@dotrino/vaultd 0.63.0 → 0.64.0

package/lib/src/enroll.js

@@ -30,7 +30,7 @@
  * `docs/pairing-protocol.md`).
  */
 import { verifyDeviceSig, pubkeyId, commitCode } from '@dotrino/identity/capabilities'
-import { verifyContinuity } from '@dotrino/identity/acta'
+import { verifyContinuity, canSeal } from '@dotrino/identity/acta'
 
 /** Un token de emparejamiento vale 5 min. */
 export const PAIRING_TTL_MS = 5 * 60 * 1000
@@ -388,9 +388,12 @@ export function createEnrollDesk ({
     const pend = [...pending.values()].find((x) => x.state === 'AWAITING_ACTA' && (x.from === from || x.dpub))
     if (!pend) return reply(from, { type: MSG_ERROR, error: 'no adoption awaiting a record' })
     if (!record || typeof record !== 'object') return reply(from, { type: MSG_ERROR, error: 'record missing or unreadable' })
-    if (record.sealer !== iss) {
+    // Ya no hay campo `sealer`: se le pregunta al PERMISO. Es la misma comprobación —«¿me
+    // nombra a mí la que me mandan?»— dicha en el idioma nuevo, y con varios selladores la
+    // respuesta puede ser que sí para más de uno, que es lo correcto.
+    if (!canSeal(record, iss)) {
       audit('rejected', { what: 'adopt', reason: 'not-sealer' })
-      return reply(from, { type: MSG_ERROR, error: 'that record does not name this vault as the sealer' })
+      return reply(from, { type: MSG_ERROR, error: 'that record does not let this vault seal it' })
     }
     if (record.sealedBy !== pend.dpub) {
       audit('rejected', { what: 'adopt', reason: 'sealed-by-other' })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.63.0",
+  "version": "0.64.0",
   "type": "module",
   "description": "Certificador personal de Dotrino: daemon headless que custodia la clave maestra y delega capacidades a tus dispositivos por el proxy. Tu CA propia.",
   "bin": {
@@ -20,7 +20,7 @@
     "node": ">=20"
   },
   "dependencies": {
-    "@dotrino/identity": "^0.66.0",
+    "@dotrino/identity": "^0.67.0",
     "@dotrino/passmanager": "^0.9.0",
     "@dotrino/proxy-client": "^0.13.1",
     "ws": "^8.18.0"

package/src/ctl.js

@@ -548,7 +548,10 @@ async function cmdMembers () {
   for (const m of record.members) {
     const who = m.label || m.id
     const marks = [
-      m.isMaster ? `${B}Master${Z}` : null,
+      // Ya no hay UN master: la marca dice quién puede sellar, y pueden ser varios. El
+      // permiso también sale abajo en la lista, pero aquí se ve de un vistazo — que es
+      // justo lo que se busca al mirar quién es quién.
+      m.isMaster ? `${B}Sella${Z}` : null,
       m.isMe ? 'este dispositivo' : null,
       m.cn ? `servicio «${m.cn}»` : null
     ].filter(Boolean)

package/src/manager.js

@@ -24,13 +24,16 @@ import { atRestFor } from './atrest.js'
  * Pura a propósito (recibe el veredicto ya calculado) para poder probar la regla sin
  * levantar un vault entero.
  */
-export function assertCanRemove ({ isMaster, memberCount, name = '' }) {
-  if (!isMaster || memberCount <= 1) return true
+export function assertCanRemove ({ isMaster, memberCount, sealerCount = 1, name = '' }) {
+  // `isMaster` ahora es «¿puedo sellar?». Y con varios selladores, borrar esta bóveda ya
+  // NO deja la cuenta muerta si queda otro que pueda sellar: eso es exactamente para lo
+  // que existe el multivault, y el freno tiene que dejar de estorbar en ese caso.
+  if (!isMaster || memberCount <= 1 || sealerCount > 1) return true
   const others = memberCount - 1
   const e = new Error(
-    `this vault is the master of account "${name}" and it has ${others} more device(s): ` +
-    'hand the master over to one that is connected first. Deleting it like this leaves them ' +
-    'with their key and nobody able to sign the record again.'
+    `this vault is the only one that can seal account "${name}", and it has ${others} more device(s): ` +
+    'give another one the sealing permission first (`caps <ID> +sella`). Deleting it like this ' +
+    'leaves them with their key and nobody able to seal the record again.'
   )
   e.code = 'MASTER_WITH_MEMBERS'
   e.members = memberCount
@@ -159,7 +162,12 @@ export async function startVaultManager ({ root = dataDir(), proxyUrl, log = con
           v.isMaster().catch(() => false),
           v.profileMembers().catch(() => ({ members: [] }))
         ])
-        assertCanRemove({ isMaster, memberCount: (record?.members || []).length, name: profiles.get(id)?.name || id })
+        assertCanRemove({
+          isMaster,
+          memberCount: (record?.members || []).length,
+          sealerCount: (record?.sealers || []).length || 1,
+          name: profiles.get(id)?.name || id
+        })
       } else {
         log('[vault] profile %s is not open: deleting without being able to check its record', id)
       }