Skip to content

Commit 1263ffd

Browse files
Simplify server auto-selection logic
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
1 parent 14b0181 commit 1263ffd

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

cli/src/cli.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -254,16 +254,13 @@ function parseArgs(): Args {
254254
const parsedConfig = JSON.parse(configContent);
255255
const servers = Object.keys(parsedConfig.mcpServers || {});
256256

257-
if (servers.includes("default-server")) {
258-
// Use default-server if it exists
259-
options.server = "default-server";
260-
} else if (servers.length === 1) {
257+
if (servers.length === 1) {
261258
// Use the only server if there's just one
262259
options.server = servers[0];
263260
} else if (servers.length === 0) {
264261
throw new Error("No servers found in config file");
265262
} else {
266-
// Multiple servers, no default-server
263+
// Multiple servers, require explicit selection
267264
throw new Error(
268265
`Multiple servers found in config file. Please specify one with --server.\nAvailable servers: ${servers.join(", ")}`,
269266
);

0 commit comments

Comments
 (0)