@arghajit/playwright-pulse-report 0.2.0 → 0.2.2

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

@@ -39,9 +39,11 @@ 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 os = __importStar(require("os"));
 const convertStatus = (status, testCase) => {
     if ((testCase === null || testCase === void 0 ? void 0 : testCase.expectedStatus) === "failed") {
-        return status === "failed" ? "failed" : "failed";
+        return "failed";
     }
     if ((testCase === null || testCase === void 0 ? void 0 : testCase.expectedStatus) === "skipped") {
         return "skipped";
@@ -94,24 +96,77 @@ class PlaywrightPulseReporter {
             : undefined;
         this._ensureDirExists(this.outputDir)
             .then(() => {
-            if (this.shardIndex === undefined) {
+            if (this.shardIndex === undefined || this.shardIndex === 0) {
                 console.log(`PlaywrightPulseReporter: Starting test run with ${suite.allTests().length} tests${this.isSharded ? ` across ${totalShards} shards` : ""}. Pulse outputting to ${this.outputDir}`);
-                return this._cleanupTemporaryFiles();
+                if (this.shardIndex === undefined ||
+                    (this.isSharded && this.shardIndex === 0)) {
+                    return this._cleanupTemporaryFiles();
+                }
             }
         })
             .catch((err) => console.error("Pulse Reporter: Error during initialization:", err));
     }
     onTestBegin(test) {
-        // console.log(`Starting test: ${test.title}`);
+        console.log(`Starting test: ${test.title}`);
+    }
+    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 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);
+        const result = parser.getResult();
+        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.
+        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"))
+                        browserName = "Chrome Mobile";
+                    else if (browserName.toLowerCase().includes("firefox"))
+                        browserName = "Firefox Mobile";
+                    else if (result.engine.name === "Blink" && !result.browser.name)
+                        browserName = "Android WebView";
+                    else if (browserName &&
+                        !browserName.toLowerCase().includes("mobile")) {
+                        // Keep it as is, e.g. "Samsung Browser" is specific enough
+                    }
+                    else {
+                        browserName = "Android Browser"; // default for android if not specific
+                    }
+                }
+                else if ((_d = result.os.name) === null || _d === void 0 ? void 0 : _d.toLowerCase().includes("ios")) {
+                    browserName = "Mobile Safari";
+                }
+            }
+            else if (browserName === "Electron") {
+                browserName = "Electron App";
+            }
+            finalString = `${browserName}${browserVersion}${osName}${osVersion}`;
+        }
+        return finalString.trim();
     }
