Skip to content

fix(mcp): scope stdio handshake timeout#909

Merged
GCWing merged 1 commit into
GCWing:mainfrom
wgqqqqq:fix/mcp-handshake-timeout-scope
May 28, 2026
Merged

fix(mcp): scope stdio handshake timeout#909
GCWing merged 1 commit into
GCWing:mainfrom
wgqqqqq:fix/mcp-handshake-timeout-scope

Conversation

@wgqqqqq
Copy link
Copy Markdown
Collaborator

@wgqqqqq wgqqqqq commented May 28, 2026

Summary

Follow-up to #888.

  • keep the 30s timeout for local stdio MCP initialize/handshake only
  • leave regular local MCP requests, especially long-running tools/call, without the handshake timeout
  • remove timed-out local requests from the pending map
  • avoid spawning a login shell for every macOS child process by caching the merged PATH once and reusing existing platform PATH expansion

Verification

  • cargo test -p bitfun-services-core
  • cargo test -p bitfun-services-integrations --features mcp connection::tests -- --nocapture
  • cargo check -p bitfun-core --features product-full
  • git diff --check

@GCWing GCWing merged commit 289714f into GCWing:main May 28, 2026
4 checks passed
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.

2 participants