feat(openai-codex): persist and send Fast priority mode - #1063
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughAdds OpenAI Codex service-tier types and provider schema support, persists tier settings, and conditionally propagates the priority tier to streaming, fallback SSE, and non-streaming Responses requests with corresponding tests. ChangesOpenAI Codex service tier
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant ApiHandlerOptions
participant OpenAiCodexHandler
participant OpenAIResponsesAPI
ApiHandlerOptions->>OpenAiCodexHandler: configured Codex service tier
OpenAiCodexHandler->>OpenAiCodexHandler: map Priority to SERVICE_TIER_KEY
OpenAiCodexHandler->>OpenAIResponsesAPI: streaming or non-streaming Responses request
Possibly related issues
Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Responsibility and scope
Adds backend/configuration support for persisting the OpenAI Codex speed preference and mapping Fast mode to the provider request priority field. Tests travel with the behavior in this PR.
Behavior
Priorityin the OpenAI Codex request.Explicit non-goals
webview-ui/changes and no user-facing selector; that is the next stack PR.Task.throttle.test.tsfix.Test evidence
Stack dependency
Replaces WebMad#2, which targeted the fork repository.
Summary by CodeRabbit
New Features
Bug Fixes