Skip to content

fix: declare root bundled runtime dependencies#1542

Open
mturac wants to merge 1 commit into
modelcontextprotocol:mainfrom
mturac:izon/fix-root-cli-commander-dep
Open

fix: declare root bundled runtime dependencies#1542
mturac wants to merge 1 commit into
modelcontextprotocol:mainfrom
mturac:izon/fix-root-cli-commander-dep

Conversation

@mturac

@mturac mturac commented Jun 28, 2026

Copy link
Copy Markdown

Summary

  • Add the runtime packages imported by the root package's bundled CLI/server/client files to the root package dependencies.
  • Keep the published mcp-inspector bin usable with strict package managers, where the root package cannot rely on dependencies declared only by sibling workspace packages.

Fixes #1513.
Addresses #873.

Verification

  • npm install --package-lock-only (ran the repository prepare build successfully)
  • npm run build
  • npm run prettier-check
  • npm run test-cli (85 tests passed)
  • git diff --check
  • Packed the root package with npm pack --ignore-scripts --json, installed the tarball into a temporary pnpm project, verified the root package can resolve commander, cors, express, express-rate-limit, and serve-handler, and ran ./node_modules/.bin/mcp-inspector --help successfully

@mturac mturac force-pushed the izon/fix-root-cli-commander-dep branch from f356225 to d2e1dc8 Compare June 29, 2026 03:09
@mturac mturac changed the title fix: declare root CLI commander dependency fix: declare root bundled runtime dependencies Jun 29, 2026
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.

unable to start using pnpx: Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'commander' imported

1 participant