local-mcp 3.0.376 → 3.0.378

package/download.js

@@ -145,15 +145,22 @@ function _resetWinHardwareIdCacheForTests() {
   _winHardwareIdCache = undefined
 }
 
-function _getMachineId() {
+// deps existe SOLO como costura de test (#1705): los 8 tests de orden ejercitaban
+// _probeWinHardwareId y el cache por separado, nunca los tres juntos por acá, asi
+// que invertir la prioridad hardware-antes-que-cache —la decision irreversible de
+// machineid.go— los pasaba a todos en verde. Sin argumentos se comporta igual que
+// antes; ningun caller de produccion pasa deps.
+function _getMachineId(deps = {}) {
+  const platform = deps.platform || process.platform
+  const cacheDir = deps.cacheDir || CACHE_DIR
   // QA override, checked before any platform probe — same rank as go-server's
   // Get() — so a smoke-* run stays a smoke-* run on every code path that
   // reports machine_id, not just the ones that already remembered to check it.
   const envId = (process.env.LMCP_MACHINE_ID || '').trim()
   if (envId) return envId
 
-  const { execSync } = require('child_process')
-  if (process.platform === 'darwin') {
+  const execSync = deps.execImpl || require('child_process').execSync
+  if (platform === 'darwin') {
     try {
       const r = execSync('security find-generic-password -s com.local-mcp.machine-id -a machine-id -w 2>/dev/null', { stdio: 'pipe' })
       const id = r.toString().trim()
@@ -164,17 +171,17 @@ function _getMachineId() {
       const match = ioreg.match(/"IOPlatformUUID"\s*=\s*"([^"]+)"/)
       if (match) return match[1]
     } catch {}
-  } else if (process.platform === 'win32') {
-    const hw = _probeWinHardwareId()
+  } else if (platform === 'win32') {
+    const hw = _probeWinHardwareId(execSync)
     if (hw) {
-      _writeWinMachineIdCache(CACHE_DIR, hw)
+      _writeWinMachineIdCache(cacheDir, hw)
       return hw
     }
     // Both probes failed or are blocked — fall back to whatever the go-server
     // already cached rather than diverging to the hostname hash (the poisoned-
     // cache case go-server's own Get() guards against does not apply here:
     // this file is only ever written with a hardware-derived id, by either side).
-    const cached = _readWinMachineIdCache(CACHE_DIR)
+    const cached = _readWinMachineIdCache(cacheDir)
     if (cached) return cached
   } else {
     try {
@@ -208,6 +215,22 @@ function versionFromArtifactUrl(url, arch) {
   return m ? m[1] : null
 }
 
+// La version contra la que ESTA plataforma tiene que compararse, sacada del
+// mismo manifiesto. Existe porque `info.version` es la linea de Mac: usarla en
+// Windows/Linux compara dos series distintas, que es lo que fastPathVersionOK ya
+// se niega a hacer en el fast path (#1394 §C).
+//
+// Devuelve null cuando no se puede determinar — sin artefacto para la
+// plataforma, o con un nombre que no parsea. El llamador NO debe caer a
+// info.version: preferimos no decidir antes que decidir sobre un numero que
+// describe otra plataforma.
+function latestVersionForPlatform(info, arch) {
+  if (!info) return null
+  if (String(arch).startsWith('darwin')) return info.version || null
+  const url = info[String(arch).replace('-', '_')]
+  return url ? versionFromArtifactUrl(url, arch) : null
+}
+
 /**
  * Obtiene la versión más reciente del binario desde el backend.
  * Si esta máquina está en beta_machines, el backend retorna la versión beta.
@@ -700,6 +723,7 @@ async function ensureSlackProxy(platform = process.platform) {
 
 module.exports = {
   ensureBinary, ensureRuntime, ensureTray, ensureTeamsProxy, ensureSlackProxy, versionFromArtifactUrl,
+  latestVersionForPlatform, getArch,
   CACHE_DIR, TRAY_DIR, _getMachineId,
   _probeWinHardwareId, _resetWinHardwareIdCacheForTests,
   _readWinMachineIdCache, _writeWinMachineIdCache, WIN_MACHINE_ID_CACHE_NAME,

package/index.js

@@ -62,6 +62,32 @@ function fastPathVersionOK(platform, cachedVersion, pkgVersion) {
   return true
 }
 
+/**
+ * Whether the 24h background check should download a newer binary.
+ *
+ * fastPathVersionOK (arriba) declara que en Windows/Linux la version cacheada y
+ * `info.version` son series distintas por construccion, se niega a compararlas,
+ * y delega en "the 24h background check below". Ese chequeo hacia exactamente la
+ * comparacion declarada invalida: `semverGte(cachedVersion, info.version)`, con
+ * info.version = la release de MAC.
+ *
+ * Medido el 2026-08-27: las 21 Windows de la flota corrian 3.0.401 con 3.0.402
+ * publicado y Mac en 3.0.377. semverGte('3.0.401','3.0.377') es true, asi que
+ * ninguna se enteraba del update — en silencio. Y cuando la linea de Mac pasa a
+ * la de Windows, la misma cuenta se da vuelta y actualiza lo que ya esta al dia.
+ *
+ * Sin version resoluble para la plataforma NO se actualiza: fail-closed, el
+ * mismo criterio que download.js ("we FAIL rather than fall back to
+ * info.version").
+ */
+function backgroundUpdateNeeded(cachedVersion, info, arch) {
+  if (!cachedVersion) return false
+  const { latestVersionForPlatform } = require('./download')
+  const latest = latestVersionForPlatform(info, arch)
+  if (!latest) return false
+  return !semverGte(cachedVersion, latest)
+}
+
 /**
  * PIDs of the lmcp/local-mcp servers running right now, or null when the answer
  * cannot be determined.
@@ -557,8 +583,13 @@ async function main() {
               res.on('data', c => buf += c)
               res.on('end', () => {
                 try {
-                  const { version: latestVersion } = JSON.parse(buf)
-                  if (latestVersion && !semverGte(_cachedVersion, latestVersion)) {
+                  const info = JSON.parse(buf)
+                  const { latestVersionForPlatform, getArch } = require('./download')
+                  const _arch = getArch()
+                  // El nombre que se escribe en .server-version si hay que bajar:
+                  // el de ESTA plataforma, no el de la release de Mac.
+                  const latestVersion = latestVersionForPlatform(info, _arch)
+                  if (backgroundUpdateNeeded(_cachedVersion, info, _arch)) {
                     // New version available — download for next restart only while no server is running.
                     // Replacing stableBin mid-session changes its mtime and used to cause sibling
                     // stdio proxies to be SIGTERM'd → Cursor transport_closed (LMCA-1044).
@@ -644,4 +675,4 @@ if (require.main === module) {
   })
 }
 
-module.exports = { semverGte, fastPathVersionOK, listRunningServerPids, countRunningServers, safeToReplaceBinary, replaceCachedBinary }
+module.exports = { semverGte, fastPathVersionOK, backgroundUpdateNeeded, listRunningServerPids, countRunningServers, safeToReplaceBinary, replaceCachedBinary }

package/package.json

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