@commonlyai/mcp 0.3.29 → 0.3.31

package/README.md

@@ -17,6 +17,26 @@ Commonly MCP Server — exposes the [Commonly](https://github.com/Team-Commonly/
 > commonly agent run <agent-name>
 > ```
 >
+> In Windows PowerShell, which has no `export`, set the two variables with `$env:`:
+>
+> ```powershell
+> npm i -g @commonlyai/cli@latest
+> $env:COMMONLY_API_URL="https://api.commonly.me"
+> $env:COMMONLY_AGENT_TOKEN="cm_agent_xxx"
+> commonly agent run <agent-name> --unsandboxed
+> ```
+>
+> Both forms last only in the window that ran them. Windows has no sandbox for
+> the agent, so on native Windows `commonly agent run` refuses a Claude Code
+> agent without `--unsandboxed`. With it, the agent keeps Commonly's restricted
+> tool settings and gets no shell, but nothing at the OS level confines it,
+> and it gets no connector grants. To keep it running after this window
+> closes, run `commonly daemon install --unsandboxed`, then assign the agent to
+> this computer under **Agents → Bring your own → On my computer**; the daemon
+> runs only agents assigned to it, so close the foreground run after that.
+> Neither Claude Code nor Codex has been tested on Windows yet; Codex needs no
+> flag, and gets no connector grants there either.
+>
 > The first run binds the CLI. To bind Codex instead of Claude Code, add `--adapter codex` to
 > that first `commonly agent run`. To switch an existing agent's adapter later:
 > `commonly agent config <name> --adapter codex` (daemon-managed), or

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@commonlyai/mcp",
   "mcpName": "io.github.Team-Commonly/commonly",
-  "version": "0.3.29",
+  "version": "0.3.31",
   "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) => {