playwright-slack-report-burak 3.0.19 → 3.0.20

package/dist/src/ResultsParser.js

@@ -44,23 +44,11 @@ class ResultsParser {
             ? 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
-    isOneBasedIndexing = false;
     constructor(options = { separateFlakyTests: false }) {
         this.result = [];
         this.separateFlakyTests = options.separateFlakyTests;
         // Log package version
         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'}`);
-        console.log(`🔍   Calculated shardIndex: ${this.shardIndex}`);
-        console.log(`🔍   Calculated totalShardCount: ${this.totalShardCount}`);
     }
     /**
      * Update shard information after detection (e.g., from GitHub API)
@@ -70,7 +58,6 @@ class ResultsParser {
     updateShardInfo(shardIndex, totalShardCount) {
         this.shardIndex = shardIndex;
         this.totalShardCount = totalShardCount;
-        console.log(`🔍 [ResultsParser] Updated shard info: shardIndex=${this.shardIndex}, totalShardCount=${this.totalShardCount}`);
     }
     async getParsedResults() {
       const summary = {
@@ -127,27 +114,17 @@ 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 === 1 && this.totalShardCount > 1) {
-          console.log(`🔍 [ResultsParser] Shard 1 detected with ${this.totalShardCount} total shards - will fetch artifacts from other shards`);
           if (process.env.CI) {
-            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('🔍 [ResultsParser] Waiting for all artifacts to exist...');
               await new Promise(resolve => setTimeout(resolve, 1000)); // Wait for 1 second
             }
           }
-        } else {
-          console.log(`🔍 [ResultsParser] Not shard 1 (current: ${this.shardIndex}) or single shard (total: ${this.totalShardCount}) - skipping artifact fetch`);
         }
 
         if (this.shardIndex === 1 && this.allNodeSummaryFilesExist() && this.allBlobZipsExist()) {
-            console.log(`🔍 [ResultsParser] Shard 1: All artifacts available, merging results from all shards...`);
             // Merge all node summaries into the final summary
             // Loop from 1 to totalShardCount (1-based indexing)
             for (let i = 1; i <= this.totalShardCount; i++) {
@@ -448,21 +425,6 @@ class ResultsParser {
         await this.fetchArtifactFromGitHubActions(i, file, filePath);
     }
 
-    // Helper to extract shard index from artifact name
-    extractShardIndexFromArtifactName(artifactName, i) {
-        // Try to extract number from artifact name (e.g., html-report-1 -> 1)
-        const match = artifactName.match(/-(\d+)$/);
-        if (match) {
-            const artifactShard = parseInt(match[1], 10);
-            // Artifacts use 1-based indexing, and we now use 1-based internally
-            // html-report-1 (workflow) -> shard 1 (internal)
-            // html-report-2 (workflow) -> shard 2 (internal)
-            return artifactShard;
-        }
-        // Fallback to the provided index
-        return i;
-    }
-
     async fetchArtifactFromGitHubActions(i, file, filePath) {
         const githubToken = process.env.SAFETYWINGTEST_GITHUB_TOKEN;
         const repo = process.env.GITHUB_REPOSITORY;
@@ -563,10 +525,7 @@ class ResultsParser {
                         const extractPath = path.join(path.dirname(filePath), `temp-artifact-${i}`);
                         zip.extractAllTo(extractPath, true);
                         
-                        // Extract shard index from artifact name to handle files named with index 0
-                        const artifactShardIndex = this.extractShardIndexFromArtifactName(artifact.name, i);
-                        
-                        // Look for the file recursively, also try with shard index from artifact name
+                        // Look for the file recursively
                         const searchInDir = (dir, targetFile, alternateTargetFile = null) => {
                             const entries = fs.readdirSync(dir, { withFileTypes: true });
                             for (const entry of entries) {

package/package.json

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