nearxos/kicad-mcp-server
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: connect MCP transport before warm-up to prevent client timeout

Browse Source 
Swap Phase 3 (transport connect) before Phase 4 (warm-up) in
server.ts startup. Previously the transport was connected only after
warm-up completed (55-125s), which exceeded the MCP client's 30s
connection timeout on macOS cold-start.

PR #210 fixed the Python-side by daemon-threading warm_cache, but
the Node-side still blocked transport connect on warm-up completion.

Complements #210. Replaces the server.ts portion of #206.
This commit is contained in:
Chaitanya Malhotra
2026-06-01 18:42:50 +05:30
parent 3b4ffef724
commit 3ecf5f38fd
1 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/server.ts
View File

@@ -538,14 +538,13 @@ export class KiCADMcpServer {
});
}
// ——— Phase 2: wait for Python READY, then send warm-up ———
// ——— Phase 2: wait for Python READY ———
logger.info("Waiting for Python process to be ready...");
await this.waitForReady(120_000);
logger.info("Python process is ready. Sending warm-up command...");
await this.runWarmup(120_000);
logger.info("Warm-up complete — pcbnew/wxApp initialised");
// ——— Phase 3: only now connect to MCP transport ———
logger.info("Python process is ready.");
// ——— Phase 3: connect MCP transport immediately ———
// The transport must be live before any client timeout fires,
// regardless of how long warm-up takes.
logger.info("Connecting MCP server to STDIO transport...");
try {
await this.server.connect(this.stdioTransport);
@@ -554,6 +553,14 @@ export class KiCADMcpServer {
logger.error(`Failed to connect to STDIO transport: ${error}`);
throw error;
}
// ——— Phase 4: background warm-up (does not block MCP) ———
// Warm-up can take 55-125 s (wxApp + symbol library parse), but
// the MCP transport is already live so the client timeout does not
// apply. Tools invoked during warm-up will work; the first
// search_symbols may be slower if warm-up hasn't completed yet.
logger.info("Sending warm-up command (background)...");
await this.runWarmup(120_000);
logger.info("Warm-up complete — pcbnew/wxApp initialised");
// Write a ready message to stderr (for debugging)