@dotrino/vaultd 0.150.0 → 0.151.0

package/lib/src/service.js

@@ -522,8 +522,9 @@ export async function registerEncKey ({ dir, ns, proxyUrl, masterPubkey, device,
  * privadas al pasar.
  *
  * Lo que decide la bóveda, no este cliente:
- *   · si hay quien apruebe, se pide aprobación siempre, y al aprobarse puede EDITAR;
- *   · si nadie puede aprobar, solo AÑADE lo que falta (una que ya exista falla con `exists`);
+ *   · AÑADIR una variable nueva es directo: se guarda en el acto;
+ *   · EDITAR una que ya existe pide aprobación (la tanda entera queda en espera);
+ *   · si nadie puede aprobar, una que ya exista falla con `exists`;
  *   · BORRAR no existe por aquí.
  *
  * @param {object} o

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.150.0",
+  "version": "0.151.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/transport.js

@@ -36,7 +36,7 @@ export async function masterPubkeyOf (identity) {
  * @param {string} [opts.url] URL del proxy (default wss://proxy.dotrino.com).
  * @returns {Promise<{ client, token:string, identify():Promise<void> }>}
  */
-export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, commKey = null, log = () => {}, makeClient = null, announce = null }) {
+export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, commKey = null, log = () => {}, makeClient = null, announce = null, recheckMs = 60_000, reannounceMs = 10 * 60_000 }) {
   installNodeGlobals(dir)
   // Import dinámico DESPUÉS de instalar los globals que el paquete usa.
   // `WebSocketProxyClient` (la clase) y NO el helper `getWebSocketProxyClient`:
@@ -174,8 +174,10 @@ export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, com
    * existe». Si el anuncio falla, la bóveda sigue sirviendo a todo lo demás y se dice en voz
    * alta: lo que se pierde es que la encuentren por su dirección, no que deje de funcionar.
    */
+  let announcedAt = 0
   const announceAccount = async () => {
     if (!announce || !identificado) return
+    announcedAt = Date.now()
     try { await client.publish(announce) } catch (e) {
       log(`[vault] could not announce this account on the proxy (${e.message}) — a borrowed machine will not find this vault by its address`)
     }
@@ -219,8 +221,9 @@ export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, com
    * `connect()` es seguro de repetir: si ya está conectado devuelve el token y no abre
    * nada.
    */
-  const RECHECK_MS = 60_000
+  const RECHECK_MS = recheckMs
   const PATIENCE_MS = 5 * 60_000
+  const REANNOUNCE_MS = reannounceMs   // la mitad de lo que el proxio guarda una entrada
   let downSince = 0
   let saidDown = false
   client.on('disconnect', ({ code, reason } = {}) => {
@@ -236,6 +239,11 @@ export async function createTransport ({ identity, dir, url = DEFAULT_PROXY, com
     if (identificado && client.token) {
       if (downSince) log(`[vault] back on the proxy after ${Math.round((Date.now() - downSince) / 1000)}s`)
       downSince = 0; saidDown = false
+      // EL ANUNCIO CADUCA: el proxio borra una entrada de canal a los 20 minutos de publicada.
+      // Sin renovarlo, entrar con usuario y contraseña solo funcionaba los primeros 20 minutos
+      // tras identificarse, y desde fuera se veía como «no hay ninguna bóveda en esa dirección»
+      // con la bóveda encendida (2026-10-12, la demo de los revisores).
+      if (announce && Date.now() - announcedAt >= REANNOUNCE_MS) await announceAccount()
       return
     }
     if (!downSince) downSince = Date.now()

package/src/vault.js

@@ -1477,9 +1477,9 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
    * firma del autor, las envolturas, la espera de aprobación. Aquí no hay otra puerta.
    *
    * Los límites (dueño, 2026-09-29):
-   *   · si hay alguien que aprueba, SIEMPRE se pide aprobación —`unattended` no exime— y con
-   *     el sí puede también EDITAR lo que ya existe;
-   *   · si nadie puede aprobar, no se pide y solo puede AÑADIR lo que falta;
+   *   · AÑADIR una variable nueva es directo, haya o no quien apruebe (dueño, 2026-10-12);
+   *   · EDITAR una que ya existe pide aprobación —`unattended` no exime—;
+   *   · si nadie puede aprobar, no se pide y lo que ya existe no se toca (`exists`);
    *   · BORRAR, nunca: eso es de la consola y del CLI de la bóveda.
    *
    * Dos operaciones: `store.recipients` (para quién se sella: solo públicas, nada que ocultar)
@@ -1696,7 +1696,7 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
         log(`[vault] ${ns}: ${deviceId || '????-????'} wants to store ${items.length} variable(s) — waiting for approval (${result.pending})`)
       } else {
         audit('store', { device: deviceId, ns, keys: result?.keys || [] })
-        log(`[vault] ${ns}: ${deviceId || '????-????'} stored ${(result?.keys || []).length} variable(s) (nobody can approve: add only)`)
+        log(`[vault] ${ns}: ${deviceId || '????-????'} stored ${(result?.keys || []).length} new variable(s)`)
         await notifyMembers('vars', { by: deviceId, keys: result?.keys || [], ns })
       }
       return await answer({ op: 'store.result', ns, ts: Date.now(), keys: result?.keys || [], pending: result?.pending || null })
@@ -2811,13 +2811,18 @@ export async function startVault ({ dir = dataDir(), proxyUrl, log = console.log
       const owner = ns ? `ns:${ns}` : `dev:${pub}`
       const keys = []
       // ¿SE PIDE APROBACIÓN? Desde la mesa del admin, lo de siempre (`needsApproval`: quien
-      // tiene `unattended` escribe solo). Desde el propio SERVICIO que guarda en su cajón, se
-      // pide SIEMPRE que haya quien apruebe, tenga o no `unattended` (dueño, 2026-09-29:
-      // «el unattended también pediría aprobación»): `unattended` es para llevarse claves al
-      // arrancar, no para reescribir su propia configuración. Sin nadie que apruebe no se
-      // pide —la regla de siempre— y entonces solo puede añadir.
+      // tiene `unattended` escribe solo). Desde el propio SERVICIO que guarda en su cajón, la
+      // aprobación es SOLO PARA EDITAR (dueño, 2026-10-12: «la aprobación solamente es para
+      // editar una env; para almacenarla es directo»): una variable nueva no le quita nada a
+      // nadie y se guarda en el acto; reemplazar una que ya existe pide el sí de quien
+      // aprueba, tenga o no `unattended` el servicio (dueño, 2026-09-29: «el unattended
+      // también pediría aprobación»). Una tanda con alguna que ya existe se pide ENTERA: va
+      // todo o nada. Sin nadie que apruebe no se pide —la regla de siempre— y lo que ya
+      // existe falla con `exists`.
       const record = await refreshActa()
-      const ask = fromService ? hasApprover(record) : needsApproval(caller, record)
+      const ask = fromService
+        ? hasApprover(record) && items.some((it) => it?.key && secrets.has(owner, it.key))
+        : needsApproval(caller, record)
       for (const it of items) {
         if (!it?.key || !it?.sealed) throw new Error('var.setMany: each variable needs its key and its sealed envelope')
         await checkAuthor(owner, it.key, it.sealed, caller, { allowOverwrite: ask })