playwright-slack-report-burak 3.0.4 → 3.0.5

package/dist/src/ResultsParser.js

@@ -34,6 +34,13 @@ class ResultsParser {
     constructor(options = { separateFlakyTests: false }) {
         this.result = [];
         this.separateFlakyTests = options.separateFlakyTests;
+        // Log shard detection in ResultsParser
+        console.log('🔍 [ResultsParser] Initialized with shard detection:');
+        console.log(`🔍   MATRIX_SHARD env: ${process.env.MATRIX_SHARD !== undefined ? `"${process.env.MATRIX_SHARD}"` : 'undefined'}`);
+        console.log(`🔍   MATRIX_INDEX env: ${process.env.MATRIX_INDEX !== undefined ? `"${process.env.MATRIX_INDEX}"` : 'undefined'}`);
+        console.log(`🔍   MATRIX_COUNT env: ${process.env.MATRIX_COUNT !== undefined ? `"${process.env.MATRIX_COUNT}"` : 'undefined'}`);
+        console.log(`🔍   Calculated shardIndex: ${this.shardIndex}`);
+        console.log(`🔍   Calculated totalShardCount: ${this.totalShardCount}`);
     }
     async getParsedResults() {
       const summary = {
@@ -90,19 +97,27 @@ class ResultsParser {
         // Create the file
         fs.writeFileSync(nodeSummaryFile, JSON.stringify(nodeSummary, null, 2));
 
+        console.log(`🔍 [ResultsParser] Current shard: ${this.shardIndex}, Total shards: ${this.totalShardCount}`);
+        console.log(`🔍 [ResultsParser] Created node summary file: ${nodeSummaryFile}`);
+
         if (this.shardIndex === 0 && this.totalShardCount > 1) {
+          console.log(`🔍 [ResultsParser] Shard 0 detected with ${this.totalShardCount} total shards - will fetch artifacts from other shards`);
           if (process.env.CI) {
-            console.log('Fetching all artifacts from GitHub Actions...');
+            console.log('🔍 [ResultsParser] CI environment detected - fetching all artifacts from GitHub Actions...');
             await this.fetchAllArtifacts();
           } else {
+            console.log('🔍 [ResultsParser] Local environment - waiting for artifacts to appear...');
             while (!this.allNodeSummaryFilesExist() || !this.allBlobZipsExist()) {
-              console.log('Waiting for all both to exist...');
+              console.log('🔍 [ResultsParser] Waiting for all artifacts to exist...');
               await new Promise(resolve => setTimeout(resolve, 1000)); // Wait for 1 second
             }
           }
+        } else {
+          console.log(`🔍 [ResultsParser] Not shard 0 (current: ${this.shardIndex}) or single shard (total: ${this.totalShardCount}) - skipping artifact fetch`);
         }
 
         if (this.shardIndex === 0 && this.allNodeSummaryFilesExist() && this.allBlobZipsExist()) {
+            console.log(`🔍 [ResultsParser] Shard 0: All artifacts available, merging results from all shards...`);
             // Merge all node summaries into the final summary
             for (let i = 0; i < this.totalShardCount; i++) {
                 const nodeSummaryFile = path.join(summariesDir, `node_summary_${i}.json`);

package/dist/src/SlackReporter.js

@@ -73,19 +73,58 @@ class SlackReporter {
         // SHARDING SUPPORT - Stop slack messages for non-zero index shard(s)
         // Only shard 0 (0-based) should post when aggregating results from multiple shards
         
+        this.log('🔍 [Shard Detection] Starting shard detection logic...');
+        
+        // Log all relevant environment variables
+        this.log(`🔍 [Shard Detection] Environment variables:`);
+        this.log(`🔍   CI: ${process.env.CI !== undefined ? `"${process.env.CI}"` : 'undefined'}`);
+        this.log(`🔍   MATRIX_SHARD: ${process.env.MATRIX_SHARD !== undefined ? `"${process.env.MATRIX_SHARD}"` : 'undefined'}`);
+        this.log(`🔍   MATRIX_INDEX: ${process.env.MATRIX_INDEX !== undefined ? `"${process.env.MATRIX_INDEX}"` : 'undefined'}`);
+        this.log(`🔍   MATRIX_COUNT: ${process.env.MATRIX_COUNT !== undefined ? `"${process.env.MATRIX_COUNT}"` : 'undefined'}`);
+        
         // Detect shard index from multiple sources (GitHub Actions compatible)
-        let currentShardIndex = process.env.strategy.job-total !== undefined 
-            ? process.env.strategy.job-total 
-            : undefined;
+        // Note: In GitHub Actions workflow, you need to set these env vars:
+        //   env:
+        //     MATRIX_SHARD: ${{ strategy.job-index }}  # 0-based
+        //     MATRIX_COUNT: ${{ strategy.job-total }}
+        let currentShardIndex = process.env.MATRIX_SHARD !== undefined 
+            ? process.env.MATRIX_SHARD 
+            : (process.env.MATRIX_INDEX !== undefined ? process.env.MATRIX_INDEX : undefined);
+        
+        this.log(`🔍 [Shard Detection] Initial currentShardIndex: ${currentShardIndex !== undefined ? `"${currentShardIndex}"` : 'undefined'}`);
+        
+        // Get shard info from ResultsParser which might have detected it
+        const parserShardIndex = this.resultsParser.shardIndex;
+        const parserTotalShards = this.resultsParser.totalShardCount;
+        
+        this.log(`🔍 [Shard Detection] ResultsParser values:`);
+        this.log(`🔍   parserShardIndex: ${parserShardIndex !== undefined ? parserShardIndex : 'undefined'}`);
+        this.log(`🔍   parserTotalShards: ${parserTotalShards !== undefined ? parserTotalShards : 'undefined'}`);
+        
+        // Use parser values if env vars not set
+        const totalShards = process.env.MATRIX_COUNT ? parseInt(process.env.MATRIX_COUNT, 10) : parserTotalShards;
+        
+        this.log(`🔍 [Shard Detection] Calculated totalShards: ${totalShards}`);
+        
+        // If we still don't have shard index, try to infer from parser
+        if (currentShardIndex === undefined && parserShardIndex !== undefined) {
+            currentShardIndex = parserShardIndex.toString();
+            this.log(`🔍 [Shard Detection] Using parserShardIndex as fallback: "${currentShardIndex}"`);
+        }
         
         // Convert to number for comparison (default to 0 if undefined)
         const shardIndexNum = currentShardIndex !== undefined ? parseInt(currentShardIndex, 10) : 0;
         
+        this.log(`🔍 [Shard Detection] Final values:`);
+        this.log(`🔍   currentShardIndex (string): ${currentShardIndex !== undefined ? `"${currentShardIndex}"` : 'undefined'}`);
+        this.log(`🔍   shardIndexNum (number): ${shardIndexNum}`);
+        this.log(`🔍   totalShards: ${totalShards}`);
+        
         // 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(`❌ [Shard Detection] BLOCKING: CI environment 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:`);
@@ -97,6 +136,7 @@ class SlackReporter {
         // 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(`❌ [Shard Detection] BLOCKING: Non-zero shard detected (shard ${currentShardIndex} of ${totalShards})`);
             this.log(`❌ Stopping reporter for non-zero index shard ${currentShardIndex} of ${totalShards}`);
             this.log(`ℹ️ Only shard 0 will post the aggregated report.`);
             return;
@@ -104,9 +144,9 @@ class SlackReporter {
         
         // Single shard - always allow posting
         if (totalShards === 1) {
-            this.log(`ℹ️ Single shard detected. Posting Slack report.`);
+            this.log(`✅ [Shard Detection] ALLOWING: Single shard detected. Posting Slack report.`);
         } else {
-            this.log(`ℹ️ Multiple shards detected (${totalShards}). Shard ${currentShardIndex} will post aggregated report.`);
+            this.log(`✅ [Shard Detection] ALLOWING: Multiple shards detected (${totalShards}). Shard ${currentShardIndex} (index ${shardIndexNum}) will post aggregated report.`);
         }
         
         const resultSummary = await this.resultsParser.getParsedResults();

package/package.json

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