fix(mcp): support stdio transport on Windows - #7596
Conversation
Windows asyncio event loops do not implement add_signal_handler, causing the stdio server to exit before serving requests. Treat signal registration as optional and cover the unsupported-loop path with a regression test.
|
Tested this on Windows 11 (build 26200). Confirmed the root cause: on a One note: this covers the crash, but not the second problem #7595 raises. |
deeleeramone
left a comment
There was a problem hiding this comment.
Thanks for the fix!
Description
NotImplementedErrorwhen registering signal handlers, so stdio now treats that registration as optional and continues serving the MCP client.How has this been tested?
python -m pytest tests -q— 174 passed, 4 skipped.ruff checkpasses on both changed files.Checklist
hotfix/mcp-stdio-windows).