@dotrino/vault 0.28.0 → 0.30.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vault",
-  "version": "0.28.0",
+  "version": "0.30.0",
   "description": "Usa ESTE dispositivo (navegador) como bóveda/CA del ecosistema Dotrino: atiende enrolamientos por el proxy y firma certificados de delegación a tus máquinas. Incluye el cliente de SERVICIO (Node): un proyecto se enrola una vez y jala sus credenciales del vault en vez del .env (`import '@dotrino/vault/config'`).",
   "type": "module",
   "main": "src/index.js",
@@ -24,6 +24,9 @@
     "./sealed": {
       "import": "./src/sealed.js"
     },
+    "./invite": {
+      "import": "./src/invite.js"
+    },
     "./protocol": {
       "import": "./src/protocol.js"
     }

package/src/admin.js

@@ -41,12 +41,11 @@ export const ADMIN_OPS = Object.freeze([
   // no debe poder dejar sin configuración a los servicios.
   // `var.setMany` es la MISMA operación con varias variables dentro de un solo sobre: no
   // añade permisos, quita reinicios (ver el enrutado abajo).
-  'vars', 'var.set', 'var.setMany',
-  // VER un valor y sus versiones anteriores. Entran aquí —y no en la lista de lo que la
-  // bóveda hace por su cuenta— porque la bóveda NO abre nada: entrega el sobre y la
-  // envoltura dirigida a ESE aparato, que lo abre con su propia llave (§8.2). Sin esto,
-  // administrar a distancia obligaba a teclear la contraseña del perfil en un navegador.
-  'var.reveal', 'var.history'
+  // Ver un valor, su histórico o volver a una versión NO están, y es a propósito: un
+  // aparato que administra no tiene sobres de lo privado (solo el servicio dueño y la
+  // recuperación). Eso lo hace la bóveda en su máquina (`secret show/history/revert`).
+  // No lo cablees.
+  'vars', 'var.set', 'var.setMany'
 ])
 
 /** Cuánto se recuerda un nonce ya usado (el doble de la ventana de frescura). */
