ccqa 1.21.0 → 1.22.0

package/dist/bin/ccqa.mjs

@@ -6842,10 +6842,26 @@ const SpecLedgerEntrySchema = z.object({
 	deployedSha: z.string().nullable().optional(),
 	deployedShaAmbiguous: z.boolean().optional()
 });
+/**
+* A red-bucket entry: where the failure happened, plus what it was. The cause
+* is copied off the run report's `analysis` so a reader learns why a spec is
+* red without fetching the report of every red spec.
+*
+* Only the red bucket carries it. A pass has no cause, so putting these on
+* `SpecLedgerEntry` would invite writing them where they cannot exist.
+*
+* Both fields are optional and mean the same thing by their absence — nothing
+* is on record. Failure analysis is opt-in (`--on-fail-explain`), and entries
+* written before this release have neither field.
+*/
+const SpecRedLedgerEntrySchema = SpecLedgerEntrySchema.extend({
+	label: PredictedLabelSchema.optional(),
+	headline: z.string().optional()
+});
 z.object({
 	green: z.record(z.string(), SpecLedgerEntrySchema).default({}),
 	run: z.record(z.string(), SpecLedgerEntrySchema).default({}),
-	red: z.record(z.string(), SpecLedgerEntrySchema).default({})
+	red: z.record(z.string(), SpecRedLedgerEntrySchema).default({})
 });
 /** One deploy, as the consumer's deploy job reported it (ADR-0010). */
 const DeployEntrySchema = z.object({
@@ -7025,7 +7041,7 @@ const SpecRerunSchema = z.object({
 	heldBy: SpecLockSchema.nullable(),
 	lastRun: SpecLedgerEntrySchema.nullable(),
 	lastGreen: SpecLedgerEntrySchema.nullable(),
-	lastRed: SpecLedgerEntrySchema.nullable(),
+	lastRed: SpecRedLedgerEntrySchema.nullable(),
 	touchedBy: z.array(z.string()).optional(),
 	touchedByDeploy: DeployRefSchema.nullable().optional()
 });
@@ -7075,7 +7091,7 @@ z.object({
 z.object({
 	entries: z.record(z.string(), SpecLedgerEntrySchema),
 	lastRun: z.record(z.string(), SpecLedgerEntrySchema).default({}),
-	lastRed: z.record(z.string(), SpecLedgerEntrySchema).default({})
+	lastRed: z.record(z.string(), SpecRedLedgerEntrySchema).default({})
 });
 /**
 * One spec's last drift audit, as recorded by `ccqa audit --report-to-hub`. Unlike the
@@ -16755,12 +16771,17 @@ function toLedger(raw) {
 }
 /** Fold `from` into `into` in place, per bucket and key, newest `at` winning. */
 function mergeLedgerInto(into, from) {
-	for (const name of BUCKET_NAMES) for (const [key, entry] of Object.entries(from[name])) {
-		const prev = into[name][key];
-		if (!prev || prev.at <= entry.at) into[name][key] = entry;
-	}
+	mergeBucket(into.green, from.green);
+	mergeBucket(into.run, from.run);
+	mergeBucket(into.red, from.red);
 	return into;
 }
+function mergeBucket(into, from) {
+	for (const [key, entry] of Object.entries(from)) {
+		const prev = into[key];
+		if (!prev || prev.at <= entry.at) into[key] = entry;
+	}
+}
 //#endregion
 //#region src/hub/api/validate.ts
 /**
@@ -16971,7 +16992,7 @@ async function updateSpecLedger(storage, run, results) {
 		const key = `${row.feature}/${row.spec}`;
 		ledger.run[key] = entry;
 		if (row.status === "passed") ledger.green[key] = entry;
-		else ledger.red[key] = entry;
+		else ledger.red[key] = redEntry(entry, row);
 	}
 	if (Object.keys(ledger.run).length === 0) return;
 	try {
@@ -16981,6 +17002,23 @@ async function updateSpecLedger(storage, run, results) {
 	}
 }
 /**
+* The red bucket's entry: the coordinate every bucket carries, plus what the
+* failure analysis concluded, so a reader of the ledger learns why a spec is
+* red without fetching its report.
+*
+* A field with nothing behind it is left out rather than written empty:
+* `analysis` is null when the run did not ask for one (`--on-fail-explain` is
+* opt-in), and a model that answered with no headline leaves `headline` "".
+*/
+function redEntry(entry, row) {
+	if (!row.analysis) return entry;
+	return {
+		...entry,
+		label: row.analysis.label,
+		...row.analysis.headline ? { headline: row.analysis.headline } : {}
+	};
+}
+/**
 * Advance the drift ledger from a terminal `kind: "drift"` run: each row's
 * `analysis` (the diagnosis `driftResultsToReport` put there) becomes that
 * spec's newest audit entry. No profile — drift asks whether the spec still
@@ -22205,6 +22243,17 @@ const CLIENT_JS = `
     badge.appendChild(document.createTextNode(" " + t("perspectives.run.state." + runState)));
     td.appendChild(badge);
 
+    // What the failure was, as the run's analysis called it — the same place
+    // the audit column names its drift label. Absent on a red entry the run
+    // never analyzed, and on entries written before the ledger carried it.
+    if (runState === "failed" && rr.lastRed && rr.lastRed.label) {
+      var cause = el("span", "cellsub", labelText(rr.lastRed.label));
+      // The one-line conclusion, for a pointer. The detail panel shows it in
+      // full, so nothing here depends on finding it.
+      if (rr.lastRed.headline) cause.title = rr.lastRed.headline;
+      td.appendChild(cause);
+    }
+
     // The sub-line is the coordinate of the result being reported — the same
     // "when is this from" evidence the audit column carries above, so the two
     // axes read as siblings. Why the currency matters belongs to the verdict
@@ -22423,6 +22472,21 @@ const CLIENT_JS = `
     return wrap;
   }
 
+  // The failure: which run it was, then what the analysis concluded. The
+  // headline is model output, already localized server-side, so it is shown as
+  // written. A run made without failure analysis carries neither field, and the
+  // row is then the coordinate alone — what it has always been.
+  function rerunFailureValue(entry) {
+    var wrap = el("div");
+    wrap.appendChild(ledgerLine(entry));
+    if (entry.label) {
+      var line = el("div", "d-prose", labelText(entry.label));
+      if (entry.headline) line.appendChild(document.createTextNode(" · " + entry.headline));
+      wrap.appendChild(line);
+    }
+    return wrap;
+  }
+
   // Detail row: a definition list of the case's fields plus the note editor.
   // Built with createElement/textContent throughout — every field here is
   // API-derived, so none of it may go through innerHTML.
@@ -22456,7 +22520,7 @@ const CLIENT_JS = `
     var rr = ledgerEntryFor(perspState.rerun, feature, spec);
     if (rr) {
       row(rerunEvidenceLabelKey(rr), rerunEvidenceValue(rr));
-      if (rerunHasFailure(rr)) row("perspectives.d.lastRed", ledgerLine(rr.lastRed));
+      if (rerunHasFailure(rr)) row("perspectives.d.lastRed", rerunFailureValue(rr.lastRed));
     }
     frag.appendChild(dl);
 

package/dist/hub-client/index.d.mts

@@ -314,6 +314,14 @@ declare const RerunReportSchema: z.ZodObject<{
       at: z.ZodString;
       deployedSha: z.ZodOptional<z.ZodNullable<z.ZodString>>;
       deployedShaAmbiguous: z.ZodOptional<z.ZodBoolean>;
+      label: z.ZodOptional<z.ZodEnum<{
+        TEST_DRIFT: "TEST_DRIFT";
+        SPEC_CHANGE: "SPEC_CHANGE";
+        PRODUCT_BUG: "PRODUCT_BUG";
+        ENVIRONMENT: "ENVIRONMENT";
+        UNKNOWN: "UNKNOWN";
+      }>>;
+      headline: z.ZodOptional<z.ZodString>;
     }, z.core.$strip>>;
     touchedBy: z.ZodOptional<z.ZodArray<z.ZodString>>;
     touchedByDeploy: z.ZodOptional<z.ZodNullable<z.ZodObject<{

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ccqa",
-  "version": "1.21.0",
+  "version": "1.22.0",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ccqa",
-  "version": "1.21.0",
+  "version": "1.22.0",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {