Skip to content

[Tool Calls] Publish tool_call_started over assistant SSE#23759

Open
philipperolet wants to merge 2 commits intomainfrom
pr/404-publish-tool-call-started
Open

[Tool Calls] Publish tool_call_started over assistant SSE#23759
philipperolet wants to merge 2 commits intomainfrom
pr/404-publish-tool-call-started

Conversation

@philipperolet
Copy link
Copy Markdown
Contributor

Description

Follows #23634, #23636, #23695, and #23703.

This publishes the normalized tool_call_started event through assistant SSE, adds the shared event typing on the front and SDK sides, and documents the private SSE payload in swagger.

The frontend accepts the new event explicitly but still ignores it in rendering for now; the pending-tool-call UI comes in the next PR.

Risks

Blast radius: assistant message SSE consumers and type contracts for agent message events.
Risk: low

Deploy Plan

  • deploy front

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
playground Ignored Ignored Preview Apr 4, 2026 4:36pm
storybook Ignored Ignored Preview Apr 4, 2026 4:36pm

Request Review

Copy link
Copy Markdown
Contributor

@dust-agent dust-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coding Rules LGTM \o/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 4, 2026

Fails
🚫

Files in **/sdks/js/ have been modified. Changing the types defined in the SDK could break existing client.
Additions (new types, new values) are generally fine but removals are NOT OK : it would break the contract of the Public API.
Please add the sdk-ack label to acknowledge that your are not breaking the existing Public API contract.

Generated by 🚫 dangerJS against 1744d02

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.

1 participant