ccqa 1.42.0 → 1.42.2

package/dist/bin/ccqa.mjs

@@ -6353,6 +6353,14 @@ function resolveCovered(prepared, ranges) {
 const VENDOR = /(^|\/)node_modules\//;
 /** `(rsc)`, `(pages-dir-browser)`, ... — which build layer, not part of the path. */
 const LAYER = /^\([^)]*\)\//;
+/**
+* What follows `[project]/` is relative to the build's project root, not to
+* where the build ran. Only this one root is stripped: Turbopack's others
+* (`[output]`, `[externals]`) name generated code, not files on disk.
+*/
+const TURBOPACK_PROJECT = /^\[project\]\//;
+/** The ` [layer] (ecmascript)` suffix Turbopack appends when it names a module. */
+const MODULE_IDENTIFIER = / \[[^\]]*\]/;
 const DEPENDENCY = { kind: "dependency" };
 const UNRESOLVED = { kind: "unresolved" };
 /** `absolute` as a posix path under `root`, or undefined when it is not under it. */
@@ -6371,9 +6379,12 @@ function normalizeSourcePath(raw, roots) {
 		path = slash < 0 ? afterScheme : afterScheme.slice(from);
 	}
 	path = posix.normalize(path.replace(LAYER, ""));
+	const rooted = TURBOPACK_PROJECT.test(path);
+	if (rooted) path = path.replace(TURBOPACK_PROJECT, "");
 	if (path === "" || path === ".") return UNRESOLVED;
 	if (path.startsWith("<") || path.startsWith("[")) return UNRESOLVED;
-	const absolute = path.startsWith("/") ? path : resolve(roots.base, path);
+	if (rooted && MODULE_IDENTIFIER.test(path)) return UNRESOLVED;
+	const absolute = path.startsWith("/") ? path : resolve(rooted ? roots.root : roots.base, path);
 	const rel = toProjectRelative(roots.root, absolute);
 	if (rel === void 0) return UNRESOLVED;
 	return VENDOR.test(rel) ? DEPENDENCY : {
@@ -6575,18 +6586,14 @@ var FrontendResolution = class {
 	* nearly all of them misses on a deployment that pushes nothing.
 	*/
 	sourceMapLoaders(script, source) {
-		const loaders = [];
 		const reference = readSourceMappingUrl(source);
-		if (reference !== void 0) {
-			const inline = decodeInlineSourceMap(reference);
-			if (inline !== void 0) loaders.push(() => Promise.resolve(inline));
-			else {
-				const target = absoluteOrUndefined(reference, script.url);
-				if (target !== void 0) loaders.push(() => this.fetchText(target));
-			}
-		}
+		const inline = reference === void 0 ? void 0 : decodeInlineSourceMap(reference);
+		const mapUrl = reference !== void 0 && inline === void 0 ? absoluteOrUndefined(reference, script.url) : void 0;
+		const loaders = [];
+		if (inline !== void 0) loaders.push(() => Promise.resolve(inline));
+		if (mapUrl !== void 0) loaders.push(() => this.fetchText(mapUrl));
 		const stored = this.fetchStoredMap;
-		if (stored !== void 0) loaders.push(() => stored(script.url));
+		if (stored !== void 0) loaders.push(() => stored(mapUrl ?? script.url, script.url));
 		return loaders;
 	}
 };
@@ -7191,7 +7198,7 @@ var Engine = class {
 			coverageDir: opts.coverageDir,
 			roots: opts.roots,
 			fetchText: (url) => this.fetchThroughBrowser(url),
-			fetchStoredMap: (scriptUrl) => this.storedMapFor(scriptUrl),
+			fetchStoredMap: (mapUrl, scriptUrl) => this.storedMapFor(mapUrl, scriptUrl),
 			warn: opts.warn
 		});
 	}
