@empiricalrun/playwright-utils 0.38.6 → 0.39.1

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @empiricalrun/playwright-utils
 
+## 0.39.1
+
+### Patch Changes
+
+- 36fc057: chore: rename telemetry fields for consistency
+- Updated dependencies [92254ba]
+  - @empiricalrun/r2-uploader@0.9.1
+  - @empiricalrun/test-gen@0.79.3
+
+## 0.39.0
+
+### Minor Changes
+
+- d1aa88c: chore: remove hardcoded asset bucket domain
+
+### Patch Changes
+
+- 5c8ac32: chore: more verbose logs for blob uploads
+  - @empiricalrun/test-gen@0.79.2
+
 ## 0.38.6
 
 ### Patch Changes

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

@@ -45,7 +45,8 @@ function getCacheDataWithTextContent({ textContent }) {
     const finalZIndex = await targetButton.evaluate((el) => window.getComputedStyle(el).getPropertyValue("z-index"));
     (0, fixtures_1.expect)(finalZIndex).toBe(initialZIndex);
 });
-(0, fixtures_1.test)("should dismiss two-step overlay for click", async ({ page, server }) => {
+// 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();

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,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AAkBnC,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;;IAM1C,OAAO,CAAC,6BAA6B,CAkCnC;IAEF,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAoFtC,KAAK,CAAC,MAAM,EAAE,UAAU;IAuG9B,OAAO,CAAC,gBAAgB;YAoBV,gBAAgB;YAOhB,iBAAiB;CAMhC;AAED,eAAe,iBAAiB,CAAC"}
+{"version":3,"file":"empirical-reporter.d.ts","sourceRoot":"","sources":["../../src/reporter/empirical-reporter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,UAAU,EACV,QAAQ,EACR,QAAQ,EACR,UAAU,EACX,MAAM,2BAA2B,CAAC;AAkBnC,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;;IAM1C,OAAO,CAAC,6BAA6B,CAkCnC;IAEF,SAAS,CAAC,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,UAAU;IAoFtC,KAAK,CAAC,MAAM,EAAE,UAAU;IA8G9B,OAAO,CAAC,gBAAgB;YAoBV,gBAAgB;YAOhB,iBAAiB;CAchC;AAED,eAAe,iBAAiB,CAAC"}

package/dist/reporter/empirical-reporter.js

@@ -156,14 +156,18 @@ class EmpiricalReporter {
         await Promise.allSettled(this._pendingTestCaseEvents);
         if (this._hasSharding) {
             const blobDir = path_1.default.join(this._currentWorkingDir, "blob-report");
+            logger_1.logger.info(`[Empirical Reporter] Sharding enabled, checking for blob-report at: ${blobDir}`);
             if (fs_1.default.existsSync(blobDir)) {
                 await this._patchBlobReport(blobDir);
                 await this._uploadBlobReport(blobDir);
             }
             else {
-                logger_1.logger.debug(`[Empirical Reporter] blob-report does not exist at: ${blobDir}`);
+                logger_1.logger.warn(`[Empirical Reporter] blob-report does not exist at: ${blobDir}`);
             }
         }
+        else {
+            logger_1.logger.info(`[Empirical Reporter] Sharding not enabled (HAS_SHARDING=${process.env.HAS_SHARDING})`);
+        }
         logger_1.logger.debug("[Empirical Reporter] All uploads finished");
         const timeDiff = Date.now() - startTime;
         logger_1.logger.debug("[Empirical Reporter] Time taken to upload after tests ended: ", timeDiff, "ms");
@@ -194,9 +198,16 @@ class EmpiricalReporter {
     async _uploadBlobReport(blobDir) {
         if (!this._uploader)
             return;
-        await this._uploader.uploadDirectory(blobDir, "blobs");
-        await this._uploader.waitForUploads();
-        logger_1.logger.debug("[Empirical Reporter] Blob report uploaded");
+        try {
+            const files = await fs_1.default.promises.readdir(blobDir);
+            logger_1.logger.info(`[Empirical Reporter] Uploading blob report from ${blobDir}, files: ${files.join(", ")}`);
+            await this._uploader.uploadDirectory(blobDir, "blobs");
+            await this._uploader.waitForUploads();
+            logger_1.logger.info("[Empirical Reporter] Blob report uploaded successfully");
+        }
+        catch (error) {
+            logger_1.logger.error("[Empirical Reporter] Failed to upload blob report:", error);
+        }
     }
 }
 exports.default = EmpiricalReporter;

package/dist/reporter/uploader.d.ts

@@ -2,8 +2,8 @@ import { StorageCredentials } from "@empiricalrun/r2-uploader";
 export interface UploaderConfig {
     projectName: string;
     runId: string;
-    baseUrl?: string;
-    uploadBucket?: string;
+    baseUrl: string;
+    uploadBucket: string;
     credentials: StorageCredentials;
 }
 export declare class Uploader {

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

@@ -1 +1 @@
-{"version":3,"file":"uploader.d.ts","sourceRoot":"","sources":["../../src/reporter/uploader.ts"],"names":[],"mappings":"AAAA,OAAO,EAEL,kBAAkB,EAGnB,MAAM,2BAA2B,CAAC;AAKnC,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,kBAAkB,CAAC;CACjC;AAED,qBAAa,QAAQ;IACnB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,eAAe,CAAS;IAChC,OAAO,CAAC,YAAY,CAAqB;gBAE7B,MAAM,EAAE,cAAc;IAWlC,OAAO,CAAC,MAAM,CAAC,iBAAiB;IAgBhC,OAAO,CAAC,MAAM,CAAC,iBAAiB;IAUhC,MAAM,CAAC,qBAAqB,CAAC,QAAQ,EAAE,OAAO,GAAG,kBAAkB,GAAG,IAAI;IAO1E,IAAI,SAAS,IAAI,OAAO,CAEvB;IAED,IAAI,OAAO,IAAI,MAAM,CAEpB;IAED,IAAI,cAAc,IAAI,MAAM,CAE3B;IAED,IAAI,SAAS,IAAI,MAAM,CAEtB;IAEK,UAAU,CACd,QAAQ,EAAE,MAAM,EAChB,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IA6BnB,eAAe,CACnB,SAAS,EAAE,MAAM,EACjB,iBAAiB,EAAE,MAAM,GACxB,OAAO,CAAC,IAAI,CAAC;IAiBV,cAAc,IAAI,OAAO,CAAC,IAAI,CAAC;CAGtC;AAED,wBAAgB,cAAc,IAAI,QAAQ,GAAG,IAAI,CAyBhD"}
+{"version":3,"file":"uploader.d.ts","sourceRoot":"","sources":["../../src/reporter/uploader.ts"],"names":[],"mappings":"AAAA,OAAO,EAEL,kBAAkB,EAGnB,MAAM,2BAA2B,CAAC;AAKnC,MAAM,WAAW,cAAc;IAC7B,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,kBAAkB,CAAC;CACjC;AAED,qBAAa,QAAQ;IACnB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,eAAe,CAAS;IAChC,OAAO,CAAC,YAAY,CAAqB;gBAE7B,MAAM,EAAE,cAAc;IAWlC,OAAO,CAAC,MAAM,CAAC,iBAAiB;IAgBhC,OAAO,CAAC,MAAM,CAAC,iBAAiB;IAUhC,MAAM,CAAC,qBAAqB,CAAC,QAAQ,EAAE,OAAO,GAAG,kBAAkB,GAAG,IAAI;IAO1E,IAAI,SAAS,IAAI,OAAO,CAEvB;IAED,IAAI,OAAO,IAAI,MAAM,CAEpB;IAED,IAAI,cAAc,IAAI,MAAM,CAE3B;IAED,IAAI,SAAS,IAAI,MAAM,CAEtB;IAEK,UAAU,CACd,QAAQ,EAAE,MAAM,EAChB,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IA6BnB,eAAe,CACnB,SAAS,EAAE,MAAM,EACjB,iBAAiB,EAAE,MAAM,GACxB,OAAO,CAAC,IAAI,CAAC;IAiBV,cAAc,IAAI,OAAO,CAAC,IAAI,CAAC;CAGtC;AAED,wBAAgB,cAAc,IAAI,QAAQ,GAAG,IAAI,CA+ChD"}

package/dist/reporter/uploader.js

@@ -14,8 +14,8 @@ class Uploader {
     _destinationDir;
     _credentials;
     constructor(config) {
-        this._baseUrl = config.baseUrl || "https://reports.empirical.run";
-        this._uploadBucket = config.uploadBucket || "test-report";
+        this._baseUrl = config.baseUrl;
+        this._uploadBucket = config.uploadBucket;
         this._destinationDir = `${config.projectName}/${config.runId}`;
         this._credentials = config.credentials;
         logger_1.logger.debug(`[Uploader] Initialized with destination: ${this._destinationDir}, provider: ${this._credentials.provider}`);
@@ -112,10 +112,32 @@ function createUploader() {
         logger_1.logger.debug(`[Uploader] ${storageType} credentials not found`);
         return null;
     }
-    const uploadBucket = enableS3 ? process.env.S3_UPLOAD_BUCKET : "test-report";
+    let baseUrl;
+    let uploadBucket;
+    if (enableS3) {
+        const region = process.env.S3_REGION;
+        const bucket = process.env.S3_UPLOAD_BUCKET;
+        if (!region || !bucket) {
+            logger_1.logger.debug("[Uploader] S3_REGION or S3_UPLOAD_BUCKET not configured");
+            return null;
+        }
+        baseUrl = `https://${bucket}.s3.${region}.amazonaws.com`;
+        uploadBucket = bucket;
+    }
+    else {
+        const accountId = process.env.R2_ACCOUNT_ID;
+        const bucket = "test-report";
+        if (!accountId) {
+            logger_1.logger.debug("[Uploader] R2_ACCOUNT_ID not configured");
+            return null;
+        }
+        baseUrl = `https://reports-r2.empirical.run`;
+        uploadBucket = bucket;
+    }
     return new Uploader({
         projectName: process.env.PROJECT_NAME,
         runId: process.env.TEST_RUN_GITHUB_ACTION_ID,
+        baseUrl,
         uploadBucket,
         credentials,
     });

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;AAKtE,wBAAsB,UAAU,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAqBrE;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,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,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,IAAI,CAoBP"}
+{"version":3,"file":"telemetry.d.ts","sourceRoot":"","sources":["../src/telemetry.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,sCAAsC,CAAC;AAKtE,wBAAsB,UAAU,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAqBrE;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,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,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,IAAI,CAqBP"}

package/dist/telemetry.js

@@ -36,7 +36,8 @@ function trackOverlayDismissed(opts) {
             original_locator: opts.original ?? "",
             cache_hit: opts.cache,
             success: opts.success,
-            project_name: process.env.PROJECT_NAME ?? "",
+            project_name: process.env.PROJECT_NAME ?? "unknown",
+            repo_name: process.env.GIT_REPO_NAME ?? "unknown",
             test_run_id: process.env.TEST_RUN_GITHUB_ACTION_ID ?? "",
             test_name: opts.testName ?? "",
             tokens_input: opts.tokens?.input ?? 0,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empiricalrun/playwright-utils",
-  "version": "0.38.6",
+  "version": "0.39.1",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"
@@ -43,8 +43,8 @@
     "puppeteer-extra-plugin-recaptcha": "^3.6.8",
     "rimraf": "^6.0.1",
     "@empiricalrun/llm": "^0.25.2",
-    "@empiricalrun/r2-uploader": "^0.9.0",
-    "@empiricalrun/test-gen": "^0.79.1"
+    "@empiricalrun/r2-uploader": "^0.9.1",
+    "@empiricalrun/test-gen": "^0.79.3"
   },
   "scripts": {
     "dev": "tsc --build --watch",