@alternative-path/testlens-playwright-reporter 0.4.7 → 0.4.8

package/index.d.ts

@@ -99,6 +99,7 @@ export interface RunMetadata {
     passedTests?: number;
     failedTests?: number;
     skippedTests?: number;
+    timedOutTests?: number;
     status?: string;
     testlensBuildName?: string;
     customMetadata?: Record<string, string | string[]>;
@@ -161,6 +162,7 @@ export declare class TestLensReporter implements Reporter {
     private runCreationFailed;
     private cliArgs;
     private pendingUploads;
+    private traceNetworkRows;
     private artifactStats;
     private artifactsSeen;
     /**

package/index.js

@@ -121,6 +121,7 @@ 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.traceNetworkRows = []; // Network requests/responses from trace zip for current test
         this.artifactStats = {
             uploaded: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
             skipped: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
@@ -406,7 +407,9 @@ class TestLensReporter {
         }
     }
     async onTestEnd(test, result) {
+        this.traceNetworkRows = []; // Reset at start of each test
         const testId = this.getTestId(test);
+        let testCaseId = '';
         let testData = this.testMap.get(testId);
         logger.debug(`[ARTIFACTS] attachments=${result.attachments?.length ?? 0}`);
         if (result.attachments && result.attachments.length > 0) {
@@ -577,10 +580,15 @@ class TestLensReporter {
                 timestamp: new Date().toISOString(),
                 test: testData
             });
+            testCaseId = testEndResponse?.testCaseId;
             // Handle artifacts (test case is now guaranteed to be in database)
             if (this.config.enableArtifacts) {
-                // Pass test case DB ID if available for faster lookups
-                await this.processArtifacts(testId, result, testEndResponse?.testCaseId);
+                // Pass test case DB ID if available for faster lookups; pass status/endTime so backend
+                // can fix up test case status if testEnd failed but artifact request succeeds
+                await this.processArtifacts(testId, result, testEndResponse?.testCaseId, {
+                    status: testData.status,
+                    endTime: testData.endTime
+                });
             }
             else if (result.attachments && result.attachments.length > 0) {
                 for (const attachment of result.attachments) {
@@ -650,7 +658,7 @@ class TestLensReporter {
                     spec: specData
                 });
                 // Send spec code blocks to API
-                await this.sendSpecCodeBlocks(specPath);
+                await this.sendSpecCodeBlocks(specPath, test.title, testData?.errors || [], this.runId, testId, testCaseId);
             }
         }
     }
@@ -815,7 +823,7 @@ class TestLensReporter {
             // Don't throw error to avoid breaking test execution
         }
     }
-    async processArtifacts(testId, result, testCaseDbId) {
+    async processArtifacts(testId, result, testCaseDbId, testEndPayload) {
         // Skip artifact processing if run creation failed
         if (this.runCreationFailed) {
             return;
@@ -839,6 +847,81 @@ class TestLensReporter {
                     this.bumpArtifactStat('skipped', artifactType);
                     continue;
                 }
+                // Trace zip (default from Playwright): extract network-like data and print to console
+                const isTraceZip = artifactType === 'trace' && attachment.path && (attachment.path.endsWith('.zip') || attachment.contentType === 'application/zip');
+                if (isTraceZip) {
+                    try {
+                        const traceStart = Date.now();
+                        const AdmZip = require('adm-zip');
+                        const zip = new AdmZip(attachment.path);
+                        const entries = zip.getEntries();
+                        const networkRows = [];
+                        for (const e of entries) {
+                            if (e.isDirectory || !e.getData)
+                                continue;
+                            const name = (e.entryName || '').toLowerCase();
+                            const isNetworkFile = name.endsWith('.network') || name.includes('.network') || name === 'network';
+                            if (!isNetworkFile)
+                                continue;
+                            const data = e.getData();
+                            if (!data)
+                                continue;
+                            const text = data.toString('utf8');
+                            if (!text || text.length < 2)
+                                continue;
+                            const isApplicationJson = (o) => {
+                                if (!o || typeof o !== 'object')
+                                    return false;
+                                const res = o.snapshot?.response ?? o.response;
+                                if (!res)
+                                    return false;
+                                const mime = res.content?.mimeType;
+                                if (typeof mime === 'string') {
+                                    const type = mime.split(';')[0].trim().toLowerCase();
+                                    return type === 'application/json';
+                                }
+                                const headers = res.headers;
+                                if (Array.isArray(headers)) {
+                                    const ct = headers.find((h) => (h.name || '').toLowerCase() === 'content-type');
+                                    const val = ct?.value;
+                                    if (typeof val === 'string') {
+                                        const type = val.split(';')[0].trim().toLowerCase();
+                                        return type === 'application/json';
+                                    }
+                                }
+                                return false;
+                            };
+                            const lines = text.split(/\r?\n/).filter(Boolean);
+                            for (const line of lines) {
+                                try {
+                                    const obj = JSON.parse(line);
+                                    if (obj && typeof obj === 'object' && isApplicationJson(obj))
+                                        networkRows.push(obj);
+                                }
+                                catch (_) { }
+                            }
+                            if (networkRows.length === 0 && lines.length > 0) {
+                                try {
+                                    const arr = JSON.parse(text);
+                                    if (Array.isArray(arr))
+                                        networkRows.push(...arr.filter((x) => x != null && typeof x === 'object' && isApplicationJson(x)));
+                                }
+                                catch (_) { }
+                            }
+                        }
+                        const durationMs = Date.now() - traceStart;
+                        if (networkRows.length > 0) {
+                            this.traceNetworkRows = networkRows;
+                        }
+                        else {
+                            this.traceNetworkRows = [];
+                        }
+                    }
+                    catch (e) {
+                        logger.warn('[TRACE] Could not read trace zip: ' + (e && e.message));
+                        this.traceNetworkRows = [];
+                    }
+                }
                 try {
                     // Determine proper filename with extension
                     // Playwright attachment.name often doesn't have extension, so we need to derive it
@@ -894,7 +977,12 @@ class TestLensReporter {
                                 fileSize: this.getFileSize(attachment.path),
                                 storageType: 's3',
                                 s3Key: s3Data.key,
-                                s3Url: s3Data.url
+                                s3Url: s3Data.url,
+                                // So backend can fix test case status if testEnd failed but artifact succeeded
+                                ...(testEndPayload && {
+                                    testStatus: testEndPayload.status,
+                                    testEndTime: testEndPayload.endTime
+                                })
                             };
                             // Send artifact data to API
                             await this.sendToApi({
@@ -928,12 +1016,12 @@ class TestLensReporter {
             }
         }
     }
-    async sendSpecCodeBlocks(specPath) {
+    async sendSpecCodeBlocks(specPath, testName, errors, runId, test_id, testCaseId) {
         try {
             // Extract code blocks using built-in parser
             const testBlocks = this.extractTestBlocks(specPath);
             // Transform blocks to match backend API expectations
-            const codeBlocks = testBlocks.map(block => ({
+            const codeBlocks = testBlocks.filter(block => block.name === testName).map(block => ({
                 type: block.type, // 'test' or 'describe'
                 name: block.name, // test/describe name
                 content: block.content, // full code content
@@ -951,7 +1039,12 @@ class TestLensReporter {
             await this.axiosInstance.post(specEndpoint, {
                 filePath: path.relative(process.cwd(), specPath),
                 codeBlocks,
-                testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
+                errors,
+                traceNetworkRows: this.traceNetworkRows,
+                testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, ''),
+                runId,
+                test_id,
+                testCaseId
             });
             logger.debug(`Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
         }
@@ -970,53 +1063,94 @@ class TestLensReporter {
             const content = fs.readFileSync(filePath, 'utf-8');
             const blocks = [];
             const lines = content.split('\n');
-            let currentDescribe = null;
-            let braceCount = 0;
+            // Use a stack to track nested describe blocks with their brace depths
+            const describeStack = [];
+            let globalBraceCount = 0;
             let inBlock = false;
             let blockStart = -1;
+            let blockBraceCount = 0;
             let blockType = 'test';
             let blockName = '';
+            let blockDescribe = undefined;
             for (let i = 0; i < lines.length; i++) {
                 const line = lines[i];
                 const trimmedLine = line.trim();
-                // Check for describe blocks
-                const describeMatch = trimmedLine.match(/describe\s*\(\s*['"`]([^'"`]+)['"`]/);
-                if (describeMatch) {
-                    currentDescribe = describeMatch[1];
-                }
-                // Check for test blocks
-                const testMatch = trimmedLine.match(/test\s*\(\s*['"`]([^'"`]+)['"`]/);
-                if (testMatch && !inBlock) {
-                    blockType = 'test';
-                    blockName = testMatch[1];
-                    blockStart = i;
-                    braceCount = 0;
-                    inBlock = true;
+                if (!inBlock) {
+                    // Check for describe blocks: describe(), test.describe(), test.describe.serial(), etc.
+                    const describeMatch = trimmedLine.match(/(?:test\.)?describe(?:\.(?:serial|parallel|only|skip|fixme))?\s*\(\s*['"`]([^'"`]+)['"`]/);
+                    if (describeMatch) {
+                        // Count braces on this line to find the opening brace
+                        let lineOpenBraces = 0;
+                        for (const char of line) {
+                            if (char === '{') {
+                                globalBraceCount++;
+                                lineOpenBraces++;
+                            }
+                            if (char === '}')
+                                globalBraceCount--;
+                        }
+                        // Push describe onto stack with the current brace depth
+                        describeStack.push({ name: describeMatch[1], braceDepth: globalBraceCount });
+                        continue;
+                    }
+                    // Check for test blocks: test(), test.only(), test.skip(), test.fixme(), it(), it.only(), etc.
+                    const testMatch = trimmedLine.match(/(?:test|it)(?:\.(?:only|skip|fixme|slow))?\s*\(\s*['"`]([^'"`]+)['"`]/);
+                    if (testMatch) {
+                        blockType = 'test';
+                        blockName = testMatch[1];
+                        blockStart = i;
+                        blockBraceCount = 0;
+                        // Capture the current innermost describe name
+                        blockDescribe = describeStack.length > 0 ? describeStack[describeStack.length - 1].name : undefined;
+                        inBlock = true;
+                    }
                 }
-                // Count braces when in a block
+                // Count braces
                 if (inBlock) {
                     for (const char of line) {
-                        if (char === '{')
-                            braceCount++;
-                        if (char === '}')
-                            braceCount--;
-                        if (braceCount === 0 && blockStart !== -1 && i > blockStart) {
-                            // End of block found
+                        if (char === '{') {
+                            blockBraceCount++;
+                            globalBraceCount++;
+                        }
+                        if (char === '}') {
+                            blockBraceCount--;
+                            globalBraceCount--;
+                        }
+                        if (blockBraceCount === 0 && blockStart !== -1 && i > blockStart) {
+                            // End of test block found
                             const blockContent = lines.slice(blockStart, i + 1).join('\n');
                             blocks.push({
                                 type: blockType,
                                 name: blockName,
                                 content: blockContent,
-                                describe: currentDescribe || undefined,
+                                describe: blockDescribe,
                                 startLine: blockStart + 1,
                                 endLine: i + 1
                             });
                             inBlock = false;
                             blockStart = -1;
+                            // Pop any describe blocks that have closed
+                            while (describeStack.length > 0 && globalBraceCount < describeStack[describeStack.length - 1].braceDepth) {
+                                describeStack.pop();
+                            }
                             break;
                         }
                     }
                 }
+                else {
+                    // Track braces outside of test blocks (for describe open/close)
+                    for (const char of line) {
+                        if (char === '{')
+                            globalBraceCount++;
+                        if (char === '}') {
+                            globalBraceCount--;
+                            // Pop any describe blocks that have closed
+                            while (describeStack.length > 0 && globalBraceCount < describeStack[describeStack.length - 1].braceDepth) {
+                                describeStack.pop();
+                            }
+                        }
+                    }
+                }
             }
             return blocks;
         }
@@ -1127,7 +1261,6 @@ class TestLensReporter {
             // Step 1: Request pre-signed URL from server
             const presignedUrlEndpoint = `${baseUrl}/api/v1/artifacts/public/presigned-url`;
             const requestBody = {
-                apiKey: this.config.apiKey,
                 testRunId: this.runId,
                 testId: testId,
                 fileName: fileName,
@@ -1169,7 +1302,6 @@ class TestLensReporter {
             // Step 3: Confirm upload with server to save metadata
             const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
             const confirmBody = {
-                apiKey: this.config.apiKey,
                 testRunId: this.runId,
                 testId: testId,
                 s3Key: s3Key,
@@ -1261,7 +1393,7 @@ class TestLensReporter {
             // Try different ways to access getType method
             const getType = mime.getType || mime.default?.getType;
             if (typeof getType === 'function') {
-                const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
+                const mimeType = getType.call(mime, ext) || (mime.default ? getType.call(mime.default, ext) : null);
                 return mimeType || 'application/octet-stream';
             }
         }

package/index.ts

@@ -61,8 +61,8 @@ const logger = pino({
           typeof args[0] === 'string'
             ? args[0]
             : typeof args[1] === 'string'
-            ? args[1]
-            : '';
+              ? args[1]
+              : '';
         if (shouldSuppressInfo(msg)) {
           return;
         }
@@ -187,6 +187,7 @@ export interface RunMetadata {
   passedTests?: number;
   failedTests?: number;
   skippedTests?: number;
+  timedOutTests?: number;
   status?: string;
   testlensBuildName?: string;
   customMetadata?: Record<string, string | string[]>;
@@ -250,6 +251,7 @@ 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 traceNetworkRows: unknown[] = []; // Network requests/responses from trace zip for current test
   private artifactStats = {
     uploaded: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
     skipped: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
@@ -263,7 +265,7 @@ export class TestLensReporter implements Reporter {
    */
   private static parseCustomArgs(): Record<string, any> {
     const customArgs: Record<string, any> = {};
-    
+
     // Common environment variable names for build metadata
     const envVarMappings: Record<string, string[]> = {
       // Support both TestLens-specific names (recommended) and common CI names
@@ -275,7 +277,7 @@ export class TestLensReporter implements Reporter {
       'project': ['PROJECT', 'PROJECT_NAME'],
       'customvalue': ['CUSTOMVALUE', 'CUSTOM_VALUE']
     };
-    
+
     // Check for each metadata key
     Object.entries(envVarMappings).forEach(([key, envVars]) => {
       for (const envVar of envVars) {
@@ -293,7 +295,7 @@ export class TestLensReporter implements Reporter {
         }
       }
     });
-    
+
     return customArgs;
   }
 
@@ -301,11 +303,11 @@ export class TestLensReporter implements Reporter {
     // Parse custom CLI arguments
     const customArgs = TestLensReporter.parseCustomArgs();
     this.cliArgs = customArgs; // Store CLI args separately for later use
-    
+
     // Allow API key from environment variable if not provided in config
     // Check multiple environment variable names in priority order (uppercase and lowercase)
-    const apiKey = options.apiKey 
-      || process.env.TESTLENS_API_KEY 
+    const apiKey = options.apiKey
+      || process.env.TESTLENS_API_KEY
       || process.env.testlens_api_key
       || process.env.TESTLENS_KEY
       || process.env.testlens_key
@@ -314,7 +316,7 @@ export class TestLensReporter implements Reporter {
       || process.env.playwright_api_key
       || process.env.PW_API_KEY
       || process.env.pw_api_key;
-    
+
     this.config = {
       apiEndpoint: options.apiEndpoint || 'https://testlens.qa-path.com/api/v1/webhook/playwright',
       apiKey: apiKey, // API key from config or environment variable
@@ -335,7 +337,7 @@ export class TestLensReporter implements Reporter {
     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) {
       logger.debug('✓ Using API key from environment variable');
     }
@@ -389,11 +391,11 @@ export class TestLensReporter implements Reporter {
       (response) => response,
       async (error: any) => {
         const originalRequest = error.config;
-        
+
         if (!originalRequest._retry && error.response?.status >= 500) {
           originalRequest._retry = true;
           originalRequest._retryCount = (originalRequest._retryCount || 0) + 1;
-          
+
           if (originalRequest._retryCount <= this.config.retryAttempts) {
             // Exponential backoff
             const delay = Math.pow(2, originalRequest._retryCount) * 1000;
@@ -401,7 +403,7 @@ export class TestLensReporter implements Reporter {
             return this.axiosInstance(originalRequest);
           }
         }
-        
+
         return Promise.reject(error);
       }
     );
@@ -411,7 +413,7 @@ export class TestLensReporter implements Reporter {
     this.specMap = new Map<string, SpecData>();
     this.testMap = new Map<string, TestData>();
     this.runCreationFailed = false;
-    
+
     // Log custom metadata if any
     if (this.config.customMetadata && Object.keys(this.config.customMetadata).length > 0) {
       logger.debug('\n[METADATA] Custom Metadata Detected:');
@@ -433,11 +435,11 @@ export class TestLensReporter implements Reporter {
       nodeVersion: process.version,
       testlensVersion: this.getTestLensVersion()
     };
-    
+
     // Add custom metadata if provided
     if (this.config.customMetadata && Object.keys(this.config.customMetadata).length > 0) {
       metadata.customMetadata = this.config.customMetadata;
-      
+
       // Extract testlensBuildName as a dedicated field for dashboard display
       if (this.config.customMetadata.testlensBuildName) {
         const buildName = this.config.customMetadata.testlensBuildName;
@@ -445,7 +447,7 @@ export class TestLensReporter implements Reporter {
         metadata.testlensBuildName = Array.isArray(buildName) ? buildName[0] : buildName;
       }
     }
-    
+
     return metadata;
   }
 
@@ -496,7 +498,7 @@ export class TestLensReporter implements Reporter {
     } else {
       logger.debug(`TestLens Reporter starting - Run ID: ${this.runId}`);
     }
-    
+
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
@@ -529,7 +531,7 @@ export class TestLensReporter implements Reporter {
   async onTestBegin(test: TestCase, result: TestResult): Promise<void> {
     // Log which test is starting
     logger.debug(`[TEST] Running test: ${test.title}`);
-    
+
     const specPath = test.location.file;
     const specKey = `${specPath}-${test.parent.title}`;
 
@@ -557,7 +559,7 @@ export class TestLensReporter implements Reporter {
     }
 
     const testId = this.getTestId(test);
-    
+
     // Only send testStart event on first attempt (retry 0)
     if (result.retry === 0) {
       // Create test data
@@ -606,7 +608,9 @@ export class TestLensReporter implements Reporter {
   }
 
   async onTestEnd(test: TestCase, result: TestResult): Promise<void> {
+    this.traceNetworkRows = []; // Reset at start of each test
     const testId = this.getTestId(test);
+    let testCaseId = '';
     let testData = this.testMap.get(testId);
 
     logger.debug(`[ARTIFACTS] attachments=${result.attachments?.length ?? 0}`);
@@ -631,7 +635,7 @@ export class TestLensReporter implements Reporter {
       // 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}`;
-      
+
       if (!this.specMap.has(specKey)) {
         const extractedTags = this.extractTags(test);
         const specData: SpecData = {
@@ -700,25 +704,25 @@ export class TestLensReporter implements Reporter {
       testData.endTime = new Date().toISOString();
       testData.errorMessages = result.errors.map((error: any) => error.message || error.toString());
       testData.currentRetry = result.retry;
-      
+
       // Capture test location
       testData.location = {
         file: path.relative(process.cwd(), test.location.file),
         line: test.location.line,
         column: test.location.column
       };
-      
+
       // Capture rich error details like Playwright's HTML report
       testData.errors = result.errors.map((error: any) => {
         const testError: TestError = {
           message: error.message || error.toString()
         };
-        
+
         // Capture stack trace
         if (error.stack) {
           testError.stack = error.stack;
         }
-        
+
         // Capture error location
         if (error.location) {
           testError.location = {
@@ -727,22 +731,22 @@ export class TestLensReporter implements Reporter {
             column: error.location.column
           };
         }
-        
+
         // Capture code snippet around error - from Playwright error object
         if (error.snippet) {
           testError.snippet = error.snippet;
         }
-        
+
         // Capture expected/actual values for assertion failures
         // Playwright stores these as specially formatted strings in the message
         const message = error.message || '';
-        
+
         // Try to parse expected pattern from toHaveURL and similar assertions
         const expectedPatternMatch = message.match(/Expected pattern:\s*(.+?)(?:\n|$)/);
         if (expectedPatternMatch) {
           testError.expected = expectedPatternMatch[1].trim();
         }
-        
+
         // Also try "Expected string:" format
         if (!testError.expected) {
           const expectedStringMatch = message.match(/Expected string:\s*["']?(.+?)["']?(?:\n|$)/);
@@ -750,13 +754,13 @@ export class TestLensReporter implements Reporter {
             testError.expected = expectedStringMatch[1].trim();
           }
         }
-        
+
         // Try to parse received/actual value
         const receivedMatch = message.match(/Received (?:string|value):\s*["']?(.+?)["']?(?:\n|$)/);
         if (receivedMatch) {
           testError.actual = receivedMatch[1].trim();
         }
-        
+
         // Parse call log entries for debugging info (timeouts, retries, etc.)
         const callLogMatch = message.match(/Call log:([\s\S]*?)(?=\n\n|\n\s*\d+\s*\||$)/);
         if (callLogMatch) {
@@ -766,19 +770,19 @@ export class TestLensReporter implements Reporter {
             testError.diff = callLog; // Reuse diff field for call log
           }
         }
-        
+
         // Parse timeout information - multiple formats
         const timeoutMatch = message.match(/(?:with timeout|Timeout:?)\s*(\d+)ms/i);
         if (timeoutMatch) {
           testError.timeout = parseInt(timeoutMatch[1], 10);
         }
-        
+
         // Parse matcher name (e.g., toHaveURL, toBeVisible)
         const matcherMatch = message.match(/expect\([^)]+\)\.(\w+)/);
         if (matcherMatch) {
           testError.matcherName = matcherMatch[1];
         }
-        
+
         // Extract code snippet from message if not already captured
         // Look for lines like "  9 |      await page.click..." or "> 11 |      await expect..."
         if (!testError.snippet) {
@@ -787,7 +791,7 @@ export class TestLensReporter implements Reporter {
             testError.snippet = codeSnippetMatch[1].trim();
           }
         }
-        
+
         return testError;
       });
 
@@ -801,11 +805,16 @@ export class TestLensReporter implements Reporter {
         timestamp: new Date().toISOString(),
         test: testData
       });
+      testCaseId = testEndResponse?.testCaseId;
 
       // Handle artifacts (test case is now guaranteed to be in database)
       if (this.config.enableArtifacts) {
-        // Pass test case DB ID if available for faster lookups
-        await this.processArtifacts(testId, result, testEndResponse?.testCaseId);
+        // Pass test case DB ID if available for faster lookups; pass status/endTime so backend
+        // can fix up test case status if testEnd failed but artifact request succeeds
+        await this.processArtifacts(testId, result, testEndResponse?.testCaseId, {
+          status: testData.status,
+          endTime: testData.endTime
+        });
       } else if (result.attachments && result.attachments.length > 0) {
         for (const attachment of result.attachments) {
           const artifactType = this.getArtifactType(attachment.name);
@@ -844,7 +853,7 @@ export class TestLensReporter implements Reporter {
             return this.testMap.get(tId);
           })
           .filter((tData: TestData | undefined): tData is TestData => !!tData);
-        
+
         if (executedTests.length > 0) {
           const allTestStatuses = executedTests.map(tData => tData.status);
           if (allTestStatuses.every(status => status === 'passed')) {
@@ -866,9 +875,9 @@ export class TestLensReporter implements Reporter {
         if (aggregatedTags.length > 0) {
           specData.tags = aggregatedTags;
         }
-        
+
         specData.endTime = new Date().toISOString();
-        
+
         // Send spec end event to API
         await this.sendToApi({
           type: 'specEnd',
@@ -878,7 +887,7 @@ export class TestLensReporter implements Reporter {
         });
 
         // Send spec code blocks to API
-        await this.sendSpecCodeBlocks(specPath);
+        await this.sendSpecCodeBlocks(specPath, test.title, testData?.errors || [], this.runId, testId, testCaseId);
       }
     }
   }
@@ -957,7 +966,7 @@ export class TestLensReporter implements Reporter {
     if (this.runCreationFailed && payload.type !== 'runStart') {
       return null;
     }
-    
+
     try {
       const response = await this.axiosInstance.post('', payload, {
         headers: {
@@ -972,14 +981,14 @@ export class TestLensReporter implements Reporter {
     } catch (error: any) {
       const errorData = error?.response?.data;
       const status = error?.response?.status;
-      
+
       // Check for limit exceeded (403)
       if (status === 403 && errorData?.error === 'limit_exceeded') {
         // Set flag to skip subsequent events
         if (payload.type === 'runStart' && errorData?.limit_type === 'test_runs') {
           this.runCreationFailed = true;
         }
-        
+
         logger.error('\n' + '='.repeat(80));
         if (errorData?.limit_type === 'test_cases') {
           logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
@@ -1001,13 +1010,13 @@ export class TestLensReporter implements Reporter {
         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') {
+        if (errorData?.error === 'trial_expired' || errorData?.error === 'subscription_inactive' ||
+          errorData?.error === 'test_cases_limit_reached' || errorData?.error === 'test_runs_limit_reached') {
           logger.error('\n' + '='.repeat(80));
-          
+
           if (errorData?.error === 'test_cases_limit_reached') {
             logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
           } else if (errorData?.error === 'test_runs_limit_reached') {
@@ -1015,7 +1024,7 @@ export class TestLensReporter implements Reporter {
           } else {
             logger.error('[ERROR] TESTLENS ERROR: Your trial plan has ended');
           }
-          
+
           logger.error('='.repeat(80));
           logger.error('');
           logger.error(errorData?.message || 'Your trial period has expired.');
@@ -1045,46 +1054,115 @@ export class TestLensReporter implements Reporter {
           method: error?.config?.method
         });
       }
-      
+
       // Don't throw error to avoid breaking test execution
     }
   }
 
-  private async processArtifacts(testId: string, result: TestResult, testCaseDbId?: string): Promise<void> {
+  private async processArtifacts(
+    testId: string,
+    result: TestResult,
+    testCaseDbId?: string,
+    testEndPayload?: { status: string; endTime: string }
+  ): Promise<void> {
     // Skip artifact processing if run creation failed
     if (this.runCreationFailed) {
       return;
     }
-    
+
     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 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.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.debug(`[SKIP] Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
           this.bumpArtifactStat('skipped', artifactType);
           continue;
         }
-        
+
+        // Trace zip (default from Playwright): extract network-like data and print to console
+        const isTraceZip = artifactType === 'trace' && attachment.path && (attachment.path.endsWith('.zip') || attachment.contentType === 'application/zip');
+        if (isTraceZip) {
+          try {
+            const traceStart = Date.now();
+            const AdmZip = require('adm-zip');
+            const zip = new AdmZip(attachment.path);
+            const entries = zip.getEntries();
+            const networkRows: unknown[] = [];
+            for (const e of entries) {
+              if (e.isDirectory || !e.getData) continue;
+              const name = (e.entryName || '').toLowerCase();
+              const isNetworkFile = name.endsWith('.network') || name.includes('.network') || name === 'network';
+              if (!isNetworkFile) continue;
+              const data = e.getData();
+              if (!data) continue;
+              const text = data.toString('utf8');
+              if (!text || text.length < 2) continue;
+              const isApplicationJson = (o: unknown): boolean => {
+                if (!o || typeof o !== 'object') return false;
+                const res = (o as any).snapshot?.response ?? (o as any).response;
+                if (!res) return false;
+                const mime = res.content?.mimeType;
+                if (typeof mime === 'string') {
+                  const type = mime.split(';')[0].trim().toLowerCase();
+                  return type === 'application/json';
+                }
+                const headers = res.headers;
+                if (Array.isArray(headers)) {
+                  const ct = headers.find((h: any) => (h.name || '').toLowerCase() === 'content-type');
+                  const val = ct?.value;
+                  if (typeof val === 'string') {
+                    const type = val.split(';')[0].trim().toLowerCase();
+                    return type === 'application/json';
+                  }
+                }
+                return false;
+              };
+              const lines = text.split(/\r?\n/).filter(Boolean);
+              for (const line of lines) {
+                try {
+                  const obj = JSON.parse(line);
+                  if (obj && typeof obj === 'object' && isApplicationJson(obj)) networkRows.push(obj);
+                } catch (_) { }
+              }
+              if (networkRows.length === 0 && lines.length > 0) {
+                try {
+                  const arr = JSON.parse(text);
+                  if (Array.isArray(arr)) networkRows.push(...arr.filter((x: unknown) => x != null && typeof x === 'object' && isApplicationJson(x)));
+                } catch (_) { }
+              }
+            }
+            const durationMs = Date.now() - traceStart;
+            if (networkRows.length > 0) {
+              this.traceNetworkRows = networkRows;
+            } else {
+              this.traceNetworkRows = [];
+            }
+          } catch (e) {
+            logger.warn('[TRACE] Could not read trace zip: ' + (e && (e as Error).message));
+            this.traceNetworkRows = [];
+          }
+        }
+
         try {
           // Determine proper filename with extension
           // Playwright attachment.name often doesn't have extension, so we need to derive it
           let fileName = attachment.name;
           const existingExt = path.extname(fileName);
-          
+
           if (!existingExt) {
             // Get extension from the actual file path
             const pathExt = path.extname(attachment.path);
@@ -1119,16 +1197,16 @@ export class TestLensReporter implements Reporter {
                 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.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - upload failed or file too large`);
                 this.bumpArtifactStat('failed', artifactType);
                 return;
               }
-              
+
               const artifactData = {
                 testId,
                 type: this.getArtifactType(attachment.name),
@@ -1138,7 +1216,12 @@ export class TestLensReporter implements Reporter {
                 fileSize: this.getFileSize(attachment.path),
                 storageType: 's3',
                 s3Key: s3Data.key,
-                s3Url: s3Data.url
+                s3Url: s3Data.url,
+                // So backend can fix test case status if testEnd failed but artifact succeeded
+                ...(testEndPayload && {
+                  testStatus: testEndPayload.status,
+                  testEndTime: testEndPayload.endTime
+                })
               };
 
               // Send artifact data to API
@@ -1148,20 +1231,20 @@ export class TestLensReporter implements Reporter {
                 timestamp: new Date().toISOString(),
                 artifact: artifactData
               });
-              
+
               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);
             }
           });
-          
+
           // Track this upload and ensure cleanup on completion
           this.pendingUploads.add(uploadPromise);
           uploadPromise.finally(() => {
             this.pendingUploads.delete(uploadPromise);
           });
-          
+
           // Don't await here - let uploads happen in parallel
           // They will be awaited in onEnd
         } catch (error) {
@@ -1174,20 +1257,20 @@ export class TestLensReporter implements Reporter {
     }
   }
 
-  private async sendSpecCodeBlocks(specPath: string): Promise<void> {
+  private async sendSpecCodeBlocks(specPath: string, testName: string, errors: unknown[], runId: string, test_id: string, testCaseId: string): Promise<void> {
     try {
       // Extract code blocks using built-in parser
       const testBlocks = this.extractTestBlocks(specPath);
-      
+
       // Transform blocks to match backend API expectations
-      const codeBlocks = testBlocks.map(block => ({
+      const codeBlocks = testBlocks.filter(block => block.name === testName).map(block => ({
         type: block.type,           // 'test' or 'describe'
         name: block.name,           // test/describe name
         content: block.content,     // full code content
         summary: null,              // optional
         describe: block.describe    // parent describe block name
       }));
-      
+
       // Send to dedicated spec code blocks API endpoint
       // Extract base URL - handle both full and partial endpoint patterns
       let baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
@@ -1196,23 +1279,28 @@ export class TestLensReporter implements Reporter {
         baseUrl = this.config.apiEndpoint.replace('/webhook/playwright', '');
       }
       const specEndpoint = `${baseUrl}/api/v1/webhook/playwright/spec-code-blocks`;
-      
+
       await this.axiosInstance.post(specEndpoint, {
         filePath: path.relative(process.cwd(), specPath),
         codeBlocks,
-        testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
+        errors,
+        traceNetworkRows: this.traceNetworkRows,
+        testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, ''),
+        runId,
+        test_id,
+        testCaseId
       });
 
       logger.debug(`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')) {
         logger.debug(`[INFO] Spec code blocks already exist for: ${path.basename(specPath)} (skipped)`);
         return;
       }
-      
+
       logger.error(`Failed to send spec code blocks: ${errorData || error?.message || 'Unknown error'}`);
     }
   }
@@ -1222,61 +1310,102 @@ export class TestLensReporter implements Reporter {
       const content = fs.readFileSync(filePath, 'utf-8');
       const blocks: CodeBlock[] = [];
       const lines = content.split('\n');
-      
-      let currentDescribe: string | null = null;
-      let braceCount = 0;
+
+      // Use a stack to track nested describe blocks with their brace depths
+      const describeStack: { name: string; braceDepth: number }[] = [];
+      let globalBraceCount = 0;
       let inBlock = false;
       let blockStart = -1;
+      let blockBraceCount = 0;
       let blockType: 'test' | 'describe' = 'test';
       let blockName = '';
-      
+      let blockDescribe: string | undefined = undefined;
+
       for (let i = 0; i < lines.length; i++) {
         const line = lines[i];
         const trimmedLine = line.trim();
-        
-        // Check for describe blocks
-        const describeMatch = trimmedLine.match(/describe\s*\(\s*['"`]([^'"`]+)['"`]/);
-        if (describeMatch) {
-          currentDescribe = describeMatch[1];
-        }
-        
-        // Check for test blocks
-        const testMatch = trimmedLine.match(/test\s*\(\s*['"`]([^'"`]+)['"`]/);
-        if (testMatch && !inBlock) {
-          blockType = 'test';
-          blockName = testMatch[1];
-          blockStart = i;
-          braceCount = 0;
-          inBlock = true;
+
+        if (!inBlock) {
+          // Check for describe blocks: describe(), test.describe(), test.describe.serial(), etc.
+          const describeMatch = trimmedLine.match(/(?:test\.)?describe(?:\.(?:serial|parallel|only|skip|fixme))?\s*\(\s*['"`]([^'"`]+)['"`]/);
+          if (describeMatch) {
+            // Count braces on this line to find the opening brace
+            let lineOpenBraces = 0;
+            for (const char of line) {
+              if (char === '{') {
+                globalBraceCount++;
+                lineOpenBraces++;
+              }
+              if (char === '}') globalBraceCount--;
+            }
+            // Push describe onto stack with the current brace depth
+            describeStack.push({ name: describeMatch[1], braceDepth: globalBraceCount });
+            continue;
+          }
+
+          // Check for test blocks: test(), test.only(), test.skip(), test.fixme(), it(), it.only(), etc.
+          const testMatch = trimmedLine.match(/(?:test|it)(?:\.(?:only|skip|fixme|slow))?\s*\(\s*['"`]([^'"`]+)['"`]/);
+          if (testMatch) {
+            blockType = 'test';
+            blockName = testMatch[1];
+            blockStart = i;
+            blockBraceCount = 0;
+            // Capture the current innermost describe name
+            blockDescribe = describeStack.length > 0 ? describeStack[describeStack.length - 1].name : undefined;
+            inBlock = true;
+          }
         }
-        
-        // Count braces when in a block
+
+        // Count braces
         if (inBlock) {
           for (const char of line) {
-            if (char === '{') braceCount++;
-            if (char === '}') braceCount--;
-            
-            if (braceCount === 0 && blockStart !== -1 && i > blockStart) {
-              // End of block found
+            if (char === '{') {
+              blockBraceCount++;
+              globalBraceCount++;
+            }
+            if (char === '}') {
+              blockBraceCount--;
+              globalBraceCount--;
+            }
+
+            if (blockBraceCount === 0 && blockStart !== -1 && i > blockStart) {
+              // End of test block found
               const blockContent = lines.slice(blockStart, i + 1).join('\n');
-              
+
               blocks.push({
                 type: blockType,
                 name: blockName,
                 content: blockContent,
-                describe: currentDescribe || undefined,
+                describe: blockDescribe,
                 startLine: blockStart + 1,
                 endLine: i + 1
               });
-              
+
               inBlock = false;
               blockStart = -1;
+
+              // Pop any describe blocks that have closed
+              while (describeStack.length > 0 && globalBraceCount < describeStack[describeStack.length - 1].braceDepth) {
+                describeStack.pop();
+              }
               break;
             }
           }
+        } else {
+          // Track braces outside of test blocks (for describe open/close)
+          for (const char of line) {
+            if (char === '{') globalBraceCount++;
+            if (char === '}') {
+              globalBraceCount--;
+              // Pop any describe blocks that have closed
+              while (describeStack.length > 0 && globalBraceCount < describeStack[describeStack.length - 1].braceDepth) {
+                describeStack.pop();
+              }
+            }
+          }
         }
       }
-      
+
       return blocks;
     } catch (error: any) {
       logger.error(`Failed to extract test blocks from ${filePath}: ${error.message}`);
@@ -1292,7 +1421,7 @@ export class TestLensReporter implements Reporter {
       const author = execSync('git log -1 --pretty=format:"%an"', { encoding: 'utf-8' }).trim();
       const commitMessage = execSync('git log -1 --pretty=format:"%s"', { encoding: 'utf-8' }).trim();
       const commitTimestamp = execSync('git log -1 --pretty=format:"%ci"', { encoding: 'utf-8' }).trim();
-      
+
       let remoteName = 'origin';
       let remoteUrl = '';
       try {
@@ -1305,9 +1434,9 @@ export class TestLensReporter implements Reporter {
         // Remote info is optional - handle gracefully
         logger.debug('[INFO] No git remote configured, skipping remote info');
       }
-      
+
       const isDirty = execSync('git status --porcelain', { encoding: 'utf-8' }).trim().length > 0;
-      
+
       return {
         branch,
         commit,
@@ -1341,7 +1470,7 @@ export class TestLensReporter implements Reporter {
 
   private extractTags(test: TestCase): string[] {
     const tags: string[] = [];
-    
+
     // Playwright stores tags in the _tags property
     const testTags = (test as any)._tags;
     if (testTags && Array.isArray(testTags)) {
@@ -1372,7 +1501,7 @@ export class TestLensReporter implements Reporter {
         : [buildTagSource];
       buildTags.forEach(tag => tags.push(`@${tag}`));
     }
-    
+
     // Remove duplicates and return
     return [...new Set(tags)];
   }
@@ -1390,15 +1519,14 @@ export class TestLensReporter implements Reporter {
       // Check file size first
       const fileSize = this.getFileSize(filePath);
       const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
-      
+
       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`;
       const requestBody: any = {
-        apiKey: this.config.apiKey,
         testRunId: this.runId,
         testId: testId,
         fileName: fileName,
@@ -1406,12 +1534,12 @@ export class TestLensReporter implements Reporter {
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       };
-      
+
       // Include DB ID if available for faster lookup (avoids query)
       if (testCaseDbId) {
         requestBody.testCaseDbId = testCaseDbId;
       }
-      
+
       const presignedResponse = await this.axiosInstance.post(presignedUrlEndpoint, requestBody, {
         timeout: 10000 // Quick timeout for metadata request
       });
@@ -1421,12 +1549,12 @@ export class TestLensReporter implements Reporter {
       }
 
       const { uploadUrl, s3Key, metadata } = presignedResponse.data;
-      
+
       // Step 2: Upload directly to S3 using presigned URL
       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
       // AWS presigned URLs are very strict about header matching
@@ -1450,7 +1578,6 @@ export class TestLensReporter implements Reporter {
       // Step 3: Confirm upload with server to save metadata
       const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
       const confirmBody: any = {
-        apiKey: this.config.apiKey,
         testRunId: this.runId,
         testId: testId,
         s3Key: s3Key,
@@ -1459,12 +1586,12 @@ export class TestLensReporter implements Reporter {
         fileSize: fileSize,
         artifactType: this.getArtifactType(fileName)
       };
-      
+
       // Include DB ID if available for direct insert (avoids query and race condition)
       if (testCaseDbId) {
         confirmBody.testCaseDbId = testCaseDbId;
       }
-      
+
       const confirmResponse = await this.axiosInstance.post(confirmEndpoint, confirmBody, {
         timeout: 10000
       });
@@ -1486,11 +1613,11 @@ export class TestLensReporter implements Reporter {
       // Check for trial expiration, subscription errors, or limit errors
       if (error?.response?.status === 401) {
         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') {
+          errorData?.error === 'test_cases_limit_reached' || errorData?.error === 'test_runs_limit_reached') {
           logger.error('\n' + '='.repeat(80));
-          
+
           if (errorData?.error === 'test_cases_limit_reached') {
             logger.error('[ERROR] TESTLENS ERROR: Test Cases Limit Reached');
           } else if (errorData?.error === 'test_runs_limit_reached') {
@@ -1498,7 +1625,7 @@ export class TestLensReporter implements Reporter {
           } else {
             logger.error('[ERROR] TESTLENS ERROR: Your trial plan has ended');
           }
-          
+
           logger.error('='.repeat(80));
           logger.error('');
           logger.error(errorData?.message || 'Your trial period has expired.');
@@ -1515,10 +1642,10 @@ export class TestLensReporter implements Reporter {
           return null;
         }
       }
-      
+
       // Better error messages for common issues
       let errorMsg = error.message;
-      
+
       if (error.code === 'ECONNABORTED' && error.message.includes('timeout')) {
         errorMsg = `Upload timeout - file may be too large or connection is slow`;
       } else if (error.response?.status === 413) {
@@ -1528,12 +1655,12 @@ export class TestLensReporter implements Reporter {
       } else if (error.response?.status === 403) {
         errorMsg = `Access denied (403) - presigned URL may have expired`;
       }
-      
+
       logger.error(`[ERROR] Failed to upload ${fileName} to S3: ${errorMsg}`);
       if (error.response?.data) {
         logger.error({ errorDetails: error.response.data }, 'Error details');
       }
-      
+
       // Don't throw, just return null to continue with other artifacts
       return null;
     }
@@ -1546,7 +1673,7 @@ export class TestLensReporter implements Reporter {
       // Try different ways to access getType method
       const getType = mime.getType || mime.default?.getType;
       if (typeof getType === 'function') {
-        const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
+        const mimeType = getType.call(mime, ext) || (mime.default ? getType.call(mime.default, ext) : null);
         return mimeType || 'application/octet-stream';
       }
     } catch (error: any) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alternative-path/testlens-playwright-reporter",
-  "version": "0.4.7",
+  "version": "0.4.8",
   "description": "Universal Playwright reporter for TestLens - works with both TypeScript and JavaScript projects",
   "main": "index.js",
   "types": "index.d.ts",
@@ -51,6 +51,7 @@
   "dependencies": {
     "@aws-sdk/client-s3": "^3.624.0",
     "@aws-sdk/s3-request-presigner": "^3.624.0",
+    "adm-zip": "^0.5.16",
     "axios": "^1.11.0",
     "cross-env": "^7.0.3",
     "dotenv": "^16.4.5",