fix: retry on No window with id CDP error (#892)

* fix: retry on No window with id CDP error

* test(browser): lock transient window-id retry behavior

---------

Co-authored-by: Yun Xiao <yunxiao@agents.com>
Co-authored-by: jackwener <jakevingoo@gmail.com>

Author: 3 people

extension/src/cdp.ts

@@ -92,8 +92,10 @@ export async function ensureAttached(tabId: number, aggressiveRetry: boolean = f
             break; // Don't retry if URL became un-debuggable
           }
         } catch {
+          // Tab is gone — don't fail early here.
+          // Later retry layers can re-resolve a fresh automation tab/window.
           lastError = `Tab ${tabId} no longer exists`;
-          break;
+          // Don't break; fall through to retry
         }
       }
     }

src/browser/errors.test.ts

@@ -0,0 +1,13 @@
+import { describe, expect, it } from 'vitest';
+
+import { isTransientBrowserError } from './errors.js';
+
+describe('isTransientBrowserError', () => {
+  it('treats "No window with id" as transient', () => {
+    expect(isTransientBrowserError(new Error('No window with id: 123'))).toBe(true);
+  });
+
+  it('does not classify unrelated browser errors as transient', () => {
+    expect(isTransientBrowserError(new Error('Permission denied'))).toBe(false);
+  });
+});

src/browser/errors.ts

@@ -20,6 +20,7 @@ const TRANSIENT_ERROR_PATTERNS = [
   'no longer exists',
   'CDP connection',
   'Daemon command failed',
+  'No window with id',
 ] as const;
 
 /**