Files
Kpa-clawbot 72161ba8fe perf: replace 169 blind sleeps with Playwright waits in coverage script
Remove all 169 waitForTimeout() calls (totaling 104.1s of blind sleeping)
from scripts/collect-frontend-coverage.js:

- Helper functions (safeClick, safeFill, safeSelect, clickAll, cycleSelect):
  removed 300-400ms waits after every interaction — Playwright's built-in
  actionability checks handle waiting for elements automatically
- Post-navigation waits: removed redundant sleeps after page.goto() calls
  that already use waitUntil: 'networkidle'
- Hash-change navigations: replaced waitForTimeout with
  waitForLoadState('networkidle') for proper SPA route settling
- Toggle/button waits: removed — event handlers execute synchronously
  before click() resolves
- Post-evaluate waits: removed — evaluate() is synchronous

Local benchmark (Windows, sparse test data):
  Before: 744.8s
  After:  484.8s (35% faster, 260s saved)

On CI runner (ARM Linux with real mesh data), savings will be
proportionally better since most elements exist and the 104s
of blind sleeping was the dominant bottleneck.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-03-26 23:02:03 -07:00
..