Add server.json and the mcp-name marker for the official MCP registry - #217
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe pull request adds MCP package metadata to ChangesMCP metadata
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to No concrete metadata or launch-contract defect remains; the PR is ready for normal merge checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
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
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.
Prepares
ctrlrun mcp-operatorfor listing in the official MCP registry atregistry.modelcontextprotocol.io. It does not publish anything; it makes the next release publishable.What this adds
server.jsonat the repo root, against the2025-12-11schema, naming the serverio.github.ctrlrun/ctrlrun-mcp-operator. The package is the PyPIctrlrundistribution, the positional argument ismcp-operator, and the transport isstreamable-httpat the loopback defaulthttp://127.0.0.1:8901/mcp, which is whatSPEC-mcp-operator §2.1already fixes.README.mdcarryingmcp-name: io.github.ctrlrun/ctrlrun-mcp-operator.Why the README line is load-bearing
The registry verifies namespace ownership for PyPI packages by looking for an
mcp-name:token in the package's long description, which is this README. I checked the live 0.12.1 description: 26,223 characters, no marker. So a publish attempted today fails ownership verification.The ordering is therefore: merge this, cut a release so PyPI carries a description with the marker, then publish. Publishing before a marked release is on PyPI cannot succeed.
Two things to decide
server.jsonsays0.12.1in both places, matching the current release. They must name whichever release first carries the marker, so if the next tag is 0.13.0 both fields move with it.io.github.ctrlrun/*authenticates through GitHub and needs nothing else. The alternative isdev.ctrlrun/mcp-operator, which authenticates by a DNS TXT record on a domain already on Cloudflare and reads as the project rather than as a GitHub account. Changing it later means republishing under a new name, so it is cheaper to decide now.Checks
Full suite: 4136 passed, 374 skipped.
tests/test_readme_assets.pypasses unchanged, including the header copy pins, since the marker sits below the last section and is a comment rather than prose.Summary by CodeRabbit