feat: add OpenAI Responses API route#2072
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise 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)
Comment |
1ed1def to
2036232
Compare
|
Review-fix sweep update: Summary:
Verification:
Remaining blocker I could not clear from the contributor account:
Confidence: high on the code/check fixes. Label checker needs a maintainer/vetter to apply the required labels. |
|
Review-fix sweep found no actionable code blockers from the contributor side. Current blockers appear repository-maintainer gated:
No new code changes made in this sweep. |
Review-Fix SummaryApplied 3 targeted fixes to the OpenAI Responses API route (
Pre-existing safe code confirmed (no changes needed):
Verification
Second-Agent Review (hermes chat -Q / MiniMax-M2.7)Verdict: CLEAN (1 minor non-blocking note) Minor note (non-blocking): No blocking findings. The type mismatch in Confidence: High. All blocking findings from the prior review are resolved. Tests pass, lint clean, second-agent confirms CLEAN. |
Signed-off-by: maxpetrusenkoagent <max.petrusenko.agent@gmail.com>
- Pass enable_interactive through add_v1_responses_route chain to post_responses_api_endpoint and _responses_stream (fixes interactive extension flag regression) - Change WORKFLOW_ERROR status_code from 422 to 500 to match OpenAI API contract (server error, not input validation failure) - Minor formatting fix: re-wrap StreamingResponse call to pass ruff E501 line-length check Signed-off-by: maxpetrusenkoagent <max.petrusenko.agent@gmail.com>
4aab349 to
fa75198
Compare
|
Label Checker is failing because this PR is missing category and breaking-change labels. Suggested labels for this change: |
Summary
Fixes #1278
Verification