@empiricalrun/playwright-utils 0.62.1 → 0.63.1

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"}

package/dist/reporter/harness.js

@@ -31,10 +31,6 @@ const PLAYWRIGHT_BIN = path_1.default.resolve(__dirname, "../../node_modules/.bi
 async function sleep(ms) {
     return new Promise((resolve) => setTimeout(resolve, ms));
 }
-/**
- * Sets up a temp directory with the IBR fixture files.
- * Returns a context object and a cleanup function.
- */
 async function setupFixture(suite) {
     const { tempDir, cleanup } = await setupReporterFixture({
         suite,
@@ -58,7 +54,6 @@ async function setupReporterFixture({ suite, reporterPath, tempPrefix, }) {
         const src = path_1.default.join(fixtureDir, entry.name);
         const dest = path_1.default.join(tempDir, entry.name);
         if (entry.isDirectory()) {
-            // Fixtures that put specs under a testDir subdir.
             await fs_1.default.promises.cp(src, dest, { recursive: true });
         }
         else if (entry.name === "playwright.config.ts") {
@@ -90,9 +85,6 @@ async function setupTestCaseSyncFixture(suite) {
         cleanup,
     };
 }
-/**
- * Cleans blob artifacts from a previous run inside the same temp dir.
- */
 async function cleanBlobArtifacts(ctx) {
     if (fs_1.default.existsSync(ctx.blobDir)) {
         await fs_1.default.promises.rm(ctx.blobDir, { recursive: true, force: true });
@@ -103,13 +95,6 @@ const WAIT_FOR_PATTERNS = {
     "first-failed": /failed|✘|×/,
 };
 const MARKER_RE = /@@ ([A-Z0-9-]+) @@/g;
-/**
- * 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)`.
- */
 async function runPlaywrightWithSignal(ctx, opts) {
     const { workers, waitForMarker, waitFor, delayBeforeSignalMs = 0, signal = "SIGINT", secondSignalAfterMs, killAfterMs, triggerTimeoutMs = 30_000, env, } = opts;
     const startedAt = Date.now();
@@ -162,9 +147,7 @@ async function runPlaywrightWithSignal(ctx, opts) {
         try {
             process.kill(-child.pid, sig);
         }
-        catch {
-            // The group is already gone.
-        }
+        catch { }
     };
     const pattern = waitFor ? WAIT_FOR_PATTERNS[waitFor] : undefined;
     const isTriggered = () => waitForMarker
@@ -199,7 +182,6 @@ async function runPlaywrightWithSignal(ctx, opts) {
     if (killAfterMs !== undefined) {
         timers.push(setTimeout(() => signalGroup("SIGKILL"), killAfterMs));
     }
-    // Backstop: never leave a runner behind if the suite misbehaves.
     timers.push(setTimeout(() => signalGroup("SIGKILL"), 120_000));
     try {
         await closed;
@@ -219,21 +201,13 @@ async function runPlaywrightWithSignal(ctx, opts) {
         startedAtEpochMs: startedAt,
     };
 }
-/**
- * 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.
- */
 async function runPlaywrightWithSigint(ctx, opts) {
     return runPlaywrightWithSignal(ctx, {
         workers: opts.workers,
         waitFor: opts.waitFor,
-        // Waiting for 1 sec to ensure second test has started
         delayBeforeSignalMs: 1_000,
     });
 }
-/** Marker names in the order they were printed. */
 function markerNames(result) {
     return result.markers.map((m) => m.name);
 }
@@ -262,15 +236,9 @@ async function runPlaywrightList(ctx) {
     }
     return { stdout, stderr };
 }
-/**
- * Reads `report.jsonl` from the blob zip and returns parsed events.
- */
 async function readBlobEvents(ctx) {
     return readEventsFromZip(ctx.zipPath);
 }
-/**
- * Reads `report.jsonl` from any blob-format zip (standard or incremental).
- */
 async function readEventsFromZip(zipPath) {
     const reportBuffer = await (0, zip_1.readZipEntry)(zipPath, "report.jsonl");
     if (!reportBuffer) {
@@ -282,10 +250,6 @@ async function readEventsFromZip(zipPath) {
         .split("\n")
         .map((line) => JSON.parse(line));
 }
-/**
- * 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.
- */
 async function setupParityFixture(suite) {
     const { tempDir, cleanup } = await setupReporterFixture({
         suite,
@@ -305,10 +269,6 @@ async function setupParityFixture(suite) {
         cleanup,
     };
 }
-/**
- * Spawns `playwright test` and lets it run to completion — the clean-finish
- * half of the matrix (the SIGINT helpers above cover interruption).
- */
 async function runPlaywrightClean(ctx, opts) {
     const args = ["test", `--workers=${opts.workers}`];
     if (opts.shard) {
@@ -319,8 +279,6 @@ async function runPlaywrightClean(ctx, opts) {
         env: {
             ...process.env,
             LOCAL_TEST: "false",
-            // With --shard, the env vars are cleared on purpose so IBR falls back to
-            // the same config.shard the built-in blob reporter sees.
             SHARD_INDEX: opts.shard ? undefined : "1",
             TOTAL_SHARDS: opts.shard ? undefined : "1",
             FORCE_COLOR: "0",
@@ -340,10 +298,6 @@ async function runPlaywrightClean(ctx, opts) {
     });
     return { stdout, stderr, exitCode };
 }
-/**
- * Runs `playwright merge-reports <blobDir> --reporter json` and returns the
- * parsed summary. Distinct output names let one temp dir hold both merges.
- */
 function mergeBlobDirToSummary(cwd, blobDir, outputName) {
     const outputPath = path_1.default.join(cwd, outputName);
     (0, child_process_1.execFileSync)(PLAYWRIGHT_BIN, ["merge-reports", blobDir, "--reporter", "json"], {
@@ -357,11 +311,6 @@ function mergeBlobDirToSummary(cwd, blobDir, outputName) {
     });
     return JSON.parse(fs_1.default.readFileSync(outputPath, "utf8"));
 }
-/**
- * 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.
- */
 async function startDashboardStub() {
     const events = [];
     const server = http_1.default.createServer((req, res) => {
@@ -379,7 +328,6 @@ async function startDashboardStub() {
                 res.end("{}");
                 return;
             }
-            // Everything else is the S3 client; r2-uploader always goes multipart.
             res.writeHead(200, { "content-type": "application/xml", etag: '"etag"' });
             if (req.method === "POST" && req.url?.includes("?uploads")) {
                 res.end("<InitiateMultipartUploadResult><UploadId>upload</UploadId></InitiateMultipartUploadResult>");

package/dist/reporter/ibr-utils.d.ts

@@ -43,11 +43,6 @@ export declare function relativeLocation(location: JsonLocation | undefined, roo
 export declare function relativeAnnotationLocations(annotations: TestAnnotation[], rootDir: string): TestAnnotation[];
 export declare function serializeTest(test: TestCase, rootDir: string): JsonTestCase;
 export declare function serializeSuite(suite: Suite, rootDir: string): JsonSuite;
-/**
- * Recursively filters a serialized suite tree to only include tests
- * whose testId is in the completedTestIds set. Removes empty parent suites.
- * Returns null if the suite has no completed test descendants.
- */
 export declare function filterSuiteByCompletedTests(suite: JsonSuite, completedTestIds: Set<string>): JsonSuite | null;
 export declare function hasValidReport(reportLines: string[]): boolean;
 export declare function buildUrlsJson(attachmentUrlMap: Map<string, string>): Record<string, string>;

package/dist/reporter/ibr-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ibr-utils.d.ts","sourceRoot":"","sources":["../../src/reporter/ibr-utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,2BAA2B,CAAC;AAO3E,MAAM,MAAM,WAAW,GAAG;IACxB,CAAC,CAAC,EAAE,MAAM,CAAC;IACX,CAAC,CAAC,EAAE;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;CACvC,CAAC;AAIF,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,CAAC,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,MAAM,GAAG,MAAM,CAAC,EAAE,GAChD,WAAW,EAAE,CAOf;AAED,wBAAgB,UAAU,IAAI,MAAM,CAEnC;AAED,wBAAgB,aAAa,CAAC,GAAG,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAO3E;AAED,wBAAgB,iBAAiB,CAC/B,GAAG,EAAE,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,EAC1B,IAAI,EAAE,QAAQ,GACb,MAAM,CAOR;AAID,KAAK,YAAY,GAAG;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAIF,MAAM,MAAM,YAAY,GAAG;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,YAAY,CAAC;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,WAAW,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CACvD,CAAC;AAIF,MAAM,MAAM,SAAS,GAAG;IACtB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,YAAY,CAAC;IACvB,OAAO,EAAE,CAAC,YAAY,GAAG,SAAS,CAAC,EAAE,CAAC;CACvC,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,YAAY,CAAC;CACzB,CAAC;AAMF,wBAAgB,iBAAiB,CAC/B,YAAY,EAAE,MAAM,GAAG,SAAS,EAChC,OAAO,EAAE,MAAM,GACd,MAAM,GAAG,SAAS,CAKpB;AAID,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,MAAM,GACd,YAAY,CAAC;AAChB,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,YAAY,GAAG,SAAS,EAClC,OAAO,EAAE,MAAM,GACd,YAAY,GAAG,SAAS,CAAC;AAa5B,wBAAgB,2BAA2B,CACzC,WAAW,EAAE,cAAc,EAAE,EAC7B,OAAO,EAAE,MAAM,GACd,cAAc,EAAE,CAKlB;AAID,wBAAgB,aAAa,CAAC,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,GAAG,YAAY,CAU3E;AAID,wBAAgB,cAAc,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,GAAG,SAAS,CAoBvE;AAMD;;;;GAIG;AACH,wBAAgB,2BAA2B,CACzC,KAAK,EAAE,SAAS,EAChB,gBAAgB,EAAE,GAAG,CAAC,MAAM,CAAC,GAC5B,SAAS,GAAG,IAAI,CAqBlB;AAED,wBAAgB,cAAc,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAK7D;AAED,wBAAgB,aAAa,CAC3B,gBAAgB,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GACpC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAMxB;AAED,wBAAgB,cAAc,IAAI,OAAO,CAExC;AAED,wBAAgB,cAAc,CAAC,YAAY,EAAE,MAAM,GAAG,MAAM,CAI3D;AAED,wBAAgB,eAAe,CAC7B,cAAc,EAAE,MAAM,GAAG,SAAS,EAClC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,MAAM,GAAG,IAAI,CAoBf"}
+{"version":3,"file":"ibr-utils.d.ts","sourceRoot":"","sources":["../../src/reporter/ibr-utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,QAAQ,EAAE,MAAM,2BAA2B,CAAC;AAO3E,MAAM,MAAM,WAAW,GAAG;IACxB,CAAC,CAAC,EAAE,MAAM,CAAC;IACX,CAAC,CAAC,EAAE;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;CACvC,CAAC;AAIF,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,CAAC,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,MAAM,GAAG,MAAM,CAAC,EAAE,GAChD,WAAW,EAAE,CAOf;AAED,wBAAgB,UAAU,IAAI,MAAM,CAEnC;AAED,wBAAgB,aAAa,CAAC,GAAG,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAO3E;AAED,wBAAgB,iBAAiB,CAC/B,GAAG,EAAE,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,EAC1B,IAAI,EAAE,QAAQ,GACb,MAAM,CAOR;AAID,KAAK,YAAY,GAAG;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAIF,MAAM,MAAM,YAAY,GAAG;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,YAAY,CAAC;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,eAAe,EAAE,MAAM,CAAC;IACxB,WAAW,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CACvD,CAAC;AAIF,MAAM,MAAM,SAAS,GAAG;IACtB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,YAAY,CAAC;IACvB,OAAO,EAAE,CAAC,YAAY,GAAG,SAAS,CAAC,EAAE,CAAC;CACvC,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,QAAQ,CAAC,EAAE,YAAY,CAAC;CACzB,CAAC;AAMF,wBAAgB,iBAAiB,CAC/B,YAAY,EAAE,MAAM,GAAG,SAAS,EAChC,OAAO,EAAE,MAAM,GACd,MAAM,GAAG,SAAS,CAKpB;AAID,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,YAAY,EACtB,OAAO,EAAE,MAAM,GACd,YAAY,CAAC;AAChB,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,YAAY,GAAG,SAAS,EAClC,OAAO,EAAE,MAAM,GACd,YAAY,GAAG,SAAS,CAAC;AAa5B,wBAAgB,2BAA2B,CACzC,WAAW,EAAE,cAAc,EAAE,EAC7B,OAAO,EAAE,MAAM,GACd,cAAc,EAAE,CAKlB;AAID,wBAAgB,aAAa,CAAC,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,GAAG,YAAY,CAU3E;AAID,wBAAgB,cAAc,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,GAAG,SAAS,CAoBvE;AAMD,wBAAgB,2BAA2B,CACzC,KAAK,EAAE,SAAS,EAChB,gBAAgB,EAAE,GAAG,CAAC,MAAM,CAAC,GAC5B,SAAS,GAAG,IAAI,CAqBlB;AAED,wBAAgB,cAAc,CAAC,WAAW,EAAE,MAAM,EAAE,GAAG,OAAO,CAK7D;AAED,wBAAgB,aAAa,CAC3B,gBAAgB,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GACpC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAMxB;AAED,wBAAgB,cAAc,IAAI,OAAO,CAExC;AAED,wBAAgB,cAAc,CAAC,YAAY,EAAE,MAAM,GAAG,MAAM,CAI3D;AAED,wBAAgB,eAAe,CAC7B,cAAc,EAAE,MAAM,GAAG,SAAS,EAClC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,MAAM,GAAG,IAAI,CAoBf"}

package/dist/reporter/ibr-utils.js

@@ -108,11 +108,6 @@ function serializeSuite(suite, rootDir) {
 function isTestEntry(entry) {
     return "testId" in entry;
 }
-/**
- * Recursively filters a serialized suite tree to only include tests
- * whose testId is in the completedTestIds set. Removes empty parent suites.
- * Returns null if the suite has no completed test descendants.
- */
 function filterSuiteByCompletedTests(suite, completedTestIds) {
     const filteredEntries = [];
     for (const entry of suite.entries) {

package/dist/reporter/incremental-blob-reporter.d.ts

@@ -1,12 +1,4 @@
 import type { FullConfig, FullResult, Reporter, Suite, TestCase, TestError, TestResult, TestStep, WorkerInfo } from "@playwright/test/reporter";
-/**
- * IncrementalBlobReporter
- *
- * This reporter builds a report.jsonl incrementally matching Playwright's blob format.
- * The zip can be uploaded for merge at any point (e.g., on SIGTERM).
- *
- * Format matches Playwright's blob reporter exactly.
- */
 export type AttachmentUrlMap = Map<string, string>;
 declare class IncrementalBlobReporter implements Reporter {
     private _currentWorkingDir;
@@ -33,12 +25,6 @@ declare class IncrementalBlobReporter implements Reporter {
     private _removeSignalHandler;
     private _buildSyntheticResult;
     private _finalizeAndMaybeUpload;
-    /**
-     * Computes the lines to write, without touching `_reportLines`: recording
-     * continues after the SIGINT salvage, and the onEnd pass re-finalizes from
-     * the full stream — mutating here would make the second pass work from the
-     * first pass's pruned leftovers.
-     */
     private _buildFinalReportLines;
     private _computeCompletedSets;
     private _removeIncompleteTestArtifacts;
@@ -60,9 +46,6 @@ declare class IncrementalBlobReporter implements Reporter {
     onStepBegin(test: TestCase, result: TestResult, step: TestStep): void;
     onStepEnd(test: TestCase, result: TestResult, step: TestStep): void;
     onEnd(result: FullResult): Promise<void>;
-    /**
-     * Add a single attachment URL mapping
-     */
     addAttachmentUrl(localPath: string, remoteUrl: string): void;
     printsToStdio(): boolean;
 }

package/dist/reporter/incremental-blob-reporter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"incremental-blob-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/incremental-blob-reporter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,UAAU,EACV,UAAU,EACV,QAAQ,EACR,KAAK,EACL,QAAQ,EACR,SAAS,EACT,UAAU,EACV,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AA8BnC;;;;;;;GAOG;AACH,MAAM,MAAM,gBAAgB,GAAG,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;AASnD,cAAM,uBAAwB,YAAW,QAAQ;IAC/C,OAAO,CAAC,kBAAkB,CAAyB;IACnD,OAAO,CAAC,WAAW,CAIjB;IACF,OAAO,CAAC,UAAU,CAGhB;IACF,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,YAAY,CAAgB;IACpC,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,YAAY,CAAkC;IACtD,OAAO,CAAC,UAAU,CAAoC;IACtD,OAAO,CAAC,sBAAsB,CAAkC;IAChE,OAAO,CAAC,SAAS,CAAyB;IAC1C,OAAO,CAAC,iBAAiB,CAA0B;IACnD,OAAO,CAAC,mBAAmB,CAA0B;IACrD,OAAO,CAAC,YAAY,CAAkC;IACtD,OAAO,CAAC,YAAY,CAAsD;IAC1E,OAAO,CAAC,UAAU,CAAsB;IACxC,OAAO,CAAC,gBAAgB,CAA8B;IACtD,OAAO,CAAC,QAAQ,CAAyB;IAEzC,cAWC;IAED,OAAO,CAAC,cAAc,CAA6B;IAEnD,OAAO,CAAC,mBAAmB;IA+B3B,OAAO,CAAC,oBAAoB;IAO5B,OAAO,CAAC,qBAAqB;YASf,uBAAuB;IAkFrC;;;;;OAKG;IACH,OAAO,CAAC,sBAAsB;IAuB9B,OAAO,CAAC,qBAAqB;IAoB7B,OAAO,CAAC,8BAA8B;IAWtC,OAAO,CAAC,mBAAmB;IAmB3B,OAAO,KAAK,QAAQ,GAKnB;IAED,OAAO,KAAK,WAAW,GAEtB;IAED,OAAO,KAAK,UAAU,GAErB;IAED,OAAO,KAAK,aAAa,GAExB;IAED,OAAO,CAAC,WAAW;IAKnB,OAAO,CAAC,YAAY;IASpB,OAAO,CAAC,aAAa;YAUP,SAAS;IAwBvB,OAAO,CAAC,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,GAAG,IAAI,CAgB9C;IAED,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,GAAG,IAAI,CAEpD;IAED,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,GAAG,IAAI,CA4BlD;IAED,QAAQ,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,IAAI,CAAC,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,UAAU,GAAG,IAAI,CAI3E;IAED,QAAQ,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,IAAI,CAAC,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,UAAU,GAAG,IAAI,CAI3E;IAED,OAAO,CAAC,KAAK,EAAE,SAAS,EAAE,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAEvD;IAED,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,EAAE,QAAQ,GAAG,IAAI,CAWpE;IAED,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,EAAE,QAAQ,GAAG,IAAI,CAalE;IAEK,KAAK,CAAC,MAAM,EAAE,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,CA+B7C;IAED;;OAEG;IACH,gBAAgB,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI,CAE3D;IAED,aAAa,IAAI,OAAO,CAEvB;CACF;AAID,wBAAgB,0BAA0B,IAAI,uBAAuB,GAAG,IAAI,CAE3E;AAED,wBAAgB,kCAAkC,CAChD,QAAQ,EAAE,uBAAuB,GAChC,IAAI,CAEN;eAEc,uBAAuB"}
+{"version":3,"file":"incremental-blob-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/incremental-blob-reporter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,UAAU,EACV,UAAU,EACV,QAAQ,EACR,KAAK,EACL,QAAQ,EACR,SAAS,EACT,UAAU,EACV,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AA8BnC,MAAM,MAAM,gBAAgB,GAAG,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;AAOnD,cAAM,uBAAwB,YAAW,QAAQ;IAC/C,OAAO,CAAC,kBAAkB,CAAyB;IACnD,OAAO,CAAC,WAAW,CAIjB;IACF,OAAO,CAAC,UAAU,CAGhB;IACF,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,YAAY,CAAgB;IACpC,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,YAAY,CAAkC;IACtD,OAAO,CAAC,UAAU,CAAoC;IACtD,OAAO,CAAC,sBAAsB,CAAkC;IAChE,OAAO,CAAC,SAAS,CAAyB;IAC1C,OAAO,CAAC,iBAAiB,CAA0B;IACnD,OAAO,CAAC,mBAAmB,CAA0B;IACrD,OAAO,CAAC,YAAY,CAAkC;IACtD,OAAO,CAAC,YAAY,CAAsD;IAC1E,OAAO,CAAC,UAAU,CAAsB;IACxC,OAAO,CAAC,gBAAgB,CAA8B;IACtD,OAAO,CAAC,QAAQ,CAAyB;IAEzC,cAWC;IAED,OAAO,CAAC,cAAc,CAA6B;IAEnD,OAAO,CAAC,mBAAmB;IA+B3B,OAAO,CAAC,oBAAoB;IAO5B,OAAO,CAAC,qBAAqB;YASf,uBAAuB;IAiFrC,OAAO,CAAC,sBAAsB;IAkB9B,OAAO,CAAC,qBAAqB;IAiB7B,OAAO,CAAC,8BAA8B;IAWtC,OAAO,CAAC,mBAAmB;IAmB3B,OAAO,KAAK,QAAQ,GAKnB;IAED,OAAO,KAAK,WAAW,GAEtB;IAED,OAAO,KAAK,UAAU,GAErB;IAED,OAAO,KAAK,aAAa,GAExB;IAED,OAAO,CAAC,WAAW;IAKnB,OAAO,CAAC,YAAY;IASpB,OAAO,CAAC,aAAa;YAUP,SAAS;IAqBvB,OAAO,CAAC,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,GAAG,IAAI,CAa9C;IAED,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,GAAG,IAAI,CAEpD;IAED,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,GAAG,IAAI,CA0BlD;IAED,QAAQ,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,IAAI,CAAC,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,UAAU,GAAG,IAAI,CAI3E;IAED,QAAQ,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,IAAI,CAAC,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,UAAU,GAAG,IAAI,CAI3E;IAED,OAAO,CAAC,KAAK,EAAE,SAAS,EAAE,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAEvD;IAED,WAAW,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,EAAE,QAAQ,GAAG,IAAI,CAWpE;IAED,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,EAAE,QAAQ,GAAG,IAAI,CAalE;IAEK,KAAK,CAAC,MAAM,EAAE,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,CA2B7C;IAED,gBAAgB,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI,CAE3D;IAED,aAAa,IAAI,OAAO,CAEvB;CACF;AAID,wBAAgB,0BAA0B,IAAI,uBAAuB,GAAG,IAAI,CAE3E;AAED,wBAAgB,kCAAkC,CAChD,QAAQ,EAAE,uBAAuB,GAChC,IAAI,CAEN;eAEc,uBAAuB"}

package/dist/reporter/incremental-blob-reporter.js

@@ -22,14 +22,14 @@ class IncrementalBlobReporter {
     _totalShards;
     _reportLines = [];
     _attachmentUrlMap = new Map();
-    _resultIdMap = new Map(); // testId+retry -> resultId
-    _stepIdMap = new Map(); // step -> stepId
-    _knownAttachmentCounts = new Map(); // resultId -> attachments already sent
+    _resultIdMap = new Map();
+    _stepIdMap = new Map();
+    _knownAttachmentCounts = new Map();
     _uploader = null;
     _completedTestIds = new Set();
     _completedResultIds = new Set();
-    _testRetries = new Map(); // testId → configured retries
-    _testResults = new Map(); // testId → retryIndex → RecordedRetry
+    _testRetries = new Map();
+    _testResults = new Map();
     _startTime = Date.now();
     _finalizePromise = null;
     _rootDir = process.cwd();
@@ -99,7 +99,6 @@ class IncrementalBlobReporter {
             console.warn("[IncrementalBlobReporterFailure] No onBegin received, skipping upload (nothing to merge)");
             return;
         }
-        // Wait for pending attachment uploads FIRST so URLs are available
         if (this._uploader) {
             await this._uploader.waitForUploads();
         }
@@ -127,33 +126,19 @@ class IncrementalBlobReporter {
         }
         logFinalizeComplete();
     }
-    /**
-     * Computes the lines to write, without touching `_reportLines`: recording
-     * continues after the SIGINT salvage, and the onEnd pass re-finalizes from
-     * the full stream — mutating here would make the second pass work from the
-     * first pass's pruned leftovers.
-     */
     _buildFinalReportLines(result, pruneIncompleteTests) {
         let lines = [...this._reportLines];
         if (pruneIncompleteTests) {
-            // Determine which tests are fully complete (passed or all retries finished)
             this._computeCompletedSets();
-            // Remove unrun tests from onProject suites
             lines = this._patchProjectSuites(lines);
-            // Remove events (onTestBegin, onStepBegin, etc.) for incomplete tests
             lines = this._removeIncompleteTestArtifacts(lines);
         }
-        // The onEnd event belongs only in the written report, never in the
-        // in-memory stream — a later finalize pass appends its own.
         lines.push(JSON.stringify((0, lifecycle_events_1.buildOnEndEvent)(result)));
         return lines;
     }
     _computeCompletedSets() {
         for (const [testId, results] of this._testResults) {
             const maxRetries = this._testRetries.get(testId);
-            // Shutdown artifacts are not finished retries. Counting them as
-            // complete would put their results in the final zip alongside the
-            // resume shard's, duplicating tests in the merge.
             const finished = new Map([...results].filter(([, r]) => !(0, reporter_state_1.isShutdownArtifact)(r.status, r.postSignal)));
             if ((0, reporter_1.areRetriesComplete)(maxRetries, finished)) {
                 this._completedTestIds.add(testId);
@@ -222,13 +207,10 @@ class IncrementalBlobReporter {
     }
     async _writeZip(lines) {
         this._ensureDirs();
-        // Write report.jsonl to the output directory
         const jsonlContent = lines.join("\n") + "\n";
         fs_1.default.writeFileSync(this._jsonlPath, jsonlContent, "utf8");
-        // Write _empirical_urls.json with attachment URL mappings
         const urlsJson = (0, ibr_utils_1.buildUrlsJson)(this._attachmentUrlMap);
         fs_1.default.writeFileSync(this._urlsJsonPath, JSON.stringify(urlsJson, null, 2), "utf8");
-        // Create zip from the staging directory
         const zipBuffer = await (0, zip_1.createZipFromDirectory)(this._stagingDir);
         fs_1.default.writeFileSync(this._zipPath, zipBuffer);
         console.log(`[IncrementalBlobReporter] Zip updated: ${this._zipPath} (${lines.length} events, ${this._attachmentUrlMap.size} attachments)`);
@@ -236,9 +218,6 @@ class IncrementalBlobReporter {
     onBegin(config, suite) {
         this._ensureDirs();
         this._rootDir = config.rootDir;
-        // The env vars win (production sets them), but a sharded run without
-        // them should still stamp the right shard into the blob metadata and
-        // the zip name instead of defaulting every shard to 1/1.
         if (!process.env.SHARD_INDEX && config.shard) {
             this._shardIndex = config.shard.current;
             this._totalShards = config.shard.total;
@@ -264,8 +243,6 @@ class IncrementalBlobReporter {
                     resultId,
                     status: result.status,
                     expectedStatus: test.expectedStatus,
-                    // The SIGINT salvage has started; this result arrived during
-                    // interruption shutdown rather than the normal run.
                     postSignal: this._finalizePromise !== null,
                 });
             }
@@ -288,10 +265,6 @@ class IncrementalBlobReporter {
     }
     async onEnd(result) {
         this._removeSignalHandler();
-        // Second phase of an interrupted run: the SIGINT pass salvaged a snapshot
-        // within milliseconds (SIGKILL insurance); by the time Playwright calls
-        // onEnd the teardown chain has finished, so re-finalize from the full
-        // stream and overwrite the salvage blob with the post-teardown report.
         if (this._finalizePromise) {
             await this._finalizePromise;
             this._finalizePromise = this._finalizeAndMaybeUpload({
@@ -314,9 +287,6 @@ class IncrementalBlobReporter {
         await this._finalizePromise;
         console.log(`[IncrementalBlobReporter] Finished with status: ${result.status}`);
     }
-    /**
-     * Add a single attachment URL mapping
-     */
     addAttachmentUrl(localPath, remoteUrl) {
         this._attachmentUrlMap.set((0, ibr_utils_1.toResourcePath)(localPath), remoteUrl);
     }