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

package/dist/chunk-2D7PWSOK.js

@@ -0,0 +1,172 @@
+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 {
+  targetFlowsOf
+} from "./chunk-GNWAOVFQ.js";
+import "./chunk-5QUOPNLN.js";
+import {
+  MAX_CHAIN_DEPTH,
+  changeImpact,
+  loadEstate
+} from "./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 {
+  loadRepos
+} from "./chunk-XOL3GFW2.js";
+import "./chunk-QHJJMZLX.js";
+import "./chunk-7AB6OWTV.js";
+import {
+  readTeams
+} from "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
+import "./chunk-TV45EUGM.js";
+
+// packages/cli/dist/change-impact.js
+var CHANGE_IMPACT_WORDS = {
+  usage: "Name the change: atticus change impact <change> [--json]",
+  noChange: (id) => `There is no change ${id}: atticus change list shows them.`,
+  noTargets: (id) => `${id} has no target flows yet, so there is nothing to weigh: run atticus change architecture ${id} first.`,
+  nothing: (id) => `${id}'s target flows break nothing the map shows: no entry or table they change loses a field, a caller or its type.`,
+  counts: (id, i) => `${id}: ${n(i.breaks, "caller")} ${i.breaks === 1 ? "breaks" : "break"}, ${n(i.mayBreak, "caller")} may break (link guessed), ${n(i.routes, "gateway route")} to update.`,
+  upstream: (u) => `Up the chain (who calls the callers): ${u.breaks} more ${u.breaks === 1 ? "breaks" : "break"}, ${u.mayBreak} may break (resting on a guessed link).`,
+  hopBreaks: (depth, via, who, owner, where) => `${"  ".repeat(depth)}Breaks via ${via}: ${who}, owned by ${owner}${where ? `; ${where}` : ""}.`,
+  hopMayBreak: (depth, via, who, owner, rests, where) => `${"  ".repeat(depth)}May break via ${via} (${rests}): ${who}, owned by ${owner}${where ? `; ${where}` : ""}. Not counted as a break.`,
+  restsOwn: (how) => `its own link guessed ${how}`,
+  restsOn: (who, how) => `rests on ${who}'s link, guessed ${how}`,
+  capped: (depth) => `  The walk up the chain stops at ${depth} hops; more callers are above the last ones listed.`,
+  item: (name, where, owner, why) => `${name} (${where}), owned by ${owner}: ${why.join("; ")}.`,
+  gone: (fields) => `  Stops crossing: ${fields.join(", ")}.`,
+  required: (fields) => `  Callers must now send: ${fields.join(", ")}.`,
+  breaks: (who, owner, where) => `  Breaks: ${who}, owned by ${owner}${where ? `; ${where}` : ""}.`,
+  mayBreak: (who, owner, how, where) => `  May break (link guessed ${how}): ${who}, owned by ${owner}${where ? `; ${where}` : ""}. Not counted as a break until someone confirms the link.`,
+  confirm: (rule, id) => `  The critic's ${rule} asks the caller's owner to confirm each guessed link: atticus critic ${id}.`,
+  nobody: "  The map shows nobody calling or using it.",
+  route: (id, what, owner, callers) => `  Gateway route to update: ${id}${what}, owned by ${owner}; ${callers.join(", ")} ${callers.length === 1 ? "calls" : "call"} through it.`,
+  teams: "Teams to involve:",
+  team: (owner, repos) => `  ${owner}: ${repos.join(", ")}`,
+  problem: (p) => `Note: ${p}`
+};
+var GUESS_WORDS = {
+  "joined-by-path": "by path",
+  "joined-by-label": "by a host's first label",
+  "joined-by-platform-setting": "by a platform-wide setting",
+  "joined-by-folder-name": "by a setting's folder name",
+  "joined-despite-conflict": "despite an open host conflict",
+  "joined-by-service-name": "by a Service's short name"
+};
+function n(count, what) {
+  return `${count} ${what}${count === 1 ? "" : "s"}`;
+}
+function ownerText(o) {
+  if ("none" in o)
+    return `no team (${o.none})`;
+  return o.name ? `${o.team} (${o.name})` : o.team;
+}
+async function changeImpactOf(root, change) {
+  const t = await targetFlowsOf(root, change);
+  if (!t)
+    return void 0;
+  const problems = [...t.problems];
+  const owners = /* @__PURE__ */ new Map();
+  try {
+    for (const r of await loadRepos(root))
+      owners.set(r.id, r.owner);
+  } catch (e) {
+    problems.push(`the repo manifests: ${e.message.split("\n")[0]}`);
+  }
+  const read = await readTeams(root);
+  const teams = read.teams;
+  if (read.problem)
+    problems.push(read.problem);
+  const impact = changeImpact(change, t.deltas, await loadEstate(root, t.records), {
+    owners,
+    teams
+  });
+  return { ...impact, flows: t.deltas.length, problems };
+}
+function impactLines(i) {
+  const W = CHANGE_IMPACT_WORDS;
+  if (!i.flows)
+    return [W.noTargets(i.change), ...i.problems.map(W.problem)];
+  if (!i.items.length)
+    return [W.nothing(i.change), ...i.problems.map(W.problem)];
+  const out = [W.counts(i.change, i)];
+  if (i.upstream && (i.upstream.breaks || i.upstream.mayBreak))
+    out.push(W.upstream(i.upstream));
+  for (const it of i.items) {
+    out.push(W.item(it.name, it.kind === "table" ? `a table ${it.flow} writes` : it.flow, ownerText(it.owner), it.why));
+    if (it.fields.gone.length)
+      out.push(W.gone(it.fields.gone));
+    if (it.fields.required.length)
+      out.push(W.required(it.fields.required));
+    if (!it.callers.length)
+      out.push(W.nobody);
+    for (const c of it.callers) {
+      const where = [
+        ...c.at.map((a) => `at ${a}`),
+        ...c.flows.map((f) => `its flow ${f}`),
+        ...c.routes.map((r) => `through ${r}`)
+      ].join(", ");
+      out.push(c.verdict === "breaks" ? W.breaks(c.component, ownerText(c.owner), where) : W.mayBreak(c.component, ownerText(c.owner), GUESS_WORDS[c.guessed ?? ""] ?? "", where));
+    }
+    for (const h of it.chain ?? []) {
+      const via = `${h.via.join(", ")}${h.entries.length ? ` (${h.entries.join(", ")})` : ""}`;
+      const where = [
+        ...h.at.map((a) => `at ${a}`),
+        ...h.flows.map((f) => `its flow ${f}`),
+        ...h.routes.map((r) => `through ${r}`)
+      ].join(", ");
+      const how = GUESS_WORDS[h.guessed ?? ""] ?? "";
+      out.push(h.verdict === "breaks" ? W.hopBreaks(h.depth, via, h.component, ownerText(h.owner), where) : W.hopMayBreak(h.depth, via, h.component, ownerText(h.owner), h.restsOn ? W.restsOn(h.restsOn, how) : W.restsOwn(how), where));
+    }
+    if (it.capped)
+      out.push(W.capped(MAX_CHAIN_DEPTH));
+    if ([...it.callers, ...it.chain ?? []].some((c) => c.verdict === "may-break"))
+      out.push(W.confirm(i.rule, i.change));
+    for (const r of it.routes) {
+      const what = [r.proxy, r.basepath, r.repo ? `in ${r.repo}` : void 0].filter(Boolean).join(", ");
+      out.push(W.route(r.id, what ? ` (${what})` : "", ownerText(r.owner), r.callers));
+    }
+  }
+  out.push(W.teams, ...i.owners.map((o) => W.team(ownerText(o.owner), o.repos)));
+  out.push(...i.problems.map(W.problem));
+  return out;
+}
+async function changeImpactCli(root, change, o, io) {
+  if (!change) {
+    io.err(CHANGE_IMPACT_WORDS.usage);
+    return 2;
+  }
+  const i = await changeImpactOf(root, change);
+  if (!i) {
+    io.err(CHANGE_IMPACT_WORDS.noChange(change));
+    return 1;
+  }
+  if (o.json)
+    io.out(JSON.stringify(i, null, 2));
+  else
+    for (const l of impactLines(i))
+      io.out(l);
+  return 0;
+}
+export {
+  CHANGE_IMPACT_WORDS,
+  changeImpactCli,
+  changeImpactOf,
+  impactLines,
+  ownerText
+};

