@dotrino/remote-agent 0.7.1 → 0.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.7.1",
+  "version": "0.9.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",

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, memberCanSign } from '@dotrino/identity/acta'
+import { sealersOf, memberCan } 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'
@@ -263,8 +263,18 @@ export async function startRemoteAgent (opts = {}) {
     // 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' })
+    //
+    // 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
+    // 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')) {
+      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/discover.js

@@ -1,33 +1,47 @@
 /**
- * discover.js — autodescubrimiento de agentes vinculados al vault (lado cliente).
+ * discover.js — autodescubrimiento de agentes de la cuenta, POR EL ACTA.
  *
- * Tus agentes = los dispositivos enrolados en TU vault (`vault.devices` trae la
- * pubkey `sub` de cada uno = su dirección en el proxy). Sin pegar nada: filtrás por
- * label y abrís una sesión con el que quieras.
+ * Quién pertenece a esta cuenta lo dice el ACTA, y solo el acta. Aquí se pregunta ahí, que
+ * es además lo único que un servicio tiene derecho a ver.
  *
- * Si pasás `label` (p. ej. `'ia-agent'`), lista solo los dispositivos con ESE label
- * exacto. Si NO pasás `label`, mantiene el comportamiento legacy de terminal: lista
- * cualquier dispositivo que tenga un label y no sea `'cli'` (los navegadores
- * enrolados quedan con label `'cli'` y no atienden agentes).
+ * **Antes se preguntaba al inventario de certificados** (`vault.devices`), y eso se rompió
+ * el 2026-08-31 por una razón correcta: un servicio pasó a recibir sus revocaciones y el
+ * acta, pero **no el inventario de aparatos del dueño** — no es asunto suyo (vaultd 0.75.1).
+ * Desde entonces cualquier servicio que buscara un agente recibía una lista vacía y
+ * concluía «no tienes ninguno». El bot social estuvo veinte horas así, reintentando cada
+ * minuto contra un node de contenido que estaba encendido y a su lado.
+ *
+ * La distinción es la que importa y no es un detalle de implementación:
+ *
+ *   · el **acta** dice QUIÉN ES DE LA CUENTA — la tiene todo miembro, y para eso existe
+ *   · el **inventario** dice qué papeles ha firmado el dueño y cuándo caducan — es suyo
+ *
+ * Para hablarle a otro agente hace falta lo primero. Se pide `listVaultDevices()` igual,
+ * pero por su efecto útil: trae el acta vigente (o la cadena) y la adopta. Lo que se lee
+ * después son sus miembros.
  */
 
 /**
  * @param {object} id   instancia de Identity (del vault) ya conectada.
- * @param {string} [label]  filtrar por label exacto (p. ej. 'ia-agent'). Omitir = todos los no-cli.
- * @returns {Promise<Array<{sub:string,label:string,deviceId:string,exp?:number}>>}
- *   dedup por pubkey (renovaciones/re-emparejes → el cert más nuevo).
+ * @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
+ *   enrolado queda como `cli` y no atiende a nadie).
+ * @returns {Promise<Array<{sub:string,label:string,cn:string|null}>>} uno por miembro.
  */
 export async function listAgentsByLabel (id, label) {
-  const { devices } = await id.listVaultDevices()
+  // 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 []
+
   const mine = id.me?.publickey
-  const now = Date.now()
-  const bySub = new Map()
-  for (const d of devices || []) {
-    if (!d.sub || d.sub === mine || (d.exp && d.exp <= now)) continue
-    if (label ? d.label !== label : (!d.label || d.label === 'cli')) continue
-    if (!bySub.has(d.sub) || (d.exp || 0) > (bySub.get(d.sub).exp || 0)) bySub.set(d.sub, d)
-  }
-  return [...bySub.values()]
+  const nombre = (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')))
+    .map((m) => ({ sub: m.pub, label: m.label || m.cn || '', cn: m.cn || null }))
 }
 
 export default { listAgentsByLabel }