Repository navigation
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
Review statusThanks for contributing. This comment tracks the review sequence and the next action. Current step: Fix the failing required CI checks; awaiting-maintainer requires CI and automated review completion. Review-state labels are managed by this workflow; do not edit them manually. |
|
Dependency updated after the backend split: this UI PR now depends directly on provider activation #1967, which requires transport #1964 and reasoning fix #1966. Merge activation before this UI to avoid exposing a checkbox without runtime support. The UI branch and all visual baselines are unchanged. |
Related GitHub Issue
Closes #1963 after all prerequisite PRs are merged.
Dependencies and merge order
Depends on provider activation #1967, which in turn requires transport #1964 and reasoning-history fix #1966.
#1966 and #1964 can merge independently → #1967 → this PR (#1965). Update this branch from the main branch after activation merges. Keep this PR as a draft until then so users never receive a checkbox that has no runtime effect.
Standalone type/build checks can fail before the prerequisites merge because the new shared provider setting and default are not yet present on the base branch.
Description
The webview-only portion of the original #1964: 28 files, all under
webview-ui.There are no backend, shared-package, or lockfile changes here. The webview subtree is byte-for-byte identical to the original unsplit PR. Combining #1966, #1964, #1967, and this PR reproduces the complete original Git tree exactly, including binary visual baselines.
Test Procedure
Validation rerun against the combined backend + UI state:
OpenAICodex.spec.tsxandSettingsView.unsaved-changes.spec.tsx.The original light/dark Playwright baselines are preserved unchanged and were not regenerated during either split. Standalone CI is not claimed until the prerequisites are merged and this branch is updated.
Manual verification after all parts are applied:
Local validation used Node 24.7.0 / pnpm 10.8.1; the repository requests Node 22.23.1.
Pre-Submission Checklist
No changeset or changelog entry is included. The original unsplit state is preserved on a backup branch in the source fork.