From 3aedd3cd9ae80b34e32fcfdb1532c6c571fa6219 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Fri, 12 Dec 2025 16:33:51 +0200 Subject: [PATCH] Fix Icon.Theme documentation to match MCP specification (#1090) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> Co-authored-by: Stephen Toub --- src/ModelContextProtocol.Core/Protocol/Icon.cs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/ModelContextProtocol.Core/Protocol/Icon.cs b/src/ModelContextProtocol.Core/Protocol/Icon.cs index 0ae9add..e187f19 100644 --- a/src/ModelContextProtocol.Core/Protocol/Icon.cs +++ b/src/ModelContextProtocol.Core/Protocol/Icon.cs @@ -79,8 +79,9 @@ public sealed class Icon /// Gets or sets the optional theme for this icon. /// /// - /// Can be "light", "dark", or a custom theme identifier. - /// The value is used to specify which UI theme the icon is designed for. + /// may be "light" or "dark". "light" indicates the icon is designed to be used with a light + /// background, and "dark" indicates the icon is designed to be used with a dark background. + /// If not provided, clients should assume the icon can be used with any theme. /// [JsonPropertyName("theme")] public string? Theme { get; set; }