ccqa 1.57.0 → 1.57.2

package/dist/bin/ccqa.mjs

@@ -8200,52 +8200,15 @@ function exportedNames(source) {
 function reviewEmittedFiles(input) {
 	const locators = collectLocators(input.files);
 	const said = input.caseText.join("\n");
-	return [
-		...decidesNothing(input),
-		...unassertedPath(input),
-		...[...input.files].flatMap(([file, source]) => [
-			...toolIdentifiers(file, source),
-			...containerOfPageText(file, source),
-			...unjustifiedFirst(file, source),
-			...describeEcho(file, source),
-			...unaskedAssertions(file, source, locators, said),
-			...weakerTwin(file, source),
-			...unreached(file, source, input)
-		])
-	];
-}
-/**
-* A path with a placeholder segment in it — `/orders/{orderId}`. Writing one
-* in prose is a statement about the address, not about the page's contents:
-* there is no other reason to name the part that changes per run.
-*/
-const PLACEHOLDER_PATH = /\/[A-Za-z0-9_\-]+(?:\/[A-Za-z0-9_\-]+)*\/\{[A-Za-z0-9_]+\}/;
-/**
-* A case that says where the run must end up, and a test that never looks at
-* the address.
-*
-* The row being on screen is not the same claim as being on the screen that
-* shows it — a product that rendered the row without navigating passes, and
-* that is the failure the expectation was written to catch. Scoped to
-* placeholder paths because those are unambiguous; a path mentioned as scenery
-* ("on /settings, the button is shown") is not a claim about the address.
-*
-* Calibrated against a small set of hand-written case definitions: a few
-* name a placeholder path in their expectations, most of those already
-* assert on the URL, and the one that does not is the defect this was
-* written for.
-*/
-function unassertedPath(input) {
-	const source = input.files.get(input.testPath);
-	if (source === void 0) return [];
-	const stated = input.caseText.find((text) => PLACEHOLDER_PATH.test(text));
-	if (stated === void 0 || /\btoHaveURL\b/.test(source)) return [];
-	return [{
-		file: input.testPath,
-		line: 1,
-		rule: "unasserted-path",
-		message: `the case says where the run ends up — "${PLACEHOLDER_PATH.exec(stated)?.[0]}" — and nothing here looks at the address. What the screen shows can be right while the screen is wrong. Assert the URL where the case says it changes`
-	}];
+	return [...decidesNothing(input), ...[...input.files].flatMap(([file, source]) => [
+		...toolIdentifiers(file, source),
+		...containerOfPageText(file, source),
+		...unjustifiedFirst(file, source),
+		...describeEcho(file, source),
+		...unaskedAssertions(file, source, locators, said),
+		...weakerTwin(file, source),
+		...unreached(file, source, input)
+	])];
 }
 /** `readonly name =`, `name(...): Locator {`, and `export … name`. */
 const DECLARED = /(?:readonly\s+([A-Za-z0-9_]+)\s*=|\b([A-Za-z0-9_]+)\([^)]*\)\s*:\s*Locator\b|export\s+(?:async\s+)?(?:const|let|function|class)\s+([A-Za-z0-9_]+))/g;
