feat: migrate MCP Registry identity to dev.zhenjia - #11
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThe MCP Registry identity changes from ChangesMCP Registry identity and publishing
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant GitHubActions
participant DNS
participant MCPPublisher
participant RegistryAPI
GitHubActions->>RegistryAPI: Inspect branded entry
GitHubActions->>DNS: Validate namespace key
GitHubActions->>MCPPublisher: Authenticate and publish branded metadata
MCPPublisher->>RegistryAPI: Create or update branded record
GitHubActions->>RegistryAPI: Verify active latest record
GitHubActions->>MCPPublisher: Authenticate for legacy migration
MCPPublisher->>RegistryAPI: Deprecate legacy versions
GitHubActions->>RegistryAPI: Verify legacy records are deprecated or deleted
✨ 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 |
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
What changed
dev.zhenjia/share-html/agentswebsite link while retaining the bot-safeworkers.devMCP endpointzhenjia.devDNS authentication via a production Environment secretWhy
The Registry namespace should represent the product domain rather than the repository owner. The old identity cannot be unpublished, so the workflow performs an ordered namespace migration: publish and verify the branded identity first, then mark the legacy identity deprecated with a migration message.
Validation
npm test— 508 passednpm run typechecknpm run buildnpx wrangler deploy --dry-runactionlint .github/workflows/publish-mcp.yml .github/workflows/ci.ymlmcp-publisher v1.7.9 validate server.jsongit diff --checkSummary by CodeRabbit
New Features
Documentation
Tests