Replies: 8 comments 10 replies
|
Hello! Noticed the same issue with gemini-3-pro via google (see simple repro in session file). I have a ugly fix that works locally, but only for google. I can't tell if this is a temporary issue because the model is still in preview, but seeing that it happens for other model it might be worth harmonising how this is handle? wdyt ? :) |
|
@garymjr GLM uses the anthropic provider code, which was just fixed in 29379ea by converting unsigned thinking blocks to plain content blocks when submitted to the LLM. This affects thinking blocks in prematurely aborted LLM responses only, so a conversion to plain content blocks is fine. For Gemini, I'm not sure if we want to fix this on our end. That's something big G has to fix in their endpoint I think. I'm not bullish on hacking around issues in providers, as that will make our codebase harder to maintain, especially since those edge cases are hard to test. @aliou 's repro above for example doesn't repro for me at all after 10 tries (using both straight Google gen-ai endpoint, and Antigravity OAuth endpoint). @garymjr I also saw you made fixes to the openai-completions code. Can you explain those? Which model/provider was this necessary for? GLM doesn't use that code. |
|
@badlogic even after 29379ea i'm still seeing thinking blocks leaking into tool calls and causing the agent to stop working. This still seems to work though. cc @nicobailon @aliou |
|
[Sorry, unrelated to the topic, will delete it later] |
|
That's right. Should add login for known providers as well I suppose, which
just saves out the api key to auth.json.
…On Fri, Dec 26, 2025, 14:52 Aliou Diallo ***@***.***> wrote:
pi looks for the ZAI_API_KEY, I think /login is mainly for providers with
oauth flows. (I might have missed wether z.ai also as a oauth flow though)
—
Reply to this email directly, view it on GitHub
<#292 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD5QBGENR7HCGAR3JNUGB34DU4RRAVCNFSM6AAAAACP4BPLXKVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTKMZUG4YTEOI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
this seems to be fixed by 04a7647. closing this now. thanks all |
|
Thanks for your help!
…On Tue, Dec 30, 2025, 10:01 PM Gary Murray ***@***.***> wrote:
this seems to be fixed by 04a7647
<04a7647>.
closing this now. thanks all
—
Reply to this email directly, view it on GitHub
<#292 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD5QBHZJRE66MVL3KBPEO34ELRY7AVCNFSM6AAAAACP4BPLXKVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTKMZXG42TKOA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|


Uh oh!
There was an error while loading. Please reload this page.
I noticed with GLM 4.7 using the z.ai coding plan that interleaved thinking seems to leak into tool calls and causes the agent to stop. Here's an example of what this looks like.
I have a fix that seems to be working locally, but I'm running it a little longer to make sure it's the correct solution. Not sure if anyone else has seen this and started working on a solution as well.
Thanks for pi, I'm really enjoying the experience over other agents.
All reactions