Skip to content

Conversation

@xiehust
Copy link
Contributor

@xiehust xiehust commented Oct 31, 2025

Description

add MCP resource operations in MCP Tools. The MCPClient now provides these resource operations:

  1. list_resources_sync(pagination_token) - List available resources
  2. read_resource_sync(uri) - Read resource content
  3. list_resource_templates_sync(pagination_token) - List resource templates

Related Issues

#1116

Documentation PR

Type of Change

New feature

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.

@mkmeral
Copy link
Contributor

mkmeral commented Dec 22, 2025

hey @xiehust , thank you for the PR :)

There seems to be some merge conflicts. Can we merge from main to resolve them? Also can you add integ tests to verify the functionality end to end?

@dbschmigelski
Copy link
Member

dbschmigelski commented Jan 5, 2026

Hi, this looks good to me. I rebased and added some integration tests

@codecov
Copy link

codecov bot commented Jan 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Member

@dbschmigelski dbschmigelski left a comment

Choose a reason for hiding this comment

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

approving, as I didn't modify any source

@mkmeral mkmeral merged commit 3bc34ac into strands-agents:main Jan 6, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants