@empiricalrun/playwright-utils 0.62.0 → 0.63.0

package/dist/reporter/uploader.js

@@ -107,9 +107,6 @@ class Uploader {
             await (0, r2_uploader_1.waitForTaskQueueToFinish)();
             return;
         }
-        // This drain runs after the last test with no other output (observed at
-        // 4+ minutes on spot runs), so make it visible: count up front, progress
-        // every 30s, duration at the end.
         const start = Date.now();
         console.log(`[Uploader] Waiting for ${pending} pending uploads (${executing} executing, ${waiting} queued)...`);
         const progress = setInterval(() => {
@@ -143,7 +140,6 @@ function createUploader() {
         logger_1.logger.debug("[Uploader] PROJECT_NAME and TEST_RUN_GITHUB_ACTION_ID must be set");
         return null;
     }
-    // REPORT_STORAGE_BACKEND selects the upload target; anything else is a misconfig.
     const backend = process.env.REPORT_STORAGE_BACKEND;
     if (backend !== "s3" && backend !== "r2") {
         logger_1.logger.debug(`[Uploader] REPORT_STORAGE_BACKEND must be "s3" or "r2" (got ${backend ?? "unset"})`);
@@ -165,8 +161,6 @@ function createUploader() {
             logger_1.logger.debug("[Uploader] S3_REGION or S3_UPLOAD_BUCKET not configured");
             return null;
         }
-        // Upload through the task role to S3, but persist CDN URLs so the bucket
-        // can stay private. Direct S3 is the fallback for envs without a CDN.
         baseUrl =
             process.env.REPORTS_BASE_URL?.replace(/\/+$/, "") ||
                 `https://${bucket}.s3.${region}.amazonaws.com`;

package/dist/reporter/util.d.ts

@@ -20,40 +20,7 @@ export declare function suitesAndProjectForTest(test: TestCase): {
 export declare function sendTestCaseUpdateToDashboard(params: TestCaseRunEndEventV2): Promise<void>;
 export declare function safelySerialiseJSON(obj: any, keyFilter?: (key: string) => boolean): any;
 export declare function updateHtmlZipFileAttachmentPaths(jsonFilePath: string, htmlFilePath: string, repoPath: string): Promise<void>;
-/**
- * Replaces media paths in test results with normalized paths from the test attachment map.
- * This function updates the attachments array in each test result with the corresponding
- * normalized attachments from the test attachment map.
- *
- * @param jsonData - The JSON data containing test results, which can be either in the format
- *                   of { files: [{ tests: [...] }] } or { tests: [...] }
- * @param testAttachmentMap - A Map where the key is the test ID and the value is an array of
- *                           objects containing attachments with updated path for each test result
- * @returns void
- */
-/**
- * Builds a map from test ID to an array of objects containing attachments for each result (retry).
- * The attachments are normalized to use relative paths starting with 'data/'.
- * @param summaryJsonFilePath Path to the summary JSON file
- * @returns Map<string, { attachments: GenericPlaywrightAttachment[] }[]>
- */
 export declare function buildTestAttachmentMapFromSummary(summaryJsonFilePath: string, repoPath: string, doNotNormalize?: boolean): TestAttachmentMap;
-/**
- * Checks asynchronously if a file exists at the given path.
- * @param filePath - The path to the file to check
- * @returns Promise<boolean> - true if the file exists, false otherwise
- */
 export declare function checkFileExistsAsync(filePath: string): Promise<boolean>;
-/**
- * Updates the attachment paths in the summary JSON file to use remote URLs.
- *
- * This function rewrites all attachment `path` fields in the given Playwright summary JSON file
- * so that they point to remote URLs based on the current `PROJECT_NAME` and `TEST_RUN_GITHUB_ACTION_ID`
- * environment variables.
- *
- * @param filePath - The path to the summary JSON file
- * @returns The updated summary JSON
- *
- */
 export declare function updateSummaryJsonAttachmentPaths(filePath: string, repoPath: string, urlPrefix: string): Promise<PlaywrightJSONReport | undefined>;
 //# sourceMappingURL=util.d.ts.map

package/dist/reporter/util.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"util.d.ts","sourceRoot":"","sources":["../../src/reporter/util.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAQhE,OAAO,EAAE,qBAAqB,EAAE,MAAM,6CAA6C,CAAC;AACpF,OAAO,EACL,UAAU,IAAI,oBAAoB,EAElC,QAAQ,EACT,MAAM,2BAA2B,CAAC;AA0BnC,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAqB7D;AAED,wBAAgB,yBAAyB,CACvC,YAAY,EAAE,MAAM,EACpB,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,MAAM,GAAG,SAAS,UAQhC;AAED,wBAAsB,0BAA0B,CAC9C,UAAU,EAAE,MAAM,EAClB,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAAC,WAAW,CAAC,EAAE,MAAM,CAAA;CAAO,GAC5E,OAAO,CAAC;IACT,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,WAAW,EAAE,MAAM,CAAC;CACrB,CAAC,CA6CD;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,QAAQ;IAqBlD,MAAM;IACN,WAAW;EAEd;AAED,wBAAsB,6BAA6B,CACjD,MAAM,EAAE,qBAAqB,GAC5B,OAAO,CAAC,IAAI,CAAC,CA2Bf;AAED,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,GAAG,EACR,SAAS,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,GACnC,GAAG,CAIL;AAED,wBAAsB,gCAAgC,CACpD,YAAY,EAAE,MAAM,EACpB,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,MAAM,iBAoFjB;AAED;;;;;;;;;;GAUG;AACH;;;;;GAKG;AACH,wBAAgB,iCAAiC,CAC/C,mBAAmB,EAAE,MAAM,EAC3B,QAAQ,EAAE,MAAM,EAChB,cAAc,GAAE,OAAe,GAC9B,iBAAiB,CAgBnB;AAED;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAO7E;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,gCAAgC,CACpD,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,6CA+BlB"}
+{"version":3,"file":"util.d.ts","sourceRoot":"","sources":["../../src/reporter/util.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAQhE,OAAO,EAAE,qBAAqB,EAAE,MAAM,6CAA6C,CAAC;AACpF,OAAO,EACL,UAAU,IAAI,oBAAoB,EAElC,QAAQ,EACT,MAAM,2BAA2B,CAAC;AA0BnC,wBAAgB,kBAAkB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAqB7D;AAED,wBAAgB,yBAAyB,CACvC,YAAY,EAAE,MAAM,EACpB,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,MAAM,GAAG,SAAS,UAQhC;AAED,wBAAsB,0BAA0B,CAC9C,UAAU,EAAE,MAAM,EAClB,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IAAC,WAAW,CAAC,EAAE,MAAM,CAAA;CAAO,GAC5E,OAAO,CAAC;IACT,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC1B,WAAW,EAAE,MAAM,CAAC;CACrB,CAAC,CA6CD;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,QAAQ;IAmBlD,MAAM;IACN,WAAW;EAEd;AAED,wBAAsB,6BAA6B,CACjD,MAAM,EAAE,qBAAqB,GAC5B,OAAO,CAAC,IAAI,CAAC,CA2Bf;AAED,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,GAAG,EACR,SAAS,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,GACnC,GAAG,CAIL;AAED,wBAAsB,gCAAgC,CACpD,YAAY,EAAE,MAAM,EACpB,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,MAAM,iBAkFjB;AAED,wBAAgB,iCAAiC,CAC/C,mBAAmB,EAAE,MAAM,EAC3B,QAAQ,EAAE,MAAM,EAChB,cAAc,GAAE,OAAe,GAC9B,iBAAiB,CAgBnB;AAED,wBAAsB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAO7E;AAED,wBAAsB,gCAAgC,CACpD,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,6CA+BlB"}

package/dist/reporter/util.js

@@ -108,8 +108,6 @@ function suitesAndProjectForTest(test) {
             parent = parent.parent;
         }
     }
-    // rawSuites looks like [ 'describe 3', 'describe 1', 'home.spec.ts', 'chromium', '' ]
-    // For suites, we will extract suites and then reverse them, since we expect top-to-bottom ordering for suites
     let suites = [];
     let iteratorIdx = 0;
     for (; iteratorIdx < rawSuites.length; iteratorIdx++) {
@@ -210,27 +208,8 @@ async function updateHtmlZipFileAttachmentPaths(jsonFilePath, htmlFilePath, repo
         logger_1.logger.error("❌ Error processing report.json or updating files:", err);
     }
     finally {
-        // Optional cleanup:
-        // await fs.promises.rm(tempDir, { recursive: true, force: true });
     }
 }
-/**
- * Replaces media paths in test results with normalized paths from the test attachment map.
- * This function updates the attachments array in each test result with the corresponding
- * normalized attachments from the test attachment map.
- *
- * @param jsonData - The JSON data containing test results, which can be either in the format
- *                   of { files: [{ tests: [...] }] } or { tests: [...] }
- * @param testAttachmentMap - A Map where the key is the test ID and the value is an array of
- *                           objects containing attachments with updated path for each test result
- * @returns void
- */
-/**
- * Builds a map from test ID to an array of objects containing attachments for each result (retry).
- * The attachments are normalized to use relative paths starting with 'data/'.
- * @param summaryJsonFilePath Path to the summary JSON file
- * @returns Map<string, { attachments: GenericPlaywrightAttachment[] }[]>
- */
 function buildTestAttachmentMapFromSummary(summaryJsonFilePath, repoPath, doNotNormalize = false) {
     const summaryJson = JSON.parse(fs_1.default.readFileSync(summaryJsonFilePath, "utf8"));
     return (0, reporter_1.buildTestAttachmentMap)(summaryJson, {
@@ -245,11 +224,6 @@ function buildTestAttachmentMapFromSummary(summaryJsonFilePath, repoPath, doNotN
             },
     });
 }
-/**
- * Checks asynchronously if a file exists at the given path.
- * @param filePath - The path to the file to check
- * @returns Promise<boolean> - true if the file exists, false otherwise
- */
 async function checkFileExistsAsync(filePath) {
     try {
         await fs_1.default.promises.access(filePath);
@@ -259,17 +233,6 @@ async function checkFileExistsAsync(filePath) {
         return false;
     }
 }
-/**
- * Updates the attachment paths in the summary JSON file to use remote URLs.
- *
- * This function rewrites all attachment `path` fields in the given Playwright summary JSON file
- * so that they point to remote URLs based on the current `PROJECT_NAME` and `TEST_RUN_GITHUB_ACTION_ID`
- * environment variables.
- *
- * @param filePath - The path to the summary JSON file
- * @returns The updated summary JSON
- *
- */
 async function updateSummaryJsonAttachmentPaths(filePath, repoPath, urlPrefix) {
     if (!process.env.PROJECT_NAME || !process.env.TEST_RUN_GITHUB_ACTION_ID) {
         console.error("PROJECT_NAME and TEST_RUN_GITHUB_ACTION_ID must be set");

package/dist/telemetry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"telemetry.d.ts","sourceRoot":"","sources":["../src/telemetry.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,sCAAsC,CAAC;AAUtE,wBAAsB,UAAU,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAyBrE;AAED,UAAU,uBAAuB;IAC/B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,uBAAuB,GAC9B,OAAO,CAAC,IAAI,CAAC,CAcf;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,QAAQ,EAAE,MAAM,GAAG,SAAS,CAAC;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,KAAK,EAAE,OAAO,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,MAAM,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IAC3C,IAAI,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;CAC1C,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBhB;AAED,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,iBAAiB,EAAE,MAAM,CAAC;CAC3B,GAAG,OAAO,CAAC,IAAI,CAAC,CAkBhB"}
+{"version":3,"file":"telemetry.d.ts","sourceRoot":"","sources":["../src/telemetry.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,sCAAsC,CAAC;AAOtE,wBAAsB,UAAU,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAmBrE;AAED,UAAU,uBAAuB;IAC/B,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,uBAAuB,GAC9B,OAAO,CAAC,IAAI,CAAC,CAcf;AAED,wBAAsB,qBAAqB,CAAC,IAAI,EAAE;IAChD,QAAQ,EAAE,MAAM,GAAG,SAAS,CAAC;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,KAAK,EAAE,OAAO,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,MAAM,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IAC3C,IAAI,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;CAC1C,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBhB;AAED,wBAAsB,yBAAyB,CAAC,IAAI,EAAE;IACpD,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,iBAAiB,EAAE,MAAM,CAAC;CAC3B,GAAG,OAAO,CAAC,IAAI,CAAC,CAkBhB"}

package/dist/telemetry.js

@@ -5,9 +5,6 @@ exports.trackReportUpload = trackReportUpload;
 exports.trackOverlayDismissed = trackOverlayDismissed;
 exports.trackLLMResponseGenerated = trackLLMResponseGenerated;
 const api_worker_client_1 = require("./api-worker-client");
-// Events go to api-worker on the project API key the run already carries, so
-// a runner needs no telemetry secret. Best-effort: no key means no telemetry,
-// and a failed send is never retried or surfaced to the test.
 async function trackEvent(event) {
     if (!process.env.EMPIRICALRUN_API_KEY) {
         return;
@@ -16,14 +13,10 @@ async function trackEvent(event) {
         const { client } = new api_worker_client_1.ApiWorkerClient();
         const { response } = await client.POST("/api/v1/telemetry/track", {
             body: {
-                // api-worker accepts only test-run events; the ones sent from here
-                // are all on its allowlist.
                 name: event.name,
                 properties: event.properties,
                 timestamp: event.timestamp?.toISOString(),
             },
-            // Plain fetch, not the client's retrying one: a test should not wait
-            // out backoff for a metric.
             fetch,
             signal: AbortSignal.timeout(10_000),
         });
@@ -31,9 +24,7 @@ async function trackEvent(event) {
             throw new Error(`Telemetry request failed: ${response.status}`);
         }
     }
-    catch {
-        // Don't throw - telemetry failures shouldn't break the application
-    }
+    catch { }
 }
 async function trackReportUpload(params) {
     const environment = process.env.ENVIRONMENT || "unknown";

package/dist/test/coverage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"coverage.d.ts","sourceRoot":"","sources":["../../src/test/coverage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAYvD,wBAAsB,uBAAuB,CAAC,OAAO,EAAE,cAAc,qBAwBpE;AAED,wBAAsB,wBAAwB,CAC5C,OAAO,EAAE,cAAc,EACvB,aAAa,EAAE,MAAM,EAAE,EACvB,QAAQ,EAAE;IACR,MAAM,EAAE,CACN,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,KAC3C,OAAO,CAAC,IAAI,CAAC,CAAC;CACpB,iBAqBF"}
+{"version":3,"file":"coverage.d.ts","sourceRoot":"","sources":["../../src/test/coverage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAYvD,wBAAsB,uBAAuB,CAAC,OAAO,EAAE,cAAc,qBAwBpE;AAED,wBAAsB,wBAAwB,CAC5C,OAAO,EAAE,cAAc,EACvB,aAAa,EAAE,MAAM,EAAE,EACvB,QAAQ,EAAE;IACR,MAAM,EAAE,CACN,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,KAC3C,OAAO,CAAC,IAAI,CAAC,CAAC;CACpB,iBAmBF"}

package/dist/test/coverage.js

@@ -31,9 +31,7 @@ async function collectAndAttachCoverage(context, coverageFiles, testInfo) {
         try {
             await page.evaluate(() => window.collectIstanbulCoverage(JSON.stringify(window.__coverage__)));
         }
-        catch {
-            // Page might be closed or coverage not available
-        }
+        catch { }
     }
     for (const filepath of coverageFiles) {
         if (fs_1.default.existsSync(filepath)) {

package/dist/test/expect/visual.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"visual.d.ts","sourceRoot":"","sources":["../../../src/test/expect/visual.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,OAAO,EAAE,IAAI,EAAY,MAAM,kBAAkB,CAAC;AAOhE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AAsE7C,wBAAsB,WAAW,CAC/B,MAAM,EAAE,IAAI,GAAG,OAAO,EACtB,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,aAAa,CAAC,CAwFxB"}
+{"version":3,"file":"visual.d.ts","sourceRoot":"","sources":["../../../src/test/expect/visual.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,OAAO,EAAE,IAAI,EAAY,MAAM,kBAAkB,CAAC;AAOhE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AAgE7C,wBAAsB,WAAW,CAC/B,MAAM,EAAE,IAAI,GAAG,OAAO,EACtB,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,aAAa,CAAC,CAsFxB"}

package/dist/test/expect/visual.js

@@ -17,7 +17,6 @@ function getCallerLine() {
         return "unknown";
     }
     const frames = stack.split("\n").slice(1);
-    // Find the test file frame (e.g. .spec.ts, .test.ts)
     const testFrame = frames.find((f) => /\.(spec|test)\.[jt]sx?:/.test(f));
     if (!testFrame) {
         return "unknown";
@@ -27,7 +26,6 @@ function getCallerLine() {
 }
 function attachScreenshot(screenshot, pageDescription) {
     try {
-        // test.info() is only available during a Playwright test run
         const { test } = require("@playwright/test");
         const testInfo = test.info();
         const slug = pageDescription
@@ -38,8 +36,6 @@ function attachScreenshot(screenshot, pageDescription) {
             .replace(/(^-|-$)/g, "");
         const line = getCallerLine();
         const attachmentName = `toLookRight-L${line}-${slug}`;
-        // Write to a temp file so the attachment has a path (required by empirical reporter).
-        // Playwright copies it into test-results/<test>/attachments/ on attach.
         const tmpFile = path_1.default.join(os_1.default.tmpdir(), `${attachmentName}-${Date.now()}.png`);
         fs_1.default.writeFileSync(tmpFile, screenshot);
         testInfo.attach(attachmentName, {
@@ -47,9 +43,7 @@ function attachScreenshot(screenshot, pageDescription) {
             contentType: "image/png",
         });
     }
-    catch {
-        // Not in a Playwright test context, skip attachment
-    }
+    catch { }
 }
 const SYSTEM_PROMPT = `
 You are a software QA tester who specializes in visual testing. You look at a
@@ -79,8 +73,6 @@ async function toLookRight(target, pageDescription) {
     attachScreenshot(screenshot, pageDescription);
     const base64Image = screenshot.toString("base64");
     const model = "gpt-4o";
-    // Same api-worker proxy the overlay-dismissal agent uses, so a test run needs
-    // no OpenAI key of its own; undefined falls back to the SDK's env defaults.
     const openai = new openai_1.default({ maxRetries: 5, ...(0, api_worker_client_1.llmProxyOpenAIConfig)() });
     const completion = await openai.chat.completions.create({
         model,

package/dist/test/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/test/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,MAAM,EACN,IAAI,EACJ,kBAAkB,EAClB,qBAAqB,EACrB,oBAAoB,EACpB,uBAAuB,EACvB,QAAQ,EACT,MAAM,kBAAkB,CAAC;AAG1B,OAAO,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAC;AAEjC,OAAO,EAGL,KAAK,qBAAqB,EAC3B,MAAM,UAAU,CAAC;AAClB,OAAO,EAAE,6BAA6B,EAAE,MAAM,WAAW,CAAC;AAC1D,OAAO,EAAE,KAAK,mBAAmB,EAAwB,MAAM,SAAS,CAAC;AACzE,OAAO,EAA8B,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAE3E,OAAO,CAAC,MAAM,CAAC;IACb,UAAU,cAAc,CAAC;QACvB,UAAU,QAAQ,CAAC,CAAC;YAClB,WAAW,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,CAAC;YAC7C,qBAAqB,CAAC,OAAO,EAAE,qBAAqB,GAAG,OAAO,CAAC,CAAC,CAAC,CAAC;SACnE;KACF;CACF;AAED,QAAA,MAAM,YAAY,mBAA6B,OAAO,MAAM,0CAG3D,CAAC;AAEF,KAAK,WAAW,GAAG;IACjB,IAAI,EAAE,IAAI,CAAC;IACX,EAAE,EAAE,QAAQ,CAAC;IACb,yBAAyB,EAAE,CACzB,OAAO,CAAC,EAAE,qBAAqB,KAC5B,OAAO,CAAC;QAAE,OAAO,EAAE,cAAc,CAAC;QAAC,IAAI,EAAE,IAAI,CAAA;KAAE,CAAC,CAAC;IACtD,UAAU,EAAE,IAAI,CAAC;CAClB,CAAC;AAYF,QAAA,MAAM,eAAe,WACX,QAAQ,CACd,kBAAkB,GAAG,qBAAqB,EAC1C,oBAAoB,GAAG,uBAAuB,CAC/C,YACQ,mBAAmB,uHAwH7B,CAAC;AAEF,OAAO,EACL,eAAe,EACf,YAAY,EACZ,6BAA6B,EAC7B,aAAa,GACd,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/test/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,MAAM,EACN,IAAI,EACJ,kBAAkB,EAClB,qBAAqB,EACrB,oBAAoB,EACpB,uBAAuB,EACvB,QAAQ,EACT,MAAM,kBAAkB,CAAC;AAG1B,OAAO,EAAE,QAAQ,EAAE,MAAM,OAAO,CAAC;AAEjC,OAAO,EAGL,KAAK,qBAAqB,EAC3B,MAAM,UAAU,CAAC;AAClB,OAAO,EAAE,6BAA6B,EAAE,MAAM,WAAW,CAAC;AAC1D,OAAO,EAAE,KAAK,mBAAmB,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAA8B,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAE3E,OAAO,CAAC,MAAM,CAAC;IACb,UAAU,cAAc,CAAC;QACvB,UAAU,QAAQ,CAAC,CAAC;YAClB,WAAW,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,CAAC;YAC7C,qBAAqB,CAAC,OAAO,EAAE,qBAAqB,GAAG,OAAO,CAAC,CAAC,CAAC,CAAC;SACnE;KACF;CACF;AAED,QAAA,MAAM,YAAY,mBAA6B,OAAO,MAAM,0CAG3D,CAAC;AAEF,KAAK,WAAW,GAAG;IACjB,IAAI,EAAE,IAAI,CAAC;IACX,EAAE,EAAE,QAAQ,CAAC;IACb,yBAAyB,EAAE,CACzB,OAAO,CAAC,EAAE,qBAAqB,KAC5B,OAAO,CAAC;QAAE,OAAO,EAAE,cAAc,CAAC;QAAC,IAAI,EAAE,IAAI,CAAA;KAAE,CAAC,CAAC;IACtD,UAAU,EAAE,IAAI,CAAC;CAClB,CAAC;AAIF,QAAA,MAAM,eAAe,WACX,QAAQ,CACd,kBAAkB,GAAG,qBAAqB,EAC1C,oBAAoB,GAAG,uBAAuB,CAC/C,YACQ,mBAAmB,uHA6G7B,CAAC;AAEF,OAAO,EACL,eAAe,EACf,YAAY,EACZ,6BAA6B,EAC7B,aAAa,GACd,CAAC"}

package/dist/test/index.js

@@ -17,23 +17,13 @@ const extendExpect = function (expectInstance) {
     return expectInstance;
 };
 exports.extendExpect = extendExpect;
-// Track pages per test to save videos with labels
 const testPages = new Map();
-const defaultOptions = {
-    mousePointerHighlighter: true,
-    locatorHighlighter: true,
-    visionMethods: true,
-    agentCapabilities: true,
-};
-const baseTestFixture = function (testFn, options = defaultOptions) {
+const baseTestFixture = function (testFn, options = {}) {
     const coverageEnabled = options.collectCoverage ?? false;
-    // Playwright draws the cursor, target box and action title into the video.
-    // 100ms per action keeps them visible without the default 500ms slowdown.
-    const showActions = options.mousePointerHighlighter === false ? undefined : { duration: 100 };
     const recordVideoOptions = (dir) => ({
         dir,
         size: { width: 1280, height: 720 },
-        showActions,
+        showActions: { duration: 100 },
     });
     const extendedTestFn = testFn.extend({
         context: async ({ browser }, use, testInfo) => {
@@ -57,10 +47,7 @@ const baseTestFixture = function (testFn, options = defaultOptions) {
             await context.close();
         },
         page: async ({ page }, use) => {
-            (0, scripts_1.injectLocatorHighlightScripts)(page, extendedTestFn, {
-                ...defaultOptions,
-                ...options,
-            });
+            (0, scripts_1.injectLocatorHighlightScripts)(page, extendedTestFn, options);
             await use(page);
         },
         kv: async ({}, use) => {
@@ -82,7 +69,6 @@ const baseTestFixture = function (testFn, options = defaultOptions) {
                 return { context: pageContext, page: customPage };
             }
             await use(createContext);
-            // Store pages for video renaming
             if (!testPages.has(testInfo.testId)) {
                 testPages.set(testInfo.testId, []);
             }
@@ -98,7 +84,6 @@ const baseTestFixture = function (testFn, options = defaultOptions) {
                 await use();
                 const pages = testPages.get(testInfo.testId) || [];
                 const usedLabels = new Set();
-                // Save videos with custom labels using saveAs
                 for (let i = 0; i < pages.length; i++) {
                     const page = pages[i];
                     if (!page) {
@@ -121,9 +106,7 @@ const baseTestFixture = function (testFn, options = defaultOptions) {
                         });
                         await video.delete();
                     }
-                    catch (e) {
-                        // Skip if video not available
-                    }
+                    catch (e) { }
                 }
                 testPages.delete(testInfo.testId);
             },

package/dist/test/scripts/agent-capabilities.js

@@ -1,8 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.addAgentCapabilities = addAgentCapabilities;
-const click_1 = require("./pw-locator-patch/highlight/click");
-const hover_1 = require("./pw-locator-patch/highlight/hover");
+const click_1 = require("./pw-locator-patch/actions/click");
+const hover_1 = require("./pw-locator-patch/actions/hover");
 function applyPatchForAutoDismissOverlay(page, testFn) {
     const LocatorClass = page.locator("").constructor;
     (0, click_1.patchClick)(LocatorClass, testFn);

package/dist/test/scripts/index.d.ts

@@ -1,5 +1,5 @@
 import type { Page } from "@playwright/test";
-import { HighlighterOpts } from "../types";
+import { PageScriptOpts } from "../types";
 import { TestFn } from "./types";
-export declare function injectLocatorHighlightScripts(page: Page, testFn: TestFn, options?: HighlighterOpts): void;
+export declare function injectLocatorHighlightScripts(page: Page, testFn: TestFn, { visionMethods, agentCapabilities }?: PageScriptOpts): void;
 //# sourceMappingURL=index.d.ts.map

package/dist/test/scripts/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/test/scripts/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAG7C,OAAO,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAI3C,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AAQjC,wBAAgB,6BAA6B,CAC3C,IAAI,EAAE,IAAI,EACV,MAAM,EAAE,MAAM,EACd,OAAO,GAAE,eAAgC,QAmB1C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/test/scripts/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAG7C,OAAO,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAG1C,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AAEjC,wBAAgB,6BAA6B,CAC3C,IAAI,EAAE,IAAI,EACV,MAAM,EAAE,MAAM,EACd,EAAE,aAAoB,EAAE,iBAAwB,EAAE,GAAE,cAAmB,QAYxE"}

package/dist/test/scripts/index.js

@@ -3,26 +3,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.injectLocatorHighlightScripts = injectLocatorHighlightScripts;
 const logger_1 = require("../../logger");
 const agent_capabilities_1 = require("./agent-capabilities");
-const locator_highlights_1 = require("./locator-highlights");
 const locator_vision_1 = require("./locator-vision");
-const defaultOptions = {
-    locatorHighlighter: true,
-    visionMethods: true,
-    agentCapabilities: true,
-};
-function injectLocatorHighlightScripts(page, testFn, options = defaultOptions) {
-    const opts = {
-        ...defaultOptions,
-        ...options,
-    };
+function injectLocatorHighlightScripts(page, testFn, { visionMethods = true, agentCapabilities = true } = {}) {
     try {
-        if (opts.locatorHighlighter) {
-            (0, locator_highlights_1.addLocatorHighlights)(page, testFn);
-        }
-        if (opts.visionMethods) {
+        if (visionMethods) {
             (0, locator_vision_1.addLocatorVisionMethods)(page);
         }
-        if (opts.agentCapabilities) {
+        if (agentCapabilities) {
             (0, agent_capabilities_1.addAgentCapabilities)(page, testFn);
         }
     }

package/dist/test/scripts/pw-locator-patch/actions/click.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"click.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/actions/click.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AAEhD,OAAO,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAOrC,wBAAgB,UAAU,CACxB,YAAY,EAAE,QAAQ,GAAG;IAAE,SAAS,EAAE,OAAO,CAAA;CAAE,EAC/C,MAAM,EAAE,MAAM,QAsDf"}

package/dist/test/scripts/pw-locator-patch/{highlight → actions}/click.js

@@ -4,7 +4,6 @@ exports.patchClick = patchClick;
 const dismiss_overlays_1 = require("../dismiss-overlays");
 const utils_1 = require("../dismiss-overlays/utils");
 const utils_2 = require("../utils");
-// Static flag to track if click has been patched
 const isClickPatched = new WeakMap();
 function patchClick(LocatorClass, testFn) {
     if (isClickPatched.get(LocatorClass)) {
@@ -23,19 +22,15 @@ function patchClick(LocatorClass, testFn) {
                 overlayAttemptsRemaining--;
                 try {
                     let result = await originalClick.apply(this, [options]);
-                    // If click is successful, we can save overlay dismissal code to cache and return
                     await overlayDismiss.saveToCache();
                     return result;
                 }
                 catch (error) {
                     originalError = error;
                     if (originalError instanceof Error) {
-                        // Overwrite the stack trace to show the original click (remove patch)
                         Error.captureStackTrace(originalError, LocatorClass.prototype.click);
                     }
                     if (options?.isExecutingForOverlayDismiss) {
-                        // While executing dismiss code from cache, we attach this custom option
-                        // which ensures that the overlay dismissal will not trigger recursively
                         throw originalError;
                     }
                     if (!(0, dismiss_overlays_1.isErrorSupported)(originalError.message)) {
@@ -45,7 +40,6 @@ function patchClick(LocatorClass, testFn) {
                         throw originalError;
                     }
                     await overlayDismiss.run(originalError, reporter, testFn);
-                    // Dismissal attempt finished (implicitly successful if no error thrown)
                     reporter(`Overlay dismissal attempted, retrying original action.`);
                 }
             }

package/dist/test/scripts/pw-locator-patch/actions/hover.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"hover.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/actions/hover.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AAEhD,OAAO,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAOrC,wBAAgB,UAAU,CACxB,YAAY,EAAE,QAAQ,GAAG;IAAE,SAAS,EAAE,OAAO,CAAA;CAAE,EAC/C,MAAM,EAAE,MAAM,QAgDf"}

package/dist/test/scripts/pw-locator-patch/{highlight → actions}/hover.js

@@ -21,7 +21,6 @@ function patchHover(LocatorClass, testFn) {
                 overlayAttemptsRemaining--;
                 try {
                     let result = await originalHover.apply(this, [options]);
-                    // If hover is successful, we can save overlay dismissal code to cache and return
                     await overlayDismiss.saveToCache();
                     return result;
                 }
@@ -31,8 +30,6 @@ function patchHover(LocatorClass, testFn) {
                         Error.captureStackTrace(originalError, LocatorClass.prototype.hover);
                     }
                     if (options?.isExecutingForOverlayDismiss) {
-                        // While executing dismiss code from cache, we attach this custom option
-                        // which ensures that the overlay dismissal will not trigger recursively
                         throw originalError;
                     }
                     if (!(0, dismiss_overlays_1.isErrorSupported)(originalError.message)) {

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/cache.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/cache.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAM7C,eAAO,MAAM,UAAU,QAItB,CAAC;AAOF,wBAAsB,cAAc,CAAC,EACnC,GAAG,EACH,IAAI,EACJ,IAAI,GACL,EAAE;IACD,GAAG,EAAE,MAAM,GAAG,SAAS,CAAC;IACxB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,IAAI,EAAE,MAAM,CAAC;CACd,iBAkCA;AAED,wBAAsB,gBAAgB,CAAC,EACrC,IAAI,EACJ,GAAG,EACH,IAAI,GACL,EAAE;IACD,IAAI,EAAE,IAAI,CAAC;IACX,GAAG,EAAE,MAAM,GAAG,SAAS,CAAC;IACxB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;CAC1B,GAAG,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAA;CAAE,CAAC,CAmChC;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,UAc5C"}
+{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/cache.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAM7C,eAAO,MAAM,UAAU,QAItB,CAAC;AAMF,wBAAsB,cAAc,CAAC,EACnC,GAAG,EACH,IAAI,EACJ,IAAI,GACL,EAAE;IACD,GAAG,EAAE,MAAM,GAAG,SAAS,CAAC;IACxB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,IAAI,EAAE,MAAM,CAAC;CACd,iBAgCA;AAED,wBAAsB,gBAAgB,CAAC,EACrC,IAAI,EACJ,GAAG,EACH,IAAI,GACL,EAAE;IACD,IAAI,EAAE,IAAI,CAAC;IACX,GAAG,EAAE,MAAM,GAAG,SAAS,CAAC;IACxB,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;CAC1B,GAAG,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAA;CAAE,CAAC,CAiChC;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,UAY5C"}

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/cache.js

@@ -11,16 +11,13 @@ const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 exports.CACHE_FILE = path_1.default.join(process.cwd(), ".empiricalrun", `overlay-cache.json`);
 function normalizeString(str) {
-    // Replace all whitespace sequences with a single space
     return str.replace(/\s+/g, " ").trim();
 }
 async function setCodeToCache({ dom, text, code, }) {
     if (!dom && !text) {
-        // At least one of the two is required
         return;
     }
     if (process.env.TEST_GEN_TOKEN) {
-        // No caching for test gen sessions (we don't want steps to get committed)
         return;
     }
     const obj = {
@@ -52,7 +49,6 @@ async function setCodeToCache({ dom, text, code, }) {
 }
 async function executeFromCache({ page, dom, text, }) {
     if (!dom && !text) {
-        // At least one of the two is required
         return { success: false };
     }
     try {
@@ -66,7 +62,6 @@ async function executeFromCache({ page, dom, text, }) {
         }
         try {
             console.log(`Executing from cache: "${match.code}" for element: ${dom}`);
-            // Ref: https://davidwalsh.name/async-function-class
             const AsyncFunction = Object.getPrototypeOf(async function () { }).constructor;
             const codeToExecute = addOverlayOption(match.code);
             const exec = new AsyncFunction("page", codeToExecute);
@@ -85,12 +80,10 @@ async function executeFromCache({ page, dom, text, }) {
 function addOverlayOption(code) {
     return code.replace(/\.(click|hover)(\s*\(\s*)(\{[^}]*\})?(\s*\))(;)?/g, (match, action, openParen, options, closeParen, semicolon) => {
         if (options) {
-            // If options exist, add isExecutingForOverlayDismiss to them
             const optionsObj = options.replace(/\s*}\s*$/, "");
             return `.${action}${openParen}${optionsObj}, isExecutingForOverlayDismiss: true }${closeParen}${semicolon || ""}`;
         }
         else {
-            // If no options, add them with isExecutingForOverlayDismiss
             return `.${action}${openParen}{ isExecutingForOverlayDismiss: true }${closeParen}${semicolon || ""}`;
         }
     });

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAIxC,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAG1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAK9C,wBAAgB,gBAAgB,CAAC,YAAY,EAAE,MAAM,GAAG,SAAS,WAKhE;AAED,qBAAa,mBAAmB;IAOlB,OAAO,CAAC,QAAQ,CAAC,IAAI;IANjC,OAAO,CAAC,iBAAiB,CAIhB;IAET,YAA6B,IAAI,EAAE,IAAI,EAAI;IAErC,WAAW,kBAIhB;IAEK,GAAG,CACP,aAAa,EAAE,GAAG,EAClB,QAAQ,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,IAAI,EACvC,MAAM,EAAE,MAAM,GACb,OAAO,CAAC,IAAI,CAAC,CAuDf;YAEa,QAAQ;CAqCvB;AAED,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,MAAM,GACnB,cAAc,GAAG,SAAS,CAiC5B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAIxC,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAG1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAK9C,wBAAgB,gBAAgB,CAAC,YAAY,EAAE,MAAM,GAAG,SAAS,WAKhE;AAED,qBAAa,mBAAmB;IAOlB,OAAO,CAAC,QAAQ,CAAC,IAAI;IANjC,OAAO,CAAC,iBAAiB,CAIhB;IAET,YAA6B,IAAI,EAAE,IAAI,EAAI;IAErC,WAAW,kBAIhB;IAEK,GAAG,CACP,aAAa,EAAE,GAAG,EAClB,QAAQ,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,IAAI,EACvC,MAAM,EAAE,MAAM,GACb,OAAO,CAAC,IAAI,CAAC,CAqDf;YAEa,QAAQ;CA+BvB;AAED,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,MAAM,GACnB,cAAc,GAAG,SAAS,CAiC5B"}

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/index.js

@@ -33,9 +33,7 @@ class OverlayDismissAgent {
         try {
             element = extractInterceptingElement(originalError.message);
         }
-        catch {
-            // Ignoring this error
-        }
+        catch { }
         const description = (0, utils_1.overlayDescription)(element) ?? "unknown";
         const testName = testFn.info().title;
         reporter(`Attempting to auto-dismiss overlay: ${description}`);
@@ -96,13 +94,7 @@ class OverlayDismissAgent {
             // only once in a particular test run.
             // Ran into this here: https://empirical.run/quizizz-tests/sessions/2825
             prefersElementFromPointCodegen: true,
-            // Overlay dismissal doesn't need page navigation — the overlay is
-            // already on the current page. Including page_goto causes the model
-            // to waste turns trying to navigate instead of clicking dismiss.
             includePageGotoTool: false,
-            // Route the model call through api-worker (Cloudflare AI Gateway) on the
-            // project API key, so test-run containers carry no OpenAI key. Undefined
-            // on a dev machine or in a pi sandbox — see the helper.
             openAIClientConfig: (0, api_worker_client_1.llmProxyOpenAIConfig)(),
         });
         this.cachingCandidates.push({

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/prompt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prompt.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/prompt.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,iBAAiB,CAAC;AAE5C,OAAO,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAGzC,wBAAsB,OAAO,CAC3B,IAAI,EAAE,IAAI,EACV,OAAO,EAAE,cAAc,GAAG,SAAS,EACnC,IAAI,EAAE,MAAM,GAAG,SAAS,mBAsCzB"}
+{"version":3,"file":"prompt.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/prompt.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,iBAAiB,CAAC;AAE5C,OAAO,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAGzC,wBAAsB,OAAO,CAC3B,IAAI,EAAE,IAAI,EACV,OAAO,EAAE,cAAc,GAAG,SAAS,EACnC,IAAI,EAAE,MAAM,GAAG,SAAS,mBAqCzB"}

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/prompt.js

@@ -13,7 +13,6 @@ ${text}
 `
         : "";
     if ((0, utils_1.isProductFruitsOverlay)(element)) {
-        // TODO: Remove this special handling with app knowledge
         promptAddition = promptForPFScenario(text);
     }
     let task = `

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/utils.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAE7C,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAE9C,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,cAAc,GAAG,SAAS,sBASrE;AAED,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,cAAc,GAAG,SAAS,uBAKzE;AAED,wBAAsB,WAAW,CAC/B,OAAO,EAAE,IAAI,EACb,OAAO,EAAE,cAAc,GAAG,SAAS,+BAmDpC;AAED,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,QAMpE;AAED,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,uDAiBrD"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/dismiss-overlays/utils.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAE7C,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,SAAS,CAAC;AAE9C,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,cAAc,GAAG,SAAS,sBASrE;AAED,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,cAAc,GAAG,SAAS,uBAKzE;AAED,wBAAsB,WAAW,CAC/B,OAAO,EAAE,IAAI,EACb,OAAO,EAAE,cAAc,GAAG,SAAS,+BAgDpC;AAED,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,QAMpE;AAED,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,uDAiBrD"}

package/dist/test/scripts/pw-locator-patch/dismiss-overlays/utils.js

@@ -29,7 +29,7 @@ async function textContent(pageRef, element) {
             let endTag;
             const match = startTag.match(/^<\s*([^\s>]+)/);
             if (match) {
-                const tagName = match[1]; // "div"
+                const tagName = match[1];
                 endTag = `</${tagName}>`;
             }
             if (!startTag || !endTag) {
@@ -41,18 +41,15 @@ async function textContent(pageRef, element) {
                 const parser = new DOMParser();
                 const doc = parser.parseFromString(htmlString, "text/html");
                 const tempEl = doc.body.firstElementChild;
-                let selector = tempEl.tagName.toLowerCase(); // start with the tag name, e.g., 'div'
-                // If there's an id, add it (id is unique and very specific)
+                let selector = tempEl.tagName.toLowerCase();
                 if (tempEl.id) {
                     selector += `#${CSS.escape(tempEl.id)}`;
                 }
-                // Add classes individually, escaping each one
                 if (tempEl.classList.length > 0) {
                     Array.from(tempEl.classList).forEach((cls) => {
                         selector += `.${CSS.escape(cls)}`;
                     });
                 }
-                // Add other attributes, being careful with escaping
                 Array.from(tempEl.attributes).forEach((attr) => {
                     if (attr.name === "id" || attr.name === "class") {
                         return;

package/dist/test/scripts/pw-locator-patch/utils/index.d.ts

@@ -1,4 +1,3 @@
 import { Locator } from "playwright-core";
 export declare function description(locator: Locator): string;
-export declare function scrollIntoView(locator: Locator): Promise<void>;
 //# sourceMappingURL=index.d.ts.map

package/dist/test/scripts/pw-locator-patch/utils/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/utils/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,iBAAiB,CAAC;AAK1C,wBAAgB,WAAW,CAAC,OAAO,EAAE,OAAO,UA0B3C;AAED,wBAAsB,cAAc,CAAC,OAAO,EAAE,OAAO,iBAgBpD"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/test/scripts/pw-locator-patch/utils/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,iBAAiB,CAAC;AAE1C,wBAAgB,WAAW,CAAC,OAAO,EAAE,OAAO,UA0B3C"}

package/dist/test/scripts/pw-locator-patch/utils/index.js

@@ -1,9 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.description = description;
-exports.scrollIntoView = scrollIntoView;
-const logger_1 = require("../../../../logger");
-const constants_1 = require("../../../constants");
 function description(locator) {
     const fallbackDescription = locator.toString();
     try {
@@ -25,18 +22,3 @@ function description(locator) {
         return fallbackDescription;
     }
 }
-async function scrollIntoView(locator) {
-    try {
-        await locator.evaluate((node) => {
-            if (node.scrollIntoViewIfNeeded) {
-                node.scrollIntoViewIfNeeded();
-            }
-            else {
-                node.scrollIntoView();
-            }
-        }, undefined, { timeout: constants_1.DEFAULT_SCRIPT_EXECUTION_TIMEOUT });
-    }
-    catch {
-        logger_1.logger.debug("Failed to scroll into view for locator method");
-    }
-}