ccqa 1.45.0 → 1.45.1

package/dist/bin/ccqa.mjs

@@ -7167,6 +7167,7 @@ async function startBrowserCoverage(opts) {
 var Engine = class {
 	client;
 	connectFn;
+	/** Last known address of the browser; see `refreshWsUrl`. */
 	wsUrl;
 	reconnects = 0;
 	reconnectInFlight = false;
@@ -7285,6 +7286,22 @@ var Engine = class {
 		this.stopped = true;
 		this.clearTimer();
 	}
+	/**
+	* Answers why the address could not be refreshed, so the retry that follows
+	* says whether it is dialling a stale address or a dead browser. Told apart
+	* because one is a driver to fix and the other is a browser to wait for, and
+	* a silent stale address is the shape of the bug this exists to prevent.
+	*/
+	async refreshWsUrl() {
+		const ask = this.opts.currentCdpUrl;
+		if (ask === void 0) return void 0;
+		try {
+			this.wsUrl = await browserWebSocketUrl(await ask());
+			return;
+		} catch (error) {
+			return message(error);
+		}
+	}
 	async reconnect(initialReason) {
 		if (this.reconnectInFlight) return;
 		this.reconnectInFlight = true;
@@ -7300,11 +7317,12 @@ var Engine = class {
 				this.opts.warn(`browser coverage transport dropped (${reason}); reconnecting in ${delay}ms (${this.reconnects}/${MAX_RECONNECTS})`);
 				await new Promise((resolve) => setTimeout(resolve, delay));
 				if (this.stopped) return;
+				const stale = await this.refreshWsUrl();
 				let fresh;
 				try {
 					fresh = await this.connectFn(this.wsUrl);
 				} catch (error) {
-					reason = `reconnect failed: ${message(error)}`;
+					reason = stale ? `reconnect failed: ${message(error)} (address may be stale: ${stale})` : `reconnect failed: ${message(error)}`;
 					continue;
 				}
 				if (this.stopped) {
@@ -7319,6 +7337,7 @@ var Engine = class {
 					reason = `reconnect failed: ${message(error)}`;
 					continue;
 				}
+				this.reconnects = 0;
 				if (this.stopped) {
 					this.clearTimer();
 					fresh.close();
@@ -7749,9 +7768,10 @@ var CoverageSession = class CoverageSession {
 	* destinations, the roots — lives here, so the caller only supplies where
 	* the browser is.
 	*/
-	armBrowser(ref, cdpUrl, coverageDir) {
+	armBrowser(ref, browser, coverageDir) {
 		return startBrowserCoverage({
-			cdpUrl,
+			cdpUrl: browser.cdpUrl,
+			currentCdpUrl: browser.currentCdpUrl?.bind(browser),
 			specId: specIdFor(this.runId, ref),
 			origins: this.origins,
 			assetOrigins: this.assetOrigins,
@@ -8257,7 +8277,7 @@ async function runOneSpec$1(ref, opts, blocks) {
 			});
 			const acquired = browserHandle;
 			opts.teardown?.onFinalize(() => acquired.dispose());
-			browserEngine = await measurement.collector.armBrowser(ref, browserHandle.cdpUrl, coverageDir);
+			browserEngine = await measurement.collector.armBrowser(ref, browserHandle, coverageDir);
 			if (browserHandle.amendCommand) command = browserHandle.amendCommand(command);
 			Object.assign(childEnv, browserHandle.env);
 		} catch (err) {
@@ -13884,6 +13904,37 @@ async function acquireAgentBrowserEndpoint(ctx) {
 		"about:blank"
 	]);
 	if (warm.status !== 0) throw new Error(`could not start the session's browser: ${warm.stderr || warm.stdout}`);
+	return {
+		cdpUrl: askCdpUrl(session),
+		currentCdpUrl: () => askCdpUrlSoon(session),
+		dispose: async () => {}
+	};
+}
+/**
+* The same question on a short leash. Its caller is racing its own backoff
+* between reconnect attempts, so an answer that arrives after the budget is
+* spent is worth less than no answer: `spawnAB` would sit through a 30s EAGAIN
+* retry and a 35s hard timeout, which is the whole reconnect window several
+* times over.
+*/
+const CDP_URL_TIMEOUT_MS = 2e3;
+async function askCdpUrlSoon(session) {
+	const { promisify } = await import("node:util");
+	const { execFile } = await import("node:child_process");
+	const { stdout } = await promisify(execFile)(resolveAgentBrowserBin$1(), [
+		"--session",
+		session,
+		"get",
+		"cdp-url"
+	], {
+		timeout: CDP_URL_TIMEOUT_MS,
+		encoding: "utf8"
+	});
+	const cdpUrl = stdout.trim().split("\n").pop()?.trim() ?? "";
+	if (!/^wss?:\/\//.test(cdpUrl)) throw new Error(`agent-browser answered no cdp-url for session ${session}`);
+	return cdpUrl;
+}
+function askCdpUrl(session) {
 	const answer = spawnAB([
 		"--session",
 		session,
@@ -13892,10 +13943,7 @@ async function acquireAgentBrowserEndpoint(ctx) {
 	]);
 	const cdpUrl = answer.stdout.trim().split("\n").pop()?.trim() ?? "";
 	if (answer.status !== 0 || !/^wss?:\/\//.test(cdpUrl)) throw new Error(`agent-browser did not answer \`get cdp-url\` for session ${session}: ${answer.stderr || answer.stdout}`);
-	return {
-		cdpUrl,
-		dispose: async () => {}
-	};
+	return cdpUrl;
 }
 //#endregion
 //#region src/diagnose/snapshot.ts
@@ -14279,7 +14327,7 @@ async function runOneSpec(args) {
 		browserEngine = await opts.coverage.armBrowser({
 			featureName,
 			specName
-		}, handle.cdpUrl, coverageDir);
+		}, handle, coverageDir);
 	} catch (err) {
 		coverageBroken = errMessage(err);
 		warn(`coverage: could not attach to the live browser (${coverageBroken})`);

package/dist/package.json

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