@arghajit/dummy 0.3.39 → 0.3.40

package/dist/reporter/playwright-pulse-reporter.d.ts

@@ -44,7 +44,6 @@ export declare class PlaywrightPulseReporter implements Reporter {
      *
      * Cleaning up at `onBegin` time guarantees each run starts with a fresh slate.
      */
-    private _cleanupStaleRunReports;
     private _ensureDirExists;
     onEnd(result: FullResult): Promise<void>;
 }

package/dist/reporter/playwright-pulse-reporter.js

@@ -109,13 +109,6 @@ class PlaywrightPulseReporter {
                     await this._cleanupTemporaryFiles();
                 }
             }
-            // When not resetting on each run, clear stale individual run files
-            // from previous sessions. Without this, _mergeAllRunReports() reads
-            // ALL accumulated files and de-duplicates by test.id, which collapses
-            // results from different sessions into fewer entries than expected.
-            if (!this.resetOnEachRun) {
-                await this._cleanupStaleRunReports();
-            }
         })
             .catch((err) => console.error("Pulse Reporter: Error during initialization:", err));
     }
@@ -524,25 +517,6 @@ class PlaywrightPulseReporter {
      *
      * Cleaning up at `onBegin` time guarantees each run starts with a fresh slate.
      */
-    async _cleanupStaleRunReports() {
-        const pulseResultsDir = path.join(this.outputDir, this.individualReportsSubDir);
-        try {
-            const files = await fs.readdir(pulseResultsDir);
-            const staleFiles = files.filter((f) => f.startsWith("playwright-pulse-report-") && f.endsWith(".json"));
-            if (staleFiles.length > 0) {
-                await Promise.all(staleFiles.map((f) => fs.unlink(path.join(pulseResultsDir, f))));
-                if (this.printsToStdio()) {
-                    console.log(`PlaywrightPulseReporter: Cleaned up ${staleFiles.length} stale run report(s) from ${pulseResultsDir}`);
-                }
-            }
-        }
-        catch (error) {
-            // ENOENT simply means no previous runs exist — that's fine
-            if ((error === null || error === void 0 ? void 0 : error.code) !== "ENOENT") {
-                console.warn("Pulse Reporter: Warning during cleanup of stale run reports:", error.message);
-            }
-        }
-    }
     async _ensureDirExists(dirPath) {
         try {
             await fs.mkdir(dirPath, { recursive: true });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arghajit/dummy",
   "author": "Arghajit Singha",
-  "version": "0.3.39",
+  "version": "0.3.40",
   "description": "A Playwright reporter and dashboard for visualizing test results.",
   "homepage": "https://arghajit47.github.io/playwright-pulse/",
   "repository": {
@@ -38,7 +38,7 @@
     "logo": "node scripts/terminal-logo.mjs",
     "generate-pulse-report": "scripts/generate-static-report.mjs",
     "generate-report": "scripts/generate-report.mjs",
-    "merge-pulse-report": "scripts/merge-pulse-report.js",
+    "merge-pulse-report": "scripts/merge-pulse-report.mjs",
     "send-email": "scripts/sendReport.mjs",
     "generate-email-report": "scripts/generate-email-report.mjs",
     "generate-trend": "scripts/generate-trend.mjs",
@@ -56,7 +56,7 @@
     "prepublishOnly": "npm run build:reporter",
     "report:static": "node ./scripts/generate-static-report.mjs",
     "report:generate": "node ./scripts/generate-report.mjs",
-    "report:merge": "node ./scripts/merge-pulse-report.js",
+    "report:merge": "node ./scripts/merge-pulse-report.mjs",
     "report:email": "node ./scripts/sendReport.mjs",
     "report:minify": "node ./scripts/generate-email-report.mjs",
     "generate-trend": "node ./scripts/generate-trend.mjs"

package/scripts/generate-email-report.mjs

@@ -2,7 +2,7 @@
 
 import * as fs from "fs/promises";
 import path from "path";
-import { getOutputDir } from "./config-reader.mjs";
+import { getReporterConfig } from "./config-reader.mjs";
 import { animate } from "./terminal-logo.mjs";
 import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
@@ -25,7 +25,6 @@ try {
 }
 
 const DEFAULT_OUTPUT_DIR = "pulse-report";
-const DEFAULT_JSON_FILE = "playwright-pulse-report.json";
 const MINIFIED_HTML_FILE = "pulse-email-summary.html"; // New minified report
 
 const args = process.argv.slice(2);
@@ -758,9 +757,12 @@ async function main() {
     await animate();
   }
   
-  const outputDir = await getOutputDir(customOutputDir);
+  const config = await getReporterConfig(customOutputDir);
+  const outputDir = config.outputDir;
+  const outputFile = config.outputFile;
+  
   await mergeSequentialReportsIfNeeded(outputDir);
-  const reportJsonPath = path.resolve(outputDir, DEFAULT_JSON_FILE);
+  const reportJsonPath = path.resolve(outputDir, outputFile);
   const minifiedReportHtmlPath = path.resolve(outputDir, MINIFIED_HTML_FILE); // Path for the new minified HTML
 
   console.log(chalk.blue(`Generating email report...`));

package/scripts/generate-report.mjs

@@ -5,7 +5,7 @@ import { readFileSync, existsSync as fsExistsSync } from "fs";
 import path from "path";
 import { fork } from "child_process";
 import { fileURLToPath } from "url";
-import { getOutputDir } from "./config-reader.mjs";
+import { getReporterConfig } from "./config-reader.mjs";
 import { animate } from "./terminal-logo.mjs";
 import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
@@ -53,7 +53,6 @@ try {
 }
 // Default configuration
 const DEFAULT_OUTPUT_DIR = "pulse-report";
-const DEFAULT_JSON_FILE = "playwright-pulse-report.json";
 const DEFAULT_HTML_FILE = "playwright-pulse-report.html";
 // Helper functions
 export function ansiToHtml(text) {
@@ -5298,9 +5297,12 @@ async function main() {
     "generate-trend.mjs", // Keeping the filename as per your request
   );
 
-  const outputDir = await getOutputDir(customOutputDir);
+  const config = await getReporterConfig(customOutputDir);
+  const outputDir = config.outputDir;
+  const outputFile = config.outputFile;
+  
   await mergeSequentialReportsIfNeeded(outputDir);
-  const reportJsonPath = path.resolve(outputDir, DEFAULT_JSON_FILE); // Current run's main JSON
+  const reportJsonPath = path.resolve(outputDir, outputFile); // Current run's main JSON
   const reportHtmlPath = path.resolve(outputDir, DEFAULT_HTML_FILE);
 
   const historyDir = path.join(outputDir, "history"); // Directory for historical JSON files

package/scripts/generate-static-report.mjs

@@ -5,7 +5,7 @@ import { readFileSync, existsSync as fsExistsSync } from "fs";
 import path from "path";
 import { fork } from "child_process";
 import { fileURLToPath } from "url";
-import { getOutputDir } from "./config-reader.mjs";
+import { getReporterConfig } from "./config-reader.mjs";
 import { animate } from "./terminal-logo.mjs";
 import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
@@ -56,18 +56,8 @@ try {
   };
 }
 
-/**
- * @constant {string} DEFAULT_OUTPUT_DIR
- * The default directory where the report will be generated.
- */
 const DEFAULT_OUTPUT_DIR = "pulse-report";
 
-/**
- * @constant {string} DEFAULT_JSON_FILE
- * The default name for the JSON file containing the test data.
- */
-const DEFAULT_JSON_FILE = "playwright-pulse-report.json";
-
 /**
  * @constant {string} DEFAULT_HTML_FILE
  * The default name for the generated HTML report file.
@@ -7163,9 +7153,12 @@ async function main() {
     "generate-trend.mjs", // Keeping the filename as per your request
   );
 
-  const outputDir = await getOutputDir(customOutputDir);
+  const config = await getReporterConfig(customOutputDir);
+  const outputDir = config.outputDir;
+  const outputFile = config.outputFile;
+
   await mergeSequentialReportsIfNeeded(outputDir);
-  const reportJsonPath = path.resolve(outputDir, DEFAULT_JSON_FILE); // Current run's main JSON
+  const reportJsonPath = path.resolve(outputDir, outputFile); // Current run's main JSON
   const reportHtmlPath = path.resolve(outputDir, DEFAULT_HTML_FILE);
 
   const historyDir = path.join(outputDir, "history"); // Directory for historical JSON files

package/scripts/generate-trend.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import * as fs from "fs/promises";
 import path from "path";
-import { getOutputDir } from "./config-reader.mjs";
+import { getReporterConfig } from "./config-reader.mjs";
 import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
 // Use dynamic import for chalk as it's ESM only for prettier console logs
@@ -19,7 +19,6 @@ try {
 }
 
 const DEFAULT_OUTPUT_DIR = "pulse-report";
-const CURRENT_RUN_JSON_FILE = "playwright-pulse-report.json"; // Source of the current run data
 const HISTORY_SUBDIR = "history"; // Subdirectory for historical JSON files
 const HISTORY_FILE_PREFIX = "trend-";
 const MAX_HISTORY_FILES = 15; // Store last 15 runs
@@ -34,9 +33,12 @@ for (let i = 0; i < args.length; i++) {
 }
 
 async function archiveCurrentRunData() {
-  const outputDir = await getOutputDir(customOutputDir);
+  const config = await getReporterConfig(customOutputDir);
+  const outputDir = config.outputDir;
+  const outputFile = config.outputFile;
+  
   await mergeSequentialReportsIfNeeded(outputDir);
-  const currentRunJsonPath = path.join(outputDir, CURRENT_RUN_JSON_FILE);
+  const currentRunJsonPath = path.join(outputDir, outputFile);
   const historyDir = path.join(outputDir, HISTORY_SUBDIR);
 
   try {

package/scripts/{merge-pulse-report.js → merge-pulse-report.mjs}

@@ -1,7 +1,10 @@
 #!/usr/bin/env node
 
-const fs = require("fs");
-const path = require("path");
+import * as fs from "fs";
+import path from "path";
+import { getReporterConfig } from "./config-reader.mjs";
+import { animate } from "./terminal-logo.mjs";
+import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
 const args = process.argv.slice(2);
 let customOutputDir = null;
@@ -13,33 +16,23 @@ for (let i = 0; i < args.length; i++) {
   }
 }
 
-const OUTPUT_FILE = "playwright-pulse-report.json";
-
 /**
- * Securely resolves the report directory.
- * Prevents Path Traversal by ensuring the output directory
- * is contained within the current working directory.
+ * Securely resolves the report directory and config.
  */
-async function getReportDir() {
+async function getFullConfig() {
+  const config = await getReporterConfig(customOutputDir);
+  
   if (customOutputDir) {
     const resolvedPath = path.resolve(process.cwd(), customOutputDir);
-
     if (!resolvedPath.startsWith(process.cwd())) {
       console.error(
         "⛔ Security Error: Custom output directory must be within the current project root.",
       );
       process.exit(1);
     }
-
-    return resolvedPath;
   }
 
-  try {
-    const { getOutputDir } = await import("./config-reader.mjs");
-    return await getOutputDir();
-  } catch (error) {
-    return path.resolve(process.cwd(), "pulse-report");
-  }
+  return config;
 }
 
 /**
@@ -54,14 +47,14 @@ function getShardDirectories(dir) {
 
   return fs
     .readdirSync(dir, { withFileTypes: true })
-    .filter((dirent) => dirent.isDirectory() && dirent.name !== "attachments")
+    .filter((dirent) => dirent.isDirectory() && dirent.name !== "attachments" && dirent.name !== "pulse-results")
     .map((dirent) => path.join(dir, dirent.name));
 }
 
 /**
  * Merges JSON reports from all shard directories.
  */
-function mergeReports(shardDirs) {
+function mergeReports(shardDirs, outputFile) {
   let combinedRun = {
     totalTests: 0,
     passed: 0,
@@ -76,10 +69,10 @@ function mergeReports(shardDirs) {
   let allEnvironments = [];
 
   for (const shardDir of shardDirs) {
-    const jsonPath = path.join(shardDir, OUTPUT_FILE);
+    const jsonPath = path.join(shardDir, outputFile);
 
     if (!fs.existsSync(jsonPath)) {
-      console.warn(`  Warning: No ${OUTPUT_FILE} found in ${path.basename(shardDir)}`);
+      console.warn(`  Warning: No ${outputFile} found in ${path.basename(shardDir)}`);
       continue;
     }
 
@@ -181,15 +174,17 @@ function cleanupShardDirectories(shardDirs) {
 
 // Main execution
 (async () => {
-  const { animate } = await import("./terminal-logo.mjs");
-  const { mergeSequentialReportsIfNeeded } = await import("./merge-sequential-reports.mjs");
   await animate();
   
-  const REPORT_DIR = await getReportDir();
+  const config = await getFullConfig();
+  const REPORT_DIR = config.outputDir;
+  const OUTPUT_FILE = config.outputFile;
+
   await mergeSequentialReportsIfNeeded(REPORT_DIR);
 
   console.log(`\n🔄 Playwright Pulse - Merge Reports (Sharding Mode)\n`);
   console.log(`  Report directory: ${REPORT_DIR}`);
+  console.log(`  Output file: ${OUTPUT_FILE}`);
   if (customOutputDir) {
     console.log(`  (from CLI argument)`);
   } else {
@@ -203,7 +198,7 @@ function cleanupShardDirectories(shardDirs) {
   if (shardDirs.length === 0) {
     console.log("❌ No shard directories found.");
     console.log(
-      "   Expected structure: <report-dir>/<shard-folder>/playwright-pulse-report.json",
+      `   Expected structure: <report-dir>/<shard-folder>/${OUTPUT_FILE}`,
     );
     process.exit(0);
   }
@@ -216,7 +211,7 @@ function cleanupShardDirectories(shardDirs) {
 
   // 2. Merge JSON Reports
   console.log(`🔀 Merging reports...`);
-  const merged = mergeReports(shardDirs);
+  const merged = mergeReports(shardDirs, OUTPUT_FILE);
   console.log(`  ✓ Merged ${shardDirs.length} report(s)`);
   console.log();
 

package/scripts/sendReport.mjs

@@ -11,7 +11,7 @@ import { fileURLToPath } from "url";
 import { animate } from "./terminal-logo.mjs";
 import { fork } from "child_process";
 import "dotenv/config";
-import { getOutputDir } from "./config-reader.mjs";
+import { getReporterConfig } from "./config-reader.mjs";
 import { mergeSequentialReportsIfNeeded } from "./merge-sequential-reports.mjs";
 
 let chalk;
@@ -43,8 +43,10 @@ for (let i = 0; i < args.length; i++) {
 let fetch;
 let projectName;
 
-function getUUID(reportDir) {
-  const reportPath = path.join(reportDir, "playwright-pulse-report.json");
+async function getUUID(reportDir) {
+  const config = await getReporterConfig(customOutputDir);
+  const outputFile = config.outputFile;
+  const reportPath = path.join(reportDir, outputFile);
   console.log("Report path:", reportPath);
 
   if (!fsExistsSync(reportPath)) {
@@ -66,8 +68,10 @@ function formatDuration(ms) {
   return `${(ms / 3600000).toFixed(1)}h`;
 }
 
-const getPulseReportSummary = (reportDir) => {
-  const reportPath = path.join(reportDir, "playwright-pulse-report.json");
+const getPulseReportSummary = async (reportDir) => {
+  const config = await getReporterConfig(customOutputDir);
+  const outputFile = config.outputFile;
+  const reportPath = path.join(reportDir, outputFile);
 
   if (!fsExistsSync(reportPath)) {
     throw new Error("Pulse report file not found.");
@@ -287,8 +291,9 @@ const sendEmail = async (credentials, reportDir) => {
 
   try {
     console.log("Starting the sendEmail function...");
-    const reportData = getPulseReportSummary(reportDir);
-    const htmlContent = generateHtmlTable(reportData);
+    const uuid = await getUUID(reportDir);
+    const summary = await getPulseReportSummary(reportDir);
+    const htmlContent = generateHtmlTable(summary);
 
     const recipients = [
       process.env.RECIPIENT_EMAIL_1 || "",