Skip to content

Commit 3d9f15b

Browse files
cliffhallclaude
andcommitted
docs(servers): fix mutation-comment caller branch label (#1359)
The previous comment named the GET handler's "seed branch" as a caller of normalizeMcpServers, but that branch returns DEFAULT_SEED_CONFIG directly without normalizing. The actual second caller is the file-present branch (after parseStore). Correcting the label so a future reader looking for the seed branch caller doesn't waste time. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f94cd85 commit 3d9f15b

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

core/mcp/remote/node/server.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -864,8 +864,10 @@ export function createRemoteApp(
864864
if (!val || typeof val !== "object") continue;
865865
// `valObj` is the per-entry object we'll mutate in place via the
866866
// `delete` calls below. Safe because the only callers
867-
// (`readMcpConfig` and the GET handler's seed branch) pass in
868-
// freshly-parsed JSON they don't retain a reference to elsewhere.
867+
// (`readMcpConfig` and the GET handler's file-present branch — the
868+
// seed branch returns `DEFAULT_SEED_CONFIG` without normalizing)
869+
// pass in freshly-parsed JSON they don't retain a reference to
870+
// elsewhere.
869871
const valObj = val as Record<string, unknown>;
870872

871873
// Strip a legacy nested `settings` node before normalizeServerType

0 commit comments

Comments
 (0)