@dotrino/remote-agent 0.6.0 → 0.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.6.0",
+  "version": "0.6.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",
@@ -45,7 +45,7 @@
     "p2p"
   ],
   "peerDependencies": {
-    "@dotrino/identity": ">=0.73.0",
+    "@dotrino/identity": ">=0.73.1",
     "@dotrino/proxy-client": ">=0.10.1",
     "@dotrino/vault": ">=0.30.0"
   },

package/src/agent.js

@@ -22,7 +22,7 @@
 import fs from 'node:fs'
 import path from 'node:path'
 import { verifyChain, signWithDevice, verifyDeviceSig, verifyDelegation, pubkeyId } from '@dotrino/identity/capabilities'
-import { sealersOf } from '@dotrino/identity/acta'
+import { sealersOf, memberCanSign } 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'
@@ -178,9 +178,16 @@ export async function startRemoteAgent (opts = {}) {
    * que una segunda bóveda pueda emitir; lo que se fija es el perfil.
    */
   async function renewCertIfNeeded () {
-    // ¿Nos quedamos atrás? Lo sabemos por el acta que la bóveda manda con cada respuesta.
+    // DOS MOTIVOS PARA PEDIR PAPEL NUEVO:
+    //   · el nuestro es del MODELO VIEJO (lleva `exp` y no `seq`). Ese hay que cambiarlo sí
+    //     o sí: se acepta por el repliegue de migración, pero muere en su fecha y con él la
+    //     máquina. Sin esta rama la migración no terminaba nunca — el disparador comparaba
+    //     `seq` y un papel viejo no tiene, así que nunca saltaba.
+    //   · o el acta que hemos visto va por delante del papel: el dueño cambió permisos.
+    const legado = typeof link.cert?.seq !== 'number'
     const visto = link.actaSeq
-    if (typeof visto !== 'number' || typeof link.cert?.seq !== 'number' || visto <= link.cert.seq) return
+    const atrasado = typeof visto === 'number' && typeof link.cert?.seq === 'number' && visto > link.cert.seq
+    if (!legado && !atrasado) return
     try {
       const res = await vaultRpc(VMSG.RENEW, VMSG.RENEWED, { op: 'renew', publickey: myPub, ts: Date.now() })
       const cert = res?.cert
@@ -251,6 +258,14 @@ export async function startRemoteAgent (opts = {}) {
     // qué decidir, así que no se decide que sí.
     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
+    // 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.
+    if (!memberCanSign(link.acta, chk.device)) {
+      return send(from, { type: ERROR, error: 'no autorizado: acta — ese aparato no firma por este perfil' })
+    }
     if (data.op !== HS || typeof data.eph !== 'string') return send(from, { type: ERROR, error: 'handshake inválido' })
 
     const eph = await makeEphemeral()