local-mcp 3.0.137 → 3.0.138

package/index.js

@@ -24,6 +24,20 @@ if (process.platform !== 'darwin') {
   process.exit(1)
 }
 
+// Node version guard — Claude Desktop can silently launch LMCP with an old
+// nvm default (e.g. v11) that makes `npx -y` fail before this file loads.
+// If this code IS running on an old Node, surface the error clearly so the
+// user sees it in Claude Desktop's MCP logs instead of a cryptic spawn error.
+if (parseInt(process.version.slice(1)) < 16) {
+  const msg = `LMCP requires Node 16+. Running ${process.version}.\n` +
+    'If you use nvm, run: nvm alias default 22\n' +
+    'Then restart Claude Desktop / Cursor.'
+  process.stderr.write(msg + '\n')
+  // Exit with a non-zero code so the MCP host marks the server as failed
+  // (vs hanging forever with no output).
+  process.exit(1)
+}
+
 const cmd = process.argv[2]
 
 async function main() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "local-mcp",
-  "version": "3.0.137",
+  "version": "3.0.138",
   "description": "LMCP — connect Claude Desktop, Cursor, Windsurf to Mail, Calendar, Contacts, Teams, OneDrive on macOS. Privacy-first: all data stays on your Mac.",
   "main": "index.js",
   "bin": {

package/setup.js

@@ -16,6 +16,49 @@ const { execSync, execFileSync } = require('child_process')
 const HOME = os.homedir()
 const NPX_COMMAND = 'npx'
 const NPX_ARGS    = ['-y', 'local-mcp@latest']
+
+// Resolve absolute path to npx from the current Node binary so Claude Desktop
+// doesn't pick up an old npx from its own PATH (common with nvm users whose
+// nvm default points to Node v11/v12 — those npx versions don't support -y).
+// Falls back to plain 'npx' if resolution fails (non-nvm setups, symlinks, etc.)
+function _resolveNpxPath() {
+  try {
+    const candidate = path.join(path.dirname(process.execPath), 'npx')
+    if (fs.existsSync(candidate)) return candidate
+  } catch {}
+  return NPX_COMMAND
+}
+
+// Write a shell launcher script that wraps npx with a Node version check and
+// curl-based telemetry. This gives us visibility into failures that happen
+// BEFORE Node loads our code (e.g. old nvm default in Claude Desktop's PATH).
+// Returns the path to the launcher, or null if writing fails.
+function _writeLaunchScript(npxAbsPath, cacheDir) {
+  try {
+    fs.mkdirSync(cacheDir, { recursive: true })
+    const launcherPath = path.join(cacheDir, 'lmcp-launch.sh')
+    const script = [
+      '#!/bin/bash',
+      '# LMCP launcher — generated by npx local-mcp setup, do not edit.',
+      '# Checks Node version before launching npx; sends telemetry if too old.',
+      'NODE_VER=$(node --version 2>/dev/null || echo "none")',
+      'NODE_MAJOR=$(echo "$NODE_VER" | sed \'s/v//\' | cut -d. -f1)',
+      'if [ "${NODE_MAJOR:-0}" -lt 16 ] 2>/dev/null; then',
+      '  # curl telemetry — no Node dependency, fire-and-forget',
+      `  curl -sf --max-time 3 -X POST "https://${BACKEND_HOST}/install-event" \\`,
+      '    -H "Content-Type: application/json" \\',
+      '    -d "{\\\"stage\\\":\\\"node_too_old\\\",\\\"version\\\":\\\"$NODE_VER\\\"}" &>/dev/null &',
+      '  printf \'LMCP requires Node 16+. Current: %s\\nFix: nvm alias default 22 — then restart Claude Desktop.\\n\' "$NODE_VER" >&2',
+      '  exit 1',
+      'fi',
+      `exec "${npxAbsPath}" -y local-mcp@latest "$@"`,
+    ].join('\n') + '\n'
+    fs.writeFileSync(launcherPath, script, { mode: 0o755 })
+    return launcherPath
+  } catch {
+    return null
+  }
+}
 const STABLE_LINK   = path.join(os.homedir(), '.local', 'share', 'local-mcp', 'bin', 'local-mcp-server')
 const BACKEND_HOST  = 'office-mcp-production.up.railway.app'
 
@@ -212,9 +255,21 @@ async function runSetup(opts = {}) {
   console.log('║     LMCP — Setup Wizard         ║')
   console.log('╚══════════════════════════════════════╝\n')
 
+  // Warn if running on an old Node version. Claude Desktop resolves 'npx' from
+  // its own PATH (not the user's shell), so an old nvm default (v11/v12) causes
+  // a silent "You must supply a command" failure on first launch.
+  const nodeMajor = parseInt(process.version.slice(1))
+  if (nodeMajor < 16) {
+    console.error(`\n⚠️  Warning: you're running Node ${process.version}.`)
+    console.error('   Claude Desktop may fail to start LMCP because it finds an old npx on its PATH.')
+    console.error('   Fix: nvm alias default 22  (then relaunch Claude Desktop)\n')
+  }
+
   // Pre-download binary so first launch is instant
-  // All clients use npx as launcher (self-healing if binary is missing/broken)
-  let stableCommand = NPX_COMMAND
+  // All clients use a shell launcher that wraps npx (self-healing if binary is
+  // missing/broken, and surfaces telemetry + clear errors if Node is too old).
+  let npxAbsPath    = _resolveNpxPath()   // baked into the launcher script
+  let stableCommand = NPX_COMMAND         // fallback; replaced below once CACHE_DIR is known
   let stableArgs    = NPX_ARGS
   let binaryVersion = ''
   try {
@@ -252,11 +307,25 @@ async function runSetup(opts = {}) {
       if (fs.existsSync(settingsSrc)) fs.copyFileSync(settingsSrc, settingsDst)
     } catch {}
     process.stderr.write('✓ Runtime ready\n\n')
+    // Write the launcher script now that we have CACHE_DIR.
+    // The launcher uses curl (no Node dependency) to send telemetry if Node is
+    // too old, then exec's npx. Claude Desktop runs this script instead of npx
+    // directly, so failures are visible in MCP logs AND in our backend events.
+    const launcherPath = _writeLaunchScript(npxAbsPath, CACHE_DIR)
+    if (launcherPath) stableCommand = launcherPath
   } catch (err) {
     process.stderr.write(`  (Runtime download failed, will download on first run: ${err.message})\n\n`)
+    // Still try to write a launcher even if binary download failed — the version
+    // check + telemetry path is independent of the binary.
+    try {
+      const { CACHE_DIR } = require('./download')
+      const launcherPath = _writeLaunchScript(npxAbsPath, CACHE_DIR)
+      if (launcherPath) stableCommand = launcherPath
+    } catch {}
   }
-  // Always use npx as the command — it has a fast-path that execs the binary directly
-  // but can self-heal if the binary is missing or broken
+  // stableCommand is now the shell launcher script path. It wraps npx with a
+  // Node version check + curl telemetry before exec'ing. If launcher creation
+  // failed it falls back to plain 'npx'.
 
   // Detectar clientes
   const detected = CLIENTS.filter(c => c.detect())