playwright-slack-report-burak 3.0.30 → 3.0.32

package/dist/src/ResultsParser.js

@@ -697,16 +697,18 @@ class ResultsParser {
             
             // Create temporary config file to specify output folder
             tempConfigPath = path.join(summariesDir, 'merge-config.js');
+            const tempConfigAbsolutePath = path.resolve(tempConfigPath);
             // Escape backslashes and single quotes in path for JavaScript string
             const escapedPath = mergedBlobsAbsolutePath.replace(/\\/g, '/').replace(/'/g, "\\'");
             const configContent = `module.exports = {
   reporter: [['html', { outputFolder: '${escapedPath}' }]]
 };`;
             fs.writeFileSync(tempConfigPath, configContent);
-            console.log(`Created temporary config file: ${tempConfigPath}`);
+            console.log(`Created temporary config file: ${tempConfigAbsolutePath}`);
             console.log(`Config content: ${configContent}`);
             
-            const mergeCommand = `npx playwright merge-reports --reporter html --config "${tempConfigPath}" "${blobReportsRelativePath}"`;
+            // Use absolute path for config file to avoid path resolution issues
+            const mergeCommand = `npx playwright merge-reports --reporter html --config "${tempConfigAbsolutePath}" "${blobReportsRelativePath}"`;
             console.log(`Executing: ${mergeCommand}`);
             const mergeResult = await execAsync(mergeCommand, { cwd: summariesDir });
             console.log('Merge command output:', mergeResult.stdout);
@@ -733,12 +735,15 @@ class ResultsParser {
             execSync(`npx playwright merge-reports --reporter html -c ./playwright-report playwright-report`, { stdio: 'inherit' });
           } finally {
             // Clean up temporary config file
-            if (tempConfigPath && fs.existsSync(tempConfigPath)) {
-              try {
-                fs.unlinkSync(tempConfigPath);
-                console.log(`Cleaned up temporary config file: ${tempConfigPath}`);
-              } catch (cleanupError) {
-                console.warn(`Warning: Failed to clean up temp config file: ${cleanupError.message}`);
+            if (tempConfigPath) {
+              const configPathToDelete = fs.existsSync(tempConfigPath) ? tempConfigPath : path.resolve(tempConfigPath);
+              if (fs.existsSync(configPathToDelete)) {
+                try {
+                  fs.unlinkSync(configPathToDelete);
+                  console.log(`Cleaned up temporary config file: ${configPathToDelete}`);
+                } catch (cleanupError) {
+                  console.warn(`Warning: Failed to clean up temp config file: ${cleanupError.message}`);
+                }
               }
             }
           }

package/dist/src/SlackReporter.js

@@ -6,27 +6,6 @@ const webhook_1 = require("@slack/webhook");
 const ResultsParser_1 = require("./ResultsParser");
 const SlackClient_1 = require("./SlackClient");
 const SlackWebhookClient_1 = require("./SlackWebhookClient");
-let axios;
-try {
-    axios = require('axios');
-} catch (e) {
-    // axios optional for GitHub Actions API detection
-    axios = null;
-}
-
-// Get package version
-const fs = require('fs');
-const path = require('path');
-let packageVersion = 'unknown';
-try {
-    const packageJsonPath = path.join(__dirname, '../../package.json');
-    if (fs.existsSync(packageJsonPath)) {
-        const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8'));
-        packageVersion = packageJson.version || 'unknown';
-    }
-} catch (e) {
-    // Version detection failed, use default
-}
 
 class SlackReporter {
     customLayout;
@@ -46,11 +25,7 @@ class SlackReporter {
     suite;
     separateFlaky = false;
     logs = [];
-    fullConfig;
     onBegin(fullConfig, suite) {
-        // Log package version
-        console.log(`📦 [SlackReporter] playwright-slack-report-burak v${packageVersion}`);
-        this.fullConfig = fullConfig;
         this.suite = suite;
         this.logs = [];
         const slackReporterConfig = fullConfig.reporter.filter((f) => f[0].toLowerCase().includes('slackreporter'))[0][1];
@@ -96,52 +71,11 @@ class SlackReporter {
             this.log(message);
             return;
         }
-        // SHARDING SUPPORT - Stop slack messages for non-shard-1 shard(s)
-        // Only shard 1 should post when aggregating results from multiple shards
-        
-        // Get shard info from environment variables
-        const currentShardIndex = process.env.SHARD_INDEX;
-        const totalShards = process.env.TOTAL_SHARDS ? parseInt(process.env.TOTAL_SHARDS, 10) : 1;
-        
-        // CRITICAL: In CI, if we can't detect shard info, block posting
-        // This prevents duplicate messages when running with multiple shards
-        if (process.env.CI && currentShardIndex === undefined && totalShards > 1) {
-            this.log(`❌ [Shard Detection] BLOCKING: CI environment detected but SHARD_INDEX not set.`);
-            this.log(`❌   This prevents duplicate messages when running with multiple shards.`);
-            this.log(`❌ Skipping Slack report to prevent duplicate messages from all shards.`);
-            this.log(`💡 Fix: Add SHARD_INDEX env var to your workflow:`);
-            this.log(`💡   env:`);
-            this.log(`💡     SHARD_INDEX: $\{\{ strategy.job-index \}\}  # Only shard 1 should post`);
-            this.log(`💡     TOTAL_SHARDS: $\{\{ strategy.job-total \}\}`);
-            return;
-        }
-        
-        // Convert to number for comparison (default to 1 if undefined, since shard 1 is the aggregator)
-        const shardIndexNum = currentShardIndex !== undefined ? parseInt(currentShardIndex, 10) : 1;
-        
-        // CRITICAL: Update ResultsParser with detected shard values immediately after detection
-        // This ensures ResultsParser always has correct values, even if we block posting
-        this.resultsParser.updateShardInfo(shardIndexNum, totalShards);
-        
-        // CRITICAL: ALL shards must create their node_summary files, even if they don't post to Slack
-        // This must happen BEFORE the early return for non-shard-1 shards
         const resultSummary = await this.resultsParser.getParsedResults();
-        
-        // Only allow shard 1 to post when there are multiple shards
-        // This ensures only one shard posts the aggregated report
-        if (totalShards > 1 && shardIndexNum !== 1) {
-            this.log(`❌ [Shard Detection] BLOCKING: Non-shard-1 detected (shard ${shardIndexNum} of ${totalShards})`);
-            this.log(`❌ Stopping reporter for shard ${shardIndexNum} of ${totalShards}`);
-            this.log(`ℹ️ Only shard 1 will post the aggregated report.`);
-            this.log(`✅ Node summary file created for shard ${shardIndexNum}`);
-            return;
-        }
-        
-        // Single shard - always allow posting
-        if (totalShards === 1) {
-            this.log(`✅ [Shard Detection] ALLOWING: Single shard detected. Posting Slack report.`);
-        } else {
-            this.log(`✅ [Shard Detection] ALLOWING: Multiple shards detected (${totalShards}). Shard ${shardIndexNum} will post aggregated report.`);
+        // SHARDING SUPPORT - Stop slack messages for non-shard-1 shard(s)
+        if (process.env.SHARD_INDEX && process.env.SHARD_INDEX !== '1') {
+          this.log(`❌ Stopping reporter for non-shard-1 index ${process.env.SHARD_INDEX} of ${process.env.TOTAL_SHARDS}`);
+          return;
         }
         resultSummary.meta = this.meta;
         const maxRetry = Math.max(...resultSummary.tests.map((o) => o.retry));

package/package.json

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