Skip to content

Conversation

sd2k
Copy link
Collaborator

@sd2k sd2k commented Oct 14, 2025

Also update the 'docker' CI workflow to publish to the registry after
building and pushing the image.

This is split out of #291 since it's actually a different thing; this PR
adds our MCP server to the registry, using the Docker image. The other
will add an MCPB manifest which we could use for the registry instead,
and which can be used for Claude Desktop.

Following the docs for publishing and the server.json format.

Closes #287.

@sd2k sd2k requested a review from a team as a code owner October 14, 2025 15:37
@github-actions

This comment has been minimized.

Also update the 'docker' CI workflow to publish to the registry after
building and pushing the image.

This is split out of #291 since it's actually a different thing; this PR
adds our MCP server to the registry, using the Docker image. The other
will add an MCPB manifest which we _could_ use for the registry instead,
and which can be used for Claude Desktop.
@sd2k sd2k force-pushed the add-mcp-registry-server-json branch from 60cdbe2 to 5988c6e Compare October 14, 2025 15:41
@sd2k sd2k linked an issue Oct 14, 2025 that may be closed by this pull request

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this only needed for the publish workflow, or does it also need to be as static content in the repo? Reason why I ask is that we would need to remember to change versions "version": "0.6.5" in server.json now. And if this is really only a file needed for the Github workflow, we might maybe just sed/replace the version in the JSON before running mcp-publisher

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's only needed for publish (at least I hope so, it would indeed be annoying to have to change it, although version bumps are done via CI so it might not be too bad). Our release CI is already doing the sed/replacing (using jq) so yeah maybe we just make it abundantly clear by setting it to $VERSION... good spot

Copy link

@svennergr svennergr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@sd2k sd2k merged commit 43bc281 into main Oct 14, 2025
13 checks passed
@sd2k sd2k deleted the add-mcp-registry-server-json branch October 14, 2025 17:25
@sd2k sd2k mentioned this pull request Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Publish to the MCP registry

2 participants