@reportforge/playwright-pdf 0.13.1 → 0.14.0

package/CHANGELOG.md

@@ -3,6 +3,34 @@
 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.14.0] - 2026-07-06
+
+### Added
+
+- **Executive report leads with a plain-language brief and a hero pass-rate stat**: a new tinted brief band composes the trend delta and root-cause tally into one sentence (e.g. "Pass rate climbed 4 points to 92%... 3 root causes remain: 2 assertion, 1 timeout"), replacing the 6-card KPI grid with a single hero pass-rate number and a compact secondary stat row. The trend chart is promoted above the smaller pass-rate/suite-results charts. `detailed` and `minimal` templates are unchanged.
+- **New `sections.briefBand` option** to toggle the brief band on any template (on by default for `executive`).
+
+### Fixed
+
+- **Flaky doughnut slice now matches the `--flaky` design token** instead of a stray hardcoded color.
+- **`minimal` template's header no longer uses a side-stripe border**, now consistent with the design system's top-accent-bar convention used everywhere else.
+
+---
+
+## [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/README.md

@@ -61,6 +61,7 @@ Three templates, one reporter. Every PDF is fully offline: fonts, charts, and sc
 - `npx @reportforge/playwright-pdf export-feedback`: exports locally-collected unrecognized failures (tokenized, local-only) for optional manual sharing.
 - **Rich execution capture**: opt-in "Steps to Reproduce" outline (a nested, copy-pasteable Markdown list of your `test.step`s and assertions; add `apiSteps` for the full click/fill/check trail), Node console tail, and trace/video links under each failed test in the defect log. Read straight from the Playwright step tree: no fixtures, no test-code changes. Enable via `capture` (`steps` / `apiSteps` / `console` / `evidence`); off by default.
 - **Configurable sections**: add or remove any report section per template via `sections`.
+- **Executive report brief band**: leads with a plain-language brief (trend + root causes) and one hero pass-rate number instead of a flat KPI grid
 - **Hybrid licensing**: one short key unlocks it; reports keep rendering offline via a cached JWT between refreshes
 
 ---
@@ -204,7 +205,7 @@ All options are the second element of the reporter tuple.
 | `flakinessTopN` | `number` | `5` | Maximum flaky tests to show in the flakiness table (`detailed` template). Set to `0` to disable the table entirely. |
 | `slowTestThreshold` | `number` | `10` | Minimum timed-test count before `SLOW` badges appear in the suite breakdown: below this, every test is trivially the "slowest" so badges stay hidden. On larger runs only genuine outliers (at least 2× the median test duration) are badged, so the badge stays rare. Set to `0` to drop the run-size gate (outliers still required). |
 | `templatePath` | `string \| string[]` | n/a | Path to a custom Handlebars (`.hbs`) template file. Takes precedence over `template`. Pass an array to generate one PDF per custom template. See the Custom Templates docs. |