@@ -13914,6 +13877,7 @@ function parseTraceEvents(jsonl) {
 	const frames = [];
 	const steps = [];
 	const pendingByCallId = /* @__PURE__ */ new Map();
+	const wallOffsets = /* @__PURE__ */ new Map();
 	let maxTimestamp = -Infinity;
 	for (const line of jsonl.split("\n")) {
 		if (line.trim() === "") continue;
@@ -13937,8 +13901,13 @@ function parseTraceEvents(jsonl) {
 		if (type === "screencast-frame") {
 			if (typeof event.sha1 === "string" && typeof event.timestamp === "number") frames.push({
 				sha1: event.sha1,
-				timestamp: event.timestamp
+				timestamp: event.timestamp,
+				...typeof event.frameSwapWallTime === "number" ? { swapWallTime: event.frameSwapWallTime } : {}
 			});
+		} else if (type === "context-options") {
+			if (typeof event.wallTime === "number" && typeof event.monotonicTime === "number") {
+				if (!wallOffsets.has(event.origin)) wallOffsets.set(event.origin, event.wallTime - event.monotonicTime);
+			}
 		} else if (type === "action") {
 			if (typeof event.callId === "string" && typeof event.startTime === "number" && typeof event.endTime === "number") steps.push({
 				title: titleOf$1(event),
@@ -13969,10 +13938,16 @@ function parseTraceEvents(jsonl) {
 		...pending,
 		endTime: maxTimestamp
 	});
-	frames.sort((a, b) => a.timestamp - b.timestamp);
+	const offset = wallOffsets.get("testRunner") ?? wallOffsets.values().next().value;
+	const painted = offset !== void 0 && frames.every((f) => f.swapWallTime !== void 0);
+	const placed = frames.map(({ sha1, timestamp, swapWallTime }) => ({
+		sha1,
+		timestamp: painted ? swapWallTime - offset : timestamp
+	}));
+	placed.sort((a, b) => a.timestamp - b.timestamp);
 	steps.sort((a, b) => a.startTime - b.startTime);
 	return {
-		frames,
+		frames: placed,
 		steps
 	};
 }
@@ -13992,13 +13967,22 @@ function titleOf$1(event) {
 	return "";
 }
 /**
-* The frame on screen at `time`: the last one captured at or before it, or —
-* when the step began before any frame was captured — the first frame after.
-* Null when the trace holds no frames at all. `frames` comes from
-* {@link parseTraceEvents}, which sorts them by timestamp.
+* One paint interval at 60fps. An assertion passes on the DOM, so a step can
+* end before the frame showing that DOM is painted — at most one interval later.
+*/
+const ONE_FRAME_MS = 17;
+/**
+* The frame showing the page as it was at `time`: the last one painted within
+* a frame interval of it, or — when the step began before any frame was
+* captured — the first frame after. Null when the trace holds no frames at
+* all. `frames` comes from {@link parseTraceEvents}, sorted by paint time.
+*
+* A step's start takes the same allowance as its end: it is the same boundary
+* as the previous step's end, and would otherwise show that step unfinished.
 */
 function frameAt(frames, time) {
-	return frames.findLast((f) => f.timestamp <= time) ?? frames[0] ?? null;
+	const painted = time + ONE_FRAME_MS;
+	return frames.findLast((f) => f.timestamp <= painted) ?? frames[0] ?? null;
 }
 //#endregion
 //#region src/targets/playwright/trace-capture.ts
@@ -27368,7 +27352,7 @@ const evidenceCommand = addLanguageOption(new Command("evidence").argument("<cas
 		anchors = await findSourceAnchors(sourceNeedles([...recording.actions, ...recording.cleanup ?? []]), roots);
 	}
 	const out = resolve(cwd, opts.out ?? join(testCase.ref.dir, "evidence.md"));
-	await writeFile(out, scrubEnvValues(renderEvidence({
+	const markdown = renderEvidence({
 		testCase,
 		recording,
 		test: {
@@ -27381,7 +27365,9 @@ const evidenceCommand = addLanguageOption(new Command("evidence").argument("<cas
 		...resolved.targetConfig.allowExpectInCleanup ? {} : { cleanupUnchecked: testCase.cleanupExpectations },
 		...review ? { review } : {},
 		...anchors ? { anchors } : {}
-	}), buildProseEnvScrubMap([], [])), "utf8");
+	});
+	await mkdir(dirname(out), { recursive: true });
+	await writeFile(out, scrubEnvValues(markdown, buildProseEnvScrubMap([], [])), "utf8");
 	process.stdout.write(`${relative(cwd, out) || out}\n`);
 	hint("paste it into the pull request, or attach it — ccqa writes the table, not the body");
 }));

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

@@ -35,9 +35,9 @@ declare const RunSchema: z.ZodObject<{
     running: "running";
   }>;
   kind: z.ZodDefault<z.ZodEnum<{
+    record: "record";
     run: "run";
     drift: "drift";
-    record: "record";
   }>>;
   drift: z.ZodDefault<z.ZodNullable<z.ZodObject<{
     specs: z.ZodNumber;
@@ -831,9 +831,9 @@ type ReportSpecResult = z.infer<typeof ReportSpecResultSchema>;
 declare const RunReportDataSchema: z.ZodObject<{
   schemaVersion: z.ZodLiteral<1>;
   kind: z.ZodDefault<z.ZodEnum<{
+    record: "record";
     run: "run";
     drift: "drift";
-    record: "record";
   }>>;
   createdAt: z.ZodString;
   runId: z.ZodNullable<z.ZodString>;

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

@@ -35,9 +35,9 @@ declare const RunSchema: z.ZodObject<{
     running: "running";
   }>;
   kind: z.ZodDefault<z.ZodEnum<{
+    record: "record";
     run: "run";
     drift: "drift";
-    record: "record";
   }>>;
   drift: z.ZodDefault<z.ZodNullable<z.ZodObject<{
     specs: z.ZodNumber;
@@ -831,9 +831,9 @@ type ReportSpecResult = z.infer<typeof ReportSpecResultSchema>;
 declare const RunReportDataSchema: z.ZodObject<{
   schemaVersion: z.ZodLiteral<1>;
   kind: z.ZodDefault<z.ZodEnum<{
+    record: "record";
     run: "run";
     drift: "drift";
-    record: "record";
   }>>;
   createdAt: z.ZodString;
   runId: z.ZodNullable<z.ZodString>;

package/dist/package.json

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