Skip to content

Document https:// direct-fetch provision in ReadResourceRequestParams#1296

Merged
stephentoub merged 2 commits intomainfrom
copilot/update-read-resource-params-docs
Feb 17, 2026
Merged

Document https:// direct-fetch provision in ReadResourceRequestParams#1296
stephentoub merged 2 commits intomainfrom
copilot/update-read-resource-params-docs

Conversation

Copy link
Contributor

Copilot AI commented Feb 17, 2026

Aligns with spec commit df1152e which clarified that clients may fetch https:// resources directly from the web instead of using resources/read.

Changes

  • Updated ReadResourceRequestParams XML docs to document the https:// direct-fetch provision
  • Restructured <remarks> into separate <para> elements for clarity

The documentation now explicitly states that servers should only use https:// scheme when clients can fetch resources independently, making this spec behavior more discoverable for SDK users.

Original prompt

Context

Spec commit df1152e297eca7afb2ef4f5aecd8f6b52177ac6a (merged via PR #802) added a clarification to the "Reading Resources" section of the spec: if the resource URI uses the https:// scheme, clients may fetch the resource directly from the web instead of going through resources/read. This was already documented in the "Common URI Schemes" section but the PR makes it more discoverable.

What to change

Update the XML doc comments on ReadResourceRequestParams in src/ModelContextProtocol.Core/Protocol/ReadResourceRequestParams.cs to mention the https:// direct-fetch provision.

Current

/// <summary>
/// Represents the parameters used with a <see cref="RequestMethods.ResourcesRead"/> request from a client to get a resource provided by a server.
/// </summary>
/// <remarks>
/// The server will respond with a <see cref="ReadResourceResult"/> containing the resulting resource data.
/// See the <see href="proxy.php?url=https://github.com/modelcontextprotocol/specification/blob/main/schema/">schema</see> for details.
/// </remarks>
public sealed class ReadResourceRequestParams : RequestParams

Proposed

/// <summary>
/// Represents the parameters used with a <see cref="RequestMethods.ResourcesRead"/> request from a client to get a resource provided by a server.
/// </summary>
/// <remarks>
/// <para>
/// The server will respond with a <see cref="ReadResourceResult"/> containing the resulting resource data.
/// </para>
/// <para>
/// Alternatively, if the resource URI uses the <c>https://</c> scheme, clients may fetch the resource
/// directly from the web instead of using <see cref="RequestMethods.ResourcesRead"/>.
/// Servers should only use the <c>https://</c> scheme when the client is able to fetch and load the
/// resource directly on its own.
/// </para>
/// <para>
/// See the <see href="proxy.php?url=https://github.com/modelcontextprotocol/specification/blob/main/schema/">schema</see> for details.
/// </para>
/// </remarks>
public sealed class ReadResourceRequestParams : RequestParams

No functional or behavioral changes are needed — this is purely a documentation update.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update XML doc comments for ReadResourceRequestParams Document https:// direct-fetch provision in ReadResourceRequestParams Feb 17, 2026
Copilot AI requested a review from stephentoub February 17, 2026 09:19
@stephentoub stephentoub marked this pull request as ready for review February 17, 2026 09:22
Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. 👍

@stephentoub stephentoub merged commit 392e47e into main Feb 17, 2026
9 of 10 checks passed
@stephentoub stephentoub deleted the copilot/update-read-resource-params-docs branch February 17, 2026 14:30
dagirard pushed a commit to dagirard/csharp-sdk that referenced this pull request Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants