@alternative-path/testlens-playwright-reporter 0.4.5 → 0.4.6

package/index.d.ts

@@ -161,6 +161,8 @@ export declare class TestLensReporter implements Reporter {
     private runCreationFailed;
     private cliArgs;
     private pendingUploads;
+    private artifactStats;
+    private artifactsSeen;
     /**
      * Parse custom metadata from environment variables
      * Checks for common metadata environment variables
@@ -184,6 +186,7 @@ export declare class TestLensReporter implements Reporter {
     private extractTestBlocks;
     private collectGitInfo;
     private getArtifactType;
+    private bumpArtifactStat;
     private extractTags;
     private getTestId;
     private uploadArtifactToS3;

package/index.js

@@ -10,9 +10,64 @@ const https = tslib_1.__importStar(require("https"));
 const axios_1 = tslib_1.__importDefault(require("axios"));
 const child_process_1 = require("child_process");
 const pino_1 = tslib_1.__importDefault(require("pino"));
+const isVerboseLogs = process.env.TESTLENS_LOG_VERBOSE === '1' ||
+    process.env.TESTLENS_LOG_VERBOSE === 'true';
+const noisyInfoPrefixes = [
+    '[OK]',
+    '[UPLOAD]',
+    '[ARTIFACT]',
+    '[SKIP]',
+    '[TestLens] onTestEnd called',
+    '[TestLens] Sending testEnd'
+];
+const noisyInfoSubstrings = [
+    'Sent runStart event to TestLens',
+    'Sent specStart event to TestLens',
+    'Sent testStart event to TestLens',
+    'Sent testEnd event to TestLens',
+    'Sent runEnd event to TestLens',
+    'Sent artifact event to TestLens',
+    'S3 upload completed',
+    'Upload confirmed',
+    'Uploading ',
+    'Processed artifact',
+    'Spec code blocks',
+    '\n[METADATA]',
+    'Custom Metadata Detected',
+    'Using API key from environment variable'
+];
+const shouldSuppressInfo = (msg) => {
+    if (isVerboseLogs)
+        return false;
+    if (!msg)
+        return false;
+    return (noisyInfoPrefixes.some(prefix => msg.startsWith(prefix)) ||
+        noisyInfoSubstrings.some(sub => msg.includes(sub)));
+};
 // Create pino logger instance
 const logger = (0, pino_1.default)({
-    level: process.env.LOG_LEVEL || 'info'
+    level: process.env.TESTLENS_LOG_LEVEL || process.env.LOG_LEVEL || 'info',
+    base: null, // remove pid/hostname
+    timestamp: false, // remove time
+    formatters: {
+        level: () => ({}) // remove level
+    },
+    hooks: {
+        logMethod(args, method, level) {
+            // pino level is numeric (info = 30)
+            if (level === 30) {
+                const msg = typeof args[0] === 'string'
+                    ? args[0]
+                    : typeof args[1] === 'string'
+                        ? args[1]
+                        : '';
+                if (shouldSuppressInfo(msg)) {
+                    return;
+                }
+            }
+            return method.apply(this, args);
+        }
+    }
 });
 // Lazy-load mime module to support ESM
 let mimeModule = null;
@@ -66,6 +121,12 @@ class TestLensReporter {
         this.runCreationFailed = false; // Track if run creation failed due to limits
         this.cliArgs = {}; // Store CLI args separately
         this.pendingUploads = new Set(); // Track pending artifact uploads
+        this.artifactStats = {
+            uploaded: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
+            skipped: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
+            failed: { screenshot: 0, video: 0, trace: 0, attachment: 0 }
+        };
+        this.artifactsSeen = 0;
         // Parse custom CLI arguments
         const customArgs = TestLensReporter.parseCustomArgs();
         this.cliArgs = customArgs; // Store CLI args separately for later use
@@ -235,19 +296,20 @@ class TestLensReporter {
         return status;
     }
     async onBegin(config, suite) {
+        logger.info(`[REPORTER] source=${__filename} enableArtifacts=${this.config.enableArtifacts}`);
         // Show Build Name if provided, otherwise show Run ID
         if (this.runMetadata.testlensBuildName) {
-            logger.info(`🚀 TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
+            logger.info(`TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
             logger.info(`   Run ID: ${this.runId}`);
         }
         else {
-            logger.info(`🚀 TestLens Reporter starting - Run ID: ${this.runId}`);
+            logger.info(`TestLens Reporter starting - Run ID: ${this.runId}`);
         }
         // Collect Git information if enabled
         if (this.config.enableGitInfo) {
             this.runMetadata.gitInfo = await this.collectGitInfo();
             if (this.runMetadata.gitInfo) {
-                logger.info(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+                logger.info(`Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
             }
             else {
                 logger.warn(`[WARN] Git info collection returned null - not in a git repository or git not available`);
@@ -273,7 +335,7 @@ class TestLensReporter {
     }
     async onTestBegin(test, result) {
         // Log which test is starting
-        logger.info(`\n[TEST] Running test: ${test.title}`);
+        logger.info(`[TEST] Running test: ${test.title}`);
         const specPath = test.location.file;
         const specKey = `${specPath}-${test.parent.title}`;
         // Create or update spec data
@@ -346,6 +408,19 @@ class TestLensReporter {
     async onTestEnd(test, result) {
         const testId = this.getTestId(test);
         let testData = this.testMap.get(testId);
+        logger.info(`[ARTIFACTS] attachments=${result.attachments?.length ?? 0}`);
+        if (result.attachments && result.attachments.length > 0) {
+            for (const attachment of result.attachments) {
+                const artifactType = this.getArtifactType(attachment.name);
+                this.artifactsSeen += 1;
+                if (attachment.path) {
+                    this.bumpArtifactStat('uploaded', artifactType);
+                }
+                else {
+                    this.bumpArtifactStat('skipped', artifactType);
+                }
+            }
+        }
         logger.info(`[TestLens] onTestEnd called for test: ${test.title}, status: ${result.status}, testData exists: ${!!testData}`);
         // For skipped tests, onTestBegin might not be called, so we need to create the test data here
         if (!testData) {
@@ -507,6 +582,13 @@ class TestLensReporter {
                 // Pass test case DB ID if available for faster lookups
                 await this.processArtifacts(testId, result, testEndResponse?.testCaseId);
             }
+            else if (result.attachments && result.attachments.length > 0) {
+                for (const attachment of result.attachments) {
+                    const artifactType = this.getArtifactType(attachment.name);
+                    this.artifactsSeen += 1;
+                    this.bumpArtifactStat('skipped', artifactType);
+                }
+            }
         }
         // Update spec status
         const specPath = test.location.file;
@@ -577,15 +659,29 @@ class TestLensReporter {
         this.runMetadata.duration = Date.now() - new Date(this.runMetadata.startTime).getTime();
         // Wait for all pending artifact uploads to complete before sending runEnd
         if (this.pendingUploads.size > 0) {
-            logger.info(`⏳ Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
+            logger.debug(`Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
             try {
                 await Promise.all(Array.from(this.pendingUploads));
-                logger.info(`[OK] All artifact uploads completed`);
+                logger.debug(`[OK] All artifact uploads completed`);
             }
             catch (error) {
                 logger.error(`[WARN] Some artifact uploads failed, continuing with runEnd`);
             }
         }
+        const uploaded = this.artifactStats.uploaded;
+        const skipped = this.artifactStats.skipped;
+        const failed = this.artifactStats.failed;
+        // Always show summary so users can confirm artifact settings
+        const summary = `[ARTIFACTS] seen=${this.artifactsSeen} | ` +
+            `uploaded screenshot=${uploaded.screenshot}, video=${uploaded.video}, trace=${uploaded.trace}, attachment=${uploaded.attachment} | ` +
+            `skipped screenshot=${skipped.screenshot}, video=${skipped.video}, trace=${skipped.trace}, attachment=${skipped.attachment} | ` +
+            `failed screenshot=${failed.screenshot}, video=${failed.video}, trace=${failed.trace}, attachment=${failed.attachment}`;
+        if (logger.levelVal > 30) {
+            console.log(summary);
+        }
+        else {
+            logger.info(summary);
+        }
         // Calculate final stats
         const totalTests = Array.from(this.testMap.values()).length;
         const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
@@ -634,7 +730,7 @@ class TestLensReporter {
                 }
             });
             if (this.config.enableRealTimeStream) {
-                logger.info(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
+                logger.debug(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
             }
             // Return response data for caller to use
             return response.data;
@@ -726,19 +822,21 @@ class TestLensReporter {
         }
         const attachments = result.attachments;
         for (const attachment of attachments) {
+            const artifactType = this.getArtifactType(attachment.name);
             if (attachment.path) {
                 // Check if attachment should be processed based on config
-                const artifactType = this.getArtifactType(attachment.name);
                 const isVideo = artifactType === 'video' || attachment.contentType?.startsWith('video/');
                 const isScreenshot = artifactType === 'screenshot' || attachment.contentType?.startsWith('image/');
                 // Skip video if disabled in config
                 if (isVideo && !this.config.enableVideo) {
-                    logger.info(`[SKIP] Skipping video artifact ${attachment.name} - video capture disabled in config`);
+                    logger.debug(`[SKIP] Skipping video artifact ${attachment.name} - video capture disabled in config`);
+                    this.bumpArtifactStat('skipped', artifactType);
                     continue;
                 }
                 // Skip screenshot if disabled in config
                 if (isScreenshot && !this.config.enableScreenshot) {
-                    logger.info(`[SKIP] Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+                    logger.debug(`[SKIP] Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+                    this.bumpArtifactStat('skipped', artifactType);
                     continue;
                 }
                 try {
@@ -776,13 +874,15 @@ class TestLensReporter {
                     const uploadPromise = Promise.resolve().then(async () => {
                         try {
                             if (!attachment.path) {
-                                logger.info(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
+                                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
+                                this.bumpArtifactStat('skipped', artifactType);
                                 return;
                             }
                             const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName, testCaseDbId);
                             // Skip if upload failed or file was too large
                             if (!s3Data) {
-                                logger.info(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - upload failed or file too large`);
+                                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - upload failed or file too large`);
+                                this.bumpArtifactStat('failed', artifactType);
                                 return;
                             }
                             const artifactData = {
@@ -803,10 +903,11 @@ class TestLensReporter {
                                 timestamp: new Date().toISOString(),
                                 artifact: artifactData
                             });
-                            logger.info(`[ARTIFACT] [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
+                            logger.debug(`[ARTIFACT] [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
                         }
                         catch (error) {
                             logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to process artifact ${attachment.name}: ${error.message}`);
+                            this.bumpArtifactStat('failed', artifactType);
                         }
                     });
                     // Track this upload and ensure cleanup on completion
@@ -819,8 +920,12 @@ class TestLensReporter {
                 }
                 catch (error) {
                     logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}: ${error.message}`);
+                    this.bumpArtifactStat('failed', artifactType);
                 }
             }
+            else {
+                this.bumpArtifactStat('skipped', artifactType);
+            }
         }
     }
     async sendSpecCodeBlocks(specPath) {
@@ -848,7 +953,7 @@ class TestLensReporter {
                 codeBlocks,
                 testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
             });
-            logger.info(`📝 Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
+            logger.info(`Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
         }
         catch (error) {
             const errorData = error?.response?.data;
@@ -968,6 +1073,12 @@ class TestLensReporter {
             return 'trace';
         return 'attachment';
     }
+    bumpArtifactStat(stat, type) {
+        const bucket = this.artifactStats[stat];
+        if (bucket[type] !== undefined) {
+            bucket[type] += 1;
+        }
+    }
     extractTags(test) {
         const tags = [];
         // Playwright stores tags in the _tags property
@@ -1011,7 +1122,7 @@ class TestLensReporter {
             // Check file size first
             const fileSize = this.getFileSize(filePath);
             const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
-            logger.info(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
+            logger.debug(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
             const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
             // Step 1: Request pre-signed URL from server
             const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
@@ -1036,7 +1147,7 @@ class TestLensReporter {
             }
             const { uploadUrl, s3Key, metadata } = presignedResponse.data;
             // Step 2: Upload directly to S3 using presigned URL
-            logger.info(`[UPLOAD] [Test: ${testId.substring(0, 8)}...] Uploading ${fileName} directly to S3...`);
+            logger.debug(`[UPLOAD] [Test: ${testId.substring(0, 8)}...] Uploading ${fileName} directly to S3...`);
             const fileBuffer = fs.readFileSync(filePath);
             // IMPORTANT: When using presigned URLs, we MUST include exactly the headers that were signed
             // The backend signs with ServerSideEncryption:'AES256', so we must send that header
@@ -1054,7 +1165,7 @@ class TestLensReporter {
             if (uploadResponse.status !== 200) {
                 throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
             }
-            logger.info(`[OK] [Test: ${testId.substring(0, 8)}...] S3 upload completed for ${fileName}`);
+            logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] S3 upload completed for ${fileName}`);
             // Step 3: Confirm upload with server to save metadata
             const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
             const confirmBody = {
@@ -1076,7 +1187,7 @@ class TestLensReporter {
             });
             if (confirmResponse.status === 201 && confirmResponse.data.success) {
                 const artifact = confirmResponse.data.artifact;
-                logger.info(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
+                logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
                 return {
                     key: s3Key,
                     url: artifact.s3Url,

package/index.ts

@@ -8,9 +8,68 @@ import type { Reporter, TestCase, TestResult, FullConfig, Suite } from '@playwri
 import { execSync } from 'child_process';
 import pino from 'pino';
 
+const isVerboseLogs =
+  process.env.TESTLENS_LOG_VERBOSE === '1' ||
+  process.env.TESTLENS_LOG_VERBOSE === 'true';
+const noisyInfoPrefixes = [
+  '[OK]',
+  '[UPLOAD]',
+  '[ARTIFACT]',
+  '[SKIP]',
+  '[TestLens] onTestEnd called',
+  '[TestLens] Sending testEnd'
+];
+const noisyInfoSubstrings = [
+  'Sent runStart event to TestLens',
+  'Sent specStart event to TestLens',
+  'Sent testStart event to TestLens',
+  'Sent testEnd event to TestLens',
+  'Sent runEnd event to TestLens',
+  'Sent artifact event to TestLens',
+  'S3 upload completed',
+  'Upload confirmed',
+  'Uploading ',
+  'Processed artifact',
+  'Spec code blocks',
+  '\n[METADATA]',
+  'Custom Metadata Detected',
+  'Using API key from environment variable'
+];
+
+const shouldSuppressInfo = (msg: string): boolean => {
+  if (isVerboseLogs) return false;
+  if (!msg) return false;
+  return (
+    noisyInfoPrefixes.some(prefix => msg.startsWith(prefix)) ||
+    noisyInfoSubstrings.some(sub => msg.includes(sub))
+  );
+};
+
 // Create pino logger instance
 const logger = pino({
-  level: process.env.LOG_LEVEL || 'info'
+  level: process.env.TESTLENS_LOG_LEVEL || process.env.LOG_LEVEL || 'info',
+  base: null, // remove pid/hostname
+  timestamp: false, // remove time
+  formatters: {
+    level: () => ({}) // remove level
+  },
+  hooks: {
+    logMethod(args, method, level) {
+      // pino level is numeric (info = 30)
+      if (level === 30) {
+        const msg =
+          typeof args[0] === 'string'
+            ? args[0]
+            : typeof args[1] === 'string'
+            ? args[1]
+            : '';
+        if (shouldSuppressInfo(msg)) {
+          return;
+        }
+      }
+      return method.apply(this, args as any);
+    }
+  }
 });
 
 // Lazy-load mime module to support ESM
@@ -191,6 +250,12 @@ export class TestLensReporter implements Reporter {
   private runCreationFailed: boolean = false; // Track if run creation failed due to limits
   private cliArgs: Record<string, any> = {}; // Store CLI args separately
   private pendingUploads: Set<Promise<any>> = new Set(); // Track pending artifact uploads
+  private artifactStats = {
+    uploaded: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
+    skipped: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
+    failed: { screenshot: 0, video: 0, trace: 0, attachment: 0 }
+  };
+  private artifactsSeen: number = 0;
 
   /**
    * Parse custom metadata from environment variables
@@ -423,19 +488,20 @@ export class TestLensReporter implements Reporter {
   }
 
   async onBegin(config: FullConfig, suite: Suite): Promise<void> {
+    logger.info(`[REPORTER] source=${__filename} enableArtifacts=${this.config.enableArtifacts}`);
     // Show Build Name if provided, otherwise show Run ID
     if (this.runMetadata.testlensBuildName) {
-      logger.info(`🚀 TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
+      logger.info(`TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
       logger.info(`   Run ID: ${this.runId}`);
     } else {
-      logger.info(`🚀 TestLens Reporter starting - Run ID: ${this.runId}`);
+      logger.info(`TestLens Reporter starting - Run ID: ${this.runId}`);
     }
     
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
       if (this.runMetadata.gitInfo) {
-        logger.info(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+        logger.info(`Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
       } else {
         logger.warn(`[WARN] Git info collection returned null - not in a git repository or git not available`);
       }
@@ -462,7 +528,7 @@ export class TestLensReporter implements Reporter {
 
   async onTestBegin(test: TestCase, result: TestResult): Promise<void> {
     // Log which test is starting
-    logger.info(`\n[TEST] Running test: ${test.title}`);
+    logger.info(`[TEST] Running test: ${test.title}`);
     
     const specPath = test.location.file;
     const specKey = `${specPath}-${test.parent.title}`;
@@ -543,6 +609,20 @@ export class TestLensReporter implements Reporter {
     const testId = this.getTestId(test);
     let testData = this.testMap.get(testId);
 
+    logger.info(`[ARTIFACTS] attachments=${result.attachments?.length ?? 0}`);
+
+    if (result.attachments && result.attachments.length > 0) {
+      for (const attachment of result.attachments) {
+        const artifactType = this.getArtifactType(attachment.name);
+        this.artifactsSeen += 1;
+        if (attachment.path) {
+          this.bumpArtifactStat('uploaded', artifactType);
+        } else {
+          this.bumpArtifactStat('skipped', artifactType);
+        }
+      }
+    }
+
     logger.info(`[TestLens] onTestEnd called for test: ${test.title}, status: ${result.status}, testData exists: ${!!testData}`);
 
     // For skipped tests, onTestBegin might not be called, so we need to create the test data here
@@ -726,6 +806,12 @@ export class TestLensReporter implements Reporter {
       if (this.config.enableArtifacts) {
         // Pass test case DB ID if available for faster lookups
         await this.processArtifacts(testId, result, testEndResponse?.testCaseId);
+      } else if (result.attachments && result.attachments.length > 0) {
+        for (const attachment of result.attachments) {
+          const artifactType = this.getArtifactType(attachment.name);
+          this.artifactsSeen += 1;
+          this.bumpArtifactStat('skipped', artifactType);
+        }
       }
     }
 
@@ -803,15 +889,30 @@ export class TestLensReporter implements Reporter {
 
     // Wait for all pending artifact uploads to complete before sending runEnd
     if (this.pendingUploads.size > 0) {
-      logger.info(`⏳ Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
+      logger.debug(`Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
       try {
         await Promise.all(Array.from(this.pendingUploads));
-        logger.info(`[OK] All artifact uploads completed`);
+        logger.debug(`[OK] All artifact uploads completed`);
       } catch (error) {
         logger.error(`[WARN] Some artifact uploads failed, continuing with runEnd`);
       }
     }
 
+    const uploaded = this.artifactStats.uploaded;
+    const skipped = this.artifactStats.skipped;
+    const failed = this.artifactStats.failed;
+    // Always show summary so users can confirm artifact settings
+    const summary =
+      `[ARTIFACTS] seen=${this.artifactsSeen} | ` +
+      `uploaded screenshot=${uploaded.screenshot}, video=${uploaded.video}, trace=${uploaded.trace}, attachment=${uploaded.attachment} | ` +
+      `skipped screenshot=${skipped.screenshot}, video=${skipped.video}, trace=${skipped.trace}, attachment=${skipped.attachment} | ` +
+      `failed screenshot=${failed.screenshot}, video=${failed.video}, trace=${failed.trace}, attachment=${failed.attachment}`;
+    if (logger.levelVal > 30) {
+      console.log(summary);
+    } else {
+      logger.info(summary);
+    }
+
     // Calculate final stats
     const totalTests = Array.from(this.testMap.values()).length;
     const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
@@ -864,7 +965,7 @@ export class TestLensReporter implements Reporter {
         }
       });
       if (this.config.enableRealTimeStream) {
-        logger.info(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
+        logger.debug(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
       }
       // Return response data for caller to use
       return response.data;
@@ -958,21 +1059,23 @@ export class TestLensReporter implements Reporter {
     const attachments = result.attachments;
     
     for (const attachment of attachments) {
+      const artifactType = this.getArtifactType(attachment.name);
       if (attachment.path) {
         // Check if attachment should be processed based on config
-        const artifactType = this.getArtifactType(attachment.name);
         const isVideo = artifactType === 'video' || attachment.contentType?.startsWith('video/');
         const isScreenshot = artifactType === 'screenshot' || attachment.contentType?.startsWith('image/');
         
         // Skip video if disabled in config
         if (isVideo && !this.config.enableVideo) {
-          logger.info(`[SKIP] Skipping video artifact ${attachment.name} - video capture disabled in config`);
+          logger.debug(`[SKIP] Skipping video artifact ${attachment.name} - video capture disabled in config`);
+          this.bumpArtifactStat('skipped', artifactType);
           continue;
         }
         
         // Skip screenshot if disabled in config
         if (isScreenshot && !this.config.enableScreenshot) {
-          logger.info(`[SKIP] Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+          logger.debug(`[SKIP] Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+          this.bumpArtifactStat('skipped', artifactType);
           continue;
         }
         
@@ -1012,7 +1115,8 @@ export class TestLensReporter implements Reporter {
           const uploadPromise = Promise.resolve().then(async () => {
             try {
               if (!attachment.path) {
-                logger.info(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
+                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
+                this.bumpArtifactStat('skipped', artifactType);
                 return;
               }
               
@@ -1020,7 +1124,8 @@ export class TestLensReporter implements Reporter {
               
               // Skip if upload failed or file was too large
               if (!s3Data) {
-                logger.info(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - upload failed or file too large`);
+                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - upload failed or file too large`);
+                this.bumpArtifactStat('failed', artifactType);
                 return;
               }
               
@@ -1044,9 +1149,10 @@ export class TestLensReporter implements Reporter {
                 artifact: artifactData
               });
               
-              logger.info(`[ARTIFACT] [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
+              logger.debug(`[ARTIFACT] [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
             } catch (error) {
               logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to process artifact ${attachment.name}: ${(error as Error).message}`);
+              this.bumpArtifactStat('failed', artifactType);
             }
           });
           
@@ -1060,7 +1166,10 @@ export class TestLensReporter implements Reporter {
           // They will be awaited in onEnd
         } catch (error) {
           logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}: ${(error as Error).message}`);
+          this.bumpArtifactStat('failed', artifactType);
         }
+      } else {
+        this.bumpArtifactStat('skipped', artifactType);
       }
     }
   }
@@ -1094,7 +1203,7 @@ export class TestLensReporter implements Reporter {
         testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
       });
 
-      logger.info(`📝 Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
+      logger.info(`Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
     } catch (error: any) {
       const errorData = error?.response?.data;
       
@@ -1223,6 +1332,13 @@ export class TestLensReporter implements Reporter {
     return 'attachment';
   }
 
+  private bumpArtifactStat(stat: 'uploaded' | 'skipped' | 'failed', type: string): void {
+    const bucket = this.artifactStats[stat] as Record<string, number>;
+    if (bucket[type] !== undefined) {
+      bucket[type] += 1;
+    }
+  }
+
   private extractTags(test: TestCase): string[] {
     const tags: string[] = [];
     
@@ -1275,7 +1391,7 @@ export class TestLensReporter implements Reporter {
       const fileSize = this.getFileSize(filePath);
       const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
       
-      logger.info(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
+      logger.debug(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
 
       const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
       
@@ -1307,7 +1423,7 @@ export class TestLensReporter implements Reporter {
       const { uploadUrl, s3Key, metadata } = presignedResponse.data;
       
       // Step 2: Upload directly to S3 using presigned URL
-      logger.info(`[UPLOAD] [Test: ${testId.substring(0, 8)}...] Uploading ${fileName} directly to S3...`);
+      logger.debug(`[UPLOAD] [Test: ${testId.substring(0, 8)}...] Uploading ${fileName} directly to S3...`);
       
       const fileBuffer = fs.readFileSync(filePath);
       
@@ -1329,7 +1445,7 @@ export class TestLensReporter implements Reporter {
         throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
       }
 
-      logger.info(`[OK] [Test: ${testId.substring(0, 8)}...] S3 upload completed for ${fileName}`);
+      logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] S3 upload completed for ${fileName}`);
 
       // Step 3: Confirm upload with server to save metadata
       const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
@@ -1355,7 +1471,7 @@ export class TestLensReporter implements Reporter {
 
       if (confirmResponse.status === 201 && confirmResponse.data.success) {
         const artifact = confirmResponse.data.artifact;
-        logger.info(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
+        logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
         return {
           key: s3Key,
           url: artifact.s3Url,
@@ -1469,4 +1585,3 @@ export class TestLensReporter implements Reporter {
 }
 
 export default TestLensReporter;
-

package/package.json

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

package/postinstall.js

@@ -1,44 +1,49 @@
-const fs = require('fs');
-const path = require('path');
-const pino = require('pino');
-
-const logger = pino({
-  level: process.env.LOG_LEVEL || 'info'
-});
-
-try {
-  // Get the parent project's node_modules/.bin directory
-  // When installed, we are in: project/node_modules/testlens-playwright-reporter/
-  // We need to go to: project/node_modules/.bin/
-  const parentBinDir = path.resolve(__dirname, '..', '.bin');
-  const localBinDir = path.resolve(__dirname, 'node_modules', '.bin');
-  
-  // Check if parent bin directory exists
-  if (!fs.existsSync(parentBinDir)) {
-    logger.info('Parent .bin directory not found, skipping cross-env setup');
-    process.exit(0);
-  }
-
-  // Check if cross-env exists in our node_modules
-  const crossEnvCmd = process.platform === 'win32' ? 'cross-env.cmd' : 'cross-env';
-  const crossEnvPs1 = 'cross-env.ps1';
-  const sourceCmdPath = path.join(localBinDir, crossEnvCmd);
-  const sourcePs1Path = path.join(localBinDir, crossEnvPs1);
-  
-  if (fs.existsSync(sourceCmdPath)) {
-    const targetCmdPath = path.join(parentBinDir, crossEnvCmd);
-    
-    // Copy the file
-    fs.copyFileSync(sourceCmdPath, targetCmdPath);
-    logger.info('✓ cross-env binary installed');
-    
-    // Also copy PowerShell script on Windows
-    if (process.platform === 'win32' && fs.existsSync(sourcePs1Path)) {
-      const targetPs1Path = path.join(parentBinDir, crossEnvPs1);
-      fs.copyFileSync(sourcePs1Path, targetPs1Path);
-    }
-  }
-} catch (error) {
-  // Don't fail installation if this doesn't work
-  logger.info('Note: Could not setup cross-env automatically:', error.message);
-}
+const fs = require('fs');
+const path = require('path');
+const pino = require('pino');
+
+const logger = pino({
+  level: process.env.LOG_LEVEL || 'info',
+  base: null,
+  timestamp: false,
+  formatters: {
+    level: () => ({})
+  }
+});
+
+try {
+  // Get the parent project's node_modules/.bin directory
+  // When installed, we are in: project/node_modules/testlens-playwright-reporter/
+  // We need to go to: project/node_modules/.bin/
+  const parentBinDir = path.resolve(__dirname, '..', '.bin');
+  const localBinDir = path.resolve(__dirname, 'node_modules', '.bin');
+  
+  // Check if parent bin directory exists
+  if (!fs.existsSync(parentBinDir)) {
+    logger.info('Parent .bin directory not found, skipping cross-env setup');
+    process.exit(0);
+  }
+
+  // Check if cross-env exists in our node_modules
+  const crossEnvCmd = process.platform === 'win32' ? 'cross-env.cmd' : 'cross-env';
+  const crossEnvPs1 = 'cross-env.ps1';
+  const sourceCmdPath = path.join(localBinDir, crossEnvCmd);
+  const sourcePs1Path = path.join(localBinDir, crossEnvPs1);
+  
+  if (fs.existsSync(sourceCmdPath)) {
+    const targetCmdPath = path.join(parentBinDir, crossEnvCmd);
+    
+    // Copy the file
+    fs.copyFileSync(sourceCmdPath, targetCmdPath);
+    logger.info('✓ cross-env binary installed');
+    
+    // Also copy PowerShell script on Windows
+    if (process.platform === 'win32' && fs.existsSync(sourcePs1Path)) {
+      const targetPs1Path = path.join(parentBinDir, crossEnvPs1);
+      fs.copyFileSync(sourcePs1Path, targetPs1Path);
+    }
+  }
+} catch (error) {
+  // Don't fail installation if this doesn't work
+  logger.info('Note: Could not setup cross-env automatically:', error.message);
+}