-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Backport changes to lifecycle.mdx from 2025-06-18 to draft.
#997
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backport changes to lifecycle.mdx from 2025-06-18 to draft.
#997
Conversation
In the PR releasing a new revision of the spec (modelcontextprotocol#783), commit e547813 changed a `SHOULD` to `MUST` in Lifecycle Operation section, but the change was not backported to the draft file. This PR backports the change avoiding potential regressions.
6e9a7e8 to
cfa1a1a
Compare
jonathanhefner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
dsp-ant
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
| ## Major changes | ||
|
|
||
| 1. Enhance authorization server discovery with support for [OpenID Connect Discovery 1.0](https://openid.net/specs/openid-connect-discovery-1_0.html). (PR [#797](https://github.com/modelcontextprotocol/modelcontextprotocol/pull/797)) | ||
| 2. Backport change _SHOULD_ to _MUST_ in [Lifecycle Operation](/specification/draft/basic/lifecycle#operation) from commit [e547813](https://github.com/modelcontextprotocol/modelcontextprotocol/commit/e54781342c25d97c9ca4ba3fede237a2f3f37b32). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed. Draft is a draft, it's really a change per-se.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed. Draft is a draft, it's really a change per-se.
Motivation and Context
In the PR releasing a new revision of the spec (#783), commit e547813 changed a
SHOULDtoMUSTin Lifecycle Operation section, but the change was not backported to the draft file.This PR backports the change avoiding potential regressions.
How Has This Been Tested?
Ran instructions specified in
CONTRIBUTING.mdBreaking Changes
None
Types of changes
Checklist
Additional context
None