local-mcp 3.0.378 → 3.0.380

package/download.js

@@ -52,6 +52,10 @@ const TRAY_APP = process.platform === 'darwin'
   ? path.join(os.homedir(), 'Applications', 'LocalMCPTray.app')
   : null
 
+// La instalación vieja que TRAY_APP supersede. Const (no literal suelto) porque el orden en
+// que se la retira es lo que arregla #2654 y los tests la inyectan.
+const SYSTEM_TRAY_APP = '/Applications/LocalMCPTray.app'
+
 // Mirror of go-server's CacheDir()/.machine-id (internal/machineid/machineid.go).
 // Only the win32 ladder reads/writes it: it's the side of the probe that can
 // come up empty on a real machine (wmic gone, PowerShell blocked by policy),
@@ -532,12 +536,65 @@ async function ensureTeamsProxy(platform = process.platform) {
  * Descarga e instala el tray SwiftUI (menu bar app) — universal binary (arm64 + Intel).
  * @returns {Promise<string|null>} Ruta al .app instalado, o null si falla
  */
-async function ensureTray() {
+// Un .app "usable" es el EJECUTABLE adentro, no el directorio: una extracción a medias deja
+// el bundle existiendo y vacío, y esa fue la falla de #1338. (#2654)
+function _isUsableTrayBundle(trayApp) {
+  try { return fs.existsSync(path.join(trayApp, 'Contents', 'MacOS', 'LocalMCPTray')) }
+  catch { return false }
+}
+
+// Retira las copias que `trayApp` supersede — SOLO si `trayApp` ya tiene un bundle usable.
+// Sin esa guarda, la migración borra la única copia que el Mac tiene antes de que exista su
+// reemplazo, y una descarga fallida deja la máquina sin cliente (#2654, hermano de #2643).
+// Devuelve lo retirado para que los tests puedan afirmar sobre el efecto, no sobre el orden.
+function _retireSupersededTrays(trayApp, supersededPaths, rm = _rmrf) {
+  if (!trayApp || !_isUsableTrayBundle(trayApp)) return []
+  const retired = []
+  for (const p of supersededPaths) {
+    if (!p) continue
+    if (_sameLocation(p, trayApp)) continue
+    if (!fs.existsSync(p)) continue
+    try { rm(p); retired.push(p) } catch { /* ignorar */ }
+  }
+  return retired
+}
+
+// Identidad, no grafía: con ~/Applications symlinkeado a /Applications los dos nombres son el
+// MISMO directorio, y un path.resolve() los ve distintos — el retiro se llevaba puesto el bundle
+// recién instalado y la máquina no volvía a tener cliente nunca. Cubre symlinks, no diferencias
+// de mayúsculas: realpathSync no normaliza case en APFS, y ningún alias así es alcanzable desde
+// TRAY_APP/SYSTEM_TRAY_APP, que son constantes. (#2654)
+function _sameLocation(a, b) {
+  try { return fs.realpathSync(a) === fs.realpathSync(b) }
+  catch { return path.resolve(a) === path.resolve(b) }
+}
+
+function _rmrf(p) {
+  execFileSync('rm', ['-rf', p], { stdio: 'pipe' })
+}
+
+// `deps` es la costura que hace testeable el ORDEN de esta función sin red ni HOME real:
+// los tests de #2654 la corren entera con dirs temporales y una descarga que falla. Mismo
+// idioma que _getMachineId(deps = {}). Producción no pasa nada y usa los valores de módulo.
+async function ensureTray(deps = {}) {
+  const {
+    platform         = process.platform,
+    trayApp          = TRAY_APP,
+    trayDir          = TRAY_DIR,
+    systemTrayApp    = SYSTEM_TRAY_APP,
+    uninstalledMarkers = null,
+    getLatest        = getLatestBinary,
+    download         = downloadFile,
+    extract          = extractTar,
+    writeLaunchAgent = _writeTrayLaunchAgent,
+    isTTY            = () => process.stdin.isTTY,
+  } = deps
+
   // Windows/Linux: the tray + background daemon are installed by the native
   // installer (LMCP-Setup.exe on Windows; the systemd unit on Linux), NOT by npm.
   // The npm package is a stdio proxy for MCP hosts, so there is no tray to fetch
   // here — no-op instead of downloading a lmcp-tray binary.
-  if (process.platform !== 'darwin') {
+  if (platform !== 'darwin') {
     return null
   }
 
@@ -545,47 +602,51 @@ async function ensureTray() {
   // Checks BOTH the runtime-dir copy and the durable App Support copy: a user who
   // "cleaned up" with `rm -rf ~/.local/share/local-mcp` removes the runtime-dir copy,
   // and only the durable one survives to stop a resurrection loop (FB-D1FA79).
-  const uninstalledMarkers = [
+  const markers = uninstalledMarkers || [
     path.join(CACHE_DIR, '..', '.uninstalled'),
     path.join(os.homedir(), 'Library', 'Application Support', 'Local MCP', '.uninstalled'),
   ]
-  if (uninstalledMarkers.some((p) => { try { return fs.existsSync(p) } catch { return false } })) return null
+  if (markers.some((p) => { try { return fs.existsSync(p) } catch { return false } })) return null
 
-  const info     = await getLatestBinary()
+  const info     = await getLatest()
   const version  = info.version
-  const trayApp  = TRAY_APP
-  const verFile  = path.join(TRAY_DIR, '.version')
-
-  // Migrar instalación vieja de ~/.local/share si existe
-  const oldTrayApp = path.join(TRAY_DIR, 'LocalMCPTray.app')
-  if (fs.existsSync(oldTrayApp)) {
-    try { execFileSync('rm', ['-rf', oldTrayApp], { stdio: 'pipe' }) } catch { /* ignorar */ }
-  }
-
-  // Migrar instalación vieja de /Applications → ~/Applications (evita diálogo App Management)
-  const systemTrayApp = '/Applications/LocalMCPTray.app'
-  if (fs.existsSync(systemTrayApp) && trayApp !== systemTrayApp) {
-    try { execFileSync('rm', ['-rf', systemTrayApp], { stdio: 'pipe' }) } catch { /* ignorar */ }
-  }
-
-  // Ya instalado y actualizado
-  if (fs.existsSync(trayApp) && fs.existsSync(verFile)) {
-    if (fs.readFileSync(verFile, 'utf8').trim() === version) return trayApp
+  const verFile  = path.join(trayDir, '.version')
+
+  // Las dos migraciones —~/.local/share (legacy) y /Applications → ~/Applications (evita el
+  // diálogo App Management)— retiran una copia que puede ser la ÚNICA del Mac. Por eso el
+  // retiro corre en exactamente dos lugares: pegado a cada early-return (no se instala nada,
+  // el tray de destino ya está verificado) y después de extraer y verificar. Nunca en el
+  // camino que descarga: ahí una falla dejaba la máquina sin cliente. (#2654)
+  const superseded = [path.join(trayDir, 'LocalMCPTray.app'), systemTrayApp]
+  const retireSuperseded = () => _retireSupersededTrays(trayApp, superseded)
+
+  // Ya instalado y actualizado. "Instalado" es el bundle USABLE, no el directorio: si no,
+  // una extracción a medias se early-returnea para siempre y nadie la repara (#1338).
+  if (_isUsableTrayBundle(trayApp) && fs.existsSync(verFile)) {
+    if (fs.readFileSync(verFile, 'utf8').trim() === version) {
+      retireSuperseded()
+      return trayApp
+    }
   }
 
   // In an active MCP session (non-TTY) the tray is running and self-updates via UpdateManager.
   // Downloading a new tray bundle and rm-rf'ing the old one mid-session kills the Unix socket
   // that local-mcp-server uses, crashing the session. Skip the update; it'll apply next login.
-  if (!process.stdin.isTTY && fs.existsSync(trayApp)) return trayApp
+  // Un bundle sin ejecutable no tiene tray corriendo que esta descarga pueda romper, así que
+  // ese caso NO se early-returnea: se repara.
+  if (!isTTY() && _isUsableTrayBundle(trayApp)) {
+    retireSuperseded()
+    return trayApp
+  }
 
   const url = `https://download.local-mcp.com/local-mcp-tray-${version}-darwin-universal.tar.gz`
   process.stderr.write(`\nDescargando tray v${version}...\n`)
 
-  fs.mkdirSync(TRAY_DIR, { recursive: true })
-  const tarPath = path.join(TRAY_DIR, `tray-${version}.tar.gz`)
+  fs.mkdirSync(trayDir, { recursive: true })
+  const tarPath = path.join(trayDir, `tray-${version}.tar.gz`)
 
   // Descargar ANTES de borrar el viejo — si la descarga falla, el tray actual sigue intacto
-  await downloadFile(url, tarPath)
+  await download(url, tarPath)
 
   // Verificar que el archivo descargado no está vacío
   const tarStat = fs.statSync(tarPath)
@@ -596,33 +657,38 @@ async function ensureTray() {
 
   // Recién ahora borrar el viejo e instalar el nuevo
   if (fs.existsSync(trayApp)) {
-    try { execFileSync('rm', ['-rf', trayApp], { stdio: 'pipe' }) } catch { /* ignorar */ }
+    try { _rmrf(trayApp) } catch { /* ignorar */ }
   }
 
   // Crear ~/Applications si no existe (ubicación estándar para apps de usuario)
-  const userAppsDir = path.join(os.homedir(), 'Applications')
+  const userAppsDir = path.dirname(trayApp)
   fs.mkdirSync(userAppsDir, { recursive: true })
 
   process.stderr.write(`  Instalando tray en ~/Applications...\n`)
-  extractTar(tarPath, userAppsDir)
+  extract(tarPath, userAppsDir)
   fs.unlinkSync(tarPath)
 
-  if (!fs.existsSync(trayApp)) {
-    throw new Error(`Tray no encontrado tras extraer: ${trayApp}`)
+  // Mismo criterio que usa el retiro, o se estampa .version sobre un bundle roto y el
+  // early-return de arriba lo da por bueno en cada corrida siguiente.
+  if (!_isUsableTrayBundle(trayApp)) {
+    throw new Error(`Tray no usable tras extraer: ${trayApp}`)
   }
 
+  // El reemplazo está instalado y verificado: recién ACÁ se retira lo que supersede.
+  retireSuperseded()
+
   // Quitar quarantine — la app viene firmada con Developer ID desde CI, no re-firmar
   try { execFileSync('xattr', ['-rd', 'com.apple.quarantine', trayApp], { stdio: 'pipe' }) } catch { /* no crítico */ }
 
   fs.writeFileSync(verFile, version, 'utf8')
   // Keep in sync with UpdateManager / Heartbeat (was .version only → fleet stuck on old tray)
-  const trayVerFile = path.join(TRAY_DIR, '.tray-version')
+  const trayVerFile = path.join(trayDir, '.tray-version')
   fs.writeFileSync(trayVerFile, version, 'utf8')
 
   // LaunchAgent — auto-start en cada reboot de sesión
-  _writeTrayLaunchAgent(trayApp)
+  writeLaunchAgent(trayApp)
 
-  process.stderr.write(`  Tray instalado en ${trayApp}${process.stdin.isTTY ? '' : ' (se iniciará en el próximo login)'}\n`)
+  process.stderr.write(`  Tray instalado en ${trayApp}${isTTY() ? '' : ' (se iniciará en el próximo login)'}\n`)
   return trayApp
 }
 
@@ -727,4 +793,5 @@ module.exports = {
   CACHE_DIR, TRAY_DIR, _getMachineId,
   _probeWinHardwareId, _resetWinHardwareIdCacheForTests,
   _readWinMachineIdCache, _writeWinMachineIdCache, WIN_MACHINE_ID_CACHE_NAME,
+  _retireSupersededTrays, _isUsableTrayBundle, SYSTEM_TRAY_APP,
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "local-mcp",
-  "version": "3.0.378",
+  "version": "3.0.380",
   "description": "Let ChatGPT, Claude, Cursor & any MCP client actually use your Mac — read & reply to email, manage your calendar, text over iMessage, find files, work with Teams, Slack & Office. On your Mac, no API keys, free.",
   "main": "index.js",
   "bin": {