Skip to content

chore: move MCP server to @pcmstack/mcp and stop hardcoding its version - #13

Merged
mpicciolli merged 2 commits into
mainfrom
chore/pcmstack-mcp-rename
Aug 31, 2026
Merged

chore: move MCP server to @pcmstack/mcp and stop hardcoding its version#13
mpicciolli merged 2 commits into
mainfrom
chore/pcmstack-mcp-rename

Conversation

@mpicciolli

@mpicciolli mpicciolli commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

What

The bundled MCP server is renamed from pcm-mcp to the scoped @pcmstack/mcp (pinned to 0.5.0), and the .mcp.json server key becomes pcmstack.

The rest of the diff makes the docs survive the next rename or version bump:

  • AGENTS.md no longer quotes a version number at all. The Security section points at .mcp.json as the single source of truth for the pin, with an explicit "never quote a version number here" so a bump stays a one-file change.
  • SKILL.md and references/database-constraints.md identify Route A by the pcm_* tool prefix an agent actually sees in its tool list, rather than by the server key or npm package name. That prefix is stable; the package name has now changed twice.
  • README.md and the Codex longDescription pick up the new package name.

No skill behaviour changes — the routing rule, the read-only invariant and the tool flow are untouched.

The bundled MCP server is renamed from `pcm-mcp` to the scoped `@pcmstack/mcp`
(pinned to 0.5.0) and its `.mcp.json` key becomes `pcmstack`.

Docs are updated so the next rename or bump touches one file:

- AGENTS.md no longer quotes a version number; it points at .mcp.json as the
  single source of truth for the pin.
- SKILL.md and database-constraints.md identify the route by the `pcm_*` tool
  prefix an agent actually sees, instead of naming the server or npm package.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 31, 2026 13:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the db-editor plugin’s bundled MCP server configuration and documentation to reflect the rename to the scoped @pcmstack/mcp, while reducing future maintenance by avoiding hardcoded version mentions in prose.

Changes:

  • Renames the bundled MCP server package reference to @pcmstack/mcp and updates .mcp.json to use a new server key (pcmstack) with a pinned version.
  • Updates repository docs to reference the new package name and to avoid hardcoding MCP server version numbers in prose.
  • Tweaks skill/references wording to describe Route A via the stable pcm_* tool prefix rather than the MCP server key/package name.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
README.md Updates docs/diagram and related-projects links to @pcmstack/mcp.
plugins/db-editor/skills/pcm-database/SKILL.md Removes hard dependency on package/server-key naming; keeps Route A described generically.
plugins/db-editor/skills/pcm-database/references/database-constraints.md Rewords guidance to refer to pcm_* tool parameters rather than the old package name.
plugins/db-editor/.mcp.json Switches MCP server key to pcmstack and pins @pcmstack/mcp@0.5.0.
plugins/db-editor/.codex-plugin/plugin.json Updates Codex longDescription to reference @pcmstack/mcp.
AGENTS.md Updates repo guidance to point to .mcp.json as the single source of truth and references @pcmstack/mcp.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread AGENTS.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mpicciolli
mpicciolli merged commit e542c4a into main Aug 31, 2026
1 check passed
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.

2 participants