Skip to content

Conversation

@dbschmigelski
Copy link
Member

Description

In strands-agents/sdk-python#419, a breaking change is being made to default the load_tools_from_directory to False. To avoid breaking the autonomous tool generating capabilities in this package, we must explicitly set load_tools_from_directory to True.

Related Issues

strands-agents/sdk-python#419

Documentation PR

[Link to related associated PR in the agent-docs repo]

Type of Change

  • Other (please describe): Fix to prepare for breaking change in sdk

[Choose one of the above types of changes]

Testing

[How have you tested the change?]

  • hatch fmt --linter
  • hatch fmt --formatter
  • hatch test --all

Checklist

  • I have read the CONTRIBUTING document
  • I have added 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
  • 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.

zastrowm
zastrowm previously approved these changes Jul 11, 2025
mehtarac
mehtarac previously approved these changes Jul 11, 2025
Unshure
Unshure previously approved these changes Jul 11, 2025
@dbschmigelski dbschmigelski dismissed stale reviews from Unshure, mehtarac, and zastrowm via 3b31a2e July 11, 2025 17:12
@zastrowm zastrowm merged commit 7388529 into strands-agents:main Jul 11, 2025
11 checks passed
JackYPCOnline added a commit that referenced this pull request Aug 15, 2025
* feat: add end to end integration test

* feat: add integration tests for the package

* feat: remove old test

* refactor: refactor the test code with patch api instead of monkeypatch

* feat: add STRANDS_ANTHROPIC_BETA environment variable (#34)

* feat: add STRANDS_ANTHROPIC_BETA environment variable

* docs(readme): document environment variables

* Windows: Remove cron tool + document missing tools when on windows (#33)

The tool can be imported and run, but without the crontab binary, it's useless on windows

* udpate agent system_prompt reference (#35)

* chore: explicitly set load_tools_from_directory on Agent initilizations (#38)

* chore: explicitly set load_tools_from_directory on Agent initilizations

* Update .prompt

* Update builder with updates from 0.3.0 SDK (#39)

+ set minimum sdk version to be 0.3.0

Co-authored-by: Mackenzie Zastrow <[email protected]>

* chore: Remove Preview (#40)

* deps: bump strangs-agents to v1.0.0 (#41)

Co-authored-by: jer <[email protected]>

* build(pyproject): update development status classifier (#42)

* Use strands logo that looks good in dark & light mode (#44)

Similar to strands-agents/sdk-python/pull/475 but using a dedicated github icon.

The github icon is the lite logo but copied/renamed to make it dedicated to github

* fix: refactor some tests

* fix: refactor tests code

* fix: make tests more accurate

* fix: remove test_custom_load_from_file to avoiding text based response assertion

---------

Co-authored-by: Jack Yuan <[email protected]>
Co-authored-by: Arron <[email protected]>
Co-authored-by: Mackenzie Zastrow <[email protected]>
Co-authored-by: Nick Clegg <[email protected]>
Co-authored-by: Dean Schmigelski <[email protected]>
Co-authored-by: Mackenzie Zastrow <[email protected]>
Co-authored-by: Jonathan Segev <[email protected]>
Co-authored-by: Jeremiah <[email protected]>
Co-authored-by: jer <[email protected]>
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