From 5cde8e2d9264167d60e391958cbf9f3aab5457ca Mon Sep 17 00:00:00 2001 From: Nishchay Mahor Date: Thu, 28 May 2026 12:43:46 -0700 Subject: [PATCH] fix: declare commander as a direct dependency (#873) The top-level package ships cli/build/cli.js as its bin, and that file imports commander at runtime. commander was only listed as a dep of the sibling inspector-cli workspace, so it was resolving by npm hoisting. pnpm and other strict installers don't hoist, so the bin failed with 'Cannot find package commander'. Adding commander to the top-level dependencies makes the bin work under any installer. --- package.json | 1 + 1 file changed, 1 insertion(+) diff --git a/package.json b/package.json index 601ac3e2d..6d64dc712 100644 --- a/package.json +++ b/package.json @@ -56,6 +56,7 @@ "@modelcontextprotocol/inspector-client": "^0.21.2", "@modelcontextprotocol/inspector-server": "^0.21.2", "@modelcontextprotocol/sdk": "^1.25.2", + "commander": "^13.1.0", "concurrently": "^9.2.0", "node-fetch": "^3.3.2", "open": "^10.2.0",