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

package/index.ts

@@ -1,1587 +1,1714 @@
-import { randomUUID } from 'crypto';
-import * as os from 'os';
-import * as path from 'path';
-import * as fs from 'fs';
-import * as https from 'https';
-import axios, { AxiosInstance } from 'axios';
-import type { Reporter, TestCase, TestResult, FullConfig, Suite } from '@playwright/test/reporter';
-import { execSync } from 'child_process';
-import pino from 'pino';
-
-const isVerboseLogs =
-  process.env.TESTLENS_LOG_VERBOSE === '1' ||
-  process.env.TESTLENS_LOG_VERBOSE === 'true';
-const noisyInfoPrefixes = [
-  '[OK]',
-  '[UPLOAD]',
-  '[ARTIFACT]',
-  '[SKIP]',
-  '[TestLens] onTestEnd called',
-  '[TestLens] Sending testEnd'
-];
-const noisyInfoSubstrings = [
-  'Sent runStart event to TestLens',
-  'Sent specStart event to TestLens',
-  'Sent testStart event to TestLens',
-  'Sent testEnd event to TestLens',
-  'Sent runEnd event to TestLens',
-  'Sent artifact event to TestLens',
-  'S3 upload completed',
-  'Upload confirmed',
-  'Uploading ',
-  'Processed artifact',
-  'Spec code blocks',
-  '\n[METADATA]',
-  'Custom Metadata Detected',
-  'Using API key from environment variable'
-];
-
-const shouldSuppressInfo = (msg: string): boolean => {
-  if (isVerboseLogs) return false;
-  if (!msg) return false;
-  return (
-    noisyInfoPrefixes.some(prefix => msg.startsWith(prefix)) ||
-    noisyInfoSubstrings.some(sub => msg.includes(sub))
-  );
-};
-
-// Create pino logger instance
-const logger = pino({
-  level: process.env.TESTLENS_LOG_LEVEL || process.env.LOG_LEVEL || 'info',
-  base: null, // remove pid/hostname
-  timestamp: false, // remove time
-  formatters: {
-    level: () => ({}) // remove level
-  },
-  hooks: {
-    logMethod(args, method, level) {
-      // pino level is numeric (info = 30)
-      if (level === 30) {
-        const msg =
-          typeof args[0] === 'string'
-            ? args[0]
-            : typeof args[1] === 'string'
-            ? args[1]
-            : '';
-        if (shouldSuppressInfo(msg)) {
-          return;
-        }
-      }
-      return method.apply(this, args as any);
-    }
-  }
-});
-
-// Lazy-load mime module to support ESM
-let mimeModule: any = null;
-async function getMime() {
-  if (!mimeModule) {
-    const imported = await import('mime');
-    // Handle both default export and named exports
-    mimeModule = imported.default || imported;
-  }
-  return mimeModule;
-}
-
-export interface TestLensReporterConfig {
-  /** TestLens API endpoint URL */
-  apiEndpoint?: string;
-  /** API key for authentication - can be provided in config or via TESTLENS_API_KEY environment variable */
-  apiKey?: string;
-  /** Enable real-time streaming of test events */
-  enableRealTimeStream?: boolean;
-  /** Enable Git information collection */
-  enableGitInfo?: boolean;
-  /** Enable artifact processing */
-  enableArtifacts?: boolean;
-  /** Enable video capture - defaults to true */
-  enableVideo?: boolean;
-  /** Enable screenshot capture - defaults to true */
-  enableScreenshot?: boolean;
-  /** Batch size for API requests */
-  batchSize?: number;
-  /** Flush interval in milliseconds */
-  flushInterval?: number;
-  /** Number of retry attempts for failed API calls */
-  retryAttempts?: number;
-  /** Request timeout in milliseconds */
-  timeout?: number;
-  /** SSL certificate validation - set to false to disable SSL verification */
-  rejectUnauthorized?: boolean;
-  /** Alternative SSL option - set to true to ignore SSL certificate errors */
-  ignoreSslErrors?: boolean;
-  /** Custom metadata from CLI arguments (automatically parsed from --key=value arguments) */
-  customMetadata?: Record<string, string | string[]>;
-}
-
-export interface TestLensReporterOptions {
-  /** TestLens API endpoint URL */
-  apiEndpoint?: string;
-  /** API key for authentication - can be provided in config or via TESTLENS_API_KEY environment variable */
-  apiKey?: string;
-  /** Enable real-time streaming of test events */
-  enableRealTimeStream?: boolean;
-  /** Enable Git information collection */
-  enableGitInfo?: boolean;
-  /** Enable artifact processing */
-  enableArtifacts?: boolean;
-  /** Enable video capture - defaults to true */
-  enableVideo?: boolean;
-  /** Enable screenshot capture - defaults to true */
-  enableScreenshot?: boolean;
-  /** Batch size for API requests */
-  batchSize?: number;
-  /** Flush interval in milliseconds */
-  flushInterval?: number;
-  /** Number of retry attempts for failed API calls */
-  retryAttempts?: number;
-  /** Request timeout in milliseconds */
-  timeout?: number;
-  /** SSL certificate validation - set to false to disable SSL verification */
-  rejectUnauthorized?: boolean;
-  /** Alternative SSL option - set to true to ignore SSL certificate errors */
-  ignoreSslErrors?: boolean;
-  /** Custom metadata from CLI arguments (automatically parsed from --key=value arguments) */
-  customMetadata?: Record<string, string | string[]>;
-}
-
-export interface GitInfo {
-  branch: string;
-  commit: string;
-  shortCommit: string;
-  author: string;
-  message: string;
-  timestamp: string;
-  isDirty: boolean;
-  remoteName: string;
-  remoteUrl: string;
-}
-
-export interface CodeBlock {
-  type: 'test' | 'describe';
-  name: string;
-  content: string;
-  summary?: string;
-  describe?: string;
-  startLine?: number;
-  endLine?: number;
-}
-
-export interface RunMetadata {
-  id: string;
-  startTime: string;
-  endTime?: string;
-  duration?: number;
-  environment: string;
-  browser: string;
-  os: string;
-  playwrightVersion: string;
-  nodeVersion: string;
-  testlensVersion?: string;
-  gitInfo?: GitInfo | null;
-  shardInfo?: {
-    current: number;
-    total: number;
-  };
-  totalTests?: number;
-  passedTests?: number;
-  failedTests?: number;
-  skippedTests?: number;
-  status?: string;
-  testlensBuildName?: string;
-  customMetadata?: Record<string, string | string[]>;
-}
-
-export interface TestError {
-  message: string;
-  stack?: string;
-  location?: {
-    file: string;
-    line: number;
-    column: number;
-  };
-  snippet?: string;
-  expected?: string;
-  actual?: string;
-  diff?: string;
-  matcherName?: string;
-  timeout?: number;
-}
-
-export interface TestData {
-  id: string;
-  name: string;
-  status: string;
-  originalStatus?: string; // Preserve original Playwright status
-  duration: number;
-  startTime: string;
-  endTime: string;
-  errorMessages: string[];
-  errors?: TestError[]; // Rich error details
-  retryAttempts: number;
-  currentRetry: number;
-  annotations: Array<{ type: string; description?: string }>;
-  projectName: string;
-  workerIndex?: number;
-  parallelIndex?: number;
-  location?: {
-    file: string;
-    line: number;
-    column: number;
-  };
-}
-
-export interface SpecData {
-  filePath: string;
-  testSuiteName: string;
-  tags?: string[];
-  startTime: string;
-  endTime?: string;
-  status: string;
-}
-
-export class TestLensReporter implements Reporter {
-  private config: Required<TestLensReporterConfig>;
-  private axiosInstance: AxiosInstance;
-  private runId: string;
-  private runMetadata: RunMetadata;
-  private specMap: Map<string, SpecData>;
-  private testMap: Map<string, TestData>;
-  private runCreationFailed: boolean = false; // Track if run creation failed due to limits
-  private cliArgs: Record<string, any> = {}; // Store CLI args separately
-  private pendingUploads: Set<Promise<any>> = new Set(); // Track pending artifact uploads
-  private artifactStats = {
-    uploaded: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
-    skipped: { screenshot: 0, video: 0, trace: 0, attachment: 0 },
-    failed: { screenshot: 0, video: 0, trace: 0, attachment: 0 }
-  };
-  private artifactsSeen: number = 0;
-
-  /**
-   * Parse custom metadata from environment variables
-   * Checks for common metadata environment variables
-   */
-  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
-      'testlensBuildTag': ['testlensBuildTag', 'TESTLENS_BUILD_TAG', 'TESTLENS_BUILDTAG', 'BUILDTAG', 'BUILD_TAG', 'TestlensBuildTag', 'TestLensBuildTag'],
-      'testlensBuildName': ['testlensBuildName', 'TESTLENS_BUILD_NAME', 'TESTLENS_BUILDNAME', 'BUILDNAME', 'BUILD_NAME', 'TestlensBuildName', 'TestLensBuildName'],
-      'environment': ['ENVIRONMENT', 'ENV', 'NODE_ENV', 'DEPLOYMENT_ENV'],
-      'branch': ['BRANCH', 'GIT_BRANCH', 'CI_COMMIT_BRANCH', 'GITHUB_REF_NAME'],
-      'team': ['TEAM', 'TEAM_NAME'],
-      'project': ['PROJECT', 'PROJECT_NAME'],
-      'customvalue': ['CUSTOMVALUE', 'CUSTOM_VALUE']
-    };
-    
-    // Check for each metadata key
-    Object.entries(envVarMappings).forEach(([key, envVars]) => {
-      for (const envVar of envVars) {
-        const value = process.env[envVar];
-        if (value) {
-          // For testlensBuildTag, support comma-separated values
-          if (key === 'testlensBuildTag' && value.includes(',')) {
-            customArgs[key] = value.split(',').map(tag => tag.trim()).filter(tag => tag);
-            logger.info(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
-          } else {
-            customArgs[key] = value;
-            logger.info(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
-          }
-          break; // Use first match
-        }
-      }
-    });
-    
-    return customArgs;
-  }
-
-  constructor(options: TestLensReporterOptions) {
-    // 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 
-      || process.env.testlens_api_key
-      || process.env.TESTLENS_KEY
-      || process.env.testlens_key
-      || process.env.testlensApiKey
-      || process.env.PLAYWRIGHT_API_KEY
-      || 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
-      enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
-      enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
-      enableArtifacts: options.enableArtifacts !== undefined ? options.enableArtifacts : true,
-      enableVideo: options.enableVideo !== undefined ? options.enableVideo : true, // Default to true, override from config
-      enableScreenshot: options.enableScreenshot !== undefined ? options.enableScreenshot : true, // Default to true, override from config
-      batchSize: options.batchSize || 10,
-      flushInterval: options.flushInterval || 5000,
-      retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
-      timeout: options.timeout || 60000,
-      rejectUnauthorized: options.rejectUnauthorized,
-      ignoreSslErrors: options.ignoreSslErrors,
-      customMetadata: { ...options.customMetadata, ...customArgs } // Config metadata first, then CLI args override
-    } as Required<TestLensReporterConfig>;
-
-    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.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 = 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;
-      logger.warn('[WARN] SSL certificate validation disabled via ignoreSslErrors option');
-    } else if (this.config.rejectUnauthorized === false) {
-      rejectUnauthorized = false;
-      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') {
-      rejectUnauthorized = false;
-      logger.warn('[WARN] SSL certificate validation disabled via NODE_TLS_REJECT_UNAUTHORIZED environment variable');
-    } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '1') {
-      rejectUnauthorized = true;
-    }
-
-    // Mirror the resolved value so all HTTPS requests in this process follow it
-    process.env.NODE_TLS_REJECT_UNAUTHORIZED = rejectUnauthorized ? '1' : '0';
-
-    // Set up axios instance with retry logic and enhanced SSL handling
-    this.axiosInstance = axios.create({
-      baseURL: this.config.apiEndpoint,
-      timeout: this.config.timeout,
-      headers: {
-        'Content-Type': 'application/json',
-        ...(this.config.apiKey && { 'X-API-Key': this.config.apiKey }),
-      },
-      // Enhanced SSL handling with flexible TLS configuration
-      httpsAgent: new https.Agent({
-        rejectUnauthorized: rejectUnauthorized,
-        // Allow any TLS version for better compatibility
-        minVersion: 'TLSv1.2',
-        maxVersion: 'TLSv1.3'
-      })
-    });
-
-    // Add retry interceptor
-    this.axiosInstance.interceptors.response.use(
-      (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;
-            await new Promise(resolve => setTimeout(resolve, delay));
-            return this.axiosInstance(originalRequest);
-          }
-        }
-        
-        return Promise.reject(error);
-      }
-    );
-
-    this.runId = randomUUID();
-    this.runMetadata = this.initializeRunMetadata();
-    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.info('\n[METADATA] Custom Metadata Detected:');
-      Object.entries(this.config.customMetadata).forEach(([key, value]) => {
-        logger.info(`   ${key}: ${value}`);
-      });
-      logger.info('');
-    }
-  }
-
-  private initializeRunMetadata(): RunMetadata {
-    const metadata: RunMetadata = {
-      id: this.runId,
-      startTime: new Date().toISOString(),
-      environment: 'production',
-      browser: 'multiple',
-      os: `${os.type()} ${os.release()}`,
-      playwrightVersion: this.getPlaywrightVersion(),
-      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;
-        // Handle both string and array (take first element if array)
-        metadata.testlensBuildName = Array.isArray(buildName) ? buildName[0] : buildName;
-      }
-    }
-    
-    return metadata;
-  }
-
-  private getPlaywrightVersion(): string {
-    try {
-      const playwrightPackage = require('@playwright/test/package.json');
-      return playwrightPackage.version;
-    } catch (error) {
-      return 'unknown';
-    }
-  }
-
-  private getTestLensVersion(): string {
-    try {
-      const testlensPackage = require('./package.json');
-      return testlensPackage.version;
-    } catch (error) {
-      return 'unknown';
-    }
-  }
-
-  private normalizeTestStatus(status: string): string {
-    // Treat timeout as failed for consistency with analytics
-    if (status === 'timedOut') {
-      return 'failed';
-    }
-    return status;
-  }
-
-  private normalizeRunStatus(status: string, hasTimeouts: boolean): string {
-    // If run has timeouts, treat as failed
-    if (hasTimeouts && status === 'passed') {
-      return 'failed';
-    }
-    // Treat timeout status as failed
-    if (status === 'timedOut') {
-      return 'failed';
-    }
-    return status;
-  }
-
-  async onBegin(config: FullConfig, suite: Suite): Promise<void> {
-    logger.info(`[REPORTER] source=${__filename} enableArtifacts=${this.config.enableArtifacts}`);
-    // Show Build Name if provided, otherwise show Run ID
-    if (this.runMetadata.testlensBuildName) {
-      logger.info(`TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
-      logger.info(`   Run ID: ${this.runId}`);
-    } else {
-      logger.info(`TestLens Reporter starting - Run ID: ${this.runId}`);
-    }
-    
-    // Collect Git information if enabled
-    if (this.config.enableGitInfo) {
-      this.runMetadata.gitInfo = await this.collectGitInfo();
-      if (this.runMetadata.gitInfo) {
-        logger.info(`Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
-      } else {
-        logger.warn(`[WARN] Git info collection returned null - not in a git repository or git not available`);
-      }
-    } else {
-      logger.info(`[INFO] Git info collection disabled (enableGitInfo: false)`);
-    }
-
-    // Add shard information if available
-    if (config.shard) {
-      this.runMetadata.shardInfo = {
-        current: config.shard.current,
-        total: config.shard.total
-      };
-    }
-
-    // Send run start event to API
-    await this.sendToApi({
-      type: 'runStart',
-      runId: this.runId,
-      timestamp: new Date().toISOString(),
-      metadata: this.runMetadata
-    });
-  }
-
-  async onTestBegin(test: TestCase, result: TestResult): Promise<void> {
-    // Log which test is starting
-    logger.info(`[TEST] Running test: ${test.title}`);
-    
-    const specPath = test.location.file;
-    const specKey = `${specPath}-${test.parent.title}`;
-
-    // Create or update spec data
-    if (!this.specMap.has(specKey)) {
-      const extractedTags = this.extractTags(test);
-      const specData: SpecData = {
-        filePath: path.relative(process.cwd(), specPath),
-        testSuiteName: test.parent.title,
-        startTime: new Date().toISOString(),
-        status: 'running'
-      };
-      if (extractedTags.length > 0) {
-        specData.tags = extractedTags;
-      }
-      this.specMap.set(specKey, specData);
-
-      // Send spec start event to API
-      await this.sendToApi({
-        type: 'specStart',
-        runId: this.runId,
-        timestamp: new Date().toISOString(),
-        spec: specData
-      });
-    }
-
-    const testId = this.getTestId(test);
-    
-    // Only send testStart event on first attempt (retry 0)
-    if (result.retry === 0) {
-      // Create test data
-      const testData: TestData = {
-        id: testId,
-        name: test.title,
-        status: 'running',
-        originalStatus: 'running',
-        duration: 0,
-        startTime: new Date().toISOString(),
-        endTime: '',
-        errorMessages: [],
-        errors: [],
-        retryAttempts: test.retries,
-        currentRetry: result.retry,
-        annotations: test.annotations.map((ann: any) => ({
-          type: ann.type,
-          description: ann.description
-        })),
-        projectName: test.parent.project()?.name || 'default',
-        workerIndex: result.workerIndex,
-        parallelIndex: result.parallelIndex,
-        location: {
-          file: path.relative(process.cwd(), test.location.file),
-          line: test.location.line,
-          column: test.location.column
-        }
-      };
-
-      this.testMap.set(testData.id, testData);
-
-      // Send test start event to API
-      await this.sendToApi({
-        type: 'testStart',
-        runId: this.runId,
-        timestamp: new Date().toISOString(),
-        test: testData
-      });
-    } else {
-      // For retries, just update the existing test data
-      const existingTestData = this.testMap.get(testId);
-      if (existingTestData) {
-        existingTestData.currentRetry = result.retry;
-      }
-    }
-  }
-
-  async onTestEnd(test: TestCase, result: TestResult): Promise<void> {
-    const testId = this.getTestId(test);
-    let testData = this.testMap.get(testId);
-
-    logger.info(`[ARTIFACTS] attachments=${result.attachments?.length ?? 0}`);
-
-    if (result.attachments && result.attachments.length > 0) {
-      for (const attachment of result.attachments) {
-        const artifactType = this.getArtifactType(attachment.name);
-        this.artifactsSeen += 1;
-        if (attachment.path) {
-          this.bumpArtifactStat('uploaded', artifactType);
-        } else {
-          this.bumpArtifactStat('skipped', artifactType);
-        }
-      }
-    }
-
-    logger.info(`[TestLens] onTestEnd called for test: ${test.title}, status: ${result.status}, testData exists: ${!!testData}`);
-
-    // For skipped tests, onTestBegin might not be called, so we need to create the test data here
-    if (!testData) {
-      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}`;
-      
-      if (!this.specMap.has(specKey)) {
-        const extractedTags = this.extractTags(test);
-        const specData: SpecData = {
-          filePath: path.relative(process.cwd(), specPath),
-          testSuiteName: test.parent.title,
-          startTime: new Date().toISOString(),
-          status: 'skipped'
-        };
-        if (extractedTags.length > 0) {
-          specData.tags = extractedTags;
-        }
-        this.specMap.set(specKey, specData);
-
-        // Send spec start event to API
-        await this.sendToApi({
-          type: 'specStart',
-          runId: this.runId,
-          timestamp: new Date().toISOString(),
-          spec: specData
-        });
-      }
-
-      // Create test data for skipped test
-      testData = {
-        id: testId,
-        name: test.title,
-        status: 'skipped',
-        originalStatus: 'skipped',
-        duration: 0,
-        startTime: new Date().toISOString(),
-        endTime: new Date().toISOString(),
-        errorMessages: [],
-        errors: [],
-        retryAttempts: test.retries,
-        currentRetry: 0,
-        annotations: test.annotations.map((ann: any) => ({
-          type: ann.type,
-          description: ann.description
-        })),
-        projectName: test.parent.project()?.name || 'default',
-        workerIndex: result.workerIndex,
-        parallelIndex: result.parallelIndex,
-        location: {
-          file: path.relative(process.cwd(), test.location.file),
-          line: test.location.line,
-          column: test.location.column
-        }
-      };
-
-      this.testMap.set(testId, testData);
-
-      // Send test start event first (so the test gets created in DB)
-      await this.sendToApi({
-        type: 'testStart',
-        runId: this.runId,
-        timestamp: new Date().toISOString(),
-        test: testData
-      });
-    }
-
-    if (testData) {
-      // Update test data with latest result
-      testData.originalStatus = result.status;
-      testData.status = this.normalizeTestStatus(result.status);
-      testData.duration = result.duration;
-      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 = {
-            file: path.relative(process.cwd(), error.location.file),
-            line: error.location.line,
-            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|$)/);
-          if (expectedStringMatch) {
-            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) {
-          // Store call log separately for display
-          const callLog = callLogMatch[1].trim();
-          if (callLog) {
-            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) {
-          const codeSnippetMatch = message.match(/((?:\s*>?\s*\d+\s*\|.*\n?)+)/);
-          if (codeSnippetMatch) {
-            testError.snippet = codeSnippetMatch[1].trim();
-          }
-        }
-        
-        return testError;
-      });
-
-      // Send testEnd event for all tests, regardless of status
-      // This ensures tests that are interrupted or have unexpected statuses are properly recorded
-      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',
-        runId: this.runId,
-        timestamp: new Date().toISOString(),
-        test: testData
-      });
-
-      // 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);
-      } 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;
-    const specKey = `${specPath}-${test.parent.title}`;
-    const specData = this.specMap.get(specKey);
-    if (specData) {
-      const normalizedStatus = this.normalizeTestStatus(result.status);
-      if (normalizedStatus === 'failed' && specData.status !== 'failed') {
-        specData.status = 'failed';
-      } else if (result.status === 'skipped' && specData.status === 'passed') {
-        specData.status = 'skipped';
-      }
-
-      // Check if all tests in spec are complete
-      // Only consider tests that were actually executed (have testData)
-      const remainingTests = test.parent.tests.filter((t: any) => {
-        const tId = this.getTestId(t);
-        const tData = this.testMap.get(tId);
-        // If testData exists but no endTime, it's still running
-        return tData && !tData.endTime;
-      });
-
-      if (remainingTests.length === 0) {
-        // Determine final spec status based on all executed tests
-        const executedTests = test.parent.tests
-          .map((t: any) => {
-            const tId = this.getTestId(t);
-            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')) {
-            specData.status = 'passed';
-          } else if (allTestStatuses.some(status => status === 'failed')) {
-            specData.status = 'failed';
-          } else if (allTestStatuses.every(status => status === 'skipped')) {
-            specData.status = 'skipped';
-          }
-        }
-        // Aggregate tags from all tests in this spec
-        const allTags = new Set<string>();
-        test.parent.tests.forEach((t: any) => {
-          const tags = this.extractTags(t);
-          tags.forEach(tag => allTags.add(tag));
-        });
-        const aggregatedTags = Array.from(allTags);
-        // Only update tags if we have any
-        if (aggregatedTags.length > 0) {
-          specData.tags = aggregatedTags;
-        }
-        
-        specData.endTime = new Date().toISOString();
-        
-        // Send spec end event to API
-        await this.sendToApi({
-          type: 'specEnd',
-          runId: this.runId,
-          timestamp: new Date().toISOString(),
-          spec: specData
-        });
-
-        // Send spec code blocks to API
-        await this.sendSpecCodeBlocks(specPath);
-      }
-    }
-  }
-
-  async onEnd(result: { status: string }): Promise<void> {
-    this.runMetadata.endTime = new Date().toISOString();
-    this.runMetadata.duration = Date.now() - new Date(this.runMetadata.startTime).getTime();
-
-    // Wait for all pending artifact uploads to complete before sending runEnd
-    if (this.pendingUploads.size > 0) {
-      logger.debug(`Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
-      try {
-        await Promise.all(Array.from(this.pendingUploads));
-        logger.debug(`[OK] All artifact uploads completed`);
-      } catch (error) {
-        logger.error(`[WARN] Some artifact uploads failed, continuing with runEnd`);
-      }
-    }
-
-    const uploaded = this.artifactStats.uploaded;
-    const skipped = this.artifactStats.skipped;
-    const failed = this.artifactStats.failed;
-    // Always show summary so users can confirm artifact settings
-    const summary =
-      `[ARTIFACTS] seen=${this.artifactsSeen} | ` +
-      `uploaded screenshot=${uploaded.screenshot}, video=${uploaded.video}, trace=${uploaded.trace}, attachment=${uploaded.attachment} | ` +
-      `skipped screenshot=${skipped.screenshot}, video=${skipped.video}, trace=${skipped.trace}, attachment=${skipped.attachment} | ` +
-      `failed screenshot=${failed.screenshot}, video=${failed.video}, trace=${failed.trace}, attachment=${failed.attachment}`;
-    if (logger.levelVal > 30) {
-      console.log(summary);
-    } else {
-      logger.info(summary);
-    }
-
-    // Calculate final stats
-    const totalTests = Array.from(this.testMap.values()).length;
-    const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
-    // failedTests already includes timedOut tests since normalizeTestStatus converts 'timedOut' to 'failed'
-    const failedTests = Array.from(this.testMap.values()).filter(t => t.status === 'failed').length;
-    const skippedTests = Array.from(this.testMap.values()).filter(t => t.status === 'skipped').length;
-    // Track timedOut separately for reporting purposes only (not for count)
-    const timedOutTests = Array.from(this.testMap.values()).filter(t => t.originalStatus === 'timedOut').length;
-
-    // Normalize run status - if there are timeouts, treat run as failed
-    const hasTimeouts = timedOutTests > 0;
-    const normalizedRunStatus = this.normalizeRunStatus(result.status, hasTimeouts);
-
-    // Send run end event to API
-    await this.sendToApi({
-      type: 'runEnd',
-      runId: this.runId,
-      timestamp: new Date().toISOString(),
-      metadata: {
-        ...this.runMetadata,
-        totalTests,
-        passedTests,
-        failedTests, // Already includes timedOut tests (normalized to 'failed')
-        skippedTests,
-        timedOutTests, // For informational purposes
-        status: normalizedRunStatus
-      }
-    });
-
-    // Show Build Name if provided, otherwise show Run ID
-    if (this.runMetadata.testlensBuildName) {
-      logger.info(`[COMPLETE] TestLens Report completed - Build: ${this.runMetadata.testlensBuildName}`);
-      logger.info(`   Run ID: ${this.runId}`);
-    } else {
-      logger.info(`[COMPLETE] TestLens Report completed - Run ID: ${this.runId}`);
-    }
-    logger.info(`[RESULTS] ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
-  }
-
-  private async sendToApi(payload: any): Promise<any> {
-    // Skip sending if run creation already failed
-    if (this.runCreationFailed && payload.type !== 'runStart') {
-      return null;
-    }
-    
-    try {
-      const response = await this.axiosInstance.post('', payload, {
-        headers: {
-          'X-API-Key': this.config.apiKey
-        }
-      });
-      if (this.config.enableRealTimeStream) {
-        logger.debug(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
-      }
-      // Return response data for caller to use
-      return response.data;
-    } 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');
-        } else if (errorData?.limit_type === 'test_runs') {
-          logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
-        } else {
-          logger.error('[ERROR] TESTLENS ERROR: Plan Limit Reached');
-        }
-        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') {
-          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') {
-            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
-          } 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.');
-          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) {
-            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
-            logger.error('');
-          }
-          logger.error('='.repeat(80));
-          logger.error('');
-        } else {
-          logger.error(`[ERROR] Authentication failed: ${errorData?.error || 'Invalid API key'}`);
-        }
-      } else if (status !== 403) {
-        // Log other errors (but not 403 which we handled above)
-        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,
-          code: error?.code,
-          url: error?.config?.url,
-          method: error?.config?.method
-        });
-      }
-      
-      // Don't throw error to avoid breaking test execution
-    }
-  }
-
-  private async processArtifacts(testId: string, result: TestResult, testCaseDbId?: 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;
-        }
-        
-        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);
-            if (pathExt) {
-              fileName = `${fileName}${pathExt}`;
-            } else if (attachment.contentType) {
-              // Fallback: derive extension from contentType
-              const mimeToExt: Record<string, string> = {
-                'image/png': '.png',
-                'image/jpeg': '.jpg',
-                'image/gif': '.gif',
-                'image/webp': '.webp',
-                'video/webm': '.webm',
-                'video/mp4': '.mp4',
-                'application/zip': '.zip',
-                'application/json': '.json',
-                'text/plain': '.txt'
-              };
-              const ext = mimeToExt[attachment.contentType];
-              if (ext) {
-                fileName = `${fileName}${ext}`;
-              }
-            }
-          }
-
-          // Upload to S3 first (pass DB ID if available for faster lookup)
-          // Create upload promise that we can track
-          const uploadPromise = Promise.resolve().then(async () => {
-            try {
-              if (!attachment.path) {
-                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
-                this.bumpArtifactStat('skipped', artifactType);
-                return;
-              }
-              
-              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName, testCaseDbId);
-              
-              // Skip if upload failed or file was too large
-              if (!s3Data) {
-                logger.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),
-                path: attachment.path,
-                name: fileName,
-                contentType: attachment.contentType,
-                fileSize: this.getFileSize(attachment.path),
-                storageType: 's3',
-                s3Key: s3Data.key,
-                s3Url: s3Data.url
-              };
-
-              // Send artifact data to API
-              await this.sendToApi({
-                type: 'artifact',
-                runId: this.runId,
-                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) {
-          logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}: ${(error as Error).message}`);
-          this.bumpArtifactStat('failed', artifactType);
-        }
-      } else {
-        this.bumpArtifactStat('skipped', artifactType);
-      }
-    }
-  }
-
-  private async sendSpecCodeBlocks(specPath: 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 => ({
-        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', '');
-      if (baseUrl === this.config.apiEndpoint) {
-        // Fallback: try alternative pattern if main pattern didn't match
-        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)$/, '')
-      });
-
-      logger.info(`Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
-    } catch (error: any) {
-      const errorData = error?.response?.data;
-      
-      // Handle duplicate spec code blocks gracefully (when re-running tests)
-      if (errorData?.error && errorData.error.includes('duplicate key value violates unique constraint')) {
-        logger.info(`[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'}`);
-    }
-  }
-
-  private extractTestBlocks(filePath: string): CodeBlock[] {
-    try {
-      const content = fs.readFileSync(filePath, 'utf-8');
-      const blocks: CodeBlock[] = [];
-      const lines = content.split('\n');
-      
-      let currentDescribe: string | null = null;
-      let braceCount = 0;
-      let inBlock = false;
-      let blockStart = -1;
-      let blockType: 'test' | 'describe' = 'test';
-      let blockName = '';
-      
-      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;
-        }
-        
-        // Count braces when in a block
-        if (inBlock) {
-          for (const char of line) {
-            if (char === '{') braceCount++;
-            if (char === '}') braceCount--;
-            
-            if (braceCount === 0 && blockStart !== -1 && i > blockStart) {
-              // End of block found
-              const blockContent = lines.slice(blockStart, i + 1).join('\n');
-              
-              blocks.push({
-                type: blockType,
-                name: blockName,
-                content: blockContent,
-                describe: currentDescribe || undefined,
-                startLine: blockStart + 1,
-                endLine: i + 1
-              });
-              
-              inBlock = false;
-              blockStart = -1;
-              break;
-            }
-          }
-        }
-      }
-      
-      return blocks;
-    } catch (error: any) {
-      logger.error(`Failed to extract test blocks from ${filePath}: ${error.message}`);
-      return [];
-    }
-  }
-
-  private async collectGitInfo(): Promise<GitInfo | null> {
-    try {
-      const branch = execSync('git rev-parse --abbrev-ref HEAD', { encoding: 'utf-8' }).trim();
-      const commit = execSync('git rev-parse HEAD', { encoding: 'utf-8' }).trim();
-      const shortCommit = commit.substring(0, 7);
-      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 {
-        const remotes = execSync('git remote', { encoding: 'utf-8' }).trim();
-        if (remotes) {
-          remoteName = remotes.split('\n')[0] || 'origin';
-          remoteUrl = execSync(`git remote get-url ${remoteName}`, { encoding: 'utf-8' }).trim();
-        }
-      } catch (e) {
-        // Remote info is optional - handle gracefully
-        logger.info('[INFO] No git remote configured, skipping remote info');
-      }
-      
-      const isDirty = execSync('git status --porcelain', { encoding: 'utf-8' }).trim().length > 0;
-      
-      return {
-        branch,
-        commit,
-        shortCommit,
-        author,
-        message: commitMessage,
-        timestamp: commitTimestamp,
-        isDirty,
-        remoteName,
-        remoteUrl
-      };
-    } catch (error: any) {
-      // Silently skip git information if not in a git repository
-      return null;
-    }
-  }
-
-  private getArtifactType(name: string): string {
-    if (name.includes('screenshot')) return 'screenshot';
-    if (name.includes('video')) return 'video';
-    if (name.includes('trace')) return 'trace';
-    return 'attachment';
-  }
-
-  private bumpArtifactStat(stat: 'uploaded' | 'skipped' | 'failed', type: string): void {
-    const bucket = this.artifactStats[stat] as Record<string, number>;
-    if (bucket[type] !== undefined) {
-      bucket[type] += 1;
-    }
-  }
-
-  private extractTags(test: TestCase): string[] {
-    const tags: string[] = [];
-    
-    // Playwright stores tags in the _tags property
-    const testTags = (test as any)._tags;
-    if (testTags && Array.isArray(testTags)) {
-      tags.push(...testTags);
-    }
-
-    // Also get tags from parent suites by walking up the tree
-    let currentSuite: Suite | undefined = test.parent;
-    while (currentSuite) {
-      const suiteTags = (currentSuite as any)._tags;
-      if (suiteTags && Array.isArray(suiteTags)) {
-        tags.push(...suiteTags);
-      }
-      currentSuite = currentSuite.parent;
-    }
-
-    // Also extract @tags from test title for backward compatibility
-    const tagMatches = test.title.match(/@[\w-]+/g);
-    if (tagMatches) {
-      tags.push(...tagMatches);
-    }
-
-    // Add testlensBuildTag: CLI args take precedence over config
-    const buildTagSource = this.cliArgs.testlensBuildTag || this.config.customMetadata?.testlensBuildTag;
-    if (buildTagSource) {
-      const buildTags = Array.isArray(buildTagSource)
-        ? buildTagSource
-        : [buildTagSource];
-      buildTags.forEach(tag => tags.push(`@${tag}`));
-    }
-    
-    // Remove duplicates and return
-    return [...new Set(tags)];
-  }
-
-  private getTestId(test: TestCase): string {
-    const cleanTitle = test.title.replace(/@[\w-]+/g, '').trim();
-    // Normalize path separators to forward slashes for cross-platform consistency
-    const normalizedFile = test.location.file.replace(/\\/g, '/');
-    return `${normalizedFile}:${test.location.line}:${cleanTitle}`;
-  }
-
-
-  private async uploadArtifactToS3(filePath: string, testId: string, fileName: string, testCaseDbId?: string): Promise<{ key: string; url: string; presignedUrl: string; fileSize: number; contentType: string } | null> {
-    try {
-      // 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,
-        fileType: await this.getContentType(fileName),
-        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
-      });
-
-      if (!presignedResponse.data.success) {
-        throw new Error(`Failed to get presigned URL: ${presignedResponse.data.error || 'Unknown error'}`);
-      }
-
-      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
-      const uploadResponse = await axios.put(uploadUrl, fileBuffer, {
-        headers: {
-          'x-amz-server-side-encryption': 'AES256'
-        },
-        maxContentLength: Infinity,
-        maxBodyLength: Infinity,
-        timeout: Math.max(600000, Math.ceil(fileSize / (1024 * 1024)) * 10000), // 10s per MB, min 10 minutes - increased for large trace files
-        // Don't use custom HTTPS agent for S3 uploads
-        httpsAgent: undefined
-      });
-
-      if (uploadResponse.status !== 200) {
-        throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
-      }
-
-      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: any = {
-        apiKey: this.config.apiKey,
-        testRunId: this.runId,
-        testId: testId,
-        s3Key: s3Key,
-        fileName: fileName,
-        fileType: await this.getContentType(fileName),
-        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
-      });
-
-      if (confirmResponse.status === 201 && confirmResponse.data.success) {
-        const artifact = confirmResponse.data.artifact;
-        logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
-        return {
-          key: s3Key,
-          url: artifact.s3Url,
-          presignedUrl: artifact.presignedUrl,
-          fileSize: artifact.fileSize,
-          contentType: artifact.contentType
-        };
-      } else {
-        throw new Error(`Upload confirmation failed: ${confirmResponse.data.error || 'Unknown error'}`);
-      }
-    } catch (error: any) {
-      // 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') {
-          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') {
-            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
-          } 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.');
-          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) {
-            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
-            logger.error('');
-          }
-          logger.error('='.repeat(80));
-          logger.error('');
-          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) {
-        errorMsg = `File too large (413) - server rejected the upload`;
-      } else if (error.response?.status === 400) {
-        errorMsg = `Bad request (400) - ${error.response.data?.error || 'check file format'}`;
-      } 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;
-    }
-  }
-
-  private async getContentType(fileName: string): Promise<string> {
-    const ext = path.extname(fileName).toLowerCase();
-    try {
-      const mime = await getMime();
-      // Try different ways to access getType method
-      const getType = mime.getType || mime.default?.getType;
-      if (typeof getType === 'function') {
-        const mimeType = getType.call(mime, ext) || getType.call(mime.default, ext);
-        return mimeType || 'application/octet-stream';
-      }
-    } catch (error: any) {
-      logger.warn(`Failed to get MIME type for ${fileName}: ${error.message}`);
-    }
-    // Fallback to basic content type mapping
-    const contentTypes: Record<string, string> = {
-      '.mp4': 'video/mp4',
-      '.webm': 'video/webm',
-      '.png': 'image/png',
-      '.jpg': 'image/jpeg',
-      '.jpeg': 'image/jpeg',
-      '.gif': 'image/gif',
-      '.json': 'application/json',
-      '.txt': 'text/plain',
-      '.html': 'text/html',
-      '.xml': 'application/xml',
-      '.zip': 'application/zip',
-      '.pdf': 'application/pdf'
-    };
-    return contentTypes[ext] || 'application/octet-stream';
-  }
-
-  // Note: S3 key generation and sanitization are handled server-side
-  // generateS3Key() and sanitizeForS3() methods removed as they were not used
-
-  private getFileSize(filePath: string): number {
-    try {
-      const stats = fs.statSync(filePath);
-      return stats.size;
-    } catch (error) {
-      logger.warn(`Could not get file size for ${filePath}: ${(error as Error).message}`);
-      return 0;
-    }
-  }
-}
-
-export default TestLensReporter;
+import { randomUUID } from 'crypto';
+import * as os from 'os';
+import * as path from 'path';
+import * as fs from 'fs';
+import * as https from 'https';
+import axios, { AxiosInstance } from 'axios';
+import type { Reporter, TestCase, TestResult, FullConfig, Suite } from '@playwright/test/reporter';
+import { execSync } from 'child_process';
+import pino from 'pino';
+
+const isVerboseLogs =
+  process.env.TESTLENS_LOG_VERBOSE === '1' ||
+  process.env.TESTLENS_LOG_VERBOSE === 'true';
+const noisyInfoPrefixes = [
+  '[OK]',
+  '[UPLOAD]',
+  '[ARTIFACT]',
+  '[SKIP]',
+  '[TestLens] onTestEnd called',
+  '[TestLens] Sending testEnd'
+];
+const noisyInfoSubstrings = [
+  'Sent runStart event to TestLens',
+  'Sent specStart event to TestLens',
+  'Sent testStart event to TestLens',
+  'Sent testEnd event to TestLens',
+  'Sent runEnd event to TestLens',
+  'Sent artifact event to TestLens',
+  'S3 upload completed',
+  'Upload confirmed',
+  'Uploading ',
+  'Processed artifact',
+  'Spec code blocks',
+  '\n[METADATA]',
+  'Custom Metadata Detected',
+  'Using API key from environment variable'
+];
+
+const shouldSuppressInfo = (msg: string): boolean => {
+  if (isVerboseLogs) return false;
+  if (!msg) return false;
+  return (
+    noisyInfoPrefixes.some(prefix => msg.startsWith(prefix)) ||
+    noisyInfoSubstrings.some(sub => msg.includes(sub))
+  );
+};
+
+// Create pino logger instance
+const logger = pino({
+  level: process.env.TESTLENS_LOG_LEVEL || process.env.LOG_LEVEL || 'info',
+  base: null, // remove pid/hostname
+  timestamp: false, // remove time
+  formatters: {
+    level: () => ({}) // remove level
+  },
+  hooks: {
+    logMethod(args, method, level) {
+      // pino level is numeric (info = 30)
+      if (level === 30) {
+        const msg =
+          typeof args[0] === 'string'
+            ? args[0]
+            : typeof args[1] === 'string'
+              ? args[1]
+              : '';
+        if (shouldSuppressInfo(msg)) {
+          return;
+        }
+      }
+      return method.apply(this, args as any);
+    }
+  }
+});
+
+// Lazy-load mime module to support ESM
+let mimeModule: any = null;
+async function getMime() {
+  if (!mimeModule) {
+    const imported = await import('mime');
+    // Handle both default export and named exports
+    mimeModule = imported.default || imported;
+  }
+  return mimeModule;
+}
+
+export interface TestLensReporterConfig {
+  /** TestLens API endpoint URL */
+  apiEndpoint?: string;
+  /** API key for authentication - can be provided in config or via TESTLENS_API_KEY environment variable */
+  apiKey?: string;
+  /** Enable real-time streaming of test events */
+  enableRealTimeStream?: boolean;
+  /** Enable Git information collection */
+  enableGitInfo?: boolean;
+  /** Enable artifact processing */
+  enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
+  /** Batch size for API requests */
+  batchSize?: number;
+  /** Flush interval in milliseconds */
+  flushInterval?: number;
+  /** Number of retry attempts for failed API calls */
+  retryAttempts?: number;
+  /** Request timeout in milliseconds */
+  timeout?: number;
+  /** SSL certificate validation - set to false to disable SSL verification */
+  rejectUnauthorized?: boolean;
+  /** Alternative SSL option - set to true to ignore SSL certificate errors */
+  ignoreSslErrors?: boolean;
+  /** Custom metadata from CLI arguments (automatically parsed from --key=value arguments) */
+  customMetadata?: Record<string, string | string[]>;
+}
+
+export interface TestLensReporterOptions {
+  /** TestLens API endpoint URL */
+  apiEndpoint?: string;
+  /** API key for authentication - can be provided in config or via TESTLENS_API_KEY environment variable */
+  apiKey?: string;
+  /** Enable real-time streaming of test events */
+  enableRealTimeStream?: boolean;
+  /** Enable Git information collection */
+  enableGitInfo?: boolean;
+  /** Enable artifact processing */
+  enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
+  /** Batch size for API requests */
+  batchSize?: number;
+  /** Flush interval in milliseconds */
+  flushInterval?: number;
+  /** Number of retry attempts for failed API calls */
+  retryAttempts?: number;
+  /** Request timeout in milliseconds */
+  timeout?: number;
+  /** SSL certificate validation - set to false to disable SSL verification */
+  rejectUnauthorized?: boolean;
+  /** Alternative SSL option - set to true to ignore SSL certificate errors */
+  ignoreSslErrors?: boolean;
+  /** Custom metadata from CLI arguments (automatically parsed from --key=value arguments) */
+  customMetadata?: Record<string, string | string[]>;
+}
+
+export interface GitInfo {
+  branch: string;
+  commit: string;
+  shortCommit: string;
+  author: string;
+  message: string;
+  timestamp: string;
+  isDirty: boolean;
+  remoteName: string;
+  remoteUrl: string;
+}
+
+export interface CodeBlock {
+  type: 'test' | 'describe';
+  name: string;
+  content: string;
+  summary?: string;
+  describe?: string;
+  startLine?: number;
+  endLine?: number;
+}
+
+export interface RunMetadata {
+  id: string;
+  startTime: string;
+  endTime?: string;
+  duration?: number;
+  environment: string;
+  browser: string;
+  os: string;
+  playwrightVersion: string;
+  nodeVersion: string;
+  testlensVersion?: string;
+  gitInfo?: GitInfo | null;
+  shardInfo?: {
+    current: number;
+    total: number;
+  };
+  totalTests?: number;
+  passedTests?: number;
+  failedTests?: number;
+  skippedTests?: number;
+  timedOutTests?: number;
+  status?: string;
+  testlensBuildName?: string;
+  customMetadata?: Record<string, string | string[]>;
+}
+
+export interface TestError {
+  message: string;
+  stack?: string;
+  location?: {
+    file: string;
+    line: number;
+    column: number;
+  };
+  snippet?: string;
+  expected?: string;
+  actual?: string;
+  diff?: string;
+  matcherName?: string;
+  timeout?: number;
+}
+
+export interface TestData {
+  id: string;
+  name: string;
+  status: string;
+  originalStatus?: string; // Preserve original Playwright status
+  duration: number;
+  startTime: string;
+  endTime: string;
+  errorMessages: string[];
+  errors?: TestError[]; // Rich error details
+  retryAttempts: number;
+  currentRetry: number;
+  annotations: Array<{ type: string; description?: string }>;
+  projectName: string;
+  workerIndex?: number;
+  parallelIndex?: number;
+  location?: {
+    file: string;
+    line: number;
+    column: number;
+  };
+}
+
+export interface SpecData {
+  filePath: string;
+  testSuiteName: string;
+  tags?: string[];
+  startTime: string;
+  endTime?: string;
+  status: string;
+}
+
+export class TestLensReporter implements Reporter {
+  private config: Required<TestLensReporterConfig>;
+  private axiosInstance: AxiosInstance;
+  private runId: string;
+  private runMetadata: RunMetadata;
+  private specMap: Map<string, SpecData>;
+  private testMap: Map<string, TestData>;
+  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 },
+    failed: { screenshot: 0, video: 0, trace: 0, attachment: 0 }
+  };
+  private artifactsSeen: number = 0;
+
+  /**
+   * Parse custom metadata from environment variables
+   * Checks for common metadata environment variables
+   */
+  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
+      'testlensBuildTag': ['testlensBuildTag', 'TESTLENS_BUILD_TAG', 'TESTLENS_BUILDTAG', 'BUILDTAG', 'BUILD_TAG', 'TestlensBuildTag', 'TestLensBuildTag'],
+      'testlensBuildName': ['testlensBuildName', 'TESTLENS_BUILD_NAME', 'TESTLENS_BUILDNAME', 'BUILDNAME', 'BUILD_NAME', 'TestlensBuildName', 'TestLensBuildName'],
+      'environment': ['ENVIRONMENT', 'ENV', 'NODE_ENV', 'DEPLOYMENT_ENV'],
+      'branch': ['BRANCH', 'GIT_BRANCH', 'CI_COMMIT_BRANCH', 'GITHUB_REF_NAME'],
+      'team': ['TEAM', 'TEAM_NAME'],
+      'project': ['PROJECT', 'PROJECT_NAME'],
+      'customvalue': ['CUSTOMVALUE', 'CUSTOM_VALUE']
+    };
+
+    // Check for each metadata key
+    Object.entries(envVarMappings).forEach(([key, envVars]) => {
+      for (const envVar of envVars) {
+        const value = process.env[envVar];
+        if (value) {
+          // For testlensBuildTag, support comma-separated values
+          if (key === 'testlensBuildTag' && value.includes(',')) {
+            customArgs[key] = value.split(',').map(tag => tag.trim()).filter(tag => tag);
+            logger.debug(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
+          } else {
+            customArgs[key] = value;
+            logger.debug(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+          }
+          break; // Use first match
+        }
+      }
+    });
+
+    return customArgs;
+  }
+
+  constructor(options: TestLensReporterOptions) {
+    // 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
+      || process.env.testlens_api_key
+      || process.env.TESTLENS_KEY
+      || process.env.testlens_key
+      || process.env.testlensApiKey
+      || process.env.PLAYWRIGHT_API_KEY
+      || 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
+      enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
+      enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
+      enableArtifacts: options.enableArtifacts !== undefined ? options.enableArtifacts : true,
+      enableVideo: options.enableVideo !== undefined ? options.enableVideo : true, // Default to true, override from config
+      enableScreenshot: options.enableScreenshot !== undefined ? options.enableScreenshot : true, // Default to true, override from config
+      batchSize: options.batchSize || 10,
+      flushInterval: options.flushInterval || 5000,
+      retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
+      timeout: options.timeout || 60000,
+      rejectUnauthorized: options.rejectUnauthorized,
+      ignoreSslErrors: options.ignoreSslErrors,
+      customMetadata: { ...options.customMetadata, ...customArgs } // Config metadata first, then CLI args override
+    } as Required<TestLensReporterConfig>;
+
+    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');
+    }
+
+    // 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 = 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;
+      logger.debug('[DEBUG] SSL certificate validation disabled via ignoreSslErrors option');
+    } else if (this.config.rejectUnauthorized === false) {
+      rejectUnauthorized = false;
+      logger.debug('[DEBUG] SSL certificate validation disabled via rejectUnauthorized option');
+    } else if (this.config.rejectUnauthorized === true) {
+      rejectUnauthorized = true;
+    } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '0') {
+      rejectUnauthorized = false;
+      logger.debug('[DEBUG] SSL certificate validation disabled via NODE_TLS_REJECT_UNAUTHORIZED environment variable');
+    } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '1') {
+      rejectUnauthorized = true;
+    }
+
+    // Mirror the resolved value so all HTTPS requests in this process follow it
+    process.env.NODE_TLS_REJECT_UNAUTHORIZED = rejectUnauthorized ? '1' : '0';
+
+    // Set up axios instance with retry logic and enhanced SSL handling
+    this.axiosInstance = axios.create({
+      baseURL: this.config.apiEndpoint,
+      timeout: this.config.timeout,
+      headers: {
+        'Content-Type': 'application/json',
+        ...(this.config.apiKey && { 'X-API-Key': this.config.apiKey }),
+      },
+      // Enhanced SSL handling with flexible TLS configuration
+      httpsAgent: new https.Agent({
+        rejectUnauthorized: rejectUnauthorized,
+        // Allow any TLS version for better compatibility
+        minVersion: 'TLSv1.2',
+        maxVersion: 'TLSv1.3'
+      })
+    });
+
+    // Add retry interceptor
+    this.axiosInstance.interceptors.response.use(
+      (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;
+            await new Promise(resolve => setTimeout(resolve, delay));
+            return this.axiosInstance(originalRequest);
+          }
+        }
+
+        return Promise.reject(error);
+      }
+    );
+
+    this.runId = randomUUID();
+    this.runMetadata = this.initializeRunMetadata();
+    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:');
+      Object.entries(this.config.customMetadata).forEach(([key, value]) => {
+        logger.debug(`   ${key}: ${value}`);
+      });
+      logger.debug('');
+    }
+  }
+
+  private initializeRunMetadata(): RunMetadata {
+    const metadata: RunMetadata = {
+      id: this.runId,
+      startTime: new Date().toISOString(),
+      environment: 'production',
+      browser: 'multiple',
+      os: `${os.type()} ${os.release()}`,
+      playwrightVersion: this.getPlaywrightVersion(),
+      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;
+        // Handle both string and array (take first element if array)
+        metadata.testlensBuildName = Array.isArray(buildName) ? buildName[0] : buildName;
+      }
+    }
+
+    return metadata;
+  }
+
+  private getPlaywrightVersion(): string {
+    try {
+      const playwrightPackage = require('@playwright/test/package.json');
+      return playwrightPackage.version;
+    } catch (error) {
+      return 'unknown';
+    }
+  }
+
+  private getTestLensVersion(): string {
+    try {
+      const testlensPackage = require('./package.json');
+      return testlensPackage.version;
+    } catch (error) {
+      return 'unknown';
+    }
+  }
+
+  private normalizeTestStatus(status: string): string {
+    // Treat timeout as failed for consistency with analytics
+    if (status === 'timedOut') {
+      return 'failed';
+    }
+    return status;
+  }
+
+  private normalizeRunStatus(status: string, hasTimeouts: boolean): string {
+    // If run has timeouts, treat as failed
+    if (hasTimeouts && status === 'passed') {
+      return 'failed';
+    }
+    // Treat timeout status as failed
+    if (status === 'timedOut') {
+      return 'failed';
+    }
+    return status;
+  }
+
+  async onBegin(config: FullConfig, suite: Suite): Promise<void> {
+    logger.debug(`[REPORTER] source=${__filename} enableArtifacts=${this.config.enableArtifacts}`);
+    // Show Build Name if provided, otherwise show Run ID
+    if (this.runMetadata.testlensBuildName) {
+      logger.debug(`TestLens Reporter starting - Build: ${this.runMetadata.testlensBuildName}`);
+      logger.debug(`   Run ID: ${this.runId}`);
+    } 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();
+      if (this.runMetadata.gitInfo) {
+        logger.debug(`Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+      } else {
+        logger.warn(`[WARN] Git info collection returned null - not in a git repository or git not available`);
+      }
+    } else {
+      logger.debug(`[INFO] Git info collection disabled (enableGitInfo: false)`);
+    }
+
+    // Add shard information if available
+    if (config.shard) {
+      this.runMetadata.shardInfo = {
+        current: config.shard.current,
+        total: config.shard.total
+      };
+    }
+
+    // Send run start event to API
+    await this.sendToApi({
+      type: 'runStart',
+      runId: this.runId,
+      timestamp: new Date().toISOString(),
+      metadata: this.runMetadata
+    });
+  }
+
+  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}`;
+
+    // Create or update spec data
+    if (!this.specMap.has(specKey)) {
+      const extractedTags = this.extractTags(test);
+      const specData: SpecData = {
+        filePath: path.relative(process.cwd(), specPath),
+        testSuiteName: test.parent.title,
+        startTime: new Date().toISOString(),
+        status: 'running'
+      };
+      if (extractedTags.length > 0) {
+        specData.tags = extractedTags;
+      }
+      this.specMap.set(specKey, specData);
+
+      // Send spec start event to API
+      await this.sendToApi({
+        type: 'specStart',
+        runId: this.runId,
+        timestamp: new Date().toISOString(),
+        spec: specData
+      });
+    }
+
+    const testId = this.getTestId(test);
+
+    // Only send testStart event on first attempt (retry 0)
+    if (result.retry === 0) {
+      // Create test data
+      const testData: TestData = {
+        id: testId,
+        name: test.title,
+        status: 'running',
+        originalStatus: 'running',
+        duration: 0,
+        startTime: new Date().toISOString(),
+        endTime: '',
+        errorMessages: [],
+        errors: [],
+        retryAttempts: test.retries,
+        currentRetry: result.retry,
+        annotations: test.annotations.map((ann: any) => ({
+          type: ann.type,
+          description: ann.description
+        })),
+        projectName: test.parent.project()?.name || 'default',
+        workerIndex: result.workerIndex,
+        parallelIndex: result.parallelIndex,
+        location: {
+          file: path.relative(process.cwd(), test.location.file),
+          line: test.location.line,
+          column: test.location.column
+        }
+      };
+
+      this.testMap.set(testData.id, testData);
+
+      // Send test start event to API
+      await this.sendToApi({
+        type: 'testStart',
+        runId: this.runId,
+        timestamp: new Date().toISOString(),
+        test: testData
+      });
+    } else {
+      // For retries, just update the existing test data
+      const existingTestData = this.testMap.get(testId);
+      if (existingTestData) {
+        existingTestData.currentRetry = result.retry;
+      }
+    }
+  }
+
+  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}`);
+
+    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.debug(`[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) {
+      logger.debug(`[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}`;
+
+      if (!this.specMap.has(specKey)) {
+        const extractedTags = this.extractTags(test);
+        const specData: SpecData = {
+          filePath: path.relative(process.cwd(), specPath),
+          testSuiteName: test.parent.title,
+          startTime: new Date().toISOString(),
+          status: 'skipped'
+        };
+        if (extractedTags.length > 0) {
+          specData.tags = extractedTags;
+        }
+        this.specMap.set(specKey, specData);
+
+        // Send spec start event to API
+        await this.sendToApi({
+          type: 'specStart',
+          runId: this.runId,
+          timestamp: new Date().toISOString(),
+          spec: specData
+        });
+      }
+
+      // Create test data for skipped test
+      testData = {
+        id: testId,
+        name: test.title,
+        status: 'skipped',
+        originalStatus: 'skipped',
+        duration: 0,
+        startTime: new Date().toISOString(),
+        endTime: new Date().toISOString(),
+        errorMessages: [],
+        errors: [],
+        retryAttempts: test.retries,
+        currentRetry: 0,
+        annotations: test.annotations.map((ann: any) => ({
+          type: ann.type,
+          description: ann.description
+        })),
+        projectName: test.parent.project()?.name || 'default',
+        workerIndex: result.workerIndex,
+        parallelIndex: result.parallelIndex,
+        location: {
+          file: path.relative(process.cwd(), test.location.file),
+          line: test.location.line,
+          column: test.location.column
+        }
+      };
+
+      this.testMap.set(testId, testData);
+
+      // Send test start event first (so the test gets created in DB)
+      await this.sendToApi({
+        type: 'testStart',
+        runId: this.runId,
+        timestamp: new Date().toISOString(),
+        test: testData
+      });
+    }
+
+    if (testData) {
+      // Update test data with latest result
+      testData.originalStatus = result.status;
+      testData.status = this.normalizeTestStatus(result.status);
+      testData.duration = result.duration;
+      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 = {
+            file: path.relative(process.cwd(), error.location.file),
+            line: error.location.line,
+            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|$)/);
+          if (expectedStringMatch) {
+            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) {
+          // Store call log separately for display
+          const callLog = callLogMatch[1].trim();
+          if (callLog) {
+            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) {
+          const codeSnippetMatch = message.match(/((?:\s*>?\s*\d+\s*\|.*\n?)+)/);
+          if (codeSnippetMatch) {
+            testError.snippet = codeSnippetMatch[1].trim();
+          }
+        }
+
+        return testError;
+      });
+
+      // Send testEnd event for all tests, regardless of status
+      // This ensures tests that are interrupted or have unexpected statuses are properly recorded
+      logger.debug(`[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',
+        runId: this.runId,
+        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; 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);
+          this.artifactsSeen += 1;
+          this.bumpArtifactStat('skipped', artifactType);
+        }
+      }
+    }
+
+    // Update spec status
+    const specPath = test.location.file;
+    const specKey = `${specPath}-${test.parent.title}`;
+    const specData = this.specMap.get(specKey);
+    if (specData) {
+      const normalizedStatus = this.normalizeTestStatus(result.status);
+      if (normalizedStatus === 'failed' && specData.status !== 'failed') {
+        specData.status = 'failed';
+      } else if (result.status === 'skipped' && specData.status === 'passed') {
+        specData.status = 'skipped';
+      }
+
+      // Check if all tests in spec are complete
+      // Only consider tests that were actually executed (have testData)
+      const remainingTests = test.parent.tests.filter((t: any) => {
+        const tId = this.getTestId(t);
+        const tData = this.testMap.get(tId);
+        // If testData exists but no endTime, it's still running
+        return tData && !tData.endTime;
+      });
+
+      if (remainingTests.length === 0) {
+        // Determine final spec status based on all executed tests
+        const executedTests = test.parent.tests
+          .map((t: any) => {
+            const tId = this.getTestId(t);
+            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')) {
+            specData.status = 'passed';
+          } else if (allTestStatuses.some(status => status === 'failed')) {
+            specData.status = 'failed';
+          } else if (allTestStatuses.every(status => status === 'skipped')) {
+            specData.status = 'skipped';
+          }
+        }
+        // Aggregate tags from all tests in this spec
+        const allTags = new Set<string>();
+        test.parent.tests.forEach((t: any) => {
+          const tags = this.extractTags(t);
+          tags.forEach(tag => allTags.add(tag));
+        });
+        const aggregatedTags = Array.from(allTags);
+        // Only update tags if we have any
+        if (aggregatedTags.length > 0) {
+          specData.tags = aggregatedTags;
+        }
+
+        specData.endTime = new Date().toISOString();
+
+        // Send spec end event to API
+        await this.sendToApi({
+          type: 'specEnd',
+          runId: this.runId,
+          timestamp: new Date().toISOString(),
+          spec: specData
+        });
+
+        // Send spec code blocks to API
+        await this.sendSpecCodeBlocks(specPath, test.title, testData?.errors || [], this.runId, testId, testCaseId);
+      }
+    }
+  }
+
+  async onEnd(result: { status: string }): Promise<void> {
+    this.runMetadata.endTime = new Date().toISOString();
+    this.runMetadata.duration = Date.now() - new Date(this.runMetadata.startTime).getTime();
+
+    // Wait for all pending artifact uploads to complete before sending runEnd
+    if (this.pendingUploads.size > 0) {
+      logger.debug(`Waiting for ${this.pendingUploads.size} artifact upload(s) to complete...`);
+      try {
+        await Promise.all(Array.from(this.pendingUploads));
+        logger.debug(`[OK] All artifact uploads completed`);
+      } catch (error) {
+        logger.error(`[WARN] Some artifact uploads failed, continuing with runEnd`);
+      }
+    }
+
+    const uploaded = this.artifactStats.uploaded;
+    const skipped = this.artifactStats.skipped;
+    const failed = this.artifactStats.failed;
+    // Always show summary so users can confirm artifact settings
+    const summary =
+      `[ARTIFACTS] seen=${this.artifactsSeen} | ` +
+      `uploaded screenshot=${uploaded.screenshot}, video=${uploaded.video}, trace=${uploaded.trace}, attachment=${uploaded.attachment} | ` +
+      `skipped screenshot=${skipped.screenshot}, video=${skipped.video}, trace=${skipped.trace}, attachment=${skipped.attachment} | ` +
+      `failed screenshot=${failed.screenshot}, video=${failed.video}, trace=${failed.trace}, attachment=${failed.attachment}`;
+    if (logger.levelVal > 30) {
+      console.log(summary);
+    } else {
+      logger.debug(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;
+    // failedTests already includes timedOut tests since normalizeTestStatus converts 'timedOut' to 'failed'
+    const failedTests = Array.from(this.testMap.values()).filter(t => t.status === 'failed').length;
+    const skippedTests = Array.from(this.testMap.values()).filter(t => t.status === 'skipped').length;
+    // Track timedOut separately for reporting purposes only (not for count)
+    const timedOutTests = Array.from(this.testMap.values()).filter(t => t.originalStatus === 'timedOut').length;
+
+    // Normalize run status - if there are timeouts, treat run as failed
+    const hasTimeouts = timedOutTests > 0;
+    const normalizedRunStatus = this.normalizeRunStatus(result.status, hasTimeouts);
+
+    // Send run end event to API
+    await this.sendToApi({
+      type: 'runEnd',
+      runId: this.runId,
+      timestamp: new Date().toISOString(),
+      metadata: {
+        ...this.runMetadata,
+        totalTests,
+        passedTests,
+        failedTests, // Already includes timedOut tests (normalized to 'failed')
+        skippedTests,
+        timedOutTests, // For informational purposes
+        status: normalizedRunStatus
+      }
+    });
+
+    // Show Build Name if provided, otherwise show Run ID
+    if (this.runMetadata.testlensBuildName) {
+      logger.debug(`[COMPLETE] TestLens Report completed - Build: ${this.runMetadata.testlensBuildName}`);
+      logger.debug(`   Run ID: ${this.runId}`);
+    } else {
+      logger.debug(`[COMPLETE] TestLens Report completed - Run ID: ${this.runId}`);
+    }
+    logger.debug(`[RESULTS] ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
+  }
+
+  private async sendToApi(payload: any): Promise<any> {
+    // Skip sending if run creation already failed
+    if (this.runCreationFailed && payload.type !== 'runStart') {
+      return null;
+    }
+
+    try {
+      const response = await this.axiosInstance.post('', payload, {
+        headers: {
+          'X-API-Key': this.config.apiKey
+        }
+      });
+      if (this.config.enableRealTimeStream) {
+        logger.debug(`[OK] Sent ${payload.type} event to TestLens (HTTP ${response.status})`);
+      }
+      // Return response data for caller to use
+      return response.data;
+    } 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');
+        } else if (errorData?.limit_type === 'test_runs') {
+          logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
+        } else {
+          logger.error('[ERROR] TESTLENS ERROR: Plan Limit Reached');
+        }
+        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') {
+          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') {
+            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
+          } 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.');
+          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) {
+            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
+            logger.error('');
+          }
+          logger.error('='.repeat(80));
+          logger.error('');
+        } else {
+          logger.error(`[ERROR] Authentication failed: ${errorData?.error || 'Invalid API key'}`);
+        }
+      } else if (status !== 403) {
+        // Log other errors (but not 403 which we handled above)
+        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,
+          code: error?.code,
+          url: error?.config?.url,
+          method: error?.config?.method
+        });
+      }
+
+      // Don't throw error to avoid breaking test execution
+    }
+  }
+
+  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);
+            if (pathExt) {
+              fileName = `${fileName}${pathExt}`;
+            } else if (attachment.contentType) {
+              // Fallback: derive extension from contentType
+              const mimeToExt: Record<string, string> = {
+                'image/png': '.png',
+                'image/jpeg': '.jpg',
+                'image/gif': '.gif',
+                'image/webp': '.webp',
+                'video/webm': '.webm',
+                'video/mp4': '.mp4',
+                'application/zip': '.zip',
+                'application/json': '.json',
+                'text/plain': '.txt'
+              };
+              const ext = mimeToExt[attachment.contentType];
+              if (ext) {
+                fileName = `${fileName}${ext}`;
+              }
+            }
+          }
+
+          // Upload to S3 first (pass DB ID if available for faster lookup)
+          // Create upload promise that we can track
+          const uploadPromise = Promise.resolve().then(async () => {
+            try {
+              if (!attachment.path) {
+                logger.debug(`[SKIP] [Test: ${testId.substring(0, 8)}...] Skipping artifact ${attachment.name} - no file path`);
+                this.bumpArtifactStat('skipped', artifactType);
+                return;
+              }
+
+              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName, testCaseDbId);
+
+              // Skip if upload failed or file was too large
+              if (!s3Data) {
+                logger.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),
+                path: attachment.path,
+                name: fileName,
+                contentType: attachment.contentType,
+                fileSize: this.getFileSize(attachment.path),
+                storageType: 's3',
+                s3Key: s3Data.key,
+                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({
+                type: 'artifact',
+                runId: this.runId,
+                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) {
+          logger.error(`[ERROR] [Test: ${testId.substring(0, 8)}...] Failed to setup artifact upload ${attachment.name}: ${(error as Error).message}`);
+          this.bumpArtifactStat('failed', artifactType);
+        }
+      } else {
+        this.bumpArtifactStat('skipped', artifactType);
+      }
+    }
+  }
+
+  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.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', '');
+      if (baseUrl === this.config.apiEndpoint) {
+        // Fallback: try alternative pattern if main pattern didn't match
+        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,
+        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'}`);
+    }
+  }
+
+  private extractTestBlocks(filePath: string): CodeBlock[] {
+    try {
+      const content = fs.readFileSync(filePath, 'utf-8');
+      const blocks: CodeBlock[] = [];
+      const lines = content.split('\n');
+
+      // 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();
+
+        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
+        if (inBlock) {
+          for (const char of line) {
+            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: 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}`);
+      return [];
+    }
+  }
+
+  private async collectGitInfo(): Promise<GitInfo | null> {
+    try {
+      const branch = execSync('git rev-parse --abbrev-ref HEAD', { encoding: 'utf-8' }).trim();
+      const commit = execSync('git rev-parse HEAD', { encoding: 'utf-8' }).trim();
+      const shortCommit = commit.substring(0, 7);
+      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 {
+        const remotes = execSync('git remote', { encoding: 'utf-8' }).trim();
+        if (remotes) {
+          remoteName = remotes.split('\n')[0] || 'origin';
+          remoteUrl = execSync(`git remote get-url ${remoteName}`, { encoding: 'utf-8' }).trim();
+        }
+      } catch (e) {
+        // 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,
+        shortCommit,
+        author,
+        message: commitMessage,
+        timestamp: commitTimestamp,
+        isDirty,
+        remoteName,
+        remoteUrl
+      };
+    } catch (error: any) {
+      // Silently skip git information if not in a git repository
+      return null;
+    }
+  }
+
+  private getArtifactType(name: string): string {
+    if (name.includes('screenshot')) return 'screenshot';
+    if (name.includes('video')) return 'video';
+    if (name.includes('trace')) return 'trace';
+    return 'attachment';
+  }
+
+  private bumpArtifactStat(stat: 'uploaded' | 'skipped' | 'failed', type: string): void {
+    const bucket = this.artifactStats[stat] as Record<string, number>;
+    if (bucket[type] !== undefined) {
+      bucket[type] += 1;
+    }
+  }
+
+  private extractTags(test: TestCase): string[] {
+    const tags: string[] = [];
+
+    // Playwright stores tags in the _tags property
+    const testTags = (test as any)._tags;
+    if (testTags && Array.isArray(testTags)) {
+      tags.push(...testTags);
+    }
+
+    // Also get tags from parent suites by walking up the tree
+    let currentSuite: Suite | undefined = test.parent;
+    while (currentSuite) {
+      const suiteTags = (currentSuite as any)._tags;
+      if (suiteTags && Array.isArray(suiteTags)) {
+        tags.push(...suiteTags);
+      }
+      currentSuite = currentSuite.parent;
+    }
+
+    // Also extract @tags from test title for backward compatibility
+    const tagMatches = test.title.match(/@[\w-]+/g);
+    if (tagMatches) {
+      tags.push(...tagMatches);
+    }
+
+    // Add testlensBuildTag: CLI args take precedence over config
+    const buildTagSource = this.cliArgs.testlensBuildTag || this.config.customMetadata?.testlensBuildTag;
+    if (buildTagSource) {
+      const buildTags = Array.isArray(buildTagSource)
+        ? buildTagSource
+        : [buildTagSource];
+      buildTags.forEach(tag => tags.push(`@${tag}`));
+    }
+
+    // Remove duplicates and return
+    return [...new Set(tags)];
+  }
+
+  private getTestId(test: TestCase): string {
+    const cleanTitle = test.title.replace(/@[\w-]+/g, '').trim();
+    // Normalize path separators to forward slashes for cross-platform consistency
+    const normalizedFile = test.location.file.replace(/\\/g, '/');
+    return `${normalizedFile}:${test.location.line}:${cleanTitle}`;
+  }
+
+
+  private async uploadArtifactToS3(filePath: string, testId: string, fileName: string, testCaseDbId?: string): Promise<{ key: string; url: string; presignedUrl: string; fileSize: number; contentType: string } | null> {
+    try {
+      // 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 = {
+        testRunId: this.runId,
+        testId: testId,
+        fileName: fileName,
+        fileType: await this.getContentType(fileName),
+        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
+      });
+
+      if (!presignedResponse.data.success) {
+        throw new Error(`Failed to get presigned URL: ${presignedResponse.data.error || 'Unknown error'}`);
+      }
+
+      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
+      const uploadResponse = await axios.put(uploadUrl, fileBuffer, {
+        headers: {
+          'x-amz-server-side-encryption': 'AES256'
+        },
+        maxContentLength: Infinity,
+        maxBodyLength: Infinity,
+        timeout: Math.max(600000, Math.ceil(fileSize / (1024 * 1024)) * 10000), // 10s per MB, min 10 minutes - increased for large trace files
+        // Don't use custom HTTPS agent for S3 uploads
+        httpsAgent: undefined
+      });
+
+      if (uploadResponse.status !== 200) {
+        throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
+      }
+
+      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: any = {
+        testRunId: this.runId,
+        testId: testId,
+        s3Key: s3Key,
+        fileName: fileName,
+        fileType: await this.getContentType(fileName),
+        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
+      });
+
+      if (confirmResponse.status === 201 && confirmResponse.data.success) {
+        const artifact = confirmResponse.data.artifact;
+        logger.debug(`[OK] [Test: ${testId.substring(0, 8)}...] Upload confirmed${testCaseDbId ? ' (fast path)' : ' (fallback)'}`);
+        return {
+          key: s3Key,
+          url: artifact.s3Url,
+          presignedUrl: artifact.presignedUrl,
+          fileSize: artifact.fileSize,
+          contentType: artifact.contentType
+        };
+      } else {
+        throw new Error(`Upload confirmation failed: ${confirmResponse.data.error || 'Unknown error'}`);
+      }
+    } catch (error: any) {
+      // 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') {
+          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') {
+            logger.error('[ERROR] TESTLENS ERROR: Test Runs Limit Reached');
+          } 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.');
+          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) {
+            logger.error(`Trial ended: ${new Date(errorData.trial_end_date).toLocaleDateString()}`);
+            logger.error('');
+          }
+          logger.error('='.repeat(80));
+          logger.error('');
+          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) {
+        errorMsg = `File too large (413) - server rejected the upload`;
+      } else if (error.response?.status === 400) {
+        errorMsg = `Bad request (400) - ${error.response.data?.error || 'check file format'}`;
+      } 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;
+    }
+  }
+
+  private async getContentType(fileName: string): Promise<string> {
+    const ext = path.extname(fileName).toLowerCase();
+    try {
+      const mime = await getMime();
+      // Try different ways to access getType method
+      const getType = mime.getType || mime.default?.getType;
+      if (typeof getType === 'function') {
+        const mimeType = getType.call(mime, ext) || (mime.default ? getType.call(mime.default, ext) : null);
+        return mimeType || 'application/octet-stream';
+      }
+    } catch (error: any) {
+      logger.warn(`Failed to get MIME type for ${fileName}: ${error.message}`);
+    }
+    // Fallback to basic content type mapping
+    const contentTypes: Record<string, string> = {
+      '.mp4': 'video/mp4',
+      '.webm': 'video/webm',
+      '.png': 'image/png',
+      '.jpg': 'image/jpeg',
+      '.jpeg': 'image/jpeg',
+      '.gif': 'image/gif',
+      '.json': 'application/json',
+      '.txt': 'text/plain',
+      '.html': 'text/html',
+      '.xml': 'application/xml',
+      '.zip': 'application/zip',
+      '.pdf': 'application/pdf'
+    };
+    return contentTypes[ext] || 'application/octet-stream';
+  }
+
+  // Note: S3 key generation and sanitization are handled server-side
+  // generateS3Key() and sanitizeForS3() methods removed as they were not used
+
+  private getFileSize(filePath: string): number {
+    try {
+      const stats = fs.statSync(filePath);
+      return stats.size;
+    } catch (error) {
+      logger.warn(`Could not get file size for ${filePath}: ${(error as Error).message}`);
+      return 0;
+    }
+  }
+}
+
+export default TestLensReporter;