ccqa 1.60.2 → 1.60.4

package/dist/bin/ccqa.mjs

@@ -338,6 +338,14 @@ function validateTestPathTemplate(template) {
 	if (!template.includes("{spec}") && !template.includes("{case}")) return "testPath must contain {spec} or {case}, or every case would generate onto the same file";
 	return null;
 }
+/**
+* The directory every test the template names sits under: its path up to the
+* first placeholder's directory. Empty when the template opens with one.
+*/
+function testPathRoot(template) {
+	const head = template.split("{")[0];
+	return head.slice(0, head.lastIndexOf("/") + 1).replace(/\/$/, "");
+}
 /** What a recording beside its test is called: whose file it is, and what it is. */
 const RECORDING_EXTENSION = ".ccqa.ir.json";
 /**
@@ -9344,8 +9352,12 @@ const ViolationsSchema = z.object({ ruleViolations: z.array(z.unknown()) });
 * treats it as one it could not obtain.
 */
 const MAX_REVIEW_TURNS = 80;
-/** A wedged call, not a slow one: a review still silent after this is stuck. */
-const REVIEW_TIMEOUT_MS = 10 * 6e4;
+/**
+* A wedged call, not a slow one. A review of a real suite reads the test, its
+* page objects and many of the suite's files to count conventions, which runs
+* past ten minutes; one that is still going after twenty is stuck.
+*/
+const REVIEW_TIMEOUT_MS = 20 * 6e4;
 /** Read-only: a reviewer reads the repository and changes nothing in it. */
 const REVIEW_TOOLS = [
 	"Read",
@@ -14788,7 +14800,7 @@ const LLM_GEN_ALLOWED_TOOLS = [
 const LlmFileSchema = z.object({
 	path: z.string().min(1),
 	contents: z.string(),
-	kind: z.string().default("test")
+	kind: z.string().optional()
 });
 const LlmOutputSchema = z.object({
 	files: z.array(LlmFileSchema),
@@ -14820,21 +14832,29 @@ function parseLlmGenOutput(raw) {
 		const issues = res.error.issues.map((i) => `${i.path.join(".") || "(root)"}: ${i.message}`);
 		throw new Error(`reply JSON does not match {files:[{path,contents,kind}],summary}: ${issues.join("; ")}`);
 	}
-	const kindWarnings = [];
 	return {
-		files: res.data.files.map((f) => {
-			if (f.kind === "test" || f.kind === "support") return {
-				...f,
-				kind: f.kind
-			};
-			kindWarnings.push(`file "${f.path}": unknown kind "${f.kind}" coerced to "test"`);
-			return {
-				...f,
-				kind: "test"
-			};
-		}),
-		summary: res.data.summary,
-		kindWarnings
+		files: res.data.files.map((f) => ({
+			path: f.path,
+			contents: f.contents,
+			kind: f.kind === "support" ? "support" : "test"
+		})),
+		summary: res.data.summary
+	};
+}
+/**
+* The file at the configured test path is the test and every other file is
+* support — whatever the reply labelled them. The path already says which one
+* a file is, and a model that labels a page object "test" would otherwise fail
+* the whole generation over a word.
+*/
+function kindByPath(output, policy) {
+	const testAbs = resolve(policy.cwd, policy.testPath);
+	return {
+		...output,
+		files: output.files.map((f) => ({
+			...f,
+			kind: resolve(policy.cwd, f.path) === testAbs ? "test" : "support"
+		}))
 	};
 }
 /**
@@ -14994,7 +15014,6 @@ async function generateWithLlmEngine(req) {
 		requireTestFile: true,
 		allowEmpty: false
 	}), "run");
-	warnings.push(...output.kindWarnings);
 	return finalizeAndVerify({
 		ctx,
 		target: req.target,
@@ -15480,10 +15499,12 @@ async function identifiersInProject(ctx, writeRoots, emitted) {
 		const names = [...source.matchAll(/\bexport\s+(?:default\s+)?(?:abstract\s+)?class\s+([A-Za-z0-9_]+)/g)].map((m) => m[1]);
 		owners.set(rel, names);
 	}
+	const testsRoot = ctx.targetConfig.testPath ? testPathRoot(ctx.targetConfig.testPath) : "";
 	const roots = new Set([
 		...writeRoots,
 		...ctx.resources.map((r) => "path" in r ? r.path : ""),
-		dirname(ctx.testPath)
+		dirname(ctx.testPath),
+		testsRoot
 	].filter((r) => r.length > 0).map((r) => resolve(ctx.cwd, r)));
 	const found = new Map([...emitted.keys()].map((rel) => [rel, /* @__PURE__ */ new Set()]));
 	const seen = /* @__PURE__ */ new Set();
@@ -15559,8 +15580,7 @@ async function invokeForFiles(p) {
 			model: p.ctx.model
 		}, () => {});
 		if (isError) throw new Error(`Claude invocation failed: ${tail(result)}`);
-		const output = parseLlmGenOutput(result);
-		for (const w of output.kindWarnings) warn(w);
+		const output = kindByPath(parseLlmGenOutput(result), p.policy);
 		const errors = validateOutput(output, p);
 		if (errors.length > 0) throw new Error(errors.join("; "));
 		return output;
@@ -15603,8 +15623,7 @@ function validateOutput(output, p) {
 		if (fileError) errors.push(fileError);
 	}
 	const tests = output.files.filter((f) => f.kind === "test");
-	if (p.requireTestFile && tests.length === 0) errors.push("output contains no \"kind\": \"test\" file");
-	if (tests.length > 1) errors.push(`output contains ${tests.length} "kind": "test" files; a spec has exactly one`);
+	if (p.requireTestFile && tests.length === 0) errors.push(`the test file must be written to ${p.policy.testPath}; the reply has no file there`);
 	return errors;
 }
 function toPromptResource(r) {

package/dist/package.json

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