Document modelcontextprotocol/go-sdk module review findings

[Copilot]

The Go Fan review of github.com/modelcontextprotocol/go-sdk (v1.7.0, already latest) found no code changes were required, but the module summary it referenced (scratchpad/mods/go-sdk.md) was never committed, and the deliberate choice to keep the streamable HTTP server stateful was undocumented in code.

Changes

• scratchpad/mods/go-sdk.md (new) — module summary following the format of the existing scratchpad/mods/jsonschema-go.md: usage inventory (server, transports, client, receiving middleware, progress notifications, tool annotations), v1.7.0 / protocol revision 2026-07-28 findings, and the "no action needed this cycle" recommendation with the Stateless mode item flagged for future review.
• pkg/cli/mcp_server_http.go — comment explaining why StreamableHTTPOptions.Stateless is intentionally left off, continuing the repo's practice of documenting why an SDK default is or is not used.

}, &mcp.StreamableHTTPOptions{
	// Stateless mode is deliberately left disabled. Stateless servers opt into
	// the 2026-07-28 protocol revision but drop session affinity, which the
	// long-running logs/audit tools rely on for progress notifications over the
	// lifetime of a session. Staying stateful lets the SDK negotiate down to
	// 2025-11-25 for peers that still expect the initialize handshake.
	SessionTimeout: 2 * time.Hour,
	...

No behavior changes.

• Fixes [go-fan] Go Module Review: modelcontextprotocol/go-sdk #53626

[github-actions]

PR Triage

• Category: docs
• Risk: low
• Priority score: 23/100 (impact 10, urgency 5, quality 8)
• Recommended action: defer

Draft PR documenting an already-resolved module review with no code changes required. Low urgency; undraft when ready for review.

Generated by 🔧 PR Triage Agent · auto · 45 AIC · ⌖ 2.09 AIC · ⊞ 8.3K · ◷

[Copilot]

Pull request overview

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Adds internal documentation for the github.com/modelcontextprotocol/go-sdk dependency and clarifies in-code rationale for keeping MCP HTTP transport stateful.

Changes:

• Added a scratchpad module summary for the Go MCP SDK, including usage locations and recommendations.
• Documented why StreamableHTTPOptions.Stateless is intentionally not enabled in the HTTP server.

Show a summary per file

File	Description
scratchpad/mods/go-sdk.md	Adds a dependency/usage research summary and recommendations for the MCP Go SDK.
pkg/cli/mcp_server_http.go	Adds an explanatory comment documenting the decision to keep MCP HTTP stateful.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 2/2 changed files
• Comments generated: 3
• Review effort level: Lite

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.87.1