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

package/index.js

@@ -9,6 +9,66 @@ const fs = tslib_1.__importStar(require("fs"));
 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.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;
 async function getMime() {
@@ -45,11 +105,11 @@ class TestLensReporter {
                     // 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
                 }
@@ -61,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
@@ -88,32 +154,43 @@ class TestLensReporter {
             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
         };
         if (!this.config.apiKey) {
             throw new Error('API_KEY is required for TestLensReporter. Pass it as apiKey option in your playwright config or set one of these environment variables: TESTLENS_API_KEY, TESTLENS_KEY, PLAYWRIGHT_API_KEY, PW_API_KEY, API_KEY, or APIKEY.');
         }
         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
-        // Check various ways SSL validation can be disabled (in order of precedence)
-        if (this.config.ignoreSslErrors) {
-            // Explicit configuration option
+        let rejectUnauthorized = process.env.NODE_TLS_REJECT_UNAUTHORIZED !== '0'; // Default to secure unless explicitly disabled
+        // 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_1.default.create({
             baseURL: this.config.apiEndpoint,
@@ -152,11 +229,11 @@ class TestLensReporter {
         this.runCreationFailed = false;
         // 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('');
         }
     }
     initializeRunMetadata() {
@@ -219,26 +296,27 @@ 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) {
-            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
         if (config.shard) {
@@ -257,7 +335,7 @@ class TestLensReporter {
     }
     async onTestBegin(test, result) {
         // 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}`;
         // Create or update spec data
@@ -330,10 +408,23 @@ class TestLensReporter {
     async onTestEnd(test, result) {
         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}`;
@@ -478,7 +569,7 @@ class TestLensReporter {
             });
             // 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',
@@ -491,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;
@@ -561,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) {
-            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;
@@ -598,13 +710,13 @@ class TestLensReporter {
         });
         // 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`);
     }
     async sendToApi(payload) {
         // Skip sending if run creation already failed
@@ -618,7 +730,7 @@ class TestLensReporter {
                 }
             });
             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;
@@ -632,65 +744,66 @@ class TestLensReporter {
                 if (payload.type === 'runStart' && errorData?.limit_type === 'test_runs') {
                     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
             }
             // Check for trial expiration, subscription errors, or limit errors (401)
             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,
@@ -709,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) {
-                    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;
                 }
                 try {
@@ -759,13 +874,15 @@ class TestLensReporter {
                     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;
                             }
                             const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName, testCaseDbId);
                             // 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;
                             }
                             const artifactData = {
@@ -786,10 +903,11 @@ class TestLensReporter {
                                 timestamp: new Date().toISOString(),
                                 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.message);
+                            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
@@ -801,9 +919,13 @@ class TestLensReporter {
                     // They will be awaited in onEnd
                 }
                 catch (error) {
-                    console.error(`❌ [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}:`, error.message);
+                    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) {
@@ -831,16 +953,16 @@ class TestLensReporter {
                 codeBlocks,
                 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) {
             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'}`);
         }
     }
     extractTestBlocks(filePath) {
@@ -899,7 +1021,7 @@ class TestLensReporter {
             return blocks;
         }
         catch (error) {
-            console.error(`Failed to extract test blocks from ${filePath}:`, error.message);
+            logger.error(`Failed to extract test blocks from ${filePath}: ${error.message}`);
             return [];
         }
     }
@@ -922,7 +1044,7 @@ class TestLensReporter {
             }
             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 = (0, child_process_1.execSync)('git status --porcelain', { encoding: 'utf-8' }).trim().length > 0;
             return {
@@ -951,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
@@ -994,7 +1122,7 @@ class TestLensReporter {
             // Check file size first
             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', '');
             // Step 1: Request pre-signed URL from server
             const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
@@ -1019,7 +1147,7 @@ class TestLensReporter {
             }
             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);
             // 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
@@ -1037,7 +1165,7 @@ class TestLensReporter {
             if (uploadResponse.status !== 200) {
                 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`;
             const confirmBody = {
@@ -1059,7 +1187,7 @@ class TestLensReporter {
             });
             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,
@@ -1078,29 +1206,29 @@ class TestLensReporter {
                 const errorData = error?.response?.data;
                 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;
                 }
             }
@@ -1118,9 +1246,9 @@ class TestLensReporter {
             else if (error.response?.status === 403) {
                 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
             return null;
@@ -1138,7 +1266,7 @@ class TestLensReporter {
             }
         }
         catch (error) {
-            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 = {
@@ -1165,7 +1293,7 @@ class TestLensReporter {
             return stats.size;
         }
         catch (error) {
-            console.warn(`Could not get file size for ${filePath}:`, error.message);
+            logger.warn(`Could not get file size for ${filePath}: ${error.message}`);
             return 0;
         }
     }