fix(infra): attach error listener to detached spawn to prevent unhandled rejection crash

tzy-17 · tzy-17 · commit a723a2183894 · 2026-07-07T16:14:46.000+08:00
spawnDetachedGatewayProcess calls child.unref() without attaching an error listener. If spawn fails asynchronously (ENOMEM, missing executable), the ChildProcess emits an unhandled 'error' event that crashes the parent Node process. Add a guard-checked noop error listener before unref() to prevent the unhandled-event crash. Fixes #101458
diff --git a/src/infra/process-respawn.ts b/src/infra/process-respawn.ts
@@ -53,6 +53,12 @@ function spawnDetachedGatewayProcess(opts: GatewayRespawnOptions = {}): {
     detached: true,
     stdio: "inherit",
   });
+  // Attach a noop error listener before unref so async spawn failures
+  // (ENOMEM, missing executable, etc.) don't crash the parent as an
+  // unhandled 'error' event on the ChildProcess.
+  if (typeof child.on === "function") {
+    child.on("error", () => {});
+  }
   child.unref();
   return { child, pid: child.pid ?? undefined };
 }
