fix: resolve Windows stdio JSON parsing errors and add Claude configuration docs#292
Merged
Merged
Conversation
…and add Windows setup guide for Claude Code MCP integration
Owner
|
Thanks for the Windows fix and docs! However, the fastmcp dependency change from |
Open
7 tasks
Contributor
Author
|
Thanks for reviewing and merging the docs! I understand the concern about the major version bump. I'll take a look at option (b) and see if I can put together a separate PR to safely migrate the codebase to the fastmcp v3 API. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe the pull request
This PR addresses a critical blocker for Windows users where the MCP server fails to connect to Claude Code, throwing
Invalid JSON: EOF while parsing a valueandConnection closederrors during thestdiostream initialization.Root Causes & Fixes:
fastmcp(locked at<3) fail to properly handle newline characters (\n) in Windowsstdiostreams. Bumped thefastmcpconstraint to>=3.2.4and regenerated theuv.lockfile to pull the upstream fix.README.mdfor Windows users. Using the typicalcmd /cwrapper in Claude's configuration injects unwanted terminal echoes into the stream, crashing the parser. Documented the direct.exeexecution approach using"env": {"PYTHONUTF8": "1"}to ensure stable communication.Testing:
Tested locally on Windows 11 with Claude Code CLI. The server now successfully connects and maintains the stream without parsing crashes.