claude-phone-local 2.1.1 → 2.1.2

package/cli/lib/commands/start.js

@@ -6,7 +6,7 @@ import { loadConfig, configExists, getInstallationType } from '../config.js';
 import { checkDocker, writeDockerConfig, startContainers } from '../docker.js';
 import { startServer, isServerRunning } from '../process-manager.js';
 import { isClaudeInstalled } from '../utils.js';
-import { checkClaudeApiServer, waitForVoiceAppReady } from '../network.js';
+import { checkClaudeApiServer, waitForVoiceAppReady, waitForClaudeApiServerReady } from '../network.js';
 import { runPrereqChecks } from '../prereqs.js';
 
 /**
@@ -91,7 +91,17 @@ async function startApiServer(config) {
       spinner.warn('Claude API server already running');
     } else {
       await startServer(config.paths.claudeApiServer, config.server.claudeApiPort);
-      spinner.succeed(`Claude API server started on port ${config.server.claudeApiPort}`);
+      // startServer only confirms the process was spawned, not that it
+      // actually bound its port and stayed up (e.g. a stale process already
+      // on that port crashes it with EADDRINUSE within milliseconds).
+      const readiness = await waitForClaudeApiServerReady(`http://localhost:${config.server.claudeApiPort}`);
+      if (readiness.healthy) {
+        spinner.succeed(`Claude API server started on port ${config.server.claudeApiPort}`);
+      } else {
+        spinner.fail(`Claude API server did not become healthy: ${readiness.error || 'timed out'}`);
+        console.log(chalk.yellow(`\n  Check the log: claude-phone logs api-server\n`));
+        process.exit(1);
+      }
     }
   } catch (error) {
     spinner.fail(`Failed to start server: ${error.message}`);
@@ -332,7 +342,17 @@ async function startBoth(config, isPiMode) {
         spinner.warn('Claude API server already running');
       } else {
         await startServer(config.paths.claudeApiServer, config.server.claudeApiPort);
-        spinner.succeed(`Claude API server started on port ${config.server.claudeApiPort}`);
+        // startServer only confirms the process was spawned, not that it
+        // actually bound its port and stayed up (e.g. a stale process
+        // already on that port crashes it with EADDRINUSE within ms).
+        const apiReadiness = await waitForClaudeApiServerReady(`http://localhost:${config.server.claudeApiPort}`);
+        if (apiReadiness.healthy) {
+          spinner.succeed(`Claude API server started on port ${config.server.claudeApiPort}`);
+        } else {
+          spinner.fail(`Claude API server did not become healthy: ${apiReadiness.error || 'timed out'}`);
+          console.log(chalk.yellow(`\n  Check the log: claude-phone logs api-server\n`));
+          process.exit(1);
+        }
       }
     } catch (error) {
       spinner.fail(`Failed to start server: ${error.message}`);

package/cli/lib/network.js

@@ -108,6 +108,33 @@ export async function checkClaudeApiServer(url) {
   });
 }
 
+/**
+ * Poll claude-api-server's /health until it responds, or the timeout
+ * elapses. startServer() only confirms the process was spawned (got a PID) -
+ * not that it actually bound its port and stayed up. A crash right after
+ * spawn (e.g. EADDRINUSE from a stale process already on that port) was
+ * previously reported as "✔ Claude API server started" and "✓ All services
+ * running!" with no indication anything was wrong.
+ * @param {string} url - claude-api-server base URL (e.g. http://localhost:3333)
+ * @param {object} [opts]
+ * @param {number} [opts.timeoutMs=10000] - Give up after this long
+ * @param {number} [opts.intervalMs=500] - Poll interval
+ * @returns {Promise<{healthy: boolean, timedOut: boolean, error?: string}>}
+ */
+export async function waitForClaudeApiServerReady(url, { timeoutMs = 10000, intervalMs = 500 } = {}) {
+  const deadline = Date.now() + timeoutMs;
+  let lastError;
+
+  while (Date.now() < deadline) {
+    const result = await checkClaudeApiServer(url);
+    if (result.reachable && result.healthy) return { healthy: true, timedOut: false };
+    lastError = result.error;
+    await new Promise((r) => setTimeout(r, intervalMs));
+  }
+
+  return { healthy: false, timedOut: true, error: lastError };
+}
+
 /**
  * Poll voice-app's /health until it reports drachtio + FreeSWITCH both
  * connected, or the timeout elapses. Fixes `claude-phone start` reporting

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-phone-local",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "description": "Local/offline fork of NetworkChuck's claude-phone: talk to Claude Code over 3CX/SIP with faster-whisper STT + Piper TTS in one Docker container.",
   "type": "module",
   "bin": {