[kcrmin]

What and why

BasicEditorInteraction.test.tsx checked toHaveFocus() right after findByRole returned the position-cap field. StrategyViews.tsx moves focus to that field inside requestAnimationFrame, so the field exists one frame before it is focused and the assertion only passed when the frame happened to fire first.

The test now waits for focus itself with waitFor, which is deterministic regardless of frame timing. No product code changes.

Verified

• Probe (not committed): with requestAnimationFrame delayed by 60 ms, the old assertion fails and the waitFor assertion passes.
• pnpm exec vitest run src/BasicEditorInteraction.test.tsx 20 consecutive runs (Node 24): 20 passed, 0 failed (32 tests each)
• node --test scripts/test-github-action-pins.mjs scripts/test-real-api-integration-workflow.mjs: passed
• pnpm typecheck: clean
• pnpm exec vitest run: 58 files, 688 tests passed
• pnpm build && node scripts/assert-production-bundle.mjs: passed
• CI=1 pnpm e2e: 26 passed

Checklist

• Test-only change
• Assertion still requires the exact invalid field to receive focus
• Stable across 20 consecutive runs