Skip to content

Conversation

@zastrowm
Copy link
Member

@zastrowm zastrowm commented Dec 5, 2025

Description

Consolidate all source code under the src directory for better organization.

This moves vended tools under src/vended-tools keeping the vended_tools export for customers & backwards compatibility. I updated our eslint config to ensure that nothing outside of vended_tools references our vended_tools.

I named it src/vended-tools instead of src/vended_tools because it matches our other folders like conversation-manager (IMHO dashes are more common/better for JS projects). Externally it's still vended_tools - I have/had a plan to migrate to vended-tools there as well, but want to discuss with the team first.

Testing

As part of this change, I added code to our test:package to ensure that consumers can access these tools. The test(s) pass both before & after this change.

We also now export the Tool class; it was previously a type only export, but that was a left over artifact from when Tool was interface and was never updated when we made it a base class.

Related Issues

Documentation PR

Type of Change

Refactor

Testing

How have you tested the change?

  • I ran npm run check

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.

Copy link
Collaborator

@afarntrog afarntrog left a comment

Choose a reason for hiding this comment

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

@zastrowm zastrowm merged commit 314a976 into strands-agents:main Dec 5, 2025
8 checks passed
@zastrowm zastrowm deleted the move_vended_tools branch December 5, 2025 21:51
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.

2 participants