@m00nsolutions/playwright-reporter 1.0.4 → 1.0.6

package/README.md

@@ -1,6 +1,6 @@
 # @m00nsolutions/playwright-reporter
 
-Official Playwright test reporter for [M00n Report](https://app.m00n.report) - a real-time test reporting dashboard.
+Official Playwright test reporter for [M00n Report](https://m00nreport.com) - a real-time test reporting dashboard.
 
 ## Features
 
@@ -24,7 +24,7 @@ pnpm add @m00nsolutions/playwright-reporter --save-dev
 
 ### 1. Get your API key
 
-1. Log in to [app.m00n.report](https://app.m00n.report)
+1. Log in to [m00nreport.com](https://m00nreport.com)
 2. Navigate to your project settings
 3. Generate or copy your project API key
 
@@ -39,7 +39,7 @@ export default defineConfig({
   reporter: [
     ['list'], // Keep default console output
     ['@m00nsolutions/playwright-reporter', {
-      serverUrl: 'https://app.m00n.report',      // Or your self-hosted URL
+      serverUrl: 'https://m00nreport.com',      // Or your self-hosted URL
       apiKey: process.env.M00N_API_KEY,           // Your project API key
       launch: 'Regression Suite',                  // Optional: run title
       tags: ['smoke', 'regression'],               // Optional: tags
@@ -75,7 +75,7 @@ That's it! Your test results will appear in the M00n Report dashboard in real-ti
 You can also configure the reporter via environment variables:
 
 ```bash
-M00N_SERVER_URL=https://app.m00n.report
+M00N_SERVER_URL=https://m00nreport.com
 M00N_API_KEY=m00n_xxxxxxxxxxxxx
 M00N_LAUNCH="Nightly Build"
 M00N_TAGS=smoke,regression
@@ -88,7 +88,7 @@ M00N_TAGS=smoke,regression
 ```typescript
 reporter: [
   ['@m00nsolutions/playwright-reporter', {
-    serverUrl: 'https://app.m00n.report',
+    serverUrl: 'https://m00nreport.com',
     apiKey: process.env.M00N_API_KEY,
   }],
 ],
@@ -132,6 +132,6 @@ MIT License - see [LICENSE](LICENSE) for details.
 
 ## Support
 
-- 📖 [Documentation](https://docs.m00n.report)
+- 📖 [Documentation](https://m00nreport.com/documentation)
 - 🐛 [Report Issues](https://github.com/m00nsolutions/m00nreport/issues)
-- 💬 [Community Discord](https://discord.gg/m00nreport)
+- 💬 [Community Discord](https://discord.gg/hzZvyVWS3Q)

package/index.mjs

@@ -38,22 +38,29 @@ import { fileURLToPath } from 'url';
 // CONSTANTS
 // ============================================================================
 
+// Maximum attachment file size - files larger than this are skipped
+// 200MB covers all realistic test artifacts (traces, videos, screenshots)
+const MAX_ATTACHMENT_SIZE = 200 * 1024 * 1024; // 200MB
+
 // Files larger than this threshold will be streamed directly to server
 // to avoid memory issues in reporter process
 const LARGE_FILE_THRESHOLD = 10 * 1024 * 1024; // 10MB
 
 // Maximum concurrent attachment uploads across ALL tests
 // Prevents server/MinIO overload with high parallelism (30+ workers)
-const MAX_GLOBAL_UPLOAD_CONCURRENCY = 10;
+// Increased from 10 to 15 to reduce queue time when many tests have attachments
+const MAX_GLOBAL_UPLOAD_CONCURRENCY = 15;
 
 // Timeout for individual attachment uploads (ms)
-const UPLOAD_TIMEOUT = 60000; // 60 seconds
+// Increased from 60s to 90s to handle larger files under load
+const UPLOAD_TIMEOUT = 90000; // 90 seconds
 
 // Timeout for streaming large file uploads (ms)
 const STREAM_UPLOAD_TIMEOUT = 300000; // 5 minutes
 
 // Timeout for waiting on pending uploads at run end (ms)
-const END_UPLOAD_WAIT_TIMEOUT = 180000; // 3 minutes
+// Increased from 3 to 5 minutes to handle many failing tests with videos
+const END_UPLOAD_WAIT_TIMEOUT = 300000; // 5 minutes
 
 // ============================================================================
 // HELPERS
@@ -557,9 +564,9 @@ class HttpClient {
         
         // Check for permanent errors (don't retry)
         const errorData = await response.json().catch(() => ({}));
-        if (['PROJECT_NOT_FOUND', 'API_KEY_REQUIRED', 'INVALID_API_KEY'].includes(errorData.code)) {
+        if (['PROJECT_NOT_FOUND', 'API_KEY_REQUIRED', 'INVALID_API_KEY', 'RUN_ATTACHMENT_LIMIT_EXCEEDED'].includes(errorData.code)) {
           endTiming(0);
-          return { error: errorData.code, permanent: true };
+          return { error: errorData.code, code: errorData.code, permanent: true, ...errorData };
         }
         
         // Transient error - retry if we have attempts left
@@ -731,6 +738,13 @@ class HttpClient {
         }
         
         const errorData = await response.json().catch(() => ({}));
+        
+        // Check for permanent errors (don't retry)
+        if (['RUN_ATTACHMENT_LIMIT_EXCEEDED', 'PROJECT_NOT_FOUND', 'API_KEY_REQUIRED', 'INVALID_API_KEY'].includes(errorData.code)) {
+          endTiming(0);
+          return { error: errorData.code, code: errorData.code, permanent: true, ...errorData };
+        }
+        
         if (attempt < maxRetries) {
           this.perfTracker?.recordRetry();
           if (this.verbose) {
@@ -887,6 +901,13 @@ class HttpClient {
         }
         
         const errorData = await response.json().catch(() => ({}));
+        
+        // Check for permanent errors (don't retry)
+        if (['RUN_ATTACHMENT_LIMIT_EXCEEDED', 'PROJECT_NOT_FOUND', 'API_KEY_REQUIRED', 'INVALID_API_KEY'].includes(errorData.code)) {
+          endTiming(0);
+          return { error: errorData.code, code: errorData.code, permanent: true, ...errorData };
+        }
+        
         if (attempt < maxRetries) {
           this.perfTracker?.recordRetry();
           if (this.verbose) {
@@ -1191,7 +1212,12 @@ class TestCollector {
         // File path - check size to decide streaming vs buffering
         try {
           const stats = await fs.promises.stat(attachment.path);
-          if (stats.size > 500 * 1024 * 1024) return null; // Skip >500MB
+          if (stats.size > MAX_ATTACHMENT_SIZE) {
+            const sizeMB = (stats.size / 1024 / 1024).toFixed(1);
+            const limitMB = (MAX_ATTACHMENT_SIZE / 1024 / 1024).toFixed(0);
+            console.warn(`[M00nReporter] Attachment skipped: "${name}" (${sizeMB}MB) exceeds ${limitMB}MB limit. File: ${attachment.path}`);
+            return null;
+          }
           
           if (name === 'trace') name = path.basename(attachment.path);
           
@@ -1237,8 +1263,15 @@ class TestCollector {
           }
         } catch (fileErr) {
           // File doesn't exist or can't be read
-          // This can happen with 'retain-on-failure' when Playwright cleans up
-          // temporary files from non-final retry attempts
+          // This commonly happens with videos when:
+          // 1. 'retain-on-failure' cleans up temporary files from non-final retry attempts
+          // 2. Race condition where video file is deleted before reporter reads it
+          // 3. Disk space issues during high-concurrency runs
+          if (fileErr.code === 'ENOENT') {
+            console.warn(`[M00nReporter] Video/attachment file not found: ${attachment.path} - file may have been cleaned up by Playwright`);
+          } else {
+            console.warn(`[M00nReporter] Failed to read attachment file ${attachment.path}: ${fileErr.message}`);
+          }
           return null;
         }
       }
@@ -1355,6 +1388,9 @@ export default class M00nReporter {
     // Prevents server/MinIO overload with high parallelism
     this.uploadSemaphore = new UploadSemaphore(MAX_GLOBAL_UPLOAD_CONCURRENCY);
     
+    // Set when server returns RUN_ATTACHMENT_LIMIT_EXCEEDED -- skip remaining uploads for this run
+    this._runLimitReached = false;
+    
     // test/start is now fire-and-forget (no need to track promises)
     // testId is generated client-side, so steps can be streamed immediately
     
@@ -1674,9 +1710,8 @@ export default class M00nReporter {
         attachment => collector.addAttachment(attachment, collector.testId, this.runId, this.binaryAttachments)
           .catch(err => {
             // Don't let attachment read errors crash the test
-            if (this.debug) {
-              this.log('debug', `Failed to read attachment: ${err.message}`);
-            }
+            // Always log warning for attachment failures (helps diagnose missing videos)
+            this.log('warn', `Failed to read attachment "${attachment.name || 'unknown'}": ${err.message}`);
             return null;
           })
       );
@@ -1799,9 +1834,17 @@ export default class M00nReporter {
   // Uses semaphore to limit concurrent uploads across ALL tests
   // Returns { uploaded: N, failed: M } for tracking
   async uploadAttachmentsWithBackpressure(runId, testId, attachments) {
-    if (!attachments || attachments.length === 0) return { uploaded: 0, failed: 0 };
+    if (!attachments || attachments.length === 0) return { uploaded: 0, failed: 0, skipped: 0 };
+    
+    // If a previous upload hit the run attachment limit, skip all remaining uploads
+    if (this._runLimitReached) {
+      if (this.debug) {
+        this.log('debug', `Skipping ${attachments.length} attachment(s) - run attachment limit already reached`);
+      }
+      return { uploaded: 0, failed: 0, skipped: attachments.length };
+    }
     
-    const results = { uploaded: 0, failed: 0 };
+    const results = { uploaded: 0, failed: 0, skipped: 0 };
     const uploadStart = performance.now();
     
     // Separate small and large files
@@ -1814,6 +1857,9 @@ export default class M00nReporter {
     // Small files - upload individually with semaphore for better parallelism
     for (const file of smallFiles) {
       uploadTasks.push(this.uploadSemaphore.run(async () => {
+        // Check limit flag before each upload (may have been set by a concurrent upload)
+        if (this._runLimitReached) return 'skipped';
+        
         try {
           const resp = await this.http.postMultipart(
             '/api/ingest/v2/attachment/upload',
@@ -1822,6 +1868,13 @@ export default class M00nReporter {
             { timeout: UPLOAD_TIMEOUT }
           );
           
+          if (resp.code === 'RUN_ATTACHMENT_LIMIT_EXCEEDED') {
+            this._runLimitReached = true;
+            const usedMB = resp.currentBytes ? Math.round(resp.currentBytes / 1024 / 1024) : '?';
+            const limitMB = resp.limitBytes ? Math.round(resp.limitBytes / 1024 / 1024) : '?';
+            this.log('warn', `Attachment skipped: run attachment limit reached (${usedMB}/${limitMB} MB used). Remaining attachments for this run will be skipped.`);
+            return 'skipped';
+          }
           if (resp.error) {
             this.addError(`Upload failed (${file.name}): ${resp.error}`);
             return false;
@@ -1837,6 +1890,9 @@ export default class M00nReporter {
     // Large files - stream upload with semaphore
     for (const largeFile of largeFiles) {
       uploadTasks.push(this.uploadSemaphore.run(async () => {
+        // Check limit flag before each upload (may have been set by a concurrent upload)
+        if (this._runLimitReached) return 'skipped';
+        
         try {
           if (this.debug) {
             this.log('debug', `Streaming large file: ${largeFile.name} (${(largeFile.size / 1024 / 1024).toFixed(1)}MB)`);
@@ -1851,6 +1907,13 @@ export default class M00nReporter {
             { timeout: STREAM_UPLOAD_TIMEOUT }
           );
           
+          if (resp.code === 'RUN_ATTACHMENT_LIMIT_EXCEEDED') {
+            this._runLimitReached = true;
+            const usedMB = resp.currentBytes ? Math.round(resp.currentBytes / 1024 / 1024) : '?';
+            const limitMB = resp.limitBytes ? Math.round(resp.limitBytes / 1024 / 1024) : '?';
+            this.log('warn', `Attachment skipped: run attachment limit reached (${usedMB}/${limitMB} MB used). Remaining attachments for this run will be skipped.`);
+            return 'skipped';
+          }
           if (resp.error) {
             this.addError(`Stream upload failed (${largeFile.name}): ${resp.error}`);
             return false;
@@ -1870,6 +1933,8 @@ export default class M00nReporter {
     for (const result of uploadResults) {
       if (result.status === 'fulfilled' && result.value === true) {
         results.uploaded++;
+      } else if (result.status === 'fulfilled' && result.value === 'skipped') {
+        results.skipped++;
       } else {
         results.failed++;
       }
@@ -1878,7 +1943,9 @@ export default class M00nReporter {
     const uploadDuration = performance.now() - uploadStart;
     
     if (this.verbose) {
-      this.log('perf', `Attachments uploaded: ${results.uploaded}/${attachments.length} in ${Math.round(uploadDuration)}ms`);
+      const parts = [`${results.uploaded}/${attachments.length} in ${Math.round(uploadDuration)}ms`];
+      if (results.skipped > 0) parts.push(`${results.skipped} skipped (limit reached)`);
+      this.log('perf', `Attachments uploaded: ${parts.join(', ')}`);
     }
     
     // Log to file
@@ -1887,6 +1954,8 @@ export default class M00nReporter {
       total: attachments.length,
       uploaded: results.uploaded,
       failed: results.failed,
+      skipped: results.skipped,
+      limitReached: this._runLimitReached,
       duration: Math.round(uploadDuration),
     });
     
@@ -1988,6 +2057,9 @@ export default class M00nReporter {
           const attachmentWaitDuration = Math.round(performance.now() - attachmentWaitStart);
           
           if (uploadWaitResult === 'timeout') {
+            const timeoutMinutes = Math.round(END_UPLOAD_WAIT_TIMEOUT / 60000);
+            this.log('warn', `Timeout waiting for attachment uploads after ${timeoutMinutes} minutes - some attachments (videos/traces) may be missing!`);
+            this.log('warn', `Tip: If this happens often, consider reducing video size or increasing END_UPLOAD_WAIT_TIMEOUT`);
             this.addError(`Timeout waiting for attachment uploads after ${END_UPLOAD_WAIT_TIMEOUT/1000}s - some attachments may be missing`);
             this.fileLogger?.event('ATTACHMENTS_WAIT_TIMEOUT', {
               duration: attachmentWaitDuration,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@m00nsolutions/playwright-reporter",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "Playwright test reporter for M00n Report dashboard - real-time test result streaming with step tracking, attachments, and retry support",
   "main": "index.mjs",
   "type": "module",
@@ -42,7 +42,7 @@
     "url": "https://github.com/m00nsolutions/m00nreport.git",
     "directory": "packages/m00n-playwright-reporter"
   },
-  "homepage": "https://m00n.report",
+  "homepage": "https://m00nreport.com",
   "bugs": {
     "url": "https://github.com/m00nsolutions/m00nreport/issues"
   },