-    async processStep(step, testId, browserName, // Changed from browserName for clarity
-    testCase) {
+    async processStep(step, testId, browserDetails, testCase) {
         var _a, _b, _c, _d;
         let stepStatus = "passed";
         let errorMessage = ((_a = step.error) === null || _a === void 0 ? void 0 : _a.message) || undefined;
         if ((_c = (_b = step.error) === null || _b === void 0 ? void 0 : _b.message) === null || _c === void 0 ? void 0 : _c.startsWith("Test is skipped:")) {
             stepStatus = "skipped";
-            errorMessage = "Info: Test is skipped:";
         }
         else {
             stepStatus = convertStatus(step.error ? "failed" : "passed", testCase);
@@ -124,28 +179,6 @@ class PlaywrightPulseReporter {
             codeLocation = `${path.relative(this.config.rootDir, step.location.file)}:${step.location.line}:${step.location.column}`;
         }
         let stepTitle = step.title;
-        // This logic had a 'status' variable that was not defined in this scope.
-        // Assuming it meant to check 'stepStatus' or 'testCase.expectedStatus' related to step.error.
-        // Corrected to reflect comparison with testCase if step.category is 'test'.
-        if (step.category === "test" && testCase) {
-            // If a test step (not a hook) resulted in an error, but the test was expected to fail,
-            // this specific logic might need refinement based on how you want to report step errors
-            // within a test that is expected to fail.
-            // The current convertStatus handles the overall testCase expectedStatus.
-            // For step-specific error messages when testCase.expectedStatus === 'failed':
-            if (testCase.expectedStatus === "failed") {
-                if (step.error) {
-                    // If the step itself has an error
-                    // errorMessage is already set from step.error.message
-                }
-                else {
-                    // If a step within an expected-to-fail test passes, it's usually not an error for the step itself.
-                }
-            }
-            else if (testCase.expectedStatus === "skipped") {
-                // errorMessage is already set if step.error.message started with "Test is skipped:"
-            }
-        }
         return {
             id: `${testId}_step_${startTime.toISOString()}-${duration}-${(0, crypto_1.randomUUID)()}`,
             title: stepTitle,
@@ -153,7 +186,7 @@ class PlaywrightPulseReporter {
             duration: duration,
             startTime: startTime,
             endTime: endTime,
-            browser: browserName,
+            browser: browserDetails,
             errorMessage: errorMessage,
             stackTrace: ((_d = step.error) === null || _d === void 0 ? void 0 : _d.stack) || undefined,
             codeLocation: codeLocation || undefined,
@@ -167,12 +200,9 @@ class PlaywrightPulseReporter {
         };
     }
     async onTestEnd(test, result) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
+        var _a, _b, _c, _d, _e, _f, _g, _h;
         const project = (_a = test.parent) === null || _a === void 0 ? void 0 : _a.project();
-        // Use project.name for a user-friendly display name
-        const browserName = ((_b = project === null || project === void 0 ? void 0 : project.use) === null || _b === void 0 ? void 0 : _b.defaultBrowserType) || "unknown";
-        // If you need the engine name (chromium, firefox, webkit)
-        // const browserEngineName = project?.use?.browserName || "unknown_engine";
+        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);
@@ -181,14 +211,10 @@ class PlaywrightPulseReporter {
                 .titlePath()
                 .join("_")
                 .replace(/[^a-zA-Z0-9]/g, "_")}_${startTime.getTime()}`;
-        const processAllSteps = async (steps
-        // parentTestStatus parameter was not used, removed for now.
-        // If needed for inherited status logic for steps, it can be re-added.
-        ) => {
+        const processAllSteps = async (steps) => {
             let processed = [];
             for (const step of steps) {
-                const processedStep = await this.processStep(step, testIdForFiles, browserName, // Pass display name
-                test);
+                const processedStep = await this.processStep(step, testIdForFiles, browserDetails, test);
                 processed.push(processedStep);
                 if (step.steps && step.steps.length > 0) {
                     processedStep.steps = await processAllSteps(step.steps);
@@ -198,7 +224,7 @@ class PlaywrightPulseReporter {
         };
         let codeSnippet = undefined;
         try {
-            if (((_c = test.location) === null || _c === void 0 ? void 0 : _c.file) && ((_d = test.location) === null || _d === void 0 ? void 0 : _d.line) && ((_e = test.location) === null || _e === void 0 ? void 0 : _e.column)) {
+            if (((_b = test.location) === null || _b === void 0 ? void 0 : _b.file) && ((_c = test.location) === null || _c === void 0 ? void 0 : _c.line) && ((_d = test.location) === null || _d === void 0 ? void 0 : _d.column)) {
                 const relativePath = path.relative(this.config.rootDir, test.location.file);
                 codeSnippet = `Test defined at: ${relativePath}:${test.location.line}:${test.location.column}`;
             }
@@ -206,69 +232,84 @@ class PlaywrightPulseReporter {
         catch (e) {
             console.warn(`Pulse Reporter: Could not extract code snippet for ${test.title}`, e);
         }
-        // --- Capture stdout and stderr ---
         const stdoutMessages = [];
         if (result.stdout && result.stdout.length > 0) {
             result.stdout.forEach((item) => {
-                if (typeof item === "string") {
-                    stdoutMessages.push(item);
-                }
-                else {
-                    // If item is not a string, Playwright's typings indicate it's a Buffer (or Buffer-like).
-                    // We must call toString() on it.
-                    // The 'item' here is typed as 'Buffer' from the 'else' branch of '(string | Buffer)[]'
-                    stdoutMessages.push(item.toString());
-                }
+                stdoutMessages.push(typeof item === "string" ? item : item.toString());
             });
         }
         const stderrMessages = [];
         if (result.stderr && result.stderr.length > 0) {
             result.stderr.forEach((item) => {
-                if (typeof item === "string") {
-                    stderrMessages.push(item);
-                }
-                else {
-                    // If item is not a string, Playwright's typings indicate it's a Buffer (or Buffer-like).
-                    // We must call toString() on it.
-                    stderrMessages.push(item.toString());
-                }
+                stderrMessages.push(typeof item === "string" ? item : item.toString());
             });
         }
-        // --- End capture stdout and stderr ---
+        const uniqueTestId = test.id;
+        // --- REFINED THIS SECTION for testData ---
+        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;
+        }
+        const testSpecificData = {
+            workerId: mappedWorkerId,
+            uniqueWorkerIndex: result.workerIndex, // We'll keep the original for diagnostics
+            totalWorkers: maxWorkers,
+            configFile: this.config.configFile,
+            metadata: this.config.metadata
+                ? JSON.stringify(this.config.metadata)
+                : undefined,
+        };
         const pulseResult = {
-            id: test.id || `${test.title}-${startTime.toISOString()}-${(0, crypto_1.randomUUID)()}`,
+            id: uniqueTestId,
             runId: "TBD",
             name: test.titlePath().join(" > "),
-            // Use project.name for suiteName if desired, or fallback
-            suiteName: (project === null || project === void 0 ? void 0 : project.name) || ((_f = this.config.projects[0]) === null || _f === void 0 ? void 0 : _f.name) || "Default Suite",
+            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",
             status: testStatus,
             duration: result.duration,
             startTime: startTime,
             endTime: endTime,
-            browser: browserName, // Use the user-friendly project name
+            browser: browserDetails,
             retries: result.retry,
-            steps: ((_g = result.steps) === null || _g === void 0 ? void 0 : _g.length) ? await processAllSteps(result.steps) : [],
-            errorMessage: (_h = result.error) === null || _h === void 0 ? void 0 : _h.message,
-            stackTrace: (_j = result.error) === null || _j === void 0 ? void 0 : _j.stack,
+            steps: ((_f = result.steps) === null || _f === void 0 ? void 0 : _f.length) ? await processAllSteps(result.steps) : [],
+            errorMessage: (_g = result.error) === null || _g === void 0 ? void 0 : _g.message,
+            stackTrace: (_h = result.error) === null || _h === void 0 ? void 0 : _h.stack,
             codeSnippet: codeSnippet,
             tags: test.tags.map((tag) => tag.startsWith("@") ? tag.substring(1) : tag),
-            screenshots: [], // Will be populated by attachFiles
+            screenshots: [],
             videoPath: undefined,
             tracePath: undefined,
-            // videoPath and tracePath might be deprecated if using the array versions above
-            // Depending on attachFiles implementation
-            // Add the captured console messages
             stdout: stdoutMessages.length > 0 ? stdoutMessages : undefined,
             stderr: stderrMessages.length > 0 ? stderrMessages : undefined,
+            // --- UPDATED THESE LINES from testSpecificData ---
+            ...testSpecificData,
         };
         try {
-            // Pass this.options which should contain the resolved outputDir
             (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}`);
         }
-        this.results.push(pulseResult);
+        const existingTestIndex = this.results.findIndex((r) => r.id === uniqueTestId);
+        if (existingTestIndex !== -1) {
+            if (pulseResult.retries >= this.results[existingTestIndex].retries) {
+                this.results[existingTestIndex] = pulseResult;
+            }
+        }
+        else {
+            this.results.push(pulseResult);
+        }
     }
     onError(error) {
         var _a;
@@ -277,9 +318,22 @@ class PlaywrightPulseReporter {
             console.error(error.stack);
         }
     }