package/dist/{chunk-JCZIKA4C.js → chunk-2HH3UCZM.js}

@@ -9,8 +9,8 @@ import {
   probeFileOf,
   probeTier,
   writeProbe
-} from "./chunk-IPKHE5D6.js";
-import "./chunk-EFHQ3TWP.js";
+} from "./chunk-3AQ2R5NL.js";
+import "./chunk-XOL3GFW2.js";
 import {
   AssistantId,
   external_exports,
@@ -18,8 +18,8 @@ import {
   orgPackDirOf,
   runsDirOf,
   runsRelOf
-} from "./chunk-S2A7YC3Q.js";
-import "./chunk-D3PO4DOZ.js";
+} from "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
 import "./chunk-TV45EUGM.js";
 
 // packages/cli/dist/probe-commands.js

package/dist/{chunk-GDLULHTC.js → chunk-2HTEMYTB.js}

@@ -10,17 +10,17 @@ import {
   parseHeader,
   renderAssistantFiles,
   writeAssistantFiles
-} from "./chunk-IPKHE5D6.js";
+} from "./chunk-3AQ2R5NL.js";
 import {
   BUILT_IN_CHECKS,
   STARTER_REVIEW_PATH,
   starterReviewText
-} from "./chunk-3HI6Z55Y.js";
+} from "./chunk-K332XW7K.js";
 import {
   changeFootprint,
   changeStore,
   loadChangeContext
-} from "./chunk-XY3P7JW7.js";
+} from "./chunk-HSZMPH5T.js";
 import {
   ORG_FLOW,
   createOpenSpec,
@@ -28,23 +28,23 @@ import {
   storeIdOf,
   writeOrgFlowSchema,
   writeStoreMarker
-} from "./chunk-4AK7IBVP.js";
+} from "./chunk-D2WSW4GU.js";
 import {
   readCatalog
-} from "./chunk-27NKUS3X.js";
+} from "./chunk-4SLBCEPD.js";
 import {
   componentExtractor
-} from "./chunk-MC3H7736.js";
+} from "./chunk-TWQRPM76.js";
 import {
   createGit,
   ensureGitignore
-} from "./chunk-EFHQ3TWP.js";
+} from "./chunk-XOL3GFW2.js";
 import {
   DEFAULT_IGNORES,
   fsRepo,
   memoryRepo,
   runExtractor
-} from "./chunk-GQ4N4WKF.js";
+} from "./chunk-7AB6OWTV.js";
 import {
   ChangeLifecycle,
   ChecklistPack,
@@ -61,10 +61,10 @@ import {
   Sources,
   loadOrgPack,
   require_dist
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 import {
   LOCAL_ONLY
-} from "./chunk-D3PO4DOZ.js";
+} from "./chunk-ODMBPDDV.js";
 import {
   __toESM
 } from "./chunk-TV45EUGM.js";
@@ -235,19 +235,6 @@ function pathsFor(root, adopted, layout) {
     inputsDir: adopted ? join2(root, ATTICUS_DIR, "inputs") : join2(root, "inputs")
   };
 }
