@dotrino/remote-agent 0.3.2 → 0.4.1

package/README.md

@@ -62,6 +62,10 @@ const ra = await startRemoteAgent({
   onRevoked () { console.log('me revocaron') }
 })
 // ra.machine, ra.machineId, ra.master, ra.close()
+// ra.client → el WebSocketProxyClient ya conectado e identificado bajo esta
+//   máquina. Si tu agente necesita algo más del transporte (anunciarse en un
+//   canal, por ejemplo), REUSA este: abrir una segunda conexión sería un segundo
+//   `identify`, una segunda identidad de transporte y una segunda cola.
 ```
 
 Enrolar la máquina (una vez, con el QR del vault):

package/node-globals.js

@@ -34,7 +34,24 @@ let _installed = false
 export function installNodeGlobals (dir) {
   if (_installed) return
   if (typeof globalThis.WebSocket === 'undefined') globalThis.WebSocket = WebSocket
-  // Node ≥22 expone un `localStorage` no funcional sin flag → forzamos el shim.
-  globalThis.localStorage = fileLocalStorage(path.join(dir, 'transport.json'))
+
+  // Node ≥22 expone un `localStorage` NO funcional sin `--localstorage-file`, así
+  // que aquí se pone el shim de archivo igual. Pero se pone con `defineProperty` y
+  // NO asignando:
+  //
+  //   · lo que Node define es un ACCESOR (getter/setter nativos), y asignarle
+  //     revienta con «Cannot assign to read only property 'localStorage'» — medido
+  //     en Node 25.9. Con la asignación, cualquier agente headless dejaba de
+  //     arrancar su plano de control en un Node moderno, y el mensaje no decía nada
+  //     de esto;
+  //   · el descriptor tampoco se LEE (nada de `globalThis.localStorage`): tocar el
+  //     getter nativo sin el flag emite un warning por consola.
+  //
+  // Es lo mismo que hace `@dotrino/vault`, que ya se topó con esto.
+  Object.defineProperty(globalThis, 'localStorage', {
+    configurable: true,
+    writable: true,
+    value: fileLocalStorage(path.join(dir, 'transport.json'))
+  })
   _installed = true
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotrino/remote-agent",
-  "version": "0.3.2",
+  "version": "0.4.1",
   "description": "Middleware compartido del ecosistema Dotrino para apps de agente remoto cifrado punto a punto (handshake E2E, emparejamiento SAS con vault, revocación, descubrimiento). Lo consumen dotrino-terminal (PTY) y dotrino-ia (chat con IA).",
   "type": "module",
   "main": "./index.js",
@@ -19,12 +19,12 @@
     "src"
   ],
   "scripts": {
-    "test": "node -e \"import('@dotrino/remote-agent').then(()=>console.log('ok'))\"",
+    "test": "node --test 'test/**/*.test.js'",
     "type-check": "tsc --noEmit"
   },
   "dependencies": {
     "@dotrino/identity": "0.53.0",
-    "@dotrino/proxy-client": "0.10.0",
+    "@dotrino/proxy-client": "0.10.1",
     "ws": "8.21.0"
   },
   "engines": {

package/src/agent.js

@@ -74,6 +74,13 @@ class AgentSession {
  * @param {()=>void} [opts.onReady]   agente listo y escuchando.
  * @param {boolean} [opts.quiet]      sin logs.
  * @param {object} [opts.client]      transporte ya conectado (SOLO pruebas).
+ *
+ * Devuelve además el **`client`** (el `WebSocketProxyClient` ya conectado e
+ * identificado bajo la pubkey de esta máquina). Se expone a propósito: un agente
+ * que necesite algo más del transporte —anunciarse en un canal, por ejemplo— debe
+ * reusar ESTA conexión y no abrir otra. Dos conexiones del mismo aparato al proxy
+ * son dos identidades de transporte, dos `identify` y dos colas: exactamente el
+ * lío que este middleware existe para evitar.
  */
 export async function startRemoteAgent (opts = {}) {
   const dir = opts.dir || dataDir()
@@ -279,7 +286,7 @@ export async function startRemoteAgent (opts = {}) {
   }
   if (opts.onReady) { try { opts.onReady({ machine: myPub, machineId: myId, master }) } catch (_) {} }
 
-  return { machine: myPub, machineId: myId, master, close: stop }
+  return { machine: myPub, machineId: myId, master, client, close: stop }
 }
 
 export default { startRemoteAgent }