@dotrino/vaultd 0.91.0 → 0.92.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.91.0",
+  "version": "0.92.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": {

package/src/manager.js

@@ -136,12 +136,37 @@ export async function startVaultManager ({ root = dataDir(), proxyUrl, log = con
    * Va aquí y no en `profiles.js` porque el registro de perfiles no sabe de bóvedas
    * corriendo: sabe de contraseñas. Quien tiene las dos cosas delante es esto.
    */
+  /**
+   * ABRIR DE VERDAD, NO SOLO QUITAR EL CANDADO.
+   *
+   * Esto decía `running.get(id)?.takeMasterKey?.()`, y con un `?.` de por medio un perfil
+   * que **no estaba corriendo** se marcaba desbloqueado sin abrir nada. Y hay un caso muy
+   * normal en el que no está corriendo: **su maestra está sellada bajo la contraseña**, así
+   * que al arrancar el servicio no hay frase con la que abrirlo y se queda fuera
+   * (`could not open profile …: vault locked`). Justo el perfil que más falta hace abrir
+   * era el único al que `unlock` no le hacía nada.
+   *
+   * Lo que se veía: `status` decía `🔓 desbloqueado` y **cada** petición contestaba
+   * `profile is not open`. La TUI se quedaba sin dispositivos, sin un error a la vista.
+   *
+   * Ahora se arranca si hace falta, y **si algo falla se vuelve a cerrar**: un candado que
+   * dice «abierto» sin estarlo es peor que uno cerrado, porque todo lo que venga detrás
+   * falla en otro sitio y por otro motivo.
+   */
   async function unlock (id, password) {
     const r = await profiles.unlock(id, password)
-    // `takeMasterKey` además la deja SELLADA la primera vez: un perfil que ya tenía
-    // contraseña guardaba la maestra bajo la llave de máquina, y se migra aquí, que es el
-    // único momento en que la llave del perfil está en la mano.
-    try { await running.get(id)?.takeMasterKey?.() } catch (e) { log('[vault] could not take the master key: %s', e.message) }
+    try {
+      // El orden importa: `profiles.unlock` ya dejó la llave del perfil disponible
+      // (`openKey`), así que ahora `startVault` sí puede abrir su maestra.
+      const v = running.get(id) || await open(id)
+      // `takeMasterKey` además la deja SELLADA la primera vez: un perfil que ya tenía
+      // contraseña guardaba la maestra bajo la llave de máquina, y se migra aquí, que es el
+      // único momento en que la llave del perfil está en la mano.
+      await v.takeMasterKey()
+    } catch (e) {
+      try { profiles.lock(id) } catch (_) {}
+      throw new Error(`unlocked the profile but could not open it, so it stays closed: ${e.message}`)
+    }
     return r
   }
 

package/src/transport.js

@@ -82,12 +82,26 @@ export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, com
     // comunicación en el acta se identifica con la maestra, como siempre. Exige el perfil
     // ABIERTO —la maestra bajo llave no firma— y por eso se dice en voz alta: mientras se
     // esté aquí, cerrar el perfil deja la bóveda sin voz.
-    const publickey = await masterPubkeyOf(identity)
-    if (!publickey) return
-    const data = { op: 'identify', publickey, token: client.token, ts: Date.now() }
-    const { signature } = await identity.signData(data)
-    log('[vault] identifying with the master key: this vault is not in its own record yet (open the profile once to fix it)')
-    await client.identify({ data, signature, acta: record })
+    //
+    // QUEDARSE SIN VOZ NO ES NO ARRANCAR, y aquí eran lo mismo. Un perfil con la maestra
+    // sellada y sin llave de comunicación todavía llegaba hasta este `signData`, que tira
+    // `vault locked`; el error subía por `createTransport` hasta `startVault` y **el perfil
+    // entero se quedaba sin abrir**. Al dueño le pasó el 2026-09-02: su perfil principal no
+    // arrancaba, `unlock` lo marcaba abierto igual y cada petición contestaba «profile is
+    // not open», con la TUI vacía y sin un error a la vista.
+    //
+    // Se dice y se sigue. Sin red se puede vivir hasta que alguien teclee la frase; sin
+    // bóveda, no.
+    try {
+      const publickey = await masterPubkeyOf(identity)
+      if (!publickey) return
+      const data = { op: 'identify', publickey, token: client.token, ts: Date.now() }
+      const { signature } = await identity.signData(data)
+      log('[vault] identifying with the master key: this vault is not in its own record yet (open the profile once to fix it)')
+      await client.identify({ data, signature, acta: record })
+    } catch (e) {
+      log(`[vault] cannot identify on the proxy yet: ${e.message} — the profile still opens; unlock it once and it gets its own communication key`)
+    }
   }
   await identify()
   // Re-identificar al reconectar (el token cambia).