ccqa 1.16.0 → 1.16.1

package/dist/bin/ccqa.mjs

@@ -8828,7 +8828,11 @@ const deployRecord = new Command("record").description("Tell the hub what a depl
 	meta("previous", previous ? previous.slice(0, 12) : "(none)");
 	meta("changed paths", changedPaths === null ? "(not reported)" : String(changedPaths.length));
 	if (opts.select) meta("selection", describeSelection(selection, diff !== null));
-	if (entry.gapBefore) warn("this deploy does not chain onto the log head, so a gap is recorded — specs whose baseline sits behind it report 'unknown' rather than 'not needed'");
+	if (entry.gapBefore) warn("this deploy does not chain onto the log head, so a gap is recorded — specs whose baseline sits behind it report 'unanswerable' rather than 'verified'");
+	if (selection && !entry.hasSelection) {
+		error("the hub recorded this deploy but could not store its spec selection — the range answers 'unanswerable' from here on, and nothing fills it in later. Re-record this deploy.");
+		process.exit(1);
+	}
 	info(`recorded deploy #${entry.index}`);
 }));
 /**
@@ -15429,6 +15433,10 @@ async function runAudit(specPath, opts) {
 		error("--only-hub-audit-needed and an explicit spec id cannot be combined; it only applies to a full sweep");
 		process.exit(2);
 	}
+	if (opts.onlyHubAuditNeeded && opts.onlyAffectedBy) {
+		error("--only-hub-audit-needed and --only-affected-by cannot be combined: a spec the hub says needs auditing that the diff drops is never audited, so it stays due forever. Pick the one that matches the job — the hub answer after a deploy, the diff on a pull request.");
+		process.exit(2);
+	}
 	let hub = null;
 	let hubProject = null;
 	let holder = null;
@@ -15445,7 +15453,7 @@ async function runAudit(specPath, opts) {
 		});
 	}
 	let targets = await collectTargets(specPath, cwd);
-	if (targets.length === 0) exitWithNoSpecs(format, "no test specs found under .ccqa/features/");
+	if (targets.length === 0) exitWithNoSpecs(format, "noSpecsFound", "no test specs found under .ccqa/features/");
 	if (format === "text") {
 		header("audit", specPath ?? `${targets.length} spec${targets.length > 1 ? "s" : ""}`);
 		if (opts.cwd) meta("cwd", cwd);
@@ -15462,12 +15470,12 @@ async function runAudit(specPath, opts) {
 			meta("hub", selection.summary);
 			meta("scoped", `${targets.length} of ${total} spec${total > 1 ? "s" : ""}`);
 		}
-		if (targets.length === 0) exitWithNoSpecs(format, "every spec has been audited since the last deploy that reached it");
+		if (targets.length === 0) exitWithNoSpecs(format, "allCurrent", "every spec has been audited since the last deploy that reached it");
 		holder = randomUUID();
 		const claimed = await claimSpecs(hub, hubProject, opts.hubProfile, targets, holder);
 		if (claimed.length < targets.length && format === "text") meta("held-elsewhere", `${targets.length - claimed.length} spec(s) another job is auditing`);
 		targets = claimed;
-		if (targets.length === 0) exitWithNoSpecs(format, "every spec that needs auditing is already being audited by another job");
+		if (targets.length === 0) exitWithNoSpecs(format, "allHeld", "every spec that needs auditing is already being audited by another job");
 	}
 	let baseRef = null;
 	if (opts.onlyAffectedBy) {
@@ -15481,7 +15489,7 @@ async function runAudit(specPath, opts) {
 		targets = selection.specs;
 		baseRef = selection.base.ref;
 		if (format === "text") meta("scoped", `${targets.length} of ${total} spec${total > 1 ? "s" : ""}`);
-		if (targets.length === 0) exitWithNoSpecs(format, "no specs intersect the changed file set; nothing to check");
+		if (targets.length === 0) exitWithNoSpecs(format, "noDiffIntersection", "no specs intersect the changed file set; nothing to check");
 	}
 	const blocks = await loadAvailableBlocks(cwd);
 	try {
@@ -15592,10 +15600,14 @@ async function pushDriftResults(args, resolveHub = resolveHubClient) {
 		throw err;
 	}
 }
-function exitWithNoSpecs(format, message) {
+function exitWithNoSpecs(format, reason, message) {
 	reportCost();
-	if (format === "json") process.stdout.write(`${JSON.stringify({ specs: [] }, null, 2)}\n`);
+	if (format === "json") process.stdout.write(`${JSON.stringify({
+		specs: [],
+		skipped: reason
+	}, null, 2)}\n`);
 	else if (format === "text") info(message);
+	else if (format === "github" && reason === "noSpecsFound") process.stdout.write(`::warning::${message}\n`);
 	process.exit(0);
 }
 async function collectTargets(specPath, cwd) {
@@ -17296,27 +17308,33 @@ function createRecordDeployHandler(storage) {
 			...ref ? { ref } : {},
 			...runUrl ? { runUrl } : {},
 			changedPaths: changedPaths ?? null,
-			hasSelection: selection !== void 0
+			hasSelection: false
+		});
+		const folded = selection !== void 0 && await foldIntoTouchIndex(storage, project, profile, entry, selection);
+		if (folded) await storage.deploys.confirmSelection(project, profile, entry.index);
+		sendJson(ctx.res, 201, {
+			...entry,
+			hasSelection: folded === true
 		});
-		if (selection !== void 0) await foldIntoTouchIndex(storage, project, profile, entry, selection);
-		sendJson(ctx.res, 201, entry);
 	};
 }
 /**
 * Record what this deploy's selection decided, so a later read can answer each
 * spec's own range with two integer comparisons.
 *
-* Deliberately after the append, and it does not fail the request: the log is
-* the record of what shipped and has to land even if the fold cannot. A lost
-* fold shows up as specs reporting `unknown` rather than as a wrong `notNeeded`
-* — the entry is already marked `hasSelection`, so the read side knows the
-* range is unresolved either way.
+* It does not fail the request: the log is the record of what shipped and has
+* to land even if the fold cannot. What a lost fold must not do is *look* like
+* a recorded one — so the caller only marks the entry `hasSelection` when this
+* returns true, and the range reads as unresolved otherwise. The client is told
+* by the entry it gets back.
 */
 async function foldIntoTouchIndex(storage, project, profile, entry, selection) {
 	try {
 		await storage.deploys.updateTouchIndex(project, profile, (current) => foldTouchIndex(current, entry, selection));
+		return true;
 	} catch (err) {
 		console.error(`hub: touch-index fold failed for deploy "${entry.sha}" of "${project}/${profile}": ${errMsg(err)}`);
+		return false;
 	}
 }
 /** GET /api/v1/projects/:project/deploys?profile=&limit= — the retained log, oldest first. */
