@reportforge/playwright-pdf 0.7.0 → 0.7.1

package/README.md

@@ -198,7 +198,7 @@ All options are the second element of the reporter tuple.
 | `historySize` | `number` | `10` | Maximum number of test runs to keep in the history file (integer ≥ 2). Older runs are pruned on append. |
 | `showTrend` | `boolean` | `true` | Show the pass-rate sparkline and delta badge in the `detailed` template. Set to `false` to disable history tracking entirely. |
 | `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. The badge marks the 10 slowest tests; below this many it stays hidden (every test would be trivially "slowest"). Set to `0` to always badge the slowest. |
+| `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[]` | — | 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. |
 <!-- AUTOGEN:options-table END -->

package/dist/cli/index.js

@@ -6473,7 +6473,7 @@ function deriveOptions(s) {
     showFullFailures: s.fullFailures
   };
 }
-var import_handlebars, import_fs, import_path, TemplateEngine;
+var import_handlebars, import_fs, import_path, SLOW_MEDIAN_FACTOR, TemplateEngine;
 var init_engine = __esm({
   "src/templates/engine.ts"() {
     "use strict";
@@ -6485,6 +6485,7 @@ var init_engine = __esm({
     init_duration();
     init_SummaryWriter();
     init_sections();
+    SLOW_MEDIAN_FACTOR = 2;
     TemplateEngine = class {
       constructor() {
         this.chartjsInline = "";
@@ -6612,8 +6613,10 @@ var init_engine = __esm({
             }
           }
         }
+        const sortedDurations = tests.map((t) => t.durationMs).sort((a, b) => a - b);
+        const medianMs = sortedDurations.length ? sortedDurations[Math.floor((sortedDurations.length - 1) / 2)] : 0;
         const ranked = tests.sort((a, b) => b.durationMs - a.durationMs).slice(0, 10);
-        return { ranked, timedTotal: tests.length };
+        return { ranked, timedTotal: tests.length, medianMs };
       }
       /**
        * Project the suite tree into its render form: attach each test's failure detail
@@ -6628,7 +6631,8 @@ var init_engine = __esm({
         const chipById = sections.failureAnalysis ? new Map((data.classifications ?? []).map((c) => [c.testId, c])) : /* @__PURE__ */ new Map();
         const slowRankById = /* @__PURE__ */ new Map();
         if (sections.slowTests && slow.timedTotal > slowTestThreshold) {
-          slow.ranked.forEach((t, i) => slowRankById.set(t.id, i + 1));
+          const floorMs = slow.medianMs * SLOW_MEDIAN_FACTOR;
+          slow.ranked.filter((t) => t.durationMs >= floorMs).forEach((t, i) => slowRankById.set(t.id, i + 1));
         }
         const enrichTest = (t) => ({
           ...t,

package/dist/index.d.mts

@@ -236,9 +236,10 @@ interface ReporterOptions {
     flakinessTopN?: number;
     /**
      * Minimum number of timed tests before `SLOW` badges appear in the suite
-     * breakdown. The badge marks the 10 slowest tests; below this many timed
-     * tests every test would trivially be the "slowest", so badges stay hidden.
-     * Set to 0 to always badge the slowest. Must be a non-negative integer.
+     * 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 are still required). Must be a non-negative integer.
      * @default 10
      */
     slowTestThreshold?: number;

package/dist/index.d.ts

@@ -236,9 +236,10 @@ interface ReporterOptions {
     flakinessTopN?: number;
     /**
      * Minimum number of timed tests before `SLOW` badges appear in the suite
-     * breakdown. The badge marks the 10 slowest tests; below this many timed
-     * tests every test would trivially be the "slowest", so badges stay hidden.
-     * Set to 0 to always badge the slowest. Must be a non-negative integer.
+     * 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 are still required). Must be a non-negative integer.
      * @default 10
      */
     slowTestThreshold?: number;

package/dist/index.js

@@ -12048,6 +12048,7 @@ function jsonForInlineScript(value) {
 function isUnknownValue(v) {
   return !v || v === "unknown";
 }
+var SLOW_MEDIAN_FACTOR = 2;
 function deriveOptions(s) {
   return {
     showCharts: s.charts,
@@ -12186,8 +12187,10 @@ var TemplateEngine = class {
         }
       }
     }
+    const sortedDurations = tests.map((t) => t.durationMs).sort((a, b) => a - b);
+    const medianMs = sortedDurations.length ? sortedDurations[Math.floor((sortedDurations.length - 1) / 2)] : 0;
     const ranked = tests.sort((a, b) => b.durationMs - a.durationMs).slice(0, 10);
-    return { ranked, timedTotal: tests.length };
+    return { ranked, timedTotal: tests.length, medianMs };
   }
   /**
    * Project the suite tree into its render form: attach each test's failure detail
@@ -12202,7 +12205,8 @@ var TemplateEngine = class {
     const chipById = sections.failureAnalysis ? new Map((data.classifications ?? []).map((c) => [c.testId, c])) : /* @__PURE__ */ new Map();
     const slowRankById = /* @__PURE__ */ new Map();
     if (sections.slowTests && slow.timedTotal > slowTestThreshold) {
-      slow.ranked.forEach((t, i) => slowRankById.set(t.id, i + 1));
+      const floorMs = slow.medianMs * SLOW_MEDIAN_FACTOR;
+      slow.ranked.filter((t) => t.durationMs >= floorMs).forEach((t, i) => slowRankById.set(t.id, i + 1));
     }
     const enrichTest = (t) => ({
       ...t,
@@ -13291,7 +13295,7 @@ var PdfReporter = class {
     this.dataCollector = new DataCollector();
     this.pdfGenerator = new PdfGenerator();
     this.options = parseOptions(rawOptions);
-    const version = true ? "0.7.0" : "0.x";
+    const version = true ? "0.7.1" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/dist/index.mjs

@@ -12049,6 +12049,7 @@ function jsonForInlineScript(value) {
 function isUnknownValue(v) {
   return !v || v === "unknown";
 }
+var SLOW_MEDIAN_FACTOR = 2;
 function deriveOptions(s) {
   return {
     showCharts: s.charts,
@@ -12187,8 +12188,10 @@ var TemplateEngine = class {
         }
       }
     }
+    const sortedDurations = tests.map((t) => t.durationMs).sort((a, b) => a - b);
+    const medianMs = sortedDurations.length ? sortedDurations[Math.floor((sortedDurations.length - 1) / 2)] : 0;
     const ranked = tests.sort((a, b) => b.durationMs - a.durationMs).slice(0, 10);
-    return { ranked, timedTotal: tests.length };
+    return { ranked, timedTotal: tests.length, medianMs };
   }
   /**
    * Project the suite tree into its render form: attach each test's failure detail
@@ -12203,7 +12206,8 @@ var TemplateEngine = class {
     const chipById = sections.failureAnalysis ? new Map((data.classifications ?? []).map((c) => [c.testId, c])) : /* @__PURE__ */ new Map();
     const slowRankById = /* @__PURE__ */ new Map();
     if (sections.slowTests && slow.timedTotal > slowTestThreshold) {
-      slow.ranked.forEach((t, i) => slowRankById.set(t.id, i + 1));
+      const floorMs = slow.medianMs * SLOW_MEDIAN_FACTOR;
+      slow.ranked.filter((t) => t.durationMs >= floorMs).forEach((t, i) => slowRankById.set(t.id, i + 1));
     }
     const enrichTest = (t) => ({
       ...t,
@@ -13292,7 +13296,7 @@ var PdfReporter = class {
     this.dataCollector = new DataCollector();
     this.pdfGenerator = new PdfGenerator();
     this.options = parseOptions(rawOptions);
-    const version = true ? "0.7.0" : "0.x";
+    const version = true ? "0.7.1" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/dist/templates/layouts/detailed.hbs

@@ -18,7 +18,7 @@
 
 {{#if sections.releaseGate}}{{> release-gate}}{{/if}}
 
-{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate) }}{{/if}}
+{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate releaseGateShown=sections.releaseGate) }}{{/if}}
 
 {{#if sections.charts}}{{> charts options=(obj showCharts=true showTrend=sections.trend) }}{{/if}}
 

package/dist/templates/layouts/executive.hbs

@@ -18,7 +18,7 @@
 
 {{#if sections.releaseGate}}{{> release-gate}}{{/if}}
 
-{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate) }}{{/if}}
+{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate releaseGateShown=sections.releaseGate) }}{{/if}}
 
 {{#if sections.charts}}{{> charts options=(obj showCharts=true showTrend=sections.trend) }}{{/if}}
 

package/dist/templates/layouts/minimal.hbs

@@ -18,7 +18,7 @@
 
 {{#if sections.releaseGate}}{{> release-gate}}{{/if}}
 
-{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate) }}{{/if}}
+{{#if sections.summary}}{{> executive-summary options=(obj showPassRate=sections.passRate releaseGateShown=sections.releaseGate) }}{{/if}}
 
 {{#if sections.charts}}{{> charts options=(obj showCharts=true showTrend=sections.trend) }}{{/if}}
 

package/dist/templates/partials/charts.hbs

@@ -166,14 +166,14 @@
         y: {
           stacked: true,
           // Widen the label gutter so the longest label isn't clipped at the canvas edge.
-          afterFit: function(scale) { if (!scale.$rfPadded) { scale.$rfPadded = true; scale.width += 12; } },
+          afterFit: function(scale) { if (!scale.$rfPadded) { scale.$rfPadded = true; scale.width += 30; } },
           ticks: {
             font: { size: 9, family: 'Inter, sans-serif' }, color: '#1F1F1C', autoSkip: false,
-            // Truncate long suite / describe labels so they fit the axis gutter
-            // instead of clipping at the canvas edge.
+            // Truncate only very long suite / describe labels; the wider gutter above
+            // lets most labels show in full instead of clipping at the canvas edge.
             callback: function(value) {
               var l = this.getLabelForValue(value);
-              return l.length > 18 ? l.slice(0, 17) + '…' : l;
+              return l.length > 24 ? l.slice(0, 23) + '…' : l;
             }
           },
           grid:   { display: false },
@@ -215,11 +215,17 @@
 
   // Smart y-axis: pad above/below the actual data range so variation is readable
   // rather than a flat line at the top of a 0–100 axis when all runs pass at 96%.
+  // Pick the step FIRST, then snap the bounds to multiples of that step, so every
+  // tick (min, min+step, … max) is a round value — never 45% / 70% from a min that
+  // isn't itself a multiple of the step. Clamps to 0/100 keep the bounds round too.
   var minRate = Math.min.apply(null, trendRates);
   var maxRate = Math.max.apply(null, trendRates);
   var yPad    = Math.max(8, Math.round((maxRate - minRate) * 0.4));
-  var yMin    = Math.max(0,   minRate - yPad);
-  var yMax    = Math.min(100, maxRate + yPad);
+  var rawMin  = Math.max(0,   minRate - yPad);
+  var rawMax  = Math.min(100, maxRate + yPad);
+  var yStep   = (rawMax - rawMin) <= 20 ? 5 : (rawMax - rawMin) <= 50 ? 10 : 25;
+  var yMin    = Math.max(0,   Math.floor(rawMin / yStep) * yStep);
+  var yMax    = Math.min(100, Math.ceil(rawMax / yStep) * yStep);
 
   // Data-label plugin: renders pass-rate % above each point.
   // Canvas ID guard prevents firing on the doughnut and bar charts.
@@ -279,7 +285,7 @@
             callback: function(v) { return v + '%'; },
             font:  { size: 8.5, family: 'Inter, sans-serif' },
             color: '#8C8B82',
-            count: 5,
+            stepSize: yStep,
           },
           grid:   { color: 'rgba(31,31,28,0.07)' },
           border: { color: 'transparent' },

package/dist/templates/partials/executive-summary.hbs

@@ -1,7 +1,13 @@
 <div class="section">
+  {{!-- The release gate (when the layout renders it) already states the verdict
+        prominently, so the chip would just repeat it — suppress it then. The layout
+        passes options.releaseGateShown; it is absent for custom templates, so the
+        chip renders by default there (a custom template may not include a gate). --}}
+  {{#unless options.releaseGateShown}}
   <span class="verdict verdict--{{stats.verdict}}">
     {{verdictLabel stats.verdict}}
   </span>
+  {{/unless}}
 
   <div class="kpi-strip">
     <div class="kpi-card kpi-card--total">
@@ -15,19 +21,21 @@
       <div class="kpi-card__rate">{{stats.passRate}}%</div>
       {{/if}}
     </div>
-    <div class="kpi-card kpi-card--fail">
+    {{!-- Zero-value status cards are dimmed: a coloured "0" for an absent status
+          draws the eye to nothing and dilutes the counts that matter. --}}
+    <div class="kpi-card kpi-card--fail{{#unless stats.failed}} kpi-card--zero{{/unless}}">
       <div class="kpi-card__value">{{stats.failed}}</div>
       <div class="kpi-card__label">Failed</div>
     </div>
-    <div class="kpi-card kpi-card--timedout">
+    <div class="kpi-card kpi-card--timedout{{#unless stats.timedOut}} kpi-card--zero{{/unless}}">
       <div class="kpi-card__value">{{stats.timedOut}}</div>
       <div class="kpi-card__label">Timed Out</div>
     </div>
-    <div class="kpi-card kpi-card--skip">
+    <div class="kpi-card kpi-card--skip{{#unless stats.skipped}} kpi-card--zero{{/unless}}">
       <div class="kpi-card__value">{{stats.skipped}}</div>
       <div class="kpi-card__label">Skipped</div>
     </div>
-    <div class="kpi-card kpi-card--flaky">
+    <div class="kpi-card kpi-card--flaky{{#unless stats.flaky}} kpi-card--zero{{/unless}}">
       <div class="kpi-card__value">{{stats.flaky}}</div>
       <div class="kpi-card__label">Flaky</div>
     </div>
@@ -38,10 +46,16 @@
       <span class="summary-row__label">Duration</span>
       <span class="summary-row__value">{{formatDuration stats.duration}}</span>
     </div>
+    {{!-- Pass rate normally shows on the Passed KPI card and the doughnut centre, so
+          it is omitted here to avoid a third restatement on one page. Fallback: if
+          BOTH of those are off (passRate modifier off AND charts off, e.g. a stripped
+          minimal), show it here so the headline metric never disappears entirely. --}}
+    {{#unless (or options.showPassRate @root.sections.charts)}}
     <div class="summary-row__item">
       <span class="summary-row__label">Pass Rate</span>
       <span class="summary-row__value" style="color:var(--pass);">{{stats.passRate}}%</span>
     </div>
+    {{/unless}}
     <div class="summary-row__item">
       <span class="summary-row__label">Generated</span>
       <span class="summary-row__value">{{formatDate meta.generatedAt}}</span>

package/dist/templates/partials/suite-breakdown.hbs

@@ -39,7 +39,7 @@
     </thead>
     <tbody>
       {{#each tests}}
-      <tr class="no-break">
+      <tr class="no-break{{#if failure}} test-row--has-detail{{/if}}">
         <td>
           <div style="font-weight:500;">{{title}}</div>
           {{#if location}}<div class="test-path">{{location}}</div>{{/if}}

package/dist/templates/styles/base.css

@@ -153,6 +153,12 @@ tr:last-child td { border-bottom: none; }
 
 /* ── Page breaks ────────────────────────────────────────────────────────── */
 .no-break { page-break-inside: avoid; }
+/* Keep a failed test row glued to its inline failure detail: the detail block is
+   page-break-inside:avoid, so when it can't fit the remaining space it moves to the
+   next page — these pull the test row along so the title never orphans above it. */
+.test-row--has-detail { page-break-after: avoid; break-after: avoid; }
+.test-row--has-detail > td { border-bottom: none; }
+.failure-row { page-break-before: avoid; break-before: avoid; }
 
 /* ── Section wrapper ────────────────────────────────────────────────────── */
 /* Large sections (suite breakdown, failures, tables) MAY flow across pages —
@@ -664,6 +670,11 @@ code {
   print-color-adjust: exact;
 }
 
+/* Dimmed zero-value KPI card — a coloured "0" for an absent status (no timeouts,
+   no flakes) draws the eye to nothing; muting it lets the real counts lead. Shared
+   modifier so every template's kpi-strip gets it. */
+.kpi-card--zero { opacity: 0.42; }
+
 /* ── Suite breakdown — shared so any template (incl. executive) can toggle it on ─ */
 /* Values match the detailed template; minimal.css keeps its own tuned overrides
    for .suite-group (margin) and .suite-stats (gap), which win via templateCSS order. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reportforge/playwright-pdf",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Enterprise-ready PDF reports for Playwright Test — minimal, detailed, and executive templates with CI/CD integrations",
   "license": "MIT",
   "author": "ReportForge",