Skip to content

fix(core): clean up mcp stdio descendants - #34525

Merged
rekram1-node merged 3 commits into
core-mcpfrom
mcp-stdio-cleanup
Jun 29, 2026
Merged

rekram1-node merged 3 commits into
core-mcpfrom
mcp-stdio-cleanup

Conversation

@rekram1-node

@rekram1-node rekram1-node commented Jun 29, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • terminate descendant processes for stdio MCP transports before closing the SDK client or transport
  • reuse the cleanup path for failed startup transport teardown
  • keep the change scoped to local MCP process cleanup

Verification

  • inspected MCP SDK v1.29.0 StdioClientTransport.close: it only closes/kills the root server process
  • confirmed with a local POSIX repro that a child process can survive after its parent is killed
  • git diff --check
  • bun typecheck could not run because tsgo is not installed in this checkout

@rekram1-node
rekram1-node merged commit d66c5f3 into core-mcp Jun 29, 2026
6 of 8 checks passed
@rekram1-node
rekram1-node deleted the mcp-stdio-cleanup branch June 29, 2026 23:07
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.

1 participant