create-cmp-cli 0.18.0 → 0.19.0

package/template/qa/lib/plan.mjs

@@ -33,6 +33,13 @@ import path from "node:path";
 
 export const PLAN_REL = "qa/.plan.json";
 export const REQUEST_REL = "qa/.request.json";
+// N5 (docs/features/drive-narration.md): closed chains leave a LOCAL trail —
+// request, steps, wall time, receipt state at close. Gitignored and excluded
+// from the hashed input surface like its siblings above, and deliberately NOT
+// a committed journal: it carries raw human prompts. Lane history that
+// belongs in the repo stays the flight recorder's.
+export const PLAN_HISTORY_REL = "qa/.plan-history.jsonl";
+const MAX_HISTORY_LINES = 50;
 
 // A marker older than this is a crashed writer, not a live run — the same
 // bound qa/watch.mjs and the preview daemon apply to the same files.
@@ -91,12 +98,17 @@ export function setPlan(root, { title, feature, steps } = {}) {
     .slice(0, MAX_STEPS)
     .map((s) => (s.length > MAX_LABEL_CHARS ? `${s.slice(0, MAX_LABEL_CHARS - 1)}…` : s));
   if (labels.length === 0) return { ok: false, reason: "a chain needs at least one step" };
+  // N1: the declaration's own write times ARE the timing data — createdAt for
+  // the whole chain, startedAt on step 1. No new claims, just timestamps the
+  // writes already imply; renderers derive durations from them.
+  const now = new Date().toISOString();
   return writeJson(path.join(root, PLAN_REL), {
     title: typeof title === "string" && title.trim() !== "" ? title.trim() : null,
     feature: typeof feature === "string" && feature.trim() !== "" ? feature.trim() : null,
-    steps: labels.map((label, i) => ({ n: i + 1, label, done: false })),
+    steps: labels.map((label, i) => ({ n: i + 1, label, done: false, ...(i === 0 ? { startedAt: now } : {}) })),
     current: 1,
-    updatedAt: new Date().toISOString(),
+    createdAt: now,
+    updatedAt: now,
   });
 }
 
