refactor: improve MCP config logging per review feedback

- Remove configPreview from error logging to avoid cluttering output
- Add informational log when merging MCP server configurations
- Simplify error message for failed config parsing

Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>
This commit is contained in:
claude[bot]
2025-05-30 14:50:13 +00:00
committed by GitHub
parent 67d2d8682a
commit feca21446a

View File

@@ -51,6 +51,8 @@ export async function prepareMcpConfig(
throw new Error("MCP config must be a valid JSON object");
}
core.info("Merging additional MCP server configuration with built-in servers");
// Merge configurations with user config overriding built-in servers
const mergedConfig = {
...baseMcpConfig,
@@ -63,12 +65,8 @@ export async function prepareMcpConfig(
return JSON.stringify(mergedConfig, null, 2);
} catch (parseError) {
const configPreview =
additionalMcpConfig.length > 100
? `${additionalMcpConfig.substring(0, 100)}...`
: additionalMcpConfig;
core.warning(
`Failed to parse additional MCP config: ${parseError}. Invalid config: "${configPreview}". Using base config only.`,
`Failed to parse additional MCP config: ${parseError}. Using base config only.`,
);
}
}