-| `sections` | `object` | per-template | Override which report sections appear, on top of the chosen template's defaults. Flat keys are the baseline for every chosen template; per-template keys (`minimal`\|`detailed`\|`executive`) override per template. Block toggles: `coverPage`, `analysisOneliner`, `releaseGate`, `summary`, `charts`, `trend`, `requirementsMatrix`, `ciEnvironment`, `suiteBreakdown`, `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`. Display modifiers: `passRate`, `fullEnvironment`, `retries`, `fullFailures`, `stackTraces`. See the Report Sections docs. |
+| `sections` | `object` | per-template | Override which report sections appear, on top of the chosen template's defaults. Flat keys are the baseline for every chosen template; per-template keys (`minimal`\|`detailed`\|`executive`) override per template. Block toggles: `coverPage`, `analysisOneliner`, `releaseGate`, `summary`, `charts`, `trend`, `requirementsMatrix`, `ciEnvironment`, `suiteBreakdown`, `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`, `briefBand`. Display modifiers: `passRate`, `fullEnvironment`, `retries`, `fullFailures`, `stackTraces`. See the Report Sections docs. |
 <!-- AUTOGEN:options-table END -->
 
 Full reference: [reportforge.org/docs/configuration](https://reportforge.org/docs/configuration).

package/dist/cli/index.js

@@ -6388,6 +6388,25 @@ function buildOneLiner(clusters) {
 function dominantCategory(clusters) {
   return clusters[0]?.category ?? "unknown";
 }
+function trendClause(passRate, delta) {
+  if (delta === 0) return `Pass rate held steady at ${passRate}%.`;
+  const verb = delta > 0 ? "climbed" : "dropped";
+  const abs = Math.abs(delta);
+  const points = abs === 1 ? "point" : "points";
+  return `Pass rate ${verb} ${abs} ${points} to ${passRate}% since the last run.`;
+}
+function causesClause(analysis) {
+  const [head, ...tallyParts] = buildOneLiner(analysis.clusters).split(" \xB7 ");
+  const verb = analysis.clusters.length === 1 ? "remains" : "remain";
+  return `${head} ${verb}: ${tallyParts.join(", ")}.`;
+}
+function buildBriefSentence(stats, analysis, trend) {
+  if (trend && trend.delta !== null) {
+    const trendSentence = trendClause(stats.passRate, trend.delta);
+    return analysis && analysis.clusters.length > 0 ? `${trendSentence} ${causesClause(analysis)}` : trendSentence;
+  }
+  return analysis && analysis.clusters.length > 0 ? causesClause(analysis) : void 0;
+}
 var CATEGORY_LABEL, SHORT_LABEL;
 var init_SummaryWriter = __esm({
   "src/analysis/SummaryWriter.ts"() {
@@ -6457,6 +6476,7 @@ var init_sections = __esm({
       "failureAnalysis",
       "slowTests",
       "defectLog",
+      "briefBand",
       // display modifiers
       "passRate",
       "fullEnvironment",
@@ -6479,6 +6499,7 @@ var init_sections = __esm({
         failureAnalysis: false,
         slowTests: false,
         defectLog: false,
+        briefBand: false,
         passRate: true,
         fullEnvironment: false,
         retries: true,
@@ -6499,6 +6520,7 @@ var init_sections = __esm({
         failureAnalysis: true,
         slowTests: true,
         defectLog: true,
+        briefBand: false,
         passRate: true,
         fullEnvironment: true,
         retries: true,
@@ -6507,7 +6529,9 @@ var init_sections = __esm({
       },
       executive: {
         coverPage: true,
-        analysisOneliner: true,
+        // Now off: its content (root-cause tally) is subsumed by briefBand below,
+        // which also adds the trend delta. Showing both would say it twice.
+        analysisOneliner: false,
         releaseGate: true,
         summary: true,
         charts: true,
@@ -6522,6 +6546,7 @@ var init_sections = __esm({
         failureAnalysis: false,
         slowTests: true,
         defectLog: false,
+        briefBand: true,
         passRate: true,
         fullEnvironment: false,
         retries: false,
@@ -7325,25 +7350,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 +7686,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 +7716,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 +7761,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",
@@ -12489,6 +12527,7 @@ async function run() {
   const options = parseOptions({ template, outputFile, failureAnalysis: { collectUnclassified: false } });
   const generator = new PdfGenerator();
   runFailureAnalysis(reportData, options, process.cwd());
+  reportData.briefSentence = buildBriefSentence(reportData.stats, reportData.analysis, reportData.charts.trend);
   const paths = await generator.generateAll(reportData, options, DEMO_LICENSE_INFO2);
   if (paths.length > 0) {
     console.log(`
@@ -12529,6 +12568,7 @@ var init_cli = __esm({
     init_schema();
     init_types2();
     init_analysis();
+    init_SummaryWriter();
     DEMO_LICENSE_INFO2 = {
       plan: "subscription",
       source: "demo",

package/dist/index.d.mts

@@ -1,11 +1,11 @@
 import { Reporter, FullConfig, Suite, TestCase, TestResult, TestStep, FullResult } from '@playwright/test/reporter';
 
 /**
- * Canonical section catalog. 13 block toggles (section present/absent) followed
+ * Canonical section catalog. 14 block toggles (section present/absent) followed
  * by 5 display modifiers (tweak a section that is on). Single source of truth —
  * the Zod schema in src/config/schema.ts builds its flag shape from this array.
  */
-declare const SECTION_KEYS: readonly ["coverPage", "analysisOneliner", "releaseGate", "summary", "charts", "trend", "requirementsMatrix", "ciEnvironment", "suiteBreakdown", "failureDeepDive", "failureAnalysis", "slowTests", "defectLog", "passRate", "fullEnvironment", "retries", "fullFailures", "stackTraces"];
+declare const SECTION_KEYS: readonly ["coverPage", "analysisOneliner", "releaseGate", "summary", "charts", "trend", "requirementsMatrix", "ciEnvironment", "suiteBreakdown", "failureDeepDive", "failureAnalysis", "slowTests", "defectLog", "briefBand", "passRate", "fullEnvironment", "retries", "fullFailures", "stackTraces"];
 type SectionKey = (typeof SECTION_KEYS)[number];
 /** A partial map of section flags — used both as the flat baseline and per-template. */
 type SectionFlags = Partial<Record<SectionKey, boolean>>;
@@ -332,7 +332,7 @@ interface ReporterOptions {
      *
      * Block toggles: `coverPage`, `analysisOneliner`, `releaseGate`, `summary`,
      * `charts`, `trend`, `requirementsMatrix`, `ciEnvironment`, `suiteBreakdown`,
-     * `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`.
+     * `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`, `briefBand`.
      * Display modifiers: `passRate`, `fullEnvironment`, `retries`, `fullFailures`,
      * `stackTraces`.
      *

package/dist/index.d.ts

@@ -1,11 +1,11 @@
 import { Reporter, FullConfig, Suite, TestCase, TestResult, TestStep, FullResult } from '@playwright/test/reporter';
 
 /**
- * Canonical section catalog. 13 block toggles (section present/absent) followed
+ * Canonical section catalog. 14 block toggles (section present/absent) followed
  * by 5 display modifiers (tweak a section that is on). Single source of truth —
  * the Zod schema in src/config/schema.ts builds its flag shape from this array.
  */
-declare const SECTION_KEYS: readonly ["coverPage", "analysisOneliner", "releaseGate", "summary", "charts", "trend", "requirementsMatrix", "ciEnvironment", "suiteBreakdown", "failureDeepDive", "failureAnalysis", "slowTests", "defectLog", "passRate", "fullEnvironment", "retries", "fullFailures", "stackTraces"];
+declare const SECTION_KEYS: readonly ["coverPage", "analysisOneliner", "releaseGate", "summary", "charts", "trend", "requirementsMatrix", "ciEnvironment", "suiteBreakdown", "failureDeepDive", "failureAnalysis", "slowTests", "defectLog", "briefBand", "passRate", "fullEnvironment", "retries", "fullFailures", "stackTraces"];
 type SectionKey = (typeof SECTION_KEYS)[number];
 /** A partial map of section flags — used both as the flat baseline and per-template. */
 type SectionFlags = Partial<Record<SectionKey, boolean>>;
@@ -332,7 +332,7 @@ interface ReporterOptions {
      *
      * Block toggles: `coverPage`, `analysisOneliner`, `releaseGate`, `summary`,
      * `charts`, `trend`, `requirementsMatrix`, `ciEnvironment`, `suiteBreakdown`,
-     * `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`.
+     * `failureDeepDive`, `failureAnalysis`, `slowTests`, `defectLog`, `briefBand`.
      * Display modifiers: `passRate`, `fullEnvironment`, `retries`, `fullFailures`,
      * `stackTraces`.
      *

package/dist/index.js

@@ -10350,6 +10350,7 @@ var SECTION_KEYS = [
   "failureAnalysis",
   "slowTests",
   "defectLog",
+  "briefBand",
   // display modifiers
   "passRate",
   "fullEnvironment",
@@ -10372,6 +10373,7 @@ var SECTION_DEFAULTS = {
     failureAnalysis: false,
     slowTests: false,
     defectLog: false,
+    briefBand: false,
     passRate: true,
     fullEnvironment: false,
     retries: true,
@@ -10392,6 +10394,7 @@ var SECTION_DEFAULTS = {
     failureAnalysis: true,
     slowTests: true,
     defectLog: true,
+    briefBand: false,
     passRate: true,
     fullEnvironment: true,
     retries: true,
@@ -10400,7 +10403,9 @@ var SECTION_DEFAULTS = {
   },
   executive: {
     coverPage: true,
-    analysisOneliner: true,
+    // Now off: its content (root-cause tally) is subsumed by briefBand below,
+    // which also adds the trend delta. Showing both would say it twice.
+    analysisOneliner: false,
     releaseGate: true,
     summary: true,
     charts: true,
@@ -10415,6 +10420,7 @@ var SECTION_DEFAULTS = {
     failureAnalysis: false,
     slowTests: true,
     defectLog: false,
+    briefBand: true,
     passRate: true,
     fullEnvironment: false,
     retries: false,
@@ -11061,6 +11067,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}`}`);
@@ -11293,6 +11304,25 @@ function buildOneLiner(clusters) {
 function dominantCategory(clusters) {
   return clusters[0]?.category ?? "unknown";
 }
+function trendClause(passRate, delta) {
+  if (delta === 0) return `Pass rate held steady at ${passRate}%.`;
+  const verb = delta > 0 ? "climbed" : "dropped";
+  const abs = Math.abs(delta);
+  const points = abs === 1 ? "point" : "points";
+  return `Pass rate ${verb} ${abs} ${points} to ${passRate}% since the last run.`;
+}
+function causesClause(analysis) {
+  const [head, ...tallyParts] = buildOneLiner(analysis.clusters).split(" \xB7 ");
+  const verb = analysis.clusters.length === 1 ? "remains" : "remain";
+  return `${head} ${verb}: ${tallyParts.join(", ")}.`;
+}
+function buildBriefSentence(stats, analysis, trend) {
+  if (trend && trend.delta !== null) {
+    const trendSentence = trendClause(stats.passRate, trend.delta);
+    return analysis && analysis.clusters.length > 0 ? `${trendSentence} ${causesClause(analysis)}` : trendSentence;
+  }
+  return analysis && analysis.clusters.length > 0 ? causesClause(analysis) : void 0;
+}
 
 // src/analysis/UnclassifiedCollector.ts
 init_cjs_shims();
@@ -11619,6 +11649,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 +11735,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 +11748,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 +11761,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 +11895,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 +11909,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 +11952,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 +11975,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 +11998,6 @@ var DataCollector = class {
         case "skipped":
           skipped++;
           break;
-        case "interrupted":
-          failed++;
-          break;
       }
     }
     const duration = fullResult.duration ?? Date.now() - this.startTime;
@@ -12123,18 +12157,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 +12282,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 +12858,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 +13118,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 +13148,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 +13193,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 +14056,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.14.0" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,
@@ -14085,6 +14140,7 @@ var PdfReporter = class {
     }
     const reportData = this._buildReportData(collected, licenseInfo);
     runFailureAnalysis(reportData, this.options, this.cwd);
+    reportData.briefSentence = buildBriefSentence(reportData.stats, reportData.analysis, reportData.charts.trend);
     let pdfPaths = [];
     try {
       pdfPaths = await this.pdfGenerator.generateAll(reportData, this.options, licenseInfo);