Skip to content

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

Merged
deathbeam merged 10 commits into
CopilotC-Nvim:mainfrom
rakotomandimby:feat/1442-responses-api-gpt5codex
Oct 24, 2025
Merged

feat(provider): Fix #1442 - Support OpenAI Responses API#1463
deathbeam merged 10 commits into
CopilotC-Nvim:mainfrom
rakotomandimby:feat/1442-responses-api-gpt5codex

Conversation

@rakotomandimby
Copy link
Copy Markdown
Contributor

…dvertizing supporting only it

… for models advertizing supporting only it
Comment thread lua/CopilotChat/config/providers.lua Outdated
Comment thread lua/CopilotChat/client.lua Outdated
Comment thread lua/CopilotChat/config/providers.lua Outdated
@deathbeam
Copy link
Copy Markdown
Collaborator

Otherwise looks pretty good, thanks!

Comment thread lua/CopilotChat/config/providers.lua Outdated
Comment thread lua/CopilotChat/config/providers.lua Outdated
@rakotomandimby
Copy link
Copy Markdown
Contributor Author

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

@rakotomandimby
Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Collaborator

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

@deathbeam
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

@deathbeam ,

My knowledge was :

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

@rakotomandimby
Copy link
Copy Markdown
Contributor Author

@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.

@deathbeam deathbeam merged commit 746a697 into CopilotC-Nvim:main Oct 24, 2025
4 checks passed
@deathbeam
Copy link
Copy Markdown
Collaborator

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

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.

2 participants