@agentvault/agentvault 0.23.7 → 0.23.9

package/dist/index.js

@@ -64792,7 +64792,7 @@ var init_channel = __esm({
        */
       sendActivitySpan(spanData) {
         if (!this._ws || this._ws.readyState !== WebSocket.OPEN) return;
-        const pluginVersion = true ? "0.23.7" : "0.0.0-dev";
+        const pluginVersion = true ? "0.23.9" : "0.0.0-dev";
         const agentName = this.config.agentName ?? "Agent";
         const resource = {
           "service.name": "agentvault-agent",
@@ -65033,10 +65033,76 @@ var init_channel = __esm({
           this._sessions.delete(convId);
           delete this._persisted.sessions[convId];
         }
+        if (room.mlsGroupId) {
+          try {
+            await deleteMlsState(this.config.dataDir, room.mlsGroupId);
+          } catch {
+          }
+        }
+        this._mlsGroups.delete(roomId);
         delete this._persisted.rooms[roomId];
         await this._persistState();
         this.emit("room_left", { roomId });
       }
+      /**
+       * #629 recurrence fix — reconcile local room state against the server on
+       * connect.
+       *
+       * On disband the server marks each member ``"left"`` and sets
+       * ``room.status="disbanded"`` but sends NO WS event to the agent's bridge, so a
+       * disbanded/left room lingers in local state and replays dead MLS commit
+       * history on every reconnect ("Desired gen in the past" / "invalid ghash tag").
+       * ``GET /rooms`` returns ONLY active rooms the caller is a member of, so any
+       * locally-persisted room ABSENT from it is terminal — prune its MLS state
+       * (file + in-memory group, keyed by roomId), its sessions, and the room entry.
+       *
+       * Fail-safe: a failed fetch / non-ok / non-array response prunes NOTHING, so a
+       * transient error can never over-prune a live room.
+       */
+      async _reconcileRoomsWithServer() {
+        const local = this._persisted?.rooms;
+        if (!local || Object.keys(local).length === 0) return;
+        if (!this._deviceJwt) return;
+        let activeIds;
+        try {
+          const res = await fetch(`${this.config.apiUrl}/api/v1/rooms`, {
+            headers: { Authorization: `Bearer ${this._deviceJwt}` }
+          });
+          if (!res.ok) return;
+          const list = await res.json();
+          if (!Array.isArray(list)) return;
+          activeIds = new Set(
+            list.map((r2) => r2?.id).filter((id) => !!id)
+          );
+        } catch (err) {
+          console.warn(
+            `[SecureChannel] room reconcile skipped (fetch failed, pruning nothing): ${err instanceof Error ? err.message : String(err)}`
+          );
+          return;
+        }
+        const stale = Object.keys(local).filter((rid) => !activeIds.has(rid));
+        if (stale.length === 0) return;
+        for (const rid of stale) {
+          const room = local[rid];
+          if (room?.mlsGroupId) {
+            try {
+              await deleteMlsState(this.config.dataDir, room.mlsGroupId);
+            } catch {
+            }
+          }
+          this._mlsGroups.delete(rid);
+          for (const convId of room?.conversationIds ?? []) {
+            this._sessions.delete(convId);
+            if (this._persisted?.sessions) delete this._persisted.sessions[convId];
+          }
+          delete local[rid];
+        }
+        await this._persistState();
+        console.log(
+          `[SecureChannel] Room reconcile: pruned ${stale.length} disbanded/left room(s) from local state (${stale.map((r2) => r2.slice(0, 8)).join(", ")})`
+        );
+        this.emit("rooms_reconciled", { pruned: stale });
+      }
       /**
        * Return info for all joined rooms.
        */
@@ -66387,6 +66453,9 @@ var init_channel = __esm({
             await this._pullDrDeliveryQueue();
             await this._flushOutboundQueue();
             this._setState("ready");
+            void this._reconcileRoomsWithServer().catch(
+              (err) => console.warn(`[SecureChannel] room reconcile failed (ignored): ${err instanceof Error ? err.message : String(err)}`)
+            );
             if (this.config.enableScanning) {
               this._scanEngine = new ScanEngine();
               await this._fetchScanRules();
@@ -66409,7 +66478,7 @@ var init_channel = __esm({
                 agentVersion: this.config.agentVersion ?? "0.0.0",
                 // __AV_VERSION__ is injected by esbuild from package.json at build time.
                 // Falls back to "0.0.0-dev" in non-bundled contexts (tests).
-                pluginVersion: true ? "0.23.7" : "0.0.0-dev"
+                pluginVersion: true ? "0.23.9" : "0.0.0-dev"
               });
               this._telemetryReporter.startAutoFlush(3e4);
             }
@@ -66488,6 +66557,10 @@ var init_channel = __esm({
               await this._handleDeviceRevoked();
               return;
             }
+            if (data.event === "connection_rejected") {
+              await this._handleConnectionRejected(data);
+              return;
+            }
             if (data.event === "device_linked") {
               await this._handleDeviceLinked(data.data);
               return;
@@ -66661,6 +66734,7 @@ var init_channel = __esm({
             if (data.event === "arming_snapshot") {
               this.emit("arming_snapshot", {
                 workAllowed: data.data?.work_allowed === true,
+                gatesRemoved: data.data?.gates_removed === true,
                 roomIds: data.data?.room_ids ?? []
               });
             }
@@ -66722,7 +66796,7 @@ var init_channel = __esm({
                     agentVersion: this.config.agentVersion ?? "0.0.0",
                     // __AV_VERSION__ is injected by esbuild from package.json at build time.
                     // Falls back to "0.0.0-dev" in non-bundled contexts (tests).
-                    pluginVersion: true ? "0.23.7" : "0.0.0-dev"
+                    pluginVersion: true ? "0.23.9" : "0.0.0-dev"
                   });
                   this._telemetryReporter.startAutoFlush(3e4);
                 }
@@ -68032,7 +68106,12 @@ ${messageText}`;
             senderName: senderLabel,
             plaintext: messageText,
             messageType,
-            timestamp: data.created_at ?? (/* @__PURE__ */ new Date()).toISOString()
+            timestamp: data.created_at ?? (/* @__PURE__ */ new Date()).toISOString(),
+            // Native bridge (claude-room-bridge) consumes this to decide reply
+            // expectation: a human/owner speaking in a room always expects a reply,
+            // an agent does not. Derived from the room roster (line ~5394) — the same
+            // signal the OpenClaw mention-filter uses for loop prevention.
+            senderIsAgent
           });
           const contextualMessage = senderIsAgent ? messageText : `[${senderLabel}]: ${messageText}`;
           Promise.resolve(this.config.onMessage?.(contextualMessage, metadata)).catch((err) => {
@@ -69909,6 +69988,37 @@ ${messageText}`;
        * re-enrollment, which SHOULD wipe creds, goes through setup --force /
        * _forceReEnroll in start().
        */
+      /**
+       * Handle a WS `connection_rejected` event.
+       *
+       * The server accepts the WS upgrade, then — for a device it will not serve —
+       * emits `{event:"connection_rejected", reason, retryable}` and closes with
+       * code 4403. `retryable:false` (e.g. `device_revoked`) is a TERMINAL
+       * credential failure: retrying can only reconnect, get rejected, and close
+       * again. Left unhandled, that generic close drove the reconnect classifier
+       * into "reconnect loop detected", which the bridge treats as a restartable
+       * exit — so launchd/systemd relaunched us every ~30s to hammer the backend
+       * with the same rejected handshake (empirically ~1150 rejects / 2h).
+       *
+       * So we route a non-retryable rejection to the SAME `auth_failed` surface the
+       * proactive/reactive reissue paths use (see channel.ts emit sites). The
+       * bridge wires `auth_failed` to a clean exit(0) (packages/claude-room-bridge
+       * /src/bridge.ts), so launchd stops relaunching. Mirrors those emitters:
+       * emit only + mark the session; the consumer decides how to stop. A
+       * `retryable` rejection is transient — fall through to a normal reconnect.
+       */
+      async _handleConnectionRejected(data) {
+        if (data?.retryable === true) {
+          this._scheduleReconnect();
+          return;
+        }
+        this._authFailedThisSession = true;
+        const authReason = data?.reason === "device_jwt_expired" ? "device_jwt_expired" : "device_revoked";
+        console.warn(
+          `[SecureChannel] connection_rejected (reason=${data?.reason ?? "unknown"}, retryable=false) \u2014 terminal; surfacing auth_failed`
+        );
+        this.emit("auth_failed", { reason: authReason });
+      }
       async _handleDeviceRevoked() {
         const now = Date.now();
         this._revokeRecoveries = this._revokeRecoveries.filter(
@@ -96900,7 +97010,7 @@ var init_index = __esm({
     init_skill_invoker();
     await init_skill_telemetry();
     await init_policy_enforcer();
-    VERSION = true ? "0.23.7" : "0.0.0-dev";
+    VERSION = true ? "0.23.9" : "0.0.0-dev";
   }
 });
 await init_index();