Repository navigation
Publish MCP Registry entry via GitHub OIDC (v1.1.0, docs link fix) - #494
Conversation
Point server.json at https://terminal49.com/docs/mcp/home and keep version 1.1.0. Co-authored-by: Akshay Dodeja <[email protected]>
Add a workflow that verifies mcp-publisher v1.8.1 and publishes server.json on main. Co-authored-by: Akshay Dodeja <[email protected]>
Record packages/mcp/llms-install.md and point the MCP README at the OIDC workflow. Co-authored-by: Akshay Dodeja <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Shadow risk calibration
Calibration result: agreement. Shadow risk never blocks or enables merge. Keep it in observation mode until roughly 50 pull requests have human-reviewed labels, then calibrate before considering any policy use. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ac89af055c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 2. Add this entry under `mcpServers` in Cline's MCP settings file | ||
| (`cline_mcp_settings.json`; in the IDE: MCP Servers → Configure → | ||
| Configure MCP Servers; CLI: `~/.cline/data/settings/cline_mcp_settings.json`). |
There was a problem hiding this comment.
Document Cline's platform-specific config paths
For users or agents editing settings directly—especially on Windows—this step names the file but provides only the Unix-style CLI path and no macOS, Windows, or Linux IDE-extension locations. That leaves the direct-edit installation path incomplete outside the UI, despite the repository's MCP documentation standard requiring all three platform paths; add those locations or make this an explicitly UI-only workflow.
AGENTS.md reference: AGENTS.md:L60-L62
Useful? React with 👍 / 👎.
| @@ -0,0 +1,61 @@ | |||
| name: Publish to MCP Registry | |||
There was a problem hiding this comment.
Split the tooling and documentation changes into separate commits
This commit adds the registry publishing workflow while also changing MCP documentation and registry metadata, so tooling and documentation can no longer be reviewed or reverted independently. Split these changes as required by the repository's commit policy.
AGENTS.md reference: AGENTS.md:L83-L87
Useful? React with 👍 / 👎.
Summary
Updates the official MCP Registry metadata for
io.github.Terminal49/terminal49and publishes it from GitHub Actions on futureserver.jsonchanges.server.jsonstays at v1.1.0 (the live registry only has v1.0.0) and setswebsiteUrltohttps://terminal49.com/docs/mcp/home. That URL returns HTTP 200 with no redirect.mcp-publisherv1.8.1validateaccepts the file, and it matches the schema in$schema..github/workflows/publish-mcp.ymlpublishes the rootserver.jsonwithmcp-publisher login github-oidcandmcp-publisher publish. Permissions areid-token: writeandcontents: read. It runs onworkflow_dispatchand on a push tomainthat changesserver.json. The publisher download is pinned tov1.8.1and verified with cosign the way the registry GitHub Actions guide pins it. Action SHAs are immutable sotools/security/workflows.test.mjspasses. The job runs only whengithub.refisrefs/heads/main.packages/mcp/llms-install.mdis the Cline marketplace install guide. MCP server runtime code and tools are unchanged.After merge
Merging this PR pushes
server.jsontomain, which starts Publish to MCP Registry automatically. No registry secret is required: theio.github.Terminal49namespace is authorized by GitHub OIDC from this organization repository.main.websiteUrlhttps://terminal49.com/docs/mcp/home:https://registry.modelcontextprotocol.io/v0.1/servers/io.github.Terminal49%2Fterminal49/versionsLater metadata edits need a new
versioninserver.jsonbefore merge. The registry rejects a second publish of the same version.Tests
mcp-publisher validate server.json(v1.8.1) againstregistry.modelcontextprotocol.ioserver.json, including URI formatsnode --test tools/security/workflows.test.mjs(7 passed)This PR does not run the publish.
Need help on this PR? Tag
@codesmith-botwith what you need.

Confidence Score: 5/5
The PR appears safe to merge; no actionable defects were established.Summary
Adds automatic publication of the MCP Registry entry and a Cline installation guide.
No actionable issues were established. No workflow or runtime tests were run during this review.
Acknowledgment: dodeja explicitly noted that the registry rejects duplicate versions and documented the need to bump
version. Keeping version 1.1.0 is intentional according to the PR description.Diagram
%%{init: {'theme': 'neutral'}}%% flowchart TD A["Push changes to server.json on main"] --> C{"Run is on main?"} B["Manual workflow run"] --> C C -->|No| D["Skip publication"] C -->|Yes| E["Check out repository"] E --> F["Download pinned publisher"] F --> G["Verify signature with cosign"] G --> H["Sign in with GitHub OIDC"] H --> I["Publish root server.json"]Reviews (1) · Last reviewed commit: "docs: add the Cline install guide and re..." · Reviewed by Greptile