Bump to version 1.10.0#305
Conversation
cliffhall
requested review from
a team,
jerome3o-anthropic,
jspahrsummers and
tadasant
tadasant
left a comment
tadasant
left a comment
There was a problem hiding this comment.
Also I think we'd want a package-lock.json bump as well right?
May be best to wait for @modelcontextprotocol/typescript-sdk to chime in here, don't want to mess up a release
package.json
Outdated
| { | ||
| "name": "@modelcontextprotocol/sdk", | ||
| "version": "1.9.0", | ||
| "version": "1.9.1", |
There was a problem hiding this comment.
It seems like most recent bumps have been minor version bumps rather than patch bumps, and adding Streamable support feels more like a backwards-compatible new-feature (minor version) addition than a bugfix (patch). Should this be 1.10.0 then?
jspahrsummers
left a comment
jspahrsummers
left a comment
There was a problem hiding this comment.
LGTM! I agree on using a minor version here. Bumping package-lock.json isn't critical, since it doesn't affect distribution (only local dev, where it will update next time someone runs npm install).
|
Sorry, but let's actually wait on cutting a release until the new transport work actually stabilizes. Will defer to @ihrpr on calling that moment. |
3 participants
In package.json
Motivation and Context
Streamable HTTP Transport is available and we need that for a PR in the inspector that supports it.
How Has This Been Tested?
Local build, test, lint.
Breaking Changes
Nope.
Types of changes
Checklist
Additional context