@@ -7458,19 +7465,24 @@ var Engine = class {
 		}
 	}
 	/**
-	* The map a deploy stored for `scriptUrl`, if this run knows where to ask.
+	* The map a deploy stored for a script, if this run knows where to ask.
 	* Addressed by the path under the asset origin rather than the URL, so the
 	* push and the read agree without either knowing the other's host.
+	*
+	* The script's pointer names the map, but it may name it somewhere this run
+	* never declared — a map-only host. Guessing from the script is worse than
+	* the pointer and better than not asking, so it stays as the fallback.
 	*/
-	async storedMapFor(scriptUrl) {
+	async storedMapFor(mapUrl, scriptUrl) {
 		const stored = this.opts.fetchStoredSourceMap;
 		if (stored === void 0) return void 0;
-		const assetPath = assetPathOf(scriptUrl, [...this.opts.origins, ...this.opts.assetOrigins ?? []]);
-		if (assetPath === void 0) return void 0;
+		const origins = [...this.opts.origins, ...this.opts.assetOrigins ?? []];
+		const key = storeKeyOf(mapUrl, origins) ?? storeKeyOf(scriptUrl, origins);
+		if (key === void 0) return void 0;
 		try {
-			return await stored(`${assetPath}.map`);
+			return await stored(key);
 		} catch (err) {
-			this.opts.warn(`could not read the stored source map for ${assetPath}: ${message(err)}`);
+			this.opts.warn(`could not read the stored source map for ${key}: ${message(err)}`);
 			return;
 		}
 	}
@@ -7494,6 +7506,16 @@ function message(error) {
 * is part of what the browser asked for, so the push has to have used it too.
 * A URL from an origin the run never declared is not ours to look up.
 */
+/**
+* The key a stored map is read by: the asset path, with `.map` appended only
+* when it is missing. A push files each map under its own path, so a URL that
+* already names a map needs no suffix; only a script URL has to guess one.
+*/
+function storeKeyOf(url, origins) {
+	const assetPath = assetPathOf(url, origins);
+	if (assetPath === void 0) return void 0;
+	return assetPath.endsWith(".map") ? assetPath : `${assetPath}.map`;
+}
 function assetPathOf(url, origins) {
 	let parsed;
 	try {
@@ -15704,7 +15726,7 @@ const CONFIG_NAMES = [
 	"playwright.config.cjs"
 ];
 async function acquirePlaywrightBrowser(ctx) {
-	await sweepStaleWrappers(ctx.cwd);
+	await sweepStaleWrappersOnce(ctx.cwd);
 	const chromium = await resolveChromium(ctx.cwd);
 	const port = await freePort();
 	const server = await chromium.launchServer({ args: [`--remote-debugging-port=${port}`] });
@@ -15735,9 +15757,26 @@ async function acquirePlaywrightBrowser(ctx) {
 	};
 }
 /**
-* Wrappers a killed earlier run left behind. Deleted on the next acquire, not
-* only guarded against: a stray one is git-status dirt in somebody's repo.
+* Wrappers a killed earlier run left behind. Deleted before this process
+* writes its first one, not only guarded against: a stray one is git-status
+* dirt in somebody's repo.
+*
+* Once per cwd per process, never per acquire: every live wrapper matches
+* the stale pattern, so under `--concurrency` a per-acquire sweep deletes
+* the wrapper a parallel spec just wrote and is about to hand to
+* `playwright test`, killing that spec before it starts. The memo makes
+* concurrent first acquires share one sweep that finished before either of
+* them wrote anything.
 */
+const sweptCwds = /* @__PURE__ */ new Map();
+function sweepStaleWrappersOnce(cwd) {
+	let sweep = sweptCwds.get(cwd);
+	if (!sweep) {
+		sweep = sweepStaleWrappers(cwd);
+		sweptCwds.set(cwd, sweep);
+	}
+	return sweep;
+}
 async function sweepStaleWrappers(cwd) {
 	const entries = await readdir(cwd).catch(() => []);
 	for (const name of entries) if (name.startsWith("ccqa-coverage.") && name.endsWith(".playwright.config.ts")) await unlink(join(cwd, name)).catch(() => void 0);

package/dist/package.json

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