Skip to content

Conversation

@ashwin-ant
Copy link
Member

Fixes #90.

How Has This Been Tested?

Testing locally:

$ npm link
$ npm run build

Without this change, running on a build of main gets:

$ npx @modelcontextprotocol/inspector uv --directory /Users/ashwin/code/example_mcp run example
Starting MCP inspector...
node:internal/errors:496
    ErrorCaptureStackTrace(err);
    ^

TypeError [ERR_PARSE_ARGS_INVALID_OPTION_VALUE]: Option '--args' argument is ambiguous.
Did you forget to specify the option argument for '--args'?
To specify an option argument starting with a dash use '--args=-XYZ'.
    at new NodeError (node:internal/errors:405:5)
    at checkOptionLikeValue (node:internal/util/parse_args/parse_args:87:11)
    at node:internal/util/parse_args/parse_args:361:9
    at Array.forEach (<anonymous>)
    at parseArgs (node:internal/util/parse_args/parse_args:357:3)
    at file:///Users/ashwin/code/mcp/inspector/server/build/index.js:14:20
    at ModuleJob.run (node:internal/modules/esm/module_job:195:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:337:24)
    at async loadESM (node:internal/process/esm_loader:34:7)
    at async handleMainPromise (node:internal/modules/run_main:106:12) {
  code: 'ERR_PARSE_ARGS_INVALID_OPTION_VALUE'
}

Node.js v18.20.4

🔍 MCP Inspector is up and running at http://localhost:5173 🚀

After this change:

$ npx @modelcontextprotocol/inspector uv --directory /Users/ashwin/code/example_mcp run example
Starting MCP inspector...

🔍 MCP Inspector is up and running at http://localhost:5173 🚀

No error.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

@dsp-ant dsp-ant merged commit 79344bd into main Nov 29, 2024
4 checks passed
@dsp-ant dsp-ant deleted the ashwin/spawnfix branch November 29, 2024 11:34
IgnacioC44 referenced this pull request in MCPJam/inspector Jun 21, 2025
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.

ConnectionError from generated sample

3 participants