@maycombhq/atticus 2.0.0-beta.2 → 2.0.0-beta.21

package/dist/{chunk-CSZ4C5N6.js → chunk-AOCWFA7T.js}

@@ -5,32 +5,40 @@ var __atticusUrl = (p) => new URL("./" + p, import.meta.url).href;
 var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   RunLevel,
+  TASK_RUNNER,
   completeTaskHandoff,
+  findTaskRunOfRun,
+  isTaskRun,
+  previewTaskRun,
   runTask
-} from "./chunk-CEXSU4S6.js";
+} from "./chunk-Y6K3CJBU.js";
+import {
+  handoffDone
+} from "./chunk-IFHDV6KQ.js";
 import {
   driftCheck,
   handDown,
   handUp
-} from "./chunk-OT2O76OS.js";
+} from "./chunk-YWAKDNMH.js";
 import {
   generateAssistantFiles,
   smokeCheck
-} from "./chunk-IPKHE5D6.js";
+} from "./chunk-3AQ2R5NL.js";
 import {
   createAdapters,
   fileStepStore
-} from "./chunk-7JUKDRSZ.js";
+} from "./chunk-4KNOJC6J.js";
 import {
   RUNTIME_MODES,
   runtimeForCli
-} from "./chunk-CNUKCS2T.js";
+} from "./chunk-QHJJMZLX.js";
 import {
   changesRelOf,
   loadOrgPack,
   orgPackDirOf,
+  runsDirOf,
   runsRelOf
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 
 // packages/cli/dist/run-commands.js
 import { readFile } from "node:fs/promises";
@@ -38,6 +46,8 @@ import { join } from "node:path";
 var KNOWN = /* @__PURE__ */ new Set([
   "SpecKitError",
   "TaskRunError",
+  "GitError",
+  "RuntimeError",
   "RuntimeConfigError",
   "AssistantFilesError",
   "ChangeError",
@@ -53,10 +63,11 @@ function fail(io, e) {
   throw e;
 }
 var plural = (n, w) => `${n} ${w}${n === 1 ? "" : "s"}`;
+var exitOf = (r) => r.state === "failed" || r.state === "needs-person" || r.level === "pr" && r.state === "built" ? 1 : 0;
 function describeRun(root, r, io) {
   const what = {
     planned: `Wrote the plan for task ${r.task}: ${changesRelOf(root)}/${r.change}/tasks/${r.task}/plan.md.`,
-    waiting: `Handed task ${r.task} to your assistant. The prompt is in ${runsRelOf(root)}/${r.change}/${r.waiting}/prompt.md; it ends with: atticus handoff done ${r.waiting}`,
+    waiting: r.level === "pr" ? `Handed task ${r.task} to your assistant. The prompt is in ${runsRelOf(root)}/${r.change}/${r.waiting}/prompt.md. When it is done, you run: atticus handoff done ${r.waiting} --approve (it pushes the branch and opens the PR; an assistant never approves a push)` : `Handed task ${r.task} to your assistant. The prompt is in ${runsRelOf(root)}/${r.change}/${r.waiting}/prompt.md; it ends with: atticus handoff done ${r.waiting}`,
     built: `Task ${r.task} is drafted on ${r.branch} and the build passes (${plural(r.attempts.length, "attempt")}); committed ${r.commit?.slice(0, 12) ?? ""}.`,
     unverified: `Task ${r.task} is drafted on ${r.branch} but wasn't built: ${r.repo} has no build command.`,
     "pr-open": `Opened PR ${r.pr?.url ?? ""} from ${r.branch} for review.`,
@@ -67,7 +78,7 @@ function describeRun(root, r, io) {
   io.out(reason ? `The run for task ${r.task} failed: ${reason}` : what[r.state]);
   for (const n of reason ? r.notes.slice(0, -1) : r.notes)
     io.out(`  ${n}`);
-  return r.state === "failed" || r.state === "needs-person" ? 1 : 0;
+  return exitOf(r);
 }
 async function taskRun(root, changeId, taskId, o, io, deps = {}) {
   if (!changeId || !taskId) {
@@ -88,6 +99,30 @@ async function taskRun(root, changeId, taskId, o, io, deps = {}) {
     io.err(`Unknown mode ${mode}. Use one of: hand-off, model, headless-claude-code, replay.`);
     return 2;
   }
+  if (o.dryRun || !o.approve) {
+    let p;
+    try {
+      p = await previewTaskRun(root, changeId, taskId, {
+        ...o.level ? { level: o.level } : {},
+        ...deps.gitHost ? { gitHost: deps.gitHost } : {}
+      });
+    } catch (e) {
+      return fail(io, e);
+    }
+    if (o.dryRun || p.level === "pr") {
+      const what = [
+        p.level === "plan" ? `Task ${p.task} of ${p.change} at the plan level: the plan is written to tasks/${p.task}/plan.md; no code changes.` : `Task ${p.task} of ${p.change} at the ${p.level} level: drafted on ${p.branch} in its own worktree of ${p.repo}, built and committed${p.level === "pr" ? p.gitHost ? `, then the branch is pushed and a PR opened into change/${p.change}.` : `; no Git host is connected (runtime.yaml adapters.gitHost), so you push it and open the PR by hand.` : "."}`,
+        // The yes is the person's: an assistant that ran this says so to them, never adds it.
+        o.dryRun || p.level !== "pr" || o.approve ? "Nothing was done: this was a dry run." : p.gitHost ? "Nothing was done: run it again with --approve to let it push and open the PR (the person's yes; an assistant never approves a push)." : "Nothing was done: run it again with --approve to draft and commit it; you push and open the PR (the person's yes; an assistant never approves a push)."
+      ];
+      if (o.json)
+        io.out(JSON.stringify({ dryRun: true, ...p, lines: what }, null, 2));
+      else
+        for (const l of what)
+          io.out(l);
+      return 0;
+    }
+  }
   const now = deps.now ?? (() => /* @__PURE__ */ new Date());
   try {
     const r = await runTask(root, changeId, taskId, {
@@ -111,7 +146,7 @@ async function taskRun(root, changeId, taskId, o, io, deps = {}) {
     });
     if (o.json) {
       io.out(JSON.stringify(r, null, 2));
-      return r.state === "failed" || r.state === "needs-person" ? 1 : 0;
+      return exitOf(r);
     }
     return describeRun(root, r, io);
   } catch (e) {
@@ -135,7 +170,7 @@ async function taskHandoff(root, runId, o, io, deps = {}) {
     if (o.json)
       io.out(JSON.stringify(r, null, 2));
     else if (r.state === "waiting")
-      io.out(`The build still fails; fix it in ${r.worktree} and run atticus handoff done ${runId} again.`);
+      io.out(r.notes.at(-1)?.startsWith("Build attempt") ? `The build still fails; fix it in ${r.worktree} and run atticus handoff done ${runId} again.` : r.notes.at(-1) ?? `Run atticus handoff done ${runId} again.`);
     else
       return describeRun(root, r, io);
     return r.state === "waiting" ? 1 : 0;
@@ -143,6 +178,52 @@ async function taskHandoff(root, runId, o, io, deps = {}) {
     return fail(io, e);
   }
 }
+async function runDone(root, runId, o, io, deps = {}) {
+  if (await isTaskRun(root, runId))
+    return taskHandoff(root, runId, {
+      ...o.by ? { by: o.by } : {},
+      ...o.approve ? { approve: true } : {},
+      ...o.json ? { json: true } : {}
+    }, io, deps);
+  const owner = await findTaskRunOfRun(root, runId);
+  if (owner) {
+    io.err(taskRunNotWaiting(runId, owner));
+    return 1;
+  }
+  if (await runAgentOf(root, runId) === TASK_RUNNER) {
+    io.err(`Run ${runId} was a task run's, and no task run waits on it now: run the task again (atticus task run <change> <task>).`);
+    return 1;
+  }
+  const agent = await runAgentOf(root, runId);
+  if (agent === "decide-propose" || agent === "decide-match") {
+    const { decideRunDone } = await import("./chunk-EFCQSMXB.js");
+    return decideRunDone(root, runId, io);
+  }
+  return handoffDone(root, runId, io, o.model ? { model: o.model } : {});
+}
+async function runAgentOf(root, runId) {
+  if (!/^run-[0-9a-z-]+$/.test(runId))
+    return void 0;
+  const { readdir } = await import("node:fs/promises");
+  const runs = runsDirOf(root);
+  for (const change of await readdir(runs).catch(() => [])) {
+    const text = await readFile(join(runs, change, runId, "run.json"), "utf8").catch(() => void 0);
+    if (!text)
+      continue;
+    try {
+      const a = JSON.parse(text).request?.agent;
+      return typeof a === "string" ? a : a?.id;
+    } catch {
+      return void 0;
+    }
+  }
+  return void 0;
+}
+function taskRunNotWaiting(runId, r) {
+  if (r.notes.at(-1)?.startsWith(`Run ${runId} was cancelled`))
+    return `Run ${runId} was cancelled; run the task again (atticus task run ${r.change} ${r.task}).`;
+  return r.state === "needs-person" ? `Run ${runId} is part of task ${r.task}'s run, which needs a person now: carry on in ${r.worktree}, or run atticus task run ${r.change} ${r.task} again.` : `Run ${runId} is part of task ${r.task}'s run, which is ${r.state} and waits on no run: run the task again (atticus task run ${r.change} ${r.task}).`;
+}
 async function specHandDown(root, changeId, taskId, o, io, deps = {}) {
   if (!changeId || !taskId) {
     io.err("Usage: atticus spec hand-down <change> <task> [--force]");
@@ -300,6 +381,8 @@ export {
   describeRun,
   taskRun,
   taskHandoff,
+  runDone,
+  taskRunNotWaiting,
   specHandDown,
   specHandUp,
   driftCommand,

package/dist/{chunk-EPTBZ3GP.js → chunk-API5TOUB.js}

@@ -5,7 +5,7 @@ var __atticusUrl = (p) => new URL("./" + p, import.meta.url).href;
 var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   parseFactLine
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 
 // packages/cli/dist/memory-room.js
 import { createReadStream } from "node:fs";

package/dist/{chunk-652QWTLZ.js → chunk-AZDITBCT.js}

@@ -15,13 +15,13 @@ import {
   releaseNotes,
   securityReview,
   writeEvidencePack
-} from "./chunk-BOHJZTEQ.js";
+} from "./chunk-36RY7ZXM.js";
 import {
   judgingRuntime
-} from "./chunk-BWZ77RQT.js";
+} from "./chunk-F6H2RJ3N.js";
 import {
   changeStore
-} from "./chunk-XY3P7JW7.js";
+} from "./chunk-HSZMPH5T.js";
 
 // packages/cli/dist/release-commands.js
 import { resolve } from "node:path";

package/dist/{chunk-DPUYDDQG.js → chunk-BA2HUMDN.js}

@@ -6,14 +6,14 @@ var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   resumedAttempt,
   stepClassOf
-} from "./chunk-CNUKCS2T.js";
+} from "./chunk-QHJJMZLX.js";
 import {
   RUN_SCHEMA_VERSION,
   STEP_CLASSES,
   changesDirOf,
   runsDirOf,
   runsRelOf
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 
 // packages/cli/dist/runs-view.js
 import { createHash } from "node:crypto";
@@ -86,6 +86,8 @@ async function item(root, s) {
     startedAt: r.startedAt,
     ...r.finishedAt ? { finishedAt: r.finishedAt } : {},
     ...r.request.outputPath ? { output: r.request.outputPath } : {},
+    // Third UX walk: a failed run's problems, which its page shows (they were left out).
+    errors: r.errors ?? [],
     ...r.usage ? { usage: r.usage } : {},
     contextManifestSha256: sha(manifest),
     promptPath: `${runsRelOf(root)}/${r.request.change}/${r.runId}/prompt.md`,
@@ -97,9 +99,28 @@ async function item(root, s) {
     // ADR 0121 addendum: the kind of step and the model that ran it. A run kept before either
     // was recorded is given its kind from its agent; its model stays unsaid.
     ...classOf(r) ? { stepClass: classOf(r) } : {},
-    ...r.model ? { model: r.model } : {}
+    ...r.model ? { model: r.model } : {},
+    // ADR 0185: the batched hand-off a lens was asked in, or a batch's lenses and where each is.
+    ...r.batch ? { batch: r.batch } : {},
+    ...r.parts?.length ? { parts: await partStates(root, r.request.change, r.parts) } : {}
   };
 }
+async function partStates(root, change, parts) {
+  return Promise.all(parts.map(async (p) => {
+    const text = RUN_ID.test(p.runId) ? await readFile(join(runsDirOf(root), change, p.runId, "run.json"), "utf8").catch(() => void 0) : void 0;
+    let state;
+    try {
+      state = text ? JSON.parse(text).state : void 0;
+    } catch {
+      state = void 0;
+    }
+    return {
+      name: p.name,
+      runId: p.runId,
+      ...state ? { state } : {}
+    };
+  }));
+}
 function classOf(r) {
   if (r.stepClass && STEP_CLASSES.includes(r.stepClass))
     return r.stepClass;
@@ -159,6 +180,7 @@ async function runTranscript(root, runId) {
 }
 
 export {
+  partStates,
   listRuns,
   runTranscript
 };

package/dist/chunk-BA3U7ALS.js

@@ -0,0 +1,99 @@
+import { createRequire as __atticusCreateRequire } from "node:module";
+import { fileURLToPath as __atticusFileURLToPath } from "node:url";
+var require = __atticusCreateRequire(import.meta.url);
+var __atticusUrl = (p) => new URL("./" + p, import.meta.url).href;
+var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
+import {
+  AUDIENCES,
+  MEASURABLE,
+  OUTCOMES_FILE,
+  REQUESTS_FILE,
+  ReleaseError,
+  breakingItem,
+  buildModel,
+  changeFiles,
+  classOf,
+  deploymentPlan,
+  estimateChange,
+  evidencePack,
+  findMeasure,
+  hasWord,
+  judgedFor,
+  loadRelease,
+  measureId,
+  measuresOf,
+  opOf,
+  outcomesView,
+  pathKey,
+  readOutcomes,
+  readServiceRequests,
+  recordOutcome,
+  releaseForecast,
+  releaseNotes,
+  secretItem,
+  sectionOf,
+  sectionsOf,
+  securityReview,
+  sentences,
+  sortStrings,
+  undoesItem,
+  writeEvidencePack
+} from "./chunk-36RY7ZXM.js";
+import "./chunk-2V2ZPQQU.js";
+import "./chunk-EZIN5E5B.js";
+import "./chunk-3LZ6P5EP.js";
+import "./chunk-BEPCKDCP.js";
+import "./chunk-HWD4JPU7.js";
+import "./chunk-5WERBKLE.js";
+import "./chunk-K332XW7K.js";
+import "./chunk-HSZMPH5T.js";
+import "./chunk-D2WSW4GU.js";
+import "./chunk-4KNOJC6J.js";
+import "./chunk-4SLBCEPD.js";
+import "./chunk-64AADMTW.js";
+import "./chunk-JYMIB5XY.js";
+import "./chunk-CPK7QSBG.js";
+import "./chunk-YO2PKTXT.js";
+import "./chunk-TWQRPM76.js";
+import "./chunk-XOL3GFW2.js";
+import "./chunk-QHJJMZLX.js";
+import "./chunk-7AB6OWTV.js";
+import "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
+import "./chunk-TV45EUGM.js";
+export {
+  AUDIENCES,
+  MEASURABLE,
+  OUTCOMES_FILE,
+  REQUESTS_FILE,
+  ReleaseError,
+  breakingItem,
+  buildModel,
+  changeFiles,
+  classOf,
+  deploymentPlan,
+  estimateChange,
+  evidencePack,
+  findMeasure,
+  hasWord,
+  judgedFor,
+  loadRelease,
+  measureId,
+  measuresOf,
+  opOf,
+  outcomesView,
+  pathKey,
+  readOutcomes,
+  readServiceRequests,
+  recordOutcome,
+  releaseForecast,
+  releaseNotes,
+  secretItem,
+  sectionOf,
+  sectionsOf,
+  securityReview,
+  sentences,
+  sortStrings,
+  undoesItem,
+  writeEvidencePack
+};

package/dist/{chunk-HVUPKCGJ.js → chunk-BC6KKZ67.js}

@@ -6,13 +6,13 @@ var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   openIndex,
   searchIndex
-} from "./chunk-T6AIP7H5.js";
-import "./chunk-CNUKCS2T.js";
+} from "./chunk-5WERBKLE.js";
+import "./chunk-QHJJMZLX.js";
 import {
   changesDirOf,
   require_dist
-} from "./chunk-S2A7YC3Q.js";
-import "./chunk-D3PO4DOZ.js";
+} from "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
 import {
   __toESM
 } from "./chunk-TV45EUGM.js";
@@ -118,7 +118,7 @@ async function pastChangeMarkdown(root, folder) {
 `;
 }
 async function attachPastChange(root, change, folder, by, now = /* @__PURE__ */ new Date()) {
-  const { attachFiles } = await import("./chunk-NE2W3OO7.js");
+  const { attachFiles } = await import("./chunk-XQ3YZ4QW.js");
   const name = `like-${folder.replace(DATED, "$2")}.md`;
   await attachFiles(root, change, [{ name, bytes: new TextEncoder().encode(await pastChangeMarkdown(root, folder)) }], by, now);
   return name;