Skip to content

feat: Add tool_invoker_kwargs to Agent#9574

Merged
sjrl merged 2 commits intomainfrom
add-max-workers-to-agent
Jul 1, 2025
Merged

feat: Add tool_invoker_kwargs to Agent#9574
sjrl merged 2 commits intomainfrom
add-max-workers-to-agent

Conversation

@sjrl
Copy link
Copy Markdown
Contributor

@sjrl sjrl commented Jul 1, 2025

Related Issues

Proposed Changes:

Add tool_invoker_kwargs to the Agent's init method so users can pass additional kwargs to the underlying ToolInvoker like max_workers to control Tool calling parallelism and enable_streaming_callback_passthrough, etc.

cc @medsriha

How did you test it?

Updated unit tests

Notes for the reviewer

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

@sjrl sjrl requested review from a team as code owners July 1, 2025 06:10
@sjrl sjrl requested review from davidsbatista and dfokina and removed request for a team July 1, 2025 06:10
@github-actions github-actions Bot added topic:tests type:documentation Improvements on the docs labels Jul 1, 2025
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 15991321411

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 13 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.002%) to 90.225%

Files with Coverage Reduction New Missed Lines %
components/agents/agent.py 13 90.8%
Totals Coverage Status
Change from base Build 15973897099: 0.002%
Covered Lines: 11731
Relevant Lines: 13002

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@davidsbatista davidsbatista left a comment

Choose a reason for hiding this comment

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

LGTM! 👍🏽

@sjrl sjrl merged commit 3aaa201 into main Jul 1, 2025
20 checks passed
@sjrl sjrl deleted the add-max-workers-to-agent branch July 1, 2025 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:tests type:documentation Improvements on the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ability to update ToolInvoker max_workers via the agent component

3 participants