@axiomatic-labs/claudeflow 2.32.22 → 2.33.0

package/bin/cli.js

@@ -27,9 +27,6 @@ const SUBCOMMANDS = new Set([
   "help",
   "--help",
   "-h",
-  "doctor",
-  "panel",
-  "preview",
 ]);
 
 function inClaudeflowProject(startDir) {
@@ -58,21 +55,6 @@ async function runSubcommand(command) {
       await version();
       return;
     }
-    case "doctor": {
-      const doctor = require("../lib/doctor.js");
-      const code = await doctor(process.argv.slice(3));
-      process.exit(code || 0);
-    }
-    case "panel": {
-      const panel = require("../lib/panel.js");
-      await panel(process.argv.slice(3));
-      return;
-    }
-    case "preview": {
-      const preview = require("../lib/preview.js");
-      await preview(process.argv.slice(3));
-      return;
-    }
     case "help":
     case "--help":
     case "-h":
@@ -85,15 +67,6 @@ async function runSubcommand(command) {
       console.log(
         `    ${ui.CYAN}install${ui.RESET}    Install or update Claudeflow in the current project`,
       );
-      console.log(
-        `    ${ui.CYAN}doctor${ui.RESET}     Diagnose local issues (CDP port, stale lockfiles); add --fix to repair`,
-      );
-      console.log(
-        `    ${ui.CYAN}panel${ui.RESET}      Open the local web dashboard for hooks, CLAUDE.md, and run state`,
-      );
-      console.log(
-        `    ${ui.CYAN}preview${ui.RESET}    Live-reload server for prototypes (.claudeflow/prototypes/) — run once, leave it`,
-      );
       console.log(`    ${ui.CYAN}version${ui.RESET}    Show version info`);
       console.log(`    ${ui.CYAN}help${ui.RESET}       Show this message`);
       console.log("");

package/lib/doctor.js

@@ -146,43 +146,9 @@ function checkStaleLockfiles(cwd) {
 // Detection strategy (in order of preference):
 //   1. error-observer.json exists → read its `port` + `pid_path`.
 //   2. If pidfile exists and PID is alive → running=true.
-//   3. Else, TCP-probe the observer port with a short `net.connect` (via a
-//      one-shot `node -e` child, so the synchronous status path stays sync).
+//   3. Else, TCP probe the observer port via `lsof -i :PORT -sTCP:LISTEN -t`.
 //      The pidfile may be missing (race) but if SOMETHING is listening on
-//      the configured port, the daemon is up. This replaces a prior `lsof`
-//      probe that was slow on macOS (often >1.5s — the timeout — so a live
-//      observer read as stopped) and absent on Windows (no `lsof` → the
-//      fallback never ran). `net.connect` is a Node built-in: fast and
-//      cross-platform. Trade-off: it proves a listener EXISTS but cannot
-//      attribute the owning PID, so a probe hit returns `pid: null`.
-//
-// Synchronous TCP liveness check WITHOUT lsof. Node has no sync net.connect,
-// so we run the connect in a one-shot `node -e` child via spawnSync — keeping
-// readObserverState (and its sync callers: getMcpInfo/getObserverInfo/
-// collectStatus) synchronous. Probes both IPv4 and IPv6 loopback (the daemon
-// may bind either); exit 0 = a listener answered. Reached only when the
-// pidfile path already failed, so the ~node-startup cost is off the hot path.
-function portHasListenerSync(port) {
-  if (!Number.isFinite(port) || port <= 0) return false;
-  const script =
-    'const net=require("net");' +
-    'const port=' + Number(port) + ';' +
-    'const hosts=["127.0.0.1","::1"];' +
-    'let pending=hosts.length,done=false;' +
-    'const finish=(ok)=>{if(done)return;if(ok){done=true;process.exit(0);}if(--pending===0){done=true;process.exit(1);}};' +
-    'for(const h of hosts){const s=net.connect({host:h,port});s.setTimeout(250);' +
-    's.once("connect",()=>{s.destroy();finish(true);});' +
-    's.once("error",()=>finish(false));' +
-    's.once("timeout",()=>{s.destroy();finish(false);});}';
-  try {
-    const { spawnSync } = require('child_process');
-    const r = spawnSync(process.execPath, ['-e', script], { timeout: 1000 });
-    return r.status === 0;
-  } catch {
-    return false;
-  }
-}
-
+//      the configured port, the daemon is up.
 function readObserverState(cwd) {
   const observerJsonPath = path.join(cwd, '.claudeflow', 'tmp', 'error-observer.json');
   // Defense-in-depth retry: even though the daemon now writes atomically
@@ -234,12 +200,18 @@ function readObserverState(cwd) {
     if (alive) return { running: true, pid, port, source: 'pidfile', observerJsonPath };
   }
 
-  // Fallback: cross-platform net.connect probe. Catches the case where the
-  // pidfile is missing (race / external start) but the daemon is still
-  // listening on the port. Can't attribute the PID, so report pid: null.
-  if (portHasListenerSync(port)) {
-    return { running: true, pid: null, port, source: 'tcp-probe', observerJsonPath };
-  }
+  // Fallback: TCP probe via lsof. Catches the case where pidfile is missing
+  // but the daemon process is still listening on the port.
+  try {
+    const { spawnSync } = require('child_process');
+    const r = spawnSync('lsof', ['-i', `:${port}`, '-sTCP:LISTEN', '-t'], { encoding: 'utf8', timeout: 1500 });
+    if (r.status === 0 && r.stdout && r.stdout.trim()) {
+      const tcpPid = parseInt(r.stdout.trim().split('\n')[0], 10);
+      if (Number.isFinite(tcpPid) && tcpPid > 0) {
+        return { running: true, pid: tcpPid, port, source: 'tcp-probe', observerJsonPath };
+      }
+    }
+  } catch {}
 
   return { running: false, pid: pid, port, source: 'no-listener', observerJsonPath };
 }