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. |
Related GitHub Issue
Related to #1963. UI follow-up #1965 closes the feature issue once all parts are merged.
Dependencies and merge order
Depends on transport #1964 and reasoning-history fix #1966. Those two PRs can be reviewed and merged independently. Merge both first, then update this branch from the main branch before merging this PR. UI #1965 follows this activation PR.
This PR is a draft while the dependencies are outstanding. It intentionally contains only the activation changes, not copies of either prerequisite. Standalone type/build/test checks can fail until the transport dependency is present on the base branch.
Description
The provider-activation portion extracted from the original #1964: 9 files, 216 added / 9 removed lines.
packages/types, including both schema exports and validation tests.ProviderSettingsManager.spec.ts.ClineProvider.spec.ts.openai-codex.tsonly when the saved preference is enabled.There are no transport implementation, transport dependency, reasoning-history, or UI changes here. The nine files are byte-for-byte identical to their original unsplit versions. All three backend PRs and UI #1965 together reproduce the original Git tree exactly.
Test Procedure
Because this PR requires its prerequisites, validation was run against the verified combined backend state:
Total combined backend validation: 570 tests passed. The separate transport-only state also passed 170 API tests and the repository-wide unused-code gate; the isolated reasoning fix passed nine tests.
With UI #1965 present, its 18 focused tests and webview type check also passed. Standalone CI for this draft is not claimed until both prerequisites merge and this branch is updated.
Manual verification after the UI is also 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.