Skip to content

Conversation

@pcarleton
Copy link
Member

Summary

  • When publishing from maintenance branches (e.g., patch-1), use a major.minor tag like v1.23 instead of defaulting to latest
  • This prevents old patch releases from becoming the latest version on npm
  • Users can install specific minor versions with npm install @modelcontextprotocol/[email protected]

Test plan

  • Create a release from a non-main branch and verify it publishes with the correct tag

🤖 Generated with Claude Code

When publishing from maintenance branches (e.g., patch-1), use a
major.minor tag like "v1.23" instead of defaulting to "latest".
This prevents old patches from becoming the latest version on npm.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@pcarleton pcarleton requested a review from a team as a code owner December 4, 2025 18:19
@pcarleton
Copy link
Member Author

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 4, 2025

Open in StackBlitz

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/sdk@1236

commit: c7a7ad7

pcarleton and others added 2 commits December 4, 2025 18:34
npm rejects tags like "v1.23" because they look like semver ranges.
Use "release-1.23" format instead.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copy link
Contributor

@felixweinberger felixweinberger left a comment

Choose a reason for hiding this comment

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

Thanks!

@felixweinberger felixweinberger enabled auto-merge (squash) December 4, 2025 19:24
@felixweinberger felixweinberger merged commit d5dba54 into main Dec 4, 2025
10 checks passed
@felixweinberger felixweinberger deleted the fix/publish-npm-tag-for-patches branch December 4, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants