feat: MCP server packaging and CI/CD setup - #2
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces MCP bundle packaging for the PCM MCP server and wires CI/CD to build (and on tags, publish) the packaged artifacts, alongside updated end-user installation docs.
Changes:
- Add MCP bundle metadata via
manifest.jsonand document multiple client installation options inREADME.md. - Add a
packscript (usingmcpb pack) and include the MCPB tooling dependency. - Update CI to build the
.mcpbbundle and add a tag-triggered publish workflow.
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Expands installation instructions (bundle + various clients) and documents npm run pack. |
| package.json | Adds pack script and MCPB tooling dependency. |
| package-lock.json | Locks new dependency graph for MCPB tooling and related packages. |
| manifest.json | Introduces MCP bundle manifest (metadata + server entrypoint + icon). |
| .github/workflows/publish.yml | Adds tag-triggered publishing/release workflow. |
| .github/workflows/ci.yml | Adds bundle build step to CI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
package.json:55
@anthropic-ai/mcpbis only used by thepackscript (CLI) and isn’t imported by the runtime server. Keeping it underdependenciesincreases install size for end-users; it should be adevDependency(and regeneratepackage-lock.json).
"dependencies": {
"@anthropic-ai/mcpb": "^2.1.2",
"@modelcontextprotocol/sdk": "^1.29.0",
"cdb-converter": "^0.1.2",
"sql.js": "^1.14.1",
"zod": "^3.25.76"
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
manifest.jsonwith full MCP server metadata (name, description, icon)mcpb packpackage.jsonwithpackscript and required packaging dependenciesREADME.mdwith clearer documentation