Use tree-kill instead of custom function to try terminating all proce…

…ss without issues

runner/package.json

@@ -14,6 +14,7 @@
     "node-fetch": "^3.3.1",
     "pidtree": "^0.6.0",
     "pidusage": "^3.0.2",
-    "systeminformation": "^5.17.16"
+    "systeminformation": "^5.17.16",
+    "tree-kill": "^1.2.2"
   }
 }

runner/runner.js

@@ -6,6 +6,7 @@ import pidusage from 'pidusage';
 import pidtree from 'pidtree';
 import os from 'os';
 import systeminformation from 'systeminformation';
+import kill from 'tree-kill';
 
 import {
 	ITERATIONS_PER_PROCESS,
@@ -81,31 +82,6 @@ async function execBuildProcess(processPath, processExe) {
 	});
 }
 
-export function killAll(pid, signal='SIGTERM'){
-	if(process.platform == "win32"){
-		exec(`taskkill /PID ${pid} /T /F`, (error, stdout, stderr)=>{
-			console.log("taskkill stdout: " + stdout)
-			console.log("taskkill stderr: " + stderr)
-			if(error){
-				console.log("error: " + error.message)
-			}
-		})
-	}
-
-	// see https://nodejs.org/api/child_process.html#child_process_options_detached
-	// If pid is less than -1, then sig is sent to every process in the process group whose ID is -pid.
-	try {
-		try {
-			process.kill(-pid, signal);
-		} catch(e) {
-			console.error(e);
-			process.kill(pid, signal);
-		}
-	} catch(e) {
-		console.error(e);
-	}
-}
-
 async function sleep(seconds) {
 	return (await new Promise((resolve, _reject) => {
 		setTimeout(resolve, seconds * 1000);
@@ -285,10 +261,10 @@ async function getMemoryUsageHistoryOfProcess(processPath, processExe, timeout=D
 				clearInterval(interval);
 
 				if(childProcess.exitCode === null && !done) {
-					killAll(childProcess.pid);
+					kill(childProcess.pid);
 				} else {
 					try {
-						killAll(childProcess.pid);
+						kill(childProcess.pid);
 					} catch {}
 				}
 			}