The AI Language
The AI Language
Hi team! This is my first pull request. PR fixes Windows compatibility for MCP STDIO clients by introducing a fallback to subprocess.Popen wrapped with async I/O streams. It ensures that...
@dsp-ant @jerome3o-anthropic would appreciate any kind of feedback on this if possible from your end. Thanks a ton! Regards [Kartik / theailanguage]
> #555 solves this in a much cleaner way Thanks for sharing 🙏 I just tested it on my setup (Windows 11 + Python 3.13), and I’m still hitting the...
> I am on windows 11; also using it in an event loop. are you using windows proactor for the async loop? > > edit: I also have this change...
One more thing, in case you'd want to look at the implementation of the streamlit code - you can have a look at Repo - https://github.com/theailanguage/mcp_client UI code - https://github.com/theailanguage/mcp_client/blob/main/streamlit_client_ui.py...
Thanks for your reply. Yes, I'll check these.
@ihrpr I checked the failing Windows checks. This doesn’t seem to be related to the code changes I made, but rather to the test setup itself. The four checks are...
@ihrpr I have requested a re-review based on my last comment itself because if the test may be skipped then the checks can be passed with the existing PR code...
@felixweinberger thanks for the suggestions, will work on this now! Regards Kartik / theailanguage
@felixweinberger I have rebased the code, included my original changes and made two additional changes as mentioned below. Kindly review and thanks a lot for your feedback! Regards Kartik /...