Repository navigation
Windows MCP SSH stdio transport closes after initialize when PROGRAMDATA is missing from inherited environment #3335
Description
Activity
I’d like to work on this issue. I’ve already prepared PR #3334 with the proposed fix. Could you please assign this issue to me so the PR can be reopened?
I reproduced this on Windows 11 with Python 3.11 against main (f1b6589). The spawned server is missing six variables the parent has: COMSPEC, PROGRAMDATA, PROGRAMFILES, PROGRAMFILES(X86), PROGRAMW6432 and WINDIR. The TypeScript SDK added the same six in typescript-sdk#2043 on September 23, so the two SDKs now differ.
The same line has a second Windows problem. With
env={"Path": ...}, the spelling Windows itself uses, the child gets both PATH and Path in its environment block. A server started as python.exe directly still sees the override. A server started withuv run, a Node server, and cmd.exe all read the inherited PATH, and underuv runthe pinned directory is gone from the server's PATH completely. So a pinned toolchain is silently ignored. I reported the TypeScript version in typescript-sdk#2859, and the fix is open as typescript-sdk#2874.I have a fix: add the six names, and on Windows drop an inherited key when an override matches it case-insensitively. POSIX is unchanged. It has three tests that go through stdio_client using the existing fake spawn seam. Two fail on main and pass with the change, and the third pins the POSIX behavior. tests/client/test_stdio.py passes on Windows. @AbarnaaSree found this, fixed PROGRAMDATA in #3334 (closed only for a missing issue link), and asked to be assigned above. If they still want it, I'm happy to hand over the repro and the extra five names. Otherwise I can build on their change with them as co-author on the commit, if you'd like to assign it to me. I used Claude Code to write the repro and the fix, and every result above comes from runs on my Windows machine.
Description
On Windows, an MCP server launched through the stdio client can terminate shortly after the
initializemessage when the spawned process is given the SDK's restricted inherited environment.During investigation of Windows MCP + SSH behavior, I found that
PROGRAMDATAis missing fromDEFAULT_INHERITED_ENV_VARSinsrc/mcp/client/stdio.py.Environment
Observed behavior
The MCP SSH subprocess can close shortly after the
initializemessage with a transport-closed error.The same SSH setup works when launched manually and behaves differently when the expected Windows environment is available.
Investigation
The stdio client constructs a restricted environment for spawned processes.
PROGRAMDATAwas not included in the Windows inherited environment list:Adding
PROGRAMDATAallows the spawned Windows process to receive this standard environment variable.Proposed fix
Add:
"PROGRAMDATA",to the Windows
DEFAULT_INHERITED_ENV_VARSlist in:src/mcp/client/stdio.pyTesting
I reproduced the Windows MCP + SSH behavior and investigated the restricted subprocess environment.
After adding
PROGRAMDATA:PROGRAMDATAis inherited by the spawned process.Popen.git diff --checkpasses.I have prepared PR #3334 with this change.
Related issue
This investigation was motivated by MCP SSH transport closing immediately after
initializeon Windows.I would appreciate maintainer confirmation that this is the appropriate fix and whether this issue can be assigned to me so PR #3334 can be reopened.