fix(e2e): stabilize semantic search fallback - #822
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Blocked merge diagnosis — blocked |
GitNexus Review · PR #822GitNexus reviewed 3 changed files. 🤖 Agent context — 4 symbols · 3 filesSummaryThis appears to be a small, self-contained end-to-end test stabilization change, with accompanying planning documentation and no detected downstream reach. 🟢 LOW blast radius. An end-to-end semantic-search test change in The code change is concentrated in No high or critical risk files, cross-repo consumers, or affected flows were detected.
What changedSymbol Changes (4)
Changed Files (3)
Full detail lives in the GitNexus check run for this commit. |
Up to standards ✅🟢 Issues
|

Summary
Stabilize the semantic-search fallback E2E test: block the app service worker so Playwright routing intercepts model requests, fulfill model/CDN requests with deterministic 404s, and assert the visible lexical TRIZ result. Set a 10s assertion timeout inside a 15s per-test timeout. No production semantic-search or workflow code changed.
Related context: #751 (this PR only changes test reliability).
Type of change
fix:isolated E2E reliability issueTesting
env -u NO_COLOR pnpm exec playwright test e2e/semantic-search.spec.ts --project=chromium --repeat-each=3 --retries=0— 9 passed (29.8s).env -u NO_COLOR pnpm exec playwright test --retries=0 --grep-invert="probe: console timeline"— 600 passed, 4 skipped (8.8m), all four projects.env -u NO_COLOR ./scripts/quality_gate.sh— passed.env -u NO_COLOR pnpm run build— passed.The untracked
e2e/zz-probe.spec.tsremains untouched and is not part of this PR. A pre-existing React hydration mismatch inHomeViewmotion styles is documented in plans/151; it is not suppressed by this change. Scheduled run 36112610311 confirmed the nightly E2E job ran on its pre-change main revision; this branch's PR checks are the validation for this patch.Risk
Low. Test-only behavior and planning evidence; no runtime application change.
📝 Summary by GitNexus
Summary
This appears to be a small, self-contained end-to-end test stabilization change, with accompanying planning documentation and no detected downstream reach.
🟢 LOW blast radius. An end-to-end semantic-search test change in
e2e/semantic-search.spec.tswith no graph-detected dependents or affected execution flows.The code change is concentrated in
e2e/semantic-search.spec.ts, while the remaining changed symbols are documentation sections inplans/149-nightly-full-viewport-e2e-2026-09-24.mdandplans/151-quality-gate-lint-cache-and-nightly-guard-2026-09-24.md. Review the semantic-search test behavior first, then verify that the follow-up documentation remains aligned with the test scope.No high or critical risk files, cross-repo consumers, or affected flows were detected.
Added by GitNexus for PR #822. Edit freely — this block is replaced on the next review, everything above it is left untouched.