@dotrino/vaultd 0.58.0 → 0.59.0

package/lib/src/enroll.js

@@ -277,7 +277,12 @@ export function createEnrollDesk ({
       pend.continuity = (okC && d.continuity.member === d.dpub) ? d.continuity : null
     }
     pend.from = from // la bóveda NO conoce el código: lo aprende cuando lo tipeas
-    if (d.label) pend.label = String(d.label).slice(0, 60)
+    // EL NOMBRE QUE PUSISTE AQUÍ MANDA. El aparato manda el suyo al enrolarse, y hasta
+    // ahora pisaba siempre al de la bóveda — como el aparato usa por defecto el apodo del
+    // PERFIL, acababas con varios dispositivos llamados igual que tú y sin forma de saber
+    // cuál era cuál. Si en `pair` le diste un nombre, ese es el nombre; el del aparato
+    // sigue valiendo como propuesta cuando no dijiste nada.
+    if (d.label && !pend.label) pend.label = String(d.label).slice(0, 60)
     // Camino A: de qué cuenta estamos hablando. Se guarda para poder comprobar, cuando
     // llegue el acta sellada, que es la que este dispositivo dijo que iba a entregar.
     if (intent === 'adopt' && typeof d.profileId === 'string') pend.profileId = d.profileId

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/vaultd",
-  "version": "0.58.0",
+  "version": "0.59.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/ctl.js

@@ -205,6 +205,17 @@ async function cmdPair (args = []) {
   // --approval: el aparato que entre pedirá tu aprobación (teléfono) en cada petición de
   // claves privadas. Por defecto NO pide; se cambia después con `caps <ID> +permiso`.
   const approval = args.includes('--approval')
+  // --name <n>: CÓMO SE VA A LLAMAR el aparato que entre, decidido aquí y antes de nada.
+  // Sin esto el nombre lo ponía el propio aparato, que por defecto usa el apodo del
+  // PERFIL: acababas con varios dispositivos llamados igual que tú, y para distinguirlos
+  // había que renombrarlos después —cuando ya no sabías cuál era cuál—.
+  const nIdx = args.indexOf('--name')
+  let label = null
+  if (nIdx >= 0) {
+    const v = args[nIdx + 1]
+    if (!v || v.startsWith('-')) { console.error('uso: dotrino-vault pair --name "teléfono de casa"'); process.exit(2) }
+    label = v.slice(0, 60)
+  }
   // --admin: el aparato que entre por esta invitación podrá ADMINISTRAR (la consola).
   // El QR no lleva nada: es una nota local de la bóveda y el permiso se aplica al aprobar
   // con el código que tecleas aquí. Ahorra el `caps <ID> +administra` de después, que en un
@@ -258,9 +269,11 @@ async function cmdPair (args = []) {
   // meter el dispositivo en una cuenta que ya vive aquí, se ESTRENA una (vacía) y
   // entra a ella; las demás no se tocan. En la TUI esto es una pregunta con sus
   // opciones; en la CLI es una bandera, para que siga sirviendo en un script.
+  // `--name` lleva su valor detrás: no puede confundirse con el nombre de la cuenta.
+  const consumido = (i) => nIdx >= 0 && i === nIdx + 1
   const naIdx = args.findIndex((a) => a === '--new-account')
   if (naIdx >= 0) {
-    const next = args[naIdx + 1]
+    const next = consumido(naIdx + 1) ? null : args[naIdx + 1]
     const name = (next && !next.startsWith('-')) ? next : `cuenta ${new Date().toISOString().slice(0, 10)}`
     const d = await profileRequest('add', { name, ...(pairKek ? { kek: pairKek } : {}) })
     if (d.error) { console.error('%s', d.error); process.exit(1) }
@@ -275,7 +288,7 @@ async function cmdPair (args = []) {
   const adIdx = args.findIndex((a) => a === '--adopt')
   const adopt = adIdx >= 0
   if (adopt) {
-    const next = args[adIdx + 1]
+    const next = consumido(adIdx + 1) ? null : args[adIdx + 1]
     const name = (next && !next.startsWith('-')) ? next : `cuenta del dispositivo`
     const d = await profileRequest('add', { name, adopt: true, ...(pairKek ? { kek: pairKek } : {}) })
     if (d.error) { console.error('%s', d.error); process.exit(1) }
@@ -298,7 +311,7 @@ async function cmdPair (args = []) {
 
   // La petición se escribe SIEMPRE (aunque no haya --service): lleva a qué perfil
   // se empareja el dispositivo.
-  writeReq('pair-request.json', { ...(service ? { service } : {}), ...(scope ? { scope } : {}), ...(adopt ? { mode: 'adopt' } : {}), ...(approval ? { approval: true } : {}), ...(admin ? { admin: true } : {}) })
+  writeReq('pair-request.json', { ...(service ? { service } : {}), ...(scope ? { scope } : {}), ...(adopt ? { mode: 'adopt' } : {}), ...(approval ? { approval: true } : {}), ...(admin ? { admin: true } : {}), ...(label ? { label } : {}) })
   sendSignal(s.pid, 'SIGUSR1')
 
   let pair = null
@@ -1469,6 +1482,9 @@ function help () {
                       estrena una cuenta VACÍA en este vault y mete ahí al dispositivo
                       (sin la bandera entra a la cuenta activa, o a la de --profile)
   pair --service <ns> empareja un SERVICIO (proxy, geo…) con acceso SOLO a sus secretos
+  pair --name <nombre>  cómo se llamará el aparato que entre. Sin esto el nombre lo pone
+                      ÉL, y por defecto usa el apodo del perfil: acabas con varios
+                      dispositivos llamados igual que tú
   pair --approval       el aparato que entre pedirá tu aprobación (teléfono) al recibir claves
   pair --admin          el aparato que entre podrá ADMINISTRAR (es lo que es una consola).
                       El QR no lleva nada: el permiso se aplica al aprobar el código aquí

package/src/tui/app.js

@@ -918,12 +918,41 @@ async function descartarCuenta (term, st, id, volverA) {
   if (r.ok) flash(st, i.accountDiscarded)
 }
 
+/**
+ * PRIMERO EL NOMBRE, y luego el QR.
+ *
+ * Sin preguntarlo, el nombre lo pone el propio aparato al enrolarse, y por defecto usa el
+ * apodo del PERFIL: te quedaban tres dispositivos llamados igual que tú y ninguna forma de
+ * saber cuál era cuál sin renombrarlos después —cuando ya no te acordabas—. Se pregunta
+ * ANTES porque después ya estás mirando un QR con el teléfono en la mano.
+ *
+ * Enter en blanco sigue valiendo: deja que lo diga el aparato, que es el comportamiento de
+ * siempre para quien no quiera decidirlo.
+ */
+function beginPairingNamed (term, st, profile, service = null, after = null) {
+  const i = L(st)
+  setInput(st, {
+    label: i.nameDeviceLabel,
+    hint: i.nameDeviceHint,
+    onSubmit: async (nombre) => {
+      st.input = null
+      const ok = await beginPairing(term, st, profile, service, (nombre || '').trim().slice(0, 60) || null)
+      // `after` es para quien tenga que limpiar si el emparejamiento no llega a abrirse
+      // (la cuenta que nació para él). Va aquí porque con el nombre de por medio ya no se
+      // puede mirar el valor de retorno desde fuera.
+      if (after) await after(ok)
+    },
+    onCancel: () => { st.input = null }
+  })
+  return true
+}
+
 /**
  * Abre el emparejamiento contra `profile` y salta a la pantalla del QR. Con `service`,
  * el QR es el de un SERVICIO de ese namespace (cert limitado a sus variables).
  */
-async function beginPairing (term, st, profile, service = null) {
-  const r = await guard(term, st, L(st).startingPairing, () => vc.startPairing({ profile, ...(service ? { service } : {}) }))
+async function beginPairing (term, st, profile, service = null, label = null) {
+  const r = await guard(term, st, L(st).startingPairing, () => vc.startPairing({ profile, ...(service ? { service } : {}), ...(label ? { label } : {}) }))
   // `service` se pega al estado porque el daemon no lo devuelve: la pantalla del QR
   // tiene que poder decir qué se está entregando, que no es lo mismo un aparato tuyo
   // que una máquina que solo va a leer la configuración del proxy.
@@ -983,7 +1012,7 @@ async function onKeyPairMode (term, st, key) {
   if (key.name === 'escape' || ch === 'b') { st.screen = 'devices'; return true }
   if (key.name !== 'enter' || !cur) return true
 
-  if (cur.mode === 'here') { await beginPairing(term, st, activeId(st)); return true }
+  if (cur.mode === 'here') { return beginPairingNamed(term, st, activeId(st)) }
 
   // SERVICIO: entra a la cuenta activa, pero con un certificado que solo sirve para
   // pedir las variables de SU namespace. El nombre del ns es el que luego pide el
@@ -1025,8 +1054,10 @@ async function onKeyPairMode (term, st, key) {
       // Si el emparejamiento no llega a abrirse, la cuenta que se creó PARA él se va con
       // él: si no, cada intento fallido dejaba una cuenta vacía —y encima activa— que
       // luego había que ir a borrar a mano adivinando cuál era.
-      if (!await beginPairing(term, st, created)) await descartarCuenta(term, st, created, previa)
-      else st.pairing.born = { id: created, from: previa }
+      beginPairingNamed(term, st, created, null, async (ok) => {
+        if (!ok) await descartarCuenta(term, st, created, previa)
+        else st.pairing.born = { id: created, from: previa }
+      })
     },
     onCancel: () => { st.input = null }
   })

package/src/tui/i18n.js

@@ -140,6 +140,8 @@ const es = {
   cantRemoveMaster: 'Esta bóveda es la que manda: no se quita a sí misma.',
   revokedCount: (n) => `  Revocados: ${n}`,
   startingPairing: 'Iniciando emparejamiento…',
+  nameDeviceLabel: '¿Cómo se va a llamar este aparato?',
+  nameDeviceHint: 'Enter para dejar que lo diga él (usará tu apodo, y así no se distinguen)',
   noPending: 'No hay ningún dispositivo pendiente',
   noPendingToReject: 'No hay ningún dispositivo pendiente para rechazar',
   rejecting: 'Rechazando…',
@@ -435,6 +437,8 @@ const en = {
   cantRemoveMaster: 'This vault is the Master: it does not remove itself.',
   revokedCount: (n) => `  Revoked: ${n}`,
   startingPairing: 'Starting pairing…',
+  nameDeviceLabel: 'What will this device be called?',
+  nameDeviceHint: 'Enter to let it say (it will use your nickname, and then they all look alike)',
   noPending: 'No device is waiting',
   noPendingToReject: 'No device is waiting to be rejected',
   rejecting: 'Rejecting…',

package/src/vaultControl.js

@@ -604,10 +604,10 @@ export function pairUrl (qr) {
   return { url: inviteUrl(qr), code, payload: JSON.stringify(qr), b64: code }
 }
 
-export async function startPairing ({ profile, service } = {}) {
+export async function startPairing ({ profile, service, label } = {}) {
   requireAlive()
   rm(F.pair); rm(F.pending)
-  await writeReq(F.pairReq, service ? { service } : {}, profile)
+  await writeReq(F.pairReq, { ...(service ? { service } : {}), ...(label ? { label } : {}) }, profile)
   signalOrCleanup('SIGUSR1', [F.pairReq])
   for (let i = 0; i < 50; i++) {
     await sleep(100)