Skip to content

feat: stream tool-response messages between generate turns#298

Open
pavelgj wants to merge 1 commit into
mainfrom
pj/generate-stream-tool-response
Open

feat: stream tool-response messages between generate turns#298
pavelgj wants to merge 1 commit into
mainfrom
pj/generate-stream-tool-response

Conversation

@pavelgj

@pavelgj pavelgj commented Jun 10, 2026

Copy link
Copy Markdown
Member

Stream the tool-response message mid-turn when the generate loop will continue, so clients like agents can observe tool execution. The tool-response message occupies the slot immediately after the model message, matching the JS implementation.

Update streaming tests to expect the additional tool-response chunk and adjusted message indices.

Stream the tool-response message mid-turn when the generate loop will
continue, so clients like agents can observe tool execution. The
tool-response message occupies the slot immediately after the model
message, matching the JS implementation.

Update streaming tests to expect the additional tool-response chunk and
adjusted message indices.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request adds support for streaming tool response messages mid-turn when streaming is requested, allowing clients to observe tool execution. It also updates the corresponding tests in generate_stream_test.dart to account for the additional tool-response chunk and verify correct message indices. I have no feedback to provide as there are no review comments.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

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