@reportforge/playwright-pdf 0.13.1 → 0.13.2

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 
+## [0.13.2] - 2026-07-04
+
+### Fixed
+
+- **License refresh no longer wipes a valid cache on rate-limit**: an HTTP 429/408 from the license server was misclassified as a denial, clearing a still-valid cached license and skipping PDF generation. Now treated as transient — falls back to the cache.
+- **`pdfPassword` encryption now actually works**: a malformed qpdf command meant every password-protected PDF shipped unencrypted while the run reported success. qpdf failures are now fatal instead of silent, and the password is no longer passed via process arguments.
+- **`test.fail()` expected failures no longer miscounted**: an expected failure was counted as a failure in the KPI stats and failure list even though the suite tree correctly showed it as passed.
+- **Sharded runs: describe blocks split across shards now merge correctly** instead of rendering as duplicate, half-populated nodes.
+- **Sharded runs: failure severity now reflects tags/annotations** instead of always showing "medium".
+- **Custom templates sharing a basename no longer overwrite each other's output.**
+- **PDF size-cap retuning now actually reduces the inline failure count** when the report has fewer real failures than the configured cap.
+
+---
+
 ## [0.13.1] - 2026-07-03
 
 ### Changed

package/dist/cli/index.js

@@ -7325,25 +7325,18 @@ var init_PdfEncryptor = __esm({
           return;
         }
         const tmpPath = import_path3.default.join(import_os2.default.tmpdir(), `rf-encrypted-${Date.now()}.pdf`);
-        const pwFile = import_path3.default.join(import_os2.default.tmpdir(), `rf-pw-${import_crypto2.default.randomBytes(8).toString("hex")}`);
+        const argFile = import_path3.default.join(import_os2.default.tmpdir(), `rf-qpdf-args-${import_crypto2.default.randomBytes(8).toString("hex")}`);
         try {
-          import_fs5.default.writeFileSync(pwFile, `${password}
-${password}`, { mode: 384 });
-          (0, import_child_process2.execFileSync)("qpdf", [
-            `--password-file=${pwFile}`,
-            "--encrypt",
-            "256",
-            "--",
-            pdfPath,
-            tmpPath
-          ], { stdio: "pipe" });
+          const args = ["--encrypt", password, password, "256", "--", pdfPath, tmpPath];
+          import_fs5.default.writeFileSync(argFile, args.join("\n"), { mode: 384 });
+          (0, import_child_process2.execFileSync)("qpdf", [`@${argFile}`], { stdio: "pipe" });
           import_fs5.default.renameSync(tmpPath, pdfPath);
           logger.info(`PDF encrypted with password.`);
         } catch (e) {
-          logger.warn(`qpdf encryption failed: ${e.message}`);
+          throw new Error(`qpdf encryption failed: ${e.message}`);
         } finally {
           if (import_fs5.default.existsSync(tmpPath)) import_fs5.default.unlinkSync(tmpPath);
-          import_fs5.default.rmSync(pwFile, { force: true });
+          import_fs5.default.rmSync(argFile, { force: true });
         }
       }
       isQpdfAvailable() {
@@ -7668,7 +7661,8 @@ var init_PdfGenerator = __esm({
             if (capBytes > 0) {
               const sizeBytes = (await import_fs8.default.promises.stat(outputPath)).size;
               if (sizeBytes > capBytes) {
-                const tighter = this.tightenForCap(compression, sizeBytes, capBytes);
+                const inlinedCount = Math.min(templateData.failures.length, compression.maxInlineFailures);
+                const tighter = this.tightenForCap(compression, sizeBytes, capBytes, inlinedCount);
                 logger.warn(
                   `PDF size ${fmtMb(sizeBytes)} exceeds cap ${fmtMb(capBytes)} \u2014 re-rendering (jpeg-q=${tighter.quality}, max-w=${tighter.maxWidth}px, inline-failures\u2264${tighter.maxInlineFailures}).`
                 );
@@ -7697,11 +7691,30 @@ var init_PdfGenerator = __esm({
       }
       resolveTemplateSources(options) {
         if (options.templatePath && options.templatePath.length > 0) {
-          return [...new Set(options.templatePath)].map((p) => ({
-            kind: "custom",
-            absolutePath: import_path6.default.isAbsolute(p) ? p : import_path6.default.resolve(process.cwd(), p),
-            label: import_path6.default.basename(p, ".hbs")
-          }));
+          const byAbsolutePath = /* @__PURE__ */ new Map();
+          for (const p of options.templatePath) {
+            const absolutePath = import_path6.default.isAbsolute(p) ? p : import_path6.default.resolve(process.cwd(), p);
+            if (!byAbsolutePath.has(absolutePath)) byAbsolutePath.set(absolutePath, p);
+          }
+          const labelCounts = /* @__PURE__ */ new Map();
+          for (const rawPath of byAbsolutePath.values()) {
+            const label = import_path6.default.basename(rawPath, ".hbs");
+            labelCounts.set(label, (labelCounts.get(label) ?? 0) + 1);
+          }
+          const seenPerLabel = /* @__PURE__ */ new Map();
+          return Array.from(byAbsolutePath.entries()).map(([absolutePath, rawPath]) => {
+            const baseLabel = import_path6.default.basename(rawPath, ".hbs");
+            let label = baseLabel;
+            if ((labelCounts.get(baseLabel) ?? 0) > 1) {
+              const n = (seenPerLabel.get(baseLabel) ?? 0) + 1;
+              seenPerLabel.set(baseLabel, n);
+              label = `${baseLabel}-${n}`;
+              logger.warn(
+                `Multiple templatePath entries resolve to the basename "${baseLabel}" \u2014 disambiguating output filenames with a numeric suffix ("${label}").`
+              );
+            }
+            return { kind: "custom", absolutePath, label };
+          });
         }
         const rawIds = Array.isArray(options.template) ? options.template : [options.template];
         return [...new Set(rawIds)].map((id) => ({ kind: "builtin", id }));
@@ -7723,8 +7736,8 @@ var init_PdfGenerator = __esm({
        * Also tightens JPEG quality and width to claw back bytes from each
        * remaining image.
        */
-      tightenForCap(prev, actualBytes, capBytes) {
-        const observedPerFailure = actualBytes / Math.max(1, prev.maxInlineFailures);
+      tightenForCap(prev, actualBytes, capBytes, actualInlinedCount) {
+        const observedPerFailure = actualBytes / Math.max(1, actualInlinedCount);
         const targetInline = Math.max(20, Math.floor(capBytes * 0.85 / observedPerFailure));
         return {
           effective: "max",

package/dist/index.js

@@ -11061,6 +11061,11 @@ var LicenseClient = class {
         body: JSON.stringify(body),
         signal: controller.signal
       });
+      if (res.status === 429 || res.status === 408) {
+        await res.body?.cancel().catch(() => {
+        });
+        throw new Error(`HTTP ${res.status}`);
+      }
       if (res.status >= 400 && res.status < 500) {
         const text = await res.text().catch(() => "");
         throw new Error(`license-denied: ${text || `HTTP ${res.status}`}`);
@@ -11619,6 +11624,17 @@ init_cjs_shims();
 // src/collector/stats-utils.ts
 init_cjs_shims();
 var TOP_SUITES_FOR_CHART = 10;
+function resolveTestStatus(test, result) {
+  if (!result) return "skipped";
+  const expected = test.expectedStatus ?? "passed";
+  const actual = result.status;
+  if (actual === "skipped") return "skipped";
+  if (actual === "timedOut") return "timedOut";
+  if (actual === "passed" && result.retry > 0) return "flaky";
+  if (actual === "passed" && expected === "passed") return "passed";
+  if (actual === "failed" && expected === "failed") return "passed";
+  return "failed";
+}
 function statsFromTests(tests) {
   return {
     total: tests.length,
@@ -11694,7 +11710,7 @@ var SuiteWalker = class {
   }
   buildTest(test, resultMap) {
     const result = resultMap.get(test.id);
-    const status = this.resolveStatus(test, result);
+    const status = resolveTestStatus(test, result);
     return {
       id: test.id,
       title: test.title,
@@ -11707,17 +11723,6 @@ var SuiteWalker = class {
       location: test.location ? `${test.location.file}:${test.location.line}` : void 0
     };
   }
-  resolveStatus(test, result) {
-    if (!result) return "skipped";
-    const expected = test.expectedStatus ?? "passed";
-    const actual = result.status;
-    if (actual === "skipped") return "skipped";
-    if (actual === "timedOut") return "timedOut";
-    if (actual === "passed" && result.retry > 0) return "flaky";
-    if (actual === "passed" && expected === "passed") return "passed";
-    if (actual === "failed" && expected === "failed") return "passed";
-    return "failed";
-  }
 };
 
 // src/collector/severity.ts
@@ -11731,6 +11736,16 @@ var SEVERITY_RANK = {
 function sortBySeverity(failures) {
   return [...failures].sort((a, b) => SEVERITY_RANK[a.severity] - SEVERITY_RANK[b.severity]);
 }
+function inferSeverity(annotations, tags) {
+  const allLabels = [
+    ...annotations.map((a) => a.type.toLowerCase()),
+    ...tags.map((t) => t.toLowerCase())
+  ];
+  if (allLabels.some((l) => l.includes("critical") || l.includes("blocker"))) return "critical";
+  if (allLabels.some((l) => l.includes("high") || l.includes("major"))) return "high";
+  if (allLabels.some((l) => l.includes("low") || l.includes("minor"))) return "low";
+  return "medium";
+}
 
 // src/utils/env.ts
 init_cjs_shims();
@@ -11855,6 +11870,10 @@ var DataCollector = class {
   constructor(captureOpts = {}) {
     this.captureOpts = captureOpts;
     this.resultMap = /* @__PURE__ */ new Map();
+    // Parallel to resultMap — needed so computeStats() (which only sees
+    // TestResult, not TestCase) can reconcile against expectedStatus the same
+    // way resolveTestStatus does for the suite tree and the failure list.
+    this.expectedStatusMap = /* @__PURE__ */ new Map();
     this.failureRecords = [];
     this.startTime = Date.now();
   }
@@ -11865,7 +11884,10 @@ var DataCollector = class {
   }
   onTestEnd(test, result) {
     this.resultMap.set(test.id, result);
-    if (result.status === "failed" || result.status === "timedOut") {
+    this.expectedStatusMap.set(test.id, test.expectedStatus);
+    const resolved = resolveTestStatus(test, result);
+    const isRealFailure = resolved === "failed" || resolved === "timedOut";
+    if (isRealFailure) {
       const existing = this.failureRecords.findIndex((f) => f.testId === test.id);
       const record = this.buildFailureRecord(test, result);
       if (existing >= 0) {
@@ -11905,7 +11927,7 @@ var DataCollector = class {
       (a) => a.name.toLowerCase().includes("screenshot") && a.contentType.startsWith("image/")
     );
     const suitePath = test.titlePath().slice(0, -1);
-    const severity = this.inferSeverity(test);
+    const severity = inferSeverity(test.annotations ?? [], test.tags ?? []);
     return {
       testId: test.id,
       testTitle: test.title,
@@ -11928,28 +11950,18 @@ var DataCollector = class {
       execution: executionFromResult(result, this.captureOpts)
     };
   }
-  inferSeverity(test) {
-    const annotations = test.annotations ?? [];
-    const tags = test.tags ?? [];
-    const allLabels = [
-      ...annotations.map((a) => a.type.toLowerCase()),
-      ...tags.map((t) => t.toLowerCase())
-    ];
-    if (allLabels.some((l) => l.includes("critical") || l.includes("blocker"))) return "critical";
-    if (allLabels.some((l) => l.includes("high") || l.includes("major"))) return "high";
-    if (allLabels.some((l) => l.includes("low") || l.includes("minor"))) return "low";
-    return "medium";
-  }
   computeStats(fullResult) {
     let total = 0, passed = 0, failed = 0, skipped = 0, flaky = 0, timedOut = 0;
-    for (const [, result] of this.resultMap) {
+    for (const [testId, result] of this.resultMap) {
       total++;
-      switch (result.status) {
-        // A test that passes on a retry (retry > 0) is flaky, NOT a clean pass.
-        // Counting it in `passed` too would double-count it against `flaky` and
-        // push passRate over 100%.
+      const expectedStatus = this.expectedStatusMap.get(testId);
+      const resolved = resolveTestStatus({ expectedStatus }, result);
+      switch (resolved) {
         case "passed":
-          result.retry > 0 ? flaky++ : passed++;
+          passed++;
+          break;
+        case "flaky":
+          flaky++;
           break;
         case "failed":
           failed++;
@@ -11961,9 +11973,6 @@ var DataCollector = class {
         case "skipped":
           skipped++;
           break;
-        case "interrupted":
-          failed++;
-          break;
       }
     }
     const duration = fullResult.duration ?? Date.now() - this.startTime;
@@ -12123,18 +12132,26 @@ var ShardMerger = class {
     });
   }
   buildDescribeSuites(jsonSuites, projectName, parentPath) {
-    return jsonSuites.map((suite) => {
-      const suitePath = [...parentPath, suite.title];
-      const tests = suite.specs.map((spec) => this.specToTestNode(spec, projectName, suitePath));
-      const childSuites = this.buildDescribeSuites(suite.suites ?? [], projectName, suitePath);
+    const groups = /* @__PURE__ */ new Map();
+    for (const suite of jsonSuites) {
+      if (!groups.has(suite.title)) groups.set(suite.title, []);
+      groups.get(suite.title).push(suite);
+    }
+    return Array.from(groups.values()).map((group) => {
+      const ref = group[0];
+      const suitePath = [...parentPath, ref.title];
+      const allSpecs = group.flatMap((s) => s.specs);
+      const allChildSuites = group.flatMap((s) => s.suites ?? []);
+      const tests = allSpecs.map((spec) => this.specToTestNode(spec, projectName, suitePath));
+      const childSuites = this.buildDescribeSuites(allChildSuites, projectName, suitePath);
       const stats = aggregateStats([
         ...childSuites.map((s) => s.stats),
         statsFromTests(tests)
       ]);
       return {
-        title: suite.title,
+        title: ref.title,
         type: "describe",
-        location: suite.file ? `${suite.file}:${suite.line ?? 0}` : void 0,
+        location: ref.file ? `${ref.file}:${ref.line ?? 0}` : void 0,
         suites: childSuites,
         tests,
         stats
@@ -12240,7 +12257,7 @@ var ShardMerger = class {
         duration: r.duration
       })),
       durationMs: lastResult?.duration ?? 0,
-      severity: "medium",
+      severity: inferSeverity(test.annotations ?? [], spec.tags ?? []),
       execution: lastResult ? buildExecution(
         {
           // Guard each field — a partial / non-conformant / older shard JSON can
@@ -12816,25 +12833,18 @@ var PdfEncryptor = class {
       return;
     }
     const tmpPath = import_path5.default.join(import_os5.default.tmpdir(), `rf-encrypted-${Date.now()}.pdf`);
-    const pwFile = import_path5.default.join(import_os5.default.tmpdir(), `rf-pw-${import_crypto7.default.randomBytes(8).toString("hex")}`);
+    const argFile = import_path5.default.join(import_os5.default.tmpdir(), `rf-qpdf-args-${import_crypto7.default.randomBytes(8).toString("hex")}`);
     try {
-      import_fs8.default.writeFileSync(pwFile, `${password}
-${password}`, { mode: 384 });
-      (0, import_child_process3.execFileSync)("qpdf", [
-        `--password-file=${pwFile}`,
-        "--encrypt",
-        "256",
-        "--",
-        pdfPath,
-        tmpPath
-      ], { stdio: "pipe" });
+      const args = ["--encrypt", password, password, "256", "--", pdfPath, tmpPath];
+      import_fs8.default.writeFileSync(argFile, args.join("\n"), { mode: 384 });
+      (0, import_child_process3.execFileSync)("qpdf", [`@${argFile}`], { stdio: "pipe" });
       import_fs8.default.renameSync(tmpPath, pdfPath);
       logger.info(`PDF encrypted with password.`);
     } catch (e) {
-      logger.warn(`qpdf encryption failed: ${e.message}`);
+      throw new Error(`qpdf encryption failed: ${e.message}`);
     } finally {
       if (import_fs8.default.existsSync(tmpPath)) import_fs8.default.unlinkSync(tmpPath);
-      import_fs8.default.rmSync(pwFile, { force: true });
+      import_fs8.default.rmSync(argFile, { force: true });
     }
   }
   isQpdfAvailable() {
@@ -13083,7 +13093,8 @@ var PdfGenerator = class {
         if (capBytes > 0) {
           const sizeBytes = (await import_fs11.default.promises.stat(outputPath)).size;
           if (sizeBytes > capBytes) {
-            const tighter = this.tightenForCap(compression, sizeBytes, capBytes);
+            const inlinedCount = Math.min(templateData.failures.length, compression.maxInlineFailures);
+            const tighter = this.tightenForCap(compression, sizeBytes, capBytes, inlinedCount);
             logger.warn(
               `PDF size ${fmtMb(sizeBytes)} exceeds cap ${fmtMb(capBytes)} \u2014 re-rendering (jpeg-q=${tighter.quality}, max-w=${tighter.maxWidth}px, inline-failures\u2264${tighter.maxInlineFailures}).`
             );
@@ -13112,11 +13123,30 @@ var PdfGenerator = class {
   }
   resolveTemplateSources(options) {
     if (options.templatePath && options.templatePath.length > 0) {
-      return [...new Set(options.templatePath)].map((p) => ({
-        kind: "custom",
-        absolutePath: import_path8.default.isAbsolute(p) ? p : import_path8.default.resolve(process.cwd(), p),
-        label: import_path8.default.basename(p, ".hbs")
-      }));
+      const byAbsolutePath = /* @__PURE__ */ new Map();
+      for (const p of options.templatePath) {
+        const absolutePath = import_path8.default.isAbsolute(p) ? p : import_path8.default.resolve(process.cwd(), p);
+        if (!byAbsolutePath.has(absolutePath)) byAbsolutePath.set(absolutePath, p);
+      }
+      const labelCounts = /* @__PURE__ */ new Map();
+      for (const rawPath of byAbsolutePath.values()) {
+        const label = import_path8.default.basename(rawPath, ".hbs");
+        labelCounts.set(label, (labelCounts.get(label) ?? 0) + 1);
+      }
+      const seenPerLabel = /* @__PURE__ */ new Map();
+      return Array.from(byAbsolutePath.entries()).map(([absolutePath, rawPath]) => {
+        const baseLabel = import_path8.default.basename(rawPath, ".hbs");
+        let label = baseLabel;
+        if ((labelCounts.get(baseLabel) ?? 0) > 1) {
+          const n = (seenPerLabel.get(baseLabel) ?? 0) + 1;
+          seenPerLabel.set(baseLabel, n);
+          label = `${baseLabel}-${n}`;
+          logger.warn(
+            `Multiple templatePath entries resolve to the basename "${baseLabel}" \u2014 disambiguating output filenames with a numeric suffix ("${label}").`
+          );
+        }
+        return { kind: "custom", absolutePath, label };
+      });
     }
     const rawIds = Array.isArray(options.template) ? options.template : [options.template];
     return [...new Set(rawIds)].map((id) => ({ kind: "builtin", id }));
@@ -13138,8 +13168,8 @@ var PdfGenerator = class {
    * Also tightens JPEG quality and width to claw back bytes from each
    * remaining image.
    */
-  tightenForCap(prev, actualBytes, capBytes) {
-    const observedPerFailure = actualBytes / Math.max(1, prev.maxInlineFailures);
+  tightenForCap(prev, actualBytes, capBytes, actualInlinedCount) {
+    const observedPerFailure = actualBytes / Math.max(1, actualInlinedCount);
     const targetInline = Math.max(20, Math.floor(capBytes * 0.85 / observedPerFailure));
     return {
       effective: "max",
@@ -14001,7 +14031,7 @@ var PdfReporter = class {
     this.liveConsole = false;
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.13.1" : "0.x";
+    const version = true ? "0.13.2" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/dist/index.mjs

@@ -11062,6 +11062,11 @@ var LicenseClient = class {
         body: JSON.stringify(body),
         signal: controller.signal
       });
+      if (res.status === 429 || res.status === 408) {
+        await res.body?.cancel().catch(() => {
+        });
+        throw new Error(`HTTP ${res.status}`);
+      }
       if (res.status >= 400 && res.status < 500) {
         const text = await res.text().catch(() => "");
         throw new Error(`license-denied: ${text || `HTTP ${res.status}`}`);
@@ -11620,6 +11625,17 @@ init_esm_shims();
 // src/collector/stats-utils.ts
 init_esm_shims();
 var TOP_SUITES_FOR_CHART = 10;
+function resolveTestStatus(test, result) {
+  if (!result) return "skipped";
+  const expected = test.expectedStatus ?? "passed";
+  const actual = result.status;
+  if (actual === "skipped") return "skipped";
+  if (actual === "timedOut") return "timedOut";
+  if (actual === "passed" && result.retry > 0) return "flaky";
+  if (actual === "passed" && expected === "passed") return "passed";
+  if (actual === "failed" && expected === "failed") return "passed";
+  return "failed";
+}
 function statsFromTests(tests) {
   return {
     total: tests.length,
@@ -11695,7 +11711,7 @@ var SuiteWalker = class {
   }
   buildTest(test, resultMap) {
     const result = resultMap.get(test.id);
-    const status = this.resolveStatus(test, result);
+    const status = resolveTestStatus(test, result);
     return {
       id: test.id,
       title: test.title,
@@ -11708,17 +11724,6 @@ var SuiteWalker = class {
       location: test.location ? `${test.location.file}:${test.location.line}` : void 0
     };
   }
-  resolveStatus(test, result) {
-    if (!result) return "skipped";
-    const expected = test.expectedStatus ?? "passed";
-    const actual = result.status;
-    if (actual === "skipped") return "skipped";
-    if (actual === "timedOut") return "timedOut";
-    if (actual === "passed" && result.retry > 0) return "flaky";
-    if (actual === "passed" && expected === "passed") return "passed";
-    if (actual === "failed" && expected === "failed") return "passed";
-    return "failed";
-  }
 };
 
 // src/collector/severity.ts
@@ -11732,6 +11737,16 @@ var SEVERITY_RANK = {
 function sortBySeverity(failures) {
   return [...failures].sort((a, b) => SEVERITY_RANK[a.severity] - SEVERITY_RANK[b.severity]);
 }
+function inferSeverity(annotations, tags) {
+  const allLabels = [
+    ...annotations.map((a) => a.type.toLowerCase()),
+    ...tags.map((t) => t.toLowerCase())
+  ];
+  if (allLabels.some((l) => l.includes("critical") || l.includes("blocker"))) return "critical";
+  if (allLabels.some((l) => l.includes("high") || l.includes("major"))) return "high";
+  if (allLabels.some((l) => l.includes("low") || l.includes("minor"))) return "low";
+  return "medium";
+}
 
 // src/utils/env.ts
 init_esm_shims();
@@ -11856,6 +11871,10 @@ var DataCollector = class {
   constructor(captureOpts = {}) {
     this.captureOpts = captureOpts;
     this.resultMap = /* @__PURE__ */ new Map();
+    // Parallel to resultMap — needed so computeStats() (which only sees
+    // TestResult, not TestCase) can reconcile against expectedStatus the same
+    // way resolveTestStatus does for the suite tree and the failure list.
+    this.expectedStatusMap = /* @__PURE__ */ new Map();
     this.failureRecords = [];
     this.startTime = Date.now();
   }
@@ -11866,7 +11885,10 @@ var DataCollector = class {
   }
   onTestEnd(test, result) {
     this.resultMap.set(test.id, result);
-    if (result.status === "failed" || result.status === "timedOut") {
+    this.expectedStatusMap.set(test.id, test.expectedStatus);
+    const resolved = resolveTestStatus(test, result);
+    const isRealFailure = resolved === "failed" || resolved === "timedOut";
+    if (isRealFailure) {
       const existing = this.failureRecords.findIndex((f) => f.testId === test.id);
       const record = this.buildFailureRecord(test, result);
       if (existing >= 0) {
@@ -11906,7 +11928,7 @@ var DataCollector = class {
       (a) => a.name.toLowerCase().includes("screenshot") && a.contentType.startsWith("image/")
     );
     const suitePath = test.titlePath().slice(0, -1);
-    const severity = this.inferSeverity(test);
+    const severity = inferSeverity(test.annotations ?? [], test.tags ?? []);
     return {
       testId: test.id,
       testTitle: test.title,
@@ -11929,28 +11951,18 @@ var DataCollector = class {
       execution: executionFromResult(result, this.captureOpts)
     };
   }
-  inferSeverity(test) {
-    const annotations = test.annotations ?? [];
-    const tags = test.tags ?? [];
-    const allLabels = [
-      ...annotations.map((a) => a.type.toLowerCase()),
-      ...tags.map((t) => t.toLowerCase())
-    ];
-    if (allLabels.some((l) => l.includes("critical") || l.includes("blocker"))) return "critical";
-    if (allLabels.some((l) => l.includes("high") || l.includes("major"))) return "high";
-    if (allLabels.some((l) => l.includes("low") || l.includes("minor"))) return "low";
-    return "medium";
-  }
   computeStats(fullResult) {
     let total = 0, passed = 0, failed = 0, skipped = 0, flaky = 0, timedOut = 0;
-    for (const [, result] of this.resultMap) {
+    for (const [testId, result] of this.resultMap) {
       total++;
-      switch (result.status) {
-        // A test that passes on a retry (retry > 0) is flaky, NOT a clean pass.
-        // Counting it in `passed` too would double-count it against `flaky` and
-        // push passRate over 100%.
+      const expectedStatus = this.expectedStatusMap.get(testId);
+      const resolved = resolveTestStatus({ expectedStatus }, result);
+      switch (resolved) {
         case "passed":
-          result.retry > 0 ? flaky++ : passed++;
+          passed++;
+          break;
+        case "flaky":
+          flaky++;
           break;
         case "failed":
           failed++;
@@ -11962,9 +11974,6 @@ var DataCollector = class {
         case "skipped":
           skipped++;
           break;
-        case "interrupted":
-          failed++;
-          break;
       }
     }
     const duration = fullResult.duration ?? Date.now() - this.startTime;
@@ -12124,18 +12133,26 @@ var ShardMerger = class {
     });
   }
   buildDescribeSuites(jsonSuites, projectName, parentPath) {
-    return jsonSuites.map((suite) => {
-      const suitePath = [...parentPath, suite.title];
-      const tests = suite.specs.map((spec) => this.specToTestNode(spec, projectName, suitePath));
-      const childSuites = this.buildDescribeSuites(suite.suites ?? [], projectName, suitePath);
+    const groups = /* @__PURE__ */ new Map();
+    for (const suite of jsonSuites) {
+      if (!groups.has(suite.title)) groups.set(suite.title, []);
+      groups.get(suite.title).push(suite);
+    }
+    return Array.from(groups.values()).map((group) => {
+      const ref = group[0];
+      const suitePath = [...parentPath, ref.title];
+      const allSpecs = group.flatMap((s) => s.specs);
+      const allChildSuites = group.flatMap((s) => s.suites ?? []);
+      const tests = allSpecs.map((spec) => this.specToTestNode(spec, projectName, suitePath));
+      const childSuites = this.buildDescribeSuites(allChildSuites, projectName, suitePath);
       const stats = aggregateStats([
         ...childSuites.map((s) => s.stats),
         statsFromTests(tests)
       ]);
       return {
-        title: suite.title,
+        title: ref.title,
         type: "describe",
-        location: suite.file ? `${suite.file}:${suite.line ?? 0}` : void 0,
+        location: ref.file ? `${ref.file}:${ref.line ?? 0}` : void 0,
         suites: childSuites,
         tests,
         stats
@@ -12241,7 +12258,7 @@ var ShardMerger = class {
         duration: r.duration
       })),
       durationMs: lastResult?.duration ?? 0,
-      severity: "medium",
+      severity: inferSeverity(test.annotations ?? [], spec.tags ?? []),
       execution: lastResult ? buildExecution(
         {
           // Guard each field — a partial / non-conformant / older shard JSON can
@@ -12817,25 +12834,18 @@ var PdfEncryptor = class {
       return;
     }
     const tmpPath = path9.join(os8.tmpdir(), `rf-encrypted-${Date.now()}.pdf`);
-    const pwFile = path9.join(os8.tmpdir(), `rf-pw-${crypto3.randomBytes(8).toString("hex")}`);
+    const argFile = path9.join(os8.tmpdir(), `rf-qpdf-args-${crypto3.randomBytes(8).toString("hex")}`);
     try {
-      fs9.writeFileSync(pwFile, `${password}
-${password}`, { mode: 384 });
-      execFileSync("qpdf", [
-        `--password-file=${pwFile}`,
-        "--encrypt",
-        "256",
-        "--",
-        pdfPath,
-        tmpPath
-      ], { stdio: "pipe" });
+      const args = ["--encrypt", password, password, "256", "--", pdfPath, tmpPath];
+      fs9.writeFileSync(argFile, args.join("\n"), { mode: 384 });
+      execFileSync("qpdf", [`@${argFile}`], { stdio: "pipe" });
       fs9.renameSync(tmpPath, pdfPath);
       logger.info(`PDF encrypted with password.`);
     } catch (e) {
-      logger.warn(`qpdf encryption failed: ${e.message}`);
+      throw new Error(`qpdf encryption failed: ${e.message}`);
     } finally {
       if (fs9.existsSync(tmpPath)) fs9.unlinkSync(tmpPath);
-      fs9.rmSync(pwFile, { force: true });
+      fs9.rmSync(argFile, { force: true });
     }
   }
   isQpdfAvailable() {
@@ -13084,7 +13094,8 @@ var PdfGenerator = class {
         if (capBytes > 0) {
           const sizeBytes = (await fs12.promises.stat(outputPath)).size;
           if (sizeBytes > capBytes) {
-            const tighter = this.tightenForCap(compression, sizeBytes, capBytes);
+            const inlinedCount = Math.min(templateData.failures.length, compression.maxInlineFailures);
+            const tighter = this.tightenForCap(compression, sizeBytes, capBytes, inlinedCount);
             logger.warn(
               `PDF size ${fmtMb(sizeBytes)} exceeds cap ${fmtMb(capBytes)} \u2014 re-rendering (jpeg-q=${tighter.quality}, max-w=${tighter.maxWidth}px, inline-failures\u2264${tighter.maxInlineFailures}).`
             );
@@ -13113,11 +13124,30 @@ var PdfGenerator = class {
   }
   resolveTemplateSources(options) {
     if (options.templatePath && options.templatePath.length > 0) {
-      return [...new Set(options.templatePath)].map((p) => ({
-        kind: "custom",
-        absolutePath: path12.isAbsolute(p) ? p : path12.resolve(process.cwd(), p),
-        label: path12.basename(p, ".hbs")
-      }));
+      const byAbsolutePath = /* @__PURE__ */ new Map();
+      for (const p of options.templatePath) {
+        const absolutePath = path12.isAbsolute(p) ? p : path12.resolve(process.cwd(), p);
+        if (!byAbsolutePath.has(absolutePath)) byAbsolutePath.set(absolutePath, p);
+      }
+      const labelCounts = /* @__PURE__ */ new Map();
+      for (const rawPath of byAbsolutePath.values()) {
+        const label = path12.basename(rawPath, ".hbs");
+        labelCounts.set(label, (labelCounts.get(label) ?? 0) + 1);
+      }
+      const seenPerLabel = /* @__PURE__ */ new Map();
+      return Array.from(byAbsolutePath.entries()).map(([absolutePath, rawPath]) => {
+        const baseLabel = path12.basename(rawPath, ".hbs");
+        let label = baseLabel;
+        if ((labelCounts.get(baseLabel) ?? 0) > 1) {
+          const n = (seenPerLabel.get(baseLabel) ?? 0) + 1;
+          seenPerLabel.set(baseLabel, n);
+          label = `${baseLabel}-${n}`;
+          logger.warn(
+            `Multiple templatePath entries resolve to the basename "${baseLabel}" \u2014 disambiguating output filenames with a numeric suffix ("${label}").`
+          );
+        }
+        return { kind: "custom", absolutePath, label };
+      });
     }
     const rawIds = Array.isArray(options.template) ? options.template : [options.template];
     return [...new Set(rawIds)].map((id) => ({ kind: "builtin", id }));
@@ -13139,8 +13169,8 @@ var PdfGenerator = class {
    * Also tightens JPEG quality and width to claw back bytes from each
    * remaining image.
    */
-  tightenForCap(prev, actualBytes, capBytes) {
-    const observedPerFailure = actualBytes / Math.max(1, prev.maxInlineFailures);
+  tightenForCap(prev, actualBytes, capBytes, actualInlinedCount) {
+    const observedPerFailure = actualBytes / Math.max(1, actualInlinedCount);
     const targetInline = Math.max(20, Math.floor(capBytes * 0.85 / observedPerFailure));
     return {
       effective: "max",
@@ -14002,7 +14032,7 @@ var PdfReporter = class {
     this.liveConsole = false;
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.13.1" : "0.x";
+    const version = true ? "0.13.2" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reportforge/playwright-pdf",
-  "version": "0.13.1",
+  "version": "0.13.2",
   "description": "Playwright Test reporter that generates designed PDF reports: minimal, detailed, and executive templates with CI/CD integrations",
   "license": "MIT",
   "author": "ReportForge",