feat: switch xai provider to responses API#18175
Conversation
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
615bc30 to
2b8a267
Compare
2b8a267 to
80d2d37
Compare
Co-authored-by: Jaaneek <jankiewiczmilosz@gmail.com>
|
@Jaaneek are you sure it's fixed? It gives 404 just because our proxy doesn't support /chat/completions api. |
|
@nikolaibogdanov will take a look within next 2 hours |
|
@nikolaibogdanov I can't reproduce cc @rekram1-node can you? I tested ACP via zed with multiple different models from xAI on newest dev branch |
|
Testing rn actually haha |
|
funny seeing the msg come in live |
|
@nikolaibogdanov I cannot reproduce either, the responses api is definitely being used, perhaps you are on an older opencode version? If u installed using brew make sure u used: |
|
Ok, you are right, it works. |
Co-authored-by: Jaaneek <jankiewiczmilosz@gmail.com>
Co-authored-by: Jaaneek <jankiewiczmilosz@gmail.com>
Issue for this PR
Closes #18179 #3890
Type of change
What does this PR do?
Switches xAI provider from completions API to responses API. The reason for this change is to use 'reasoning.encrypted_content' with reasoning models. This should improve performance in long multi turn conversations.
How did you verify your code works?
I did a lot of manual testing also told agent to test the changes e2e. I tried reading files, writing code, opening images etc..
Screenshots / recordings
Checklist
If you do not follow this template your PR will be automatically rejected.