Repository navigation
npx-migration: flip plugin launcher to npx; delete bun launcher + workspace-freeze staging (comms-iw8w.5) - #25
Merged
Merged
Conversation
…p; delete bun launcher + workspace-freeze staging (comms-iw8w.5)
The Claude Code plugin now launches its MCP server with
`npx -y @codeforbreakfast/commy-mcp` and cwd ${CLAUDE_PLUGIN_ROOT},
replacing the launch.sh bun bootstrap. npx resolves the published,
self-contained node bundle from the registry for external consumers; a
local node_modules/@codeforbreakfast/commy-mcp install (staged by
publish-marketplace) shadows the registry so a host running its own copy
resolves it with zero registry hits. The consumer prerequisite drops from
bun to node.
- .mcp.json: command=npx, args=[-y, @codeforbreakfast/commy-mcp] (bare, no
version pin so the local override wins), cwd=${CLAUDE_PLUGIN_ROOT}.
- Delete launch.sh + launch.test.ts; add launcher.test.ts pinning the npx
shape, the bare-name/cwd contract, and the env block.
- Rewrite publish-marketplace.ts: stage the node bundle as a local override
under the plugin's node_modules instead of freezing the whole bun
workspace + bun install; verifyBoots runs the staged bundle under node.
- Update the dependencies.test.ts framing (bundle inlines deps; the test
still guards that every runtime import is a declared dependency).
- Docs (README, clients README, self-hosting): node-only consumer contract.
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.
Flips the Claude Code plugin's MCP launcher from the
launch.shbun bootstrap tonpx -y @codeforbreakfast/commy-mcpwithcwd: ${CLAUDE_PLUGIN_ROOT}, dropping the consumer prerequisite from bun to node.What changes
.mcp.json:command: npx,args: [-y, @codeforbreakfast/commy-mcp](bare name — no version pin on the command line, so a local install can shadow the registry),cwd: ${CLAUDE_PLUGIN_ROOT}. Env block unchanged.launch.sh+launch.test.ts; addlauncher.test.tspinning the npx command shape, the bare-name/cwd contract, and the realm-credential env keys.publish-marketplace.ts: stage the self-contained node bundle as a localnode_modules/@codeforbreakfast/commy-mcpoverride under the plugin dir, instead of freezing the whole bun workspace and runningbun install.verifyBootsnow runs the staged bundle undernodefor the MCP initialize smoke test. No source tree, no workspace, no install step.dependencies.test.ts: comment reframed for the bundle model (the test still guards that every runtime import is a declared dependency so the bundle build can resolve it).README.md,clients/claude-code/README.md,docs/self-hosting.md): node-only consumer contract —npxpulls the published bundle; bun/nix are development-only.How resolution works
npxresolves a package from its cwd'snode_modulesbefore the registry. Withcwd: ${CLAUDE_PLUGIN_ROOT}:@codeforbreakfast/commy-mcpfrom npm;Both paths, the
-ybehaviour, and thecwdfield support were verified empirically against the published package.bun run checkgreen (833 pass).