ccqa 1.31.1 → 1.31.3

package/dist/bin/ccqa.mjs

@@ -15579,10 +15579,11 @@ async function runRecord(specPath, opts) {
 	let sealed = true;
 	let tracingStep;
 	let abortCause;
+	let traceFailureNote;
 	const teardown = createRunTeardown();
 	teardown.onFinalize(async () => {
 		if (!push) return;
-		sealed = await sealRecordPush(push, featureName, specName, recorded, abortCause !== void 0 ? abortNote(abortCause, tracingStep) : void 0);
+		sealed = await sealRecordPush(push, featureName, specName, recorded, abortCause !== void 0 ? abortNote(abortCause, tracingStep) : traceFailureNote);
 	});
 	const disposeSignalHandlers = installTeardownSignalHandlers(teardown, (sig) => {
 		abortCause = `terminated by signal (${sig})`;
@@ -15609,6 +15610,7 @@ async function runRecord(specPath, opts) {
 				}
 			});
 			tracingStep = void 0;
+			if (traceResult.status !== "passed") traceFailureNote = "trace finished FAILED — a step did not complete, so the recording does not demonstrate the spec";
 			blank();
 			await learnFromTrace({
 				enabled: opts.learnHubTracePrompt === true,
@@ -15633,14 +15635,17 @@ async function runRecord(specPath, opts) {
 		} finally {
 			await releaseLock();
 		}
-		recorded = generated;
+		recorded = generated && traceFailureNote === void 0;
 	} finally {
 		if (deadline !== void 0) clearTimeout(deadline);
 		await teardown.run();
 		disposeSignalHandlers();
 	}
 	if (!sealed) process.exit(2);
-	if (!recorded) process.exit(1);
+	if (!recorded) {
+		if (traceFailureNote !== void 0) error(traceFailureNote);
+		process.exit(1);
+	}
 }
 /** `--timeout <seconds>`: a positive whole number of seconds. */
 function parseTimeoutSeconds(raw) {
@@ -16487,19 +16492,42 @@ async function fetchAuditNeed(ctx, profile) {
 		throw new RunUsageError(`${FLAG}: could not ask the hub which specs need auditing: ${errMessage(err)}`);
 	}
 }
+/**
+* Specs whose drift-ledger entry is still open. The hub's audit-need answer is
+* deploy-based, and a merged fix changes only the spec tree — no deploy lands
+* on the spec, so the hub would never call it due again and the entry would
+* stay open forever. A drifted spec is due until the audit itself clears it.
+* Unreadable ledger degrades to the deploy-based answer alone: the sweep must
+* not die over the supplementary question.
+*/
+async function fetchStillDrifted(ctx) {
+	try {
+		const ledger = await ctx.hub.getDriftLedger(ctx.project);
+		return new Set(Object.entries(ledger.specs ?? {}).filter(([, entry]) => entry.label != null).map(([key]) => key));
+	} catch {
+		return /* @__PURE__ */ new Set();
+	}
+}
 /** Worst-known-first, so the line leads with what has never been looked at. */
 const SUMMARY_ORDER = rankedOrder({
 	neverAudited: 0,
-	cannotTell: 1,
-	deployReached: 2,
-	held: 3,
-	current: 4
+	stillDrifted: 1,
+	cannotTell: 2,
+	deployReached: 3,
+	held: 4,
+	current: 5
 });
-function selectSpecsNeedingAudit(targets, report) {
+function selectSpecsNeedingAudit(targets, report, stillDrifted = /* @__PURE__ */ new Set()) {
 	const counts = /* @__PURE__ */ new Map();
 	const selected = [];
 	for (const target of targets) {
-		const need = report.specs[specKey(target)] ?? { because: "neverAudited" };
+		const key = specKey(target);
+		if (stillDrifted.has(key)) {
+			counts.set("stillDrifted", (counts.get("stillDrifted") ?? 0) + 1);
+			selected.push(target);
+			continue;
+		}
+		const need = report.specs[key] ?? { because: "neverAudited" };
 		counts.set(need.because, (counts.get(need.because) ?? 0) + 1);
 		if (needsAudit(need)) selected.push(target);
 	}
@@ -16511,7 +16539,7 @@ function selectSpecsNeedingAudit(targets, report) {
 //#endregion
 //#region src/cli/audit.ts
 const DEFAULT_CONCURRENCY = 3;
-const auditCommand = addProfileOption(addLanguageOption(new Command("audit").argument("[feature/spec]", "Optional spec id. If omitted, every spec under .ccqa/features/ is checked.").description("Read each spec against the code it describes and report where the two have drifted. Static: no browser is run, so this is the cheap check to put in front of `ccqa run`.").optionsGroup("Which specs to audit:").option("--only-affected-by <ref>", "Only specs `ccqa select-specs` judges reached by the diff against <ref> (e.g. origin/main). In pull_request CI, pass $GITHUB_BASE_REF. Costs one model call; specs it cannot decide are audited rather than skipped.").option("--only-hub-audit-needed", "Only specs the hub says a deploy has landed on since the audit last read them. A spec that was never audited is always included, and one the hub cannot answer for is audited rather than skipped. No git diff involved. Requires a hub connection and --hub-profile.").optionsGroup("How to run it:").option("--concurrency <n>", `Parallel spec checks (default: ${DEFAULT_CONCURRENCY})`).option("-m, --model <name>", "Claude model alias ('sonnet'|'opus'|'haiku') or full ID. Overrides CCQA_MODEL.").optionsGroup("What to do with the results:").option("--report-format <fmt>", "Output format: text | json | github", "text").option("--report-to-hub", "Push the result to a ccqa hub as a run (kind: drift), which is what updates the drift ledger. A spec it finds drifted answers `needsRepair` to `ccqa run --only-hub-rerun-needed`, and is not run until a person repairs it.").option("--exit-on <level>", "Exit non-zero on this severity or higher: warn | error", "error").optionsGroup("Environment and connection:").option("--cwd <path>", "Working directory used as both the .ccqa root and the codebase Claude reads. Useful for monorepos. Defaults to process.cwd().").option("--project <name>", "Logical project name for the pushed run. Defaults to the current directory's name.").option(...hubUrlOption).option(...hubTokenOption).option(...hubHeaderOption))).action(withUsageErrors(async (specPath, opts) => {
+const auditCommand = addProfileOption(addLanguageOption(new Command("audit").argument("[feature/spec]", "Optional spec id. If omitted, every spec under .ccqa/features/ is checked.").description("Read each spec against the code it describes and report where the two have drifted. Static: no browser is run, so this is the cheap check to put in front of `ccqa run`.").optionsGroup("Which specs to audit:").option("--only-affected-by <ref>", "Only specs `ccqa select-specs` judges reached by the diff against <ref> (e.g. origin/main). In pull_request CI, pass $GITHUB_BASE_REF. Costs one model call; specs it cannot decide are audited rather than skipped.").option("--only-hub-audit-needed", "Only specs the hub says a deploy has landed on since the audit last read them. A spec that was never audited is always included, one the hub cannot answer for is audited rather than skipped, and one whose drift entry is still open is always re-audited — a merged fix changes only the spec tree, which no deploy answer covers. No git diff involved. Requires a hub connection and --hub-profile.").optionsGroup("How to run it:").option("--concurrency <n>", `Parallel spec checks (default: ${DEFAULT_CONCURRENCY})`).option("-m, --model <name>", "Claude model alias ('sonnet'|'opus'|'haiku') or full ID. Overrides CCQA_MODEL.").optionsGroup("What to do with the results:").option("--report-format <fmt>", "Output format: text | json | github", "text").option("--report-to-hub", "Push the result to a ccqa hub as a run (kind: drift), which is what updates the drift ledger. A spec it finds drifted answers `needsRepair` to `ccqa run --only-hub-rerun-needed`, and is not run until a person repairs it.").option("--exit-on <level>", "Exit non-zero on this severity or higher: warn | error", "error").optionsGroup("Environment and connection:").option("--cwd <path>", "Working directory used as both the .ccqa root and the codebase Claude reads. Useful for monorepos. Defaults to process.cwd().").option("--project <name>", "Logical project name for the pushed run. Defaults to the current directory's name.").option(...hubUrlOption).option(...hubTokenOption).option(...hubHeaderOption))).action(withUsageErrors(async (specPath, opts) => {
 	await withCostReporting("audit", () => runAudit(specPath, opts));
 }));
 async function runAudit(specPath, opts) {
@@ -16555,11 +16583,12 @@ async function runAudit(specPath, opts) {
 	}
 	if (opts.onlyHubAuditNeeded) {
 		const total = targets.length;
-		const report = await fetchAuditNeed({
+		const ctx = {
 			hub,
 			project: hubProject
-		}, opts.hubProfile);
-		const selection = selectSpecsNeedingAudit(targets, report);
+		};
+		const [report, stillDrifted] = await Promise.all([fetchAuditNeed(ctx, opts.hubProfile), fetchStillDrifted(ctx)]);
+		const selection = selectSpecsNeedingAudit(targets, report, stillDrifted);
 		targets = selection.selected;
 		if (format === "text") {
 			meta("hub", selection.summary);

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

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

package/dist/package.json

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