@@ -217,34 +216,6 @@ export function createAdminDesk ({
         return { ok: true, result: result || { ok: true } }
       }
 
-      // VER un valor (y sus versiones anteriores). La bóveda entrega material sellado a
-      // ESTE aparato; abrirlo es cosa suya. Se AUDITA, porque leer un secreto a distancia
-      // es exactamente lo que hay que poder revisar después.
-      if (data.op === 'var.reveal' || data.op === 'var.history') {
-        if (!vars) return { ok: false, error: 'admin: this vault does not serve environment variables' }
-        if (typeof vars.reveal !== 'function') return { ok: false, error: 'admin: this vault cannot show values (update it)' }
-        const toScope = typeof data.ns === 'string' && !!data.ns
-        const toDevice = typeof data.pub === 'string' && !!data.pub
-        if (toScope === toDevice) return { ok: false, error: 'admin: it needs exactly one target (ns or pub)' }
-        const target = { ns: toScope ? data.ns : null, pub: toDevice ? data.pub : null }
-
-        if (data.op === 'var.history') {
-          const result = await vars.history({ ...target, key: typeof data.key === 'string' ? data.key : null })
-          audit('admin.var.history', { by, ns: target.ns, key: data.key || null })
-          return { ok: true, result }
-        }
-        if (typeof data.key !== 'string' || !data.key) return { ok: false, error: 'admin: var.reveal needs a key' }
-        const result = await vars.reveal({
-          ...target,
-          key: data.key,
-          ts: Number.isFinite(data.ts) ? data.ts : null,
-          device: chk.device,
-          by
-        })
-        audit('admin.var.reveal', { by, ns: target.ns, key: data.key, ...(data.ts ? { ts: data.ts } : {}) })
-        return { ok: true, result }
-      }
-
       // QUITAR UN DISPOSITIVO se hace por `sub` (su llave): sale del acta Y se le retiran
       // todos los certificados. Las dos cosas o ninguna.
       //

package/src/service.js

@@ -224,47 +224,37 @@ function writeServiceIdentity (dir, obj) {
  *   Se llama ANTES de enrolar si ya había una identidad: la que va a descartarse.
  * @returns {Promise<{device, cert, iss:string, replaced:object|null}>}
  */
-export async function enrollService ({ qr, ns, dir, label, onCode, onReplace, approveTimeoutMs = 180000 } = {}) {
+/**
+ * EL ENROLAMIENTO, a secas: con una invitación de la bóveda (en cualquiera de sus
+ * formas) crea las DOS llaves del aparato —firma y cifrado—, pide el cert y lo
+ * verifica. **No persiste nada**: devuelve la identidad y quien llama decide dónde
+ * vive (`enrollService` la guarda como servicio; `@dotrino/remote-agent` como
+ * `link.json`). Es el único sitio del ecosistema donde se enrola un agente headless:
+ * si falta algo al enrolar, se añade aquí y lo heredan todos.
+ *
+ * @param {Object} opts
+ * @param {object|string} opts.qr   La invitación: objeto, URL del QR o código pegado.
+ * @param {string} [opts.label]
+ * @param {string|null} [opts.expectedScope]  Scope que el cert DEBE traer (null = no se exige).
+ * @param {(c:{deviceId:string, code:string})=>void} [opts.onCode]
+ * @param {number} [opts.approveTimeoutMs]
+ * @returns {Promise<{device, enc:{publickey:string, privateJwk:object}, cert, iss:string, proxy:string}>}
+ */
+export async function enrollWithVault ({ qr, label = 'agent', expectedScope = null, onCode, approveTimeoutMs = 180000 } = {}) {
   // `parseInvite` y NO `JSON.parse`: el vault no imprime JSON desde hace rato.
-  // `dotrino-vault pair --service` emite la URL del QR y el código compacto
-  // (`c…`/`t…`, ver invite.js), así que un `JSON.parse` fallaba SIEMPRE con
-  // «qr inválido: no es JSON» y el enrolamiento de un servicio era imposible por
-  // este camino. Lo tapaba que el único servicio enrolado del ecosistema lo hizo
-  // cuando el formato todavía era JSON. `parseInvite` acepta todas las formas,
-  // incluida la vieja, así que esto entiende cualquier invitación.
+  // `dotrino-vault pair` emite la URL del QR y el código compacto (`c…`/`t…`, ver
+  // invite.js); `parseInvite` acepta todas las formas, incluida la vieja.
   if (typeof qr === 'string') {
     const o = parseInvite(qr)
-    if (!o) throw new Error('that does not look like a vault invitation (paste the output of `dotrino-vault pair --service <ns>`)')
+    if (!o) throw new Error('that does not look like a vault invitation (paste the output of `dotrino-vault pair`)')
     qr = o
   }
   if (!qr?.sn || !(qr.iss || qr.conn)) throw new Error('invalid qr: missing the vault or the nonce')
   rejectAdoption(qr.m)
-  if (!isValidSecretsNs(ns)) throw new Error('invalid ns (use [a-z0-9-]{1,32}, e.g. "proxy")')
-  if (!dir) throw new Error('dir required (where to persist the service identity)')
-  label = label || 'service:' + ns
-
-  // La identidad que va a quedar descartada. Se avisa antes de tocar nada: para
-  // el proxy, por ejemplo, esta llave es además su identidad de red, así que
-  // reemplazarla le cambia el id de nodo y sus peers dejan de reconocerlo hasta
-  // que se re-pineen a mano.
-  const previous = readServiceIdentity(dir)
-  let replaced = null
-  if (previous?.device?.publickey) {
-    replaced = {
-      ns: previous.ns,
-      enrolledAt: previous.enrolledAt,
-      deviceId: (await pubkeyId(previous.device.publickey)).slice(0, 8).toUpperCase()
-    }
-    try { onReplace?.(replaced) } catch (_) {}
-  }
 
   const client = await freshClient(qr.proxy || 'wss://proxy.dotrino.com')
   // QR CORTO: se le pregunta a la bóveda quién es, punto a punto, presentando el `sn`.
   if (!qr.iss) {
-    // `qr.conn` es una CITA (código de 6 caracteres, un solo uso): hay que
-    // canjearla para saber a qué conexión apunta. El canje lo resuelve el proxio
-    // que la emitió —lo dice el prefijo del propio código—, así que funciona
-    // aunque la bóveda esté en otro proxio de la malla.
     const target = await resolveAppointment(client, qr.conn)
     const hello = await new Promise((resolve, reject) => {
       const off = client.on('message', (_f, p) => {
@@ -279,22 +269,14 @@ export async function enrollService ({ qr, ns, dir, label, onCode, onReplace, ap
   }
   try {
     const device = await makeDeviceKey({ label })
-    // La llave de CIFRADO, hermana de la de firma. Sin ella el vault no puede
-    // sellarle sus variables a ESTE aparato, y arrancaría sin configuración sin
-    // que nadie sepa por qué. Su pública viaja en el enrolamiento y acaba en el
-    // acta como `encPub`; la privada no sale de este disco.
+    // La llave de CIFRADO: es a la que la bóveda sella cada variable. Sin ella el
+    // aparato entra al acta pero no le llega ningún secreto, y no da error.
     const enc = await makeDeviceEncKey()
     const deviceId = (await pubkeyId(device.publickey)).slice(0, 8).toUpperCase().replace(/(.{4})(.{4})/, '$1-$2')
-    // Código ALEATORIO generado AQUÍ: el vault no lo conoce; solo puede echarlo
-    // de vuelta si el dueño lo tipeó (= tiene esta pantalla a la vista).
+    // Código de emparejamiento ALEATORIO: se muestra y NO se envía. La bóveda lo
+    // aprende solo cuando un humano lo tipea → aprobar exige TENER esta máquina.
     const code = makePairingCode()
-    // El COMPROMISO del código (nunca el código): la bóveda lo recompone con lo que
-    // tipeas y solo entonces firma el cert → aprobar exige haber leído esta pantalla.
     const commit = await commitCode({ code, dpub: device.publickey, sn: qr.sn })
-    // `intent: 'join'` EXPLÍCITO. La bóveda lo compara con el modo que abrió y,
-    // si falta, asume `join` — pero un agente no debe apoyarse en un default
-    // para algo que decide de quién es la cuenta. Yendo dentro de `data`, viaja
-    // firmado: nadie en el medio puede convertirlo en una adopción.
     const data = { op: 'enroll', intent: 'join', dpub: device.publickey, encPub: enc.encPublickey, token: qr.token || qr.sn, sn: qr.sn, commit, label, ts: Date.now() }
     const { signature } = await signWithDevice({ privateJwk: device.privateJwk, data })
 
@@ -302,7 +284,7 @@ export async function enrollService ({ qr, ns, dir, label, onCode, onReplace, ap
       const off = client.on('message', (_from, p) => {
         if (!p || typeof p !== 'object') return
         if (p.type === MSG.ENROLL_CHALLENGE) {
-          const show = onCode || (({ deviceId, code }) => console.log(`[vault-service] device ${deviceId} · approve it on the vault:  dotrino-vault approve ${code}`))
+          const show = onCode || (({ deviceId, code }) => console.log(`[vault] device ${deviceId} · approve it on the vault:  dotrino-vault approve ${code}`))
           show({ deviceId, code })
         } else if (p.type === MSG.ENROLLED) { cleanup(); resolve(p) } else if (p.type === MSG.ERROR) { cleanup(); reject(new Error(p.error)) }
       })
@@ -312,35 +294,71 @@ export async function enrollService ({ qr, ns, dir, label, onCode, onReplace, ap
     client.sendByPubkey(qr.iss, { type: MSG.ENROLL, data, signature })
     const res = await enrolled
 
-    // Validación estricta (igual que un dispositivo): cert de la maestra VISTA,
-    // para ESTA llave, y el código echado debe ser el nuestro (anti vault falso).
     if (res.code !== code) throw new Error('the vault echoed a code other than the one shown (possible malicious relay)')
-    const v = await verifyDelegation({ cert: res.cert, expectedSub: device.publickey, expectedScope: secretsScope(ns) })
+    const v = await verifyDelegation({ cert: res.cert, expectedSub: device.publickey, ...(expectedScope ? { expectedScope } : {}) })
     if (!v.ok) throw new Error('invalid cert: ' + v.reason)
     if (res.cert.iss !== qr.iss) throw new Error('cert signed by a master other than the one in the QR')
 
-    // Reemplazo, no acumulación: el archivo se sobrescribe entero y la identidad
-    // anterior deja de existir en este agente.
-    // v2: suma `enc`. El `device` (la llave de FIRMA) no se toca — de él sale el
-    // `nodeId` del proxio, y cambiarlo le costaría su cajón de variables.
-    writeServiceIdentity(dir, { v: 2, ns, iss: qr.iss, proxy: qr.proxy, device, enc: { publickey: enc.encPublickey, privateJwk: enc.encPrivateJwk }, cert: res.cert, enrolledAt: Date.now() })
-    return { device, enc, cert: res.cert, iss: qr.iss, replaced }
+    return { device, enc: { publickey: enc.encPublickey, privateJwk: enc.encPrivateJwk }, cert: res.cert, iss: qr.iss, proxy: qr.proxy || 'wss://proxy.dotrino.com' }
   } finally { client.close() }
 }
 
 /**
- * Se asegura de que este servicio TENGA llave de cifrado, generándola si le falta.
+ * Enrola ESTE servicio contra el vault y persiste su identidad.
  *
- * Existe para los agentes que se enrolaron ANTES de que la llave existiera: su
- * `service-identity.json` es v1 y solo lleva la de firma. Sin esto, la única forma de
- * conseguirla sería re-enrolarse — y re-enrolar le cambia la pubkey, con lo que pierde
- * su cajón de variables (va indexado por ella) y arrancaría sin configuración, en
- * silencio. Ese es justo el accidente que esta función existe para evitar.
+ * UN AGENTE TIENE UNA SOLA IDENTIDAD, Y SE LA DA EL VAULT. A diferencia de un
+ * aparato —que puede llevar varios perfiles y hasta meter su cuenta al vault por
+ * adopción—, un agente no acumula identidades ni transfiere la suya: se enrola,
+ * el vault le cede una (llave propia + cert de la maestra) y **la anterior, si
+ * había, se descarta**. No hay fusión ni convivencia, y no hace falta: un agente
+ * es un servicio, no una persona; no tiene por qué "ser varios".
  *
- * La llave de FIRMA no se toca: de ella sale el `nodeId` del proxio.
+ * Enrolar dos veces, entonces, no es un error a bloquear sino un REEMPLAZO — que
+ * es además la forma de rotar la identidad de un agente comprometido. Lo que sí
+ * hace falta es que se vea: se avisa por `onReplace` qué identidad se tira.
  *
- * @returns {Promise<{ encPub: string, created: boolean }>}
+ * En el vault se corre antes `dotrino-vault pair --service <ns>`; la invitación
+ * que imprime ese comando es el `qr` de aquí (en cualquiera de sus formas).
+ * Muestra un código por `onCode`: el dueño lo tipea en el vault
+ * (`dotrino-vault approve <código>`).
+ *
+ * @param {Object} opts
+ * @param {object|string} opts.qr  La invitación: objeto, URL del QR o código pegado.
+ * @param {string} opts.ns     Namespace de secretos del servicio (el mismo del pair).
+ * @param {string} opts.dir    Dónde persistir `service-identity.json`.
+ * @param {string} [opts.label]
+ * @param {(c:{deviceId:string, code:string})=>void} [opts.onCode]
+ * @param {(prev:{ns:string, enrolledAt:number, deviceId:string})=>void} [opts.onReplace]
+ *   Se llama ANTES de enrolar si ya había una identidad: la que va a descartarse.
+ * @returns {Promise<{device, cert, iss:string, replaced:object|null}>}
  */
+export async function enrollService ({ qr, ns, dir, label, onCode, onReplace, approveTimeoutMs = 180000 } = {}) {
+  if (!isValidSecretsNs(ns)) throw new Error('invalid ns (use [a-z0-9-]{1,32}, e.g. "proxy")')
+  if (!dir) throw new Error('dir required (where to persist the service identity)')
+  label = label || 'service:' + ns
+
+  // La identidad que va a quedar descartada. Se avisa antes de tocar nada: para
+  // el proxy, por ejemplo, esta llave es además su identidad de red, así que
+  // reemplazarla le cambia el id de nodo y sus peers dejan de reconocerlo hasta
+  // que se re-pineen a mano.
+  const previous = readServiceIdentity(dir)
+  let replaced = null
+  if (previous?.device?.publickey) {
+    replaced = {
+      ns: previous.ns,
+      enrolledAt: previous.enrolledAt,
+      deviceId: (await pubkeyId(previous.device.publickey)).slice(0, 8).toUpperCase()
+    }
+    try { onReplace?.(replaced) } catch (_) {}
+  }
+
+  const { device, enc, cert, iss, proxy } = await enrollWithVault({ qr, label, expectedScope: secretsScope(ns), onCode, approveTimeoutMs })
+  // v2: suma `enc`. El `device` (la llave de FIRMA) no se toca — de él sale el
+  // id de nodo del proxio y la fila del acta.
+  writeServiceIdentity(dir, { v: 2, ns, iss, proxy, device, enc, cert, enrolledAt: Date.now() })
+  return { device, enc, cert, iss, replaced }
+}
+
 export async function ensureEncKey ({ dir } = {}) {
   const saved = readServiceIdentity(dir)
   if (!saved) throw new Error('service not enrolled: run enrollService() first')
@@ -389,9 +407,13 @@ export async function registerEncKey ({ dir, ns, proxyUrl, masterPubkey, device,
  * Renueva el cert automáticamente si está por vencer (best-effort).
  * @returns {Promise<Record<string,string>>}  secretos KEY→valor
  */
-export async function fetchSecrets ({ dir, ns, proxyUrl, masterPubkey, device, cert, timeoutMs = 30000 } = {}) {
+export async function fetchSecrets ({ dir, ns, proxyUrl, masterPubkey, device, cert, enc, timeoutMs = 30000 } = {}) {
   let saved = null
   if (dir) saved = readServiceIdentity(dir)
+  // Sin `dir`: la identidad viene entera por parámetros — es el caso de un agente
+  // enrolado por `@dotrino/remote-agent` (su `link.json` trae `device`, `cert` y `enc`).
+  // Un mismo enrolamiento sirve para el plano de control y para los secretos.
+  if (!saved && enc) saved = { ns, iss: masterPubkey, proxy: proxyUrl, device, cert, enc }
   ns = ns || saved?.ns
   proxyUrl = proxyUrl || saved?.proxy
   masterPubkey = masterPubkey || saved?.iss
@@ -860,11 +882,11 @@ function fingerprintOf (secrets) {
  * servicio no opera hasta que esto resuelva — esa es la regla.
  * @returns {Promise<Record<string,string>>}
  */
-export async function waitForSecrets ({ dir, ns, proxyUrl, masterPubkey, device, cert, retryMs = 5000, maxRetryMs = 60000, onRetry } = {}) {
+export async function waitForSecrets ({ dir, ns, proxyUrl, masterPubkey, device, cert, enc, retryMs = 5000, maxRetryMs = 60000, onRetry } = {}) {
   let delay = retryMs
   for (;;) {
     try {
-      return await fetchSecrets({ dir, ns, proxyUrl, masterPubkey, device, cert })
+      return await fetchSecrets({ dir, ns, proxyUrl, masterPubkey, device, cert, enc })
     } catch (e) {
       // Lo NO transitorio no se arregla reintentando: falta de enrolamiento,
       // cert revocado/vencido o scope equivocado exigen re-emparejar → se corta.