Merge "tool_call" and "tool_call_update" sessionUpdate events #177
Replies: 3 comments 1 reply
-
|
Yeah I think we can simplify here. I don't think we can drop either method, but maybe we can at least have them be aliases? The only issue is that clients currently see tool_call as "replace all fields" and tool call update as "replace fields that are present" |
Beta Was this translation helpful? Give feedback.
-
|
Just wanted to provide a +1 to this unification, I'm coming from zed-industries/claude-code-acp#122 after running into the same issue as the OP. |
Beta Was this translation helpful? Give feedback.
-
|
another subtlelty is that, at least with Gemini, if there is no pre-premission-approval, you wont get a tool-call event before a tool-call-update, the initial call info is embedded in the request permission object. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Currently the only difference is that
tool_callrequires atitleInstead, specification should say that
agent MUST provide a title on the first event about tool callCompatibility
Beta Was this translation helpful? Give feedback.
All reactions