FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: propagate stdio process exit during initialization by DragonFSKY · Pull Request #960 · modelcontextprotocol/java-sdk · GitHub

fix: propagate stdio process exit during initialization - #960

Open
DragonFSKY wants to merge 1 commit into
modelcontextprotocol:mainfrom
DragonFSKY:fix/959-stdio-process-exit
Open

fix: propagate stdio process exit during initialization#960
DragonFSKY wants to merge 1 commit into
modelcontextprotocol:mainfrom
DragonFSKY:fix/959-stdio-process-exit

Conversation

DragonFSKY commented May 15, 2026
edited
Loading

Copy link
Copy Markdown

Summary

  • Detect unexpected stdio child-process exit and report a typed exception with the command and exit code.
  • Propagate that failure to an in-progress client initialization instead of waiting for the request timeout.
  • Add one subprocess regression test for the reported behavior.

Fixes #959

Tests

  • ./mvnw -pl mcp-core test — 385 tests passed
  • ./mvnw -pl mcp-test -am -Dtest=StdioMcpClientInitializationFailureTests -Dsurefire.failIfNoSpecifiedTests=false test — 1 test passed
  • ./mvnw -DskipTests package — all 11 modules built successfully
  • git diff --check

Signed-off-by: Dongliang Xie <dragonfsky@gmail.com>
DragonFSKY force-pushed the fix/959-stdio-process-exit branch from bc6b03e to 73cfe88 Compare August 25, 2026 10:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

StdioClientTransport does not propagate unexpected child process exit during initialization

1 participant


Back | FazBrowse Home | New Git URL