+    _getEnvDetails() {
+        return {
+            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
+                cores: os.cpus().length,
+            },
+            memory: `${(os.totalmem() / 1024 ** 3).toFixed(2)}GB`, // Total RAM in GB
+            node: process.version,
+            v8: process.versions.v8,
+            cwd: process.cwd(),
+        };
+    }
     async _writeShardResults() {
         if (this.shardIndex === undefined) {
-            // console.warn("Pulse Reporter: _writeShardResults called unexpectedly in main process. Skipping.");
             return;
         }
         const tempFilePath = path.join(this.outputDir, `${TEMP_SHARD_FILE_PREFIX}${this.shardIndex}.json`);
@@ -291,29 +345,38 @@ class PlaywrightPulseReporter {
         }
     }
     async _mergeShardResults(finalRunData) {
-        let allResults = [];
+        let allShardProcessedResults = [];
         const totalShards = this.config.shard ? this.config.shard.total : 1;
         for (let i = 0; i < totalShards; i++) {
             const tempFilePath = path.join(this.outputDir, `${TEMP_SHARD_FILE_PREFIX}${i}.json`);
             try {
                 const content = await fs.readFile(tempFilePath, "utf-8");
                 const shardResults = JSON.parse(content);
-                shardResults.forEach((r) => (r.runId = finalRunData.id));
-                allResults = allResults.concat(shardResults);
+                allShardProcessedResults =
+                    allShardProcessedResults.concat(shardResults);
             }
             catch (error) {
                 if ((error === null || error === void 0 ? void 0 : error.code) === "ENOENT") {
-                    console.warn(`Pulse Reporter: Shard results file not found: ${tempFilePath}.`);
+                    console.warn(`Pulse Reporter: Shard results file not found: ${tempFilePath}. This might be normal if a shard had no tests or failed early.`);
                 }
                 else {
                     console.error(`Pulse Reporter: Could not read/parse results from shard ${i} (${tempFilePath}). Error:`, error);
                 }
             }
         }
-        finalRunData.passed = allResults.filter((r) => r.status === "passed").length;
-        finalRunData.failed = allResults.filter((r) => r.status === "failed").length;
-        finalRunData.skipped = allResults.filter((r) => r.status === "skipped").length;
-        finalRunData.totalTests = allResults.length;
+        let finalUniqueResultsMap = new Map();
+        for (const result of allShardProcessedResults) {
+            const existing = finalUniqueResultsMap.get(result.id);
+            if (!existing || result.retries >= existing.retries) {
+                finalUniqueResultsMap.set(result.id, result);
+            }
+        }
+        const finalResultsList = Array.from(finalUniqueResultsMap.values());
+        finalResultsList.forEach((r) => (r.runId = finalRunData.id));
+        finalRunData.passed = finalResultsList.filter((r) => r.status === "passed").length;
+        finalRunData.failed = finalResultsList.filter((r) => r.status === "failed").length;
+        finalRunData.skipped = finalResultsList.filter((r) => r.status === "skipped").length;
+        finalRunData.totalTests = finalResultsList.length;
         const reviveDates = (key, value) => {
             const isoDateRegex = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(\.\d+)?Z$/;
             if (typeof value === "string" && isoDateRegex.test(value)) {
@@ -322,10 +385,10 @@ class PlaywrightPulseReporter {
             }
             return value;
         };
-        const finalParsedResults = JSON.parse(JSON.stringify(allResults), reviveDates);
+        const properlyTypedResults = JSON.parse(JSON.stringify(finalResultsList), reviveDates);
         return {
             run: finalRunData,
-            results: finalParsedResults,
+            results: properlyTypedResults,
             metadata: { generatedAt: new Date().toISOString() },
         };
     }
@@ -339,12 +402,11 @@ class PlaywrightPulseReporter {
         }
         catch (error) {
             if ((error === null || error === void 0 ? void 0 : error.code) !== "ENOENT") {
-                console.error("Pulse Reporter: Error cleaning up temporary files:", error);
+                console.warn("Pulse Reporter: Warning during cleanup of temporary files:", error.message);
             }
         }
     }
     async _ensureDirExists(dirPath) {
-        // Removed 'clean' parameter as it was unused
         try {
             await fs.mkdir(dirPath, { recursive: true });
         }
@@ -356,15 +418,16 @@ class PlaywrightPulseReporter {
         }
     }
     async onEnd(result) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
