Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(provider): Fix #1442 - Support OpenAI Responses API#1463

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@rakotomandimby
Copy link
Contributor

…dvertizing supporting only it

@deathbeam
Copy link
Collaborator

Otherwise looks pretty good, thanks!

@rakotomandimby
Copy link
ContributorAuthor

Hello@deathbeam ,
Hope you are healthy.
Is there anything left I must do for the approbation of this PR?
Regards

@rakotomandimby
Copy link
ContributorAuthor

The last commit above is just a catchup to the main branch. No changes made my side. If there is anything I can do to make this PR merged, do not hesitate. Regards.

@deathbeam
Copy link
Collaborator

Sorry did not had much time to look at this lately, I will try to review tomorrow

@deathbeam
Copy link
Collaborator

https://platform.openai.com/docs/guides/function-calling hmm so i went through this but it looks the wawy you are handling tool calls here is very different from the official documentation, does the tool calling part work?

@rakotomandimby
Copy link
ContributorAuthor

@deathbeam ,

My knowledge was :

I am sorry to see they have documentation divergence...

@rakotomandimby
Copy link
ContributorAuthor

@deathbeam , I pushed a fix for tool calling. You were right there were a problem. I think it is good now. Feel free to review and comment.

@deathbeamdeathbeam merged commit746a697 intoCopilotC-Nvim:mainOct 24, 2025
4 checks passed
@deathbeam
Copy link
Collaborator

Look fine for now thanks a lot. sorry it took me a while to review

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@deathbeamdeathbeamAwaiting requested review from deathbeam

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rakotomandimby@deathbeam

[8]ページ先頭

©2009-2025 Movatter.jp