fix(tracing): do not hang on unresponsive page when capturing snapshots

[Yury Semikhatsky (yury-s)]

Summary

• Race the DOM snapshot in _captureSnapshot against progress so a busy page cannot block before/after hooks forever.
• Make await-must-use-progress lint rule see through optional chaining, which hid this case.
• Version the progress eslint plugin by its source hash so rule changes invalidate the eslint cache.
• Fix react-hooks/exhaustive-deps warning in trace viewer workbench.

Fixes #42903

[Pavel Feldman (pavelfeldman)]

Nothing works on these pages, this test and change do not make sense.

[github-actions]

Test results for "MCP"

3 failed
❌ [firefox] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent) @mcp-ubuntu-latest-firefox
❌ [firefox] › mcp/annotate.spec.ts:446 › should switch screencast to -s session on show --annotate @mcp-windows-latest-firefox
❌ [webkit] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent) @mcp-ubuntu-latest-webkit

8735 passed, 1480 skipped

---

Merge workflow run.

[github-actions]

Test results for "tests 1"

4 flaky

⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@frozen-time-library-chromium-linux`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`

52302 passed, 1241 skipped

---

Merge workflow run.

[github-actions]

Hi, I'm the Playwright bot and I took a first look at the CI failures here.

🟢 The failures look like pre-existing flakes, not this PR

The latest "tests 1" run only has flaky tests, which retries rescued. The three MCP failures are all known flaky tests that also fail on main and on unrelated PRs. This PR touches tracing snapshot capture, the trace viewer workbench and an eslint rule, and none of the failing MCP tests exercise that code.

Details

I checked the latest reports on head a7c5f54: MCP run and tests 1 run. I then looked up each failing test's history in the aggregated test-results DB, after refreshing it with the last 2 days of runs.

Pre-existing flake / infra

• [firefox] and [webkit] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent): pre-existing flake. The test fails in the same way (the expect(...).toEqual on the "42" result doesn't match) on Ubuntu across chromium, firefox and webkit. That includes main at 1996640f, ff19f730, 3847b985 and 22dc1227, plus unrelated PRs fix(route): do not drop requests matching case-insensitive URLPattern #42920, fix(chromium): report numpad location on keyup #42913, feat(chromium): roll to r1247 #42915, feat(reporter): and printOnlyFailures and --reporter-only-failures #42766, test(locators): parse nested locators in locator(), and() and or() #42899, fix(html reporter): print show-report path relative to npm invocation dir #42897 and feat(test-runner): add --shuffle to run tests in random order #42877, several of them in the last day. The SSE transport doesn't go through tracing.
• [firefox] › mcp/annotate.spec.ts:446 › should switch screencast to -s session on show --annotate (Windows): pre-existing flake. On mcp-windows-latest-firefox it failed 78 of 709 runs (~11%). 37 of those failures were on main and the rest were spread across 38 other PRs.
• "tests 1" had 4 flaky tests and no failures: video.spec.ts:762, beforeunload.spec.ts:130 and two browsercontext-cookies-third-party.spec.ts tests. Retries rescued all four, so they don't block the PR.

_{Triaged by the Playwright bot - agent run}