ccqa 1.47.0 → 1.47.1

package/dist/bin/ccqa.mjs

@@ -13535,7 +13535,27 @@ const STEP_EVIDENCE_BEFORE = "ccqaStepBefore";
 const STEP_EVIDENCE_AFTER = "ccqaStepAfter";
 /** The exact boundary call for one step, as emitted and as the gate greps for it. */
 function stepEvidenceCall(fn, marker) {
-	return `await ${fn}(page, ${j(marker.stepId)}, ${j(marker.source)});`;
+	return injectedCall(fn, [j(marker.stepId), j(marker.source)]);
+}
+/**
+* A receiver expression: anything up to the argument's comma, allowing one
+* level of parentheses so `await ctx.newPage()` and `page.context().pages()[1]`
+* — how a rewrite names a tab a click opened — count as receivers.
+*/
+const RECEIVER = String.raw`(?:[^,()]|\([^()]*\))+`;
+/**
+* The call as emitted, plus the pattern that accepts it back on any page. The
+* receiver is the one part a rewrite is right to change: a click that opens a
+* new tab has to act on that tab. Everything else is required verbatim, the
+* `await` and `;` included — an unawaited capture races the end of the test,
+* and a mention in a comment is not a call.
+*/
+function injectedCall(name, args) {
+	const tail = args.map(escapeRegExp).join(String.raw`\s*,\s*`);
+	return {
+		code: `await ${name}(page, ${args.join(", ")});`,
+		pattern: new RegExp(String.raw`await\s+${name}\s*\(\s*${RECEIVER}\s*,\s*${tail}\s*,?\s*\)\s*;`)
+	};
 }
 /**
 * The "preserve the step-evidence calls" rule the library-rewrite prompt must
@@ -13545,7 +13565,7 @@ function stepEvidenceCall(fn, marker) {
 * engine, and the prompt then omits the rule entirely.
 */
 function stepEvidencePreserveRule() {
-	return `**Keep the \`${STEP_EVIDENCE_MODULE}\` calls.** The draft's \`await ${STEP_EVIDENCE_BEFORE}(page, ...)\` / \`await ${STEP_EVIDENCE_AFTER}(page, ...)\` lines are load-bearing: ccqa run reads the per-step screenshots they capture. Keep both calls for every step, in place around that step's actions, with their exact \`(page, "<stepId>", "<source>")\` arguments — and keep the import. If you move a step's actions into a page-object method, leave these two calls in the test body around the call to that method; do NOT move them inside the page object. Never wrap a step in a closure to hold them.`;
+	return `**Keep the \`${STEP_EVIDENCE_MODULE}\` calls.** The draft's \`await ${STEP_EVIDENCE_BEFORE}(page, ...)\` / \`await ${STEP_EVIDENCE_AFTER}(page, ...)\` lines are load-bearing: ccqa run reads the per-step screenshots they capture. Keep both calls for every step, in place around that step's actions, with their exact \`"<stepId>", "<source>"\` arguments — and keep the import. Pass a different page only when the step genuinely acts on one (a click that opens a new tab). If you move a step's actions into a page-object method, leave these two calls in the test body around the call to that method; do NOT move them inside the page object. Never wrap a step in a closure to hold them.`;
 }
 /**
 * Told to the rewrite, because the alternative failure is silent: a claim
@@ -13564,22 +13584,22 @@ function emitPlaywrightDraft(input) {
 	const flushJudgements = (afterActionIndex) => {
 		for (const { step } of judgements.filter((j) => j.afterActionIndex === afterActionIndex)) {
 			if (openMarker) {
-				lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker));
+				lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker).code);
 				openMarker = null;
 			}
 			if (lines.length > 0) lines.push("");
 			lines.push(`// step: ${step.id} [${step.source}]`);
-			lines.push(judgeCall(step));
+			lines.push(judgeCall(step).code);
 		}
 	};
 	flushJudgements(-1);
 	for (let i = 0; i < actions.length; i++) {
 		const marker = markerByIndex.get(i);
 		if (marker) {
-			if (openMarker) lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker));
+			if (openMarker) lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker).code);
 			if (lines.length > 0) lines.push("");
 			lines.push(`// step: ${marker.stepId} [${marker.source}]`);
-			lines.push(stepEvidenceCall(STEP_EVIDENCE_BEFORE, marker));
+			lines.push(stepEvidenceCall(STEP_EVIDENCE_BEFORE, marker).code);
 			openMarker = marker;
 		}
 		const action = actions[i];
@@ -13591,7 +13611,7 @@ function emitPlaywrightDraft(input) {
 		}
 		flushJudgements(i);
 	}
-	if (openMarker) lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker));
+	if (openMarker) lines.push(stepEvidenceCall(STEP_EVIDENCE_AFTER, openMarker).code);
 	if (judgements.length > 0) lines.unshift("test.slow();", "");
 	const body = lines.map((l) => l === "" ? "" : `  ${l}`).join("\n");
 	return [
@@ -13779,8 +13799,9 @@ const j = (s) => JSON.stringify(s);
 const jExpr = (s) => envRefsToJsExpression(s);
 /** One claim, asserted through the judge. Exported so the generation gate can require it back. */
 function judgeCall(step) {
-	const from = step.from === void 0 ? "" : `, ${jExpr(step.from)}`;
-	return `await ${JUDGE_CALL}(page, ${bracedRefsToJsExpression(step.judgeByLlm.trim())}${from});`;
+	const args = [bracedRefsToJsExpression(step.judgeByLlm.trim())];
+	if (step.from !== void 0) args.push(jExpr(step.from));
+	return injectedCall(JUDGE_CALL, args);
 }
 //#endregion
 //#region src/targets/playwright/browser-server.ts
@@ -14031,12 +14052,12 @@ async function missingInjectedCalls(result, markers, judgements) {
 	const corpus = (await Promise.all(result.files.filter((f) => f.kind === "test").map((f) => readFile(f.path, "utf8").catch(() => "")))).join("\n");
 	const warnings = [];
 	for (const m of markers) {
-		const hasBefore = corpus.includes(stepEvidenceCall(STEP_EVIDENCE_BEFORE, m));
-		const hasAfter = corpus.includes(stepEvidenceCall(STEP_EVIDENCE_AFTER, m));
+		const hasBefore = stepEvidenceCall(STEP_EVIDENCE_BEFORE, m).pattern.test(corpus);
+		const hasAfter = stepEvidenceCall(STEP_EVIDENCE_AFTER, m).pattern.test(corpus);
 		if (!hasBefore || !hasAfter) warnings.push(`step ${m.stepId}: generated test is missing its ${STEP_EVIDENCE_BEFORE}/${STEP_EVIDENCE_AFTER} call(s) — that step will have no report screenshots. A rewrite pass must not drop them.`);
 	}
 	for (const { step } of judgements) {
-		if (corpus.includes(judgeCall(step))) continue;
+		if (judgeCall(step).pattern.test(corpus)) continue;
 		warnings.push(`step ${step.id}: generated test is missing its ${JUDGE_CALL} call — that claim is never decided and the spec passes without testing it. A rewrite pass must not drop or reword it.`);
 	}
 	return warnings;

package/dist/package.json

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