playwright-slack-report-burak 3.0.16 → 3.0.18

package/dist/src/ResultsParser.js

@@ -35,10 +35,14 @@ class ResultsParser {
     separateFlakyTests;
     // Handle both 0-based and 1-based shard indexing
     // If MATRIX_SHARD is not set, try to infer from workflow (1-based) or default to 0
-    totalShardCount = process.env.MATRIX_COUNT ? parseInt(process.env.MATRIX_COUNT, 10) : 1;
-    shardIndex = process.env.MATRIX_SHARD !== undefined 
-        ? parseInt(process.env.MATRIX_SHARD, 10) 
-        : (process.env.MATRIX_INDEX !== undefined ? parseInt(process.env.MATRIX_INDEX, 10) : 0);
+    totalShardCount = process.env.TOTAL_SHARDS 
+        ? parseInt(process.env.TOTAL_SHARDS, 10) 
+        : (process.env.MATRIX_COUNT ? parseInt(process.env.MATRIX_COUNT, 10) : 1);
+    shardIndex = process.env.SHARD_INDEX !== undefined
+        ? parseInt(process.env.SHARD_INDEX, 10)
+        : (process.env.MATRIX_SHARD !== undefined 
+            ? parseInt(process.env.MATRIX_SHARD, 10) 
+            : (process.env.MATRIX_INDEX !== undefined ? parseInt(process.env.MATRIX_INDEX, 10) : 0));
     
     // Determine if we're using 1-based indexing (workflow-style) or 0-based
     // This is detected by checking if artifacts exist with 1-based naming
@@ -50,6 +54,8 @@ class ResultsParser {
         console.log(`📦 [ResultsParser] playwright-slack-report-burak v${packageVersion}`);
         // Log shard detection in ResultsParser
         console.log('🔍 [ResultsParser] Initialized with shard detection:');
+        console.log(`🔍   SHARD_INDEX env: ${process.env.SHARD_INDEX !== undefined ? `"${process.env.SHARD_INDEX}"` : 'undefined'}`);
+        console.log(`🔍   TOTAL_SHARDS env: ${process.env.TOTAL_SHARDS !== undefined ? `"${process.env.TOTAL_SHARDS}"` : 'undefined'}`);
         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'}`);

package/dist/src/SlackReporter.js

@@ -341,14 +341,50 @@ class SlackReporter {
         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'}`);
         
+        // Check for Playwright shard environment variables (Playwright may set these)
+        this.log(`🔍   PLAYWRIGHT_SHARD: ${process.env.PLAYWRIGHT_SHARD !== undefined ? `"${process.env.PLAYWRIGHT_SHARD}"` : 'undefined'}`);
+        this.log(`🔍   SHARD_INDEX: ${process.env.SHARD_INDEX !== undefined ? `"${process.env.SHARD_INDEX}"` : 'undefined'}`);
+        this.log(`🔍   TOTAL_SHARDS: ${process.env.TOTAL_SHARDS !== undefined ? `"${process.env.TOTAL_SHARDS}"` : 'undefined'}`);
+        
         // Check Playwright's shard config (highest priority for local runs)
         let playwrightShardInfo = null;
-        if (this.fullConfig && this.fullConfig.shard) {
-            playwrightShardInfo = {
-                current: this.fullConfig.shard.current,
-                total: this.fullConfig.shard.total
-            };
-            this.log(`🔍 [Shard Detection] Found Playwright shard config: ${playwrightShardInfo.current}/${playwrightShardInfo.total}`);
+        if (this.fullConfig) {
+            // Debug: log config structure
+            this.log(`🔍 [Shard Detection] Checking Playwright config for shard info...`);
+            this.log(`🔍   fullConfig.shard: ${this.fullConfig.shard ? JSON.stringify(this.fullConfig.shard) : 'undefined'}`);
+            
+            if (this.fullConfig.shard) {
+                // Playwright shard can be either { current, total } or undefined
+                if (this.fullConfig.shard.current !== undefined && this.fullConfig.shard.total !== undefined) {
+                    playwrightShardInfo = {
+                        current: this.fullConfig.shard.current,
+                        total: this.fullConfig.shard.total
+                    };
+                    this.log(`🔍 [Shard Detection] Found Playwright shard config: ${playwrightShardInfo.current}/${playwrightShardInfo.total}`);
+                } else {
+                    this.log(`🔍 [Shard Detection] Playwright shard config exists but missing current/total properties`);
+                }
+            } else {
+                this.log(`🔍 [Shard Detection] No Playwright shard config found in fullConfig`);
+            }
+        } else {
+            this.log(`🔍 [Shard Detection] fullConfig not available`);
+        }
+        
+        // Also check if shard info is in process.argv (command line)
+        // This is a fallback if config doesn't have it but --shard was passed
+        if (!playwrightShardInfo && process.argv) {
+            const shardArg = process.argv.find(arg => arg && arg.includes('--shard'));
+            if (shardArg) {
+                const shardMatch = shardArg.match(/--shard[=:]?(\d+)\/(\d+)/);
+                if (shardMatch) {
+                    playwrightShardInfo = {
+                        current: parseInt(shardMatch[1], 10),
+                        total: parseInt(shardMatch[2], 10)
+                    };
+                    this.log(`🔍 [Shard Detection] Found shard in command line args: ${playwrightShardInfo.current}/${playwrightShardInfo.total}`);
+                }
+            }
         }
         
         // Step 1: Try GitHub Actions API detection (only if actually in GitHub Actions)
@@ -366,6 +402,9 @@ class SlackReporter {
             // Playwright uses 1-based indexing, convert to 0-based
             currentShardIndex = (playwrightShardInfo.current - 1).toString();
             this.log(`🔍 [Shard Detection] Using Playwright shard config (converted to 0-based): "${currentShardIndex}"`);
+        } else if (process.env.SHARD_INDEX !== undefined) {
+            currentShardIndex = process.env.SHARD_INDEX;
+            this.log(`🔍 [Shard Detection] Using SHARD_INDEX env var: "${currentShardIndex}"`);
         } else if (process.env.MATRIX_SHARD !== undefined) {
             currentShardIndex = process.env.MATRIX_SHARD;
             this.log(`🔍 [Shard Detection] Using MATRIX_SHARD env var: "${currentShardIndex}"`);
@@ -384,6 +423,9 @@ class SlackReporter {
         if (playwrightShardInfo) {
             totalShards = playwrightShardInfo.total;
             this.log(`🔍 [Shard Detection] Using Playwright shard config totalShards: ${totalShards}`);
+        } else if (process.env.TOTAL_SHARDS !== undefined) {
+            totalShards = parseInt(process.env.TOTAL_SHARDS, 10);
+            this.log(`🔍 [Shard Detection] Using TOTAL_SHARDS env var: ${totalShards}`);
         } else if (process.env.MATRIX_COUNT !== undefined) {
             totalShards = parseInt(process.env.MATRIX_COUNT, 10);
             this.log(`🔍 [Shard Detection] Using MATRIX_COUNT env var: ${totalShards}`);

package/package.json

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