@btraut/browser-bridge 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -8,6 +8,19 @@ The format is based on "Keep a Changelog", and this project adheres to Semantic
 
 _TBD_
 
+## [0.7.2] - 2026-02-12
+
+### Fixed
+
+- Core debugger bridge now self-heals stale attach state: when a command fails with "Debugger is not attached to the requested tab.", it marks the tab detached, re-attaches once, and retries the command once. It also clears cached attachment state after extension disconnects to avoid false "attached" assumptions that can trigger inspect recovery loops/timeouts.
+
+## [0.7.1] - 2026-02-11
+
+### Fixed
+
+- Diagnostics doctor: stop failing by default when `session_id` is omitted, treat detached debugger as expected idle behavior, and downgrade stale drive/inspect errors to warnings.
+- Core error latching: clear drive/inspect/debugger `last_error` state after successful operations so recovered sessions report healthy diagnostics.
+
 ## [0.7.0] - 2026-02-10
 
 ### Fixed

package/dist/api.js

@@ -848,6 +848,10 @@ var DriveController = class {
     this.lastError = error;
     this.lastErrorAt = (/* @__PURE__ */ new Date()).toISOString();
   }
+  clearLastError() {
+    this.lastError = void 0;
+    this.lastErrorAt = void 0;
+  }
   async execute(sessionId, action, params, timeoutMs) {
     return await driveMutex.runExclusive(async () => {
       try {
@@ -888,6 +892,7 @@ var DriveController = class {
             timeoutMs
           );
           if (response.status === "ok") {
+            this.clearLastError();
             return {
               ok: true,
               result: response.result
@@ -2800,6 +2805,7 @@ var InspectService = class {
     });
   }
   markInspectConnected(sessionId) {
+    this.clearLastError();
     try {
       const session = this.registry.require(sessionId);
       if (session.state === "INIT" /* INIT */ || session.state === "DRIVE_READY" /* DRIVE_READY */) {
@@ -2814,6 +2820,10 @@ var InspectService = class {
     this.lastError = error;
     this.lastErrorAt = (/* @__PURE__ */ new Date()).toISOString();
   }
+  clearLastError() {
+    this.lastError = void 0;
+    this.lastErrorAt = void 0;
+  }
   buildUnavailableError() {
     return new InspectError(
       "INSPECT_UNAVAILABLE",
@@ -3474,10 +3484,20 @@ var registerArtifactsRoutes = (router, options = {}) => {
 };
 
 // packages/core/src/diagnostics.ts
+var STALE_ERROR_THRESHOLD_MS = 2 * 60 * 1e3;
+var getErrorAgeMs = (timestamp) => {
+  const parsed = Date.parse(timestamp);
+  if (!Number.isFinite(parsed)) {
+    return void 0;
+  }
+  return Math.max(0, Date.now() - parsed);
+};
 var buildDiagnosticReport = (sessionId, context = {}) => {
   const extensionConnected = context.extension?.connected ?? false;
   const debuggerAttached = context.debugger?.attached ?? false;
   const sessionState = context.sessionState;
+  const hasSessionId = Boolean(sessionId);
+  const warnings = [];
   const checks = [
     {
       name: "extension.connected",
@@ -3486,13 +3506,13 @@ var buildDiagnosticReport = (sessionId, context = {}) => {
     },
     {
       name: "debugger.attached",
-      ok: debuggerAttached,
-      message: debuggerAttached ? "Debugger is attached." : "Debugger is not attached."
+      ok: true,
+      message: debuggerAttached ? "Debugger is attached." : "Debugger is not attached (inspect is idle)."
     },
     {
       name: "session.state",
-      ok: Boolean(sessionState),
-      message: sessionState ? `Session state is ${sessionState}.` : sessionId ? "Session state unavailable." : "Session id not provided.",
+      ok: hasSessionId ? Boolean(sessionState) : true,
+      message: sessionState ? `Session state is ${sessionState}.` : hasSessionId ? "Session state unavailable." : "Session id not provided.",
       details: {
         session_id: sessionId || null,
         state: sessionState ?? "UNKNOWN"
@@ -3500,26 +3520,42 @@ var buildDiagnosticReport = (sessionId, context = {}) => {
     }
   ];
   if (context.driveLastError) {
+    const ageMs = getErrorAgeMs(context.driveLastError.at);
+    const isStale = ageMs !== void 0 && ageMs > STALE_ERROR_THRESHOLD_MS;
+    if (isStale) {
+      warnings.push(
+        `Ignoring stale drive error (${Math.round(ageMs / 1e3)}s old): ${context.driveLastError.message}`
+      );
+    }
     checks.push({
       name: "drive.last_error",
-      ok: false,
+      ok: isStale,
       message: context.driveLastError.message,
       details: {
         code: context.driveLastError.code,
         retryable: context.driveLastError.retryable,
-        at: context.driveLastError.at
+        at: context.driveLastError.at,
+        ...ageMs !== void 0 ? { age_ms: ageMs } : {}
       }
     });
   }
   if (context.inspectLastError) {
+    const ageMs = getErrorAgeMs(context.inspectLastError.at);
+    const isStale = ageMs !== void 0 && ageMs > STALE_ERROR_THRESHOLD_MS;
+    if (isStale) {
+      warnings.push(
+        `Ignoring stale inspect error (${Math.round(ageMs / 1e3)}s old): ${context.inspectLastError.message}`
+      );
+    }
     checks.push({
       name: "inspect.last_error",
-      ok: false,
+      ok: isStale,
       message: context.inspectLastError.message,
       details: {
         code: context.inspectLastError.code,
         retryable: context.inspectLastError.retryable,
-        at: context.inspectLastError.at
+        at: context.inspectLastError.at,
+        ...ageMs !== void 0 ? { age_ms: ageMs } : {}
       }
     });
   }
@@ -3578,6 +3614,7 @@ var buildDiagnosticReport = (sessionId, context = {}) => {
         loop_detected: context.recoveryMetrics.loopDetected
       } : {}
     } : void 0,
+    ...warnings.length > 0 ? { warnings } : {},
     notes: ["Diagnostics include runtime status; some checks may be stubbed."]
   };
   return report;
@@ -4271,6 +4308,7 @@ var DebuggerBridge = class {
       }
       state.attached = true;
       this.touch(tabId, state);
+      this.clearLastError();
       return { ok: true, result: { attached: true } };
     } catch (error) {
       const info = this.handleBridgeError(error);
@@ -4298,6 +4336,7 @@ var DebuggerBridge = class {
         return { ok: false, error };
       }
       this.markDetached(tabId);
+      this.clearLastError();
       return { ok: true, result: { attached: false } };
     } catch (error) {
       const info = this.handleBridgeError(error);
@@ -4311,7 +4350,7 @@ var DebuggerBridge = class {
       return attachResult;
     }
     try {
-      const response = await this.bridge.requestDebugger(
+      const runCommand = async () => await this.bridge.requestDebugger(
         "debugger.command",
         {
           tab_id: tabId,
@@ -4320,6 +4359,15 @@ var DebuggerBridge = class {
         },
         timeoutMs
       );
+      let response = await runCommand();
+      if (response.status === "error" && this.shouldRetryAfterStaleAttach(response.error)) {
+        this.markDetached(tabId);
+        const reattach = await this.attach(tabId);
+        if (!reattach.ok) {
+          return reattach;
+        }
+        response = await runCommand();
+      }
       if (response.status === "error") {
         const error = response.error ?? {
           code: "INSPECT_UNAVAILABLE",
@@ -4330,6 +4378,7 @@ var DebuggerBridge = class {
         return { ok: false, error };
       }
       this.touch(tabId, this.ensureTab(tabId));
+      this.clearLastError();
       return { ok: true, result: response.result };
     } catch (error) {
       const info = this.handleBridgeError(error);
@@ -4415,8 +4464,15 @@ var DebuggerBridge = class {
     this.lastError = error;
     this.lastErrorAt = (/* @__PURE__ */ new Date()).toISOString();
   }
+  clearLastError() {
+    this.lastError = void 0;
+    this.lastErrorAt = void 0;
+  }
   handleBridgeError(error) {
     if (error instanceof ExtensionBridgeError) {
+      if (error.code === "EXTENSION_DISCONNECTED") {
+        this.markAllDetached();
+      }
       const info2 = toDriveError(error);
       this.recordError(info2);
       return info2;
@@ -4429,6 +4485,18 @@ var DebuggerBridge = class {
     this.recordError(info);
     return info;
   }
+  shouldRetryAfterStaleAttach(error) {
+    if (!error) return false;
+    if (error.code !== "FAILED_PRECONDITION" && error.code !== "INSPECT_UNAVAILABLE") {
+      return false;
+    }
+    return error.message.toLowerCase().includes("not attached");
+  }
+  markAllDetached() {
+    for (const tabId of this.tabs.keys()) {
+      this.markDetached(tabId);
+    }
+  }
 };
 
 // packages/core/src/server.ts