@wrongstack/tools 0.281.3 → 0.282.0

package/dist/exec.js

@@ -455,14 +455,34 @@ function redactCommand(cmd) {
   return result;
 }
 var DEFAULT_GRACE_MS = 2e3;
-function killWin32Tree(pid) {
+var WIN32_TASKKILL_TIMEOUT_MS = 5e3;
+function killWin32Tree(pid, opts = {}) {
   try {
     const child = spawn("taskkill", ["/pid", String(pid), "/T", "/F"], {
       stdio: "ignore",
       windowsHide: true
     });
-    child.on("error", () => {
-    });
+    let settled = false;
+    let timeout;
+    const settle = () => {
+      if (settled) return;
+      settled = true;
+      if (timeout) clearTimeout(timeout);
+      try {
+        opts.onSettled?.();
+      } catch {
+      }
+    };
+    child.on("error", settle);
+    child.on("close", settle);
+    timeout = setTimeout(() => {
+      try {
+        child.kill();
+      } catch {
+      }
+      settle();
+    }, Math.max(1, opts.timeoutMs ?? WIN32_TASKKILL_TIMEOUT_MS));
+    timeout.unref?.();
     child.unref();
     return true;
   } catch {
@@ -692,17 +712,18 @@ var ProcessRegistryImpl = class {
     const isWin2 = os.platform() === "win32";
     if (isWin2) {
       const liveRealChild = p.child.exitCode === null && typeof p.child.pid === "number";
-      if (liveRealChild && killWin32Tree(pid)) {
-        const fallback = setTimeout(() => {
-          if (p.child.exitCode === null) {
-            try {
-              p.child.kill("SIGKILL");
-            } catch {
-            }
+      const directFallback = () => {
+        if (p.child.exitCode === null) {
+          try {
+            p.child.kill("SIGKILL");
+          } catch {
           }
-        }, graceMs);
-        fallback.unref?.();
-      } else {
+        }
+      };
+      if (liveRealChild && killWin32Tree(pid, {
+        timeoutMs: Math.max(graceMs, WIN32_TASKKILL_TIMEOUT_MS),
+        onSettled: directFallback
+      })) ; else {
         try {
           p.child.kill(force ? "SIGKILL" : "SIGTERM");
         } catch {