Skip to content

add tool name to tool call delta streaming events#1222

Merged
joshua-mo-143 merged 2 commits into0xPlaygrounds:mainfrom
caido:tl-fix-tooldelta-missing-funcname
Jan 5, 2026
Merged

add tool name to tool call delta streaming events#1222
joshua-mo-143 merged 2 commits into0xPlaygrounds:mainfrom
caido:tl-fix-tooldelta-missing-funcname

Conversation

@bebiksior
Copy link
Contributor

@bebiksior bebiksior commented Jan 5, 2026

Adds tool name to ToolCallDelta streaming events so apps can display which tool is being called during streaming

Uses a ToolCallDeltaContent enum with Name(String) and Delta(String) variants since providers send either the name (first event) or argument deltas (subsequent events)

pub enum ToolCallDeltaContent {
    Name(String),
    Delta(String),
}

RawStreamingChoice::ToolCallDelta {
    id: String,
    content: ToolCallDeltaContent,
}

ToolCallDelta now includes the tool name so apps can display which tool is being called during streaming
@Sytten
Copy link
Contributor

Sytten commented Jan 5, 2026

@joshua-mo-143 FYI @bebiksior here is working for caido, I support this PR. This is a breaking change though so let us know if you prefer to have a nullable name instead but this new enum is IMO better to convey the intent of the delta.

Copy link
Collaborator

@joshua-mo-143 joshua-mo-143 left a comment

Choose a reason for hiding this comment

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

lgtm!

@joshua-mo-143 joshua-mo-143 added this pull request to the merge queue Jan 5, 2026
Merged via the queue into 0xPlaygrounds:main with commit fd9da14 Jan 5, 2026
5 checks passed
@github-actions github-actions bot mentioned this pull request Jan 5, 2026
ThomasMarches pushed a commit to ThomasMarches/rig that referenced this pull request Feb 13, 2026
* add tool name to tool call delta streaming events

ToolCallDelta now includes the tool name so apps can display which tool is being called during streaming

* improve ToolCallDelta structure
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.

3 participants