Skip to content

Conversation

@joelrobin18
Copy link
Contributor

Description

When following the model provider example, the calculator tool is used, which depends on the strands-agents-tools package. This package is not currently mentioned in the documentation.
This PR updates the installation instructions to include strands-agents-tools in the pip command, ensuring users can run the example without missing dependency errors.

Type of Change

  • New content addition
  • Content update/revision
  • Structure/organization improvement
  • Typo/formatting fix
  • Bug fix
  • Other (please describe):

This PR updates the installation instructions to include strands-agents-tools in the pip command, ensuring users can run the example without missing dependency errors.

Motivation and Context

The model provider example in the documentation uses the calculator tool, which depends on the strands-agents-tools package. Since this dependency was not mentioned, users following the example would encounter a ModuleNotFoundError. This change ensures the package is included in the installation instructions, preventing runtime errors and improving the developer experience.

Areas Affected

docs/user-guide/concepts/model-providers/anthropic.md
docs/user-guide/concepts/model-providers/cohere.md
docs/user-guide/concepts/model-providers/litellm.md
docs/user-guide/concepts/model-providers/llamaapi.md
docs/user-guide/concepts/model-providers/mistral.md
docs/user-guide/concepts/model-providers/ollama.md
docs/user-guide/concepts/model-providers/openai.md
docs/user-guide/concepts/model-providers/sagemaker.md
docs/user-guide/concepts/model-providers/writer.md

Screenshots

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using mkdocs serve
  • Links in the documentation are valid and working
  • Images/diagrams are properly sized and formatted
  • All new and existing tests pass

Additional Notes

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

Signed-off-by: joelrobin18 <joelrobin1818@gmail.com>
@joelrobin18
Copy link
Contributor Author

@zastrowm @jer96 Could you please review this PR? Thanks!

@zastrowm zastrowm merged commit b14afa7 into strands-agents:main Aug 13, 2025
2 checks passed
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