ccqa 1.58.0 → 1.58.2

package/dist/bin/ccqa.mjs

@@ -8710,16 +8710,14 @@ function rulesReview(guides) {
 				""
 			])
 		],
-		answerShape: "{ \"findings\": [ { \"stepId\": \"step-05\", \"problem\": \"…\" } ], \"ruleViolations\": [ { \"file\": \"…\", \"guide\": \"…\", \"rule\": \"…\", \"code\": \"…\", \"severity\": \"blocking\" } ] }",
+		answerShape: "{ \"findings\": [ { \"stepId\": \"step-05\", \"problem\": \"…\" } ], \"ruleViolations\": [ { \"file\": \"…\", \"guide\": \"…\", \"rule\": \"…\", \"code\": \"…\" } ] }",
 		answerNote: [
 			"",
 			"In `ruleViolations`: `file` is the file's path in this repository, and `code`",
 			"the lines that break the rule. `guide` and `rule` are where the rule comes",
 			"from and what it says — the document's path and its own words when you can",
 			"quote one; otherwise the path of a file you read that shows the convention,",
-			"and what you counted (\"N of M files under <dir> do X\"). `severity` is",
-			"\"blocking\" when you would hold the change until it is fixed, and \"advisory\"",
-			"when it is worth saying but you would approve anyway. An empty",
+			"and what you counted (\"N of M files under <dir> do X\"). An empty",
 			"`ruleViolations` means every file follows every rule you could quote or",
 			"count — write the key either way."
 		]
@@ -9150,8 +9148,7 @@ const ViolationSchema = z.object({
 	file: z.string(),
 	guide: z.string(),
 	rule: z.string(),
-	code: z.string(),
-	severity: z.enum(["blocking", "advisory"]).catch("blocking")
+	code: z.string()
 });
 const ViolationsSchema = z.object({ ruleViolations: z.array(z.unknown()) });
 /**
@@ -9239,14 +9236,11 @@ function formatFinding(finding, labels = evidenceLabels()) {
 }
 /**
 * The warning a rule violation becomes: the file, the rule in the words it was
-* judged by, and where that came from. An advisory one says so — a reader must
-* be able to tell a line that was worth a fix round from one that was not. The
-* offending code is left out: it is what the fix pass needs, not what a reader
-* scanning a log does.
+* judged by, and where that came from. The offending code is left out: it is
+* what the fix pass needs, not what a reader scanning a log does.
 */
 function formatViolation(violation) {
-	const advisory = violation.severity === "advisory" ? ", advisory" : "";
-	return `${violation.file}: ${violation.rule} (${violation.guide}${advisory})`;
+	return `${violation.file}: ${violation.rule} (${violation.guide})`;
 }
 /**
 * Hand the written files to a reviewer who knows nothing about where they came
@@ -15179,13 +15173,12 @@ function uncheckedSteps(review, alreadyAsked = /* @__PURE__ */ new Set()) {
 * arrived. Absent violations are absent either way — there is nothing here to
 * act on when that half never came.
 *
-* Only what the reviewer would hold the change for. It is asked to sort its
-* own violations, because the alternative is a round spent on a line it would
-* have approved anyway. Every violation is still reported: `warnings` carries
-* the advisory ones too, marked as such.
+* Every violation the reviewer reports: a rule the project wrote down, or a
+* convention its files share, is the project's decision, not the reviewer's to
+* waive.
 */
 function guideViolations(review, alreadyAsked = /* @__PURE__ */ new Set()) {
-	const violations = (review?.ruleViolations ?? []).filter((v) => v.severity === "blocking" && !alreadyAsked.has(violationKey(v)));
+	const violations = (review?.ruleViolations ?? []).filter((v) => !alreadyAsked.has(violationKey(v)));
 	if (violations.length === 0) return null;
 	return {
 		exitCode: 1,
@@ -15464,7 +15457,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 +15547,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 +16201,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;

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.58.0",
+  "version": "1.58.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.58.0",
+  "version": "1.58.2",
   "type": "module",
   "description": "Browser test recorder powered by Claude Code and agent-browser",
   "repository": {