-
Notifications
You must be signed in to change notification settings - Fork 219
feat: Add MCP client tool #49
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.
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
Conversation
|
Related strands-agents/agent-builder#4 |
- Add streamable_http transport option alongside stdio and sse - Support headers, timeout, sse_read_timeout, terminate_on_close, and auth parameters - Add comprehensive tests for streamable HTTP functionality - Update README with streamable HTTP example
cagataycali
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Approving and leaving a room to team to take a look!
dbschmigelski
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving after offline discussion about consistency, but want to reiterate that mcp_client as the name seems potentially dangerous.
Description
This PR introduces a new MCP (Model Context Protocol) client tool for Strands Agents that provides a high-level interface for connecting to any MCP server with simplified configuration and enhanced functionality.
Key Features:
Usage Examples:
Related Issues
N/A
Documentation PR
N/A - Documentation is included in the README.md update
Type of Change
Testing
The implementation includes comprehensive unit tests covering:
Testing commands run:
hatch fmt --linter✓hatch fmt --formatter✓hatch test --all✓Manual Testing
Key achievements:
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.