local-mcp 3.0.253 → 3.0.255

package/package.json

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

@@ -535,97 +535,12 @@ async function runSetup(opts = {}) {
     } catch { /* config not created yet — will be created on first run */ }
   }
 
-  // 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 skip.
-  // ENXIO fix (LMCA-401): pre-test /dev/tty open on macOS/Linux. ENXIO means
-  // 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', '', '')
-  } else if (!email) {
-    let emailPromptResult = 'not_shown_no_tty'
-    let emailPromptError = ''
-    try {
-      let rl
-      let ttyFd = null
-
-      if (_IS_WIN) {
-        // 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.
-        // 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)
-            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() },
-        })
-        rl = require('readline').createInterface({ input: ttyIn, output: ttyOut, terminal: true })
-      }
-
-      process.stderr.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(); if (ttyFd !== null) try { fs.closeSync(ttyFd) } catch {} ; res((a || '').trim()) })
-        }),
-        new Promise(res => setTimeout(() => {
-          try { rl.close() } catch {}
-          if (ttyFd !== null) try { fs.closeSync(ttyFd) } catch {}
-          res('')
-        }, 30000)),
-      ])
-
-      if (ans && ans.includes('@')) {
-        email = ans
-        emailPromptResult = 'submitted'
-        try {
-          const r = _safeReadConfig(cfgFile)
-          const cfg = r.data || {}
-          if (!cfg.license_email) { cfg.license_email = email; _atomicWriteConfig(cfgFile, cfg) }
-        } catch {}
-        console.log('  ✓ Email saved')
-      } else {
-        emailPromptResult = 'skipped'
-      }
-    } 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)
-  }
+  // Interactive email prompt removed (2026-06-05) — same change already made in
+  // install.sh on 2026-04-19. It paused `curl | bash` up to 30s on a confusing
+  // mid-install prompt while ground-truth telemetry showed ~98% of machines that
+  // hit it reached a heartbeat anyway (no conversion impact). Email is still
+  // captured non-interactively: via LMCP_EMAIL above, the anon tunnel register,
+  // and the tray/settings UI later.
 
   // Health check — verify binary works before showing success
   const healthOk = _runHealthCheck()
@@ -950,34 +865,6 @@ function _trackConfigWritten(clientId, clientName) {
   } catch { /* non-fatal */ }
 }
 
-function _trackEmailPrompt(result, email, errorMsg) {
-  // result: 'shown' | 'submitted' | 'skipped' | 'not_shown_no_tty' | 'failed'
-  try {
-    const https = require('https')
-    const machineId = _getMachineId()
-    const payload = {
-      stage: 'email_prompt',
-      email_prompt_result: result,
-      machine_id: machineId,
-      install_id: process.env.INSTALL_ID || '',
-    }
-    if (email) payload.email = email
-    if (errorMsg) payload.ref = errorMsg.slice(0, 200)  // reuse ref field; fits in existing schema
-    const data = JSON.stringify(payload)
-    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('response', (res) => res.resume())
-    req.on('error', () => {})
-    req.write(data)
-    req.end()
-  } catch { /* non-fatal */ }
-}
-
 function _trackRestartPrompted(clientName) {
   try {
     const https = require('https')