harnessed 4.1.1 → 4.1.3

package/dist/cli.mjs

@@ -186,6 +186,20 @@ var init_schemaVersion = __esm({
     ]);
   }
 });
+async function writeFileAtomic(path, data) {
+  const tmp = `${path}.tmp`;
+  await writeFile(tmp, data, "utf8");
+  await rename(tmp, path);
+}
+function writeFileSyncAtomic(path, data) {
+  const tmp = `${path}.tmp`;
+  writeFileSync(tmp, data, "utf8");
+  renameSync(tmp, path);
+}
+var init_atomicWrite = __esm({
+  "src/checkpoint/atomicWrite.ts"() {
+  }
+});
 var CheckpointStatus, CheckpointV1;
 var init_checkpoint_v1 = __esm({
   "src/checkpoint/schema/checkpoint.v1.ts"() {
@@ -305,7 +319,7 @@ async function writeCurrentWorkflow(s) {
   const path = statePath();
   await mkdir(dirname(path), { recursive: true });
   await withLock(async () => {
-    await writeFile(path, JSON.stringify(s, null, 2), "utf8");
+    await writeFileAtomic(path, JSON.stringify(s, null, 2));
   });
 }
 async function activate(phase, checkpointPath = null) {
@@ -332,6 +346,7 @@ var init_state = __esm({
   "src/checkpoint/state.ts"() {
     init_harnessedRoot();
     init_schemaVersion();
+    init_atomicWrite();
     init_schema();
     WorkflowStateError = class extends Error {
       constructor(message) {
@@ -375,13 +390,14 @@ function writeCheckpoint(c, customPath) {
   const enforced = enforceBudget(c);
   const path = join(harnessedSubdir("checkpoints"), `${enforced.phase}.json`);
   mkdirSync(dirname(path), { recursive: true });
-  writeFileSync(path, JSON.stringify(enforced, null, 2), "utf8");
+  writeFileSyncAtomic(path, JSON.stringify(enforced, null, 2));
   return path;
 }
 var BUDGET_TOKEN, CheckpointTooLargeError, CheckpointWriteError;
 var init_template = __esm({
   "src/checkpoint/template.ts"() {
     init_harnessedRoot();
+    init_atomicWrite();
     init_schema();
     BUDGET_TOKEN = 1e3;
     CheckpointTooLargeError = class extends Error {
@@ -1271,7 +1287,7 @@ var init_auto_install = __esm({
 
 // package.json
 var package_default = {
-  version: "4.1.1"};
+  version: "4.1.3"};
 
 // src/manifest/errors.ts
 function instancePathToKeyPath(instancePath) {
@@ -2572,23 +2588,18 @@ async function resolveJudgmentGate(gateRef, context, packageRoot) {
   }
   return evalGate(expr, context);
 }
-function getPackageRoot() {
-  const thisFile = fileURLToPath(import.meta.url);
-  const thisDir = dirname(thisFile);
-  if (thisDir.endsWith("dist") || thisDir.replace(/\\/g, "/").endsWith("/dist")) {
-    return resolve(thisDir, "..");
-  }
-  return resolve(thisDir, "..", "..", "..");
-}
 
-// src/cli/gates.ts
-var VALID_MASTERS = /* @__PURE__ */ new Set(["auto", "discuss", "plan", "task", "verify"]);
-var STAGE_MASTERS = /* @__PURE__ */ new Set(["discuss", "plan", "task", "verify"]);
-var PARALLELISM_GATE = "judgments.parallelism-gate.agent-teams-upgrade.fires";
-function defaultContext(task, stage) {
+// src/cli/lib/gateContext.ts
+function buildDefaultGateContext(task, stage) {
   return {
     task,
     user_understanding_unclear: false,
+    // v4.1.2 — parallelism-gate.agent-teams-upgrade team-routing facts (default off).
+    teammate_send_message_needed: false,
+    subagent_context_overflow: false,
+    shared_task_list: false,
+    opposing_hypothesis_debate: false,
+    fullstack_three_way: false,
     phase: {
       stage,
       is_critical_module: true,
@@ -2627,6 +2638,30 @@ function defaultContext(task, stage) {
     }
   };
 }
+function mergeGateContext(base, extra) {
+  const merged = { ...base };
+  for (const [k, v] of Object.entries(extra)) {
+    if ((k === "phase" || k === "subtask") && v && typeof v === "object" && !Array.isArray(v)) {
+      merged[k] = { ...base[k], ...v };
+    } else {
+      merged[k] = v;
+    }
+  }
+  return merged;
+}
+function getPackageRoot() {
+  const thisFile = fileURLToPath(import.meta.url);
+  const thisDir = dirname(thisFile);
+  if (thisDir.endsWith("dist") || thisDir.replace(/\\/g, "/").endsWith("/dist")) {
+    return resolve(thisDir, "..");
+  }
+  return resolve(thisDir, "..", "..", "..");
+}
+
+// src/cli/gates.ts
+var VALID_MASTERS = /* @__PURE__ */ new Set(["auto", "discuss", "plan", "task", "verify"]);
+var STAGE_MASTERS = /* @__PURE__ */ new Set(["discuss", "plan", "task", "verify"]);
+var PARALLELISM_GATE = "judgments.parallelism-gate.agent-teams-upgrade.fires";
 function resolveMasterYamlPath(master, packageRoot) {
   return master === "auto" ? resolve(packageRoot, "workflows", "auto", "workflow.yaml") : resolve(packageRoot, "workflows", master, "auto", "workflow.yaml");
 }
@@ -2670,7 +2705,7 @@ function registerGates(program2) {
     const clauses = delegates;
     const task = typeof raw.task === "string" ? raw.task : "";
     const stage = master;
-    const ctx = defaultContext(task, stage);
+    let ctx = buildDefaultGateContext(task, stage);
     if (typeof raw.context === "string" && raw.context.length > 0) {
       let extra;
       try {
@@ -2680,15 +2715,14 @@ function registerGates(program2) {
         process.exit(1);
         return;
       }
-      if (extra && typeof extra === "object") {
-        Object.assign(ctx, extra);
+      if (extra && typeof extra === "object" && !Array.isArray(extra)) {
+        ctx = mergeGateContext(ctx, extra);
       }
     }
     const skipSubs = new Set(
       typeof raw.skipSub === "string" && raw.skipSub.length > 0 ? raw.skipSub.split(",").map((s) => s.trim()).filter((s) => s.length > 0) : []
     );
     ctx.skip_subs = [...skipSubs];
-    ctx.task = task;
     const fire = [];
     const skip = [];
     for (const clause of clauses) {
@@ -2700,13 +2734,13 @@ function registerGates(program2) {
         continue;
       }
       if (!clause.gate) {
-        fire.push(fireEntry(clause));
+        fire.push(fireEntry(clause, master));
         continue;
       }
       try {
         const passes = await resolveJudgmentGate(clause.gate, ctx, packageRoot);
         if (passes) {
-          fire.push(fireEntry(clause));
+          fire.push(fireEntry(clause, master));
         } else {
           skip.push({ sub: clause.sub, reason: `gate ${clause.gate} = false` });
         }
@@ -2714,7 +2748,7 @@ function registerGates(program2) {
         console.warn(
           `\u26A0\uFE0F master ${master} sub ${clause.sub} gate ${clause.gate} eval failed (${e.message}); firing sub as if gate=true (ADR 0029 fail-soft).`
         );
-        fire.push(fireEntry(clause));
+        fire.push(fireEntry(clause, master));
       }
     }
     let parallelism = { escalate_to_teams: false, reason: null };
@@ -2729,14 +2763,14 @@ function registerGates(program2) {
     process.exit(0);
   });
 }
-function fireEntry(clause) {
-  const entry = { sub: clause.sub };
+function fireEntry(clause, master) {
+  const isMaster = STAGE_MASTERS.has(clause.sub);
+  const sub = master !== "auto" && !isMaster ? `${master}-${clause.sub}` : clause.sub;
+  const entry = { sub };
   if (clause.order !== void 0) entry.order = clause.order;
   if (clause.mode !== void 0) entry.mode = clause.mode;
   if (clause.gate !== void 0) entry.gate = clause.gate;
-  if (STAGE_MASTERS.has(clause.sub)) {
-    entry.is_master = true;
-  }
+  if (isMaster) entry.is_master = true;
   return entry;
 }
 var DURATION_RE = /^(\d+)([dhmw])$/;
@@ -2790,7 +2824,7 @@ ${t("gc.invalid_duration.fix")}`
       return;
     }
     const cutoff = Date.now() - olderMs;
-    const kept = new Set(dirs.slice(-keepLast));
+    const kept = new Set(keepLast > 0 ? dirs.slice(-keepLast) : []);
     const candidates = [];
     for (const ts of dirs) {
       if (kept.has(ts)) continue;
@@ -4537,7 +4571,7 @@ function nameToYamlHintPath(name) {
   }
   return `${name}/workflow.yaml`;
 }
-var HARNESSED_MARKER_RX = /<!--\s*harnessed-generated:v3\.4\.\d+\s*-->/;
+var HARNESSED_MARKER_RX = /<!--\s*harnessed-generated:v\d+\.\d+\.\d+\s*-->/;
 var V3_4_3_SIGNATURE_SUB_RX = /\*\*Preferred path\*\*[\s\S]*use the SlashCommand tool[\s\S]*\*\*Fallback path\*\*[\s\S]*use the Task tool to spawn/;
 var V3_4_3_SIGNATURE_MASTER_RX = /\*\*Preferred path\*\*[\s\S]*dispatch to the per-sub-workflow/;
 function shouldOverwriteFile(content) {
@@ -5801,63 +5835,7 @@ function registerRun(program2) {
     }
     const stage = name.includes("-") ? name.split("-")[0] ?? "" : name;
     const gateContext = {
-      task,
-      // v3.9.24 — top-level fact set by runAutoPreFlight() when /auto super-master
-      // Phase 0.5 understanding-check prompt fires. CLI path skips preflight
-      // (no readline hooks), so default to false → research sub skips by default.
-      user_understanding_unclear: false,
-      phase: {
-        stage,
-        // verify-stage gates
-        is_critical_module: true,
-        // verify-paranoid fires
-        is_final_step: true,
-        // verify-simplify fires
-        is_major_release: false,
-        // verify-multispec only for major
-        has_auth_or_secrets: false,
-        has_design_changes: false,
-        has_ui_changes: false,
-        requires_creative_polish: false,
-        // plan-stage gates
-        is_complex_architecture: true,
-        // plan-architecture fires
-        // discuss-stage gates
-        has_cross_phase_data_flow: true,
-        // discuss-phase fires
-        open_decisions: 2,
-        // ≥2 fires phase-gate
-        scope_days: 2,
-        // >1 day fires phase-gate
-        scope_locked_in_history: false,
-        single_task: false,
-        type: "general"
-      },
-      subtask: {
-        // subtask brainstorming gate
-        approaches: 2,
-        // ≥2 fires
-        core_algorithm: true,
-        has_api_contract: true,
-        error_cost: "high",
-        lines: 50,
-        // ≥20 → no skip
-        type: "general",
-        // not crud/standard_lib_call → no skip
-        // tdd gate
-        is_core_business_logic: true,
-        is_algorithm: true,
-        is_data_processing: true,
-        regression_risk: "high",
-        reliability_required: true,
-        // misc
-        communication_needed: false,
-        needs_lib_docs: false,
-        needs_web_search: false,
-        parallel_count: 1,
-        search_type: "general",
-        test_type: "general"
-      },
+      ...buildDefaultGateContext(task, stage),
       ...raw.model ? { modelOverride: raw.model } : {},
       ...raw.maxIterations ? { maxIterations: raw.maxIterations } : {},
       ...raw.staged ? { staged: true } : {},
@@ -5989,14 +5967,17 @@ var LANG_NAMES = {
   "zh-Hant": "\u7E41\u9AD4\u4E2D\u6587 (Traditional Chinese)",
   "zh-TW": "\u7E41\u9AD4\u4E2D\u6587 (Traditional Chinese)"
 };
+async function loadSubArrayField(sub, packageRoot, field) {
+  const subYaml = await resolveWorkflowYaml(sub, resolve(packageRoot, "workflows"));
+  if (!subYaml) return [];
+  const wf = parse(await readFile(subYaml, "utf8"));
+  const v = wf?.[field];
+  return Array.isArray(v) ? v : [];
+}
 async function buildToolsSection(sub, packageRoot) {
   try {
     const workflowsDir = resolve(packageRoot, "workflows");
-    const subYaml = await resolveWorkflowYaml(sub, workflowsDir);
-    if (!subYaml) return "";
-    const wfRaw = await readFile(subYaml, "utf8");
-    const wf = parse(wfRaw);
-    const tools = Array.isArray(wf?.tools_available) ? wf.tools_available : [];
+    const tools = await loadSubArrayField(sub, packageRoot, "tools_available");
     if (tools.length === 0) return "";
     const capRaw = await readFile(resolve(workflowsDir, "capabilities.yaml"), "utf8");
     const capDoc = parse(capRaw);
@@ -6020,11 +6001,7 @@ ${lines.join("\n")}
 async function buildDisciplinesSection(sub, packageRoot) {
   try {
     const workflowsDir = resolve(packageRoot, "workflows");
-    const subYaml = await resolveWorkflowYaml(sub, workflowsDir);
-    if (!subYaml) return "";
-    const wfRaw = await readFile(subYaml, "utf8");
-    const wf = parse(wfRaw);
-    const applied = Array.isArray(wf?.disciplines_applied) ? wf.disciplines_applied : [];
+    const applied = await loadSubArrayField(sub, packageRoot, "disciplines_applied");
     const names = applied.filter((d) => d !== "language");
     if (names.length === 0) return "";
     const blocks = [];
@@ -6097,7 +6074,7 @@ function registerPrompt(program2) {
     const packageRoot = getPackageRoot();
     const workflowsDir = resolve(packageRoot, "workflows");
     const rolePrompts = await loadRolePrompts(workflowsDir);
-    const def = buildAgentDef(sub, rolePrompts, void 0, void 0, void 0);
+    const def = buildAgentDef(sub, rolePrompts);
     const body = def.prompt;
     const taskSection = typeof raw.task === "string" && raw.task.length > 0 ? `## Task
 ${raw.task}
@@ -6121,7 +6098,6 @@ ${toolsSection}${disciplinesSection}${PROTOCOLS}${buildLanguageSection()}`;
         })
       );
       process.exit(0);
-      return;
     }
     console.log(fullPrompt);
     process.exit(0);
@@ -6216,21 +6192,21 @@ ${t("rollback.metadata_unreadable.fix")}`
       process.exit(1);
       return;
     }
-    for (const entry of [...meta.files].reverse()) {
+    const ordered = [...meta.files].reverse();
+    const planned = [];
+    for (const entry of ordered) {
       if (entry.backup === "") {
-        try {
-          await unlink(entry.target);
-        } catch (err2) {
-          const code = err2.code;
-          if (code !== "ENOENT") {
-            console.error(`error: cannot unlink ${entry.target}: ${err2.message}`);
-            process.exit(1);
-            return;
-          }
-        }
+        planned.push({ target: entry.target, action: "unlink" });
         continue;
       }
-      const buf = await readFile(entry.backup);
+      let buf;
+      try {
+        buf = await readFile(entry.backup);
+      } catch (err2) {
+        console.error(`error: cannot read backup ${entry.backup}: ${err2.message}`);
+        process.exit(1);
+        return;
+      }
       const sha1 = createHash("sha1").update(buf).digest("hex");
       if (sha1 !== entry.sha1) {
         console.error(
@@ -6243,7 +6219,23 @@ ${t("rollback.metadata_unreadable.fix")}`
         process.exit(1);
         return;
       }
-      await writeFile(entry.target, normalizeEol(buf, entry.eol));
+      planned.push({ target: entry.target, action: "write", data: normalizeEol(buf, entry.eol) });
+    }
+    for (const op of planned) {
+      if (op.action === "unlink") {
+        try {
+          await unlink(op.target);
+        } catch (err2) {
+          const code = err2.code;
+          if (code !== "ENOENT") {
+            console.error(`error: cannot unlink ${op.target}: ${err2.message}`);
+            process.exit(1);
+            return;
+          }
+        }
+      } else {
+        await writeFile(op.target, op.data);
+      }
     }
     console.log(t("rollback.restored", { count: meta.files.length, timestamp }));
   });