Skip to content

feat(ai-proxy): add support for OpenAI Responses API #6513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 23, 2025

Conversation

iutx
Copy link
Member

@iutx iutx commented Apr 23, 2025

What this PR does / why we need it:

  • Fix Missing body, will cause method like GET or DELETE.. error.
  • Add new route and filters for handling Responses API requests- Implement context filter to extract user prompts from API input
  • Update request path constants to include new Responses API endpoint- Add unit tests for new context filter functionality

Which issue(s) this PR fixes:

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English add support for OpenAI Responses API
🇨🇳 中文 支持 OpenAI Responses API

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

- Add new route and filters for handling Responses API requests- Implement context filter to extract user prompts from API input
- Update request path constants to include new Responses API endpoint- Add unit tests for new context filter functionality
@erda-bot erda-bot requested a review from sfwn April 23, 2025 06:57
Copy link

codecov bot commented Apr 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 15.17%. Comparing base (9ee67c6) to head (7c66b9b).
Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6513   +/-   ##
=======================================
  Coverage   15.16%   15.17%           
=======================================
  Files        3683     3683           
  Lines      377077   377077           
=======================================
+ Hits        57202    57209    +7     
+ Misses     314426   314417    -9     
- Partials     5449     5451    +2     
Flag Coverage Δ
by-github-actions 15.17% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@iutx iutx marked this pull request as ready for review April 23, 2025 07:16
@iutx iutx requested a review from sfwn April 23, 2025 08:10
@sfwn
Copy link
Member

sfwn commented Apr 23, 2025

/approve

@erda-bot erda-bot merged commit 3cdab4d into erda-project:master Apr 23, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants