Skip to content

Commit c2c0e59

Browse files
author
Griffin Evans
committed
Fix double normalize in canonicalizeStdioConfig
1 parent c2d0ef7 commit c2c0e59

1 file changed

Lines changed: 11 additions & 7 deletions

File tree

packages/plugins/mcp/src/sdk/stdio-config.ts

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -121,13 +121,17 @@ const normalizeEnv = (
121121

122122
export const canonicalizeStdioConfig = (
123123
config: Pick<McpStdioIntegrationConfig, "command" | "args" | "env" | "cwd">,
124-
): CanonicalStdioConfig => ({
125-
transport: "stdio",
126-
command: config.command.trim(),
127-
...(config.args !== undefined && config.args.length > 0 ? { args: [...config.args] } : {}),
128-
...(normalizeEnv(config.env) !== undefined ? { env: normalizeEnv(config.env) } : {}),
129-
...(config.cwd !== undefined && config.cwd.trim().length > 0 ? { cwd: config.cwd.trim() } : {}),
130-
});
124+
): CanonicalStdioConfig => {
125+
const env = normalizeEnv(config.env);
126+
const cwd = config.cwd?.trim();
127+
return {
128+
transport: "stdio",
129+
command: config.command.trim(),
130+
...(config.args !== undefined && config.args.length > 0 ? { args: [...config.args] } : {}),
131+
...(env !== undefined ? { env } : {}),
132+
...(cwd !== undefined && cwd.length > 0 ? { cwd } : {}),
133+
};
134+
};
131135

132136
export const canonicalizeStdioDraft = (input: {
133137
readonly command: string;

0 commit comments

Comments
 (0)