microlinkhq / browserless / 28009818944 / 5

81%

master: 81%

push

github

fix(lighthouse): retry context teardown masked as perf-mark error (#801)

When the browser context is torn down mid-run, lighthouse's marky teardown
calls performance.measure for a phase whose start mark never fired, throwing
`SyntaxError: The "start lh:…" performance mark has not been set`. That message
masks the real cause (a destroyed context), so it escaped the retry machinery
as an opaque error — surfacing as a misleading 500 and Sentry noise instead of
being retried.

Map the perf-mark error back to errors.contextDisconnected() so withPage
recreates the context and retries the run; persistent failures now surface as
EBRWSRCONTEXTCONNRESET (the accurate cause) rather than five phantom SyntaxError
groups.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

10 of 13 branches covered (76.92%)

Branch coverage included in aggregate %.

46 of 46 relevant lines covered (100.0%)

4.83 hits per line