ccqa 1.35.1 → 1.36.0

package/dist/bin/ccqa.mjs

@@ -19419,9 +19419,11 @@ function computeRerun(input) {
 		const runMoved = specMovedSince(spec.changedAt, coords.lastRun?.deployedSha ?? null, coords.lastRun?.at ?? "", deployTimes);
 		if (auditMoved && audit.audit !== "due") audit = { audit: "due" };
 		if (runMoved && execution.execution === "passed") execution = { execution: "stale" };
+		const manualState = readAttestation(attestations.specs[spec.key], spec, coords.lastRed, range, log, deployTimes);
+		const lapsedOverRed = manualState?.kind === "lapsed" && manualState.because !== "newerRed";
+		if (execution.execution === "failed" && lapsedOverRed) execution = { execution: "stale" };
 		const held = heldBy(locks, spec.key, now);
 		let verdict = decide(audit.audit, execution.execution, held);
-		const manualState = readAttestation(attestations.specs[spec.key], spec, coords.lastRed, range, log, deployTimes);
 		if (manualState?.kind === "covers" && !held && verdict !== "verified") verdict = "manuallyVerified";
 		out[spec.key] = {
 			verdict,
@@ -19530,8 +19532,10 @@ function auditState(drift, key, range) {
 * buckets advance from the same terminal-run trigger, so the run that wrote
 * `run` wrote exactly one of `green` or `red`.
 *
-* `failed` outranks `stale`: a red result is current information whatever has
-* deployed since, and re-running it teaches nothing until someone repairs it.
+* `failed` outranks `stale` here: a red result is current information whatever
+* has deployed since, and re-running it teaches nothing until someone repairs
+* it. Its caller retires a red a lapsed attestation already answered
+* (ADR-0020) — this function does not know about attestations.
 */
 function executionState(coords, since) {
 	const { lastRun, lastRed } = coords;
@@ -24015,6 +24019,13 @@ const CLIENT_JS = `
     return null;
   }
 
+  // Did the last run end red? Read off the ledger the same way the execution
+  // axis reads it, because a red a lapsed attestation retired ships as stale
+  // and the sites that need the failure itself must still find it (ADR-0020).
+  function lastFailed(rr) {
+    return !!(rr && rr.lastRun && rr.lastRed && rr.lastRed.runId === rr.lastRun.runId);
+  }
+
   // The execution axis as one state. "What did it say last time" and "is that
   // still true" are two faces of one question, so the row answers it once and
   // the detail panel keeps the coordinates.
@@ -24545,7 +24556,9 @@ const CLIENT_JS = `
     }
     // Only once the audit has actually cleared the spec: while it is still
     // due, nobody knows yet whether the environment was the only thing wrong.
-    if (rr.audit === "clean" && rr.execution === "failed" && rr.lastRed && rr.lastRed.label === "ENVIRONMENT") {
+    // Read off the ledger, not the axis: an environment red is demoted to
+    // stale so it re-runs (ADR-0020), and the offer has to survive that.
+    if (rr.audit === "clean" && lastFailed(rr) && rr.lastRed.label === "ENVIRONMENT") {
       return buildOverrideOffer("environment", feature, spec);
     }
     return null;
@@ -24585,7 +24598,7 @@ const CLIENT_JS = `
   // report loads — and the whole content if it never arrives.
   function reasonFindingSource(rr, driftEntry) {
     if (auditOpen(rr) && driftEntry && driftEntry.runId && driftEntry.label) return driftEntry;
-    if (rr.execution === "failed" && rr.lastRed && rr.lastRed.runId && rr.lastRed.label) return rr.lastRed;
+    if (lastFailed(rr) && rr.lastRed.runId && rr.lastRed.label) return rr.lastRed;
     return null;
   }
 

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ccqa",
-  "version": "1.35.1",
+  "version": "1.36.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.35.1",
+  "version": "1.36.0",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {