Skip to content

docs: document Windows stdio subprocess stdin handling#3079

Merged
felixweinberger merged 2 commits into
modelcontextprotocol:mainfrom
AndreKalberer:andrekalberer/issue-671-stdio-subprocess-windows
Jul 16, 2026
Merged

docs: document Windows stdio subprocess stdin handling#3079
felixweinberger merged 2 commits into
modelcontextprotocol:mainfrom
AndreKalberer:andrekalberer/issue-671-stdio-subprocess-windows

Conversation

@AndreKalberer

Copy link
Copy Markdown
Contributor

Summary

  • document the Windows stdio subprocess hang when a child inherits the server stdin pipe
  • show stdin=subprocess.DEVNULL for asyncio.create_subprocess_exec
  • add a troubleshooting entry and link to it from the stdio run docs

Addresses #671.

Validation

  • Reproduced the issue on Windows current main: a v2 MCPServer stdio tool that starts sys.executable -c "print('Successfully Call!')" without redirecting stdin timed out/faulted after about 7.8s.
  • Confirmed the same repro succeeds in about 3.4s with stdin=subprocess.DEVNULL.
  • py -3.13 -m uv run --frozen mkdocs build --strict

AI disclosure

AI assistance was used while preparing this PR. I reviewed the final change, ran the validation listed above, and take responsibility for it.

@AndreKalberer AndreKalberer marked this pull request as ready for review July 8, 2026 22:37

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@AndreKalberer

Copy link
Copy Markdown
Contributor Author

@felixweinberger Would you be able to review this when you have a chance? It documents the confirmed Windows-only stdio subprocess hang from #671 and the stdin=subprocess.DEVNULL workaround. All CI checks are passing.

@felixweinberger

Copy link
Copy Markdown
Contributor

@felixweinberger Would you be able to review this when you have a chance? It documents the confirmed Windows-only stdio subprocess hang from #671 and the stdin=subprocess.DEVNULL workaround. All CI checks are passing.

Hi thank you for submitting and following up - yes, I will come back to reviewing inbound issues/PRs ~next week, thanks for your patience!

@felixweinberger felixweinberger merged commit e464f72 into modelcontextprotocol:main Jul 16, 2026
35 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