@dotrino/remote-agent 0.18.1 → 0.18.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.18.1",
+  "version": "0.18.3",
   "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/src/agent.js

@@ -31,7 +31,7 @@ import { HS, ACK, DATA, PING, PONG, ERROR, VMSG, SIGN_SCOPE, SESSION_TTL_MS, SES
 import { loadLink, saveLink, dataDir } from './link.js'
 
 /** Sesión cifrada con un dispositivo cliente. La app recibe esto en `onSession`. */
-class AgentSession {
+export class AgentSession {
   constructor ({ sid, key, from, device, send, isDirect = null }) {
     this.sid = sid
     this.key = key
@@ -44,10 +44,16 @@ class AgentSession {
   }
 
   /** Envía un payload de dominio cifrado al cliente. */
-  async send (payload) {
-    if (this._closed) return
-    const env = await seal(this.key, payload).catch(() => null)
-    if (env) this._send(this.from, { type: DATA, sid: this.sid, env })
+  send (payload) {
+    if (this._closed) return Promise.resolve()
+    // SALEN EN EL ORDEN EN QUE SE PIDIERON. Cifrar tarda según el tamaño: sin esta cola, un
+    // mensaje pequeño adelantaba a uno grande pedido antes (en la terminal, lo que la consola
+    // pintaba llegaba antes que su pantalla inicial, y esta lo tapaba después).
+    const sealed = seal(this.key, payload).catch(() => null)
+    this._out = (this._out || Promise.resolve()).then(() => sealed).then((env) => {
+      if (env && !this._closed) this._send(this.from, { type: DATA, sid: this.sid, env })
+    })
+    return this._out
   }
 
   /**
@@ -79,10 +85,13 @@ class AgentSession {
 
   on (ev, cb) { this._h[ev]?.push(cb); return this }
 
-  async _ingest (env) {
-    let msg
-    try { msg = await open(this.key, env) } catch { return }
-    for (const h of this._h.message) { try { h(msg) } catch (_) {} }
+  /** Lo que llega se entrega en el orden en que llegó, tarde lo que tarde en descifrarse cada uno. */
+  _ingest (env) {
+    const opened = open(this.key, env).then((msg) => ({ msg }), () => null)
+    this._in = (this._in || Promise.resolve()).then(() => opened).then((r) => {
+      if (r) for (const h of this._h.message) { try { h(r.msg) } catch (_) {} }
+    })
+    return this._in
   }
 
   close () {

package/src/client.js

@@ -115,10 +115,13 @@ export class RemoteAgentClient {
       if (token === this.agentToken) this.agentToken = null
     })
 
-    this.client.on('message', async (_from, p) => {
+    this.client.on('message', (_from, p) => {
       if (!p || typeof p !== 'object') return
       if (p.type === DATA && p.sid === this.sid) {
-        try { const m = await open(this.key, p.env); this._emit('message', m) } catch {}
+        // Se entregan en el orden en que llegaron: descifrar tarda según el tamaño, y sin
+        // esta cola un mensaje pequeño adelantaba a uno grande que llegó antes.
+        const opened = open(this.key, p.env).then((m) => ({ m }), () => null)
+        this._in = (this._in || Promise.resolve()).then(() => opened).then((r) => { if (r) this._emit('message', r.m) })
       } else if (p.type === ERROR) {
         this._onAgentError(p)
       }
@@ -214,12 +217,18 @@ export class RemoteAgentClient {
   async send (payload) {
     if (this._resuming) await this._resuming
     if (!this.key) throw Object.assign(new Error('no session with the agent'), { code: 'no-session' })
-    const env = await seal(this.key, payload)
-    const msg = { type: DATA, sid: this.sid, env }
-    // Por token en cuanto se sabe: sube al canal directo, y si el token murió el pilar lo
-    // saca por la pubkey. Por pubkey solo mientras no hay token.
-    if (this.agentToken) this.client.sendToOrQueue(this.agentToken, msg, { peerPubkey: this.agentPubkey })
-    else this.client.sendByPubkey(this.agentPubkey, msg)
+    // Salen en el orden en que se pidieron, tarde lo que tarde en cifrarse cada uno.
+    const sid = this.sid
+    const sealed = seal(this.key, payload)
+    const turn = (this._out || Promise.resolve()).then(() => sealed).then((env) => {
+      const msg = { type: DATA, sid, env }
+      // Por token en cuanto se sabe: sube al canal directo, y si el token murió el pilar lo
+      // saca por la pubkey. Por pubkey solo mientras no hay token.
+      if (this.agentToken) this.client.sendToOrQueue(this.agentToken, msg, { peerPubkey: this.agentPubkey })
+      else this.client.sendByPubkey(this.agentPubkey, msg)
+    })
+    this._out = turn.catch(() => {})
+    return turn
   }
 
   /** Sonda de presencia (liveness) sin abrir sesión. Devuelve true si responde. */

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