+        var _a, _b, _c;
         if (this.shardIndex !== undefined) {
             await this._writeShardResults();
             return;
         }
         const runEndTime = Date.now();
         const duration = runEndTime - this.runStartTime;
-        // Consider making the UUID part truly random for each run if this ID needs to be globally unique over time
-        const runId = `run-${this.runStartTime}-${(0, crypto_1.randomUUID)()}`;
+        const runId = `run-${this.runStartTime}-581d5ad8-ce75-4ca5-94a6-ed29c466c815`; // Need not to change
+        // --- CALLING _getEnvDetails HERE ---
+        const environmentDetails = this._getEnvDetails();
         const runData = {
             id: runId,
             timestamp: new Date(this.runStartTime),
@@ -373,10 +436,16 @@ class PlaywrightPulseReporter {
             failed: 0,
             skipped: 0,
             duration,
+            // --- ADDED environmentDetails HERE ---
+            environment: environmentDetails,
         };
-        let finalReport;
+        let finalReport = undefined; // Initialize as 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;
+            }
         }
         else {
             this.results.forEach((r) => (r.runId = runId));
@@ -384,42 +453,84 @@ class PlaywrightPulseReporter {
             runData.failed = this.results.filter((r) => r.status === "failed").length;
             runData.skipped = this.results.filter((r) => r.status === "skipped").length;
             runData.totalTests = this.results.length;
+            const reviveDates = (key, value) => {
+                const isoDateRegex = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(\.\d+)?Z$/;
+                if (typeof value === "string" && isoDateRegex.test(value)) {
+                    const date = new Date(value);
+                    return !isNaN(date.getTime()) ? date : value;
+                }
+                return value;
+            };
+            const properlyTypedResults = JSON.parse(JSON.stringify(this.results), reviveDates);
             finalReport = {
                 run: runData,
-                results: this.results,
+                results: properlyTypedResults,
                 metadata: { generatedAt: new Date().toISOString() },
             };
         }
-        // This block seems redundant as finalReport is already assigned above.
-        // if (this.isSharded) {
-        //   finalReport = await this._mergeShardResults(runData);
-        // } else {
-        //   this.results.forEach((r) => (r.runId = runId));
-        //   runData.passed = this.results.filter((r) => r.status === "passed").length;
-        //   runData.failed = this.results.filter((r) => r.status === "failed").length;
-        //   runData.skipped = this.results.filter((r) => r.status === "skipped").length;
-        //   runData.totalTests = this.results.length;
-        //   finalReport = {
-        //     run: runData, results: this.results,
-        //     metadata: { generatedAt: new Date().toISOString() },
-        //   };
-        // }
-        const finalRunStatus = ((_b = (_a = finalReport.run) === null || _a === void 0 ? void 0 : _a.failed) !== null && _b !== void 0 ? _b : 0 > 0)
+        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"
-            : ((_c = finalReport.run) === null || _c === void 0 ? void 0 : _c.totalTests) === 0
-                ? "no tests"
+            : ((_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:    ${(_e = (_d = finalReport.run) === null || _d === void 0 ? void 0 : _d.totalTests) !== null && _e !== void 0 ? _e : "N/A"}
-  Passed:         ${(_g = (_f = finalReport.run) === null || _f === void 0 ? void 0 : _f.passed) !== null && _g !== void 0 ? _g : "N/A"}
-  Failed:         ${(_j = (_h = finalReport.run) === null || _h === void 0 ? void 0 : _h.failed) !== null && _j !== void 0 ? _j : "N/A"}
-  Skipped:        ${(_l = (_k = finalReport.run) === null || _k === void 0 ? void 0 : _k.skipped) !== null && _l !== void 0 ? _l : "N/A"}
-  Duration:       ${(duration / 1000).toFixed(2)}s
+  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 
 -----------------------------------------`;
-        console.log(summary);
+        if (this.printsToStdio()) {
+            console.log(summary);
+        }
         const finalOutputPath = path.join(this.outputDir, this.baseOutputFile);
         try {
             await this._ensureDirExists(this.outputDir);
@@ -430,7 +541,9 @@ PlaywrightPulseReporter: Run Finished
                     return value.toString();
                 return value;
             }, 2));
-            console.log(`PlaywrightPulseReporter: JSON report written to ${finalOutputPath}`);
+            if (this.printsToStdio()) {
+                console.log(`PlaywrightPulseReporter: JSON report written to ${finalOutputPath}`);
+            }
         }
         catch (error) {
             console.error(`Pulse Reporter: Failed to write final JSON report to ${finalOutputPath}. Error: ${error.message}`);

package/dist/types/index.d.ts

@@ -36,6 +36,10 @@ export interface TestResult {
     tracePath?: string;
     stdout?: string[];
     stderr?: string[];
+    workerId?: number;
+    totalWorkers?: number;
+    configFile?: string;
+    metadata?: string;
 }
 export interface TestRun {
     id: string;
@@ -45,6 +49,7 @@ export interface TestRun {
     failed: number;
     skipped: number;
     duration: number;
+    environment?: EnvDetails;
 }
 export interface TrendDataPoint {
     date: string;
@@ -63,3 +68,15 @@ export interface PlaywrightPulseReporterOptions {
     outputDir?: string;
     base64Images?: boolean;
 }
+export interface EnvDetails {
+    host: string;
+    os: string;
+    cpu: {
+        model: string;
+        cores: number;
+    };
+    memory: string;
+    node: string;
+    v8: string;
+    cwd: string;
+}