ccqa 1.57.3 → 1.58.1

package/dist/bin/ccqa.mjs

@@ -15464,7 +15464,7 @@ const JUDGE_CALL = "judgeByLlm";
 * would notice.
 */
 function headerPreserveRule(header, titleSuffix) {
-	const parts = [header ? `the comment block at the top of the draft, verbatim` : "", titleSuffix ? `the \`${titleSuffix.trim()}\` suffix on the test's name` : ""].filter(Boolean);
+	const parts = [header ? `the comment block right below the draft's imports, verbatim` : "", titleSuffix ? `the \`${titleSuffix.trim()}\` suffix on the test's name` : ""].filter(Boolean);
 	return `**Keep ${parts.join(" and ")}.** Written from the case's own record, not decided per test; do not reword, move, or drop ${parts.length > 1 ? "either" : "it"}.`;
 }
 /**
@@ -15554,11 +15554,11 @@ function emitPlaywrightDraft(input) {
 		] : []
 	];
 	const source = [
-		...input.header ? [input.header.trimEnd(), ""] : [],
 		`import { test, expect } from "@playwright/test";`,
 		...judgements.length > 0 ? [`import { ${JUDGE_CALL} } from ${j(JUDGE_MODULE)};`] : [],
 		...runId ? [runId.import] : [],
 		"",
+		...input.header ? [input.header.trimEnd(), ""] : [],
 		...declaration,
 		""
 	].join("\n");
@@ -16208,7 +16208,7 @@ function injectedCallGaps(corpus, { markers, judgements, header, titleSuffix })
 		warnings.push(`step ${stepId}: the generated test no longer opens that step with the \`test.step\` title the draft wrote. The evidence table, the run's screenshots and the spec review read it back to attribute assertions, so a reshaped one makes all three report the step as deciding nothing. Keep the title unchanged.`);
 	}
 	const firstHeaderLine = stamped.header.split("\n")[0]?.trim() ?? "";
-	if (firstHeaderLine && !corpus.includes(firstHeaderLine)) warnings.push("the generated test no longer opens with the configured header — a rewrite pass dropped it, so the file does not say which case it came from");
+	if (firstHeaderLine && !corpus.includes(firstHeaderLine)) warnings.push("the generated test no longer carries the configured header — a rewrite pass dropped it, so the file does not say which case it came from");
 	if (stamped.titleSuffix && !corpus.includes(stamped.titleSuffix.trim())) warnings.push(`the generated test's name no longer ends with "${stamped.titleSuffix.trim()}" — a rewrite pass dropped the tag, and whatever selects tests by it will skip this one`);
 	for (const { step } of judgements) {
 		if (judgeCall(step).pattern.test(corpus)) continue;
@@ -17910,8 +17910,9 @@ const AttestationSchema = z.object({
 const AttestationsSchema = z.object({ specs: z.record(z.string(), AttestationSchema).default({}) });
 /**
 * Why an attestation stopped covering the spec. One reason is named even when
-* several hold, in the order the checks run (coverage, then the spec's own
-* edits, then a later red run) — enough for a reader to see what to verify
+* several hold, in the order the checks run (a later red that is still the
+* latest run, then coverage, then the spec's own edits, then a later red since
+* replaced by a green) — enough for a reader to see what to verify
 * before attesting again.
 */
 const AttestationLapseSchema = z.enum([
@@ -31915,7 +31916,7 @@ 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 manualState = readAttestation(attestations.specs[spec.key], spec, coords, 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);
@@ -31946,18 +31947,24 @@ function computeRerun(input) {
 	return out;
 }
 /**
-* Does the attestation still speak for what is deployed? Checked in the order
-* the lapse enum documents: deploy coverage first (a sha the log cannot place
-* reads as reached, ADR-0014, with the hole kept as an annotation), then the
-* spec's own edits — compared against when the person looked, because they
-* read the spec as it stood that moment, which `specMovedSince` covers via a
-* null baseline sha — then a red run recorded after them, which is newer
-* information than their word. The null-sha case is the profile that had no
-* deploy log when they checked: their word covers exactly as long as that
-* stays true.
+* Does the attestation still speak for what is deployed? A red run recorded
+* after the person looked and still the latest run is checked first, since it
+* alone keeps the spec parked (see the caller). Then deploy coverage (a sha the
+* log cannot place reads as reached, ADR-0014, with the hole kept as an
+* annotation), then the spec's own edits — compared against when the person
+* looked, because they read the spec as it stood that moment, which
+* `specMovedSince` covers via a null baseline sha — then a later red a green
+* run has since replaced. The null-sha case is the profile that had no deploy
+* log when they checked: their word covers exactly as long as that stays true.
 */
-function readAttestation(attest, spec, lastRed, range, log, deployTimes) {
+function readAttestation(attest, spec, { lastRun, lastRed }, range, log, deployTimes) {
 	if (!attest) return null;
+	const newerRed = lastRed !== null && lastRed.at > attest.at;
+	if (newerRed && lastRed.runId === lastRun?.runId) return {
+		kind: "lapsed",
+		attest,
+		because: "newerRed"
+	};
 	const coverage = attest.deployedSha === null ? log.entries.length === 0 ? { kind: "current" } : {
 		kind: "unanswerable",
 		reason: "unknownDeployedSha"
@@ -31979,7 +31986,7 @@ function readAttestation(attest, spec, lastRed, range, log, deployTimes) {
 		attest,
 		because: "specEdited"
 	};
-	if (lastRed !== null && lastRed.at > attest.at) return {
+	if (newerRed) return {
 		kind: "lapsed",
 		attest,
 		because: "newerRed"

package/dist/package.json

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