Skip to content

Conversation

@kpavlov
Copy link
Contributor

@kpavlov kpavlov commented Nov 28, 2025

The StdioClientTransport class has been enhanced to monitor the standard error (stderr) stream, enabling the detection of fatal errors that should terminate the connection. Its internal threading model was refactored to use structured concurrency, ensuring that the separate coroutines for standard input, output, and error are properly supervised and cancelled together. The API was expanded with new constructors that accept an error source and a predicate for identifying fatal error lines. Furthermore, the helper was updated to expose functionality, supporting the processing of line-based text from the error stream.

Motivation and Context

#107 Handle errors in Stdio transport

How Has This Been Tested?

Unit/untegraiton tests

Breaking Changes

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
  • I have added or updated documentation as needed

Additional context

The class has been enhanced to monitor the standard error (stderr) stream, enabling the detection of fatal errors that should terminate the connection.
Its internal threading model was refactored to use structured concurrency, ensuring that the separate coroutines for standard input, output, and error are properly supervised and cancelled together.
The API was expanded with new constructors that accept an error source and a predicate for identifying fatal error lines. Furthermore, the helper was updated to expose functionality, supporting the processing of line-based text from the error stream.
@kpavlov kpavlov added the bugfix Something was fixed 🎉 label Nov 28, 2025
@kpavlov kpavlov requested a review from devcrocod November 28, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Something was fixed 🎉

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants