Skip to content

fix(opencode): terminate local MCP process trees - #46254

Closed
YidiDev wants to merge 1 commit into
anomalyco:devfrom
YidiDev:mcp-process-cleanup
Closed

fix(opencode): terminate local MCP process trees#46254
YidiDev wants to merge 1 commit into
anomalyco:devfrom
YidiDev:mcp-process-cleanup

Conversation

@YidiDev

@YidiDev YidiDev commented Aug 30, 2026

Copy link
Copy Markdown

Summary

  • reuse the existing descendant-process cleanup whenever a connected MCP client is released
  • apply it to disconnect, replacement, and post-connect initialization failure
  • add a real stdio regression covering parent and child termination for replacement and disconnect

Verification

  • bun test test/mcp/lifecycle.test.ts --timeout 30000 (22 pass)
  • bun run typecheck
  • Prettier and git diff --check

Closes #46253
Related to #46035

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Aug 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label Aug 30, 2026
@github-actions github-actions Bot closed this Aug 30, 2026
@YidiDev

YidiDev commented Aug 30, 2026

Copy link
Copy Markdown
Author

Resubmitted with the required PR template as #46312.

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.

Local MCP disconnect leaves descendant processes running

1 participant