local-mcp 3.0.126 → 3.0.128

package/package.json

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

@@ -244,27 +244,60 @@ async function runSetup(opts = {}) {
     } catch { /* config not created yet — will be created on first run */ }
   }
 
-  // Interactive email prompt — only if TTY and no email from env (LMC-833)
-  // Lets the activation nudge reach this machine. Optional, non-blocking.
-  if (!email && process.stdin.isTTY) {
+  // Interactive email prompt — uses /dev/tty so it works even from `curl | bash`
+  // (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 catch and continue.
+  if (!email) {
+    let emailPromptResult = 'not_shown_no_tty'
     try {
-      const rl = require('readline').createInterface({ input: process.stdin, output: process.stdout })
-      const ans = await new Promise(res => {
-        rl.question('\n  Email for support & update notifications (optional, press Enter to skip): ', a => {
-          rl.close()
-          res((a || '').trim())
-        })
+      const ttyFd = fs.openSync('/dev/tty', 'r+')
+      const ttyIn  = require('stream').Readable.from(
+        (function* () {
+          const buf = Buffer.alloc(256)
+          let line = ''
+          while (true) {
+            const n = fs.readSync(ttyFd, buf, 0, 1, null)
+            if (n === 0) break
+            const ch = buf.slice(0, n).toString()
+            if (ch === '\n' || ch === '\r') break
+            line += ch
+          }
+          yield line
+        })()
+      )
+      const ttyOut = new (require('stream').Writable)({
+        write(chunk, _enc, cb) { fs.writeSync(ttyFd, chunk); cb() },
       })
+      const rl = require('readline').createInterface({ input: ttyIn, output: ttyOut, terminal: true })
+
+      process.stdout.write('\n  📧 Email for update notifications (optional, press Enter to skip): ')
+      emailPromptResult = 'shown'
+      const ans = await Promise.race([
+        new Promise(res => {
+          rl.once('line', a => { rl.close(); try { fs.closeSync(ttyFd) } catch {} ; res((a || '').trim()) })
+        }),
+        new Promise(res => setTimeout(() => {
+          try { rl.close() } catch {}
+          try { fs.closeSync(ttyFd) } catch {}
+          res('')
+        }, 30000)),
+      ])
+
       if (ans && ans.includes('@')) {
         email = ans
+        emailPromptResult = 'submitted'
         try {
           const cfg = _readJson(cfgFile)
           if (!cfg.license_email) { cfg.license_email = email; _writeJson(cfgFile, cfg) }
         } catch {}
-        // Email will be sent to the backend on the next heartbeat via license_email
-        console.log(`  ✓ Email saved`)
+        console.log('  ✓ Email saved')
+      } else {
+        emailPromptResult = 'skipped'
       }
-    } catch { /* non-fatal — install continues without email */ }
+    } catch { emailPromptResult = 'failed' }
+
+    // Track prompt outcome so we can measure conversion and diagnose failures
+    _trackEmailPrompt(emailPromptResult)
   }
 
   // Health check — verify binary works before showing success
@@ -501,6 +534,30 @@ function _migrateCurlLaunchAgent() {
   } catch { /* non-fatal — don't block install on unexpected plist content */ }
 }
 
+function _trackEmailPrompt(result) {
+  // result: 'shown' | 'submitted' | 'skipped' | 'not_shown_no_tty' | 'failed'
+  try {
+    const https = require('https')
+    const machineId = _getMachineId()
+    const data = JSON.stringify({
+      stage: 'email_prompt',
+      email_prompt_result: result,
+      machine_id: machineId,
+      install_id: process.env.INSTALL_ID || '',
+    })
+    const req = https.request({
+      hostname: BACKEND_HOST,
+      path: '/install-event',
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json', 'Content-Length': data.length },
+      timeout: 5000,
+    })
+    req.on('error', () => {})
+    req.write(data)
+    req.end()
+  } catch { /* non-fatal */ }
+}
+
 function _pingInstall(clients, method, email = '') {
   try {
     const https = require('https')