File: e2e/helpers/touch.ts:68
Severity: tech-debt
Obvious? no
After dispatching touchend, real-time longPress sleeps 300ms for "settle" with no observable condition. Pair test (longPressWithClock at line 137) uses page.clock.runFor(300) — proves a clock-driven path exists.
Fix: migrate the real-time variant to wait on the post-press outcome (e.g. .mobile-action-sheet visibility) rather than a fixed timer; otherwise this guarantees a 300ms tax per call and risks flake on slow runners.
Filed by /general-audit @ 6404719 (2026-05-03). master: #567.
File:
e2e/helpers/touch.ts:68Severity: tech-debt
Obvious? no
After dispatching
touchend, real-timelongPresssleeps 300ms for "settle" with no observable condition. Pair test (longPressWithClockat line 137) usespage.clock.runFor(300)— proves a clock-driven path exists.Fix: migrate the real-time variant to wait on the post-press outcome (e.g.
.mobile-action-sheetvisibility) rather than a fixed timer; otherwise this guarantees a 300ms tax per call and risks flake on slow runners.Filed by
/general-audit@6404719(2026-05-03). master: #567.