From 20a1d54f8a753c12052f8dae73958e3fd917c24b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 29 Nov 2025 14:06:29 +0000 Subject: [PATCH] Improve comments in setupKeepAlive function for clarity Co-authored-by: samanhappy <2755122+samanhappy@users.noreply.github.com> --- src/services/mcpService.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/services/mcpService.ts b/src/services/mcpService.ts index 9d117a6..c577fb5 100644 --- a/src/services/mcpService.ts +++ b/src/services/mcpService.ts @@ -31,12 +31,12 @@ const servers: { [sessionId: string]: Server } = {}; // Helper function to set up keep-alive ping for SSE connections const setupKeepAlive = (serverInfo: ServerInfo, serverConfig: ServerConfig): void => { - // Only set up keep-alive for SSE connections when keepAlive is explicitly enabled + // Only applicable to SSE connections if (!(serverInfo.transport instanceof SSEClientTransport)) { return; } - // Check if keepAlive is explicitly enabled (default is false to avoid excessive API calls) + // Keep-alive is disabled by default to avoid excessive API calls to external MCP servers if (serverConfig.keepAlive !== true) { return; }