Skip to content

iterative tools #345

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
Jul 7, 2025
Merged

iterative tools #345

merged 8 commits into from
Jul 7, 2025

Conversation

@pgrayy
Copy link
Member

@pgrayy pgrayy commented Jul 3, 2025 •
edited by zastrowm
Loading

Description

Support iterative tool calls. This will allow customers to yield back events, which we can pass to the callback handler on their behalf and yield up through the Agent.stream_async method.

Note, customers still cannot define an iterative (generator) based tool. The support is only internal to the SDK. We are waiting on support for strongly typed events before allowing customers to define their tools as generators.

Related Issues

#83

Documentation PR

Will work on documentation next week.

Type of Change

  • Bug fix
  • New feature
  • Breaking change: See "BREAKING CHANGE" comments for details
  • Documentation update
  • Other (please describe):

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 15:03 — with GitHub Actions Inactive
pgrayy
pgrayy commented Jul 3, 2025
View reviewed changes
@pgrayy pgrayy force-pushed the iterative-tools branch from 9df9353 to 0ae14ce Compare July 3, 2025 15:54
@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 15:55 — with GitHub Actions Inactive
@pgrayy pgrayy force-pushed the iterative-tools branch from 0ae14ce to 06c90a1 Compare July 3, 2025 16:11
@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 16:11 — with GitHub Actions Inactive
@pgrayy pgrayy force-pushed the iterative-tools branch from 06c90a1 to f27c290 Compare July 3, 2025 17:01
@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 17:01 — with GitHub Actions Inactive
@pgrayy pgrayy requested review from Unshure and zastrowm July 3, 2025 18:07
@pgrayy pgrayy marked this pull request as ready for review July 3, 2025 18:07
@pgrayy pgrayy force-pushed the iterative-tools branch from f27c290 to 999df44 Compare July 3, 2025 18:08
@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 18:08 — with GitHub Actions Inactive
@pgrayy pgrayy mentioned this pull request Jul 3, 2025
Closed
zastrowm
zastrowm reviewed Jul 3, 2025
View reviewed changes
pgrayy
pgrayy commented Jul 3, 2025
View reviewed changes
@pgrayy pgrayy temporarily deployed to auto-approve July 3, 2025 20:57 — with GitHub Actions Inactive
pgrayy
pgrayy commented Jul 3, 2025
View reviewed changes
pgrayy
pgrayy commented Jul 3, 2025
View reviewed changes
@pgrayy pgrayy marked this pull request as draft July 7, 2025 18:31
@pgrayy pgrayy temporarily deployed to auto-approve July 7, 2025 18:32 — with GitHub Actions Inactive
pgrayy
pgrayy commented Jul 7, 2025
View reviewed changes
pgrayy
pgrayy commented Jul 7, 2025
View reviewed changes
@pgrayy pgrayy temporarily deployed to auto-approve July 7, 2025 18:49 — with GitHub Actions Inactive
@pgrayy pgrayy marked this pull request as ready for review July 7, 2025 18:51
zastrowm
zastrowm approved these changes Jul 7, 2025
View reviewed changes
@pgrayy pgrayy merged commit ebcd4c0 into strands-agents:main Jul 7, 2025
20 of 22 checks passed
jsamuel1 pushed a commit to jsamuel1/sdk-python that referenced this pull request Jul 9, 2025
@pgrayy @jsamuel1
iterative tools (strands-agents#345)
79f6a52
@pgrayy pgrayy mentioned this pull request Jul 11, 2025
Closed
@pgrayy pgrayy deleted the iterative-tools branch July 14, 2025 13:18
dbschmigelski pushed a commit to Ketansuhaas/sdk-python that referenced this pull request Jul 24, 2025
@pgrayy @dbschmigelski
iterative tools (strands-agents#345)
3f25063
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zastrowm zastrowm zastrowm approved these changes

@Unshure Unshure Awaiting requested review from Unshure

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pgrayy @zastrowm