@@ -23434,6 +23452,18 @@ function createFileDeployStore(root) {
 			const log = await updateJson(deployLogPath(root, project, profile), (current) => appendDeploy(current, input));
 			return log.entries[log.entries.length - 1];
 		},
+		async confirmSelection(project, profile, index) {
+			await updateJson(deployLogPath(root, project, profile), (current) => {
+				const log = current ?? emptyDeployLog();
+				return {
+					...log,
+					entries: log.entries.map((e) => e.index === index ? {
+						...e,
+						hasSelection: true
+					} : e)
+				};
+			});
+		},
 		getLog: readLog,
 		async head(project, profile) {
 			const { entries } = await readLog(project, profile);

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

@@ -388,8 +388,8 @@ declare const ReportSpecResultSchema: z.ZodObject<{
   title: z.ZodNullable<z.ZodString>;
   target: z.ZodOptional<z.ZodString>;
   mode: z.ZodOptional<z.ZodEnum<{
-    live: "live";
     deterministic: "deterministic";
+    live: "live";
   }>>;
   status: z.ZodEnum<{
     passed: "passed";
@@ -574,8 +574,8 @@ declare const RunReportDataSchema: z.ZodObject<{
     title: z.ZodNullable<z.ZodString>;
     target: z.ZodOptional<z.ZodString>;
     mode: z.ZodOptional<z.ZodEnum<{
-      live: "live";
       deterministic: "deterministic";
+      live: "live";
     }>>;
     status: z.ZodEnum<{
       passed: "passed";

package/dist/package.json

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