@arghajit/playwright-pulse-report 0.2.2 → 0.2.3

package/README.md

@@ -7,53 +7,7 @@ _The ultimate Playwright reporter — Interactive dashboard with historical tren
 
 ## ![Features](https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images/features.svg)
 
-## 📸 Screenshots
-
-### 🖥️ Desktop View
-
-<div align="center" style="display: flex; gap: 20px; justify-content: center; flex-wrap: wrap;">
-  <a href="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//playwright-pulse-static-report-desktop.html.png" target="_blank"> <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//playwright-pulse-static-report-desktop.html.png" alt="Dashboard Overview" width="300"/>
-   <p align="center"><strong>Dashboard Overview</strong></p>
-  </a>
-  <a href="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-run-desktop.png" target="_blank"> <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-run-desktop.png" alt="Test Details" width="300"/>
-   <p align="center"><strong>Test Details</strong>
-   </p>
-  </a>
-  <a href="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-error-desktop.png" target="_blank"> <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-run-desktop.png" alt="Test Failure Details" width="300"/>
-   <p align="center"><strong>Test Failure Details</strong>
-   </p>
-  </a>
-  <a href="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-trends-desktop.png" target="_blank"> <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Test-trends-desktop.png" alt="Filter View" width="300"/>
-  <p align="center"><strong>Test Trends</strong></p>
-  </a>
-</div>
-
-### 📱 Mobile View
-
-<div align="center" style="display: flex; gap: 20px; justify-content: center; flex-wrap: wrap;">
-
-  <a href="https://postimg.cc/CzJBLR5N" target="_blank">
-    <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//playwright-pulse-static-report-Dashboard.html.png" alt="Mobile Dashboard Overview" width="300"/>
-    <p align="center"><strong>Dashboard Overview</strong></p>
-  </a>
-
-  <a href="https://postimg.cc/G8YTczT8" target="_blank">
-    <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//playwright-pulse-static-report_Test-results.html.png" alt="Test Details" width="300"/>
-    <p align="center"><strong>Test Details</strong></p>
-  </a>
-
-  <a href="https://postimg.cc/G8YTczT8" target="_blank">
-    <img src="https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//playwright-pulse-static-report-Trends.html.png" alt="Test Trends" width="300"/>
-    <p align="center"><strong>Test Trends</strong></p>
-  </a>
-
-</div>
-
-### Email Report Example
-
-[![Email Report Template](https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Email-report-mobile-template.jpeg)](https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//Email-report-mobile-template.jpeg)
-
-[![Email Report](https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//pulse-email-summary.html.png)](https://ocpaxmghzmfbuhxzxzae.supabase.co/storage/v1/object/public/images//pulse-email-summary.html.png)
+## **Documentation**: [Pulse Report](https://playwright-pulse-report.netlify.app/)
 
 ## Available Scripts
 
@@ -91,10 +45,6 @@ Run with `npm run <command>`
 
 ```bash
 npm install @arghajit/playwright-pulse-report@latest --save-dev
-# or
-yarn add @arghajit/playwright-pulse-report@latest --dev
-# or
-pnpm add @arghajit/playwright-pulse-report@latest --save-dev
 ```
 
 ### 2. Configure Playwright
@@ -161,8 +111,8 @@ npx generate-report
 1. Configure `.env`:
 
    ```bash
-   SENDER_EMAIL_1=recipient1@example.com
-   SENDER_EMAIL_2=recipient2@example.com
+   RECIPIENT_EMAIL_1=recipient1@example.com
+   RECIPIENT_EMAIL_2=recipient2@example.com
    # ... up to 5 recipients
    ```
 
@@ -300,7 +250,7 @@ npm run pulse-dashboard
 
 *(Run from project root containing `pulse-report/` directory)*  
 
-**NPM Package**: [pulse-dashboard](https://www.npmjs.com/package/pulse-dashboard)  
+**NPM Package**: [playwright-pulse-report](https://www.npmjs.com/package/@arghajit/playwright-pulse-report) 
 
 **Tech Stack**: Next.js, TypeScript, Tailwind CSS, Playwright  
 

package/dist/reporter/attachment-utils.js

@@ -45,11 +45,10 @@ const ATTACHMENTS_SUBDIR = "attachments"; // Consistent subdirectory name
  * @param config The reporter configuration options.
  */
 function attachFiles(testId, pwResult, pulseResult, config) {
-    const baseReportDir = config.outputDir || "pulse-report"; // Base output directory
-    // Ensure attachments are relative to the main outputDir
-    const attachmentsBaseDir = path.resolve(baseReportDir, ATTACHMENTS_SUBDIR); // Absolute path for FS operations
-    const attachmentsSubFolder = testId.replace(/[^a-zA-Z0-9_-]/g, "_"); // Sanitize testId for folder name
-    const testAttachmentsDir = path.join(attachmentsBaseDir, attachmentsSubFolder); // e.g., pulse-report/attachments/test_id_abc
+    const baseReportDir = config.outputDir || "pulse-report";
+    const attachmentsBaseDir = path.resolve(baseReportDir, ATTACHMENTS_SUBDIR);
+    const attachmentsSubFolder = testId.replace(/[^a-zA-Z0-9_-]/g, "_");
+    const testAttachmentsDir = path.join(attachmentsBaseDir, attachmentsSubFolder);
     try {
         if (!fs.existsSync(testAttachmentsDir)) {
             fs.mkdirSync(testAttachmentsDir, { recursive: true });
@@ -57,53 +56,49 @@ function attachFiles(testId, pwResult, pulseResult, config) {
     }
     catch (error) {
         console.error(`Pulse Reporter: Failed to create attachments directory: ${testAttachmentsDir}`, error);
-        return; // Stop processing if directory creation fails
+        return;
     }
     if (!pwResult.attachments)
         return;
-    const { base64Images } = config; // Get base64 embedding option
-    pulseResult.screenshots = []; // Initialize screenshots array
+    const { base64Images } = config;
+    // --- MODIFICATION: Initialize all attachment arrays to prevent errors ---
+    pulseResult.screenshots = [];
+    pulseResult.videoPath = [];
+    pulseResult.attachments = [];
     pwResult.attachments.forEach((attachment) => {
         const { contentType, name, path: attachmentPath, body } = attachment;
-        // Skip attachments without path or body
         if (!attachmentPath && !body) {
             console.warn(`Pulse Reporter: Attachment "${name}" for test ${testId} has no path or body. Skipping.`);
             return;
         }
-        // Determine filename
-        const safeName = name.replace(/[^a-zA-Z0-9_.-]/g, "_"); // Sanitize original name
+        const safeName = name.replace(/[^a-zA-Z0-9_.-]/g, "_");
         const extension = attachmentPath
             ? path.extname(attachmentPath)
             : `.${getFileExtension(contentType)}`;
         const baseFilename = attachmentPath
             ? path.basename(attachmentPath, extension)
             : safeName;
-        // Ensure unique filename within the test's attachment folder
         const fileName = `${baseFilename}_${Date.now()}${extension}`;
-        // Relative path for storing in JSON (relative to baseReportDir)
         const relativePath = path.join(ATTACHMENTS_SUBDIR, attachmentsSubFolder, fileName);
-        // Full path for file system operations
         const fullPath = path.join(testAttachmentsDir, fileName);
         if (contentType === null || contentType === void 0 ? void 0 : contentType.startsWith("image/")) {
-            // Handle all image types consistently
             handleImage(attachmentPath, body, base64Images, fullPath, relativePath, pulseResult, name);
         }
         else if (name === "video" || (contentType === null || contentType === void 0 ? void 0 : contentType.startsWith("video/"))) {
-            handleAttachment(attachmentPath, body, fullPath, relativePath, "videoPath", pulseResult);
+            handleAttachment(attachmentPath, body, fullPath, relativePath, "videoPath", pulseResult, attachment);
         }
         else if (name === "trace" || contentType === "application/zip") {
-            // Trace files are zips
-            handleAttachment(attachmentPath, body, fullPath, relativePath, "tracePath", pulseResult);
+            handleAttachment(attachmentPath, body, fullPath, relativePath, "tracePath", pulseResult, attachment);
         }
         else {
-            // Handle other generic attachments if needed (e.g., log files)
-            // console.log(`Pulse Reporter: Processing generic attachment "${name}" (Type: ${contentType}) for test ${testId}`);
-            // handleAttachment(attachmentPath, body, fullPath, relativePath, 'otherAttachments', pulseResult); // Example for storing other types
+            // --- MODIFICATION: Enabled handling for all other file types ---
+            handleAttachment(attachmentPath, body, fullPath, relativePath, "attachments", pulseResult, attachment);
         }
     });
 }
 /**
  * Handles image attachments, either embedding as base64 or copying the file.
+ * (This function is unchanged)
  */
 function handleImage(attachmentPath, body, base64Embed, fullPath, relativePath, pulseResult, attachmentName) {
     let screenshotData = undefined;
@@ -123,14 +118,10 @@ function handleImage(attachmentPath, body, base64Embed, fullPath, relativePath,
         }
     }
     else if (body) {
-        // Always embed if only body is available
         screenshotData = `data:image/${getFileExtension(attachmentName)};base64,${body.toString("base64")}`;
         if (!base64Embed) {
-            // Optionally save the buffer to a file even if embedding is off,
-            // but the primary representation will be base64.
             try {
                 fs.writeFileSync(fullPath, body);
-                // console.log(`Pulse Reporter: Saved screenshot buffer to ${fullPath}`);
             }
             catch (error) {
                 console.error(`Pulse Reporter: Failed to save screenshot buffer: ${fullPath}. Error: ${error.message}`);
@@ -147,21 +138,36 @@ function handleImage(attachmentPath, body, base64Embed, fullPath, relativePath,
 /**
  * Handles non-image attachments by copying the file or writing the buffer.
  */
-function handleAttachment(attachmentPath, body, fullPath, relativePath, resultKey, // Add more keys if needed
-pulseResult) {
+function handleAttachment(attachmentPath, body, fullPath, relativePath, resultKey, // MODIFIED: Added 'attachments'
+pulseResult, originalAttachment // MODIFIED: Pass original attachment
+) {
+    var _a, _b;
     try {
         if (attachmentPath) {
             fs.copyFileSync(attachmentPath, fullPath);
-            pulseResult[resultKey] = relativePath;
         }
         else if (body) {
             fs.writeFileSync(fullPath, body);
-            pulseResult[resultKey] = relativePath; // Store relative path even if from buffer
+        }
+        // --- MODIFICATION: Logic to handle different properties correctly ---
+        switch (resultKey) {
+            case "videoPath":
+                (_a = pulseResult.videoPath) === null || _a === void 0 ? void 0 : _a.push(relativePath);
+                break;
+            case "tracePath":
+                pulseResult.tracePath = relativePath;
+                break;
+            case "attachments":
+                (_b = pulseResult.attachments) === null || _b === void 0 ? void 0 : _b.push({
+                    name: originalAttachment.name,
+                    path: relativePath,
+                    contentType: originalAttachment.contentType,
+                });
+                break;
         }
     }
     catch (error) {
         console.error(`Pulse Reporter: Failed to copy/write attachment to ${fullPath}. Error: ${error.message}`);
-        // Don't set the path in pulseResult if saving failed
     }
 }
 /**
@@ -172,8 +178,7 @@ pulseResult) {
 function getFileExtension(contentType) {
     var _a;
     if (!contentType)
-        return "bin"; // Default binary extension
-    // More robust mapping
+        return "bin";
     const extensions = {
         "image/png": "png",
         "image/jpeg": "jpg",
@@ -182,9 +187,12 @@ function getFileExtension(contentType) {
         "image/svg+xml": "svg",
         "video/webm": "webm",
         "video/mp4": "mp4",
-        "application/zip": "zip", // For traces
+        "application/zip": "zip",
         "text/plain": "txt",
         "application/json": "json",
+        "text/html": "html",
+        "application/pdf": "pdf",
+        "text/csv": "csv",
     };
     return (extensions[contentType.toLowerCase()] ||
         ((_a = contentType.split("/")[1]) === null || _a === void 0 ? void 0 : _a.split("+")[0]) ||

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

@@ -1,5 +1,4 @@
 "use strict";
-// input_file_0.ts
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
     var desc = Object.getOwnPropertyDescriptor(m, k);
@@ -38,8 +37,7 @@ exports.PlaywrightPulseReporter = void 0;
 const fs = __importStar(require("fs/promises"));
 const path = __importStar(require("path"));
 const crypto_1 = require("crypto");
-const attachment_utils_1 = require("./attachment-utils"); // Use relative path
-const ua_parser_js_1 = require("ua-parser-js"); // Added UAParser import
+const ua_parser_js_1 = require("ua-parser-js");
 const os = __importStar(require("os"));
 const convertStatus = (status, testCase) => {
     if ((testCase === null || testCase === void 0 ? void 0 : testCase.expectedStatus) === "failed") {
@@ -111,8 +109,8 @@ class PlaywrightPulseReporter {
     }
     getBrowserDetails(test) {
         var _a, _b, _c, _d;
-        const project = (_a = test.parent) === null || _a === void 0 ? void 0 : _a.project(); // project() can return undefined if not in a project context
-        const projectConfig = project === null || project === void 0 ? void 0 : project.use; // This is where options like userAgent, defaultBrowserType are
+        const project = (_a = test.parent) === null || _a === void 0 ? void 0 : _a.project();
+        const projectConfig = project === null || project === void 0 ? void 0 : project.use;
         const userAgent = projectConfig === null || projectConfig === void 0 ? void 0 : projectConfig.userAgent;
         const configuredBrowserType = (_b = projectConfig === null || projectConfig === void 0 ? void 0 : projectConfig.browserName) === null || _b === void 0 ? void 0 : _b.toLowerCase();
         const parser = new ua_parser_js_1.UAParser(userAgent);
@@ -120,20 +118,18 @@ class PlaywrightPulseReporter {
         let browserName = result.browser.name;
         const browserVersion = result.browser.version
             ? ` v${result.browser.version.split(".")[0]}`
-            : ""; // Major version
+            : "";
         const osName = result.os.name ? ` on ${result.os.name}` : "";
         const osVersion = result.os.version
             ? ` ${result.os.version.split(".")[0]}`
-            : ""; // Major version
-        const deviceType = result.device.type; // "mobile", "tablet", etc.
+            : "";
+        const deviceType = result.device.type;
         let finalString;
-        // If UAParser couldn't determine browser name, fallback to configured type
         if (browserName === undefined) {
             browserName = configuredBrowserType;
             finalString = `${browserName}`;
         }
         else {
-            // Specific refinements for mobile based on parsed OS and device type
             if (deviceType === "mobile" || deviceType === "tablet") {
                 if ((_c = result.os.name) === null || _c === void 0 ? void 0 : _c.toLowerCase().includes("android")) {
                     if (browserName.toLowerCase().includes("chrome"))
@@ -144,10 +140,10 @@ class PlaywrightPulseReporter {
                         browserName = "Android WebView";
                     else if (browserName &&
                         !browserName.toLowerCase().includes("mobile")) {
-                        // Keep it as is, e.g. "Samsung Browser" is specific enough
+                        // Keep it as is
                     }
                     else {
-                        browserName = "Android Browser"; // default for android if not specific
+                        browserName = "Android Browser";
                     }
                 }
                 else if ((_d = result.os.name) === null || _d === void 0 ? void 0 : _d.toLowerCase().includes("ios")) {
@@ -178,10 +174,9 @@ class PlaywrightPulseReporter {
         if (step.location) {
             codeLocation = `${path.relative(this.config.rootDir, step.location.file)}:${step.location.line}:${step.location.column}`;
         }
-        let stepTitle = step.title;
         return {
             id: `${testId}_step_${startTime.toISOString()}-${duration}-${(0, crypto_1.randomUUID)()}`,
-            title: stepTitle,
+            title: step.title,
             status: stepStatus,
             duration: duration,
             startTime: startTime,
@@ -200,21 +195,16 @@ class PlaywrightPulseReporter {
         };
     }
     async onTestEnd(test, result) {
-        var _a, _b, _c, _d, _e, _f, _g, _h;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
         const project = (_a = test.parent) === null || _a === void 0 ? void 0 : _a.project();
         const browserDetails = this.getBrowserDetails(test);
         const testStatus = convertStatus(result.status, test);
         const startTime = new Date(result.startTime);
         const endTime = new Date(startTime.getTime() + result.duration);
-        const testIdForFiles = test.id ||
-            `${test
-                .titlePath()
-                .join("_")
-                .replace(/[^a-zA-Z0-9]/g, "_")}_${startTime.getTime()}`;
         const processAllSteps = async (steps) => {
             let processed = [];
             for (const step of steps) {
-                const processedStep = await this.processStep(step, testIdForFiles, browserDetails, test);
+                const processedStep = await this.processStep(step, test.id, browserDetails, test);
                 processed.push(processedStep);
                 if (step.steps && step.steps.length > 0) {
                     processedStep.steps = await processAllSteps(step.steps);
@@ -232,39 +222,14 @@ class PlaywrightPulseReporter {
         catch (e) {
             console.warn(`Pulse Reporter: Could not extract code snippet for ${test.title}`, e);
         }
-        const stdoutMessages = [];
-        if (result.stdout && result.stdout.length > 0) {
-            result.stdout.forEach((item) => {
-                stdoutMessages.push(typeof item === "string" ? item : item.toString());
-            });
-        }
-        const stderrMessages = [];
-        if (result.stderr && result.stderr.length > 0) {
-            result.stderr.forEach((item) => {
-                stderrMessages.push(typeof item === "string" ? item : item.toString());
-            });
-        }
-        const uniqueTestId = test.id;
-        // --- REFINED THIS SECTION for testData ---
+        const stdoutMessages = result.stdout.map((item) => typeof item === "string" ? item : item.toString());
+        const stderrMessages = result.stderr.map((item) => typeof item === "string" ? item : item.toString());
         const maxWorkers = this.config.workers;
-        let mappedWorkerId;
-        // First, check for the special case where a test is not assigned a worker (e.g., global setup failure).
-        if (result.workerIndex === -1) {
-            mappedWorkerId = -1; // Keep it as -1 to clearly identify this special case.
-        }
-        else if (maxWorkers && maxWorkers > 0) {
-            // If there's a valid worker, map it to the concurrency slot...
-            const zeroBasedId = result.workerIndex % maxWorkers;
-            // ...and then shift it to be 1-based (1 to n).
-            mappedWorkerId = zeroBasedId + 1;
-        }
-        else {
-            // Fallback for when maxWorkers is not defined: just use the original index (and shift to 1-based).
-            mappedWorkerId = result.workerIndex + 1;
-        }
+        let mappedWorkerId = result.workerIndex === -1
+            ? -1
+            : (result.workerIndex % (maxWorkers > 0 ? maxWorkers : 1)) + 1;
         const testSpecificData = {
             workerId: mappedWorkerId,
-            uniqueWorkerIndex: result.workerIndex, // We'll keep the original for diagnostics
             totalWorkers: maxWorkers,
             configFile: this.config.configFile,
             metadata: this.config.metadata
@@ -272,7 +237,7 @@ class PlaywrightPulseReporter {
                 : undefined,
         };
         const pulseResult = {
-            id: uniqueTestId,
+            id: test.id,
             runId: "TBD",
             name: test.titlePath().join(" > "),
             suiteName: (project === null || project === void 0 ? void 0 : project.name) || ((_e = this.config.projects[0]) === null || _e === void 0 ? void 0 : _e.name) || "Default Suite",
@@ -288,20 +253,56 @@ class PlaywrightPulseReporter {
             codeSnippet: codeSnippet,
             tags: test.tags.map((tag) => tag.startsWith("@") ? tag.substring(1) : tag),
             screenshots: [],
-            videoPath: undefined,
+            videoPath: [],
             tracePath: undefined,
+            attachments: [],
             stdout: stdoutMessages.length > 0 ? stdoutMessages : undefined,
             stderr: stderrMessages.length > 0 ? stderrMessages : undefined,
-            // --- UPDATED THESE LINES from testSpecificData ---
             ...testSpecificData,
         };
-        try {
-            (0, attachment_utils_1.attachFiles)(testIdForFiles, result, pulseResult, this.options);
-        }
-        catch (attachError) {
-            console.error(`Pulse Reporter: Error processing attachments for test ${pulseResult.name} (ID: ${testIdForFiles}): ${attachError.message}`);
+        // --- CORRECTED ATTACHMENT PROCESSING LOGIC ---
+        for (const [index, attachment] of result.attachments.entries()) {
+            if (!attachment.path)
+                continue;
+            try {
+                // Create a sanitized, unique folder name for this specific test
+                const testSubfolder = test.id.replace(/[^a-zA-Z0-9_-]/g, "_");
+                // Sanitize the original attachment name to create a safe filename
+                const safeAttachmentName = path
+                    .basename(attachment.path)
+                    .replace(/[^a-zA-Z0-9_.-]/g, "_");
+                // Create a unique filename to prevent collisions, especially in retries
+                const uniqueFileName = `${index}-${Date.now()}-${safeAttachmentName}`;
+                // This is the relative path that will be stored in the JSON report
+                const relativeDestPath = path.join(ATTACHMENTS_SUBDIR, testSubfolder, uniqueFileName);
+                // This is the absolute path used for the actual file system operation
+                const absoluteDestPath = path.join(this.outputDir, relativeDestPath);
+                // Ensure the unique, test-specific attachment directory exists
+                await this._ensureDirExists(path.dirname(absoluteDestPath));
+                await fs.copyFile(attachment.path, absoluteDestPath);
+                // Categorize the attachment based on its content type
+                if (attachment.contentType.startsWith("image/")) {
+                    (_j = pulseResult.screenshots) === null || _j === void 0 ? void 0 : _j.push(relativeDestPath);
+                }
+                else if (attachment.contentType.startsWith("video/")) {
+                    (_k = pulseResult.videoPath) === null || _k === void 0 ? void 0 : _k.push(relativeDestPath);
+                }
+                else if (attachment.name === "trace") {
+                    pulseResult.tracePath = relativeDestPath;
+                }
+                else {
+                    (_l = pulseResult.attachments) === null || _l === void 0 ? void 0 : _l.push({
+                        name: attachment.name, // The original, human-readable name
+                        path: relativeDestPath, // The safe, relative path for linking
+                        contentType: attachment.contentType,
+                    });
+                }
+            }
+            catch (err) {
+                console.error(`Pulse Reporter: Failed to process attachment "${attachment.name}" for test ${pulseResult.name}. Error: ${err.message}`);
+            }
         }
-        const existingTestIndex = this.results.findIndex((r) => r.id === uniqueTestId);
+        const existingTestIndex = this.results.findIndex((r) => r.id === test.id);
         if (existingTestIndex !== -1) {
             if (pulseResult.retries >= this.results[existingTestIndex].retries) {
                 this.results[existingTestIndex] = pulseResult;
@@ -323,10 +324,10 @@ class PlaywrightPulseReporter {
             host: os.hostname(),
             os: `${os.platform()} ${os.release()}`,
             cpu: {
-                model: os.cpus()[0] ? os.cpus()[0].model : "N/A", // Handle cases with no CPU info
+                model: os.cpus()[0] ? os.cpus()[0].model : "N/A",
                 cores: os.cpus().length,
             },
-            memory: `${(os.totalmem() / 1024 ** 3).toFixed(2)}GB`, // Total RAM in GB
+            memory: `${(os.totalmem() / 1024 ** 3).toFixed(2)}GB`,
             node: process.version,
             v8: process.versions.v8,
             cwd: process.cwd(),
@@ -418,15 +419,13 @@ class PlaywrightPulseReporter {
         }
     }
     async onEnd(result) {
-        var _a, _b, _c;
         if (this.shardIndex !== undefined) {
             await this._writeShardResults();
             return;
         }
         const runEndTime = Date.now();
         const duration = runEndTime - this.runStartTime;
-        const runId = `run-${this.runStartTime}-581d5ad8-ce75-4ca5-94a6-ed29c466c815`; // Need not to change
-        // --- CALLING _getEnvDetails HERE ---
+        const runId = `run-${this.runStartTime}-581d5ad8-ce75-4ca5-94a6-ed29c466c815`;
         const environmentDetails = this._getEnvDetails();
         const runData = {
             id: runId,
@@ -436,13 +435,11 @@ class PlaywrightPulseReporter {
             failed: 0,
             skipped: 0,
             duration,
-            // --- ADDED environmentDetails HERE ---
             environment: environmentDetails,
         };
-        let finalReport = undefined; // Initialize as undefined
+        let finalReport = undefined;
         if (this.isSharded) {
             finalReport = await this._mergeShardResults(runData);
-            // Ensured environment details are on the final merged runData if not already
             if (finalReport && finalReport.run && !finalReport.run.environment) {
                 finalReport.run.environment = environmentDetails;
             }
@@ -470,67 +467,8 @@ class PlaywrightPulseReporter {
         }
         if (!finalReport) {
             console.error("PlaywrightPulseReporter: CRITICAL - finalReport object was not generated. Cannot create summary.");
-            const errorSummary = `
-PlaywrightPulseReporter: Run Finished
------------------------------------------
-  Overall Status: ERROR (Report data missing)
-  Total Tests:    N/A
-  Passed:         N/A
-  Failed:         N/A
-  Skipped:        N/A
-  Duration:       N/A
------------------------------------------`;
-            if (this.printsToStdio()) {
-                console.log(errorSummary);
-            }
-            const errorReport = {
-                run: {
-                    id: runId,
-                    timestamp: new Date(this.runStartTime),
-                    totalTests: 0,
-                    passed: 0,
-                    failed: 0,
-                    skipped: 0,
-                    duration: duration,
-                    environment: environmentDetails,
-                },
-                results: [],
-                metadata: {
-                    generatedAt: new Date().toISOString(),
-                },
-            };
-            const finalOutputPathOnError = path.join(this.outputDir, this.baseOutputFile);
-            try {
-                await this._ensureDirExists(this.outputDir);
-                await fs.writeFile(finalOutputPathOnError, JSON.stringify(errorReport, null, 2));
-                console.warn(`PlaywrightPulseReporter: Wrote an error report to ${finalOutputPathOnError} as finalReport was missing.`);
-            }
-            catch (writeError) {
-                console.error(`PlaywrightPulseReporter: Failed to write error report: ${writeError.message}`);
-            }
             return;
         }
-        const reportRunData = finalReport.run;
-        const finalRunStatus = ((_a = reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.failed) !== null && _a !== void 0 ? _a : 0) > 0
-            ? "failed"
-            : ((_b = reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.totalTests) !== null && _b !== void 0 ? _b : 0) === 0 && result.status !== "passed"
-                ? result.status === "interrupted"
-                    ? "interrupted"
-                    : "no tests or error"
-                : "passed";
-        const summary = `
-PlaywrightPulseReporter: Run Finished
------------------------------------------
-  Overall Status: ${finalRunStatus.toUpperCase()}
-  Total Tests:    ${(reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.totalTests) || 0}
-  Passed:         ${reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.passed}
-  Failed:         ${reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.failed}
-  Skipped:        ${reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.skipped}
-  Duration:       ${(((_c = reportRunData === null || reportRunData === void 0 ? void 0 : reportRunData.duration) !== null && _c !== void 0 ? _c : 0) / 1000).toFixed(2)}s 
------------------------------------------`;
-        if (this.printsToStdio()) {
-            console.log(summary);
-        }
         const finalOutputPath = path.join(this.outputDir, this.baseOutputFile);
         try {
             await this._ensureDirExists(this.outputDir);

package/dist/types/index.d.ts

@@ -32,8 +32,13 @@ export interface TestResult {
     runId: string;
     browser: string;
     screenshots?: string[];
-    videoPath?: string;
+    videoPath?: string[];
     tracePath?: string;
+    attachments?: {
+        name: string;
+        path: string;
+        contentType: string;
+    }[];
     stdout?: string[];
     stderr?: string[];
     workerId?: number;

package/package.json

@@ -1,8 +1,9 @@
 {
   "name": "@arghajit/playwright-pulse-report",
   "author": "Arghajit Singha",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "A Playwright reporter and dashboard for visualizing test results.",
+  "homepage": "https://playwright-pulse-report.netlify.app/",
   "keywords": [
     "playwright",
     "reporter",
@@ -11,10 +12,13 @@
     "reporting",
     "nextjs",
     "playwright-pulse",
+    "playwright-pulse-report",
     "report",
     "email-report",
     "send-report",
-    "email"
+    "email",
+    "playwright-report",
+    "pulse"
   ],
   "main": "dist/reporter/index.js",
   "types": "dist/reporter/index.d.ts",
@@ -46,7 +50,8 @@
     "report:generate": "node ./scripts/generate-report.mjs",
     "report:merge": "node ./scripts/merge-pulse-report.js",
     "report:email": "node ./scripts/sendReport.mjs",
-    "report:minify": "node ./scripts/generate-email-report.mjs"
+    "report:minify": "node ./scripts/generate-email-report.mjs",
+    "generate-trend": "node ./scripts/generate-trend.mjs"
   },
   "dependencies": {
     "archiver": "^7.0.1",