fix(utils): remove unsafe shell:true option from executeProcess

Author: matejchalk

packages/nx-plugin/src/internal/execute-process.ts

@@ -151,8 +151,7 @@ export function executeProcess(cfg: ProcessConfig): Promise<ProcessResult> {
   );
 
   return new Promise((resolve, reject) => {
-    // shell:true tells Windows to use shell command for spawning a child process
-    const process = spawn(command, args, { cwd, shell: true });
+    const process = spawn(command, args, { cwd });
     // eslint-disable-next-line functional/no-let
     let stdout = '';
     // eslint-disable-next-line functional/no-let

packages/utils/src/lib/execute-process.ts

@@ -158,9 +158,7 @@ export function executeProcess(cfg: ProcessConfig): Promise<ProcessResult> {
   }
 
   return new Promise((resolve, reject) => {
-    // shell:true tells Windows to use shell command for spawning a child process
     const spawnedProcess = spawn(command, args ?? [], {
-      shell: true,
       windowsHide: true,
       ...options,
     }) as ChildProcessByStdio<Writable, Readable, Readable>;