Skip to content

Feature/rule engine parallel execution - #1426

Merged
iceljc merged 5 commits into
SciSharp:masterfrom
iceljc:feature/rule-engine-parallel-execution
Sep 5, 2026
Merged

Feature/rule engine parallel execution#1426
iceljc merged 5 commits into
SciSharp:masterfrom
iceljc:feature/rule-engine-parallel-execution

Conversation

@iceljc

@iceljc iceljc commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Jicheng Lu and others added 5 commits September 3, 2026 14:52
The Response API paths kept only the first function call the model asked
for, so parallel tool calls were dropped and the model re-asked for the
rest on the next turn. The streaming path was worse: each output-item-done
update overwrote the previous call, keeping only the last.

Collect every FunctionCallResponseItem and populate ToolCalls on the
returned RoleDialogModel, matching the Chat API paths. ToolCallId,
FunctionName and FunctionArgs still carry the first call, so existing
single-call consumers are unaffected.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown
Contributor

Code Review by Qodo

Grey Divider

No Changes in PR

Qodo reviewed your PR and found no changes in the code

Grey Divider

Tip of the day
💡 Did you know, you can keep summaries lean with Finding overflow, which tucks the rest behind 'View more'

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@iceljc
iceljc merged commit 403df81 into SciSharp:master Sep 5, 2026
0 of 4 checks passed
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