feat(mcp): publish vouch to the mcp registry as io.github.vouchdev/vouch - #410
Conversation
adds the packaging needed to list vouch in the official mcp registry (registry.modelcontextprotocol.io, mirrored at github.com/mcp/plind-junior/vouch): - server.json at the repo root — pypi `vouch-kb` artifact, run over stdio via `uvx vouch-kb serve`. - `vouch-kb` console-script alias so the registry can launch the package by its pypi identifier; the existing `vouch` script name wouldn't match. - an `mcp-name: io.github.vouchdev/vouch` marker in the readme, which the registry matches against server.json to verify package ownership. bumps to 1.2.2 across the four version sites so the marker ships in a published pypi release — pypi descriptions are immutable per version, so the ownership check needs a fresh release to read.
📝 WalkthroughWalkthroughThis PR releases version 1.2.2 across package manifests, adds an MCP registry server manifest (server.json), introduces a vouch-kb console-script alias, adds an MCP package-name marker in README.md, and documents the release in CHANGELOG.md. Changesv1.2.2 Release and MCP Registry Packaging
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
server.json (1)
11-35: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueConsider adding
registryBaseUrlfor the pypi package.Other example manifests explicitly set
registryBaseUrl(e.g."https://pypi.org") alongsideregistryType: "pypi". It's optional but adds clarity/robustness against future registry ambiguity.Optional addition
{ "registryType": "pypi", + "registryBaseUrl": "https://pypi.org", "identifier": "vouch-kb", "version": "1.2.2",🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@server.json` around lines 11 - 35, The pypi package entry in server.json is missing an explicit registryBaseUrl, which other manifests use for clarity. Update the package object for vouch-kb to include registryBaseUrl with the standard PyPI URL alongside registryType and identifier, keeping the existing runtimeHint, packageArguments, and environmentVariables unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@server.json`:
- Around line 11-35: The pypi package entry in server.json is missing an
explicit registryBaseUrl, which other manifests use for clarity. Update the
package object for vouch-kb to include registryBaseUrl with the standard PyPI
URL alongside registryType and identifier, keeping the existing runtimeHint,
packageArguments, and environmentVariables unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 4a1396c0-2091-4034-ac11-ee1661d12578
📒 Files selected for processing (7)
CHANGELOG.mdREADME.mdopenclaw.plugin.jsonpackage.jsonpyproject.tomlserver.jsonsrc/vouch/__init__.py
adds the packaging needed to list vouch in the official mcp registry (registry.modelcontextprotocol.io, mirrored at
github.com/mcp/plind-junior/vouch):
vouch-kbartifact, run over stdio viauvx vouch-kb serve.vouch-kbconsole-script alias so the registry can launch the package by its pypi identifier; the existingvouchscript name wouldn't match.mcp-name: io.github.vouchdev/vouchmarker in the readme, which the registry matches against server.json to verify package ownership.bumps to 1.2.2 across the four version sites so the marker ships in a published pypi release — pypi descriptions are immutable per version, so the ownership check needs a fresh release to read.
What changed
Why
What might break
VEP
Tests
make checkpasses locally (lint + mypy + pytest)CHANGELOG.mdupdated under## [Unreleased]Summary by CodeRabbit
New Features
vouch-kblaunch alias for starting the service.Chores