Skip to content

[Docs] Condense and group the 0.10.0 changelog - #728

Merged
chr-hertel merged 4 commits into
modelcontextprotocol:mainfrom
chr-hertel:prepate-0.10.0
Nov 10, 2026
Merged

chr-hertel merged 4 commits into
modelcontextprotocol:mainfrom
chr-hertel:prepate-0.10.0

Conversation

@chr-hertel

Copy link
Copy Markdown
Member

Adds a TL;DR, moves the namespace split and cross-cutting changes above the components, groups Server and Client entries by theme and merges the duplicated ones, like the 0.9.0 changelog. JSON-RPC folds into Schema, where MessageFactory lives now.

@chr-hertel chr-hertel added the documentation Improvements or additions to documentation label Nov 10, 2026
@chr-hertel
chr-hertel requested a balanced review from Copilot November 10, 2026 12:58
@chr-hertel chr-hertel added this to the 0.10.0 milestone Nov 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The changelog incorrectly attributes rejection of all 3xx responses to all three classes, although AuthenticatingHttpClient returns redirects unchanged and the error guarantee applies only to HttpTransport.

1 open issue
What changed in this PR

Reorganizes the SDK’s 0.10.0 changelog to make release highlights and migration details easier to scan, following the 0.9.0 layout. No runtime code changes.

Changes:

  • Adds a TL;DR and places namespace and cross-cutting changes before component details.
  • Groups Server and Client entries by theme and merges duplicate entries.
  • Moves JSON-RPC entries into Schema, where MessageFactory resides.
File Description
CHANGELOG.md Condenses and groups the 0.10.0 release notes.

Review effort: Balanced

Comment thread CHANGELOG.md Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The changelog incorrectly implies handlers run under an event loop by default, when this behavior is opt-in via Builder::setFiberScheduler().

No open issues 🎉

1 previously missed issue

In code that hasn't changed since last review

Low severity Clarify event-loop support requires setFiberScheduler()

CHANGELOG.md:14

Event-loop execution is opt-in (Builder::$fiberScheduler defaults to null), so this reads as a new default execution model that the SDK does not actually introduce. Please describe support for running handlers under an event loop with Builder::setFiberScheduler(), rather than stating that handlers run under one.

1 resolved since last review

Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changelog reorganization has no active review issues.

No open issues 🎉

Review effort: Balanced

@chr-hertel
chr-hertel merged commit 9e0f6c9 into modelcontextprotocol:main Nov 10, 2026
30 checks passed
@chr-hertel
chr-hertel deleted the prepate-0.10.0 branch November 10, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants