Skip to content

Group consecutive tool calls into a shared render block#190

Open
annzhang-db wants to merge 4 commits intodatabricks:mainfrom
annzhang-db:parallel-tool-group-rendering
Open

Group consecutive tool calls into a shared render block#190
annzhang-db wants to merge 4 commits intodatabricks:mainfrom
annzhang-db:parallel-tool-group-rendering

Conversation

@annzhang-db
Copy link
Copy Markdown

Summary

  • Adds groupConsecutiveToolSegments to collapse runs of consecutive dynamic-tool segments into a single tool-group render block
  • Replaces the inline partSegments.map render loop with renderBlocks.map, routing tool groups to a new MessageToolGroup component
  • Extracts all tool rendering logic (MCP approval branch + regular tool branch) into a ToolPartRenderer component

Test plan

  • Verify single tool calls render identically to before
  • Verify consecutive tool calls are grouped into a shared wrapper
  • Verify MCP approval flow still works (approve/deny buttons, state transitions)
  • Verify OAuth inline errors still render correctly

This pull request and its description were written by Isaac.

Signed-off-by: Ann Zhang <ann.zhang@databricks.com>
@annzhang-db annzhang-db force-pushed the parallel-tool-group-rendering branch from 0897f13 to b0e0251 Compare April 21, 2026 22:22
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