-
Notifications
You must be signed in to change notification settings - Fork 76
Ci/add mcp registry publish #86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR extends the release workflow to automatically publish the Redis MCP Server to the Model Context Protocol Registry alongside PyPI releases, using GitHub OIDC authentication for secure, keyless publishing.
Key Changes:
- Added
publish-mcp-registryjob that updatesserver.jsonwith release version and publishes to MCP Registry via OIDC - Removed OCI/Docker package entry from
server.json(deferred until Docker versioning is resolved) - Enhanced success notifications to include MCP Registry links
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| server.json | Removed Docker/OCI package entry, keeping only PyPI configuration for registry publishing |
| .github/workflows/release.yml | Added new publish-mcp-registry job with version sync, MCP Publisher CLI installation, OIDC authentication, and publishing steps; updated success notifications to reflect dual publishing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
bobymicroby
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
…ase workflow Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Add MCP Registry Publishing to Release Workflow
Summary
Extends the release workflow to automatically publish the Redis MCP Server to the Model Context Protocol Registry alongside PyPI releases.
What's Changed
New Job:
publish-mcp-registryAdds automated publishing to the MCP Registry using GitHub OIDC authentication:
id-token: writepermission) for secure, keyless publishingserver.jsonwith the release versionserver.jsonis valid JSON before publishingdry_runinput to test without publishingUpdated
server.jsonEnhanced Success Notifications
How It Works
mcp-publisherCLI toolserver.jsonis valid JSONWorkflow Steps