ci: auto-sync server.json versions before MCP Registry publish#330
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughTwo CI workflow files ( Manifest Version Sync in CI Workflows
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Problem
The MCP Registry publishes whatever version is committed in
server.jsonandserver-vaara-server.json. Nothing bumps those files on release, so a forgotten manual manifest bump publishes a stale version and the post-publish assertion fails. This is the cause of the failedReleaseruns for v1.17.0 / v1.18.0 and the 3 straightRepublishfailures. PyPI and npm were fine, so the failures were invisible externally while the MCP manifest fell behind.Fix
Add a step in both
release.ymlandrepublish.yml, right before publishing, that rewrites.versionand.packages[].versionin both manifests to the released tag (release) or the requested input (republish). The manifests are now always in sync at publish time regardless of what is committed.jqis pre-installed onubuntu-latestrunners.No effect on the current registry state (already at 1.19.0, isLatest, active for both listings). This prevents the next release from recurring the failure.
Summary by CodeRabbit