local-mcp 3.0.166 → 3.0.168

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "local-mcp",
-  "version": "3.0.166",
+  "version": "3.0.168",
   "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

@@ -511,12 +511,17 @@ async function runSetup(opts = {}) {
   // (stdin is the pipe in that context; /dev/tty is always the real terminal).
   // Non-fatal: if /dev/tty is unavailable (CI, automated scripts) we skip.
   // ENXIO fix (LMCA-401): pre-test /dev/tty open on macOS/Linux. ENXIO means
-  // no controlling terminal (launchd, SSH -T, background process). 11 machines
-  // were logging prompt failures — this avoids the noisy error path entirely.
-  let _hasTty = _IS_WIN ? true : false
-  if (!_IS_WIN) {
-    try { const fd = fs.openSync('/dev/tty', 'r'); fs.closeSync(fd); _hasTty = true }
-    catch { /* ENXIO or ENOENT — no terminal available */ }
+  // no controlling terminal (launchd, SSH -T, background process).
+  // LMCA-415: Windows also needs a TTY check (npm subprocess has no stdin.isTTY
+  // when launched from Go tray installer). Pre-test uses 'r+' (same flags as
+  // the actual open) to avoid a race where the pre-test passes but actual open fails.
+  let _hasTty = false
+  if (_IS_WIN) {
+    // Windows: check stdin.isTTY — false when spawned as a subprocess without console
+    _hasTty = process.stdin && process.stdin.isTTY === true
+  } else {
+    try { const fd = fs.openSync('/dev/tty', 'r+'); fs.closeSync(fd); _hasTty = true }
+    catch { /* ENXIO or ENOENT — no controlling terminal */ }
   }
   if (!email && !_hasTty) {
     _trackEmailPrompt('skipped_no_tty', '', '')
@@ -531,8 +536,15 @@ async function runSetup(opts = {}) {
         // Windows: use process.stdin directly (works in PowerShell and cmd)
         rl = require('readline').createInterface({ input: process.stdin, output: process.stderr })
       } else {
-        // macOS/Linux: open /dev/tty to read from terminal even when stdin is piped
-        ttyFd = fs.openSync('/dev/tty', 'r+')
+        // macOS/Linux: open /dev/tty to read from terminal even when stdin is piped.
+        // If this throws (ENXIO/ENOENT), treat as no-tty, not a failure.
+        try { ttyFd = fs.openSync('/dev/tty', 'r+') }
+        catch (ttyErr) {
+          _trackEmailPrompt('skipped_no_tty', '', '')
+          ttyFd = null
+          // Skip the rest of the prompt block
+          throw Object.assign(new Error('no_tty'), { _skipPrompt: true })
+        }
         const ttyIn  = require('stream').Readable.from(
           (function* () {
             const buf = Buffer.alloc(256)
@@ -578,7 +590,10 @@ async function runSetup(opts = {}) {
       } else {
         emailPromptResult = 'skipped'
       }
-    } catch (err) { emailPromptResult = 'failed'; emailPromptError = (err && err.message) || String(err) }
+    } catch (err) {
+      if (err && err._skipPrompt) { /* skipped_no_tty already tracked above */ }
+      else { emailPromptResult = 'failed'; emailPromptError = (err && err.message) || String(err) }
+    }
 
     // Track prompt outcome so we can measure conversion and diagnose failures
     _trackEmailPrompt(emailPromptResult, emailPromptResult === 'submitted' ? email : '', emailPromptError)