Skip to content

fix: MCP bin alias for npx -y @vibegrid/mcp#85

Merged
jcanizalez merged 1 commit intomainfrom
fix/mcp-npx-bin-name
Mar 24, 2026
Merged

fix: MCP bin alias for npx -y @vibegrid/mcp#85
jcanizalez merged 1 commit intomainfrom
fix/mcp-npx-bin-name

Conversation

@jcanizalez
Copy link
Copy Markdown
Owner

Summary

  • Add mcp bin alias to @vibegrid/mcp package so npx -y @vibegrid/mcp resolves correctly
  • npx looks for a bin matching the unscoped package name (mcp), but only vibegrid-mcp existed
  • Tested locally: both bin names create correct symlinks and the server initializes successfully

Test plan

  • yarn workspace @vibegrid/mcp build succeeds
  • Local tarball install creates both mcp and vibegrid-mcp bin links
  • npx mcp starts the server and responds to JSON-RPC initialize

npx resolves scoped packages by the unscoped name. The bin was only
`vibegrid-mcp`, so `npx -y @vibegrid/mcp` failed with "command not
found". Adding `mcp` as an additional bin entry fixes this.
Copilot AI review requested due to automatic review settings March 24, 2026 22:24
@jcanizalez jcanizalez merged commit 63b7a94 into main Mar 24, 2026
2 of 3 checks passed
@jcanizalez jcanizalez deleted the fix/mcp-npx-bin-name branch March 24, 2026 22:25
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an unscoped mcp executable alias to the @vibegrid/mcp package so npx -y @vibegrid/mcp can resolve the expected bin name (mcp) while retaining the existing vibegrid-mcp bin for backwards compatibility.

Changes:

  • Add mcp entry to @vibegrid/mcp bin map (pointing to dist/index.js)
  • Keep existing vibegrid-mcp bin unchanged

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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