@integrity-labs/agt-cli 0.28.575 → 0.28.577

package/dist/lib/manager-worker.js

@@ -53,7 +53,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-Q76HILWC.js";
+} from "../chunk-FOMHYTFV.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -11596,7 +11596,7 @@ var agentRestartTimezoneInputs = /* @__PURE__ */ new Map();
 var lastVersionCheckAt = 0;
 var VERSION_CHECK_INTERVAL_MS = 5 * 60 * 1e3;
 var lastResponsivenessProbeAt = 0;
-var agtCliVersion = true ? "0.28.575" : "dev";
+var agtCliVersion = true ? "0.28.577" : "dev";
 function resolveBrewPath(execFileSync2) {
   try {
     const out = execFileSync2("which", ["brew"], { timeout: 5e3 }).toString().trim();

package/dist/mcp/index.js

@@ -6802,6 +6802,15 @@ var require_dist = __commonJS({
 
 // src/index.ts
 import { mkdtemp, writeFile } from "fs/promises";
+import {
+  closeSync,
+  constants as fsConstants,
+  fstatSync,
+  lstatSync,
+  openSync,
+  readSync,
+  realpathSync
+} from "fs";
 
 // src/kanban-add-result.ts
 function kanbanAddResultText(data, submittedTitle, status) {
@@ -6818,8 +6827,134 @@ ${first.duplicate_hint}` : added;
 }
 
 // src/index.ts
-import { tmpdir } from "os";
-import { basename, join as join3 } from "path";
+import { homedir as homedir3, tmpdir } from "os";
+import { basename, isAbsolute, join as join3, resolve as pathResolve, sep } from "path";
+
+// src/skill-body-path.ts
+var SKILL_BODY_PATH_TOOLS = /* @__PURE__ */ new Set(["skill_create", "skill_update"]);
+var SKILL_BODY_PATH_MAX_BYTES = 512 * 1024;
+function allowedSkillBodyRoots(agentDir, path) {
+  return [path.join(agentDir, "project"), path.join(agentDir, "scratch")];
+}
+function assertNoSymlinkComponents(target, roots, path, fs) {
+  const root = roots.find((r) => isContained(target, r, path.sep));
+  if (!root) return "resolved outside the allowed roots";
+  const rest = target.slice(root.length).split(path.sep).filter(Boolean);
+  let current = root;
+  for (const segment of rest) {
+    current = path.join(current, segment);
+    try {
+      if (fs.lstatSync(current).isSymbolicLink()) {
+        return `'${current}' is a symlink, so the path changed after it was resolved`;
+      }
+    } catch {
+      return `'${current}' could not be inspected`;
+    }
+  }
+  return null;
+}
+function isContained(candidate, root, sep2) {
+  if (candidate === root) return true;
+  const rootWithSep = root.endsWith(sep2) ? root : root + sep2;
+  return candidate.startsWith(rootWithSep);
+}
+function resolveSkillBodyPath(toolName, args, opts) {
+  if (!SKILL_BODY_PATH_TOOLS.has(toolName)) return { ok: true, args };
+  const raw = args["body_path"];
+  if (raw === void 0 || raw === null) return { ok: true, args };
+  if (typeof raw !== "string" || raw.trim() === "") {
+    return { ok: false, error: "body_path must be a non-empty string." };
+  }
+  if (typeof args["body"] === "string") {
+    return {
+      ok: false,
+      error: "Pass either body or body_path, not both \u2014 they are two ways to say the same thing and I will not guess which one you meant."
+    };
+  }
+  const { agentDir, path, fs } = opts;
+  const maxBytes = opts.maxBytes ?? SKILL_BODY_PATH_MAX_BYTES;
+  const roots = allowedSkillBodyRoots(agentDir, path);
+  const absolute = path.isAbsolute(raw) ? path.resolve(raw) : path.resolve(path.join(agentDir, "project"), raw);
+  if (!roots.some((r) => isContained(absolute, r, path.sep))) {
+    return {
+      ok: false,
+      error: `body_path must be inside your own project or scratch directory. Got '${raw}'. Allowed: ${roots.join(", ")}`
+    };
+  }
+  let real;
+  try {
+    real = fs.realpathSync(absolute);
+  } catch {
+    return { ok: false, error: `body_path does not exist or is unreadable: '${raw}'` };
+  }
+  if (!roots.some((r) => isContained(real, r, path.sep))) {
+    return {
+      ok: false,
+      error: `body_path resolves outside your own directories (symlink?). '${raw}' \u2192 '${real}'. Allowed: ${roots.join(", ")}`
+    };
+  }
+  const walkError = assertNoSymlinkComponents(real, roots, path, fs);
+  if (walkError) return { ok: false, error: `body_path: ${walkError}` };
+  let vetted = null;
+  try {
+    vetted = fs.lstatSync(real);
+  } catch {
+    return { ok: false, error: `body_path vanished before it could be read: '${raw}'` };
+  }
+  let fd;
+  try {
+    fd = fs.openSync(real, opts.openFlags);
+  } catch (err) {
+    const msg = err.message;
+    return {
+      ok: false,
+      error: `body_path could not be opened (it may have been replaced mid-check): '${raw}' \u2014 ${msg}`
+    };
+  }
+  try {
+    let stat;
+    try {
+      stat = fs.fstatSync(fd);
+    } catch {
+      return { ok: false, error: `body_path could not be inspected: '${raw}'` };
+    }
+    if (!stat.isFile()) {
+      return { ok: false, error: `body_path is not a regular file: '${raw}'` };
+    }
+    if (stat.ino !== void 0 && vetted.ino !== void 0) {
+      if (stat.ino !== vetted.ino || stat.dev !== vetted.dev) {
+        return {
+          ok: false,
+          error: `body_path changed identity between the check and the open (it was replaced): '${raw}'`
+        };
+      }
+    }
+    if (stat.size > maxBytes) {
+      return {
+        ok: false,
+        error: `body_path is ${stat.size} bytes, over the ${maxBytes}-byte limit for a skill body.`
+      };
+    }
+    let bytes;
+    try {
+      bytes = fs.readFdSync(fd, maxBytes);
+    } catch (err) {
+      return { ok: false, error: `body_path could not be read: ${err.message}` };
+    }
+    if (bytes.byteLength > maxBytes) {
+      return { ok: false, error: `body_path grew past the ${maxBytes}-byte limit while being read.` };
+    }
+    const body = fs.decodeUtf8(bytes);
+    const next = { ...args, body };
+    delete next["body_path"];
+    return { ok: true, args: next };
+  } finally {
+    try {
+      fs.closeSync(fd);
+    } catch {
+    }
+  }
+}
 
 // ../../node_modules/.pnpm/zod@3.25.76/node_modules/zod/v3/external.js
 var external_exports = {};
@@ -24345,16 +24480,72 @@ async function discoverApiTools() {
     return [];
   }
 }
+function canonicalAgentDir(dir) {
+  try {
+    return realpathSync(dir);
+  } catch {
+    return dir;
+  }
+}
 async function forwardToolCall(toolName, args) {
   if (!AGT_AGENT_ID) {
     throw new Error("Cannot forward tool call: AGT_AGENT_ID is not set");
   }
+  let outgoing = args;
+  if (SKILL_BODY_PATH_TOOLS.has(toolName) && args["body_path"] !== void 0) {
+    if (!AGT_AGENT_CODE_NAME) {
+      throw new Error("Cannot resolve body_path: AGT_AGENT_CODE_NAME is not set");
+    }
+    const resolved = resolveSkillBodyPath(toolName, args, {
+      // ADR-0049 / ENG-7891: the per-agent host dir is agent_id-keyed and the
+      // codename path is only a compatibility SYMLINK. That is not a style
+      // point here, it is a correctness one: the containment check below
+      // realpath-resolves the caller's path, which resolves THROUGH the symlink
+      // to the agent_id-keyed tree - so a codename-based root would fail
+      // containment for every legitimate file and refuse them all. Resolve the
+      // root to its canonical form so both sides of the comparison are real
+      // paths. Falls back to the unresolved join if the dir is missing, which
+      // then simply refuses (there is nothing to read anyway).
+      agentDir: canonicalAgentDir(join3(homedir3(), ".augmented", AGT_AGENT_CODE_NAME)),
+      // agent-dir-allow: immediately realpath-resolved to the ADR-0049 agent_id-keyed dir before use as a containment root
+      path: { isAbsolute, resolve: pathResolve, join: join3, sep },
+      // O_NOFOLLOW is the whole point of opening by flags: the path is already
+      // symlink-resolved, so a symlink at the final component means it was
+      // swapped after the check, and this turns that into an ELOOP.
+      openFlags: fsConstants.O_RDONLY | fsConstants.O_NOFOLLOW,
+      fs: {
+        realpathSync,
+        openSync,
+        fstatSync,
+        closeSync,
+        // Bounded read straight off the descriptor: at most maxBytes + 1 bytes,
+        // so an oversize file is detected rather than buffered whole.
+        lstatSync,
+        // Raw bytes, decoded only after the byte-length check — see
+        // readFdSync's docblock in skill-body-path.ts.
+        readFdSync: (fd, maxBytes) => {
+          const buf = Buffer.alloc(maxBytes + 1);
+          let read = 0;
+          for (; ; ) {
+            const n = readSync(fd, buf, read, buf.length - read, null);
+            if (n <= 0) break;
+            read += n;
+            if (read >= buf.length) break;
+          }
+          return buf.subarray(0, read);
+        },
+        decodeUtf8: (bytes) => Buffer.from(bytes).toString("utf8")
+      }
+    });
+    if (!resolved.ok) throw new Error(resolved.error);
+    outgoing = resolved.args;
+  }
   return apiPost(
     "/host/mcp/tools/call",
     {
       agent_id: AGT_AGENT_ID,
       name: toolName,
-      arguments: args,
+      arguments: outgoing,
       run_id: AGT_RUN_ID ?? null
     },
     false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@integrity-labs/agt-cli",
-  "version": "0.28.575",
+  "version": "0.28.577",
   "description": "Augmented Team CLI — agent provisioning and management",
   "type": "module",
   "engines": {