@dotrino/remote-agent 0.9.0 → 0.9.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.9.0",
+  "version": "0.9.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",
@@ -46,12 +46,12 @@
   ],
   "peerDependencies": {
     "@dotrino/identity": ">=0.73.1",
-    "@dotrino/proxy-client": ">=0.10.1",
+    "@dotrino/proxy-client": ">=0.18.2",
     "@dotrino/vault": ">=0.30.0"
   },
   "devDependencies": {
     "@dotrino/identity": "0.73.0",
-    "@dotrino/proxy-client": "0.13.1",
+    "@dotrino/proxy-client": "0.18.2",
     "@dotrino/vault": "0.35.0"
   }
 }

package/src/agent.js

@@ -111,9 +111,8 @@ export async function startRemoteAgent (opts = {}) {
   // Identificarse bajo la pubkey de ESTA máquina (firmado con su D).
   const identify = async () => {
     if (!client.token) return
-    const data = { op: 'identify', publickey: myPub, token: client.token, ts: Date.now() }
-    const { signature } = await signWithDevice({ privateJwk: link.device.privateJwk, data })
-    await client.identify({ data, signature })
+    // El sobre lo arma el pilar (`identifyAs`), que le pone el destinatario.
+    await client.identifyAs({ publickey: myPub, sign: (d) => signWithDevice({ privateJwk: link.device.privateJwk, data: d }) })
   }
   await identify()
   client.on('token', () => { identify().catch(() => {}) })
@@ -259,21 +258,21 @@ export async function startRemoteAgent (opts = {}) {
     const chk = await verifyChain({ data, signature, cert, expectedScope: SIGN_SCOPE, ...contextoActa(), revoked: revokedSet })
     if (!chk.ok) return send(from, { type: ERROR, error: 'no autorizado: ' + chk.reason })
     // CERT ∩ ACTA. `verifyChain` comprueba QUIÉN EMITIÓ el papel; que el APARATO siga siendo
-    // miembro y pueda firmar lo dice el acta, y hay que preguntárselo aparte. Sin esto, un
+    // isMember y pueda firmar lo dice el acta, y hay que preguntárselo aparte. Sin esto, un
     // papel bien firmado por una selladora abría sesión aunque el acta no nombrara a ese
     // aparato — o ya no lo nombrara. Bajo el modelo viejo daba igual, porque el papel ERA la
     // autoridad y caducaba; con el papel atado al acta, el acta tiene que decidir.
     //
     // SE PREGUNTA POR EL PERMISO, NO POR «FIRMA POR LA PERSONA». Aquí se usaba
-    // `memberCanSign`, que a un miembro CON `cn` le dice que no salvo que se le nombre su
+    // `memberCanSign`, que a un isMember CON `cn` le dice que no salvo que se le nombre su
     // cajón — y con razón: un servicio habla por la bóveda, no por su dueño. Pero eso es la
     // pregunta de OTRO mostrador. Lo que pasa aquí es un saludo, y lo que prueba es «soy
     // este aparato», no «hablo por ti»: exigirle lo segundo dejaba a TODO servicio sin poder
     // abrir sesión con otro agente. El bot social se pasó así el 1 de septiembre, con un
     // node de contenido encendido a su lado que le contestaba «ese aparato no firma por este
     // perfil». Se le pide lo que corresponde: que el acta lo nombre y le reconozca `sign`.
-    const miembro = (link.acta?.members || []).some((m) => m?.pub === chk.device)
-    if (!miembro || !memberCan(link.acta, chk.device, 'sign')) {
+    const isMember = (link.acta?.members || []).some((m) => m?.pub === chk.device)
+    if (!isMember || !memberCan(link.acta, chk.device, 'sign')) {
       return send(from, { type: ERROR, error: 'no autorizado: acta — este perfil no reconoce a ese aparato' })
     }
     if (data.op !== HS || typeof data.eph !== 'string') return send(from, { type: ERROR, error: 'handshake inválido' })

package/src/client.js

@@ -21,6 +21,7 @@
  * `on('message', cb)` recibe los payloads del agente, ya descifrados.
  */
 import { verifyChain } from '@dotrino/identity/capabilities'
+import { sealersOf } from '@dotrino/identity/acta'
 import { makeEphemeral, deriveKey, seal, open } from '../e2e.js'
 import { HS, ACK, DATA, PING, PONG, ERROR } from '../protocol.js'
 
@@ -52,6 +53,23 @@ export class RemoteAgentClient {
 
   _emit (ev, ...args) { for (const h of (this._h[ev] || [])) { try { h(...args) } catch (_) {} } }
 
+  /**
+   * El acta con la que se juzga el papel del agente. Se pide UNA vez por cliente y se
+   * guarda: es la política vigente del perfil, no cambia a mitad de una sesión.
+   *
+   * Si no se puede traer, se devuelve `null` a propósito y `verifyChain` corta con
+   * `no-acta`. No hay repliegue: sin acta no hay con qué decidir, y no decidir es que no.
+   */
+  async _acta () {
+    if (this._actaCache !== undefined) return this._actaCache
+    let acta = this.link.acta || null
+    if (!acta) {
+      try { acta = (await this.link.id?.listVaultDevices?.())?.acta || null } catch (_) { acta = null }
+    }
+    this._actaCache = acta
+    return acta
+  }
+
   async _identify () {
     // En modo self NO identificamos esta conexión como P: el proxy enruta por token
     // las respuestas del agente (no por pubkey), y así no recibimos el fan-out de
@@ -64,9 +82,8 @@ export class RemoteAgentClient {
     const { id, cert } = this.link
     const publickey = id.me?.publickey
     if (!publickey) return
-    const data = { op: 'identify', publickey, token: this.client.token, ts: Date.now() }
-    const { signature } = await id.signData(data)
-    await this.client.identify({ data, signature, cert })
+    // El sobre lo arma el pilar (`identifyAs`), que le pone el destinatario.
+    await this.client.identifyAs({ publickey, sign: (d) => id.signData(d), cert })
   }
 
   async connect () {
@@ -115,7 +132,19 @@ export class RemoteAgentClient {
 
     // El ack debe: (1) encadenar a NUESTRA maestra, (2) estar firmado por el agente
     // que apuntamos, (3) atar nuestra pub efímera y el sid.
-    const chk = await verifyChain({ data: res.ack, signature: res.signature, cert: res.cert, trustedIssuer: this.link.iss })
+    //
+    // CON EL ACTA, o no se juzga. Desde que el papel no vence, `verifyChain` necesita el
+    // `seq` del acta y su lista de SELLADORAS: sin eso no puede decir si quien firmó el
+    // papel del agente podía hacerlo, y contesta `no-acta` — que es lo correcto, porque
+    // decir «vale» sin haber comprobado nada es como entraba un papel viejo. Aquí no se le
+    // pasaba: el lado del agente sí lo hacía (`contextoActa`) y este lado se quedó atrás,
+    // así que TODA sesión moría con «el agente no está certificado por tu vault: no-acta».
+    // `clientLink` no trae el acta, así que se le pide a la bóveda una vez y se guarda.
+    const acta = await this._acta()
+    const chk = await verifyChain({
+      data: res.ack, signature: res.signature, cert: res.cert, trustedIssuer: this.link.iss,
+      actaSeq: acta?.seq ?? null, sealers: acta ? sealersOf(acta) : null
+    })
     if (!chk.ok) throw new Error('el agente no está certificado por tu vault: ' + chk.reason)
     if (res.ack.machine !== this.agentPubkey) throw new Error('el ack vino de otro agente')
     if (res.ack.ceph !== eph.pub || res.ack.sid !== res.sid) throw new Error('ack no corresponde a este handshake')

package/src/discover.js

@@ -24,7 +24,7 @@
 /**
  * @param {object} id   instancia de Identity (del vault) ya conectada.
  * @param {string} [label]  filtrar por servicio/label exacto (p. ej. `'content'`,
- *   `'ia-agent'`). Omitir = todos los que tengan nombre y no sean `'cli'` (un navegador
+ *   `'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.
  */
@@ -37,10 +37,10 @@ export async function listAgentsByLabel (id, label) {
   if (!acta) return []
 
   const mine = id.me?.publickey
-  const nombre = (m) => m.cn || m.label || null
+  const nameOf = (m) => m.cn || m.label || null
   return (acta.members || [])
     .filter((m) => m?.pub && m.pub !== mine)
-    .filter((m) => (label ? nombre(m) === label : (nombre(m) && nombre(m) !== 'cli')))
+    .filter((m) => (label ? nameOf(m) === label : (nameOf(m) && nameOf(m) !== 'cli')))
     .map((m) => ({ sub: m.pub, label: m.label || m.cn || '', cn: m.cn || null }))
 }