ccqa 1.8.2 → 1.8.3

package/dist/bin/ccqa.mjs

@@ -6932,6 +6932,68 @@ function verifySessionRestores(statePath, verifyUrl) {
 		]);
 	}
 }
+/**
+* Non-destructive mid-run health probe of an already-running live session.
+* Reads the current page URL (`eval location.href` — the same read
+* {@link verifySessionRestores} makes, and it does NOT navigate, so it never
+* clobbers the page the model is working on) and detects only the unambiguous
+* signals that agent-browser's daemon was replaced/wedged mid-run:
+*
+*   - the probe exits non-zero — the daemon is wedged or was replaced
+*     (`spawnAB`'s hard timeout bounds a hung daemon, so this returns rather
+*     than hanging the run);
+*   - the page is blank/absent (`about:blank`, empty, `chrome://…`) — a
+*     restarted daemon comes up with no page and no in-memory auth-state.
+*
+* Deliberately NOT flagged: a non-blank page that left the verify URL's origin.
+* That can't be told apart from a spec legitimately roaming to another origin
+* mid-flow (e.g. Slack → a separate admin app it logs into at runtime), and a
+* false "unhealthy" would re-inject the saved state and wipe auth the spec
+* acquired live — breaking a spec that was fine. Missing a same-origin-ish
+* sign-in wall just leaves that step failing as before (no regression), so the
+* asymmetry favours only firing on a provably dead daemon. `verifyUrl` is still
+* required (it's the re-anchor target for recovery) but no longer compared here.
+*/
+function checkLiveSessionHealth(sessionName) {
+	const probe = spawnAB([
+		"--session",
+		sessionName,
+		"eval",
+		"location.href"
+	]);
+	if (probe.status !== 0) return {
+		healthy: false,
+		reason: (probe.stderr || probe.stdout || `probe exited ${probe.status}`).trim()
+	};
+	const href = unwrapEvalString(probe.stdout);
+	if (!href || href === "about:blank" || href.startsWith("chrome://") || href.startsWith("chrome-error://")) return {
+		healthy: false,
+		reason: `blank/absent page (${href || "empty"})`
+	};
+	return { healthy: true };
+}
+/**
+* Recover a live session whose daemon was replaced mid-run (detected by
+* {@link checkLiveSessionHealth}). The restart drops the in-memory auth-state
+* injected at run start, so the session fell to a sign-in wall. Re-boot +
+* re-attach the saved state ({@link loadStateIntoSession} is idempotent —
+* `state load` is load-only, never writes back) and then navigate to
+* `verifyUrl`, a known signed-in page, so the retrying model has an
+* authenticated anchor to continue from instead of a login screen. Returns the
+* injection result; the trailing `open` is best-effort (a failed nav still
+* leaves the state attached for the model's own next navigation).
+*/
+function recoverLiveSession(sessionName, statePath, verifyUrl) {
+	const injected = loadStateIntoSession(sessionName, statePath);
+	if (!injected.ok) return injected;
+	spawnAB([
+		"--session",
+		sessionName,
+		"open",
+		verifyUrl
+	]);
+	return { ok: true };
+}
 /** Take the last non-empty line of `agent-browser eval` stdout and JSON-unquote it. */
 function unwrapEvalString(stdout) {
 	const lines = stdout.split("\n").map((l) => l.trim()).filter((l) => l.length > 0);
@@ -7514,6 +7576,7 @@ async function runLiveExecutor(input) {
 	const stepResults = [];
 	let overallFailed = false;
 	const statePath = input.statePath ?? null;
+	const verifyUrl = input.verifyUrl ?? null;
 	const promptPrefix = buildLiveSystemPromptPrefix({
 		title: input.spec.title,
 		allSteps: input.steps,
@@ -7545,12 +7608,25 @@ async function runLiveExecutor(input) {
 		const systemPrompt = promptPrefix + buildLiveSystemPromptStepSection(step$1) + suffixBlock + langDirective;
 		const userPrompt = buildLiveUserPrompt(step$1);
 		let attempt = 0;
+		let recoveredOnce = false;
 		let lastOutcome = null;
-		while (attempt <= retries) {
-			if (attempt > 0) info(`  retry ${attempt}/${retries} for ${step$1.id}`);
+		for (;;) {
 			lastOutcome = await executeStepAttempt(step$1, paths, systemPrompt, userPrompt);
 			if (lastOutcome.status === "passed") break;
+			if (!recoveredOnce && statePath && verifyUrl) {
+				const health = checkLiveSessionHealth(input.sessionName);
+				if (!health.healthy) {
+					warn(`session lost mid-step for ${step$1.id} (${health.reason}); re-injecting auth-state and retrying`);
+					const rec = recoverLiveSession(input.sessionName, statePath, verifyUrl);
+					if (!rec.ok) warn(`session recovery failed: ${rec.error}`);
+					recoveredOnce = true;
+					attempt++;
+					continue;
+				}
+			}
+			if (attempt >= retries) break;
 			attempt++;
+			info(`  retry ${attempt}/${retries} for ${step$1.id}`);
 		}
 		const outcome = lastOutcome;
 		stepResults.push({
@@ -8101,6 +8177,7 @@ async function resolveSessionState(names, hubCtx, profile, verify = verifySessio
 	const profileFlag = profile ? ` --profile ${profile}` : "";
 	const loaded = [];
 	const broken = [];
+	let verifyUrl;
 	for (const name of names) {
 		let state;
 		try {
@@ -8115,6 +8192,7 @@ async function resolveSessionState(names, hubCtx, profile, verify = verifySessio
 		}
 		const embedded = state[SESSION_VERIFY_URL_KEY];
 		if (typeof embedded === "string") {
+			verifyUrl ??= embedded;
 			const memoKey = `${resolvedProfile}/${name}`;
 			if (!verifiedSessions.has(memoKey)) {
 				const tmp = await writeMergedTempState(mergeStorageStates([state]));
@@ -8139,6 +8217,7 @@ async function resolveSessionState(names, hubCtx, profile, verify = verifySessio
 	return {
 		ok: true,
 		statePath,
+		...verifyUrl ? { verifyUrl } : {},
 		cleanup: () => removeTempStateDir(statePath)
 	};
 }
@@ -8167,6 +8246,7 @@ async function runOneSpec(args) {
 	meta("session", sessionName);
 	opts.teardown?.trackSession(sessionName);
 	let statePath = null;
+	let verifyUrl = null;
 	let cleanupSession = null;
 	if (spec.session && spec.session.length > 0) {
 		const resolution = await resolveSessionState(spec.session, opts.hubContext ?? null, opts.profile);
@@ -8181,6 +8261,7 @@ async function runOneSpec(args) {
 			};
 		}
 		statePath = resolution.statePath;
+		verifyUrl = resolution.verifyUrl ?? null;
 		cleanupSession = resolution.cleanup;
 		meta("state", spec.session.join(", "));
 	}
@@ -8196,6 +8277,7 @@ async function runOneSpec(args) {
 			runDir,
 			sessionName,
 			statePath,
+			verifyUrl,
 			systemPromptSuffix: userPromptSuffix,
 			model: opts.model,
 			language: opts.language,

package/dist/package.json

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