Refactor asyncExec to simplify spawn options handling

Revised the `asyncExec` function to use `Omit<SpawnOptions, 'shell'|'stdio'>`, ensuring better type safety and cleaner code. Standardized `spawn` options by including predefined `stdio` settings for consistent process output handling.

Author: cod1k

dev-packages/e2e-tests/run.ts

@@ -1,5 +1,5 @@
 /* eslint-disable no-console */
-import { spawn } from 'child_process';
+import { type SpawnOptions, spawn } from 'child_process';
 import * as dotenv from 'dotenv';
 import { mkdtemp, rm } from 'fs/promises';
 import { sync as globSync } from 'glob';
@@ -13,17 +13,9 @@ const DEFAULT_DSN = 'https://username@domain/123';
 const DEFAULT_SENTRY_ORG_SLUG = 'sentry-javascript-sdks';
 const DEFAULT_SENTRY_PROJECT = 'sentry-javascript-e2e-tests';
 
-function asyncExec(command: string, options: { env: Record<string, string | undefined>; cwd: string }): Promise<void> {
+function asyncExec(command: string, options: Omit<SpawnOptions, 'shell'|'stdio'>): Promise<void> {
   return new Promise((resolve, reject) => {
-    const process = spawn(command, { ...options, shell: true });
-
-    process.stdout.on('data', data => {
-      console.log(`${data}`);
-    });
-
-    process.stderr.on('data', data => {
-      console.error(`${data}`);
-    });
+    const process = spawn(command, { ...options, shell: true, stdio: ['ignore', 'inherit', 'inherit'] });
 
     process.on('error', error => {
       reject(error);