@commonlyai/mcp 0.3.28 → 0.3.30

package/package.json

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

package/src/client.js

@@ -132,7 +132,9 @@ export const loadConfig = (env = process.env, opts = {}) => {
  * `query` is a flat object — all values stringified and URL-encoded.
  * `_fetchImpl` is a test seam (same convention as `cli/src/lib/adapters/*.js`).
  */
-export const request = async (config, { method, path, query, body, _fetchImpl = fetch } = {}) => {
+export const request = async (config, {
+  method, path, query, body, signal, _fetchImpl = fetch,
+} = {}) => {
   let url = `${config.baseUrl}${path}`;
   if (query) {
     const params = new URLSearchParams();
@@ -154,6 +156,7 @@ export const request = async (config, { method, path, query, body, _fetchImpl =
     method,
     headers,
     body: body !== undefined ? JSON.stringify(body) : undefined,
+    signal,
   });
 
   // Read body once. Prefer JSON; fall back to text for non-JSON error pages

package/src/index.js

@@ -25,11 +25,12 @@ import {
   CallToolRequestSchema,
 } from '@modelcontextprotocol/sdk/types.js';
 
-import { loadConfig } from './client.js';
+import { loadConfig, request } from './client.js';
 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,8 +65,34 @@ const main = async () => {
     return tool.call(req.params.arguments || {});
   });
 
+  // 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. 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) => {