Skip to content

fix: unblock event loop for analyze_repo, detach git subprocess stdin on Windows - #112

Open
huanceng wants to merge 1 commit into
FSoft-AI4Code:mainfrom
huanceng:fix/mcp-windows-stdio-hang
Open

huanceng wants to merge 1 commit into
FSoft-AI4Code:mainfrom
huanceng:fix/mcp-windows-stdio-hang

Conversation

@huanceng

Copy link
Copy Markdown
  • Run analyze_repo via asyncio.to_thread under an asyncio.Lock: keeps tree-sitter usage serialized without blocking the MCP stdio pump (aligns with the to_thread pattern used by all other sync handlers)
  • Add stdin=DEVNULL to GitIgnoreFilter's git calls so children never inherit the MCP JSON-RPC stdin pipe, which hangs on Windows ProactorEventLoop

… on Windows

- Run analyze_repo via asyncio.to_thread under an asyncio.Lock: keeps
  tree-sitter usage serialized without blocking the MCP stdio pump
  (aligns with the to_thread pattern used by all other sync handlers)
- Add stdin=DEVNULL to GitIgnoreFilter's git calls so children never
  inherit the MCP JSON-RPC stdin pipe, which hangs on Windows
  ProactorEventLoop
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.

1 participant