Skip to content

fix: example clients_everything_stdio#770

Open
adrianromero wants to merge 2 commits intomodelcontextprotocol:mainfrom
adrianromero:Fixes/769
Open

fix: example clients_everything_stdio#770
adrianromero wants to merge 2 commits intomodelcontextprotocol:mainfrom
adrianromero:Fixes/769

Conversation

@adrianromero
Copy link
Copy Markdown

@adrianromero adrianromero commented Mar 22, 2026

This change fixes #769

Motivation and Context

Very simple fix that resolves the execution of the example Full-Featured Standard I/O Client (everything_stdio.rs)

How Has This Been Tested?

Just running the example locally:
Full-Featured Standard I/O Client (everything_stdio.rs)

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling (Not required)
  • I have added or updated documentation as needed (not required

Additional context

@adrianromero adrianromero requested a review from a team as a code owner March 22, 2026 22:09
@github-actions github-actions bot added the T-examples Example code changes label Mar 22, 2026
@adrianromero adrianromero changed the title fix: Example clients_everything_stdio (#769) fix: example clients_everything_stdio Mar 25, 2026
@adrianromero
Copy link
Copy Markdown
Author

adrianromero commented Mar 25, 2026

Commit message fixed due to CI failure

DaleSeo
DaleSeo previously approved these changes Mar 25, 2026
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @adrianromero! Could update the comment to match the new behavior? Other than that, it looks great!

Co-authored-by: Dale Seo <5466341+DaleSeo@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@alexhancock alexhancock left a comment

Choose a reason for hiding this comment

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

Nice find. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-examples Example code changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Example *clients_everything_stdio* fails when requesting the first resource

3 participants