claude-phone-local 2.1.1 → 2.2.0

package/claude-api-server/server.js

@@ -159,10 +159,22 @@ SLACK DELIVERY: When the caller requests delivery to Slack (phrases like "send t
 
 The caller may hang up while you're working (they'll hear hold music). That's fine - complete the work and send to Slack. They'll see it there.
 
+END OF CALL: The phone app matches a fixed list of goodbye words (bye, goodbye, hang up, etc.) to end the call automatically, so it misses anything phrased differently - "that's everything, thanks", "I'm all set", "nothing else for now", "we're done here". You understand intent better than a keyword match does, so ALSO add a third line whenever the caller's message signals the conversation is actually finished (a clear goodbye/thanks-and-done in any wording, in any language) - not for a mid-conversation pause, not for "give me a second", not merely because their immediate question was answered:
+
+🔚 END_CALL: true
+
+Omit this line entirely (do not write "false") on every turn that isn't a real goodbye - most turns won't have it.
+
 Example query: "What's the weather in Royce City?"
 Example response:
 🗣️ VOICE_RESPONSE: It's 65 degrees and partly cloudy in Royce City right now. Great weather for being outside!
 🎯 COMPLETED: Weather lookup for Royce City done.
+
+Example query: "Great, that's everything, thanks!"
+Example response:
+🗣️ VOICE_RESPONSE: You're welcome! Have a great day.
+🎯 COMPLETED: Caller signed off, no further requests.
+🔚 END_CALL: true
 [END VOICE CONTEXT]
 
 `;

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.2.0",
   "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": {

package/voice-app/lib/sip-handler.js

@@ -162,6 +162,17 @@ function extractVoiceLine(response) {
   return response.substring(0, 500).trim();
 }
 
+/**
+ * Whether Claude's raw response signals the conversation is actually over.
+ * Complements isGoodbye()'s fixed phrase match on the caller's own words -
+ * Claude sees the full exchange and can recognize a sign-off phrased in ways
+ * the keyword list doesn't cover ("that's everything, thanks", "I'm all
+ * set"), in any language, without maintaining a growing keyword list.
+ */
+function hasEndCallMarker(response) {
+  return /🔚\s*END_CALL:\s*true/i.test(response);
+}
+
 /**
  * Play a clip the caller is allowed to interrupt.
  *
@@ -443,6 +454,13 @@ async function conversationLoop(endpoint, dialog, callUuid, options, deviceConfi
         const responseBarge = await playInterruptible(endpoint, session, responseUrl);
         if (responseBarge) {
           pendingUtterance = responseBarge;
+        } else if (hasEndCallMarker(claudeResponse)) {
+          // Claude judged this a real sign-off from the caller's own words
+          // (not just isGoodbye()'s fixed phrase match) - she already said
+          // her goodbye as part of the normal response above, so just end
+          // the call rather than waiting for another turn.
+          console.log('[' + new Date().toISOString() + '] CONVERSATION Claude signaled end of call');
+          break;
         }
       }