Skip to content

feat: execute parallel tool calls sequentially#428

Merged
andreitava-uip merged 1 commit intomainfrom
feat/sequential-tool-execution
Jan 20, 2026
Merged

feat: execute parallel tool calls sequentially#428
andreitava-uip merged 1 commit intomainfrom
feat/sequential-tool-execution

Conversation

@andreitava-uip
Copy link
Copy Markdown
Contributor

@andreitava-uip andreitava-uip commented Jan 19, 2026

Support parallel tool calls by executing each tool sequentially.

It determines the current tool call by checking how many tool calls have corresponding ToolMessages already in the message stack.

This requires no new state fields or nodes, making the logic more graph-agnostic.

Sections that assumed the AIMessage will be last in the stack were changed to now find the last AIMessage, skipping over any ToolMessage.

True Parallel execution was NOT implemented due to maintainability and extensibility concerns.

@andreitava-uip andreitava-uip force-pushed the feat/sequential-tool-execution branch 2 times, most recently from da66268 to d7aaabf Compare January 19, 2026 10:08
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure about this router, since the code did not match the docstring for what it claimed to do

@andreitava-uip andreitava-uip force-pushed the feat/sequential-tool-execution branch from d7aaabf to d8c7b6c Compare January 19, 2026 10:28
@andreitava-uip andreitava-uip force-pushed the feat/sequential-tool-execution branch from d8c7b6c to d147578 Compare January 20, 2026 06:45
@andreitava-uip andreitava-uip force-pushed the feat/sequential-tool-execution branch from d147578 to 99985f6 Compare January 20, 2026 07:10
@andreitava-uip andreitava-uip merged commit ba961da into main Jan 20, 2026
60 of 61 checks passed
@andreitava-uip andreitava-uip deleted the feat/sequential-tool-execution branch January 20, 2026 07:14
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.

4 participants