@dotrino/remote-agent 0.18.0 → 0.18.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.18.0",
+  "version": "0.18.2",
   "description": "Middleware compartido del ecosistema Dotrino para apps de agente remoto cifrado punto a punto (handshake E2E, emparejamiento SAS con vault, revocación, descubrimiento). Lo consumen dotrino-terminal (PTY) y dotrino-ia (chat con IA).",
   "type": "module",
   "main": "./index.js",

package/protocol.js

@@ -39,6 +39,11 @@ export const SIGN_SCOPE = 'vault:sign'
 /** Vigencia de una sesión inactiva y refresco de la lista de revocados. */
 export const SESSION_TTL_MS = 30 * 60 * 1000
 export const REVOKE_REFRESH_MS = 5 * 60 * 1000
+/**
+ * Cuánto se espera a un cliente cuya conexión se fue antes de cerrarle la sesión. Un cliente
+ * que reconecta sigue con su sesión desde un token nuevo: si en este plazo manda algo, se queda.
+ */
+export const SESSION_GONE_MS = 20 * 1000
 
 /**
  * RENOVACIÓN del cert de esta máquina. El cert dura 30 días; se pide uno fresco

package/src/agent.js

@@ -27,7 +27,7 @@ import { adoptRecord } from './record.js'
 import { installNodeGlobals } from '../node-globals.js'
 import { makeEphemeral, deriveKey, seal, open } from '../e2e.js'
 import { makeCatchUp } from './catch-up.js'
-import { HS, ACK, DATA, PING, PONG, ERROR, VMSG, SIGN_SCOPE, SESSION_TTL_MS, REVOKE_REFRESH_MS } from '../protocol.js'
+import { HS, ACK, DATA, PING, PONG, ERROR, VMSG, SIGN_SCOPE, SESSION_TTL_MS, SESSION_GONE_MS, REVOKE_REFRESH_MS } from '../protocol.js'
 import { loadLink, saveLink, dataDir } from './link.js'
 
 /** Sesión cifrada con un dispositivo cliente. La app recibe esto en `onSession`. */
@@ -88,6 +88,7 @@ class AgentSession {
   close () {
     if (this._closed) return
     this._closed = true
+    clearTimeout(this._gone)
     for (const h of this._h.close) { try { h() } catch (_) {} }
   }
 }
@@ -105,6 +106,7 @@ class AgentSession {
  * @param {()=>void} [opts.onRevoked] se llamó al auto-borrarse por revocación.
  * @param {()=>void} [opts.onReady]   agente listo y escuchando.
  * @param {boolean} [opts.quiet]      sin logs.
+ * @param {number} [opts.sessionGoneMs]  cuánto se espera a un cliente cuya conexión se fue (`SESSION_GONE_MS`).
  * @param {number} [opts.netLogMs]    cada cuánto van al log las estadísticas de red (5 min).
  * @param {object} [opts.client]      transporte ya conectado (SOLO pruebas).
  *
@@ -420,9 +422,27 @@ export async function startRemoteAgent (opts = {}) {
     if (!s) return send(from, { type: ERROR, code: 'unknown-session', sid: p.sid, error: 'sesión desconocida o expirada' })
     s._exp = Date.now() + SESSION_TTL_MS
     s.from = from
+    if (s._gone) { clearTimeout(s._gone); s._gone = null }
     s._ingest(p.env).catch(() => send(from, { type: ERROR, error: 'sobre inválido' }))
   }
 
+  // LA CONEXIÓN DEL CLIENTE SE FUE: su sesión no se queda 30 minutos. El proxio avisa de que ese
+  // token murió (`peer_disconnected`); un token no vuelve, así que quien sigue vivo manda desde
+  // otro y la sesión se queda (`handleData`). Si en `SESSION_GONE_MS` no llega nada, se cierra.
+  // Antes solo vencía por `SESSION_TTL_MS`: un teléfono que volvía del fondo abría sesión nueva
+  // y la vieja seguía enganchada a su consola (seis del mismo aparato en una, el 2026-10-09).
+  const goneMs = opts.sessionGoneMs ?? SESSION_GONE_MS
+  client.on('peer_disconnected', (token) => {
+    for (const [sid, s] of sessions) {
+      if (s.from !== token || s._gone) continue
+      s._gone = setTimeout(() => {
+        s.close(); sessions.delete(sid)
+        audit('session-gone', { sid: sid.slice(0, 8) })
+      }, goneMs)
+      s._gone.unref?.()
+    }
+  })
+
   // POR DÓNDE VA EL TRÁFICO, AL LOG. Un agente no tiene topbar: sin esto, saber si una
   // sesión subió a WebRTC o sigue dando la vuelta por el proxio pedía una prueba aparte.
   // Lo escribe el pilar (`logStats`), que lee los clientes del proceso; aquí solo se enciende.

package/src/discover.js

@@ -29,14 +29,22 @@ import { PING, PONG } from '../protocol.js'
  *   `'ia-agent'`). Omitir = todos los que tengan nameOf y no sean `'cli'` (un navegador
  *   enrolado queda como `cli` y no atiende a nadie).
  * @returns {Promise<Array<{sub:string,label:string,cn:string|null}>>} uno por miembro.
+ * @throws `code: 'no-acta'` si ni la bóveda ni lo guardado dan el acta: no se sabe, y no es «ninguno».
  */
 export async function listAgentsByLabel (id, label) {
   // Por su efecto: trae el acta vigente y la adopta. La lista de aparatos que devuelva —si
   // devuelve alguna— no se mira: quién es de la cuenta lo dice el acta.
   let acta = null
-  try { acta = (await id.listVaultDevices())?.acta || null } catch (_) {}
-  if (!acta) acta = (await id.profileActa?.().catch(() => null))?.acta || null
-  if (!acta) return []
+  let why = null
+  try { acta = (await id.listVaultDevices())?.acta || null } catch (e) { why = e }
+  // La bóveda no contestó: vale el acta que ya se tiene guardada.
+  if (!acta) acta = (await id.profileActa?.().catch((e) => { why = why || e; return null }))?.acta || null
+  // SIN ACTA NO SE SABE quién es de la cuenta, y eso no es «no hay nadie». Devolver una lista
+  // vacía aquí hacía que una máquina con la bóveda inalcanzable dijera «no tienes otras
+  // máquinas» con toda tranquilidad. Se para y se dice.
+  if (!acta) {
+    throw Object.assign(new Error(`could not get the account record (acta) from the vault${why ? `: ${why.message}` : ''}`), { code: 'no-acta', cause: why || undefined })
+  }
 
   const mine = id.me?.publickey
   const nameOf = (m) => m.cn || m.label || null