fix(e2e): make authenticated browser stack reproducible

[seonghobae]

Outcome

This bounded repair is complete and was normally merged into #831.

• exact repair head: b013b8b7bc717c543b0d2ca8458c60dcc1f2144e
• exact parent before merge: feat(reports): preserve grouping comparison cross-share delta for serialized successor #831 ac50985c681bb3184af53803e460ef918d601076
• authoritative Tests: 34612342413 — terminal SUCCESS on the unchanged repair head
• merge commit on feat(reports): preserve grouping comparison cross-share delta for serialized successor #831: 93e1b4fb5601b5733e9a4b0f7b4a9f63ff44354e

Causal repairs retained

The lane repaired the clean-checkout authenticated browser path without provider/model substitution, copied contextual-orchestrator source, timeout inflation, test skipping, or gate weakening:

• repository-local contextual-orchestrator is explicit/optional while backend/MCP remain Null/fail-closed by default;
• disposable synthetic seeding uses the committed dependency lock while the long-running backend runtime stays lean;
• the synthetic Keycloak issuer is pinned to the public product-auth boundary;
• Playwright Pixel-7 setup uses only supported browser-context fields instead of spreading worker-scoped device options;
• browser artifacts/logs are retained only when the browser step itself fails;
• locale options are enumerated with a real browser callback;
• the existing semantic WorkspaceNav and language tools remain reachable below 768px with touch-sized responsive layout, closing the buyer-visible gap caused by hiding the nav for a drawer that did not exist.

The exact-head SUCCESS covered the PostgreSQL full suite plus frontend lint/tests, production build, Storybook, Chromium installation, authenticated product startup, synthetic seed, persisted cross-share fixture, desktop/responsive pointer and keyboard/focus checks, locale iteration, and Pixel-7 touch acceptance.

No predecessor receipt transfers to the moved #831 head. Fresh #831 exact-head validation is tracked on merge result 93e1b4fb... before descendant semantic reconstruction.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 6569f7ed-b107-42d7-818f-cbadccd84e8b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[seonghobae]

Owner-boundary note: protected-main foundation #899 is the canonical LineageWeave→contextual-orchestrator separation lane and removes the repository-local orchestrator wrapper entirely. This #831-stack repair does not supersede or copy that owner implementation; it only makes the existing historical stack honor its already-documented optional/fail-closed contract. When this serialized report stack later converges with #899 or its accepted successor, #899's external-service boundary wins and the transitional llm profile/wrapper must disappear rather than become a competing canonical runtime.