@slack/radar-mcp 1.7.0 → 1.9.0

package/dist/web/public/next/store.js

@@ -100,6 +100,12 @@ class Store {
     _localPushKey = null;
     _lastBoundLabel = null;
     _backfillToken = 0;
+    // True while the connection is in a dropped/waiting state. Lets pollHealth detect the
+    // drop->recover EDGE (device came back) so it can re-open the stream + backfill ONCE on
+    // recovery, instead of every poll. Without this, an auto-recovered tab flips the dot to
+    // connected but the dead EventSource is never re-opened and the device ring is never pulled,
+    // so the tab shows nothing until a manual refresh.
+    _dropped = false;
     _rafPending = false;
     _intervals = [];
     // ---- shared detail cache (bounded FIFO) ----
@@ -204,6 +210,7 @@ class Store {
         this._localPushKey = null;
         this._lastBoundLabel = null;
         this._backfillToken = 0;
+        this._dropped = false;
         this._rafPending = false;
     }
     // ---- start everything ----
@@ -317,7 +324,12 @@ class Store {
         this.closeStream();
         const es = new EventSource("/stream");
         es.onmessage = (m) => this.onFrame(m);
-        es.onerror = () => this.setConn("waiting", "reconnecting…");
+        // An SSE error is the most direct signal the stream died; mark dropped so the next healthy
+        // poll re-opens + backfills (recovered()), not just flips the dot green.
+        es.onerror = () => {
+            this._dropped = true;
+            this.setConn("waiting", "reconnecting…");
+        };
         this._es = es;
     }
     closeStream() {
@@ -349,6 +361,10 @@ class Store {
         if (!STREAM_KINDS.includes(f.type))
             return;
         this._lastEventTs = Date.now();
+        // A live frame proves the stream healed itself (the browser EventSource auto-reconnects on a
+        // transient blip). Clear _dropped here so recovered() does NOT later tear down this healthy
+        // stream and re-open it: recovered() should only fire when the stream did NOT self-heal.
+        this._dropped = false;
         this.setConn("connected", "");
         const ev = f.event || {};
         ev._type = f.type;
@@ -593,6 +609,7 @@ class Store {
     async pollHealth() {
         if (Date.now() - this._lastEventTs < 6000) {
             this._healthMisses = 0;
+            this.recovered();
             this.setConn("connected", "");
             return;
         }
@@ -600,17 +617,31 @@ class Store {
             const h = await (await fetch("/health")).json();
             if (h.device) {
                 this._healthMisses = 0;
+                this.recovered();
                 this.setConn("connected", "");
             }
             else if (++this._healthMisses >= 2) {
+                this._dropped = true;
                 this.setConn("waiting", "device offline, retrying…");
             }
         }
         catch {
-            if (++this._healthMisses >= 2)
+            if (++this._healthMisses >= 2) {
+                this._dropped = true;
                 this.setConn("waiting", "reconnecting…");
+            }
         }
     }
+    // Called when health flips back to connected. If we were in a dropped state, the EventSource
+    // did not survive the drop and the tab has been silent, so re-open the stream + backfill the
+    // device ring ONCE on the recovery edge. The _dropped guard makes this fire on the transition,
+    // not on every healthy poll (resyncStream clears the flag). The <6s-since-last-event fast path
+    // means a tab that never lost its stream skips this (live frames keep _lastEventTs fresh).
+    recovered() {
+        if (!this._dropped)
+            return;
+        this.resyncStream();
+    }
     async pollProfile() {
         try {
             const [profilesR, healthR] = await Promise.all([
@@ -635,6 +666,26 @@ class Store {
             /* leave the last label */
         }
     }
+    // Re-open the SSE stream and backfill the device ring for the ACTIVE streaming source, so the
+    // current tab repopulates from what the device already holds (not just new live events).
+    // Needed because the EventSource does not survive a device drop and pollSpec only re-applies
+    // (and so re-opens+backfills) on a spec-KEY change, which a reconnect is not. Guarded for
+    // streaming sources only, matching applySpec: db + blank own no SSE feed; log opens the stream
+    // but backfill() self-skips it (host-side capture, no device ring to replay).
+    //
+    // Clears _dropped itself so the two recovery entry points -- the manual reconnect() button and
+    // the auto recovered() edge -- collapse into one idempotent op: whichever fires first wins, the
+    // other sees _dropped false and no-ops. Without this, a manual reconnect on a waiting tab fires
+    // resyncStream() here AND again from recovered() once the health poll resolves, so the stream is
+    // opened, torn down, reopened, and the ring is backfilled twice.
+    resyncStream() {
+        this._dropped = false;
+        const src = this.spec.source;
+        if (this.spec.blank || !src || src === "db")
+            return;
+        this.openStream();
+        this.backfill();
+    }
     async reconnect() {
         try {
             await fetch("/enable", { method: "POST" });
@@ -645,6 +696,7 @@ class Store {
         this.pollHealth();
         this.pollProfile();
         this.pollSpec();
+        this.resyncStream();
     }
 }
 // ---- aggregate math (moved here from widgets per the plan: it belongs WITH the aggregate

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@slack/radar-mcp",
-  "version": "1.7.0",
+  "version": "1.9.0",
   "description": "MCP server and web dashboard for on-device debugging of the Slack Android app via ADB",
   "type": "module",
   "main": "dist/mcp/index.js",
@@ -22,7 +22,8 @@
     "start": "node dist/mcp/index.js",
     "start:web": "node dist/web/bin.js",
     "watch": "tsc --watch",
-    "test": "npm run build && node --test tests/*.test.mjs",
+    "test": "npm run build && node tests/run.mjs",
+    "test:tap": "npm run build && node --test --test-force-exit tests/*.test.mjs",
     "test:smoke": "npm run build && bash tests/smoke.sh",
     "test:all": "npm run test && npm run test:smoke",
     "prepare": "npm run build"