File: e2e/helpers/ui.ts:161 (and many sibling lines: 259, 264, 266, 268, 278, 288, 299, 303, 305, 307, 309, 322, 334, 380, 385)
Severity: tech-debt
Obvious? yes
switchTab clicks the tab then unconditionally sleeps 200ms. Per the repo's own guideline (e2e/README.md:102 — waitForTimeout "blocked by no-restricted-syntax"), helpers exempted via lint but slated to migrate to selector waits.
Fix: waitFor({state:'visible'}) on the destination panel selector. Same pattern repeats at the lines noted — wholesale migration is one sweep.
Filed by /general-audit @ 6404719 (2026-05-03). master: #567.
File:
e2e/helpers/ui.ts:161(and many sibling lines: 259, 264, 266, 268, 278, 288, 299, 303, 305, 307, 309, 322, 334, 380, 385)Severity: tech-debt
Obvious? yes
switchTabclicks the tab then unconditionally sleeps 200ms. Per the repo's own guideline (e2e/README.md:102—waitForTimeout"blocked by no-restricted-syntax"), helpers exempted via lint but slated to migrate to selector waits.Fix:
waitFor({state:'visible'})on the destination panel selector. Same pattern repeats at the lines noted — wholesale migration is one sweep.Filed by
/general-audit@6404719(2026-05-03). master: #567.