fix: use taskkill on Windows for process tree cleanup

process.kill(-pid) is Unix-only; on Windows it fails to kill child
processes, causing handle.kill() to hang and the NUT job to timeout.
Use taskkill /pid /t /f on Windows for cross-platform process cleanup.

Made-with: Cursor

Author: deepu-mungamuri94

test/commands/webapp/helpers/devServerUtils.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { spawn, type ChildProcess } from 'node:child_process';
+import { execSync, spawn, type ChildProcess } from 'node:child_process';
 import { createServer as createHttpServer, type Server as HttpServer } from 'node:http';
 import { join } from 'node:path';
 import { createServer, type Server } from 'node:net';
@@ -51,8 +51,14 @@ export function spawnWebappDev(args: string[], options: { cwd: string; timeout?:
   let stderrData = '';
 
   const killProcessGroup = (signal: NodeJS.Signals = 'SIGTERM'): void => {
+    const pid = proc.pid;
+    if (pid == null) return;
     try {
-      process.kill(-proc.pid!, signal);
+      if (process.platform === 'win32') {
+        execSync(`taskkill /pid ${pid} /t /f`, { stdio: 'ignore' });
+      } else {
+        process.kill(-pid, signal);
+      }
     } catch {
       /* already dead */
     }
@@ -90,7 +96,7 @@ export function spawnWebappDev(args: string[], options: { cwd: string; timeout?:
                     killProcessGroup('SIGKILL');
                     res();
                   }, 5_000);
-                  proc.on('close', () => {
+                  proc.once('close', () => {
                     clearTimeout(forceKillTimeout);
                     res();
                   });