-function inPlacePath(url) {
-  if (url === ".")
-    return ".";
-  if (url.startsWith("./")) {
-    const rest = url.slice(2).replace(/\/+$/, "");
-    if (rest === "")
-      return ".";
-    if (rest.split("/").includes(".."))
-      return void 0;
-    return rest;
-  }
-  return void 0;
-}
 
 // packages/adopt/dist/detect.js
 var REPO_TYPES = [
@@ -755,7 +742,7 @@ var yaml = (data, header) => `${header ? `# ${header}
 var DEFAULT_HEADER = "Built-in default written out by Atticus. Change it through a pull request; delete the file to go back to the default.";
 var CHECKLISTS_README = "# Checklists\n\nCritic rule packs (contract C3), one YAML file per pack. The constitution is built in; packs here add organisation rules.\n";
 var STARTER_CHECKLISTS_README = `${CHECKLISTS_README}
-\`review.yaml\` is the starter review (ADR 0120): AI rules the critic's lenses apply to every draft, judging its meaning against the estate and the brief, and two code checks (a call's timeout, a store write's rollback step). It is this organisation's configuration, not the constitution: change, waive or retire its rules, or delete one, through a pull request. Delete the file to turn the starter review off; \`atticus upgrade\` won't add it back.
+\`review.yaml\` is the starter review (ADR 0120): AI rules the critic's lenses apply to every draft, judging its meaning against the estate and the brief, and three code checks (a call's timeout, a store write's rollback step, a story criterion from an attached note quoting the note's line). It is this organisation's configuration, not the constitution: change, waive or retire its rules, or delete one, through a pull request. Delete the file to turn the starter review off; \`atticus upgrade\` won't add it back.
 `;
 var FOLDER_READMES = [
   [
@@ -1210,7 +1197,7 @@ async function unadopt(path) {
       if (now === before[f])
         continue;
       const text2 = await readText(full) ?? "";
-      const back = withoutSection(text2).find((t) => sha256(t) === before[f]);
+      const back = withoutSection(text2).flatMap((t) => [t, t.replace(/\r?\n/g, "\r\n")]).find((t) => sha256(t) === before[f]);
       if (back !== void 0) {
         await writeFile4(full, back, "utf8");
         res.restored.push(f);
@@ -2868,7 +2855,6 @@ export {
   resolveMetarepo,
   orgPackDirOf,
   pathsFor,
-  inPlacePath,
   REPO_TYPES,
   typeFromDescription,
   typeFromFrameworks,

package/dist/chunk-2UCRFXFZ.js

@@ -0,0 +1,130 @@
+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 {
+  estateFacts,
+  measureLinks,
+  parseLinkReference
+} from "./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 {
+  loadOrgPack,
+  orgPackDirOf
+} from "./chunk-PAINMSID.js";
+import "./chunk-ODMBPDDV.js";
+import "./chunk-TV45EUGM.js";
+
+// packages/cli/dist/measure-commands.js
+import { readFile } from "node:fs/promises";
+import { isAbsolute, resolve } from "node:path";
+var MEASURE_WORDS = {
+  usage: "Usage: atticus join measure <reference.yaml> [--json] (docs/guide/measure.md)",
+  noFile: (file) => `There is no reference file at ${file}. List a few known repos' entries and calls in it (docs/guide/measure.md), then run atticus join measure again.`,
+  badFile: (file) => `${file} isn't a reference Atticus can read:`,
+  nothing: (file) => `${file} lists no calls or entries, so there is nothing to measure. List a few known repos' entries and calls (docs/guide/measure.md).`,
+  head: (file, m, repos) => `Measured against ${file}${m.estate ? ` (${m.estate})` : ""}: ${plural(repos, "repo")}, ${plural(m.reading.entries.want, "entry", "entries")} and ${plural(m.reading.calls.want, "call")} listed.`,
+  reading: (m) => `Reading: ${m.reading.entries.found} of ${plural(m.reading.entries.want, "entry", "entries")} and ${m.reading.calls.found} of ${plural(m.reading.calls.want, "call")} are in the facts${pct(m.reading.entries.found + m.reading.calls.found, m.reading.entries.want + m.reading.calls.want)}.`,
+  precision: (m) => m.links.precision === null ? "Join precision: no listed call is joined to a repo yet." : `Join precision ${m.links.precision}: ${m.links.right} of the ${plural(m.links.joined, "listed call")} joined to a repo reach the one you named.`,
+  recall: (m) => m.links.recall === null ? "Join recall: no listed call should reach a repo of the estate." : `Join recall ${m.links.recall}: ${m.links.right} of the ${plural(m.links.want, "call")} that should reach a repo of the estate do.`,
+  rest: (m) => [
+    `${m.links.asked} asked`,
+    `${m.links.inferred.joined} guessed (${m.links.inferred.right} right)`,
+    `${m.links.outside.right} of ${m.links.outside.want} outside right`,
+    ...m.links.none.want ? [`${m.links.none.right} of ${m.links.none.want} "none" right`] : [],
+    `${m.links.unlinked} unlinked`,
+    ...m.links.missed ? [`${m.links.missed} with no entry serving them`] : [],
+    `${m.links.notRead} not read`
+  ].join(", ") + ".",
+  calls: "Each listed call:",
+  missed: "Listed but not in the facts (read the repo again, or check the line and path you wrote):",
+  readOnly: "Nothing was written."
+};
+var plural = (n, one, many = `${one}s`) => `${n} ${n === 1 ? one : many}`;
+var pct = (a, b) => b ? ` (${Math.round(a / b * 100)}%)` : "";
+var wantText = (w) => "repo" in w ? w.repo : "outside" in w ? `outside ${w.outside}` : "none";
+function gotText(g) {
+  switch (g.kind) {
+    case "joined":
+      return `${g.repo} ${g.entry}${g.via?.length ? ` via ${g.via.join(", ")}` : ""}${g.inferred ? ` (a guess: ${g.inferred})` : ""}`;
+    case "outside":
+      return `outside ${g.system}`;
+    case "asked":
+      return `asked (${g.reason}): atticus questions`;
+    case "missed":
+      return `named ${g.target}, which serves nothing like it`;
+    case "unlinked":
+      return "nothing";
+    case "not-read":
+      return "not in the facts";
+  }
+}
+async function joinMeasure(root, file, o, io) {
+  if (!file) {
+    io.err(MEASURE_WORDS.usage);
+    return 2;
+  }
+  const path = isAbsolute(file) ? file : resolve(root, file);
+  const text = await readFile(path, "utf8").catch(() => void 0);
+  if (text === void 0) {
+    io.err(MEASURE_WORDS.noFile(file));
+    return 2;
+  }
+  const parsed = parseLinkReference(text);
+  if (!parsed.ok) {
+    io.err(MEASURE_WORDS.badFile(file));
+    for (const i of parsed.issues)
+      io.err(`  ${i}`);
+    return 2;
+  }
+  const ref = parsed.reference;
+  if (!ref.repos.some((r) => r.calls.length || r.entries.length)) {
+    io.err(MEASURE_WORDS.nothing(file));
+    return 2;
+  }
+  const op = await loadOrgPack(orgPackDirOf(root));
+  const estate = await estateFacts(root, op, "");
+  const m = measureLinks({ op, repos: estate.repos, targets: estate.targets, reference: ref });
+  const notes = [...estate.notes, ...m.notes];
+  if (o.json) {
+    io.out(JSON.stringify({ reference: file, ...m, notes }, null, 2));
+    return 0;
+  }
+  io.out(MEASURE_WORDS.head(file, m, ref.repos.length));
+  io.out(MEASURE_WORDS.reading(m));
+  io.out(MEASURE_WORDS.precision(m));
+  io.out(MEASURE_WORDS.recall(m));
+  io.out(MEASURE_WORDS.rest(m));
+  if (m.calls.length) {
+    io.out("");
+    io.out(MEASURE_WORDS.calls);
+    for (const c of m.calls)
+      io.out(`  ${c.right ? "ok" : "--"}  ${c.repo} ${c.call}: wanted ${wantText(c.want)}, got ${gotText(c.got)}`);
+  }
+  if (m.reading.missed.length) {
+    io.out("");
+    io.out(MEASURE_WORDS.missed);
+    for (const x of m.reading.missed)
+      io.out(`  ${x}`);
+  }
+  if (notes.length) {
+    io.out("");
+    for (const n of notes)
+      io.out(n);
+  }
+  io.out("");
+  io.out(MEASURE_WORDS.readOnly);
+  return 0;
+}
+export {
+  MEASURE_WORDS,
+  joinMeasure
+};

package/dist/{chunk-MVZVGYXH.js → chunk-2V2ZPQQU.js}

@@ -5,17 +5,17 @@ var __atticusUrl = (p) => new URL("./" + p, import.meta.url).href;
 var __atticusPath = (p) => __atticusFileURLToPath(__atticusUrl(p));
 import {
   changeStore
-} from "./chunk-XY3P7JW7.js";
+} from "./chunk-HSZMPH5T.js";
 import {
   ManualNotifier,
   fileStepStore
-} from "./chunk-7JUKDRSZ.js";
+} from "./chunk-4KNOJC6J.js";
 import {
   readCatalog
-} from "./chunk-27NKUS3X.js";
+} from "./chunk-4SLBCEPD.js";
 import {
   readServiceRequestExports
-} from "./chunk-23W7VWKR.js";
+} from "./chunk-CPK7QSBG.js";
 import {
   Initiative,
   IsoDate,
@@ -32,10 +32,11 @@ import {
   serializeRecord,
   stateDirOf,
   toCanonicalYaml
-} from "./chunk-S2A7YC3Q.js";
+} from "./chunk-PAINMSID.js";
 import {
-  SCHEMA_VERSION
-} from "./chunk-D3PO4DOZ.js";
+  SCHEMA_VERSION,
+  renameRetrying
+} from "./chunk-ODMBPDDV.js";
 import {
   __toESM
 } from "./chunk-TV45EUGM.js";