@@ -112,10 +124,90 @@ export function markStep(root, n) {
   const step = Number(n);
   if (!Number.isInteger(step) || step < 1 || step > plan.steps.length + 1)
     return { ok: false, reason: `step must be 1..${plan.steps.length + 1} (=${plan.steps.length + 1} closes the chain), got ${n}` };
-  for (const s of plan.steps) s.done = s.n < step;
+  const now = new Date().toISOString();
+  for (const s of plan.steps) {
+    const willBeDone = s.n < step;
+    // N1: stamp doneAt the first time a step closes and startedAt the first
+    // time it becomes current — first-write-wins, so re-marking never
+    // rewrites history.
+    if (willBeDone && !s.done && !s.doneAt) s.doneAt = now;
+    s.done = willBeDone;
+    if (s.n === step && !s.startedAt) s.startedAt = now;
+  }
+  const closing = step > plan.steps.length && !plan.closedAt;
   plan.current = step > plan.steps.length ? null : step;
-  plan.updatedAt = new Date().toISOString();
-  return writeJson(path.join(root, PLAN_REL), plan).ok ? { ok: true, plan } : { ok: false, reason: "could not write the chain" };
+  if (closing) plan.closedAt = now;
+  plan.updatedAt = now;
+  if (!writeJson(path.join(root, PLAN_REL), plan).ok) return { ok: false, reason: "could not write the chain" };
+  // N5: the FIRST close leaves the trail entry; a re-close of an already
+  // closed chain never double-writes. Fail-soft — a trail that cannot be
+  // written must not fail the advance that was asked for.
+  if (closing) appendPlanHistory(root, plan);
+  return { ok: true, plan };
+}
+
+/** The receipt's verdict + rung right now, for the trail — fail-soft glance. */
+function receiptGlance(root) {
+  try {
+    const r = JSON.parse(fs.readFileSync(path.join(root, "qa/evidence/latest.json"), "utf8"));
+    return { verdict: r?.verdict ?? null, rung: r?.evidenceLevel?.rung ?? null };
+  } catch {
+    return null;
+  }
+}
+
+function appendPlanHistory(root, plan) {
+  try {
+    const started = Date.parse(plan.createdAt ?? "");
+    const closed = Date.parse(plan.closedAt ?? "");
+    const entry = {
+      schema: "cmp-plan-history/1",
+      at: plan.closedAt ?? new Date().toISOString(),
+      request: readRequest(root)?.text ?? null,
+      title: plan.title ?? null,
+      feature: plan.feature ?? null,
+      steps: plan.steps.map((s) => s.label),
+      durationMs: Number.isNaN(started) || Number.isNaN(closed) ? null : Math.max(0, closed - started),
+      receipt: receiptGlance(root),
+    };
+    const p = path.join(root, PLAN_HISTORY_REL);
+    let lines = [];
+    try {
+      lines = fs.readFileSync(p, "utf8").split("\n").filter((l) => l.trim() !== "");
+    } catch {
+      /* first entry */
+    }
+    lines.push(JSON.stringify(entry));
+    fs.writeFileSync(p, `${lines.slice(-MAX_HISTORY_LINES).join("\n")}\n`);
+    return { ok: true };
+  } catch (err) {
+    return { ok: false, reason: err?.message ?? String(err) };
+  }
+}
+
+/**
+ * The last `limit` closed chains, NEWEST FIRST — Drive's "Recent requests"
+ * fold. Absent trail or unparsable lines read as an empty/shorter list,
+ * never an error.
+ * @returns {object[]}
+ */
+export function readPlanHistory(root, limit = 5) {
+  try {
+    const raw = fs.readFileSync(path.join(root, PLAN_HISTORY_REL), "utf8");
+    const out = [];
+    for (const line of raw.split("\n")) {
+      if (!line.trim()) continue;
+      try {
+        const e = JSON.parse(line);
+        if (e && typeof e === "object") out.push(e);
+      } catch {
+        /* skip the line, keep the trail */
+      }
+    }
+    return out.slice(-Math.max(0, limit)).reverse();
+  } catch {
+    return [];
+  }
 }
 
 /** @returns {object|null} the declared chain, or null. */
@@ -134,33 +226,130 @@ export function clearPlan(root) {
   }
 }
 
