feat(openai-codex): add 1.5x Fast mode - #1038
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Addressed all review comments in
Validation after the changes:
|
WebMad
left a comment
There was a problem hiding this comment.
Reviewed the current head. I found no actionable or blocking issues.
The provider schema and persistence path preserve both Standard and Fast preferences, Standard is omitted from outgoing Codex requests, and Fast is consistently mapped to service_tier: "priority" for SDK streaming, manual SSE fallback, and non-streaming requests. The settings UI uses the supplied configuration buffer/callback and clearly communicates the quota trade-off.
I also ran the focused type/schema, provider/persistence, and WebView tests locally: 105 tests passed across 6 test files. git diff --check also passed.
|
This combined draft has been superseded by a three-PR stack split by responsibility:
Tests travel with the behavior in each layer. The unrelated |
Summary
service_tier: "priority"for SDK streaming, manual SSE fallback, and non-streaming Codex requestsservice_tierfor Standard requestsTest coverage
Validation
packages/types,src, andwebview-uigit diff --checkpassedCloses #1037