@cruxy/cli 0.20.0 → 0.21.0

package/dist/utils/child-tree.js

@@ -0,0 +1,76 @@
+/**
+ * Shared no-orphan machinery for long-lived child processes (C.12 LSP servers,
+ * C.27 MCP servers). A child is spawned `detached` so it leads its own process
+ * group; every kill here uses a **negative-PID** `SIGKILL` so the whole tree —
+ * the child AND any grandchildren it forked (gopls's `go`, an MCP server's
+ * helper) — dies together.
+ *
+ * A per-session graceful shutdown covers the normal path, but a hard exit
+ * (Ctrl-C, an uncaught throw) would otherwise orphan these trees. So every live
+ * child's pid is tracked in one process-wide set and force-killed on teardown.
+ * Handlers are installed ONCE, lazily, on the first registration — so unit tests
+ * that never spawn a real process never install them.
+ *
+ * This module is deliberately transport-agnostic: LSP (Content-Length framing)
+ * and MCP (newline-delimited JSON) share the SAME backstop, so `killTrackedTrees`
+ * on exit reaps both and there is a single source of truth for "no orphans".
+ */
+/**
+ * Kill a process's entire group (POSIX negative-PID `SIGKILL`), falling back to
+ * a direct kill when there is no group (or on win32). Swallows errors — the
+ * process may already be gone.
+ */
+export function killTree(pid) {
+    if (pid === undefined)
+        return;
+    try {
+        process.kill(-pid, "SIGKILL");
+    }
+    catch {
+        try {
+            process.kill(pid, "SIGKILL");
+        }
+        catch {
+            /* already exited */
+        }
+    }
+}
+const livePids = new Set();
+let handlersInstalled = false;
+/**
+ * Force-kill the process group of every tracked-but-not-yet-shut-down child,
+ * then forget them. This is exactly what the `exit`/`SIGINT`/`SIGTERM`/`SIGHUP`
+ * handlers run — the last line against orphaned server trees on a hard exit.
+ * Exported so it is directly testable without raising real process signals.
+ * Idempotent: a second call is a no-op.
+ */
+export function killTrackedTrees() {
+    for (const pid of livePids)
+        killTree(pid);
+    livePids.clear();
+}
+/** Number of child trees currently tracked by the exit backstop (for tests). */
+export function trackedTreeCount() {
+    return livePids.size;
+}
+function installExitHandlers() {
+    if (handlersInstalled)
+        return;
+    handlersInstalled = true;
+    process.once("exit", killTrackedTrees);
+    for (const sig of ["SIGINT", "SIGTERM", "SIGHUP"]) {
+        process.once(sig, () => {
+            killTrackedTrees();
+            // Restore default behavior and re-raise so the exit code is correct.
+            process.exit(130);
+        });
+    }
+}
+/** Track a live child for the exit backstop; returns a deregister callback. */
+export function registerForCleanup(pid) {
+    if (pid === undefined)
+        return () => { };
+    installExitHandlers();
+    livePids.add(pid);
+    return () => livePids.delete(pid);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cruxy/cli",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "an agentic coding CLI",
   "type": "module",
   "bin": {