-function markerFresh(root, name) {
+/**
+ * A marker read WITH its content (N2, docs/features/drive-narration.md):
+ * every other marker consumer is mtime-only, so the content is free to carry
+ * the lane's own narration — verify.mjs rewrites the lane marker at each
+ * step start with {step, index, total, stepStartedAt, expectedStepMs,
+ * expectedLaneMs}. Legacy "pid iso" content (older lanes, the render marker)
+ * reads as a bare truthy {} — busy, no narration. Stale/absent -> false.
+ * @returns {object|false}
+ */
+function markerInfo(root, name) {
+  const p = path.join(root, "composeApp", "build", name);
   try {
-    const st = fs.statSync(path.join(root, "composeApp", "build", name));
-    return Date.now() - st.mtimeMs < MARKER_FRESH_MS;
+    const st = fs.statSync(p);
+    if (Date.now() - st.mtimeMs >= MARKER_FRESH_MS) return false;
+    const raw = fs.readFileSync(p, "utf8").trim();
+    if (raw.startsWith("{")) {
+      try {
+        const parsed = JSON.parse(raw);
+        return parsed && typeof parsed === "object" ? parsed : {};
+      } catch {
+        return {};
+      }
+    }
+    return {};
   } catch {
     return false;
   }
 }
 
+// The build stage's observed tier (evidence-economics S3). Between the prompt
+// and the lane — most of the working time — the chain moved only if the agent
+// volunteered `plan.mjs --step`, so an undeclared chain was a still photo until
+// the lane landed. The lane marker already corroborates the lane mechanically;
+// this corroborates the build stage the same way: writes in the working tree
+// since the current request began. No agent cooperation required — which is
+// the point.
+const ACTIVITY_ROOTS = ["composeApp/src", "specs", "qa", "docs"];
+const ACTIVITY_SKIP_DIRS = new Set(["build", ".gradle", ".kotlin", ".git", ".idea", "node_modules", "evidence"]);
+// Machinery, not work: the chain's own files and the lane's outputs must not
+// count as "the agent wrote something", or the pulse would corroborate itself.
+const ACTIVITY_SKIP_FILES = new Set([".plan.json", ".request.json", ".plan-history.jsonl", "flight-recorder.jsonl", "approvals.log.jsonl", ".DS_Store"]);
+// Nothing written for this long, with no lane or render running, is a stall
+// worth naming — the human is watching a strip that has stopped moving.
+export const ACTIVITY_STALL_MS = 10 * 60 * 1000;
+
+/**
+ * Files written under the working roots since `sinceIso` — the request stamp.
+ * Pure filesystem, no git: a scaffold before `git init` still answers, and a
+ * mtime is a fact regardless of what is staged.
+ * @param {string} root
+ * @param {string|null|undefined} sinceIso the request's `at`
+ * @param {{now?: number}} [opts]
+ * @returns {{filesChanged: number, lastWriteAgoMs: (number|null), since: string}|null}
+ *   null when there is no request to measure from
+ */
+export function observeActivity(root, sinceIso, { now = Date.now() } = {}) {
+  const since = Date.parse(sinceIso ?? "");
+  if (Number.isNaN(since)) return null;
+  let filesChanged = 0;
+  let newest = -Infinity;
+  const walk = (dir) => {
+    let entries;
+    try {
+      entries = fs.readdirSync(dir, { withFileTypes: true });
+    } catch {
+      return;
+    }
+    for (const e of entries) {
+      if (e.isDirectory()) {
+        if (!ACTIVITY_SKIP_DIRS.has(e.name)) walk(path.join(dir, e.name));
+        continue;
+      }
+      if (!e.isFile() || ACTIVITY_SKIP_FILES.has(e.name)) continue;
+      let m;
+      try {
+        m = fs.statSync(path.join(dir, e.name)).mtimeMs;
+      } catch {
+        continue;
+      }
+      if (m > since) {
+        filesChanged += 1;
+        if (m > newest) newest = m;
+      }
+    }
+  };
+  for (const rel of ACTIVITY_ROOTS) walk(path.join(root, rel));
+  return {
+    filesChanged,
+    lastWriteAgoMs: filesChanged > 0 ? Math.max(0, now - newest) : null,
+    since: new Date(since).toISOString(),
+  };
+}
+
 /**
  * Everything a chain-rendering surface needs, with provenance attached:
  * request (tier 1) + plan with its age (tier 2) + what is ACTUALLY running
- * (tier 3 — the markers the lane and preview daemon already stamp).
+ * (tier 3 — the markers the lane and preview daemon already stamp, the lane's
+ * now carrying its own step narration) + the local trail of closed chains
+ * (N5). `busy.lane`/`busy.render` are truthy objects while fresh — existing
+ * truthiness consumers keep working unchanged.
  * @returns {{request: (object|null), plan: (object|null), planAgeMs: (number|null),
- *   busy: {lane: boolean, render: boolean}}}
+ *   busy: {lane: (object|false), render: (object|false)}, history: object[]}}
  */
 export function deriveChain(root) {
   const plan = readPlan(root);
   const at = plan ? Date.parse(plan.updatedAt) : NaN;
+  const busy = {
+    lane: markerInfo(root, ".cmp-lane-in-progress"),
+    render: markerInfo(root, ".cmp-render-in-progress"),
+  };
+  const request = readRequest(root);
+  // S3: the build stage's observed tier — writes since the request began.
+  const activity = observeActivity(root, request ? request.at : null);
   return {
-    request: readRequest(root),
+    request,
     plan,
     planAgeMs: Number.isNaN(at) ? null : Math.max(0, Date.now() - at),
-    busy: {
-      lane: markerFresh(root, ".cmp-lane-in-progress"),
-      render: markerFresh(root, ".cmp-render-in-progress"),
-    },
+    busy,
+    activity,
+    // Pre-rendered so every surface (chat, CLI, studio) speaks the observed
+    // tier in identical words — the console renders this string, never its
+    // own paraphrase of the marker.
+    busyText: describeBusy(busy, Date.now(), activity),
+    history: readPlanHistory(root, 5),
   };
 }
 
@@ -172,29 +361,106 @@ export function formatAge(ms) {
   return `${Math.round(ms / 3600000)}h ago`;
 }
 
+/** "12s" / "~3 min" — a plain duration (formatAge's sibling, no "ago"). */
+export function formatDuration(ms) {
+  if (!(ms >= 0)) return "";
+  if (ms < 120000) return `${Math.max(1, Math.round(ms / 1000))}s`;
+  return `~${Math.round(ms / 60000)} min`;
+}
+
+/** A done step's wall time from its own N1 stamps, or null pre-N1. */
+function stepDurationMs(s) {
+  const a = Date.parse(s.startedAt ?? "");
+  const b = Date.parse(s.doneAt ?? "");
+  return Number.isNaN(a) || Number.isNaN(b) ? null : Math.max(0, b - a);
+}
+
+/**
+ * The tier-3 corroboration as one phrase (N2): the lane's own narration when
+ * the marker carries it ("full check — unitTests (10/16) · 12s of ~3s,
+ * usually ~52s total"), the legacy phrase when it does not. "" when nothing
+ * is running. Shared by the text and HTML renderers so the observed tier
+ * speaks identically everywhere.
+ */
+export function describeBusy(busy, now = Date.now(), activity = null) {
+  if (!busy) return describeActivity(activity);
+  const lane = busy.lane;
+  if (lane) {
+    if (typeof lane === "object" && typeof lane.step === "string" && lane.step !== "") {
+      const pos = Number.isInteger(lane.index) && Number.isInteger(lane.total) ? ` (${lane.index}/${lane.total})` : "";
+      const started = Date.parse(lane.stepStartedAt ?? "");
+      const elapsed = Number.isNaN(started) ? null : Math.max(0, now - started);
+      const stepExpect = typeof lane.expectedStepMs === "number" && lane.expectedStepMs > 0 ? ` of ~${formatDuration(lane.expectedStepMs)}` : "";
+      const laneExpect =
+        typeof lane.expectedLaneMs === "number" && lane.expectedLaneMs > 0 ? `, usually ${formatDuration(lane.expectedLaneMs)} total` : "";
+      return `full check — ${lane.step}${pos}${elapsed !== null ? ` · ${formatDuration(elapsed)}${stepExpect}` : ""}${laneExpect}`;
+    }
+    return "the full check is running NOW";
+  }
+  if (busy.render) return "a preview render is in flight";
+  // Nothing mechanical is running — but the working tree may still be moving.
+  return describeActivity(activity);
+}
+
+/**
+ * The build stage's phrase (S3). Files written since the request, and how
+ * long ago the last one landed; a stall when the tree has stopped moving.
+ * "" when there is no request to measure from, or nothing has happened yet.
+ */
+export function describeActivity(activity) {
+  if (!activity) return "";
+  if (activity.filesChanged === 0) return "";
+  const n = activity.filesChanged;
+  const ago = activity.lastWriteAgoMs;
+  const files = `${n} file${n === 1 ? "" : "s"} written since the request`;
+  if (typeof ago === "number" && ago >= ACTIVITY_STALL_MS) return `${files} · stalled — nothing written for ${formatDuration(ago)}`;
+  return typeof ago === "number" ? `${files} · last ${formatDuration(ago)} ago` : files;
+}
+
 /**
  * The chain as one text block — the CLI's and the inject's rendering.
- * Numbered steps: done ✓, current ◉ (with tier-3 corroboration when a lane
- * or render is genuinely in flight), pending ○. "" when nothing is declared
- * AND no request is recorded (silence, never an empty frame).
+ * Numbered steps: done ✓ with wall time, current ◉ with elapsed, pending ○
+ * (N1); the tier-3 corroboration is prefixed "observed:" so the machine's
+ * word is visibly distinct from the agent's declaration (N3). "" when
+ * nothing is declared AND no request is recorded (silence, never an empty
+ * frame).
  */
 export function renderChain(chain) {
   if (!chain || (!chain.plan && !chain.request)) return "";
+  const now = Date.now();
   const lines = [];
   const title = chain.plan?.title ?? chain.request?.text ?? null;
   if (title) lines.push(`Request: ${title}`);
   if (chain.plan) {
     const p = chain.plan;
     const seq = p.steps
-      .map((s) => `${s.done ? "✓" : s.n === p.current ? "◉" : "○"} ${s.n}. ${s.label}`)
+      .map((s) => {
+        if (s.done) {
+          const d = stepDurationMs(s);
+          return `✓ ${s.n}. ${s.label}${d !== null ? ` (${formatDuration(d)})` : ""}`;
+        }
+        if (s.n === p.current) {
+          const a = Date.parse(s.startedAt ?? "");
+          return `◉ ${s.n}. ${s.label}${Number.isNaN(a) ? "" : ` · ${formatDuration(Math.max(0, now - a))} in`}`;
+        }
+        return `○ ${s.n}. ${s.label}`;
+      })
       .join("  →  ");
     lines.push(seq);
     const cur = p.steps.find((s) => s.n === p.current) ?? null;
-    const busy = chain.busy?.lane ? " · the full check is running NOW" : chain.busy?.render ? " · a preview render is in flight" : "";
+    const busyText = typeof chain.busyText === "string" ? chain.busyText : describeBusy(chain.busy, now, chain.activity ?? null);
+    const busy = busyText !== "" ? ` · observed: ${busyText}` : "";
     const age = chain.planAgeMs !== null ? ` · declared by the agent, updated ${formatAge(chain.planAgeMs)}` : "";
     lines.push(cur ? `now: step ${cur.n} of ${p.steps.length} — ${cur.label}${busy}${age}` : `chain complete${busy}${age}`);
   } else {
     lines.push("(no declared chain for this request yet — node qa/plan.mjs --set \"step | step | …\")");
+    // S3: an undeclared chain is no longer a still photo. The observed tier —
+    // a running lane, a render, or writes since the request — is printed even
+    // when the agent declared nothing, because it is the machine's word and
+    // needs no declaration to exist. This is the case that used to show nothing
+    // for forty minutes.
+    const observed = typeof chain.busyText === "string" ? chain.busyText : describeBusy(chain.busy, Date.now(), chain.activity ?? null);
+    if (observed !== "") lines.push(`observed: ${observed}`);
   }
   return lines.join("\n");
 }

package/template/qa/lib/receipt-validate.mjs

@@ -134,7 +134,10 @@ export function checkExecutionPlausibility(receipt, { minExecutedMs = DEFAULT_PO
   if (!steps || steps.length === 0) {
     return { ok: false, detail: "receipt lists no verify-lane steps — nothing was executed" };
   }
-  const executed = steps.filter((s) => s && s.verdict !== "SKIP");
+  // Executed = produced a verdict about the tree. SKIP did not try; ERROR
+  // tried and could not (a deadline, zero tests, a throw) — neither measured
+  // anything, so neither counts toward "this lane verified something".
+  const executed = steps.filter((s) => s && s.verdict !== "SKIP" && s.verdict !== "ERROR");
   if (executed.length === 0) {
     return { ok: false, detail: "every step in the receipt is a SKIP — the lane verified nothing" };
   }

package/template/qa/lib/spec-coverage.mjs

@@ -14,6 +14,20 @@ import path from "node:path";
 
 /** `- **HOME-01** — …` (live) or `- ~~**HOME-01**~~ — …` (withdrawn). */
 export const CLAUSE_LINE_RE = /^-\s+(~~)?\*\*([A-Z][A-Z0-9]*-\d{2,})\*\*/;
+// An OPTIONAL tier requirement on the clause line itself:
+//
+//   - **MOTION-13** [tier: device] — Given a cold start, When … Then …
+//
+// The clause declares what it takes to OBSERVE it, which is a property of the
+// promise, not of whatever test happened to cite it. Note this attaches to the
+// clause line, not to `[enforced: …]` — that tags docs/ARCHITECTURE.md prose and
+// is a different grammar entirely.
+const CLAUSE_TIER_RE = /\[tier:\s*(device|e2e)\]/i;
+/** Which citing tiers satisfy a declared requirement. */
+export const TIERS_SATISFYING = Object.freeze({
+  device: ["androidInstrumentedTest", "e2e"],
+  e2e: ["e2e"],
+});
 
 const TAG_LINE_RE = /^(?:\/\/|#)\s*SPEC:/;
 const TAG_IDS_RE = /SPEC:\s*([A-Z0-9,\s-]+)/;
@@ -45,7 +59,12 @@ export function scanSpecClauses(root) {
     for (const line of fs.readFileSync(abs, "utf8").split("\n")) {
       const m = line.match(CLAUSE_LINE_RE);
       if (!m) continue;
-      clauses.set(m[2], { file: path.relative(root, abs), withdrawn: Boolean(m[1]) });
+      const tierMatch = line.match(CLAUSE_TIER_RE);
+      clauses.set(m[2], {
+        file: path.relative(root, abs),
+        withdrawn: Boolean(m[1]),
+        requiredTier: tierMatch ? tierMatch[1].toLowerCase() : null,
+      });
     }
   }
   return clauses;
@@ -108,6 +127,9 @@ export function scanCitations(root) {
  * (commonTest/desktopTest) — behavior claims no device-tier evidence backs.
  * `summaryLine` is the one line the lane's specCoverage step (and any other
  * consumer) can print verbatim; null when nothing is desktop-only.
+ * `unmetTier` is the PRESCRIPTIVE half — clauses that declared `[tier: …]` and
+ * have no citation from a tier that could observe them. specCoverage FAILS on it:
+ * "instrument before you police" was the right first move, and this is the second.
  * @param {Map<string, {file: string, withdrawn: boolean}>} clauses from scanSpecClauses
  * @param {Array<{id: string, tier: string}>} tags from scanCitations
  * @returns {{tiersByClause: Record<string, string[]>, desktopOnly: string[], summaryLine: string|null}}
@@ -117,6 +139,17 @@ export function clauseTierCoverage(clauses, tags) {
   for (const t of tags) {
     (tiersByClause[t.id] ??= []).includes(t.tier) || tiersByClause[t.id].push(t.tier);
   }
+  // The gate input. A clause that DECLARED the tier it needs and has no citation
+  // from that tier is not covered — it is cited by tests structurally incapable
+  // of observing it, which is the exact hole `desktopOnly` below could only ever
+  // describe. MOTION-13 promised an animation "plays once per process start" and
+  // was cited by a desktop Compose test, a tier with no process lifecycle at all:
+  // the citation existed, the gate went green, and nothing ever observed the
+  // promise. Declared requirements are checked; undeclared clauses are unchanged.
+  const unmetTier = [...clauses.entries()]
+    .filter(([, c]) => !c.withdrawn && c.requiredTier)
+    .map(([id, c]) => ({ id, requiredTier: c.requiredTier, tiers: tiersByClause[id] ?? [], file: c.file }))
+    .filter((u) => !(TIERS_SATISFYING[u.requiredTier] ?? []).some((t) => u.tiers.includes(t)));
   const desktopOnly = [...clauses.entries()]
     .filter(([, c]) => !c.withdrawn)
     .map(([id]) => id)
@@ -127,5 +160,5 @@ export function clauseTierCoverage(clauses, tags) {
   const summaryLine = desktopOnly.length
     ? `${desktopOnly.length} clause${desktopOnly.length === 1 ? "" : "s"} cited only from desktop-tier tests (${desktopOnly.join(", ")})`
     : null;
-  return { tiersByClause, desktopOnly, summaryLine };
+  return { tiersByClause, desktopOnly, unmetTier, summaryLine };
 }

package/template/qa/lib/step-cache.mjs

@@ -138,7 +138,7 @@ export function loadStepCache(root) {
 export function lookupCachedPass(root, stepName, inputsHash) {
   const entry = loadStepCache(root).steps[stepName];
   if (!entry || typeof entry !== "object") return null;
-  if (entry.verdict !== "PASS") return null; // FAIL/SKIP are never reused
+  if (entry.verdict !== "PASS") return null; // FAIL/SKIP/ERROR are never reused
   if (typeof inputsHash !== "string" || entry.inputsHash !== inputsHash) return null;
   if (typeof entry.at !== "string") return null;
   return entry;

package/template/qa/lib/step-outcomes.mjs

@@ -0,0 +1,123 @@
+// step-outcomes.mjs — a step's VERDICT, separated from its INVOCATION.
+//
+// A step that ran zero tests knows nothing about behaviour and must not speak
+// as though it does. Observed 2026-09-02 (create-cmp-showcase): a concurrent
+// adb session collided with androidChecks, Gradle exited non-zero having
+// executed no tests, and the step reported "an on-device behavior claim is
+// broken. Fix the behavior, not the test." The identical task passed 8 tests
+// moments later. Believed, that sends the reader hunting a defect that does not
+// exist; disbelieved once, it teaches them to discount every future red from
+// the step — a gate that misattributes its own failures corrodes the gates that
+// are right.
+//
+// Pure, so the wording and the rule are testable without Gradle or a device.
+// (docs/proposals/evidence-economics.md C3, S4.)
+//
+// FOUR VERDICTS. PASS / FAIL / SKIP had no way to say "I could not run": a
+// step whose infrastructure broke reported a behaviour failure. ERROR is that
+// fourth word — zero tests executed, a deadline passed, a tool vanished, a
+// step threw. An ERROR never accuses the change, never counts as evidence
+// (evidence-level derives no rung over it; the plausibility check does not
+// count it as executed), is visibly distinct from FAIL (⊘, not ✗), and is
+// never silently retried. It still makes the lane FAIL — "could not check" is
+// not green. This is JUnit's error-vs-failure, Bazel's FAILED_TO_BUILD /
+// TIMEOUT vs FAILED, pytest's error vs failed — the distinction every mature
+// runner makes and this one did not.
+
+/**
+ * The androidChecks outcome from Gradle's exit and the JUnit summary.
+ *
+ * @param {{ok: boolean, out: string}} res the Gradle invocation
+ * @param {{tests: number, failures: number, errors: number}|null} summary parsed JUnit
+ *   results, or null when none were written
+ * @param {{gradlew?: string}} [opts]
+ * @returns {{verdict: "PASS"|"FAIL"|"ERROR", executed: boolean, reason?: string}}
+ */
+export function androidChecksOutcome(res, summary, { gradlew = "./gradlew" } = {}) {
+  const executed = Boolean(summary && summary.tests > 0);
+  if (res.ok) return { verdict: "PASS", executed };
+  const tail = String(res.out ?? "")
+    .split("\n")
+    .filter((l) => /FAILED|error:|failed/i.test(l))
+    .slice(0, 12)
+    .join("\n");
+  if (executed) {
+    return {
+      verdict: "FAIL",
+      executed,
+      reason:
+        `connectedDebugAndroidTest failed (${summary.failures + summary.errors} of ${summary.tests} tests) — ` +
+        `an on-device behavior claim is broken. Fix the behavior, not the test:\n${tail}`,
+    };
+  }
+  // ERROR, not FAIL: the step could not execute. A device tier that could not
+  // run is not evidence (the lane still FAILs), and going green would be the
+  // worse lie — but "your behaviour is broken" is withdrawn, and the receipt
+  // can tell a red that measured something from a red that measured nothing.
+  return {
+    verdict: "ERROR",
+    executed,
+    reason:
+      "connectedDebugAndroidTest DID NOT EXECUTE — the run reported no tests at all, so this step has observed " +
+      "nothing about your change and is not accusing it. Usual cause: another adb/Gradle session touching the same " +
+      "device (a manual `adb` command, a second lane, a running preview), or an install that never landed. " +
+      `Re-run this step alone with nothing else on the device before suspecting the code:\n  ${gradlew} :composeApp:connectedDebugAndroidTest --rerun\n${tail}`,
+  };
+}
+
+/** Thrown by the lane's subprocess helper when a step's deadline passes. */
+export class StepTimeout extends Error {
+  constructor(cmd, deadlineMs) {
+    super(`deadline of ${Math.round(deadlineMs / 60000)} min passed: ${cmd}`);
+    this.name = "StepTimeout";
+    this.cmd = cmd;
+    this.deadlineMs = deadlineMs;
+  }
+}
+
+/**
+ * Did a spawnSync result hit its deadline? Node reports ETIMEDOUT on
+ * `error.code` and the kill signal on `signal`; either alone is enough — an
+ * older Node sets only one of them.
+ * @param {{error?: {code?: string}, signal?: string|null}} res
+ * @returns {boolean}
+ */
+export function spawnTimedOut(res) {
+  if (!res) return false;
+  if (res.error && res.error.code === "ETIMEDOUT") return true;
+  return res.signal === "SIGTERM" && (res.status === null || res.status === undefined);
+}
+
+/**
+ * A step's own deadline, from the journal's last measured duration for it:
+ * three times what it usually takes, never under five minutes (a cold Gradle
+ * daemon is slow, not wedged), never over thirty (past that it IS wedged).
+ * Unknown steps get the ceiling — a first run is never cut short.
+ * @param {number|null|undefined} expectedMs
+ * @returns {number}
+ */
+export function stepDeadlineMs(expectedMs, { floorMs = 5 * 60_000, ceilingMs = 30 * 60_000 } = {}) {
+  if (!(expectedMs > 0)) return ceilingMs;
+  return Math.min(ceilingMs, Math.max(floorMs, Math.round(expectedMs * 3)));
+}
+
+/**
+ * The step result for a step that could not run — a deadline, or any throw
+ * out of the step's own body (which used to crash the whole lane; now it is
+ * one ERROR row and the lane keeps going, because the other steps' verdicts
+ * are still worth having).
+ * @param {string} name the step's display name
+ * @param {unknown} err
+ * @param {number} durationMs
+ * @returns {{name: string, verdict: "ERROR", reason: string, durationMs: number, details: {executed: false, kind: string}}}
+ */
+export function stepErrorResult(name, err, durationMs) {
+  const timeout = err instanceof StepTimeout;
+  const reason = timeout
+    ? `DID NOT COMPLETE — no result within its deadline (${Math.round(err.deadlineMs / 60000)} min). This step has observed nothing about your change and is not accusing it. ` +
+      `A wedged Gradle daemon or a device that stopped answering are the usual causes; check \`./gradlew --status\` and \`adb devices\`, then re-run the step alone.
+  ${err.cmd}`
+    : `DID NOT RUN — the step threw before producing a verdict: ${err && err.message ? err.message : String(err)}. ` +
+      `Nothing here is a claim about your change.`;
+  return { name, verdict: "ERROR", reason, durationMs, details: { executed: false, kind: timeout ? "deadline" : "threw" } };
+}