@dotrino/remote-agent 0.9.1 → 0.10.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.9.1",
+  "version": "0.10.0",
   "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",
@@ -45,13 +45,13 @@
     "p2p"
   ],
   "peerDependencies": {
-    "@dotrino/identity": ">=0.73.1",
-    "@dotrino/proxy-client": ">=0.10.1",
-    "@dotrino/vault": ">=0.30.0"
+    "@dotrino/identity": ">=0.92.0",
+    "@dotrino/proxy-client": ">=0.18.2",
+    "@dotrino/vault": ">=0.64.0"
   },
   "devDependencies": {
-    "@dotrino/identity": "0.73.0",
-    "@dotrino/proxy-client": "0.13.1",
-    "@dotrino/vault": "0.35.0"
+    "@dotrino/identity": "0.92.0",
+    "@dotrino/proxy-client": "0.18.2",
+    "@dotrino/vault": "0.64.0"
   }
 }

package/src/agent.js

@@ -21,8 +21,8 @@
  */
 import fs from 'node:fs'
 import path from 'node:path'
-import { verifyChain, signWithDevice, verifyDeviceSig, verifyDelegation, pubkeyId } from '@dotrino/identity/capabilities'
-import { sealersOf, memberCan } from '@dotrino/identity/acta'
+import { verifyChain, signWithDevice, verifyDeviceSig, pubkeyId } from '@dotrino/identity/capabilities'
+import { sealersOf, memberCan, checkVaultReply } from '@dotrino/identity/acta'
 import { installNodeGlobals } from '../node-globals.js'
 import { makeEphemeral, deriveKey, seal, open } from '../e2e.js'
 import { HS, ACK, DATA, PING, PONG, ERROR, VMSG, SIGN_SCOPE, SESSION_TTL_MS, REVOKE_REFRESH_MS } from '../protocol.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(() => {}) })
@@ -192,14 +191,10 @@ export async function startRemoteAgent (opts = {}) {
       const res = await vaultRpc(VMSG.RENEW, VMSG.RENEWED, { op: 'renew', publickey: myPub, ts: Date.now() })
       const cert = res?.cert
       const acta = res?.acta
-      if (!cert || cert.sub !== myPub) throw new Error('the vault returned a cert that is not for this machine')
-      if (!acta) throw new Error('the vault did not send its record: cannot check who signed this cert')
-      if (acta.profileId !== master) throw new Error('the record is from a profile other than the pinned one')
-      const v = await verifyDelegation({
-        cert, expectedSub: myPub, expectedScope: SIGN_SCOPE,
-        actaSeq: acta.seq, sealers: sealersOf(acta)
-      })
-      if (!v.ok) throw new Error(v.reason)
+      // La regla del pilar, contra la bóveda a la que se le pidió (`master`). Aquí se exigía
+      // `acta.profileId === master`, que con una segunda bóveda nunca es verdad.
+      const chk = await checkVaultReply({ acta, cert, vault: master, sub: myPub, scope: SIGN_SCOPE })
+      if (!chk.ok) throw new Error('invalid renewed cert: ' + chk.reason)
       link.cert = cert
       link.actaSeq = acta.seq
       saveLink(dir, link)
@@ -259,21 +254,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

@@ -82,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 () {

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 }))
 }
 

package/src/link.js

@@ -16,8 +16,7 @@
 import fs from 'node:fs'
 import path from 'node:path'
 import os from 'node:os'
-import { signWithDevice, pubkeyId, verifyDelegation } from '@dotrino/identity/capabilities'
-import { sealersOf } from '@dotrino/identity/acta'
+import { signWithDevice, pubkeyId } from '@dotrino/identity/capabilities'
 import { requestDevices, requestRenew } from '@dotrino/identity/vault/remote.js'
 import { installNodeGlobals } from '../node-globals.js'
 
@@ -169,17 +168,11 @@ export async function renewLink (link, { dir = dataDir(), force = false } = {})
   if (!force && !legado && !atrasado) return { renewed: false, seq: link.cert.seq, reason: 'not due' }
   installNodeGlobals(dir)
   try {
-    const res = await requestRenew({ master: link.iss, proxy: link.proxy || 'wss://proxy.dotrino.com', device: link.device, cert: link.cert })
-    const cert = res?.cert
-    const acta = res?.acta
-    if (!cert || cert.sub !== link.device.publickey) throw new Error('the vault returned a cert that is not for this machine')
-    // El acta viaja con el papel y hace falta para juzgarlo: quien lo firmó tiene que ser
-    // SELLADORA de este perfil. Ya no se compara `cert.iss` con una llave fija — con varias
-    // bóvedas el emisor puede ser otra del mismo perfil; lo que se fija es el PERFIL.
-    if (!acta) throw new Error('the vault did not send its record: cannot check who signed this cert')
-    if (acta.profileId !== link.iss) throw new Error('the record is from a profile other than the pinned one')
-    const v = await verifyDelegation({ cert, expectedSub: link.device.publickey, actaSeq: acta.seq, sealers: sealersOf(acta) })
-    if (!v.ok) throw new Error(v.reason)
+    // `requestRenew` ya JUZGA lo que devuelve (`checkVaultReply` del pilar: acta bien
+    // firmada, esta bóveda puede sellarla, el papel lo firmó ella y es para esta llave) y
+    // lanza si no cuadra. Aquí se comparaba `acta.profileId` con `link.iss`, que solo vale
+    // en una cuenta que nació en esa bóveda: con una segunda bóveda no renovaba nunca.
+    const { cert, acta } = await requestRenew({ master: link.iss, proxy: link.proxy || 'wss://proxy.dotrino.com', device: link.device, cert: link.cert })
     // SE GUARDA ANTES DE DAR NADA POR BUENO. Emitir un papel RETIRA el anterior, así que si
     // esto no se persiste la máquina se queda usando uno revocado y fuera para siempre. Le
     // pasó al registro de selladores en la migración, y por eso está dicho aquí.