claude-phone-local 2.0.4 → 2.0.6

package/cli/bin/postinstall.js

@@ -1,6 +1,50 @@
 #!/usr/bin/env node
-// Friendly next-steps message after `npm install -g claude-phone-local`.
-// Never fails the install (package.json calls this with `|| true`).
+// Post-install: installs host-side subpackage dependencies, then prints
+// next-steps. Never fails the parent install (package.json calls this with
+// `|| true`) - a failed subpackage install here just means `claude-phone
+// start`/`claude-phone doctor` will report it later with the exact fix.
+//
+// claude-api-server and mcp-server run directly on the host (not in Docker,
+// unlike voice-app), so npm doesn't install their deps automatically - `npm
+// install -g` only installs the top-level package's own dependencies.
+// Without this, users hit "Dependencies not installed" only when they later
+// run `claude-phone start`, despite the message below promising zero manual
+// steps.
+
+import { execFileSync } from 'node:child_process';
+import { existsSync } from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const packageRoot = path.resolve(__dirname, '..', '..');
+
+const HOST_SUBPACKAGES = ['claude-api-server', 'mcp-server'];
+
+for (const dir of HOST_SUBPACKAGES) {
+  const dirPath = path.join(packageRoot, dir);
+  const nodeModulesPath = path.join(dirPath, 'node_modules');
+
+  if (!existsSync(path.join(dirPath, 'package.json'))) continue;
+  if (existsSync(nodeModulesPath)) continue;
+
+  try {
+    console.log(`Installing dependencies for ${dir}...`);
+    // npm resolves to npm.cmd (a shell shim) on Windows, so it needs
+    // shell:true there to run at all. Node warns when shell:true is combined
+    // with an args array (each element gets shell-interpolated); passing a
+    // single fixed command string instead avoids that, and there is no user
+    // input in this string to make interpolation a real risk.
+    execFileSync('npm install --omit=dev', {
+      cwd: dirPath,
+      stdio: 'inherit',
+      shell: true
+    });
+  } catch (error) {
+    console.log(`\x1b[33mWarning: could not install ${dir} dependencies automatically.\x1b[0m`);
+    console.log(`  Run manually: cd ${dirPath} && npm install\n`);
+  }
+}
 
 console.log(`
 \x1b[36m claude-phone-local installed \x1b[0m

package/cli/lib/utils.js

@@ -51,11 +51,18 @@ export function getLocalIP() {
  * @returns {Promise<boolean>}
  */
 export async function isClaudeInstalled() {
+  // `which` doesn't exist on Windows (it's `where`), and a spawn ENOENT with
+  // no 'error' handler crashes the whole process rather than resolving false.
+  const lookupCmd = process.platform === 'win32' ? 'where' : 'which';
+
   return new Promise((resolve) => {
-    const check = spawn('which', ['claude']);
+    const check = spawn(lookupCmd, ['claude']);
     check.on('close', (code) => {
       resolve(code === 0);
     });
+    check.on('error', () => {
+      resolve(false);
+    });
   });
 }
 

package/mcp-server/package.json

@@ -5,5 +5,8 @@
   "type": "module",
   "bin": {
     "claude-phone-mcp": "./index.js"
+  },
+  "dependencies": {
+    "@modelcontextprotocol/sdk": "^1.0.0"
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-phone-local",
-  "version": "2.0.4",
+  "version": "2.0.6",
   "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": {