Skip to content

Commit cea1f2a

Browse files
Copilotstephentoub
andcommitted
Fix code review feedback - remove paramref from property setter exception docs
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
1 parent 28aaeab commit cea1f2a

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

src/ModelContextProtocol.Core/Client/HttpClientTransportOptions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ public sealed class HttpClientTransportOptions
1010
/// <summary>
1111
/// Gets or sets the base address of the server for SSE connections.
1212
/// </summary>
13-
/// <exception cref="ArgumentNullException"><paramref name="value"/> is <see langword="null"/>.</exception>
14-
/// <exception cref="ArgumentException"><paramref name="value"/> is not an absolute URI, or does not use the HTTP or HTTPS scheme.</exception>
13+
/// <exception cref="ArgumentNullException">The value is <see langword="null"/>.</exception>
14+
/// <exception cref="ArgumentException">The value is not an absolute URI, or does not use the HTTP or HTTPS scheme.</exception>
1515
public required Uri Endpoint
1616
{
1717
get;

src/ModelContextProtocol.Core/Client/StdioClientTransportOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ public sealed class StdioClientTransportOptions
88
/// <summary>
99
/// Gets or sets the command to execute to start the server process.
1010
/// </summary>
11-
/// <exception cref="ArgumentException"><paramref name="value"/> is <see langword="null"/>, empty, or composed entirely of whitespace.</exception>
11+
/// <exception cref="ArgumentException">The value is <see langword="null"/>, empty, or composed entirely of whitespace.</exception>
1212
public required string Command
1313
{
1414
get;

src/ModelContextProtocol.Core/Protocol/ElicitRequestParams.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ public sealed class ElicitRequestParams : RequestParams
2020
/// <item><description><b>url</b>: Client navigates user to a URL for out-of-band interaction. Sensitive data is not exposed to the client.</description></item>
2121
/// </list>
2222
/// </remarks>
23-
/// <exception cref="ArgumentException"><paramref name="value"/> is not "form" or "url".</exception>
23+
/// <exception cref="ArgumentException">The value is not "form" or "url".</exception>
2424
[JsonPropertyName("mode")]
2525
[field: MaybeNull]
2626
public string Mode

src/ModelContextProtocol.Core/Protocol/Tool.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public sealed class Tool : IBaseMetadata
3939
/// <summary>
4040
/// Gets or sets a JSON Schema object defining the expected parameters for the tool.
4141
/// </summary>
42-
/// <exception cref="ArgumentException"><paramref name="value"/> is not a valid MCP tool JSON schema.</exception>
42+
/// <exception cref="ArgumentException">The value is not a valid MCP tool JSON schema.</exception>
4343
/// <remarks>
4444
/// <para>
4545
/// The schema must be a valid JSON Schema object with the "type" property set to "object".
@@ -74,7 +74,7 @@ public JsonElement InputSchema
7474
/// <summary>
7575
/// Gets or sets a JSON Schema object defining the expected structured outputs for the tool.
7676
/// </summary>
77-
/// <exception cref="ArgumentException"><paramref name="value"/> is not a valid MCP tool JSON schema.</exception>
77+
/// <exception cref="ArgumentException">The value is not a valid MCP tool JSON schema.</exception>
7878
/// <remarks>
7979
/// <para>
8080
/// The schema must be a valid JSON Schema object with the "type" property set to "object".

0 commit comments

Comments
 (0)