Abort capability run when nav dumps browser on chrome-error page

When a navigation fails with a transport-level error (e.g.
ERR_HTTP2_PROTOCOL_ERROR), the browser ends up on
chrome-error://chromewebdata/ and every subsequent click/type
runs against that error page. Previously the executor kept going,
wasting ~10s on selector probes + AI healing calls, and returned
a misleading "site_changed" result with an empty data object.

Now: after any failed step, check page.url(); if it's a chrome-error
page, stop immediately, capture a screenshot, and return a clean
'transient' errorType with "Site unreachable: <original error>".
Skips extraction too — nothing to extract from chrome://.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: mreider

core/browser/playwright-engine.ts

@@ -108,6 +108,7 @@ export class PlaywrightEngine {
       this.addLog(`Optimized: ${resolvedActions.length} actions → ${optimized.length} (removed waits + duplicate navs)`)
 
       let failedSteps = 0
+      let unreachableError: string | null = null
       for (let i = 0; i < optimized.length; i++) {
         const action = optimized[i]
         this.addLog(`Step ${i + 1}/${optimized.length}: ${action.type} ${action.selector || action.url || ''} ${action.label ? '(' + action.label + ')' : ''}`.trim())
@@ -118,6 +119,29 @@ export class PlaywrightEngine {
         } catch (err: any) {
           failedSteps++
           this.addLog(`  -> FAILED (skipping): ${err.message}`)
+
+          // If a navigation dumped us onto Chrome's network-error page, no later
+          // step can succeed (DOM is chrome-error://chromewebdata/). Bail instead
+          // of spending 10+ seconds running selectors against the error page.
+          if (this.page.url().startsWith('chrome-error://')) {
+            unreachableError = err.message
+            const skipped = optimized.length - i - 1
+            this.addLog(`  -> Site unreachable; aborting${skipped > 0 ? ` remaining ${skipped} step(s)` : ''}.`)
+            break
+          }
+        }
+      }
+
+      if (unreachableError) {
+        const screenshotBuffer = await this.page.screenshot({ type: 'png' })
+        return {
+          success: false,
+          data: {},
+          error: `Site unreachable: ${unreachableError}`,
+          errorType: 'transient',
+          durationMs: Date.now() - startTime,
+          screenshot: screenshotBuffer.toString('base64'),
+          log: this.log
         }
       }
 

tests/core/playwright-engine.test.ts

@@ -299,6 +299,24 @@ describe('PlaywrightEngine', () => {
       const result = await engine.execute(actions, [], {}, [])
       expect(result.errorType).toBe('site_changed')
     })
+
+    it('aborts remaining steps when nav dumps browser on chrome-error page', async () => {
+      mockPage.goto.mockRejectedValueOnce(new Error('page.goto: net::ERR_HTTP2_PROTOCOL_ERROR at https://united.com/'))
+      mockPage.url.mockReturnValue('chrome-error://chromewebdata/')
+      const actions = [
+        buildNavigateAction({ url: 'https://united.com' }),
+        buildClickAction({ locators: [{ strategy: 'css', value: '#nav' }] }),
+        buildNavigateAction({ url: 'https://united.com/account' })
+      ]
+      const result = await engine.execute(actions, [], {}, [])
+
+      expect(mockPage.goto).toHaveBeenCalledTimes(1)
+      expect(result.success).toBe(false)
+      expect(result.errorType).toBe('transient')
+      expect(result.error).toMatch(/Site unreachable.*ERR_HTTP2_PROTOCOL_ERROR/)
+      expect(result.log.some(l => l.includes('Site unreachable; aborting remaining 2 step(s)'))).toBe(true)
+      expect(result.screenshot).toBeDefined()
+    })
   })
 
   // ════════════════════════════════════════════════════════════════════════════