@monoes/monomindcli 2.9.10 → 2.9.12

package/.claude/helpers/control-start.cjs

@@ -277,19 +277,37 @@ async function main() {
   // pid matches our child. A foreign server answering on DEFAULT_PORT must
   // NOT be mistaken for ours — that lie misroutes every event emitter in
   // this project (root cause of orgs running invisibly).
+  // Absolute safety net so a wedged-but-still-alive process can't hang this
+  // hook forever — CONFIRM_ATTEMPTS below is a minimum grace period, not a
+  // hard budget (#142 follow-up).
+  const HARD_CEILING_ATTEMPTS = 600; // 500ms/attempt: 5 min
   async function confirmPort() {
     let sawForeignOnDefault = false;
-    for (let attempt = 0; attempt < CONFIRM_ATTEMPTS; attempt++) {
+    let attempt = 0;
+    for (; attempt < HARD_CEILING_ATTEMPTS; attempt++) {
       await new Promise(r => setTimeout(r, 500));
-      // 1) Authoritative: child self-reported its bound port
+      // Past the minimum grace period, a dead child means it's not coming
+      // back — stop waiting immediately instead of burning the rest of the
+      // budget. A LIVE child that simply hasn't reported yet (npm/AV
+      // contention on a freshly-written node_modules right after install,
+      // measured once at ~142s vs the normal ~5-9s — #142 follow-up) keeps
+      // getting the benefit of the doubt up to HARD_CEILING_ATTEMPTS instead
+      // of being killed on a fixed guess.
+      if (attempt >= CONFIRM_ATTEMPTS && !isPidAlive(child.pid)) break;
+      // 1) Authoritative: child self-reported its bound port. Identity comes
+      // from BOUND_REPORT's own per-invocation-unique path (nothing else on
+      // the machine knows it, since it's only handed to this child via env),
+      // NOT a pid match: under shell:true (#141's Windows EINVAL fix),
+      // child.pid is the wrapping cmd.exe's pid, not the real server's —
+      // rep.pid !== child.pid always, so a pid comparison here can never
+      // succeed on the npx-fallback path regardless of how long we wait
+      // (#143). rep.pid is the real server pid — use it for control.json.
       try {
         const rep = JSON.parse(fs.readFileSync(BOUND_REPORT, 'utf8'));
-        if (rep && rep.pid === child.pid && typeof rep.port === 'number') {
+        if (rep && typeof rep.pid === 'number' && typeof rep.port === 'number') {
           try { fs.unlinkSync(BOUND_REPORT); } catch { /* ignore */ }
-          if (rep.port !== DEFAULT_PORT) {
-            writeStatus(child.pid, rep.port);
-            if (String(process.env.MONOMIND_HOOK_QUIET || "") !== "1") process.stdout.write(`[control] server bound to port ${rep.port} (updated control.json)\n`);
-          }
+          writeStatus(rep.pid, rep.port);
+          if (String(process.env.MONOMIND_HOOK_QUIET || "") !== "1") process.stdout.write(`[control] server bound to port ${rep.port} (pid ${rep.pid})\n`);
           return;
         }
       } catch { /* not written yet or old server — fall through to probe */ }
@@ -365,7 +383,7 @@ async function main() {
     // The next session-start simply retries.
     try { process.kill(child.pid, 'SIGTERM'); } catch { /* already gone */ }
     try { fs.unlinkSync(STATUS_FILE); } catch { /* ignore */ }
-    process.stdout.write(`[control] server did not respond within ${CONFIRM_ATTEMPTS / 2} s — killed orphan, will retry next session\n`);
+    process.stdout.write(`[control] server did not respond within ${((attempt + 1) * 500 / 1000).toFixed(1)} s — killed orphan, will retry next session\n`);
   }
 
   confirmPort().catch(() => {}).finally(() => { releaseSpawnLock(); process.exit(0); });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@monoes/monomindcli",
-  "version": "2.9.10",
+  "version": "2.9.12",
   "type": "module",
   "description": "CLI engine for Monomind — an open-source MCP server that extends Claude Code with a codebase knowledge graph (tree-sitter + SQLite), persistent memory, multi-agent task coordination, and session hooks. Apache 2.0 licensed, fully local.",
   "main": "dist/src/index.js",
@@ -119,10 +119,10 @@
     "@opencode-ai/sdk": "^1.18.15",
     "monofence-ai": "^1.0.2",
     "sql.js": "^1.14.1",
-    "@monoes/routing": "1.0.4",
+    "@monoes/mcp": "1.0.3",
     "@monoes/hooks": "1.0.5",
     "@monoes/memory": "1.0.15",
-    "@monoes/mcp": "1.0.3"
+    "@monoes/routing": "1.0.4"
   },
   "publishConfig": {
     "access": "public",