@commonlyai/mcp 0.3.29 → 0.3.30

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@commonlyai/mcp",
   "mcpName": "io.github.Team-Commonly/commonly",
-  "version": "0.3.29",
+  "version": "0.3.30",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",

package/src/index.js

@@ -30,6 +30,7 @@ import { buildTools } from './tools.js';
 import { PACKAGE_VERSION } from './version.js';
 
 const PACKAGE_NAME = '@commonlyai/mcp';
+const CHECK_IN_WAIT_MS = 1000;
 
 const main = async () => {
   // Fail fast on missing env. The MCP host (codex/claude) will surface this
@@ -64,22 +65,34 @@ const main = async () => {
     return tool.call(req.params.arguments || {});
   });
 
-  const transport = new StdioServerTransport();
-  await server.connect(transport);
-
   // One authenticated call at start (TASK-345). Until a tool ran, nothing
   // reached Commonly, so a working install and a broken one looked the same and
   // the agent never showed as connected. A failure goes to stderr and never
   // ends the server: stdout belongs to the MCP transport.
   // Bounded, so a server that never answers cannot keep the process alive
-  // after the host has gone.
-  request(config, {
+  // after the host has gone. Started before the transport, so it is already
+  // on its way when the client's initialize arrives.
+  const checkIn = request(config, {
     method: 'GET', path: '/api/agents/runtime/installations', signal: AbortSignal.timeout(10000),
   }).catch((e) => {
     // The status says which kind of failure this is; see docs/MCP_INTEGRATION.md.
     const status = e.status ? `HTTP ${e.status}: ` : '';
     process.stderr.write(`[commonly-mcp] could not reach Commonly at start: ${status}${e.message}\n`);
   });
+
+  const transport = new StdioServerTransport();
+  await server.connect(transport);
+
+  // `claude mcp list` initializes, prints "Connected" and closes the server,
+  // which once came before the check-in landed (Growth's walk, 11-04): the
+  // person who added it looked like one who never had. So initialize waits for
+  // the check-in, for at most a second, so a slow API never holds startup.
+  const deliver = transport.onmessage;
+  transport.onmessage = (message, extra) => {
+    if (message?.method !== 'initialize') return deliver(message, extra);
+    const cap = new Promise((resolve) => { setTimeout(resolve, CHECK_IN_WAIT_MS).unref(); });
+    return Promise.race([checkIn, cap]).then(() => deliver(message, extra));
+  };
 };
 
 main().catch((e) => {