playwright-slack-report-burak 3.0.0 → 3.0.3

package/dist/src/ResultsParser.js

@@ -343,19 +343,53 @@ class ResultsParser {
 
     async fetchAllArtifacts() {
         const summariesDir = path.join('./', 'playwright-report');
-        while (!this.allNodeSummaryFilesExist() || !this.allBlobZipsExist()) {
-          console.log('Waiting for all blob zips to exist...');
-          if (!fs.existsSync(summariesDir)) {
-              fs.mkdirSync(summariesDir, { recursive: true });
-          }
-          // Fetch artifacts for shards 1 to totalShardCount-1 (if using 1-based) or 0 to totalShardCount-1 (if 0-based)
-          // Since we're shard 0, we need to fetch from other shards
-          // The workflow uses 1-based shards (1, 2, 3...), but we need to map them to 0-based internally (0, 1, 2...)
-          // So shard 1 in workflow = shard 0 internally, shard 2 = shard 1, etc.
-          for (let i = 1; i < this.totalShardCount; i++) {            
-              await this.fetchArtifact(i, `node_summary_${i}.json`, summariesDir);
-              await this.fetchArtifact(i, `blob-report-node-${i}.zip`, summariesDir);
-          }
+        if (!fs.existsSync(summariesDir)) {
+            fs.mkdirSync(summariesDir, { recursive: true });
+        }
+        
+        console.log(`Shard 0: Waiting for artifacts from ${this.totalShardCount - 1} other shard(s)...`);
+        
+        // Fetch artifacts for shards 1 to totalShardCount-1 (since we're shard 0)
+        // We need to fetch from all other shards before proceeding
+        for (let i = 1; i < this.totalShardCount; i++) {
+            console.log(`Shard 0: Fetching artifacts from shard ${i}...`);
+            await this.fetchArtifact(i, `node_summary_${i}.json`, summariesDir);
+            await this.fetchArtifact(i, `blob-report-node-${i}.zip`, summariesDir);
+        }
+        
+        // After fetching, verify all files exist and wait if needed
+        let retryCount = 0;
+        const maxRetries = 60; // Wait up to 10 minutes (60 * 10 seconds)
+        
+        while ((!this.allNodeSummaryFilesExist() || !this.allBlobZipsExist()) && retryCount < maxRetries) {
+            console.log(`Shard 0: Waiting for all artifacts to be available (attempt ${retryCount + 1}/${maxRetries})...`);
+            
+            // Re-fetch any missing artifacts
+            for (let i = 1; i < this.totalShardCount; i++) {
+                const nodeSummaryFile = path.join(summariesDir, `node_summary_${i}.json`);
+                const blobZipFile = path.join(summariesDir, `blob-report-node-${i}.zip`);
+                
+                if (!fs.existsSync(nodeSummaryFile)) {
+                    console.log(`Shard 0: Re-fetching missing node_summary_${i}.json...`);
+                    await this.fetchArtifact(i, `node_summary_${i}.json`, summariesDir);
+                }
+                
+                if (!fs.existsSync(blobZipFile)) {
+                    console.log(`Shard 0: Re-fetching missing blob-report-node-${i}.zip...`);
+                    await this.fetchArtifact(i, `blob-report-node-${i}.zip`, summariesDir);
+                }
+            }
+            
+            retryCount++;
+            if (!this.allNodeSummaryFilesExist() || !this.allBlobZipsExist()) {
+                await new Promise(resolve => setTimeout(resolve, 10000)); // Wait 10 seconds before retry
+            }
+        }
+        
+        if (this.allNodeSummaryFilesExist() && this.allBlobZipsExist()) {
+            console.log(`Shard 0: Successfully fetched all artifacts from ${this.totalShardCount - 1} other shard(s).`);
+        } else {
+            console.warn(`Shard 0: Warning - Some artifacts may still be missing after ${maxRetries} attempts.`);
         }
     }
 

package/dist/src/SlackReporter.js

@@ -70,21 +70,58 @@ class SlackReporter {
             this.log(message);
             return;
         }
-        const resultSummary = await this.resultsParser.getParsedResults();
         // SHARDING SUPPORT - Stop slack messages for non-zero index shard(s)
-        // Support both explicit MATRIX_SHARD/MATRIX_INDEX and workflow-style 1-based indexing
-        // If MATRIX_SHARD is not set, shard 0 will still send reports (handles single shard case)
-        const currentShardIndex = process.env.MATRIX_SHARD !== undefined 
+        // Only shard 0 (0-based) should post when aggregating results from multiple shards
+        
+        // Detect shard index from multiple sources (GitHub Actions compatible)
+        let currentShardIndex = process.env.MATRIX_SHARD !== undefined 
             ? process.env.MATRIX_SHARD 
             : (process.env.MATRIX_INDEX !== undefined ? process.env.MATRIX_INDEX : undefined);
         
-        // Only skip if explicitly set and not 0 (or '0')
-        // This allows shard 0 to send reports even when MATRIX_SHARD is not set
-        if (currentShardIndex !== undefined && currentShardIndex !== '0' && currentShardIndex !== 0) {
-            const totalShards = process.env.MATRIX_COUNT || '1';
+        // Get shard info from ResultsParser which might have detected it
+        const parserShardIndex = this.resultsParser.shardIndex;
+        const parserTotalShards = this.resultsParser.totalShardCount;
+        
+        // Use parser values if env vars not set
+        const totalShards = process.env.MATRIX_COUNT ? parseInt(process.env.MATRIX_COUNT, 10) : parserTotalShards;
+        
+        // If we still don't have shard index, try to infer from parser
+        if (currentShardIndex === undefined && parserShardIndex !== undefined) {
+            currentShardIndex = parserShardIndex.toString();
+        }
+        
+        // Convert to number for comparison (default to 0 if undefined)
+        const shardIndexNum = currentShardIndex !== undefined ? parseInt(currentShardIndex, 10) : 0;
+        
+        // CRITICAL: For GitHub Actions with multiple shards, only shard 0 should post
+        // If we're in CI with multiple shards and don't know which shard we are, block all posts
+        // This prevents duplicate messages when MATRIX_SHARD/MATRIX_INDEX is not set
+        if (process.env.CI && totalShards > 1 && currentShardIndex === undefined) {
+            this.log(`❌ GitHub Actions detected with ${totalShards} shards but shard index not set (MATRIX_SHARD/MATRIX_INDEX missing).`);
+            this.log(`❌ Skipping Slack report to prevent duplicate messages from all shards.`);
+            this.log(`💡 Fix: Add these env vars to your workflow:`);
+            this.log(`💡   env:`);
+            this.log(`💡     MATRIX_SHARD: $\{\{ strategy.job-index \}\}  # 0-based (only shard 0 posts)`);
+            this.log(`💡     MATRIX_COUNT: $\{\{ strategy.job-total \}\}`);
+            return;
+        }
+        
+        // Only allow shard 0 (0-based) to post when there are multiple shards
+        // This ensures only one shard posts the aggregated report
+        if (totalShards > 1 && shardIndexNum !== 0) {
             this.log(`❌ Stopping reporter for non-zero index shard ${currentShardIndex} of ${totalShards}`);
+            this.log(`ℹ️ Only shard 0 will post the aggregated report.`);
             return;
         }
+        
+        // Single shard - always allow posting
+        if (totalShards === 1) {
+            this.log(`ℹ️ Single shard detected. Posting Slack report.`);
+        } else {
+            this.log(`ℹ️ Multiple shards detected (${totalShards}). Shard ${currentShardIndex} will post aggregated report.`);
+        }
+        
+        const resultSummary = await this.resultsParser.getParsedResults();
         resultSummary.meta = this.meta;
         const maxRetry = Math.max(...resultSummary.tests.map((o) => o.retry));
         if (this.sendResults === 'on-failure'

package/package.json

@@ -31,7 +31,7 @@
     "lint-fix": "npx eslint . --ext .ts --fix"
   },
   "name": "playwright-slack-report-burak",
-  "version": "3.0.0",
+  "version": "3.0.3",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "author": "Burak B. <burak.boluk@hotmail.com>",