@integrity-labs/agt-cli 0.28.762 → 0.28.764

package/dist/{chunk-BTED7TYA.js → chunk-SMY25FCN.js}

@@ -7,13 +7,14 @@ import {
   buildRemoteMcpEntry,
   expandTemplateVars,
   getProjectDir,
+  hardBoundedExecFile,
   isolationMode,
   log,
   mergeEnvIntegrationsContent,
   parseEnvFileEntries,
   parseEnvIntegrations,
   shellQuote
-} from "./chunk-EU6RSU5V.js";
+} from "./chunk-LVBINNTQ.js";
 import {
   BIND_FAILURE_QUARANTINE_THRESHOLD,
   INTEGRATIONS_SECTION_END,
@@ -63,7 +64,7 @@ import {
   sessionTranscriptDir,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-L4KDOGHB.js";
+} from "./chunk-23KTLVYX.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -6436,7 +6437,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.762" : "dev";
+var agtCliVersion = true ? "0.28.764" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -7043,6 +7044,12 @@ var HostFlagStore = class {
 
 // src/lib/stale-mcp-reaper.ts
 import { execFileSync } from "child_process";
+function livenessFromProbe(r, onNoAnswer) {
+  if (r.kind === "ok") return true;
+  if (r.kind === "failed" && r.status !== null) return false;
+  onNoAnswer?.(r.kind === "timed-out" ? "did not answer within its bound" : `could not run: ${r.message}`);
+  return true;
+}
 var rotationTimestamps = /* @__PURE__ */ new Map();
 var DEFAULT_ROTATION_GRACE_MS = 15e3;
 function rotationKey(codeName, serverKey) {
@@ -7256,19 +7263,30 @@ function findMcpChildrenForAgent(args) {
 function reapStaleMcpChildren(args) {
   const { log: log2, codeName, serverKeys, mcpJson, graceMs = 5e3, isolated = false } = args;
   if (serverKeys.length === 0) return [];
-  const runPs = args.runPs ?? (isolated ? () => execFileSync("docker", ["exec", `agt-${codeName}`, "ps", "-eo", "pid,ppid,args"], {
-    encoding: "utf-8",
-    timeout: 8e3
-  }) : () => execFileSync("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 }));
-  const killProcess = args.killProcess ?? (isolated ? (pid, signal) => {
-    try {
-      execFileSync(
+  const runPs = args.runPs ?? (isolated ? (
+    // ENG-9781: hard-bounded. See bounded-exec.ts — `execFileSync`'s own
+    // `timeout` is not a bound, and this runs per isolated agent per cycle
+    // on the manager-worker's critical path. Throws on a non-answer so the
+    // caller's existing catch skips the agent rather than acting on an
+    // empty process list.
+    () => {
+      const r = hardBoundedExecFile(
         "docker",
-        ["exec", `agt-${codeName}`, "kill", signal === "SIGKILL" ? "-KILL" : "-TERM", String(pid)],
-        { timeout: 8e3, stdio: "ignore" }
+        ["exec", `agt-${codeName}`, "ps", "-eo", "pid,ppid,args"],
+        { timeoutMs: 8e3, log: log2 }
+      );
+      if (r.kind === "ok") return r.stdout;
+      throw new Error(
+        r.kind === "timed-out" ? `docker exec ps did not answer within its bound (bounded=${r.bounded})` : `docker exec ps failed (status=${r.status}): ${r.message}`
       );
-    } catch {
     }
+  ) : () => execFileSync("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 }));
+  const killProcess = args.killProcess ?? (isolated ? (pid, signal) => {
+    hardBoundedExecFile(
+      "docker",
+      ["exec", `agt-${codeName}`, "kill", signal === "SIGKILL" ? "-KILL" : "-TERM", String(pid)],
+      { timeoutMs: 8e3, log: log2 }
+    );
   } : (pid, signal) => {
     try {
       process.kill(pid, signal);
@@ -7276,15 +7294,17 @@ function reapStaleMcpChildren(args) {
     }
   });
   const isAlive = args.isAlive ?? (isolated ? (pid) => {
-    try {
-      execFileSync("docker", ["exec", `agt-${codeName}`, "kill", "-0", String(pid)], {
-        timeout: 8e3,
-        stdio: "ignore"
-      });
-      return true;
-    } catch {
-      return false;
-    }
+    const r = hardBoundedExecFile(
+      "docker",
+      ["exec", `agt-${codeName}`, "kill", "-0", String(pid)],
+      { timeoutMs: 8e3, log: log2 }
+    );
+    return livenessFromProbe(
+      r,
+      (why) => log2(
+        `[stale-mcp-reaper] liveness probe for pid ${pid} in 'agt-${codeName}' ${why} \u2014 treating as ALIVE (no reap) rather than guessing`
+      )
+    );
   } : (pid) => {
     try {
       process.kill(pid, 0);
@@ -7453,10 +7473,33 @@ function reapMissingMcpSessions(args) {
     currentGenerationExternallyStarted = false
   } = args;
   const now = args.now ?? Date.now;
-  const runPs = args.runPs ?? (isolated ? () => execFileSync2("docker", ["exec", `agt-${codeName}`, "ps", "-eo", "pid,ppid,args"], {
-    encoding: "utf-8",
-    timeout: 8e3
-  }) : () => execFileSync2("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 }));
+  const runPs = args.runPs ?? (isolated ? (
+    // ENG-9781: HARD-bounded, and the distinction is the whole ticket. The
+    // previous `execFileSync(..., { timeout: 8_000 })` bounded nothing —
+    // Node fires SIGTERM at the deadline and then keeps waiting, so a
+    // docker CLI talking to a wedged daemon blocked the manager-worker's
+    // event loop and stopped scheduling for EVERY agent on the host
+    // (33m46s, twice, on 2026-09-01). This call runs once per isolated
+    // agent per poll cycle, so it is the single worst place on the manager
+    // for an unbounded subprocess.
+    //
+    // Throwing on a non-answer is deliberate rather than lazy: the caller
+    // already catches, logs and SKIPS the agent for the cycle, which is
+    // exactly right. A timeout must not become an empty process list —
+    // that reads as "every declared MCP server is missing" and restarts the
+    // session, turning a docker hiccup into a fleet-wide restart storm.
+    () => {
+      const r = hardBoundedExecFile(
+        "docker",
+        ["exec", `agt-${codeName}`, "ps", "-eo", "pid,ppid,args"],
+        { timeoutMs: 8e3, log: log2 }
+      );
+      if (r.kind === "ok") return r.stdout;
+      throw new Error(
+        r.kind === "timed-out" ? `docker exec ps did not answer within its bound (bounded=${r.bounded})` : `docker exec ps failed (status=${r.status}): ${r.message}`
+      );
+    }
+  ) : () => execFileSync2("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 }));
   if (!mcpJson?.mcpServers) {
     return { missing: [], restarted: false, reason: "no-mcp-json" };
   }
@@ -10593,4 +10636,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-BTED7TYA.js.map
+//# sourceMappingURL=chunk-SMY25FCN.js.map