Skip to content

Commit 304d726

Browse files
Copilothalter73
andcommitted
Apply doc comment suggestions from review
Co-authored-by: halter73 <54385+halter73@users.noreply.github.com>
1 parent 8498b79 commit 304d726

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

src/Common/Experimentals.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ internal static class Experimentals
7373

7474
/// <summary>
7575
/// Diagnostic ID for experimental SDK APIs unrelated to the MCP specification,
76-
/// such as subclassing <c>McpClient</c>/<c>McpServer</c> and <c>RunSessionHandler</c>.
76+
/// such as subclassing <c>McpClient</c>/<c>McpServer</c> or referencing <c>RunSessionHandler</c>.
7777
/// </summary>
7878
/// <remarks>
7979
/// This diagnostic ID covers experimental SDK-level extensibility APIs. All constants

src/ModelContextProtocol.AspNetCore/HttpServerTransportOptions.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,11 @@ public class HttpServerTransportOptions
2525
/// <remarks>
2626
/// This callback is useful for running logic before a session starts and after it completes.
2727
/// <para>
28+
/// The <see cref="HttpContext"/> parameter comes from the request that initiated the session (e.g., the
29+
/// initialize request) and may not be usable after <see cref="McpServer.RunAsync"/> starts, since that
30+
/// request will have already completed.
31+
/// </para>
32+
/// <para>
2833
/// Consider using <see cref="ConfigureSessionOptions"/> instead, which provides access to the
2934
/// <see cref="HttpContext"/> of the initializing request with fewer known issues.
3035
/// </para>

0 commit comments

Comments
 (0)