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

package/index.ts

@@ -6,7 +6,71 @@ 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 FormData from 'form-data';
+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.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
 let mimeModule: any = null;
@@ -186,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
@@ -214,10 +284,10 @@ export class TestLensReporter implements Reporter {
           // For testlensBuildTag, support comma-separated values
           if (key === 'testlensBuildTag' && value.includes(',')) {
             customArgs[key] = value.split(',').map(tag => tag.trim()).filter(tag => tag);
-            console.log(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
+            logger.info(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
           } else {
             customArgs[key] = value;
-            console.log(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+            logger.info(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
           }
           break; // Use first match
         }
@@ -257,6 +327,8 @@ export class TestLensReporter implements Reporter {
       flushInterval: options.flushInterval || 5000,
       retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
       timeout: options.timeout || 60000,
+      rejectUnauthorized: options.rejectUnauthorized,
+      ignoreSslErrors: options.ignoreSslErrors,
       customMetadata: { ...options.customMetadata, ...customArgs } // Config metadata first, then CLI args override
     } as Required<TestLensReporterConfig>;
 
@@ -265,27 +337,36 @@ export class TestLensReporter implements Reporter {
     }
     
     if (apiKey !== options.apiKey) {
-      console.log('✓ Using API key from environment variable');
+      logger.info('✓ Using API key from environment variable');
+    }
+
+    // Default environment to allow self-signed certs unless explicitly set
+    if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === undefined) {
+      process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0';
     }
 
     // Determine SSL validation behavior
-    let rejectUnauthorized = true; // Default to secure
+    let rejectUnauthorized = process.env.NODE_TLS_REJECT_UNAUTHORIZED !== '0'; // Default to secure unless explicitly disabled
 
-    // Check various ways SSL validation can be disabled (in order of precedence)
-    if (this.config.ignoreSslErrors) {
-      // Explicit configuration option
+    // Check various ways SSL validation can be disabled or enforced (in order of precedence)
+    if (this.config.ignoreSslErrors === true) {
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via ignoreSslErrors option');
+      logger.warn('[WARN] SSL certificate validation disabled via ignoreSslErrors option');
     } else if (this.config.rejectUnauthorized === false) {
-      // Explicit configuration option
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via rejectUnauthorized option');
+      logger.warn('[WARN] SSL certificate validation disabled via rejectUnauthorized option');
+    } else if (this.config.rejectUnauthorized === true) {
+      rejectUnauthorized = true;
     } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '0') {
-      // Environment variable override
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via NODE_TLS_REJECT_UNAUTHORIZED environment variable');
+      logger.warn('[WARN] SSL certificate validation disabled via NODE_TLS_REJECT_UNAUTHORIZED environment variable');
+    } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '1') {
+      rejectUnauthorized = true;
     }
 
+    // Mirror the resolved value so all HTTPS requests in this process follow it
+    process.env.NODE_TLS_REJECT_UNAUTHORIZED = rejectUnauthorized ? '1' : '0';
+
     // Set up axios instance with retry logic and enhanced SSL handling
     this.axiosInstance = axios.create({
       baseURL: this.config.apiEndpoint,
@@ -333,11 +414,11 @@ export class TestLensReporter implements Reporter {
     
     // Log custom metadata if any
     if (this.config.customMetadata && Object.keys(this.config.customMetadata).length > 0) {
-      console.log('\n📋 Custom Metadata Detected:');
+      logger.info('\n[METADATA] Custom Metadata Detected:');
       Object.entries(this.config.customMetadata).forEach(([key, value]) => {
-        console.log(`   ${key}: ${value}`);
+        logger.info(`   ${key}: ${value}`);
       });
-      console.log('');
+      logger.info('');
     }
   }
 
@@ -407,24 +488,25 @@ 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) {
-      console.log(`🚀 TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
-      console.log(`   Run ID: ${this.runId}`);
+      logger.info(`TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
+      logger.info(`   Run ID: ${this.runId}`);
     } else {
-      console.log(`🚀 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) {
-        console.log(`📦 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 {
-        console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+        logger.warn(`[WARN] Git info collection returned null - not in a git repository or git not available`);
       }
     } else {
-      console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
+      logger.info(`[INFO] Git info collection disabled (enableGitInfo: false)`);
     }
 
     // Add shard information if available
@@ -446,7 +528,7 @@ export class TestLensReporter implements Reporter {
 
   async onTestBegin(test: TestCase, result: TestResult): Promise<void> {
     // Log which test is starting
-    console.log(`\n▶️  Running test: ${test.title}`);
+    logger.info(`[TEST] Running test: ${test.title}`);
     
     const specPath = test.location.file;
     const specKey = `${specPath}-${test.parent.title}`;
@@ -527,11 +609,25 @@ export class TestLensReporter implements Reporter {
     const testId = this.getTestId(test);
     let testData = this.testMap.get(testId);
 
-    console.log(`[TestLens] onTestEnd called for test: ${test.title}, status: ${result.status}, testData exists: ${!!testData}`);
+    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) {
-      console.log(`[TestLens] Creating test data for skipped/uncreated test: ${test.title}`);
+      logger.info(`[TestLens] Creating test data for skipped/uncreated test: ${test.title}`);
       // Create spec data if not exists (skipped tests might not have spec data either)
       const specPath = test.location.file;
       const specKey = `${specPath}-${test.parent.title}`;
@@ -697,7 +793,7 @@ export class TestLensReporter implements Reporter {
 
       // Send testEnd event for all tests, regardless of status
       // This ensures tests that are interrupted or have unexpected statuses are properly recorded
-      console.log(`[TestLens] Sending testEnd - testId: ${testData.id}, status: ${testData.status}, originalStatus: ${testData.originalStatus}`);
+      logger.info(`[TestLens] Sending testEnd - testId: ${testData.id}, status: ${testData.status}, originalStatus: ${testData.originalStatus}`);
       // Send test end event to API and get response
       const testEndResponse = await this.sendToApi({
         type: 'testEnd',
@@ -710,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);
+        }
       }
     }
 
@@ -787,15 +889,30 @@ export class TestLensReporter implements Reporter {
 
     // Wait for all pending artifact uploads to complete before sending runEnd
     if (this.pendingUploads.size > 0) {
-      console.log(`⏳ 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));
-        console.log(`✅ All artifact uploads completed`);
+        logger.debug(`[OK] All artifact uploads completed`);
       } catch (error) {
-        console.error(`⚠️  Some artifact uploads failed, continuing with runEnd`);
+        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;
@@ -827,12 +944,12 @@ export class TestLensReporter implements Reporter {
 
     // Show Build Name if provided, otherwise show Run ID
     if (this.runMetadata.testlensBuildName) {
-      console.log(`📊 TestLens Report completed - Build: ${this.runMetadata.testlensBuildName}`);
-      console.log(`   Run ID: ${this.runId}`);
+      logger.info(`[COMPLETE] TestLens Report completed - Build: ${this.runMetadata.testlensBuildName}`);
+      logger.info(`   Run ID: ${this.runId}`);
     } else {
-      console.log(`📊 TestLens Report completed - Run ID: ${this.runId}`);
+      logger.info(`[COMPLETE] TestLens Report completed - Run ID: ${this.runId}`);
     }
-    console.log(`🎯 Results: ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
+    logger.info(`[RESULTS] ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
   }
 
   private async sendToApi(payload: any): Promise<any> {
@@ -848,7 +965,7 @@ export class TestLensReporter implements Reporter {
         }
       });
       if (this.config.enableRealTimeStream) {
-        console.log(`✅ 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;
@@ -863,25 +980,25 @@ export class TestLensReporter implements Reporter {
           this.runCreationFailed = true;
         }
         
-        console.error('\n' + '='.repeat(80));
+        logger.error('\n' + '='.repeat(80));
         if (errorData?.limit_type === 'test_cases') {
-          console.error('❌ TESTLENS ERROR: Test Cases Limit Reached');
+          logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
         } else if (errorData?.limit_type === 'test_runs') {
-          console.error('❌ TESTLENS ERROR: Test Runs Limit Reached');
+          logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
         } else {
-          console.error('❌ TESTLENS ERROR: Plan Limit Reached');
+          logger.error('[ERROR] TESTLENS ERROR: Plan Limit Reached');
         }
-        console.error('='.repeat(80));
-        console.error('');
-        console.error(errorData?.message || 'You have reached your plan limit.');
-        console.error('');
-        console.error(`Current usage: ${errorData?.current_usage || 'N/A'} / ${errorData?.limit || 'N/A'}`);
-        console.error('');
-        console.error('To continue, please upgrade your plan.');
-        console.error('Contact: support@alternative-path.com');
-        console.error('');
-        console.error('='.repeat(80));
-        console.error('');
+        logger.error('='.repeat(80));
+        logger.error('');
+        logger.error(errorData?.message || 'You have reached your plan limit.');
+        logger.error('');
+        logger.error(`Current usage: ${errorData?.current_usage || 'N/A'} / ${errorData?.limit || 'N/A'}`);
+        logger.error('');
+        logger.error('To continue, please upgrade your plan.');
+        logger.error('Contact: support@alternative-path.com');
+        logger.error('');
+        logger.error('='.repeat(80));
+        logger.error('');
         return; // Don't log the full error object for limit errors
       }
       
@@ -889,36 +1006,37 @@ export class TestLensReporter implements Reporter {
       if (status === 401) {
         if (errorData?.error === 'trial_expired' || errorData?.error === 'subscription_inactive' || 
             errorData?.error === 'test_cases_limit_reached' || errorData?.error === 'test_runs_limit_reached') {
-          console.error('\n' + '='.repeat(80));
+          logger.error('\n' + '='.repeat(80));
           
           if (errorData?.error === 'test_cases_limit_reached') {
-            console.error('❌ TESTLENS ERROR: Test Cases Limit Reached');
+            logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
           } else if (errorData?.error === 'test_runs_limit_reached') {
-            console.error('❌ TESTLENS ERROR: Test Runs Limit Reached');
+            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
           } else {
-            console.error('❌ TESTLENS ERROR: Your trial plan has ended');
+            logger.error('[ERROR] TESTLENS ERROR: Your trial plan has ended');
           }
           
-          console.error('='.repeat(80));
-          console.error('');
-          console.error(errorData?.message || 'Your trial period has expired.');
-          console.error('');
-          console.error('To continue using TestLens, please upgrade to Enterprise plan.');
-          console.error('Contact: ' + (errorData?.contactEmail || 'support@alternative-path.com'));
-          console.error('');
+          logger.error('='.repeat(80));
+          logger.error('');
+          logger.error(errorData?.message || 'Your trial period has expired.');
+          logger.error('');
+          logger.error('To continue using TestLens, please upgrade to Enterprise plan.');
+          logger.error('Contact: ' + (errorData?.contactEmail || 'support@alternative-path.com'));
+          logger.error('');
           if (errorData?.trial_end_date) {
-            console.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
-            console.error('');
+            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
+            logger.error('');
           }
-          console.error('='.repeat(80));
-          console.error('');
+          logger.error('='.repeat(80));
+          logger.error('');
         } else {
-          console.error(`❌ Authentication failed: ${errorData?.error || 'Invalid API key'}`);
+          logger.error(`[ERROR] Authentication failed: ${errorData?.error || 'Invalid API key'}`);
         }
       } else if (status !== 403) {
         // Log other errors (but not 403 which we handled above)
-        console.error(`❌ Failed to send ${payload.type} event to TestLens:`, {
-          message: error?.message || 'Unknown error',
+        logger.error({
+          message: `[ERROR] Failed to send ${payload.type} event to TestLens`,
+          error: error?.message || 'Unknown error',
           status: status,
           statusText: error?.response?.statusText,
           data: errorData,
@@ -941,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) {
-          console.log(`⏭️  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) {
-          console.log(`⏭️  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;
         }
         
@@ -995,7 +1115,8 @@ export class TestLensReporter implements Reporter {
           const uploadPromise = Promise.resolve().then(async () => {
             try {
               if (!attachment.path) {
-                console.log(`⏭️  [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;
               }
               
@@ -1003,7 +1124,8 @@ export class TestLensReporter implements Reporter {
               
               // Skip if upload failed or file was too large
               if (!s3Data) {
-                console.log(`⏭️  [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;
               }
               
@@ -1027,9 +1149,10 @@ export class TestLensReporter implements Reporter {
                 artifact: artifactData
               });
               
-              console.log(`📎 [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
+              logger.debug(`[ARTIFACT] [Test: ${testId.substring(0, 8)}...] Processed artifact: ${fileName}`);
             } catch (error) {
-              console.error(`❌ [Test: ${testId.substring(0, 8)}...] Failed to process artifact ${attachment.name}:`, (error as Error).message);
+              logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to process artifact ${attachment.name}: ${(error as Error).message}`);
+              this.bumpArtifactStat('failed', artifactType);
             }
           });
           
@@ -1042,8 +1165,11 @@ export class TestLensReporter implements Reporter {
           // Don't await here - let uploads happen in parallel
           // They will be awaited in onEnd
         } catch (error) {
-          console.error(`❌ [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}:`, (error as Error).message);
+          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);
       }
     }
   }
@@ -1077,17 +1203,17 @@ export class TestLensReporter implements Reporter {
         testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
       });
 
-      console.log(`📝 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;
       
       // Handle duplicate spec code blocks gracefully (when re-running tests)
       if (errorData?.error && errorData.error.includes('duplicate key value violates unique constraint')) {
-        console.log(`ℹ️  Spec code blocks already exist for: ${path.basename(specPath)} (skipped)`);
+        logger.info(`[INFO] Spec code blocks already exist for: ${path.basename(specPath)} (skipped)`);
         return;
       }
       
-      console.error('Failed to send spec code blocks:', errorData || error?.message || 'Unknown error');
+      logger.error(`Failed to send spec code blocks: ${errorData || error?.message || 'Unknown error'}`);
     }
   }
 
@@ -1153,7 +1279,7 @@ export class TestLensReporter implements Reporter {
       
       return blocks;
     } catch (error: any) {
-      console.error(`Failed to extract test blocks from ${filePath}:`, error.message);
+      logger.error(`Failed to extract test blocks from ${filePath}: ${error.message}`);
       return [];
     }
   }
@@ -1177,7 +1303,7 @@ export class TestLensReporter implements Reporter {
         }
       } catch (e) {
         // Remote info is optional - handle gracefully
-        console.log('ℹ️  No git remote configured, skipping remote info');
+        logger.info('[INFO] No git remote configured, skipping remote info');
       }
       
       const isDirty = execSync('git status --porcelain', { encoding: 'utf-8' }).trim().length > 0;
@@ -1206,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[] = [];
     
@@ -1258,7 +1391,7 @@ export class TestLensReporter implements Reporter {
       const fileSize = this.getFileSize(filePath);
       const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
       
-      console.log(`📤 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', '');
       
@@ -1290,7 +1423,7 @@ export class TestLensReporter implements Reporter {
       const { uploadUrl, s3Key, metadata } = presignedResponse.data;
       
       // Step 2: Upload directly to S3 using presigned URL
-      console.log(`⬆️  [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);
       
@@ -1312,7 +1445,7 @@ export class TestLensReporter implements Reporter {
         throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
       }
 
-      console.log(`✅ [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`;
@@ -1338,7 +1471,7 @@ export class TestLensReporter implements Reporter {
 
       if (confirmResponse.status === 201 && confirmResponse.data.success) {
         const artifact = confirmResponse.data.artifact;
-        console.log(`✅ [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,
@@ -1356,29 +1489,29 @@ export class TestLensReporter implements Reporter {
         
         if (errorData?.error === 'trial_expired' || errorData?.error === 'subscription_inactive' ||
             errorData?.error === 'test_cases_limit_reached' || errorData?.error === 'test_runs_limit_reached') {
-          console.error('\n' + '='.repeat(80));
+          logger.error('\n' + '='.repeat(80));
           
           if (errorData?.error === 'test_cases_limit_reached') {
-            console.error('❌ TESTLENS ERROR: Test Cases Limit Reached');
+            logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
           } else if (errorData?.error === 'test_runs_limit_reached') {
-            console.error('❌ TESTLENS ERROR: Test Runs Limit Reached');
+            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
           } else {
-            console.error('❌ TESTLENS ERROR: Your trial plan has ended');
+            logger.error('[ERROR] TESTLENS ERROR: Your trial plan has ended');
           }
           
-          console.error('='.repeat(80));
-          console.error('');
-          console.error(errorData?.message || 'Your trial period has expired.');
-          console.error('');
-          console.error('To continue using TestLens, please upgrade to Enterprise plan.');
-          console.error('Contact: ' + (errorData?.contactEmail || 'support@alternative-path.com'));
-          console.error('');
+          logger.error('='.repeat(80));
+          logger.error('');
+          logger.error(errorData?.message || 'Your trial period has expired.');
+          logger.error('');
+          logger.error('To continue using TestLens, please upgrade to Enterprise plan.');
+          logger.error('Contact: ' + (errorData?.contactEmail || 'support@alternative-path.com'));
+          logger.error('');
           if (errorData?.trial_end_date) {
-            console.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
-            console.error('');
+            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
+            logger.error('');
           }
-          console.error('='.repeat(80));
-          console.error('');
+          logger.error('='.repeat(80));
+          logger.error('');
           return null;
         }
       }
@@ -1396,9 +1529,9 @@ export class TestLensReporter implements Reporter {
         errorMsg = `Access denied (403) - presigned URL may have expired`;
       }
       
-      console.error(`❌ Failed to upload ${fileName} to S3:`, errorMsg);
+      logger.error(`[ERROR] Failed to upload ${fileName} to S3: ${errorMsg}`);
       if (error.response?.data) {
-        console.error('Error details:', error.response.data);
+        logger.error({ errorDetails: error.response.data }, 'Error details');
       }
       
       // Don't throw, just return null to continue with other artifacts
@@ -1417,7 +1550,7 @@ export class TestLensReporter implements Reporter {
         return mimeType || 'application/octet-stream';
       }
     } catch (error: any) {
-      console.warn(`Failed to get MIME type for ${fileName}:`, error.message);
+      logger.warn(`Failed to get MIME type for ${fileName}: ${error.message}`);
     }
     // Fallback to basic content type mapping
     const contentTypes: Record<string, string> = {
@@ -1445,11 +1578,10 @@ export class TestLensReporter implements Reporter {
       const stats = fs.statSync(filePath);
       return stats.size;
     } catch (error) {
-      console.warn(`Could not get file size for ${filePath}:`, (error as Error).message);
+      logger.warn(`Could not get file size for ${filePath}: ${(error as Error).message}`);
       return 0;
     }
   }
 }
 
 export default TestLensReporter;
-