Unable to run tests via MCP server tool (on Windows?)

[DanTup]

I created a new Dart project and asked Copilot to run the tests. It invoked the MCP tool but failed with an error so had to fall back to using the command line:

<<< (dart and flutter tooling) {"jsonrpc":"2.0","id":0,"result":{"roots":[{"uri":"file:///d%3A/Dev/Test%20Projects/mcp_test_dart","name":"mcp_test_dart"}]}}
>>> (dart and flutter tooling) {"jsonrpc":"2.0","result":{"content":[{"text":"dart|flutter test failed in d:\\Dev\\Test Projects\\mcp_test_dart:\n\n\nErrors\n'\\\"D:\\Tools\\Dart\\Latest (Bleeding Edge)\\bin\\dart.exe\\\"' is not recognized as an internal or external command,\r\noperable program or batch file.","type":"text"}],"isError":true},"id":3}

The path D:\Tools\Dart\Latest (Bleeding Edge)\bin\dart.exe is definitely correct, but a) I don't know if those backslashes need escaping, b) it contains spaces and c) it contains parens.

I had a quick scan on GitHub but couldn't see the code that was spawning the process to see whether there's any escaping going on (or required).

cc @jakemac53

[jakemac53]

Fwiw, I just tested with spaces and parenthesis in the package path (without issue), so I think this is more specific to the dart executable path having some of those. I will see if I can narrow that down.

[jakemac53]

Ok, I was able to reproduce this when I added parenthesis to the flutter SDK path

[jakemac53]

We are using package:process which has some handling here https://github.com/dart-lang/tools/blob/main/pkgs/process/lib/src/interface/common.dart#L23 but only for spaces

[DanTup]

That's odd - my path has both, so I would've expected it to end up quoted based on the string above, so it seems odd that it still failed.

[jakemac53]

I think the right place to try and fix this is in package:process, and hopefully we can write a test there to validate that it works.