Skip to content

feat: implement Responses API #437

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

AdrianMiska
Copy link

Q A
Bug fix? no
New feature? yes
BC breaks? no
Related Issue TBD

Describe your change

This PR implements the new Responses API.

Note: I've only tested my use cases happy paths so far. Streaming support is still missing, so this isn’t the final state. Depending on feedback, this might go through another iteration or two, or it may be merged with the remaining bits addressed later. Just wanted to get this out there to share progress :)

What problem is this fixing?

The library currently lacks support for the Responses API, which means features like built-in tools weren't usable. This change adds initial support and sets the groundwork for full integration.

@Komdosh
Copy link

Komdosh commented Apr 22, 2025

Hope it will be merged soon!

@aallam
Copy link
Owner

aallam commented Apr 23, 2025

Thank you for contribution 🙌 ! let's merge your progress for now, and add streaming in a separate PR.

Copy link
Owner

@aallam aallam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please fix lint issues ? thanks!

…Responses-API

# Conflicts:
#	openai-client/src/commonMain/kotlin/com.aallam.openai.client/OpenAI.kt
@AdrianMiska AdrianMiska requested a review from aallam April 23, 2025 09:43
@AdrianMiska
Copy link
Author

Is there something I need to do to fix the workflow?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants