testlens-playwright-reporter 0.2.4 → 0.2.6

package/index.d.ts

@@ -55,8 +55,8 @@ export interface GitInfo {
   commit: string;
   shortCommit: string;
   author: string;
-  commitMessage: string;
-  commitTimestamp: string;
+  message: string;
+  timestamp: string;
   isDirty: boolean;
   remoteName: string;
   remoteUrl: string;

package/index.js

@@ -6,12 +6,23 @@ const path = require('path');
 const fs = require('fs');
 const https = require('https');
 const axios = require('axios');
-const mime = require('mime');
 const FormData = require('form-data');
+
+// Lazy-load mime module to support ESM
+let mimeModule = null;
+async function getMime() {
+  if (!mimeModule) {
+    const imported = await import('mime');
+    // Handle both default export and named exports
+    mimeModule = imported.default || imported;
+  }
+  return mimeModule;
+}
+
 class TestLensReporter {
   constructor(options = {}) {
     this.config = {
-      apiEndpoint: 'https://testlens.qa-path.com/api/v1/webhook/playwright',
+      apiEndpoint: options.apiEndpoint || 'https://testlens.qa-path.com/api/v1/webhook/playwright',
       apiKey: options.apiKey, // API key must come from config file
       enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
       enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
@@ -136,6 +147,13 @@ class TestLensReporter {
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
+      if (this.runMetadata.gitInfo) {
+        console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+      } else {
+        console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+      }
+    } else {
+      console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
     }
 
     // Add shard information if available
@@ -371,20 +389,14 @@ 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, fileName);
+              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
               
               const artifactData = {
                 testId,
-                type: this.getArtifactType(fileName),
+                type: this.getArtifactType(attachment.name),
                 path: attachment.path,
-                name: fileName,
+                name: attachment.name,
                 contentType: attachment.contentType,
                 fileSize: this.getFileSize(attachment.path),
                 storageType: 's3',
@@ -538,8 +550,8 @@ class TestLensReporter {
         commit,
         shortCommit,
         author,
-        commitMessage,
-        commitTimestamp,
+        message: commitMessage,
+        timestamp: commitTimestamp,
         isDirty,
         remoteName,
         remoteUrl
@@ -579,24 +591,6 @@ class TestLensReporter {
     return `${test.location.file}:${test.location.line}:${cleanTitle}`;
   }
 
-  getContentType(ext) {
-    const contentTypes = {
-      '.mp4': 'video/mp4',
-      '.webm': 'video/webm',
-      '.png': 'image/png',
-      '.jpg': 'image/jpeg',
-      '.jpeg': 'image/jpeg',
-      '.gif': 'image/gif',
-      '.json': 'application/json',
-      '.txt': 'text/plain',
-      '.html': 'text/html',
-      '.xml': 'application/xml',
-      '.zip': 'application/zip',
-      '.pdf': 'application/pdf'
-    };
-    return contentTypes[ext] || 'application/octet-stream';
-  }
-
   getFileSize(filePath) {
     try {
       const stats = fs.statSync(filePath);
@@ -624,7 +618,7 @@ class TestLensReporter {
         testRunId: this.runId,
         testId: testId,
         fileName: fileName,
-        fileType: this.getContentType(fileName),
+        fileType: await this.getContentType(fileName),
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       }, {
@@ -670,7 +664,7 @@ class TestLensReporter {
         testId: testId,
         s3Key: s3Key,
         fileName: fileName,
-        fileType: this.getContentType(fileName),
+        fileType: await this.getContentType(fileName),
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       }, {
@@ -714,10 +708,35 @@ class TestLensReporter {
     }
   }
 
-  getContentType(fileName) {
+  async getContentType(fileName) {
     const ext = path.extname(fileName).toLowerCase();
-    const mimeType = mime.getType(ext) || 'application/octet-stream';
-    return mimeType;
+    try {
+      const mime = await getMime();
+      // Try different ways to access getType method
+      const getType = mime.getType || mime.default?.getType;
+      if (typeof getType === 'function') {
+        const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
+        return mimeType || 'application/octet-stream';
+      }
+    } catch (error) {
+      console.warn(`Failed to get MIME type for ${fileName}:`, error.message);
+    }
+    // Fallback to basic content type mapping
+    const contentTypes = {
+      '.mp4': 'video/mp4',
+      '.webm': 'video/webm',
+      '.png': 'image/png',
+      '.jpg': 'image/jpeg',
+      '.jpeg': 'image/jpeg',
+      '.gif': 'image/gif',
+      '.json': 'application/json',
+      '.txt': 'text/plain',
+      '.html': 'text/html',
+      '.xml': 'application/xml',
+      '.zip': 'application/zip',
+      '.pdf': 'application/pdf'
+    };
+    return contentTypes[ext] || 'application/octet-stream';
   }
 }
 

package/index.ts

@@ -8,8 +8,19 @@ import * as https from 'https';
 import axios, { AxiosInstance } from 'axios';
 import type { Reporter, TestCase, TestResult, FullConfig, Suite } from '@playwright/test/reporter';
 import { execSync } from 'child_process';
-import * as mime from 'mime';
 import FormData from 'form-data';
+
+// Lazy-load mime module to support ESM
+let mimeModule: any = null;
+async function getMime() {
+  if (!mimeModule) {
+    const imported = await import('mime');
+    // Handle both default export and named exports
+    mimeModule = imported.default || imported;
+  }
+  return mimeModule;
+}
+
 export interface TestLensReporterConfig {
   /** TestLens API endpoint URL */
   apiEndpoint?: string;
@@ -65,8 +76,8 @@ export interface GitInfo {
   commit: string;
   shortCommit: string;
   author: string;
-  commitMessage: string;
-  commitTimestamp: string;
+  message: string;
+  timestamp: string;
   isDirty: boolean;
   remoteName: string;
   remoteUrl: string;
@@ -265,6 +276,13 @@ export class TestLensReporter implements Reporter {
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
+      if (this.runMetadata.gitInfo) {
+        console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+      } else {
+        console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+      }
+    } else {
+      console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
     }
 
     // Add shard information if available
@@ -483,26 +501,20 @@ 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, fileName);
+          const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
           
           // Skip if upload failed or file was too large
           if (!s3Data) {
-            console.log(`⏭️  Skipping artifact ${fileName} - upload failed or file too large`);
+            console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
             continue;
           }
           
           const artifactData = {
             testId,
-            type: this.getArtifactType(fileName),
+            type: this.getArtifactType(attachment.name),
             path: attachment.path,
-            name: fileName,
+            name: attachment.name,
             contentType: attachment.contentType,
             fileSize: this.getFileSize(attachment.path),
             storageType: 's3',
@@ -652,8 +664,8 @@ export class TestLensReporter implements Reporter {
         commit,
         shortCommit,
         author,
-        commitMessage,
-        commitTimestamp,
+        message: commitMessage,
+        timestamp: commitTimestamp,
         isDirty,
         remoteName,
         remoteUrl
@@ -711,7 +723,7 @@ export class TestLensReporter implements Reporter {
         testRunId: this.runId,
         testId: testId,
         fileName: fileName,
-        fileType: this.getContentType(fileName),
+        fileType: await this.getContentType(fileName),
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       }, {
@@ -757,7 +769,7 @@ export class TestLensReporter implements Reporter {
         testId: testId,
         s3Key: s3Key,
         fileName: fileName,
-        fileType: this.getContentType(fileName),
+        fileType: await this.getContentType(fileName),
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       }, {
@@ -801,10 +813,35 @@ export class TestLensReporter implements Reporter {
     }
   }
 
-  private getContentType(fileName: string): string {
+  private async getContentType(fileName: string): Promise<string> {
     const ext = path.extname(fileName).toLowerCase();
-    const mimeType = mime.default.getType(ext) || 'application/octet-stream';
-    return mimeType;
+    try {
+      const mime = await getMime();
+      // Try different ways to access getType method
+      const getType = mime.getType || mime.default?.getType;
+      if (typeof getType === 'function') {
+        const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
+        return mimeType || 'application/octet-stream';
+      }
+    } catch (error: any) {
+      console.warn(`Failed to get MIME type for ${fileName}:`, error.message);
+    }
+    // Fallback to basic content type mapping
+    const contentTypes: Record<string, string> = {
+      '.mp4': 'video/mp4',
+      '.webm': 'video/webm',
+      '.png': 'image/png',
+      '.jpg': 'image/jpeg',
+      '.jpeg': 'image/jpeg',
+      '.gif': 'image/gif',
+      '.json': 'application/json',
+      '.txt': 'text/plain',
+      '.html': 'text/html',
+      '.xml': 'application/xml',
+      '.zip': 'application/zip',
+      '.pdf': 'application/pdf'
+    };
+    return contentTypes[ext] || 'application/octet-stream';
   }
 
   private generateS3Key(runId: string, testId: string, fileName: string): string {

package/lib/index.js

@@ -45,7 +45,16 @@ const fs = __importStar(require("fs"));
 const https = __importStar(require("https"));
 const axios_1 = __importDefault(require("axios"));
 const child_process_1 = require("child_process");
-const mime = __importStar(require("mime"));
+// Lazy-load mime module to support ESM
+let mimeModule = null;
+async function getMime() {
+    if (!mimeModule) {
+        const imported = await Promise.resolve().then(() => __importStar(require('mime')));
+        // Handle both default export and named exports
+        mimeModule = imported.default || imported;
+    }
+    return mimeModule;
+}
 class TestLensReporter {
     constructor(options) {
         this.config = {
@@ -159,6 +168,15 @@ class TestLensReporter {
         // Collect Git information if enabled
         if (this.config.enableGitInfo) {
             this.runMetadata.gitInfo = await this.collectGitInfo();
+            if (this.runMetadata.gitInfo) {
+                console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+            }
+            else {
+                console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+            }
+        }
+        else {
+            console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
         }
         // Add shard information if available
         if (config.shard) {
@@ -352,23 +370,18 @@ 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, fileName);
+                    const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
                     // Skip if upload failed or file was too large
                     if (!s3Data) {
-                        console.log(`⏭️  Skipping artifact ${fileName} - upload failed or file too large`);
+                        console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
                         continue;
                     }
                     const artifactData = {
                         testId,
-                        type: this.getArtifactType(fileName),
+                        type: this.getArtifactType(attachment.name),
                         path: attachment.path,
-                        name: fileName,
+                        name: attachment.name,
                         contentType: attachment.contentType,
                         fileSize: this.getFileSize(attachment.path),
                         storageType: 's3',
@@ -503,8 +516,8 @@ class TestLensReporter {
                 commit,
                 shortCommit,
                 author,
-                commitMessage,
-                commitTimestamp,
+                message: commitMessage,
+                timestamp: commitTimestamp,
                 isDirty,
                 remoteName,
                 remoteUrl
@@ -555,7 +568,7 @@ class TestLensReporter {
                 testRunId: this.runId,
                 testId: testId,
                 fileName: fileName,
-                fileType: this.getContentType(fileName),
+                fileType: await this.getContentType(fileName),
                 fileSize: fileSize,
                 artifactType: this.getArtifactType(fileName)
             }, {
@@ -593,7 +606,7 @@ class TestLensReporter {
                 testId: testId,
                 s3Key: s3Key,
                 fileName: fileName,
-                fileType: this.getContentType(fileName),
+                fileType: await this.getContentType(fileName),
                 fileSize: fileSize,
                 artifactType: this.getArtifactType(fileName)
             }, {
@@ -637,10 +650,36 @@ class TestLensReporter {
             return null;
         }
     }
-    getContentType(fileName) {
+    async getContentType(fileName) {
         const ext = path.extname(fileName).toLowerCase();
-        const mimeType = mime.default.getType(ext) || 'application/octet-stream';
-        return mimeType;
+        try {
+            const mime = await getMime();
+            // Try different ways to access getType method
+            const getType = mime.getType || mime.default?.getType;
+            if (typeof getType === 'function') {
+                const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
+                return mimeType || 'application/octet-stream';
+            }
+        }
+        catch (error) {
+            console.warn(`Failed to get MIME type for ${fileName}:`, error.message);
+        }
+        // Fallback to basic content type mapping
+        const contentTypes = {
+            '.mp4': 'video/mp4',
+            '.webm': 'video/webm',
+            '.png': 'image/png',
+            '.jpg': 'image/jpeg',
+            '.jpeg': 'image/jpeg',
+            '.gif': 'image/gif',
+            '.json': 'application/json',
+            '.txt': 'text/plain',
+            '.html': 'text/html',
+            '.xml': 'application/xml',
+            '.zip': 'application/zip',
+            '.pdf': 'application/pdf'
+        };
+        return contentTypes[ext] || 'application/octet-stream';
     }
     generateS3Key(runId, testId, fileName) {
         const date = new Date().toISOString().slice(0, 10);

package/package.json

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

package/CHANGELOG.md

@@ -1,23 +0,0 @@
-# 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