modelcontextprotocol/inspector
Watch
1
0
Fork
You've already forked inspector
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Simplify server auto-selection logic

Browse source 
Only auto-select when there's exactly one server in the config file.
Removes special handling for "default-server" naming convention.

Addresses review feedback from @ochafik on PR #661
This commit is contained in:
Felix Weinberger 2025-08-07 20:45:23 +01:00
commit 1263ffdf58
No known key found for this signature in database
1 changed files with 2 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
cli/src/cli.ts
View file

@ -254,16 +254,13 @@ function parseArgs(): Args {
const parsedConfig = JSON.parse(configContent);
const servers = Object.keys(parsedConfig.mcpServers || {});
if (servers.includes("default-server")) {
// Use default-server if it exists
options.server = "default-server";
} else if (servers.length === 1) {
if (servers.length === 1) {
// Use the only server if there's just one
options.server = servers[0];
} else if (servers.length === 0) {
throw new Error("No servers found in config file");
} else {
// Multiple servers, no default-server
// Multiple servers, require explicit selection
throw new Error(
`Multiple servers found in config file. Please specify one with --server.\nAvailable servers: ${servers.join(", ")}`,
);