Skip to content

Commit 9e4eba3

Browse files
Simplify Icons property documentation as suggested in code review
Co-authored-by: MackinnonBuck <[email protected]>
1 parent 3ac0678 commit 9e4eba3

File tree

4 files changed

+0
-32
lines changed

4 files changed

+0
-32
lines changed

src/ModelContextProtocol.Core/Protocol/Implementation.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,7 @@ public sealed class Implementation : IBaseMetadata
4141
/// Gets or sets an optional list of icons for this implementation.
4242
/// </summary>
4343
/// <remarks>
44-
/// <para>
4544
/// This can be used by clients to display the implementation in a user interface.
46-
/// Multiple icons can be provided to support different display contexts and resolutions.
47-
/// Clients should select the most appropriate icon based on their UI requirements.
48-
/// </para>
49-
/// <para>
50-
/// Each icon should specify a source URI that points to the icon file or data representation,
51-
/// and may also include MIME type and size information to help clients choose the best icon.
52-
/// </para>
5345
/// </remarks>
5446
[JsonPropertyName("icons")]
5547
public IList<Icon>? Icons { get; set; }

src/ModelContextProtocol.Core/Protocol/Prompt.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -56,15 +56,7 @@ public sealed class Prompt : IBaseMetadata
5656
/// Gets or sets an optional list of icons for this prompt.
5757
/// </summary>
5858
/// <remarks>
59-
/// <para>
6059
/// This can be used by clients to display the prompt's icon in a user interface.
61-
/// Multiple icons can be provided to support different display contexts and resolutions.
62-
/// Clients should select the most appropriate icon based on their UI requirements.
63-
/// </para>
64-
/// <para>
65-
/// Each icon should specify a source URI that points to the icon file or data representation,
66-
/// and may also include MIME type and size information to help clients choose the best icon.
67-
/// </para>
6860
/// </remarks>
6961
[JsonPropertyName("icons")]
7062
public IList<Icon>? Icons { get; set; }

src/ModelContextProtocol.Core/Protocol/Resource.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,7 @@ public sealed class Resource : IBaseMetadata
8484
/// Gets or sets an optional list of icons for this resource.
8585
/// </summary>
8686
/// <remarks>
87-
/// <para>
8887
/// This can be used by clients to display the resource's icon in a user interface.
89-
/// Multiple icons can be provided to support different display contexts and resolutions.
90-
/// Clients should select the most appropriate icon based on their UI requirements.
91-
/// </para>
92-
/// <para>
93-
/// Each icon should specify a source URI that points to the icon file or data representation,
94-
/// and may also include MIME type and size information to help clients choose the best icon.
95-
/// </para>
9688
/// </remarks>
9789
[JsonPropertyName("icons")]
9890
public IList<Icon>? Icons { get; set; }

src/ModelContextProtocol.Core/Protocol/Tool.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -111,15 +111,7 @@ public JsonElement? OutputSchema
111111
/// Gets or sets an optional list of icons for this tool.
112112
/// </summary>
113113
/// <remarks>
114-
/// <para>
115114
/// This can be used by clients to display the tool's icon in a user interface.
116-
/// Multiple icons can be provided to support different display contexts and resolutions.
117-
/// Clients should select the most appropriate icon based on their UI requirements.
118-
/// </para>
119-
/// <para>
120-
/// Each icon should specify a source URI that points to the icon file or data representation,
121-
/// and may also include MIME type and size information to help clients choose the best icon.
122-
/// </para>
123115
/// </remarks>
124116
[JsonPropertyName("icons")]
125117
public IList<Icon>? Icons { get; set; }

0 commit comments

Comments
 (0)