@empiricalrun/playwright-utils 0.62.0 → 0.63.0

package/dist/overlay-tests/cache.spec.js

@@ -13,7 +13,6 @@ const useCache = (data) => {
         version: "2025-03-06",
         data,
     };
-    // Ensure the directory exists
     const dir = path_1.default.dirname(cache_1.CACHE_FILE);
     if (!fs_1.default.existsSync(dir)) {
         fs_1.default.mkdirSync(dir, { recursive: true });
@@ -43,13 +42,11 @@ fixtures_1.test.beforeEach(async ({ page }) => {
     (0, test_1.injectLocatorHighlightScripts)(page, fixtures_1.test);
 });
 fixtures_1.test.afterEach(() => {
-    // Delete overlay cache
     if (fs_1.default.existsSync(cache_1.CACHE_FILE)) {
         fs_1.default.unlinkSync(cache_1.CACHE_FILE);
     }
 });
 (0, fixtures_1.test)("overlay dismiss should not trigger recursively when executing from cache", async ({ page, server, }) => {
-    // Set up cache with a bad value: click on "Target" to dimiss the glass pane
     useCache([
         {
             element: { dom: '<div class="glass-pane">', textContent: "" },
@@ -58,15 +55,8 @@ fixtures_1.test.afterEach(() => {
     ]);
     await page.goto(`${server.baseURL}/glass-pane.html`);
     let error;
-    // The only two routes the browser agent has to a model: the project API key
-    // (api-worker's proxy) and a sandbox's OPENAI_BASE_URL. Clearing both makes
-    // llmProxyOpenAIConfig throw before any request — which matters here more
-    // than elsewhere, because this is the recursive case and every extra
-    // dismissal would spawn another agent and another set of screenshots.
     const restoreEnv = clearEnv(["EMPIRICALRUN_API_KEY", "OPENAI_BASE_URL"]);
     try {
-        // Target element click will trigger the cache - which will invoke another
-        // click to the same element (recursive overlay dismissal)
         await page.getByRole("button", { name: "Target" }).click();
     }
     catch (e) {

package/dist/overlay-tests/click.spec.js

@@ -11,7 +11,6 @@ fixtures_1.test.beforeEach(async ({ page }) => {
     (0, test_1.injectLocatorHighlightScripts)(page, fixtures_1.test);
 });
 fixtures_1.test.afterAll(async () => {
-    // Delete overlay cache
     if (fs_1.default.existsSync(cache_1.CACHE_FILE)) {
         fs_1.default.unlinkSync(cache_1.CACHE_FILE);
     }
@@ -36,28 +35,20 @@ function getCacheDataWithTextContent({ textContent }) {
 }
 (0, fixtures_1.test)("should dismiss survicate for hover", async ({ page, server }) => {
     await page.goto(`${server.baseURL}/survey.html`);
-    // Assert that Survicate and button loads
     await (0, fixtures_1.expect)(page.getByText("Start survey")).toBeVisible();
     const targetButton = page.getByRole("button", { name: "Target" });
     await (0, fixtures_1.expect)(targetButton).toBeVisible();
-    // Get initial z-index
     const initialZIndex = await targetButton.evaluate((el) => window.getComputedStyle(el).getPropertyValue("z-index"));
-    // Do the hover, which should pass after survey overlay is dismissed
     await targetButton.hover();
-    // Assert z-index hasn't changed
     const finalZIndex = await targetButton.evaluate((el) => window.getComputedStyle(el).getPropertyValue("z-index"));
     (0, fixtures_1.expect)(finalZIndex).toBe(initialZIndex);
 });
-// Skipped since it's flaky
 fixtures_1.test.skip("should dismiss two-step overlay for click", async ({ page, server, }) => {
     await page.goto(`${server.baseURL}/tos.html`);
-    // Assert that button and overlay load
     await (0, fixtures_1.expect)(page.getByRole("button", { name: "Target" })).toBeVisible();
     await (0, fixtures_1.expect)(page.getByRole("heading", { name: "Terms of Service" })).toBeVisible();
-    // Do the click, which should pass
     await page.getByRole("button", { name: "Target" }).click();
     await (0, fixtures_1.expect)(page.getByRole("heading", { name: "Terms of Service" })).not.toBeVisible();
-    // Assert cache is correct
     const withDom = getCacheDataWithDom({
         dom: '<div class="terms-text">',
     });
@@ -75,7 +66,6 @@ fixtures_1.test.skip("should dismiss two-step overlay for click", async ({ page,
     await page.goto(`${server.baseURL}/overlay-form.html`);
     await page.getByRole("button", { name: "Target" }).click();
     await (0, fixtures_1.expect)(page.getByText("Target was clicked")).toBeVisible();
-    // Assert cache is correct
     const withDom = getCacheDataWithDom({
         dom: 'id="hearAboutUs"',
     });
@@ -92,16 +82,12 @@ fixtures_1.test.skip("should dismiss two-step overlay for click", async ({ page,
 });
 (0, fixtures_1.test)("should choose and dismiss the correct overlay", async ({ page, server, }) => {
     await page.goto(`${server.baseURL}/choose-an-overlay.html`);
-    // Assert that button and overlays load
     await (0, fixtures_1.expect)(page.getByRole("button", { name: "Target" })).toBeVisible();
     await (0, fixtures_1.expect)(page.getByText("This is a toast message")).toBeVisible();
     await (0, fixtures_1.expect)(page.getByRole("button", { name: "Close" })).toBeVisible();
-    // Do the click, which should pass
     await page.getByRole("button", { name: "Target" }).click();
-    // Assert correct overlay was dismissed
     await (0, fixtures_1.expect)(page.getByText("This is a toast message")).not.toBeVisible();
     await (0, fixtures_1.expect)(page.getByRole("button", { name: "Close" })).toBeVisible();
-    // Assert cache is correct
     const withDom = getCacheDataWithDom({
         dom: '<div id="toast" class="z-[100]">',
     });
@@ -115,19 +101,15 @@ fixtures_1.test.skip("should dismiss two-step overlay for click", async ({ page,
     (0, fixtures_1.expect)(code).toBeDefined();
     (0, fixtures_1.expect)(code).toContain("await page.getByRole('button', { name: '×' }).click()");
 });
-// Test is flaky - skipping it
 fixtures_1.test.skip("should fallback when the overlay cannot be dismissed", async ({ page, server, }) => {
     await page.goto(`${server.baseURL}/no-overlay.html`);
     let error;
     try {
-        // Click on element that is hidden below another button, which is not
-        // dismissible
         await page.getByRole("button", { name: "Bottom Button" }).click();
     }
     catch (e) {
         error = e;
     }
-    // TODO: add better assertions with result from master agent
     console.log(fixtures_1.test.info().annotations);
     (0, fixtures_1.expect)(error).toBeDefined();
     (0, fixtures_1.expect)(error.message).toContain('<button type="button" class="top-button">Top Button</button> intercepts pointer events');

package/dist/playwright-extensions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"playwright-extensions.d.ts","sourceRoot":"","sources":["../src/playwright-extensions.ts"],"names":[],"mappings":"AAKA,MAAM,MAAM,eAAe,GAAG;IAC5B,KAAK,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;CAC7C,CAAC;AAEF,OAAO,QAAQ,iBAAiB,CAAC;IAC/B,UAAU,OAAQ,SAAQ,eAAe;KAAG;CAC7C;AAED,OAAO,QAAQ,kBAAkB,CAAC;IAChC,UAAU,OAAQ,SAAQ,eAAe;KAAG;CAC7C"}
+{"version":3,"file":"playwright-extensions.d.ts","sourceRoot":"","sources":["../src/playwright-extensions.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,eAAe,GAAG;IAC5B,KAAK,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;CAC7C,CAAC;AAEF,OAAO,QAAQ,iBAAiB,CAAC;IAC/B,UAAU,OAAQ,SAAQ,eAAe;KAAG;CAC7C;AAED,OAAO,QAAQ,kBAAkB,CAAC;IAChC,UAAU,OAAQ,SAAQ,eAAe;KAAG;CAC7C"}

package/dist/playwright-extensions.js

@@ -1,3 +1,2 @@
 "use strict";
-// Extend Playwright core interfaces transparently to the end user.
 Object.defineProperty(exports, "__esModule", { value: true });

package/dist/reporter/attachment-cleanup-test-reporter.d.ts

@@ -1,9 +1,3 @@
-/**
- * Test-only reporter used by attachment-cleanup integration tests.
- *
- * Records a timestamped event log of register/markUploaded/eviction events
- * so tests can make precise assertions about ordering and timing.
- */
 import type { FullResult, Reporter, TestCase, TestResult } from "@playwright/test/reporter";
 declare class AttachmentCleanupTestReporter implements Reporter {
     private _timeline;

package/dist/reporter/attachment-cleanup-test-reporter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"attachment-cleanup-test-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/attachment-cleanup-test-reporter.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EACV,UAAU,EACV,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AA0CnC,cAAM,6BAA8B,YAAW,QAAQ;IACrD,OAAO,CAAC,SAAS,CAAuB;IACxC,OAAO,CAAC,QAAQ,CAA2B;IAC3C,OAAO,CAAC,mBAAmB,CAAgB;IAC3C,OAAO,CAAC,eAAe,CAAuB;IAC9C,OAAO,CAAC,MAAM,CAAkC;IAEhD,cAQC;IAED,SAAS,CAAC,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,QAyB5C;IAEK,KAAK,CAAC,OAAO,EAAE,UAAU,iBAyB9B;CACF;eAEc,6BAA6B"}
+{"version":3,"file":"attachment-cleanup-test-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/attachment-cleanup-test-reporter.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,UAAU,EACV,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AA0CnC,cAAM,6BAA8B,YAAW,QAAQ;IACrD,OAAO,CAAC,SAAS,CAAuB;IACxC,OAAO,CAAC,QAAQ,CAA2B;IAC3C,OAAO,CAAC,mBAAmB,CAAgB;IAC3C,OAAO,CAAC,eAAe,CAAuB;IAC9C,OAAO,CAAC,MAAM,CAAkC;IAEhD,cAQC;IAED,SAAS,CAAC,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,QAyB5C;IAEK,KAAK,CAAC,OAAO,EAAE,UAAU,iBAyB9B;CACF;eAEc,6BAA6B"}

package/dist/reporter/attachment-cleanup-test-reporter.js

@@ -1,10 +1,4 @@
 "use strict";
-/**
- * Test-only reporter used by attachment-cleanup integration tests.
- *
- * Records a timestamped event log of register/markUploaded/eviction events
- * so tests can make precise assertions about ordering and timing.
- */
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };

package/dist/reporter/attachment-cleanup.d.ts

@@ -1,28 +1,3 @@
-/**
- * Tracks test attachments and evicts the oldest uploaded ones from disk
- * when total size exceeds the configured limit.
- *
- * Two-phase tracking:
- * 1. register() — called immediately when an attachment is created.
- *    Adds the file to the total size accounting so the LRU knows about
- *    all disk usage, even while uploads are still queued.
- * 2. markUploaded() — called after the file is uploaded to R2.
- *    Marks the file as safe to evict (delete from disk).
- *
- * Eviction only deletes files that have been uploaded. This ensures we
- * never delete a file before its upload completes.
- *
- * Why this exists:
- * Test attachments (videos, traces, screenshots) accumulate on disk during a
- * test run. On Fargate workers with 50 GB ephemeral storage, large test suites
- * can exhaust disk space. Since attachments are already uploaded to R2, we can
- * safely delete them from disk — but we can't delete them immediately because
- * Playwright's blob reporter reads them during onEnd to embed as resources/.
- *
- * The blob reporter's statSync guard (Playwright 1.57+) gracefully skips
- * missing files, and patchBlobZip replaces resources/ with
- * _empirical_urls.json pointing to R2 URLs anyway.
- */
 export declare class AttachmentCleanup {
     private _pending;
     private _uploaded;
@@ -36,17 +11,9 @@ export declare class AttachmentCleanup {
     private _diskLogTick;
     private _onEvict;
     constructor(limitBytes?: number);
-    /**
-     * Phase 1: Register an attachment immediately when it's created.
-     * Adds to total size accounting and attempts eviction of already-uploaded files.
-     */
     register(attachment: {
         path?: string | undefined;
     }): void;
-    /**
-     * Phase 2: Mark an attachment as uploaded (safe to evict).
-     * Moves from pending to the uploaded queue and attempts eviction.
-     */
     markUploaded(attachmentPath: string): void;
     private _evict;
     private _logDiskStatus;

package/dist/reporter/attachment-cleanup.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"attachment-cleanup.d.ts","sourceRoot":"","sources":["../../src/reporter/attachment-cleanup.ts"],"names":[],"mappings":"AAgEA;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,qBAAa,iBAAiB;IAC5B,OAAO,CAAC,QAAQ,CAAkC;IAClD,OAAO,CAAC,SAAS,CAAwC;IACzD,OAAO,CAAC,UAAU,CAAK;IACvB,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,gBAAgB,CAAK;IAC7B,OAAO,CAAC,cAAc,CAAK;IAC3B,OAAO,CAAC,aAAa,CAAK;IAC1B,OAAO,CAAC,aAAa,CAAK;IAC1B,OAAO,CAAC,aAAa,CAA+C;IACpE,OAAO,CAAC,YAAY,CAAK;IACzB,OAAO,CAAC,QAAQ,CAAyC;IAEzD,YAAY,UAAU,GAAE,MAAmC,EAU1D;IAED;;;OAGG;IACH,QAAQ,CAAC,UAAU,EAAE;QAAE,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAA;KAAE,QAiBjD;IAED;;;OAGG;IACH,YAAY,CAAC,cAAc,EAAE,MAAM,QASlC;IAED,OAAO,CAAC,MAAM;IA4Bd,OAAO,CAAC,cAAc;IAStB,UAAU,SAaT;IAED,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,EAE3C;IAED,IAAI,KAAK;QAEL,eAAe;QACf,aAAa;QACb,YAAY;QACZ,YAAY;QACZ,YAAY;QACZ,WAAW;QACX,SAAS;QACT,UAAU;MAEb;CACF"}
+{"version":3,"file":"attachment-cleanup.d.ts","sourceRoot":"","sources":["../../src/reporter/attachment-cleanup.ts"],"names":[],"mappings":"AA6DA,qBAAa,iBAAiB;IAC5B,OAAO,CAAC,QAAQ,CAAkC;IAClD,OAAO,CAAC,SAAS,CAAwC;IACzD,OAAO,CAAC,UAAU,CAAK;IACvB,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,gBAAgB,CAAK;IAC7B,OAAO,CAAC,cAAc,CAAK;IAC3B,OAAO,CAAC,aAAa,CAAK;IAC1B,OAAO,CAAC,aAAa,CAAK;IAC1B,OAAO,CAAC,aAAa,CAA+C;IACpE,OAAO,CAAC,YAAY,CAAK;IACzB,OAAO,CAAC,QAAQ,CAAyC;IAEzD,YAAY,UAAU,GAAE,MAAmC,EAM1D;IAED,QAAQ,CAAC,UAAU,EAAE;QAAE,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAA;KAAE,QAejD;IAED,YAAY,CAAC,cAAc,EAAE,MAAM,QASlC;IAED,OAAO,CAAC,MAAM;IA4Bd,OAAO,CAAC,cAAc;IAItB,UAAU,SAWT;IAED,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,EAE3C;IAED,IAAI,KAAK;QAEL,eAAe;QACf,aAAa;QACb,YAAY;QACZ,YAAY;QACZ,YAAY;QACZ,WAAW;QACX,SAAS;QACT,UAAU;MAEb;CACF"}

package/dist/reporter/attachment-cleanup.js

@@ -16,7 +16,7 @@ function getDefaultDiskLimitBytes() {
         }
         logger_1.logger.warn(`[AttachmentCleanup] Invalid ATTACHMENT_CLEANUP_LIMIT_BYTES=${raw}, using default`);
     }
-    return 10 * 1024 * 1024 * 1024; // 10 GB
+    return 10 * 1024 * 1024 * 1024;
 }
 const DISK_LOG_INTERVAL_MS = 30_000;
 function formatBytes(bytes) {
@@ -39,12 +39,9 @@ function getDiskUsage() {
         return "unavailable";
     }
 }
-// Where the test-run worker checks out the test repo.
 const WORKER_REPO_DIR = "/app/source-repo";
 function getDiskBreakdown() {
     try {
-        // /tmp also holds browser profiles and download dirs that no attachment
-        // accounts for, so it is listed whole alongside its artifact dirs.
         const output = (0, child_process_1.execSync)("du -sh test-results/.playwright-artifacts-* test-results/ blob-report/ blob-report-incremental/ playwright-report/ /tmp/playwright-artifacts-* /tmp 2>/dev/null | sort -rh", { encoding: "utf8", timeout: 8000, cwd: WORKER_REPO_DIR }).trim();
         if (!output) {
             return "none";
@@ -58,31 +55,6 @@ function getDiskBreakdown() {
         return "none";
     }
 }
-/**
- * Tracks test attachments and evicts the oldest uploaded ones from disk
- * when total size exceeds the configured limit.
- *
- * Two-phase tracking:
- * 1. register() — called immediately when an attachment is created.
- *    Adds the file to the total size accounting so the LRU knows about
- *    all disk usage, even while uploads are still queued.
- * 2. markUploaded() — called after the file is uploaded to R2.
- *    Marks the file as safe to evict (delete from disk).
- *
- * Eviction only deletes files that have been uploaded. This ensures we
- * never delete a file before its upload completes.
- *
- * Why this exists:
- * Test attachments (videos, traces, screenshots) accumulate on disk during a
- * test run. On Fargate workers with 50 GB ephemeral storage, large test suites
- * can exhaust disk space. Since attachments are already uploaded to R2, we can
- * safely delete them from disk — but we can't delete them immediately because
- * Playwright's blob reporter reads them during onEnd to embed as resources/.
- *
- * The blob reporter's statSync guard (Playwright 1.57+) gracefully skips
- * missing files, and patchBlobZip replaces resources/ with
- * _empirical_urls.json pointing to R2 URLs anyway.
- */
 class AttachmentCleanup {
     _pending = new Map();
     _uploaded = [];
@@ -97,19 +69,11 @@ class AttachmentCleanup {
     _onEvict = null;
     constructor(limitBytes = getDefaultDiskLimitBytes()) {
         this._limitBytes = limitBytes;
-        // logger.debug(
-        //   `[AttachmentCleanup] Initialized with limit: ${formatBytes(limitBytes)}`,
-        // );
         this._diskLogTimer = setInterval(() => {
             this._logDiskStatus();
         }, DISK_LOG_INTERVAL_MS);
-        // Don't keep the process alive just for this timer
         this._diskLogTimer.unref();
     }
-    /**
-     * Phase 1: Register an attachment immediately when it's created.
-     * Adds to total size accounting and attempts eviction of already-uploaded files.
-     */
     register(attachment) {
         try {
             if (!attachment.path) {
@@ -125,14 +89,8 @@ class AttachmentCleanup {
             this._registeredCount++;
             this._evict();
         }
-        catch {
-            // Ignore errors (file may not exist)
-        }
+        catch { }
     }
-    /**
-     * Phase 2: Mark an attachment as uploaded (safe to evict).
-     * Moves from pending to the uploaded queue and attempts eviction.
-     */
     markUploaded(attachmentPath) {
         const size = this._pending.get(attachmentPath);
         if (size === undefined) {
@@ -170,19 +128,12 @@ class AttachmentCleanup {
     }
     _logDiskStatus() {
         this._diskLogTick++;
-        // const artifactsSuffix =
-        //   this._diskLogTick % 15 === 0 ? `, breakdown=${getDiskBreakdown()}` : "";
-        // logger.info(
-        //   `[AttachmentCleanup] Periodic: registered=${this._registeredCount}, pending=${this._pending.size}, uploaded=${this._uploadedCount}, evicted=${this._evictedCount} (${formatBytes(this._evictedBytes)}), totalSize=${formatBytes(this._totalSize)}, limit=${formatBytes(this._limitBytes)}, disk=${getDiskUsage()}${artifactsSuffix}`,
-        // );
     }
     logSummary() {
         if (this._diskLogTimer) {
             clearInterval(this._diskLogTimer);
             this._diskLogTimer = null;
         }
-        // One line per run, only on the worker: the only record of what filled the
-        // disk when a run hits ENOSPC. Local runs stay quiet.
         if (!fs_1.default.existsSync(WORKER_REPO_DIR)) {
             return;
         }

package/dist/reporter/empirical-reporter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"empirical-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/empirical-reporter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,UAAU,EACV,UAAU,EACV,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AAuBnC,cAAM,iBAAkB,YAAW,QAAQ;IACzC,OAAO,CAAC,kBAAkB,CAAyB;IACnD,OAAO,CAAC,oBAAoB,CAG1B;IACF,OAAO,CAAC,sBAAsB,CAAuB;IACrD,OAAO,CAAC,YAAY,CAAgD;IACpE,OAAO,CAAC,iBAAiB,CAAyC;IAClE,OAAO,CAAC,gBAAgB,CAAkC;IAC1D,OAAO,CAAC,SAAS,CAAyB;IAC1C,OAAO,CAAC,kBAAkB,CAA2B;IAGrD,OAAO,CAAC,YAAY,CAAqB;IAEzC,cAEC;IAED,OAAO,CAAC,MAAM,EAAE,UAAU,QAEzB;IAED,OAAO,CAAC,aAAa;IAUrB,OAAO,CAAC,6BAA6B,CA2CnC;IAEF,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,QAoJ3C;IAEK,KAAK,CAAC,MAAM,EAAE,UAAU,iBAoH7B;IAED,OAAO,CAAC,qBAAqB;YAkBf,gBAAgB;YAOhB,iBAAiB;CAqChC;eAEc,iBAAiB"}
+{"version":3,"file":"empirical-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/empirical-reporter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,UAAU,EACV,UAAU,EACV,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AAuBnC,cAAM,iBAAkB,YAAW,QAAQ;IACzC,OAAO,CAAC,kBAAkB,CAAyB;IACnD,OAAO,CAAC,oBAAoB,CAG1B;IACF,OAAO,CAAC,sBAAsB,CAAuB;IACrD,OAAO,CAAC,YAAY,CAAgD;IACpE,OAAO,CAAC,iBAAiB,CAAyC;IAClE,OAAO,CAAC,gBAAgB,CAAkC;IAC1D,OAAO,CAAC,SAAS,CAAyB;IAC1C,OAAO,CAAC,kBAAkB,CAA2B;IACrD,OAAO,CAAC,YAAY,CAAqB;IAEzC,cAEC;IAED,OAAO,CAAC,MAAM,EAAE,UAAU,QAEzB;IAED,OAAO,CAAC,aAAa;IAUrB,OAAO,CAAC,6BAA6B,CA2CnC;IAEF,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,QA+I3C;IAEK,KAAK,CAAC,MAAM,EAAE,UAAU,iBAkH7B;IAED,OAAO,CAAC,qBAAqB;YAkBf,gBAAgB;YAOhB,iBAAiB;CAqChC;eAEc,iBAAiB"}

package/dist/reporter/empirical-reporter.js

@@ -23,8 +23,6 @@ class EmpiricalReporter {
     _testRetryCounts = new Map();
     _uploader = null;
     _attachmentCleanup = new attachment_cleanup_1.AttachmentCleanup();
-    // Resolved Playwright maxFailures (0 = no cap), sent on every run-end event
-    // so the server-side early-end enforces the client's configured cap.
     _maxFailures;
     constructor() {
         this._uploader = (0, uploader_1.getUploader)();
@@ -62,11 +60,6 @@ class EmpiricalReporter {
         }
     };
     onTestEnd(test, result) {
-        // After the SIGINT salvage, an attempt that really finished (its body
-        // completed, only its teardown overlapped the signal) is still reported:
-        // the blob keeps it, and a ledger that misses it calls the test unfinished,
-        // so the resume shard re-runs it and the merged report holds it twice.
-        // Shutdown artifacts stay out, as they do in the blob.
         if ((0, reporter_state_1.isFinalized)() && (0, reporter_state_1.isShutdownArtifact)(result.status, true)) {
             return;
         }
@@ -186,8 +179,6 @@ class EmpiricalReporter {
     }
     async onEnd(result) {
         if ((0, reporter_state_1.isFinalized)()) {
-            // The salvage owns the report upload. Only flush the test-case events
-            // sent during shutdown, so they land before the process exits.
             await Promise.allSettled(this._pendingTestCaseEvents);
             return;
         }

package/dist/reporter/harness.d.ts

@@ -8,10 +8,6 @@ export interface TestCaseSyncTestContext {
     outputPath: string;
 }
 export declare function sleep(ms: number): Promise<void>;
-/**
- * Sets up a temp directory with the IBR fixture files.
- * Returns a context object and a cleanup function.
- */
 export declare function setupFixture(suite: string): Promise<{
     ctx: IBRTestContext;
     cleanup: () => Promise<void>;
@@ -20,16 +16,12 @@ export declare function setupTestCaseSyncFixture(suite: string): Promise<{
     ctx: TestCaseSyncTestContext;
     cleanup: () => Promise<void>;
 }>;
-/**
- * Cleans blob artifacts from a previous run inside the same temp dir.
- */
 export declare function cleanBlobArtifacts(ctx: IBRTestContext): Promise<void>;
 export interface RunResult {
     stdout: string;
     stderr: string;
 }
 export type WaitFor = "first-passed" | "first-failed";
-/** A `@@ NAME @@` line printed by a fixture suite, with its offset from run start. */
 export interface Marker {
     name: string;
     atMs: number;
@@ -38,79 +30,40 @@ export interface SignalRunResult extends RunResult {
     exitCode: number | null;
     exitSignal: NodeJS.Signals | null;
     markers: Marker[];
-    /** Offset from run start at which the first signal was delivered. */
     signalSentAtMs: number;
-    /** Wall clock at run start, so `atMs` offsets can be compared to file mtimes. */
     startedAtEpochMs: number;
 }
 export interface SignalRunOptions {
     workers: number;
-    /**
-     * Signal as soon as this `@@ MARKER @@` is printed. Preferred over `waitFor`:
-     * it pins the signal to a known point in the run instead of a sleep.
-     */
     waitForMarker?: string;
-    /** Legacy trigger: wait for the first pass/fail line in stdout. */
     waitFor?: WaitFor;
-    /** Settling delay between the trigger and the signal. */
     delayBeforeSignalMs?: number;
-    /** Defaults to SIGINT, which is what production sends. */
     signal?: NodeJS.Signals;
-    /** Send the same signal again this long after the first one. */
     secondSignalAfterMs?: number;
-    /** SIGKILL the group this long after the first signal, like the cmd.ts grace cap. */
     killAfterMs?: number;
-    /** How long to wait for the trigger before giving up and signalling anyway. */
     triggerTimeoutMs?: number;
-    /** Extra env for the runner, e.g. from `startDashboardStub`. */
     env?: NodeJS.ProcessEnv;
 }
-/**
- * Spawns `playwright test` with the IBR reporter in its own process group,
- * signals the group mid-run, and waits for exit.
- *
- * The process group and the signal both match production: `cmd.ts` spawns the
- * runner detached and does `process.kill(-pid, signal)`.
- */
 export declare function runPlaywrightWithSignal(ctx: IBRTestContext, opts: SignalRunOptions): Promise<SignalRunResult>;
-/**
- * Spawns `playwright test` with the IBR reporter, waits for a test outcome,
- * sends SIGINT, and waits for the process to exit.
- *
- * Returns captured stdout/stderr.
- */
 export declare function runPlaywrightWithSigint(ctx: IBRTestContext, opts: {
     workers: number;
     waitFor: WaitFor;
 }): Promise<RunResult>;
-/** Marker names in the order they were printed. */
 export declare function markerNames(result: SignalRunResult): string[];
 export declare function runPlaywrightList(ctx: TestCaseSyncTestContext): Promise<RunResult>;
 export interface BlobEvent {
     method: string;
     params?: Record<string, unknown>;
 }
-/**
- * Reads `report.jsonl` from the blob zip and returns parsed events.
- */
 export declare function readBlobEvents(ctx: IBRTestContext): Promise<BlobEvent[]>;
-/**
- * Reads `report.jsonl` from any blob-format zip (standard or incremental).
- */
 export declare function readEventsFromZip(zipPath: string): Promise<BlobEvent[]>;
 export interface ParityTestContext {
     tempDir: string;
-    /** Playwright's built-in blob reporter output: blob-report/report-1.zip */
     standardZipPath: string;
     standardBlobDir: string;
-    /** IBR output: blob-report-incremental/incremental-report-1.zip */
     incrementalZipPath: string;
     incrementalBlobDir: string;
 }
-/**
- * Sets up a fixture whose config runs BOTH the built-in blob reporter and the
- * IBR on the same invocation, so one event stream feeds both zips.
- */
 export declare function setupParityFixture(suite: string): Promise<{
     ctx: ParityTestContext;
     cleanup: () => Promise<void>;
@@ -118,10 +71,6 @@ export declare function setupParityFixture(suite: string): Promise<{
 export interface CleanRunResult extends RunResult {
     exitCode: number | null;
 }
-/**
- * Spawns `playwright test` and lets it run to completion — the clean-finish
- * half of the matrix (the SIGINT helpers above cover interruption).
- */
 export declare function runPlaywrightClean(ctx: ParityTestContext, opts: {
     workers: number;
     shard?: {
@@ -129,22 +78,11 @@ export declare function runPlaywrightClean(ctx: ParityTestContext, opts: {
         total: number;
     };
 }): Promise<CleanRunResult>;
-/**
- * Runs `playwright merge-reports <blobDir> --reporter json` and returns the
- * parsed summary. Distinct output names let one temp dir hold both merges.
- */
 export declare function mergeBlobDirToSummary(cwd: string, blobDir: string, outputName: string): Record<string, unknown>;
 export interface DashboardStub {
-    /** `test_case_run_end` payloads EmpiricalReporter sent, in arrival order. */
     testCaseEvents: () => Array<Record<string, any>>;
-    /** Runner env that points EmpiricalReporter's API calls and uploads here. */
     env: NodeJS.ProcessEnv;
     close: () => Promise<void>;
 }
-/**
- * A local stand-in for api-worker and the report bucket, so EmpiricalReporter
- * runs unmodified inside a real Playwright run. Records test-case events and
- * accepts every upload.
- */
 export declare function startDashboardStub(): Promise<DashboardStub>;
 //# sourceMappingURL=harness.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"harness.d.ts","sourceRoot":"","sources":["../../src/reporter/harness.ts"],"names":[],"mappings":"AA0BA,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,uBAAuB;IACtC,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,wBAAsB,KAAK,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAErD;AAED;;;GAGG;AACH,wBAAsB,YAAY,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IACzD,GAAG,EAAE,cAAc,CAAC;IACpB,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAaD;AA4CD,wBAAsB,wBAAwB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IACrE,GAAG,EAAE,uBAAuB,CAAC;IAC7B,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAaD;AAED;;GAEG;AACH,wBAAsB,kBAAkB,CAAC,GAAG,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAI3E;AAED,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,MAAM,OAAO,GAAG,cAAc,GAAG,cAAc,CAAC;AAOtD,sFAAsF;AACtF,MAAM,WAAW,MAAM;IACrB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;CACd;AAED,MAAM,WAAW,eAAgB,SAAQ,SAAS;IAChD,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,CAAC,OAAO,GAAG,IAAI,CAAC;IAClC,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,qEAAqE;IACrE,cAAc,EAAE,MAAM,CAAC;IACvB,iFAAiF;IACjF,gBAAgB,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,MAAM,CAAC;IAChB;;;OAGG;IACH,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,mEAAmE;IACnE,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,yDAAyD;IACzD,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,0DAA0D;IAC1D,MAAM,CAAC,EAAE,MAAM,CAAC,OAAO,CAAC;IACxB,gEAAgE;IAChE,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,qFAAqF;IACrF,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,+EAA+E;IAC/E,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,gEAAgE;IAChE,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;CACzB;AAID;;;;;;GAMG;AACH,wBAAsB,uBAAuB,CAC3C,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,gBAAgB,GACrB,OAAO,CAAC,eAAe,CAAC,CAgI1B;AAED;;;;;GAKG;AACH,wBAAsB,uBAAuB,CAC3C,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,GAC1C,OAAO,CAAC,SAAS,CAAC,CAOpB;AAED,mDAAmD;AACnD,wBAAgB,WAAW,CAAC,MAAM,EAAE,eAAe,GAAG,MAAM,EAAE,CAE7D;AAED,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,uBAAuB,GAC3B,OAAO,CAAC,SAAS,CAAC,CA6BpB;AAED,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CAClC;AAED;;GAEG;AACH,wBAAsB,cAAc,CAClC,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,SAAS,EAAE,CAAC,CAEtB;AAED;;GAEG;AACH,wBAAsB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC,CAU7E;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,2EAA2E;IAC3E,eAAe,EAAE,MAAM,CAAC;IACxB,eAAe,EAAE,MAAM,CAAC;IACxB,mEAAmE;IACnE,kBAAkB,EAAE,MAAM,CAAC;IAC3B,kBAAkB,EAAE,MAAM,CAAC;CAC5B;AAED;;;GAGG;AACH,wBAAsB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IAC/D,GAAG,EAAE,iBAAiB,CAAC;IACvB,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAqBD;AAED,MAAM,WAAW,cAAe,SAAQ,SAAS;IAC/C,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;CACzB;AAED;;;GAGG;AACH,wBAAsB,kBAAkB,CACtC,GAAG,EAAE,iBAAiB,EACtB,IAAI,EAAE;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAA;CAAE,GACpE,OAAO,CAAC,cAAc,CAAC,CAgCzB;AAED;;;GAGG;AACH,wBAAgB,qBAAqB,CACnC,GAAG,EAAE,MAAM,EACX,OAAO,EAAE,MAAM,EACf,UAAU,EAAE,MAAM,GACjB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAgBzB;AAED,MAAM,WAAW,aAAa;IAC5B,6EAA6E;IAC7E,cAAc,EAAE,MAAM,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,CAAC;IACjD,6EAA6E;IAC7E,GAAG,EAAE,MAAM,CAAC,UAAU,CAAC;IACvB,KAAK,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC5B;AAED;;;;GAIG;AACH,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,aAAa,CAAC,CAyDjE"}
+{"version":3,"file":"harness.d.ts","sourceRoot":"","sources":["../../src/reporter/harness.ts"],"names":[],"mappings":"AA0BA,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,uBAAuB;IACtC,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,wBAAsB,KAAK,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAErD;AAED,wBAAsB,YAAY,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IACzD,GAAG,EAAE,cAAc,CAAC;IACpB,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAaD;AA2CD,wBAAsB,wBAAwB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IACrE,GAAG,EAAE,uBAAuB,CAAC;IAC7B,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAaD;AAED,wBAAsB,kBAAkB,CAAC,GAAG,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAI3E;AAED,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,MAAM,OAAO,GAAG,cAAc,GAAG,cAAc,CAAC;AAOtD,MAAM,WAAW,MAAM;IACrB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;CACd;AAED,MAAM,WAAW,eAAgB,SAAQ,SAAS;IAChD,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,CAAC,OAAO,GAAG,IAAI,CAAC;IAClC,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,cAAc,EAAE,MAAM,CAAC;IACvB,gBAAgB,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,MAAM,CAAC,EAAE,MAAM,CAAC,OAAO,CAAC;IACxB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;CACzB;AAID,wBAAsB,uBAAuB,CAC3C,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,gBAAgB,GACrB,OAAO,CAAC,eAAe,CAAC,CA6H1B;AAED,wBAAsB,uBAAuB,CAC3C,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,GAC1C,OAAO,CAAC,SAAS,CAAC,CAMpB;AAED,wBAAgB,WAAW,CAAC,MAAM,EAAE,eAAe,GAAG,MAAM,EAAE,CAE7D;AAED,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,uBAAuB,GAC3B,OAAO,CAAC,SAAS,CAAC,CA6BpB;AAED,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CAClC;AAED,wBAAsB,cAAc,CAClC,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,SAAS,EAAE,CAAC,CAEtB;AAED,wBAAsB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC,CAU7E;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,eAAe,EAAE,MAAM,CAAC;IACxB,eAAe,EAAE,MAAM,CAAC;IACxB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,kBAAkB,EAAE,MAAM,CAAC;CAC5B;AAED,wBAAsB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC;IAC/D,GAAG,EAAE,iBAAiB,CAAC;IACvB,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC,CAqBD;AAED,MAAM,WAAW,cAAe,SAAQ,SAAS;IAC/C,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;CACzB;AAED,wBAAsB,kBAAkB,CACtC,GAAG,EAAE,iBAAiB,EACtB,IAAI,EAAE;IAAE,OAAO,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAA;CAAE,GACpE,OAAO,CAAC,cAAc,CAAC,CA8BzB;AAED,wBAAgB,qBAAqB,CACnC,GAAG,EAAE,MAAM,EACX,OAAO,EAAE,MAAM,EACf,UAAU,EAAE,MAAM,GACjB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAgBzB;AAED,MAAM,WAAW,aAAa;IAC5B,cAAc,EAAE,MAAM,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,CAAC;IACjD,GAAG,EAAE,MAAM,CAAC,UAAU,CAAC;IACvB,KAAK,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC5B;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,aAAa,CAAC,CAwDjE"}