testlens-playwright-reporter 0.2.2 → 0.2.4

package/CHANGELOG.md

@@ -0,0 +1,23 @@
+# Changelog
+
+All notable changes to the testlens-playwright-reporter package will be documented in this file.
+
+## [0.2.4] - 2025-11-18
+
+### Fixed
+- **TypeScript source updated**: Applied trace file extension preservation fix to TypeScript source (`index.ts`) and rebuilt `lib/index.js` to ensure both TypeScript and JavaScript implementations have the fix.
+
+### Fixed
+- **Package entry point**: Changed main entry point from `lib/index.js` to `index.js` to use the correct implementation with trace file extension preservation fix.
+
+## [0.2.3] - 2025-11-18
+
+### Fixed
+- **Trace file extension preservation**: Fixed issue where Playwright trace files were being uploaded to S3 without their `.zip` extension, causing "Could not load trace" errors in Playwright Trace Viewer. The reporter now preserves the original file extension from `attachment.path` when uploading artifacts, ensuring trace files maintain their `.zip` format required by the viewer.
+
+## [0.2.2] - Previous release
+
+### Features
+- Direct S3 uploads with presigned URLs
+- Asynchronous artifact processing
+- Enhanced error handling and retry logic

package/index.js

@@ -371,14 +371,20 @@ class TestLensReporter {
             try {
               console.log(`📤 Processing ${attachment.name} asynchronously...`);
               
+              // Preserve original file extension from path (important for traces which must be .zip)
+              const originalExt = path.extname(attachment.path);
+              const fileName = attachment.name.includes(originalExt) 
+                ? attachment.name 
+                : attachment.name + originalExt;
+              
               // Upload to S3 first
-              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
               
               const artifactData = {
                 testId,
-                type: this.getArtifactType(attachment.name),
+                type: this.getArtifactType(fileName),
                 path: attachment.path,
-                name: attachment.name,
+                name: fileName,
                 contentType: attachment.contentType,
                 fileSize: this.getFileSize(attachment.path),
                 storageType: 's3',

package/index.ts

@@ -483,20 +483,26 @@ export class TestLensReporter implements Reporter {
     for (const attachment of attachments) {
       if (attachment.path) {
         try {
+          // Preserve original file extension from path (important for traces which must be .zip)
+          const originalExt = path.extname(attachment.path);
+          const fileName = attachment.name.includes(originalExt) 
+            ? attachment.name 
+            : attachment.name + originalExt;
+          
           // Upload to S3 first
-          const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+          const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
           
           // Skip if upload failed or file was too large
           if (!s3Data) {
-            console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
+            console.log(`⏭️  Skipping artifact ${fileName} - upload failed or file too large`);
             continue;
           }
           
           const artifactData = {
             testId,
-            type: this.getArtifactType(attachment.name),
+            type: this.getArtifactType(fileName),
             path: attachment.path,
-            name: attachment.name,
+            name: fileName,
             contentType: attachment.contentType,
             fileSize: this.getFileSize(attachment.path),
             storageType: 's3',

package/lib/index.js

@@ -352,18 +352,23 @@ class TestLensReporter {
         for (const attachment of attachments) {
             if (attachment.path) {
                 try {
+                    // Preserve original file extension from path (important for traces which must be .zip)
+                    const originalExt = path.extname(attachment.path);
+                    const fileName = attachment.name.includes(originalExt)
+                        ? attachment.name
+                        : attachment.name + originalExt;
                     // Upload to S3 first
-                    const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+                    const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
                     // Skip if upload failed or file was too large
                     if (!s3Data) {
-                        console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
+                        console.log(`⏭️  Skipping artifact ${fileName} - upload failed or file too large`);
                         continue;
                     }
                     const artifactData = {
                         testId,
-                        type: this.getArtifactType(attachment.name),
+                        type: this.getArtifactType(fileName),
                         path: attachment.path,
-                        name: attachment.name,
+                        name: fileName,
                         contentType: attachment.contentType,
                         fileSize: this.getFileSize(attachment.path),
                         storageType: 's3',

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "testlens-playwright-reporter",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "description": "Universal Playwright reporter for TestLens - works with both TypeScript and JavaScript projects",
-  "main": "lib/index.js",
+  "main": "index.js",
   "types": "index.d.ts",
   "files": [
     "index.js",