Hey :wave: PR to enable parallel tool calls in ag...
# proj-agent-canvas
m
Hey 👋 PR to enable parallel tool calls in agent-canvas: add parallel tool calls (tool_concurrency_limit) to Agent Settings This PR adds parallel tool calls via
tool_concurrency_limit
to the Agent Settings page. The default value is set to
1
, meaning execution remains sequential by default (matching current behavior). However, this unlocks the ability to experiment with higher concurrency limits. A couple of remarks: • The UI can be slightly misleading. While you can run up to 10 tools in parallel, you won't see step-by-step progress, only the final results. (Enabling real-time progress visualization will require changes in the SDK). • This feature can be highly resource-intensive. For that reason, the default is strictly set to
1
(sequential).
🙌 2
h
I just pulled that down to test and got an error in the console after setting the limit.
Copy code
user-message-body.tsx:57 Cannot update a component (`ChatMessage`) while rendering a different component (`UserMessageBody`). To locate the bad setState() call inside `UserMessageBody`, follow the stack trace as described in <https://react.dev/link/setstate-in-render>
I set the limit to 5 then asked an agent to run 5 tool calls at once to see if it worked. I made a branch to have an agent do a fix and didn't realize it was pointed back at your branch when I just pushed it up to show what I found and fixed 😞
I thought I'd set it back to my fork
m
No problem… I will give a look. Probably something changes when i updated the branch. Thanks to try it out. Ah yes, you have write permission in agent-canvas now :) With great power, comes great responsability 💪
Also I checked and the problem was real but not associated with parallel tool call. It was a regression we introduced. I cherry picked your commit and opened another PR to fix that: https://github.com/OpenHands/agent-canvas/pull/1431 such that the fix is documented. Can you review it? 😉
🙌 1
✅ 1
h
Thank you! I should keep the console open more often. I only noticed it checking the PR and didn't seem to get it when I switched back to main. Sorry for the extra steps once again 🫠
m
No problem… Enjoy the weekend :)
@hundreds-twilight-32833 could you re-try when you have a moment? 🙂
h
Done!
m
thanks