SEP-2577: Deprecate Roots, Sampling, and Logging #2577
Open
kurtisvg wants to merge 3 commits intomodelcontextprotocol:mainfrom
Open
SEP-2577: Deprecate Roots, Sampling, and Logging #2577kurtisvg wants to merge 3 commits intomodelcontextprotocol:mainfrom
kurtisvg wants to merge 3 commits intomodelcontextprotocol:mainfrom
Conversation
Proposes deprecating three core protocol features (Roots, Sampling, and Logging) with a migration window tied to the one-year-per-version support policy. Features remain functional in all spec versions released within one year of the deprecating version.
8f1868b to
38825e8
Compare
SamMorrowDrums
approved these changes
Apr 15, 2026
Contributor
There was a problem hiding this comment.
I agree in principle with all of this, I see two challenges:
- servers can no longer use inference without either paying for it or finding a way to charge the user, doesn't change the decision but it matters.
- roots were one of the only concepts of server configuration in the protocol itself, I think it was not quite the right implementation but more broadly we need to seriously consider if there is some way of providing mutually understood configuration via MCP conceptually or not, related problems are configurations that change tool surface etc. if clients/gateways can't know a configuration impacts the server, then lots of things cannot be effectively monitored like "why are tools suddenly different", so just throwing it out there this is a step away from configuration being part of the protocol, and my open question is whether that's a directional choice or not. @tadasant @dsp-ant and I recently discussed this and @cliffhall and @chughtapan also considering it a lot, I think it also materially impacts whether primitive grouping would ever be an in-protocol concept or not.
No action needed, just wanted to state some possible consequences of merging this.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Motivation
These features were identified during a core contributor meeting as having low adoption relative to their implementation complexity: