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
|
Review statusThanks for contributing. This comment tracks the review sequence and the next action. Current step: Resolve the merge conflicts. The review sequence resumes after the branch is mergeable. 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. |
|
This PR has been awaiting author changes for 14 days and will be automatically closed in 7 days. Please address the review comments or leave a comment if you need more time. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Closing due to author inactivity after requested changes. Feel free to reopen once the requested changes have been addressed. |
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.