@wrongstack/core 0.24.0 → 0.32.0

package/dist/coordination/index.js

@@ -1,6 +1,7 @@
 import { randomUUID, randomBytes } from 'crypto';
-import * as fsp5 from 'fs/promises';
+import * as fsp6 from 'fs/promises';
 import * as path4 from 'path';
+import { isAbsolute, resolve } from 'path';
 import { EventEmitter } from 'events';
 
 var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
@@ -11,16 +12,16 @@ var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require
 });
 async function atomicWrite(targetPath, content, opts = {}) {
   const dir = path4.dirname(targetPath);
-  await fsp5.mkdir(dir, { recursive: true });
+  await fsp6.mkdir(dir, { recursive: true });
   const tmp = path4.join(dir, `.${path4.basename(targetPath)}.${randomBytes(6).toString("hex")}.tmp`);
   try {
     if (typeof content === "string") {
-      await fsp5.writeFile(tmp, content, { flag: "wx", encoding: opts.encoding ?? "utf8" });
+      await fsp6.writeFile(tmp, content, { flag: "wx", encoding: opts.encoding ?? "utf8" });
     } else {
-      await fsp5.writeFile(tmp, content, { flag: "wx" });
+      await fsp6.writeFile(tmp, content, { flag: "wx" });
     }
     try {
-      const fh = await fsp5.open(tmp, "r+");
+      const fh = await fsp6.open(tmp, "r+");
       try {
         await fh.sync();
       } finally {
@@ -30,37 +31,37 @@ async function atomicWrite(targetPath, content, opts = {}) {
     }
     let mode;
     try {
-      const stat3 = await fsp5.stat(targetPath);
-      mode = stat3.mode & 511;
+      const stat4 = await fsp6.stat(targetPath);
+      mode = stat4.mode & 511;
     } catch {
       mode = opts.mode;
     }
     if (mode !== void 0) {
-      await fsp5.chmod(tmp, mode);
+      await fsp6.chmod(tmp, mode);
     }
     await renameWithRetry(tmp, targetPath);
   } catch (err) {
     try {
-      await fsp5.unlink(tmp);
+      await fsp6.unlink(tmp);
     } catch {
     }
     throw err;
   }
 }
 async function ensureDir(dir) {
-  await fsp5.mkdir(dir, { recursive: true });
+  await fsp6.mkdir(dir, { recursive: true });
 }
 var TRANSIENT_RENAME_CODES = /* @__PURE__ */ new Set(["EPERM", "EBUSY", "EACCES", "ENOTEMPTY"]);
 async function renameWithRetry(from, to) {
   if (process.platform !== "win32") {
-    await fsp5.rename(from, to);
+    await fsp6.rename(from, to);
     return;
   }
   const delays = [10, 25, 60, 120, 250];
   let lastErr;
   for (let i = 0; i <= delays.length; i++) {
     try {
-      await fsp5.rename(from, to);
+      await fsp6.rename(from, to);
       return;
     } catch (err) {
       lastErr = err;
@@ -68,15 +69,89 @@ async function renameWithRetry(from, to) {
       if (!code || !TRANSIENT_RENAME_CODES.has(code) || i === delays.length) {
         throw err;
       }
-      await new Promise((resolve) => setTimeout(resolve, delays[i]));
+      await new Promise((resolve2) => setTimeout(resolve2, delays[i]));
     }
   }
   throw lastErr;
 }
+
+// src/coordination/large-answer-store.ts
+var LargeAnswerStore = class {
+  /**
+   * Responses above this size (in characters) are stored out-of-context.
+   * Below this, the full answer is returned inline (no overhead).
+   * Default: 2000 chars ≈ 400-600 tokens.
+   */
+  sizeThreshold;
+  store = /* @__PURE__ */ new Map();
+  constructor(sizeThreshold = 2e3) {
+    this.sizeThreshold = sizeThreshold;
+  }
+  /**
+   * Store a value, returning a summary + key for inline use.
+   * If the value is below sizeThreshold, returns it as-is (no store entry).
+   */
+  storeAnswer(value) {
+    if (value === void 0 || value === null) {
+      return { summary: String(value), inline: true };
+    }
+    const serialized = typeof value === "string" ? value : JSON.stringify(value);
+    const size = serialized.length;
+    if (size <= this.sizeThreshold) {
+      return { summary: serialized.slice(0, 500), inline: true };
+    }
+    const key = `a-${hashStr(serialized)}`;
+    this.store.set(key, {
+      key,
+      value,
+      size,
+      storedAt: Date.now()
+    });
+    return {
+      key,
+      summary: `[stored: ${size} chars \u2014 use roll_up or ask_result tool to retrieve, key=${key}]`,
+      inline: false
+    };
+  }
+  /**
+   * Retrieve a previously stored answer by its key.
+   * Returns undefined if the key is unknown or the store was cleared.
+   */
+  retrieveAnswer(key) {
+    return this.store.get(key)?.value;
+  }
+  /**
+   * Check if a key exists in the store.
+   */
+  hasAnswer(key) {
+    return this.store.has(key);
+  }
+  /** Number of stored entries. */
+  get size() {
+    return this.store.size;
+  }
+  /** Total characters stored. */
+  get totalChars() {
+    let total = 0;
+    for (const e of this.store.values()) total += e.size;
+    return total;
+  }
+  /** Clear all stored entries. Call at the end of a director run. */
+  clear() {
+    this.store.clear();
+  }
+};
+function hashStr(s) {
+  let h = 5381;
+  for (let i = 0; i < s.length; i++) {
+    h = h * 33 ^ s.charCodeAt(i);
+  }
+  return (h >>> 0).toString(36);
+}
 async function acquireDirectorStateLock(lockPath, processId = process.pid) {
   let existing;
   try {
-    existing = await fsp5.readFile(lockPath, "utf8");
+    existing = await fsp6.readFile(lockPath, "utf8");
   } catch {
   }
   if (existing) {
@@ -100,7 +175,7 @@ async function acquireDirectorStateLock(lockPath, processId = process.pid) {
 }
 async function releaseDirectorStateLock(lockPath) {
   try {
-    await fsp5.unlink(lockPath);
+    await fsp6.unlink(lockPath);
   } catch {
   }
 }
@@ -334,7 +409,7 @@ var InMemoryAgentBridge = class {
       );
     }
     this.inflightGuards.add(correlationId);
-    return new Promise((resolve, reject) => {
+    return new Promise((resolve2, reject) => {
       const timer = setTimeout(() => {
         this.inflightGuards.delete(correlationId);
         this.pendingRequests.delete(correlationId);
@@ -346,7 +421,7 @@ var InMemoryAgentBridge = class {
         return;
       }
       this.pendingRequests.set(correlationId, {
-        resolve,
+        resolve: resolve2,
         reject,
         timer
       });
@@ -707,6 +782,10 @@ var NICKNAME_POOL = {
   "pasteur": { name: "Pasteur", domain: "biology" },
   "hawking": { name: "Hawking", domain: "cosmology" },
   "sagan": { name: "Sagan", domain: "cosmology" },
+  // Exploration & navigation
+  "columbus": { name: "Columbus", domain: "exploration" },
+  "polo": { name: "Polo", domain: "exploration" },
+  "magellan": { name: "Magellan", domain: "exploration" },
   // Chemistry / materials
   "lavoisier": { name: "Lavoisier", domain: "chemistry" },
   "mendeleev": { name: "Mendeleev", domain: "chemistry" }
@@ -1083,12 +1162,12 @@ var SubagentBudget = class _SubagentBudget {
           if (!bus || !bus.hasListenerFor("budget.threshold_reached")) {
             return Promise.resolve("stop");
           }
-          return new Promise((resolve) => {
+          return new Promise((resolve2) => {
             let resolved = false;
             const respond = (d) => {
               if (resolved) return;
               resolved = true;
-              resolve(d);
+              resolve2(d);
             };
             const fallback = setTimeout(
               () => respond("stop"),
@@ -3686,7 +3765,7 @@ var FLEET_ROSTER_WITHACP = {
 };
 
 // src/coordination/multi-agent-coordinator.ts
-var DefaultMultiAgentCoordinator = class extends EventEmitter {
+var DefaultMultiAgentCoordinator = class _DefaultMultiAgentCoordinator extends EventEmitter {
   coordinatorId;
   config;
   runner;
@@ -3701,8 +3780,12 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
    * names) to memorable ones here — that's what surfaces in the fleet monitor.
    */
   usedNicknames = /* @__PURE__ */ new Set();
+  /** Maps subagentId → nickname key (e.g. 'einstein'). Used to free the slot on remove(). */
+  subagentNicknames = /* @__PURE__ */ new Map();
   pendingTasks = [];
   completedResults = [];
+  /** Prevents completedResults from growing unbounded in long-running coordinators. */
+  static MAX_COMPLETED_RESULTS = 1e4;
   totalIterations = 0;
   inFlight = 0;
   /**
@@ -3757,7 +3840,7 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
    * Explicit, human-chosen names — including nicknames already assigned by
    * `Director.spawn()` — are left untouched, so this never double-assigns.
    */
-  withNickname(subagent) {
+  withNickname(subagent, subagentId) {
     const role = subagent.role ?? "subagent";
     const name = subagent.name?.trim() ?? "";
     const isPlaceholder = name === "" || name.toLowerCase() === role.toLowerCase() || name === "subagent" || name === "adhoc" || name === "generic" || /^slot-/.test(name);
@@ -3765,11 +3848,12 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
     const nickname = assignNickname(role, this.usedNicknames);
     const baseKey = nickname.split(" ")[0].toLowerCase().replace(/[^a-z0-9-]/g, "-");
     this.usedNicknames.add(baseKey);
+    this.subagentNicknames.set(subagentId, baseKey);
     return { ...subagent, name: nickname };
   }
   async spawn(subagent) {
-    subagent = this.withNickname(subagent);
     const id = subagent.id || randomUUID();
+    subagent = this.withNickname(subagent, id);
     if (this.subagents.has(id)) {
       throw new Error(`Subagent id "${id}" already exists \u2014 refusing to overwrite`);
     }
@@ -3913,7 +3997,7 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
       taskIds.map((id) => {
         const cached = this.completedResults.find((r) => r.taskId === id);
         if (cached) return cached;
-        return new Promise((resolve, reject) => {
+        return new Promise((resolve2, reject) => {
           const timeout = setTimeout(() => {
             this.off("task.completed", handler);
             reject(new Error(`awaitTasks timed out waiting for task "${id}"`));
@@ -3922,7 +4006,7 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
             if (result.taskId === id) {
               clearTimeout(timeout);
               this.off("task.completed", handler);
-              resolve(result);
+              resolve2(result);
             }
           };
           this.on("task.completed", handler);
@@ -4211,6 +4295,12 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
   }
   recordCompletion(result) {
     this.completedResults.push(result);
+    if (this.completedResults.length > _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS) {
+      this.completedResults.splice(
+        0,
+        this.completedResults.length - _DefaultMultiAgentCoordinator.MAX_COMPLETED_RESULTS
+      );
+    }
     this.totalIterations += result.iterations;
     if (this.inFlight > 0) {
       this.inFlight--;
@@ -4280,7 +4370,29 @@ var DefaultMultiAgentCoordinator = class extends EventEmitter {
     }
     this.subagents.delete(subagentId);
     this.terminating.delete(subagentId);
+    const nicknameKey = this.subagentNicknames.get(subagentId);
+    if (nicknameKey) {
+      this.usedNicknames.delete(nicknameKey);
+      this.subagentNicknames.delete(subagentId);
+    }
+    const orphaned = this.pendingTasks.filter((t) => t.subagentId === subagentId);
     this.pendingTasks = this.pendingTasks.filter((t) => t.subagentId !== subagentId);
+    for (const t of orphaned) {
+      const synthetic = {
+        subagentId,
+        taskId: t.id,
+        status: "stopped",
+        error: {
+          kind: "aborted_by_parent",
+          message: `Subagent "${subagentId}" was removed while task "${t.id}" was pending`,
+          retryable: false
+        },
+        iterations: 0,
+        toolCalls: 0,
+        durationMs: 0
+      };
+      this.recordCompletion(synthetic);
+    }
     this.fleetBus?.emit({
       subagentId,
       ts: Date.now(),
@@ -4528,9 +4640,12 @@ function makeSpawnTool(director, roster) {
       provider: { type: "string", description: 'Provider id (e.g. "anthropic", "openai"). Defaults to the leader provider when omitted.' },
       model: { type: "string", description: "Model id within the provider. Defaults to the leader model when omitted." },
       systemPromptOverride: { type: "string", description: "Extra prompt text appended after the role-base prompt." },
-      maxIterations: { type: "number" },
-      maxToolCalls: { type: "number" },
-      maxCostUsd: { type: "number" }
+      maxIterations: { type: "number", minimum: 1 },
+      maxToolCalls: { type: "number", minimum: 1 },
+      maxCostUsd: { type: "number", minimum: 0 },
+      timeoutMs: { type: "number", minimum: 1, description: "Hard wall-clock cap in milliseconds. Defaults to none (idle timeout is the default reaper)." },
+      idleTimeoutMs: { type: "number", minimum: 1, description: "Idle timeout in ms: reap the subagent after this long with no activity. Resets on every iteration/tool call. Default is role/coordinator-specific." },
+      maxTokens: { type: "number", minimum: 1, description: "Maximum total tokens (input + output) the subagent may use." }
     },
     required: []
   };
@@ -4576,6 +4691,9 @@ function makeSpawnTool(director, roster) {
       if (typeof i.maxIterations === "number") cfg.maxIterations = i.maxIterations;
       if (typeof i.maxToolCalls === "number") cfg.maxToolCalls = i.maxToolCalls;
       if (typeof i.maxCostUsd === "number") cfg.maxCostUsd = i.maxCostUsd;
+      if (typeof i.timeoutMs === "number") cfg.timeoutMs = i.timeoutMs;
+      if (typeof i.idleTimeoutMs === "number") cfg.idleTimeoutMs = i.idleTimeoutMs;
+      if (typeof i.maxTokens === "number") cfg.maxTokens = i.maxTokens;
       try {
         const subagentId = await director.spawn(cfg);
         return { subagentId, provider: cfg.provider, model: cfg.model, name: cfg.name, role: cfg.role };
@@ -4603,10 +4721,10 @@ function makeAssignTool(director) {
   const inputSchema = {
     type: "object",
     properties: {
-      subagentId: { type: "string", description: "Target subagent id. Required." },
-      description: { type: "string", description: "The task in natural language \u2014 what you want this subagent to do." },
-      maxToolCalls: { type: "number", description: "Optional per-task tool-call budget override." },
-      timeoutMs: { type: "number", description: "Optional per-task timeout in ms." }
+      subagentId: { type: "string", minLength: 1, description: "Target subagent id. Required." },
+      description: { type: "string", minLength: 1, description: "The task in natural language \u2014 what you want this subagent to do." },
+      maxToolCalls: { type: "number", minimum: 1, description: "Optional per-task tool-call budget override." },
+      timeoutMs: { type: "number", minimum: 1, description: "Optional per-task timeout in ms." }
     },
     required: ["subagentId", "description"]
   };
@@ -4647,9 +4765,9 @@ function makeAskTool(director) {
     inputSchema: {
       type: "object",
       properties: {
-        subagentId: { type: "string", description: "Subagent to ask. Must be a previously spawned id." },
-        question: { type: "string", description: "The question or instruction." },
-        timeoutMs: { type: "number", description: "Optional timeout in ms (default 30s)." }
+        subagentId: { type: "string", minLength: 1, description: "Subagent to ask. Must be a previously spawned id." },
+        question: { type: "string", minLength: 1, description: "The question or instruction." },
+        timeoutMs: { type: "number", minimum: 1, description: "Optional timeout in ms (default 30s)." }
       },
       required: ["subagentId", "question"]
     },
@@ -4657,13 +4775,49 @@ function makeAskTool(director) {
       const i = input;
       try {
         const answer = await director.ask(i.subagentId, { question: i.question }, i.timeoutMs);
-        return { ok: true, answer };
+        const stored = director.largeAnswerStore.storeAnswer(answer);
+        if (stored.inline) {
+          return { ok: true, answer: stored.summary };
+        }
+        return {
+          ok: true,
+          answer: stored.summary,
+          _answerKey: stored.key,
+          _hint: "Response was large and stored. Use ask_result with the key to retrieve it."
+        };
       } catch (err) {
         return { ok: false, error: err instanceof Error ? err.message : String(err) };
       }
     }
   };
 }
+function makeAskResultTool(director) {
+  return {
+    name: "ask_result",
+    description: "Retrieve a large `ask_subagent` response that was stored out-of-context (>2K chars). Returns the full stored value.",
+    permission: "auto",
+    mutating: false,
+    inputSchema: {
+      type: "object",
+      properties: {
+        key: {
+          type: "string",
+          minLength: 1,
+          description: "The `_answerKey` returned by `ask_subagent` for a large response."
+        }
+      },
+      required: ["key"]
+    },
+    async execute(input) {
+      const i = input;
+      const value = director.largeAnswerStore.retrieveAnswer(i.key);
+      if (value === void 0) {
+        return { ok: false, error: `No stored answer found for key "${i.key}" \u2014 it may have been cleared or the key is invalid.` };
+      }
+      return { ok: true, value };
+    }
+  };
+}
 function makeRollUpTool(director) {
   return {
     name: "roll_up",
@@ -4817,7 +4971,18 @@ function makeCollabDebugTool(director) {
         },
         timeoutMs: {
           type: "number",
+          minimum: 1,
           description: "Timeout in ms. Default: 600000 (10 minutes)."
+        },
+        maxTargetFiles: {
+          type: "number",
+          minimum: 1,
+          description: "Maximum number of files to include in the snapshot. If not set, the limit is computed dynamically from contextWindow or falls back to the default (30)."
+        },
+        contextWindow: {
+          type: "number",
+          minimum: 1,
+          description: "Context window size (tokens) of the model. When provided and maxTargetFiles is not set, the file limit is computed dynamically as floor((contextWindow * 0.4) / 2000)."
         }
       },
       required: ["targetPaths"]
@@ -4829,7 +4994,9 @@ function makeCollabDebugTool(director) {
       }
       const options = {
         targetPaths: i.targetPaths,
-        timeoutMs: i.timeoutMs
+        timeoutMs: i.timeoutMs,
+        maxTargetFiles: i.maxTargetFiles,
+        contextWindow: i.contextWindow
       };
       try {
         const report = await director.spawnCollab(options);
@@ -4896,6 +5063,123 @@ function makeWorkCompleteTool(director) {
     }
   };
 }
+var GLOB_CHARS = /* @__PURE__ */ new Set(["*", "?", "["]);
+var IS_WINDOWS = process.platform === "win32";
+var SEP = IS_WINDOWS ? "\\" : "/";
+function isGlob(p) {
+  for (const c of p) {
+    if (GLOB_CHARS.has(c)) return true;
+  }
+  return false;
+}
+function globToRegex(pat) {
+  let i = 0, re = "^";
+  while (i < pat.length) {
+    const c = pat[i];
+    if (c === "*") {
+      if (pat[i + 1] === "*") {
+        re += ".*";
+        i += 2;
+        if (pat[i] === "/") i++;
+      } else {
+        re += "[^/\\\\]*";
+        i++;
+      }
+    } else if (c === "?") {
+      re += "[^/\\\\]";
+      i++;
+    } else if (c === "[") {
+      let cls = "[";
+      i++;
+      if (pat[i] === "!" || pat[i] === "^") {
+        cls += "^";
+        i++;
+      }
+      while (i < pat.length && pat[i] !== "]") {
+        const ch = pat[i] ?? "";
+        if (ch === "\\") cls += "\\\\";
+        else if (ch === "]" || ch === "^") cls += `\\${ch}`;
+        else cls += ch;
+        i++;
+      }
+      cls += "]";
+      re += cls;
+      i++;
+    } else {
+      re += c.replace(/[.+^${}()|\\]/g, "\\$&");
+      i++;
+    }
+  }
+  return new RegExp(re + "$");
+}
+function baseDir(pat) {
+  let i = pat.length - 1;
+  while (i >= 0 && !GLOB_CHARS.has(pat[i]) && pat[i] !== SEP && pat[i] !== "/") i--;
+  const cut = i >= 0 ? pat.lastIndexOf(SEP, i) : pat.lastIndexOf("/", i);
+  return cut < 0 ? "." : pat.slice(0, cut);
+}
+async function expandGlob(pattern) {
+  if (!isGlob(pattern)) return [pattern];
+  const results = /* @__PURE__ */ new Set();
+  const abs = isAbsolute(pattern);
+  const base = abs ? baseDir(pattern) : baseDir(pattern);
+  const relPat = base === "." ? pattern : pattern.slice(base.length + 1);
+  async function walk(dir, pat) {
+    let entries;
+    try {
+      entries = await fsp6.readdir(dir);
+    } catch {
+      return;
+    }
+    const firstGlob = pat.search(/[*?[\[]/);
+    if (firstGlob < 0) {
+      const re = globToRegex(pat);
+      for (const e of entries) {
+        if (re.test(e)) {
+          const full = `${dir}${SEP}${e}`;
+          results.add(abs ? resolve(full) : full);
+        }
+      }
+      return;
+    }
+    const before = pat.slice(0, firstGlob);
+    const rest = pat.slice(firstGlob);
+    if (before.endsWith("**")) {
+      await walk(dir, rest);
+      for (const e of entries) {
+        const full = `${dir}${SEP}${e}`;
+        try {
+          const stat4 = await fsp6.stat(full);
+          if (stat4.isDirectory()) await walk(full, rest);
+        } catch {
+        }
+      }
+    } else if (before === "") {
+      const re = globToRegex(rest);
+      for (const e of entries) {
+        if (re.test(e)) {
+          const full = `${dir}${SEP}${e}`;
+          results.add(abs ? resolve(full) : full);
+        }
+      }
+    } else {
+      const seg = before.replace(/[*?[\]]/g, "").replace(/\/$/, "");
+      if (entries.includes(seg)) {
+        const full = `${dir}${SEP}${seg}`;
+        try {
+          const stat4 = await fsp6.stat(full);
+          if (stat4.isDirectory()) await walk(full, rest);
+        } catch {
+        }
+      }
+    }
+  }
+  await walk(base === "." ? "." : base, relPat);
+  return [...results];
+}
+
+// src/coordination/collab-debug.ts
+var DEFAULT_MAX_TARGET_FILES = 30;
 var DirectorAlertLevel = /* @__PURE__ */ ((DirectorAlertLevel2) => {
   DirectorAlertLevel2["WARNING"] = "warning";
   DirectorAlertLevel2["CRITICAL"] = "critical";
@@ -4958,11 +5242,36 @@ var CollabSession = class extends EventEmitter {
   getSubagentIds() {
     return new Map(this.subagentIds);
   }
+  /**
+   * Returns the effective file limit for this session.
+   * Priority: explicit `maxTargetFiles` > dynamic from `contextWindow` > `DEFAULT_MAX_TARGET_FILES`.
+   */
+  effectiveFileLimit() {
+    if (this.options.maxTargetFiles !== void 0) {
+      return this.options.maxTargetFiles;
+    }
+    if (this.options.contextWindow !== void 0) {
+      return Math.max(5, Math.floor(this.options.contextWindow * 0.4 / 2e3));
+    }
+    return DEFAULT_MAX_TARGET_FILES;
+  }
   async buildSnapshot() {
     if (this.snapshot.files.length > 0) return this.snapshot;
-    for (const filePath of this.options.targetPaths) {
+    const allFiles = [];
+    for (const pattern of this.options.targetPaths) {
+      const expanded = await expandGlob(pattern);
+      allFiles.push(...expanded);
+    }
+    const limit = this.effectiveFileLimit();
+    if (allFiles.length > limit) {
+      const hint = this.options.contextWindow ? `contextWindow=${this.options.contextWindow} \u2192 calculated limit=${limit}` : `default limit=${DEFAULT_MAX_TARGET_FILES}`;
+      throw new Error(
+        `[collab_debug] Target has ${allFiles.length} files, which exceeds the limit (${hint}). Narrow the target or pass maxTargetFiles / contextWindow to override. For large codebases, run package-by-package or module-by-module sessions instead of targeting the entire repo.`
+      );
+    }
+    for (const filePath of allFiles) {
       try {
-        const content = await fsp5.readFile(filePath, "utf8");
+        const content = await fsp6.readFile(filePath, "utf8");
         const ext = filePath.split(".").pop() ?? "";
         const language = ext === "ts" || ext === "tsx" ? "typescript" : ext === "js" || ext === "jsx" ? "javascript" : ext === "md" ? "markdown" : ext === "json" ? "json" : void 0;
         this.snapshot.files.push({ path: filePath, content, language });
@@ -5016,7 +5325,7 @@ var CollabSession = class extends EventEmitter {
         reject(new Error(`CollabSession timed out after ${this.timeoutMs}ms`));
       }, this.timeoutMs);
     });
-    let results;
+    let results = null;
     try {
       results = await Promise.race([
         Promise.all([
@@ -5447,7 +5756,7 @@ var FleetContextOverflowError = class extends Error {
     this.observed = observed;
   }
 };
-var Director = class {
+var Director = class _Director {
   /** Alias for the ICoordinator contract. `id` is retained for backward compatibility. */
   get coordinatorId() {
     return this.id;
@@ -5505,6 +5814,8 @@ var Director = class {
    *  coordinator already fired the event — `awaitTasks(['t-1'])` after
    *  t-1 finished should resolve immediately, not hang. */
   completed = /* @__PURE__ */ new Map();
+  /** Prevents the completed Map from growing unbounded in long-running directors. */
+  static MAX_COMPLETED = 1e4;
   /** Per-subagent provider/model metadata, captured at spawn time so the
    *  FleetUsageAggregator's metaLookup can surface readable rows. */
   subagentMeta = /* @__PURE__ */ new Map();
@@ -5584,6 +5895,8 @@ var Director = class {
   _leaderBtwNotes = [];
   /** Active collab sessions tracked by sessionId (see spawnCollab). */
   _activeCollabSessions = /* @__PURE__ */ new Map();
+  /** Prevents large `ask_subagent` answers from bloating the leader's context window. */
+  largeAnswerStore;
   constructor(opts) {
     this.id = opts.config.coordinatorId || randomUUID();
     this.manifestPath = opts.manifestPath;
@@ -5612,7 +5925,7 @@ var Director = class {
     }, opts.checkpointDebounceMs ?? 250) : null;
     this.fleetManager = opts.fleetManager;
     if (this.sharedScratchpadPath) {
-      void fsp5.mkdir(this.sharedScratchpadPath, { recursive: true }).catch(
+      void fsp6.mkdir(this.sharedScratchpadPath, { recursive: true }).catch(
         (err) => this.logShutdownError("shared_scratchpad_mkdir", err)
       );
     }
@@ -5644,6 +5957,11 @@ var Director = class {
     this.taskCompletedListener = (payload) => {
       const r = payload.result;
       this.completed.set(r.taskId, r);
+      if (this.completed.size > _Director.MAX_COMPLETED) {
+        const toDelete = this.completed.size - _Director.MAX_COMPLETED;
+        const keys = [...this.completed.keys()].slice(0, toDelete);
+        for (const k of keys) this.completed.delete(k);
+      }
       const waiter = this.taskWaiters.get(r.taskId);
       if (waiter) {
         waiter.resolve(r);
@@ -5750,6 +6068,7 @@ var Director = class {
         payload.extend(extra);
       });
     });
+    this.largeAnswerStore = new LargeAnswerStore(2e3);
   }
   /**
    * Record a granted budget extension and broadcast it on the FleetBus so
@@ -5976,6 +6295,20 @@ var Director = class {
     );
     this.coordinator.setSubagentBridge(result.subagentId, subagentBridge);
     this.subagentBridges.set(result.subagentId, subagentBridge);
+    this.fleet.emit({
+      subagentId: result.subagentId,
+      ts: Date.now(),
+      type: "subagent.spawned",
+      payload: {
+        subagentId: result.subagentId,
+        taskId: "",
+        // taskId will be set when assign() is called
+        name: config.name,
+        role: config.role,
+        provider: config.provider,
+        model: config.model
+      }
+    });
     if (!this.fleetManager) {
       this.manifestEntries.set(result.subagentId, {
         subagentId: result.subagentId,
@@ -6115,7 +6448,7 @@ var Director = class {
       })),
       usage: this.usage.snapshot()
     };
-    await fsp5.mkdir(path4.dirname(this.manifestPath), { recursive: true });
+    await fsp6.mkdir(path4.dirname(this.manifestPath), { recursive: true });
     await atomicWrite(this.manifestPath, JSON.stringify(manifest, null, 2), { mode: 384 });
     return this.manifestPath;
   }
@@ -6230,11 +6563,11 @@ var Director = class {
         if (cached) return cached;
         const existing = this.taskWaiters.get(id);
         if (existing) return existing.promise;
-        let resolve;
+        let resolve2;
         const promise = new Promise((res) => {
-          resolve = res;
+          resolve2 = res;
         });
-        this.taskWaiters.set(id, { promise, resolve });
+        this.taskWaiters.set(id, { promise, resolve: resolve2 });
         return promise;
       })
     );
@@ -6247,7 +6580,24 @@ var Director = class {
   }
   async remove(subagentId) {
     await this.coordinator.remove(subagentId);
+    const bridge = this.subagentBridges.get(subagentId);
+    if (bridge) {
+      await bridge.stop();
+      this.subagentBridges.delete(subagentId);
+    }
     this.usage.removeSubagent(subagentId);
+    if (this.fleetManager) {
+      this.fleetManager.removeSubagent(subagentId);
+    } else {
+      const entry = this.manifestEntries.get(subagentId);
+      if (entry?.name) {
+        const nicknameKey = entry.name.split(" ")[0].toLowerCase().replace(/[^a-z0-9-]/g, "-");
+        this._usedNicknames.delete(nicknameKey);
+      }
+    }
+    this.manifestEntries.delete(subagentId);
+    this.taskOwners.delete(subagentId);
+    this.taskDescriptions.delete(subagentId);
   }
   status() {
     const base = this.coordinator.getStatus();
@@ -6303,7 +6653,7 @@ var Director = class {
     const filePath = path4.join(this.sessionsRoot, this.directorRunId, `${subagentId}.jsonl`);
     let raw;
     try {
-      raw = await fsp5.readFile(filePath, "utf8");
+      raw = await fsp6.readFile(filePath, "utf8");
     } catch {
       return null;
     }
@@ -6409,6 +6759,7 @@ var Director = class {
       makeAssignTool(this),
       makeAwaitTasksTool(this),
       makeAskTool(this),
+      makeAskResultTool(this),
       makeRollUpTool(this),
       makeTerminateTool(this),
       makeTerminateAllTool(this),
@@ -6491,15 +6842,33 @@ function createDelegateTool(opts) {
       },
       timeoutMs: {
         type: "number",
+        minimum: 1,
         description: `Wall-clock budget for this delegate in milliseconds. No hard cap \u2014 set as high as the task realistically needs (a monorepo audit can take hours, a single-file lint takes seconds). Default ${Math.round(defaultTimeoutMs / 1e3 / 60)} minutes.`
       },
       maxIterations: {
         type: "number",
+        minimum: 1,
         description: "Maximum LLM iterations the subagent may take. Unset = use the role/coordinator default. Raise this for tasks with many tool-think-tool cycles (deep code analysis, multi-file refactors)."
       },
       maxToolCalls: {
         type: "number",
+        minimum: 1,
         description: "Maximum number of tool invocations the subagent may make. Unset = use the role/coordinator default. Raise this for tasks that touch many files (large grep + read + report)."
+      },
+      idleTimeoutMs: {
+        type: "number",
+        minimum: 1,
+        description: "Idle timeout in ms: reap the subagent after this long with no activity. Resets on every iteration/tool call. Unset = use the role/coordinator default."
+      },
+      maxTokens: {
+        type: "number",
+        minimum: 1,
+        description: "Maximum total tokens (input + output) the subagent may use. Unset = use the role/coordinator default."
+      },
+      maxCostUsd: {
+        type: "number",
+        minimum: 0,
+        description: "Maximum estimated USD cost the subagent may incur. Unset = use the role/coordinator default."
       }
     },
     required: ["task"]
@@ -6557,12 +6926,21 @@ function createDelegateTool(opts) {
           };
           cfg = applyRosterBudget({ ...cfg, name: i.name });
         }
-        if (typeof i.maxIterations === "number" && i.maxIterations > 0) {
+        if (typeof i.maxIterations === "number") {
           cfg.maxIterations = i.maxIterations;
         }
-        if (typeof i.maxToolCalls === "number" && i.maxToolCalls > 0) {
+        if (typeof i.maxToolCalls === "number") {
           cfg.maxToolCalls = i.maxToolCalls;
         }
+        if (typeof i.idleTimeoutMs === "number") {
+          cfg.idleTimeoutMs = i.idleTimeoutMs;
+        }
+        if (typeof i.maxTokens === "number") {
+          cfg.maxTokens = i.maxTokens;
+        }
+        if (typeof i.maxCostUsd === "number") {
+          cfg.maxCostUsd = i.maxCostUsd;
+        }
         const SUBAGENT_TIMEOUT_BUFFER_MS = opts.subagentTimeoutBufferMs ?? 6e4;
         if (!cfg.timeoutMs) {
           cfg.timeoutMs = Math.max(3e4, timeoutMs - SUBAGENT_TIMEOUT_BUFFER_MS);
@@ -6574,7 +6952,7 @@ function createDelegateTool(opts) {
           subagentId
         });
         const dir = director;
-        const result = await new Promise((resolve) => {
+        const result = await new Promise((resolve2) => {
           let settled = false;
           let timer;
           const finish = (value) => {
@@ -6584,7 +6962,7 @@ function createDelegateTool(opts) {
             offTool();
             offIter();
             offProgress();
-            resolve(value);
+            resolve2(value);
           };
           const arm = () => {
             if (timer) clearTimeout(timer);
@@ -6731,7 +7109,7 @@ async function readSubagentPartial(opts, subagentId) {
     candidates.push(path4.join(opts.sessionsRoot, opts.directorRunId, `${subagentId}.jsonl`));
   } else {
     try {
-      const entries = await fsp5.readdir(opts.sessionsRoot, { withFileTypes: true });
+      const entries = await fsp6.readdir(opts.sessionsRoot, { withFileTypes: true });
       for (const entry of entries) {
         if (entry.isDirectory()) {
           candidates.push(path4.join(opts.sessionsRoot, entry.name, `${subagentId}.jsonl`));
@@ -6744,7 +7122,7 @@ async function readSubagentPartial(opts, subagentId) {
   for (const file of candidates) {
     let raw;
     try {
-      raw = await fsp5.readFile(file, "utf8");
+      raw = await fsp6.readFile(file, "utf8");
     } catch {
       continue;
     }
@@ -7063,7 +7441,7 @@ var DefaultSessionStore = class {
     const file = path4.join(shardDir, `${id}.jsonl`);
     let handle;
     try {
-      handle = await fsp5.open(file, "a", 384);
+      handle = await fsp6.open(file, "a", 384);
     } catch (err) {
       throw new Error(
         `Failed to open session file: ${err instanceof Error ? err.message : String(err)}`,
@@ -7087,7 +7465,7 @@ var DefaultSessionStore = class {
     const data = await this.load(id);
     let handle;
     try {
-      handle = await fsp5.open(file, "a", 384);
+      handle = await fsp6.open(file, "a", 384);
     } catch (err) {
       throw new Error(
         `Failed to open session "${id}" for append: ${err instanceof Error ? err.message : String(err)}`,
@@ -7116,7 +7494,7 @@ var DefaultSessionStore = class {
   }
   async load(id) {
     const file = this.sessionPath(id, ".jsonl");
-    const raw = await fsp5.readFile(file, "utf8");
+    const raw = await fsp6.readFile(file, "utf8");
     const lines = raw.split("\n").filter((l) => l.trim());
     const events = [];
     for (const line of lines) {
@@ -7151,7 +7529,7 @@ var DefaultSessionStore = class {
   /** Recursively collect all session IDs from shard subdirectories. */
   async collectSessionIds(dir) {
     const ids = [];
-    const entries = await fsp5.readdir(dir, { withFileTypes: true });
+    const entries = await fsp6.readdir(dir, { withFileTypes: true });
     for (const entry of entries) {
       const full = path4.join(dir, entry.name);
       if (entry.isDirectory()) {
@@ -7165,12 +7543,12 @@ var DefaultSessionStore = class {
   async summaryFor(id) {
     const manifest = this.sessionPath(id, ".summary.json");
     try {
-      const raw = await fsp5.readFile(manifest, "utf8");
+      const raw = await fsp6.readFile(manifest, "utf8");
       return JSON.parse(raw);
     } catch {
       const full = this.sessionPath(id, ".jsonl");
-      const stat3 = await fsp5.stat(full);
-      const summary = await this.summarize(id, stat3.mtime.toISOString());
+      const stat4 = await fsp6.stat(full);
+      const summary = await this.summarize(id, stat4.mtime.toISOString());
       await atomicWrite(manifest, JSON.stringify(summary), { mode: 384 }).catch((err) => {
         console.warn(
           `[session-store] Failed to write manifest for "${id}":`,
@@ -7181,8 +7559,8 @@ var DefaultSessionStore = class {
     }
   }
   async delete(id) {
-    await fsp5.unlink(this.sessionPath(id, ".jsonl"));
-    await fsp5.unlink(this.sessionPath(id, ".summary.json")).catch(() => void 0);
+    await fsp6.unlink(this.sessionPath(id, ".jsonl"));
+    await fsp6.unlink(this.sessionPath(id, ".summary.json")).catch(() => void 0);
   }
   async clearHistory(id) {
     await this.ensureShardDir(id);
@@ -7196,8 +7574,8 @@ var DefaultSessionStore = class {
       provider: "unknown"
     })}
 `;
-    await fsp5.writeFile(file, record, "utf8");
-    await fsp5.unlink(meta).catch(() => void 0);
+    await fsp6.writeFile(file, record, "utf8");
+    await fsp6.unlink(meta).catch(() => void 0);
   }
   async summarize(id, mtime) {
     try {
@@ -7383,7 +7761,7 @@ var FileSessionWriter = class {
 `;
     try {
       if (this.filePath) {
-        await fsp5.writeFile(this.filePath, record, { flag: "a", mode: 384 });
+        await fsp6.writeFile(this.filePath, record, { flag: "a", mode: 384 });
       }
     } catch {
     }
@@ -7476,7 +7854,7 @@ var FileSessionWriter = class {
   }
   async truncateToCheckpoint(targetPromptIndex) {
     if (!this.filePath) return 0;
-    const raw = await fsp5.readFile(this.filePath, "utf8");
+    const raw = await fsp6.readFile(this.filePath, "utf8");
     const lines = raw.split("\n");
     const kept = [];
     let removedCount = 0;
@@ -7514,13 +7892,13 @@ var FileSessionWriter = class {
     }
     const truncated = kept.join("\n");
     const tmpPath = `${this.filePath}.rewind.tmp`;
-    await fsp5.writeFile(tmpPath, truncated + "\n", "utf8");
+    await fsp6.writeFile(tmpPath, truncated + "\n", "utf8");
     try {
       await this.handle.close();
-      await fsp5.rename(tmpPath, this.filePath);
-      this.handle = await fsp5.open(this.filePath, "a", 384);
+      await fsp6.rename(tmpPath, this.filePath);
+      this.handle = await fsp6.open(this.filePath, "a", 384);
     } catch (err) {
-      await fsp5.unlink(tmpPath).catch(() => void 0);
+      await fsp6.unlink(tmpPath).catch(() => void 0);
       throw err;
     }
     await this.append({
@@ -7546,7 +7924,7 @@ var FileSessionWriter = class {
       provider: this.meta.provider ?? "unknown"
     })}
 `;
-    await fsp5.writeFile(this.filePath, record, "utf8");
+    await fsp6.writeFile(this.filePath, record, "utf8");
   }
   /**
    * Idea #1 — write an in-flight marker. The agent loop should call
@@ -7892,7 +8270,7 @@ var FleetManager = class {
       })),
       usage: this.usage.snapshot()
     };
-    await fsp5.mkdir(path4.dirname(this.manifestPath), { recursive: true });
+    await fsp6.mkdir(path4.dirname(this.manifestPath), { recursive: true });
     await atomicWrite(this.manifestPath, JSON.stringify(manifest, null, 2), { mode: 384 });
     return this.manifestPath;
   }
@@ -8002,6 +8380,23 @@ var FleetManager = class {
     }));
     return { pending, live: [] };
   }
+  /**
+   * Clean up all fleet-manager state associated with a removed subagent:
+   * - Frees the nickname slot so the same name can be reused
+   * - Removes any pending tasks for this subagent
+   */
+  removeSubagent(subagentId) {
+    const entry = this.manifestEntries.get(subagentId);
+    if (entry?.name) {
+      const nicknameKey = entry.name.split(" ")[0].toLowerCase().replace(/[^a-z0-9-]/g, "-");
+      this._usedNicknames.delete(nicknameKey);
+    }
+    for (const [taskId, task] of this.pendingTasks) {
+      if (task.subagentId === subagentId) {
+        this.pendingTasks.delete(taskId);
+      }
+    }
+  }
   /** Release all resources: clear the manifest debounce timer and dispose the usage aggregator. */
   dispose() {
     if (this.manifestTimer) {
@@ -8012,6 +8407,6 @@ var FleetManager = class {
   }
 };
 
-export { ACP_AGENTS, AGENTS_BY_PHASE, AGENT_CATALOG, TOOLS as AGENT_TOOL_PRESETS, ALL_AGENT_DEFINITIONS, ALL_FLEET_AGENTS, AUDIT_LOG_AGENT, BUG_HUNTER_AGENT, BUILD_AGENTS, BudgetExceededError, BudgetThresholdSignal, CollabSession, DEFAULT_DIRECTOR_PREAMBLE, DEFAULT_DISPATCH_ROLE, DEFAULT_SUBAGENT_BASELINE, DELIVERY_AGENTS, DISCOVERY_AGENTS, DOMAIN_AGENTS, DefaultMultiAgentCoordinator, Director, DirectorAlertLevel, FLEET_ROSTER, FLEET_ROSTER_BUDGETS, FLEET_ROSTER_WITHACP, FleetBus, FleetCostCapError, FleetManager, FleetSpawnBudgetError, FleetUsageAggregator, HEAVY_BUDGET, InMemoryAgentBridge, InMemoryBridgeTransport, KNOWLEDGE_AGENTS, LIGHT_BUDGET, MEDIUM_BUDGET, META_AGENTS, NULL_FLEET_BUS, PLANNING_AGENTS, REFACTOR_PLANNER_AGENT, REVIEW_AGENTS, SECURITY_SCANNER_AGENT, SubagentBudget, VERIFY_AGENTS, applyRosterBudget, attachAutoExtend, composeDirectorPrompt, composeSubagentPrompt, createDelegateTool, createMessage, dispatchAgent, getAgentDefinition, makeAgentSubagentRunner, makeAskTool, makeAssignTool, makeAwaitTasksTool, makeCollabDebugTool, makeDirectorSessionFactory, makeFleetEmitTool, makeFleetHealthTool, makeFleetSessionTool, makeFleetStatusTool, makeFleetUsageTool, makeLLMClassifier, makeRollUpTool, makeSpawnTool, makeTerminateTool, makeWorkCompleteTool, rosterSummaryFromConfigs, scoreAgents };
+export { ACP_AGENTS, AGENTS_BY_PHASE, AGENT_CATALOG, TOOLS as AGENT_TOOL_PRESETS, ALL_AGENT_DEFINITIONS, ALL_FLEET_AGENTS, AUDIT_LOG_AGENT, BUG_HUNTER_AGENT, BUILD_AGENTS, BudgetExceededError, BudgetThresholdSignal, CollabSession, DEFAULT_DIRECTOR_PREAMBLE, DEFAULT_DISPATCH_ROLE, DEFAULT_SUBAGENT_BASELINE, DELIVERY_AGENTS, DISCOVERY_AGENTS, DOMAIN_AGENTS, DefaultMultiAgentCoordinator, Director, DirectorAlertLevel, FLEET_ROSTER, FLEET_ROSTER_BUDGETS, FLEET_ROSTER_WITHACP, FleetBus, FleetCostCapError, FleetManager, FleetSpawnBudgetError, FleetUsageAggregator, HEAVY_BUDGET, InMemoryAgentBridge, InMemoryBridgeTransport, KNOWLEDGE_AGENTS, LIGHT_BUDGET, LargeAnswerStore, MEDIUM_BUDGET, META_AGENTS, NULL_FLEET_BUS, PLANNING_AGENTS, REFACTOR_PLANNER_AGENT, REVIEW_AGENTS, SECURITY_SCANNER_AGENT, SubagentBudget, VERIFY_AGENTS, applyRosterBudget, attachAutoExtend, composeDirectorPrompt, composeSubagentPrompt, createDelegateTool, createMessage, dispatchAgent, getAgentDefinition, makeAgentSubagentRunner, makeAskResultTool, makeAskTool, makeAssignTool, makeAwaitTasksTool, makeCollabDebugTool, makeDirectorSessionFactory, makeFleetEmitTool, makeFleetHealthTool, makeFleetSessionTool, makeFleetStatusTool, makeFleetUsageTool, makeLLMClassifier, makeRollUpTool, makeSpawnTool, makeTerminateTool, makeWorkCompleteTool, rosterSummaryFromConfigs, scoreAgents };
 //# sourceMappingURL=index.js.map
 //# sourceMappingURL=index.js.map