@intentius/behold 0.29.10 → 0.29.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@intentius/behold",
-  "version": "0.29.10",
+  "version": "0.29.12",
   "license": "Apache-2.0",
   "type": "module",
   "description": "behold — a live control plane on chant. See your whole estate (every substrate in one graph), coloured by drift; act through delegated, gated Ops.",

package/web/kiosk-annotations.js

@@ -123,7 +123,7 @@ export const ANNOTATIONS = [
   {
     id: "T12", product: "chant", feature: "Waiting for approval",
     pattern: String.raw`^Nothing in wave \d+ or after it ran\. (?:Read the plans, then|It waits on .+\.) approve: (chant approve \S+ \S+ --plan \S+) \(with --resume, the approval also starts this job again\)$`,
-    note: "The job ended without applying this wave or any later one; the printed chant approve command records the approval and can restart the job.",
+    note: "The job ends at this wave; the printed chant approve command records the approval and can restart the job.",
     docs: { product: "chant", path: "/guide/op-waves/#resume-the-waiting-job" },
     source: "chant:packages/core/src/op/op-waves-run.ts:1587",
     sample: "Nothing in wave 2 or after it ran. Read the plans, then approve: chant approve tf-apply wave-2 --plan jcs1-sha256:95b79f4946fffd67b2d3da8971084852c4182623d8223b4a307f7067375fe37a (with --resume, the approval also starts this job again)",
@@ -163,7 +163,7 @@ export const ANNOTATIONS = [
   {
     id: "T17", product: "terragucci", feature: "Changed after approval",
     pattern: String.raw`^wave \d+ changed after it was approved, so nothing in it was applied\. approved: (\S+); planned now: (\S+)\. `,
-    note: "The wave planned to a different digest than the one approved, so nothing applied; the new plan needs its own approval.",
+    note: "The wave planned to a different digest than the one approved; the new plan needs its own approval.",
     docs: { product: "terragucci", path: "/guides/fix-a-refused-wave/" },
     source: "terragucci:packages/terragucci/src/apply.ts:2983",
     sample: "wave 1 changed after it was approved, so nothing in it was applied. approved: jcs1-sha256:c7d57fcf0bf559242e2104f4d0ed562888f6bd99ed8c5af350db462771169877; planned now: jcs1-sha256:0be8353d2143e463577e05eca00b62cabd51791aaa383051930e384340812052. Read the new plan, then approve it: terragucci approve wave-1 --plan jcs1-sha256:0be8353d2143e463577e05eca00b62cabd51791aaa383051930e384340812052",
@@ -171,7 +171,7 @@ export const ANNOTATIONS = [
   {
     id: "T18", product: "terragucci", feature: "Changed after approval",
     pattern: String.raw`^wave \d+ of \d+: approved by (\S+), but these roots planned differently since: (.+)$`,
-    note: "Names the roots whose plans differ from the approved ones, which is why the approval no longer covers the wave.",
+    note: "Names the roots whose plans differ from the approved ones, so the wave needs a new approval.",
     docs: { product: "terragucci", path: "/tutorial/changed-wave/" },
     source: "terragucci:packages/terragucci/src/apply.ts:2984",
     sample: "wave 1 of 3: approved by terragucci-admin, but these roots planned differently since: network",
@@ -187,7 +187,7 @@ export const ANNOTATIONS = [
   {
     id: "T20", product: "terragucci", feature: "Unreviewed outputs",
     pattern: String.raw`^wave \d+ of \d+: nobody reviewed the plan it made on those outputs, so it waits for an approval of it(?: although gate is \S+)?$`,
-    note: "This wave planned on outputs nobody could see in review, so it waits for an approval even when its gate would let it through.",
+    note: "This wave planned on outputs that got their values during the apply, so a person approves it even when its gate would let it through.",
     docs: { product: "terragucci", path: "/concepts/waves-and-approvals/#linked-roots" },
     source: "terragucci:packages/terragucci/src/apply.ts:2562",
     sample: "wave 1 of 3: nobody reviewed the plan it made on those outputs, so it waits for an approval of it although gate is on-destructive",
@@ -203,7 +203,7 @@ export const ANNOTATIONS = [
   {
     id: "T22", product: "terragucci", feature: "Unreviewed outputs",
     pattern: String.raw`^wave \d+ of \d+: (\S+) reads? no output of (\S+) that (.+) left unknown$`,
-    note: "Every output this root reads was already known when it was planned, so its plan holds no values nobody reviewed.",
+    note: "Every output this root reads was known when it was planned, so its plan holds only reviewed values.",
     docs: { product: "terragucci", path: "/concepts/waves-and-approvals/#linked-roots" },
     source: "terragucci:packages/terragucci/src/unknown-reads.ts:142",
     sample: "wave 2 of 3: app reads no output of network that wave 1's plan of network in this push (2026/10/e8863ee426177316d84499a5b8401c4b6adfb546/tf-apply-wave-1) left unknown",
@@ -235,7 +235,7 @@ export const ANNOTATIONS = [
   {
     id: "T26", product: "terragucci", feature: "Unreviewed outputs",
     pattern: String.raw`^wave \d+ of \d+: (\S+) reads (.+), unknown at plan time and computed in this apply$`,
-    note: "Inside a composed run this root reads a value that only gets computed during the apply, so nobody saw it in the plan.",
+    note: "Inside a composed run this root reads a value computed during the apply.",
     docs: { product: "terragucci", path: "/concepts/waves-and-approvals/#approval" },
     source: "terragucci:packages/terragucci/src/apply.ts:2022",
     sample: "wave 1 of 3: app reads subnet_id of network, unknown at plan time and computed in this apply",
@@ -275,7 +275,7 @@ export const ANNOTATIONS = [
   {
     id: "T31", product: "chant", feature: "Apply failed",
     pattern: String.raw`^wave \d+ \(([^)]+)\): apply failed, so none of (.+) counts as applied$`,
-    note: "The wave's apply failed. None of its roots count as applied and later waves do not run.",
+    note: "The wave's apply failed. Its roots and the later waves wait for a re-run.",
     docs: { product: "terragucci", path: "/reference/responses/#apply-failed" },
     source: "chant:packages/core/src/op/op-waves-run.ts:2092",
     sample: "wave 2 (wave-2): apply failed, so none of app counts as applied",
@@ -379,7 +379,7 @@ export const ANNOTATIONS = [
   {
     id: "T44", product: "terragucci", feature: "Eligible approvers",
     pattern: String.raw`^wave \d+ of \d+: (\S+) (.+), so \1's (approval|review) doesn't count$`,
-    note: "This person had a hand in the change. Their approval or review does not count toward the gate.",
+    note: "This person worked on the change, so the gate counts approvals from someone else.",
     docs: { product: "terragucci", path: "/guides/approve-a-wave/#eligible-approvers" },
     source: "terragucci:packages/terragucci/src/involved.ts:175",
     sample: "wave 2 of 3: demo-author opened pull request 1086, so demo-author's review doesn't count", // made from the source, no run printed it yet
@@ -387,7 +387,7 @@ export const ANNOTATIONS = [
   {
     id: "T45", product: "terragucci", feature: "Eligible approvers",
     pattern: String.raw`^wave \d+ of \d+: an approval counts only from someone who had no hand in the change, so not from (.+)$`,
-    note: "An approval from the people named here does not count because they had a hand in the change.",
+    note: "The people named here worked on the change, so the gate counts approvals from someone else.",
     docs: { product: "terragucci", path: "/guides/approve-a-wave/#eligible-approvers" },
     source: "terragucci:packages/terragucci/src/apply.ts:3001",
     sample: "wave 2 of 3: an approval counts only from someone who had no hand in the change, so not from demo-author", // made from the source, no run printed it yet

package/web/kiosk-approval.js

@@ -123,7 +123,7 @@ export function approvalModel({ demo, runs = [], history = null, lifecycle = nul
       commit: requested && requested.source ? requested.source.commit || null : null,
       line: reqLine,
     });
-  } else Object.assign(reqRow, { state: "dim", text: `Wave ${wave} has not asked for an approval yet.` });
+  } else Object.assign(reqRow, { state: "dim", text: `Wave ${wave} asks for an approval when it reaches its gate.` });
 
   // Approved.
   const apRow = { id: "approved", label: "Approved" };
@@ -159,7 +159,7 @@ export function approvalModel({ demo, runs = [], history = null, lifecycle = nul
       line: start2 ? { run: resume.id, ...start2 } : found,
     });
   } else if (resume) {
-    Object.assign(rsRow, { state: "going", text: `/terragucci apply started the waves again; wave ${wave} has not reached its gate yet.`, at: resume.started_at || resume.created_at, runId: resume.id, line: start2 ? { run: resume.id, ...start2 } : null });
+    Object.assign(rsRow, { state: "going", text: `/terragucci apply started the waves again; wave ${wave} reaches its gate next.`, at: resume.started_at || resume.created_at, runId: resume.id, line: start2 ? { run: resume.id, ...start2 } : null });
   } else {
     Object.assign(rsRow, { state: "dim", text: driver && driver.comment ? `Then a comment of ${driver.comment} on ${driver.on} starts the waves again.` : "A comment of /terragucci apply starts the waves again." });
   }

package/web/kiosk-popover.js

@@ -31,9 +31,9 @@ export const ACTION_HEAD = {
 
 export const WORDS = {
   same: "The plan leaves this resource as it is.",
-  untouched: "This plan does not touch its root, so nothing changes here.",
+  untouched: "This plan leaves its root as it is.",
   noPlan: "No plan yet for this resource.",
-  unknown: "known after apply, so no review shows this value",
+  unknown: "known after apply",
   long: "too long to show here",
   noneYet: "none yet",
   gone: "gone",

package/web/kiosk-popover.test.js

@@ -49,11 +49,11 @@ describe("what the popover says about a resource (behold#571)", () => {
     expect(p.applied).toBeNull();
   });
 
-  it("says an attribute known only after apply in the unknown tone, and that no review shows it", () => {
+  it("says an attribute known only after apply in the unknown tone", () => {
     const p = resourcePopover(answer(), "repo/app/terraform_data.listener_port");
     expect(p.unknown).toBe(true);
-    expect(p.lines).toEqual([{ text: "input: 8070 -> known after apply, so no review shows this value", tone: "unknown" }]);
-    expect(attributeLine({ path: "input", unknown: true })).toEqual({ text: "input: known after apply, so no review shows this value", tone: "unknown" });
+    expect(p.lines).toEqual([{ text: "input: 8070 -> known after apply", tone: "unknown" }]);
+    expect(attributeLine({ path: "input", unknown: true })).toEqual({ text: "input: known after apply", tone: "unknown" });
   });
 
   it("says no change for a card the plan leaves alone, and that a root the plan does not hold is untouched", () => {

package/web/kiosk.js

@@ -137,7 +137,7 @@ export function countdownWords(secs) {
 
 /** The driver's next action, as the kiosk says it. */
 export function nextWords(demo, now = new Date()) {
-  if (!demo.next) return { action: "nothing until the driver resets", when: "the driver stalled; it resets and starts the next cycle", secs: null };
+  if (!demo.next) return { action: "reset and start the next cycle", when: "the driver stalled; it resets and starts the next cycle", secs: null };
   const secs = secondsUntil(demo.next.at, now);
   return {
     action: demo.next.action,
@@ -198,9 +198,9 @@ export function runPanel(demo, answer, progress) {
   if (run) return { stage: "apply", run };
   if (pr) return { stage: "pull-request", pr };
   if (stage === "pull-request") {
-    return { stage: "pull-request", note: demo.pull === null ? "The pull request isn't open yet." : `The plan of pull request #${demo.pull} isn't in the reports yet: the pipeline is planning it.` };
+    return { stage: "pull-request", note: demo.pull === null ? "The pull request opens next." : `The pipeline is planning pull request #${demo.pull}.` };
   }
-  return { stage: "apply", note: demo.commit ? `The run view of ${demo.commit.slice(0, 8)} isn't in the reports yet: the pipeline writes it as the waves start.` : "No run yet." };
+  return { stage: "apply", note: demo.commit ? `The pipeline writes the run view of ${demo.commit.slice(0, 8)} as the waves start.` : "The first run starts next." };
 }
 
 /** The pull request's waves and reads, when the panel shows its plan. */
@@ -457,7 +457,7 @@ export function factLines(demo, answer) {
   if (totals) out.push(`pull request #${pr.run.pull_request} changes ${rootsCount(pr.totals_roots) || "across roots"}: ${totals}`);
   const b = blastRows(answer);
   if (b && b.changed.length) out.push(`blast radius: ${b.changed.join(", ")}${b.downstream.length ? `; downstream, nearest first: ${b.downstream.join(", ")}` : ""}`);
-  if (b && b.unplanned.length) out.push(`not touched: ${b.unplanned.join(", ")}`);
+  if (b && b.unplanned.length) out.push(`unchanged: ${b.unplanned.join(", ")}`);
   const cost = costLine(pr.cost);
   if (cost) out.push(cost);
   return out;
@@ -644,7 +644,7 @@ export function approvalPanel(model, s = {}) {
 }
 
 /** behold#571: the header's line when this behold serves another repo than the page's. */
-export const NO_GRAPH_WORDS = "This page shows the runs and their logs; the graph of this repo isn't served here.";
+export const NO_GRAPH_WORDS = "This page shows the runs and their logs.";
 
 /** behold#571: a tool's name, and its logo where its owner allows one, linking to its official site. */
 function brandEl(vendor) {
@@ -691,7 +691,7 @@ export const APPROVAL_POLL_MS = 4_000;
 export const LEDGER_POLL_MS = 10_000;
 
 /** behold#571: the approval panel's words for the approval the driver gives next, over its command. */
-export const DRIVER_SIGNS = "Next the driver plays a reviewer with no hand in the change and signs this plan with that reviewer's key:";
+export const DRIVER_SIGNS = "Next the driver plays a reviewer outside the change and signs this plan with that reviewer's key:";
 
 /** behold#571: "What is this?" says what behold is; the demo's own intro stays on the card between scenarios. */
 export const WHAT_TEXT = "This is behold, a live view of infrastructure. It draws the setup as a graph and shows its pipeline runs, their logs and the approvals they wait for, as they happen. Here it watches a demo pipeline that runs one change through several Terraform states in a loop.";

package/web/kiosk.test.js

@@ -49,7 +49,7 @@ describe("frameSrc (#569)", () => {
 
 describe("factLines (#569)", () => {
   it("says the pull request's totals and blast radius", () => {
-    expect(factLines(demo("planned"), answer)).toEqual(["pull request #7 changes across 3 roots: 3 update", "blast radius: app, edge, network", "not touched: other"]);
+    expect(factLines(demo("planned"), answer)).toEqual(["pull request #7 changes across 3 roots: 3 update", "blast radius: app, edge, network", "unchanged: other"]);
   });
 });
 
@@ -81,7 +81,7 @@ describe("before the merge, no other cycle's plan (#569)", () => {
       expect(matchingPull(ours, now(step))).toBe(ours.pull_request);
       expect(focusRoots(now(step), older, null)).toBe(null);
       expect(factLines(now(step), older)).toEqual([]);
-      expect(runPanel(now(step), older, null).note).toBe("The plan of pull request #989 isn't in the reports yet: the pipeline is planning it.");
+      expect(runPanel(now(step), older, null).note).toBe("The pipeline is planning pull request #989.");
     }
   });
   it("from the merge on, takes the plan by its number: demo.json's commit is the merge's", () => {

package/web/terragucci-change.js

@@ -23,7 +23,7 @@
 import { ago, clock, hrefOf } from "./terragucci.js";
 
 export const STATE_WORDS = {
-  "not-started": "not started",
+  "not-started": "queued",
   planned: "planned",
   waiting: "waiting for an approval",
   applying: "applying",
@@ -119,13 +119,13 @@ export function readCardIds(answer, e) {
   return [`${r.member}/${r.name}/data.terraform_remote_state.${data}`, `${r.member}/${r.name}/data.terraform_estate_outputs.${data}`];
 }
 
-/** "5 create · 1 update", no-ops and zeros left out; "no changes" when nothing moves. */
+/** "5 create · 1 update", no-ops and zeros left out; "unchanged" when nothing moves. */
 export function totalsLine(totals) {
   if (!totals) return null;
   const order = ["create", "update", "replace", "delete", "forget", "import", "read"];
   const keys = [...order.filter((k) => k in totals), ...Object.keys(totals).filter((k) => !order.includes(k) && k !== "no-op").sort()];
   const parts = keys.filter((k) => totals[k] > 0).map((k) => `${totals[k]} ${k}`);
-  return parts.length ? parts.join(" · ") : "no changes";
+  return parts.length ? parts.join(" · ") : "unchanged";
 }
 
 /** #568 step 2: "across 3 roots", from `totals_roots` (report minor 35); null on an older report, which has no count. */
@@ -216,7 +216,7 @@ export function composeLine(pr) {
   return null;
 }
 
-const APPLY_WORDS = { applied: "applied", partial: "applied in part", failed: "failed", "not-started": "not started", "no-changes": "no changes" };
+const APPLY_WORDS = { applied: "applied", partial: "applied in part", failed: "failed", "not-started": "queued", "no-changes": "unchanged" };
 const STATE_AS = { written: "state written", unchanged: "state as it was, nothing to write", "not-written": "state not written" };
 
 /** One member of a composed run as planned: "app as module.app: 1 create". */
@@ -494,7 +494,7 @@ export function rootStates(answer, progress) {
         const blast = pr.blast && Array.isArray(pr.blast.roots) ? pr.blast.roots : null;
         const readsUnknown = (pr.reads || []).some((r) => r.root === root && r.unknown && r.unknown.length) || !!(c && c.held_by_unknown && c.held_by_unknown.includes(root));
         if (blast && !blast.includes(root) && !readsUnknown) {
-          roots[root] = { wave: w.number, state: "unchanged", tone: STATE_TONE.unchanged, words: `wave ${w.number} · no changes`, facts: [] };
+          roots[root] = { wave: w.number, state: "unchanged", tone: STATE_TONE.unchanged, words: `wave ${w.number} · unchanged`, facts: [] };
           continue;
         }
         const facts = [];
@@ -515,7 +515,7 @@ export function rootStates(answer, progress) {
       const inRun = [...composedOf].filter(([, n]) => n === w.number).map(([r]) => r);
       for (const root of [...new Set([...w.roots, ...(c ? c.roots : []), ...inRun])]) {
         if (!rootChanges(w, root)) {
-          roots[root] = { wave: w.number, state: "unchanged", tone: STATE_TONE.unchanged, words: `wave ${w.number} · no changes`, facts: [] };
+          roots[root] = { wave: w.number, state: "unchanged", tone: STATE_TONE.unchanged, words: `wave ${w.number} · unchanged`, facts: [] };
           continue;
         }
         const m = c && c.members ? c.members.find((x) => x.root === root) : null;
@@ -529,7 +529,7 @@ export function rootStates(answer, progress) {
         if (m && m.apply) facts.push(memberApplyLine(m).replace(`${root}: `, "")); // this root's own part, not the wave's
         else if (c && !composedHere) {
           // Beside a composed run in its wave: the wave's figures are the run's, not this root's.
-        } else if (totals && state !== "not-started") facts.push(totals === "no changes" ? `${together ? "the wave: " : ""}no changes` : `${together ? "the wave " : ""}${state === "applied" ? "applied" : state === "failed" || state === "refused" ? "planned" : "plans"} ${totals}`);
+        } else if (totals && state !== "not-started") facts.push(totals === "unchanged" ? `${together ? "the wave: " : ""}unchanged` : `${together ? "the wave " : ""}${state === "applied" ? "applied" : state === "failed" || state === "refused" ? "planned" : "plans"} ${totals}`);
         if (state === "failed" || state === "refused") {
           const before = progress.states.filter((x) => x.number < w.number && x.state === "applied").map((x) => x.number);
           if (before.length) facts.push(`wave ${list(before)} stays applied`);

package/web/terragucci-change.test.js

@@ -60,7 +60,7 @@ describe("rootStates, a root that plans no change (#569 review)", () => {
     const p = run([{ number: 1, state: "waiting", roots: ["network", "other"], reads: [], digest: "jcs1-sha256:365a4f0d54d7aa", totals: { create: 5 }, changed: ["network"] }], []);
     const { roots: r } = rootStates(prAnswer, p);
     expect(r.network.state).toBe("waiting");
-    expect(r.other).toMatchObject({ state: "unchanged", tone: "idle", words: "wave 1 · no changes", facts: [] });
+    expect(r.other).toMatchObject({ state: "unchanged", tone: "idle", words: "wave 1 · unchanged", facts: [] });
   });
   it("takes a composed member's own no-changes over the wave's state", () => {
     const p = run([{ number: 1, state: "failed", roots: ["network"], reads: [], composed: { roots: ["network", "edge"], members: [{ root: "network", module: "m", apply: "failed" }, { root: "edge", module: "e", apply: "no-changes" }] } }], []);

package/web/terragucci-components-live.js

@@ -30,13 +30,13 @@ export const JOB_TONE = { queued: "idle", running: "busy", waiting: "wait", succ
 /** A card's state in one or two words, when the full words don't fit the card. */
 export const SHORT_WORDS = {
   planned: "planned",
-  "not-started": "not started",
+  "not-started": "queued",
   waiting: "waiting",
   applying: "applying",
   applied: "applied",
   refused: "refused",
   failed: "failed",
-  unchanged: "no changes",
+  unchanged: "unchanged",
   outside: "not in this run",
 };
 

package/web/terragucci-components-live.test.js

@@ -98,7 +98,7 @@ describe("the component marks (#571)", () => {
       network: ["applied", "ok", "applied"],
       other: ["outside", "idle", "not in this run"],
       app: ["waiting", "wait", "waiting for an approval"],
-      edge: ["not-started", "idle", "not started"],
+      edge: ["not-started", "idle", "queued"],
     });
     expect(m.cards.app.short).toBe("waiting");
     expect(m.waves[2]).toMatchObject({ job: "apply-wave-2", state: "waiting", tone: "wait", title: "apply-wave-2 · held at the gate" });

package/web/terragucci-estate.js

@@ -211,7 +211,7 @@ export function renderEstateSvg(answer, now = new Date(), doc = document) {
     const run = runLine(p, now);
     g.appendChild(titled(doc, svgEl(doc, "text", { x: b.x + PAD, y: b.y + 55, class: `tg-project-run tg-${run.tone}` }, run.text), run.title));
     for (const w of b.waves) {
-      g.appendChild(svgEl(doc, "text", { x: w.x, y: w.y + 14, class: `tg-wave-label tg-wave-${w.state}`, "data-wave": w.number }, `wave ${w.number} · ${w.state === "not-started" ? "not started" : w.state}`));
+      g.appendChild(svgEl(doc, "text", { x: w.x, y: w.y + 14, class: `tg-wave-label tg-wave-${w.state}`, "data-wave": w.number }, `wave ${w.number} · ${w.state === "not-started" ? "queued" : w.state}`));
       for (const r of w.roots) {
         const node = svgEl(doc, "g", { class: "tg-root-node", "data-root": r.root });
         node.appendChild(svgEl(doc, "rect", { x: r.x, y: r.y, width: r.w, height: r.h, rx: 4 }));

package/web/terragucci.js

@@ -86,7 +86,7 @@ export function stageCell(verdict, stage, now = new Date()) {
     if (verdict.approval === "waiting") return { text: `wave ${verdict.run.wave} waiting, ${age}`, tone: "wave", title };
     return { text: `wave ${verdict.run.wave} ${verdict.applied ? "applied" : verdict.approval || "ran"}, ${age}`, tone: "ok", title };
   }
-  if (verdict.changes === 0) return { text: `${stage === "tf-drift" ? "none" : "no changes"}, ${age}`, tone: "ok", title };
+  if (verdict.changes === 0) return { text: `${stage === "tf-drift" ? "none" : "unchanged"}, ${age}`, tone: "ok", title };
   const noun = stage === "tf-drift" ? "drifted" : `change${verdict.changes === 1 ? "" : "s"}`;
   const pr = verdict.run.pull_request ? ` (PR #${verdict.run.pull_request})` : "";
   return { text: `${verdict.changes} ${noun}${pr}, ${age}`, tone: stage === "tf-drift" ? "drift" : "plan", title };