@@ -717,7 +718,7 @@ var stepsOf = (j) => j.phases.flatMap((p) => p.steps.map((s) => ({ ...s, phase:
 
 // packages/initiatives/dist/store.js
 var import_yaml2 = __toESM(require_dist(), 1);
-import { mkdir, readFile as readFile2, rename, rm, writeFile } from "node:fs/promises";
+import { mkdir, readFile as readFile2, rm, writeFile } from "node:fs/promises";
 import { dirname, join as join2 } from "node:path";
 var curatedDirOf = (root) => join2(catalogDirOf(root), "curated");
 var initiativesDirOf = (root) => join2(stateDirOf(root), "initiatives");
@@ -736,7 +737,7 @@ async function writeAtomic(path, text) {
   const tmp = `${path}.tmp`;
   try {
     await writeFile(tmp, text, "utf8");
-    await rename(tmp, path);
+    await renameRetrying(tmp, path);
   } catch (e) {
     await rm(tmp, { force: true });
     throw e;
@@ -854,6 +855,44 @@ async function readPlan(root, id) {
     throw new InitiativeError(`initiatives/${id}/plan.yaml: ${r.error.issues.map((i) => `${i.path.join(".")} ${i.message}`).join("; ")}`);
   return r.data;
 }
+var CommitmentNote = external_exports.strictObject({
+  at: IsoDateTime,
+  by: external_exports.string().min(1),
+  commitment: RecordId,
+  status: external_exports.enum(["open", "at-risk", "done", "slipped"]),
+  note: external_exports.string().min(1)
+});
+var NotesFile = external_exports.strictObject({
+  schemaVersion: external_exports.literal(1),
+  initiative: RecordId,
+  notes: external_exports.array(CommitmentNote).default([])
+});
+var notesPath = (root, id) => join2(initiativeDir(root, id), "notes.yaml");
+async function readNotes(root, id) {
+  let text;
+  try {
+    text = await readFile2(notesPath(root, id), "utf8");
+  } catch {
+    return [];
+  }
+  const r = NotesFile.safeParse((0, import_yaml2.parse)(text));
+  if (!r.success)
+    throw new InitiativeError(`initiatives/${id}/notes.yaml: ${r.error.issues.map((i) => `${i.path.join(".")} ${i.message}`).join("; ")}`);
+  return r.data.notes;
+}
+async function appendNote(root, id, note) {
+  const notes = [...await readNotes(root, id), CommitmentNote.parse(note)];
+  await writeAtomic(notesPath(root, id), toCanonicalYaml(NotesFile.parse({ schemaVersion: 1, initiative: id, notes }), [
+    "schemaVersion",
+    "initiative",
+    "at",
+    "by",
+    "commitment",
+    "status",
+    "note"
+  ]));
+  return `initiatives/${id}/notes.yaml`;
+}
 var slugOf = (s) => s.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 80).replace(/-+$/, "");
 
 // packages/initiatives/dist/dates.js
@@ -1285,6 +1324,9 @@ async function initiativeStatus(root, id, o = {}) {
   }
   const commitmentRecords = view.all("Commitment").filter((c) => c.initiative === id);
   const recordOf = new Map(commitmentRecords.map((c) => [c.id, c]));
+  const lastNote = /* @__PURE__ */ new Map();
+  for (const n of await readNotes(root, id))
+    lastNote.set(n.commitment, { at: n.at, by: n.by, status: n.status, note: n.note });
   const items = [];
   let projectedRelease;
   if (plan) {
@@ -1331,9 +1373,14 @@ async function initiativeStatus(root, id, o = {}) {
       const d = done.get(i.key);
       const finish = ef.get(i.key);
       const float = d ? 0 : workingDays(finish, lf.get(i.key));
-      const status2 = d ? "done" : today > i.due ? "slipped" : finish > i.due ? "at-risk" : "open";
+      let status2 = d ? "done" : today > i.due ? "slipped" : finish > i.due ? "at-risk" : "open";
+      const said = i.commitment ? commitmentRecords.find((c) => c.id === i.commitment)?.commitmentStatus : void 0;
+      const recorded = said === "at-risk" || said === "slipped" ? said : void 0;
+      const rank = { slipped: 3, "at-risk": 2 };
+      if (!d && recorded && rank[recorded] > (rank[status2] ?? 0))
+        status2 = recorded;
       const lateBefore = i.after.filter((a) => !done.get(a) && ef.get(a) > byKey.get(a).due);
-      const why = status2 === "done" ? "Done." : status2 === "slipped" ? `Due ${i.due} and not done.` : status2 === "at-risk" ? `Projected to finish ${finish}, after its due date ${i.due}${lateBefore.length ? `, because ${lateBefore.join(", ")} ${lateBefore.length === 1 ? "is" : "are"} late` : ""}.` : `On plan: due ${i.due}.`;
+      const why = status2 === "done" ? "Done." : recorded && status2 === recorded ? recorded === "slipped" ? "Recorded as slipped." : "Recorded as at risk." : status2 === "slipped" ? `Due ${i.due} and not done.` : status2 === "at-risk" ? `Projected to finish ${finish}, after its due date ${i.due}${lateBefore.length ? `, because ${lateBefore.join(", ")} ${lateBefore.length === 1 ? "is" : "are"} late` : ""}.` : `On plan: due ${i.due}.`;
       items.push({
         key: i.key,
         kind: i.kind,
@@ -1377,6 +1424,13 @@ async function initiativeStatus(root, id, o = {}) {
       [status2, why] = ["at-risk", `Needed by ${c.neededBy}: two working days or less left.`];
     else
       [status2, why] = ["open", `Needed by ${c.neededBy}.`];
+    const rank = { slipped: 3, "at-risk": 2 };
+    const said = c.commitmentStatus;
+    if ((said === "at-risk" || said === "slipped") && rank[said] > (rank[status2] ?? 0))
+      [status2, why] = [
+        said,
+        said === "slipped" ? "Recorded as slipped." : "Recorded as at risk."
+      ];
     return {
       id: c.id,
       what: c.what,
@@ -1384,7 +1438,8 @@ async function initiativeStatus(root, id, o = {}) {
       neededBy: c.neededBy,
       recorded: c.commitmentStatus,
       status: status2,
-      why
+      why,
+      ...lastNote.get(c.id) ? { note: lastNote.get(c.id) } : {}
     };
   });
   const changes = initiative.changes.map((c) => {
@@ -1442,7 +1497,19 @@ async function recordCommitment(root, id, o) {
   const by = o.by.replace(/^@/, "");
   const evidence = c.evidence.some((e) => e.source === "manual" && e.by === by) ? c.evidence : [...c.evidence, ...manualEvidence(by)];
   const updated = { ...c, commitmentStatus: o.status, evidence };
-  return { commitment: updated, written: await writeCurated(root, updated) };
+  const written = await writeCurated(root, updated);
+  const note = o.note?.trim();
+  if (!note)
+    return { commitment: updated, written };
+  const at = (o.now ?? /* @__PURE__ */ new Date()).toISOString().replace(/\.\d{3}Z$/, "Z");
+  const notes = await appendNote(root, c.initiative, {
+    at,
+    by,
+    commitment: id,
+    status: o.status,
+    note
+  });
+  return { commitment: updated, written, notes };
 }
 
 // packages/initiatives/dist/card.js
@@ -1599,6 +1666,7 @@ export {
   workingDays,
   laterOf,
   JOURNEYS,
+  initiativesDirOf,
   InitiativeError,
   readView,
   readPlan,