test(windows): guard clean MCP stdio startup - #2213
lorenzozanee wants to merge 1 commit into
Conversation
|
Thanks for opening this — it has been seen, and it is queued. This note is automated, but it is not a brush-off: it exists so you know where your PR stands instead of having to guess from silence. Current review status: working through a backlog. What that means for this PR, concretely:
Things that will genuinely speed it up whenever review does happen:
If this fixes a bug, a reproduction we can run is worth more than a description of the symptom. Thanks for contributing, and sorry in advance for the wait. |
|
Thank you for building a native Windows MCP stdio guard for #654 — that platform is where our hardest bugs hide and guards there are worth a lot. Before you spend any time on this one: I think it may be a double-submit, and I would rather ask than close it on you. This PR and #2217 are the same guard for the same issue, opened thirteen seconds apart. #2217 looks like the fuller version:
If that is right, #2217 is the one to keep and this can be closed — but it is your work and your call, so tell me which you want and I will do the rest. If they are actually different and I have misread, say so and I will review both. Whichever survives needs the same two things: The
#2241 and #2242 both merged today after review. Good work on those. |
What does this PR do?
Adds a native Windows MCP stdio regression guard for #654. The guard isolates daemon runtime cleanup and rejects the reported path-not-found diagnostics after a successful initialize/tools/list exchange.
Checklist
git commit -s) — required, CI rejectsunsigned commits (DCO, see CONTRIBUTING.md)
make -f Makefile.cbm test)make -f Makefile.cbm lint-ci)Fixes #654
Supersedes #2097(原PR分支已删,按原提交重建)