Executing tools with option `shell: true` to allow executing tool (e.g. `go.cmd`, `go.ps1`) from tool management tool like `mise`

[Trung0246]

Is your feature request related to a problem? Please describe.
Currently with this code works that spawn a process internally that depends on node child_process:

cp8.execFile(
      goRuntimePath,
      // -json is supported since go1.9
      ["env", "-json", "GOPATH", "GOROOT", "GOPROXY", "GOBIN", "GOMODCACHE"],
      { env: env3, cwd },
      (err, stdout, stderr) => {
        if (err) {
          outputChannel.info(
            `Failed to run '${goRuntimePath} env' (cwd: ${getWorkspaceFolderPath()}): ${err}
${stderr}`
          );
          outputChannel.show();
          vscode14.window.showErrorMessage(
            `Failed to run '${goRuntimePath} env. The config change may not be applied correctly.`
          );
          return reject();
        }
        if (stderr) {
          outputChannel.info(`'${goRuntimePath} env': ${stderr}`);
          outputChannel.show();
        }
        outputChannel.trace(`${goRuntimePath} env ...:
${stdout}`);
        const envOutput = JSON.parse(stdout);
        if (envOutput.GOROOT && envOutput.GOROOT.trim()) {
          outputChannel.debug("setCurrentGOROOT:", envOutput.GOROOT);
          setCurrentGoRoot(envOutput.GOROOT.trim());
          delete envOutput.GOROOT;
        }
        for (const envName in envOutput) {
          if (!process.env[envName] && envOutput[envName] && envOutput[envName].trim()) {
            process.env[envName] = envOutput[envName].trim();
          }
        }
        if (why !== "path") {
          addGoRuntimeBaseToPATH(path6.join(getCurrentGoRoot(), "bin"));
        } else {
          clearGoRuntimeBaseFromPATH();
        }
        initGoStatusBar(goCtx2);
        return resolve();
      }
    );

As https://stackoverflow.com/questions/78420489/error-when-running-npm-script-with-yarn-err-spawn-einval states, if go.cmd is passed, it will throws EINTVAL, and go.ps1 will causes Uncaught Error: spawn EFTYPE.

Describe the solution you'd like
Allows an option to make {"shell": true} for such case. This is necessary since the popularize of tools like https://github.com/jdx/mise that use shims technique heavily to substitute for real go.exe (which is usually achieved through go.cmd and similar).

Describe alternatives you've considered
No current alternative solution so far, but I did consider making a small C wrapper program to bypass this but it's annoying.

[h9jiang]

Hi @Trung0246

Just to be clear, you should be able to set the .ps1 and .cmd as alternateTools for go but the issue is the child_process will fail.

I'm not a windows user so I can't tell whether this is worth doing. Are you suggesting the only change needed is to add this option {"shell": true} to the cp.ExecFileOptions?

I will need to investigate a little to figure out what this means...

[Trung0246]

Yes that's correct since looks like cp8.execFile actually calls child_process.spawnSync/child_process.spawn.

Here's the excerpt from node docs: https://nodejs.org/api/child_process.html#spawning-bat-and-cmd-files-on-windows

[h9jiang]

Let me give it a try. I have never used the go.cmd or go.ps1 and I normally don't do any development in Windows, I will need to see whether this is possible after try it in windows.