-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issues on Windows 11 #4
Comments
Yeah, figured it out around the same time. Nice one! I think once you get it doing something easier (and using cmd commands instead of PS), it's fine - and lets its guard down more for other commands. So for anyone else - don't start off with asking it to do anything that could trigger its guard rails - things related to claude or stuff like that. I also commented out the blacklisted commands just to give it more freedom (I'm happy to own the risk - wouldn't recommend if using any prod server or server you're not happy to have broken). Now to see if I can get it connected to my linux server which was the initial plan for this. Be a bit trickier but I'm sure there's a way. |
Patched the first one in #6. It would be better if it was always one command, sets config for you, etc. Now -- as for our commands here ... I've been trying to figure out how to get that settled. I wrote a utility to find our Powershell and use that as a shell but it seems like it's our string escaping that's a little weird, perhaps? I also think execa isn't piping our environment into the shell environment like it does on Unix machines... They have an ongoing issue about Windows. |
Hello, I've just tested it on Win11 and here's my notes.
npx mcp-shell
won't work.npm install -g mcp-shell
claude_desktop_config.json
:it works with commands like
npm --version
but it doesn't work with folder specific commands:
because it adds double backslash while win11 only works with one backslash.
The text was updated successfully, but these errors were encountered: