@reportforge/playwright-pdf 0.8.0 → 0.8.1

package/dist/index.js

@@ -11407,6 +11407,7 @@ var DEFAULT_MAX_STEPS = 50;
 var DEFAULT_MAX_CONSOLE_LINES = 50;
 var PW_API_WINDOW = 10;
 var isTeardown = (category) => category === "hook" || category === "fixture";
+var inHookSubtree = (flat, f) => isTeardown(f.category) || f.ancestors.some((a) => isTeardown(flat[a].category));
 function flatten(steps) {
   const out = [];
   const walk = (nodes, depth, ancestors) => {
@@ -11425,7 +11426,7 @@ function flatten(steps) {
   walk(steps ?? [], 0, []);
   return out;
 }
-function materialize(n) {
+function materialize(n, isFailing) {
   const loc = n.raw.location;
   return {
     title: clampWithEllipsis(n.raw.title ?? "", MAX_TITLE_CHARS),
@@ -11433,17 +11434,16 @@ function materialize(n) {
     durationMs: Math.max(0, Math.round(n.raw.duration ?? 0)),
     depth: n.depth,
     failed: n.hasError,
-    errorMessage: n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
+    errorMessage: isFailing && n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
     location: loc?.file != null ? `${loc.file}:${loc.line ?? 0}` : void 0
   };
 }
 function pickFailingIdx(flat) {
-  const inTeardown = (f) => isTeardown(f.category) || f.ancestors.some((a) => isTeardown(flat[a].category));
   const pick = (allowTeardown) => {
     let idx = -1;
     for (let i = 0; i < flat.length; i++) {
       const f = flat[i];
-      if (!f.hasError || !allowTeardown && inTeardown(f)) continue;
+      if (!f.hasError || !allowTeardown && inHookSubtree(flat, f)) continue;
       if (idx === -1 || f.depth >= flat[idx].depth) idx = i;
     }
     return idx;
@@ -11466,7 +11466,7 @@ function compactSteps(rawSteps, opts = {}) {
     if (opts.apiSteps) {
       const pwApiBefore = [];
       for (let i = 0; i < failingIdx; i++) {
-        if (flat[i].category === "pw:api") pwApiBefore.push(i);
+        if (flat[i].category === "pw:api" && !inHookSubtree(flat, flat[i])) pwApiBefore.push(i);
       }
       for (const i of pwApiBefore.slice(-PW_API_WINDOW)) keep.add(i);
     }
@@ -11482,7 +11482,7 @@ function compactSteps(rawSteps, opts = {}) {
   const mat = (i) => {
     let es = cache.get(i);
     if (!es) {
-      es = materialize(flat[i]);
+      es = materialize(flat[i], i === failingIdx);
       cache.set(i, es);
     }
     return es;
@@ -13495,7 +13495,7 @@ var PdfReporter = class {
     this.pdfGenerator = new PdfGenerator();
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.8.0" : "0.x";
+    const version = true ? "0.8.1" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/dist/index.mjs

@@ -11408,6 +11408,7 @@ var DEFAULT_MAX_STEPS = 50;
 var DEFAULT_MAX_CONSOLE_LINES = 50;
 var PW_API_WINDOW = 10;
 var isTeardown = (category) => category === "hook" || category === "fixture";
+var inHookSubtree = (flat, f) => isTeardown(f.category) || f.ancestors.some((a) => isTeardown(flat[a].category));
 function flatten(steps) {
   const out = [];
   const walk = (nodes, depth, ancestors) => {
@@ -11426,7 +11427,7 @@ function flatten(steps) {
   walk(steps ?? [], 0, []);
   return out;
 }
-function materialize(n) {
+function materialize(n, isFailing) {
   const loc = n.raw.location;
   return {
     title: clampWithEllipsis(n.raw.title ?? "", MAX_TITLE_CHARS),
@@ -11434,17 +11435,16 @@ function materialize(n) {
     durationMs: Math.max(0, Math.round(n.raw.duration ?? 0)),
     depth: n.depth,
     failed: n.hasError,
-    errorMessage: n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
+    errorMessage: isFailing && n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
     location: loc?.file != null ? `${loc.file}:${loc.line ?? 0}` : void 0
   };
 }
 function pickFailingIdx(flat) {
-  const inTeardown = (f) => isTeardown(f.category) || f.ancestors.some((a) => isTeardown(flat[a].category));
   const pick = (allowTeardown) => {
     let idx = -1;
     for (let i = 0; i < flat.length; i++) {
       const f = flat[i];
-      if (!f.hasError || !allowTeardown && inTeardown(f)) continue;
+      if (!f.hasError || !allowTeardown && inHookSubtree(flat, f)) continue;
       if (idx === -1 || f.depth >= flat[idx].depth) idx = i;
     }
     return idx;
@@ -11467,7 +11467,7 @@ function compactSteps(rawSteps, opts = {}) {
     if (opts.apiSteps) {
       const pwApiBefore = [];
       for (let i = 0; i < failingIdx; i++) {
-        if (flat[i].category === "pw:api") pwApiBefore.push(i);
+        if (flat[i].category === "pw:api" && !inHookSubtree(flat, flat[i])) pwApiBefore.push(i);
       }
       for (const i of pwApiBefore.slice(-PW_API_WINDOW)) keep.add(i);
     }
@@ -11483,7 +11483,7 @@ function compactSteps(rawSteps, opts = {}) {
   const mat = (i) => {
     let es = cache.get(i);
     if (!es) {
-      es = materialize(flat[i]);
+      es = materialize(flat[i], i === failingIdx);
       cache.set(i, es);
     }
     return es;
@@ -13496,7 +13496,7 @@ var PdfReporter = class {
     this.pdfGenerator = new PdfGenerator();
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.8.0" : "0.x";
+    const version = true ? "0.8.1" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,

package/dist/templates/partials/defect-log.hbs

@@ -11,8 +11,8 @@
         <th style="width:72px;text-align:right;">Duration</th>
       </tr>
     </thead>
-    <tbody>
       {{#each failures}}
+      <tbody class="defect-row-group">
       <tr class="no-break">
         <td class="defect-id">DEF-{{addOne @index}}</td>
         <td style="font-weight:500;">{{testTitle}}</td>
@@ -24,9 +24,14 @@
       <tr class="defect-detail-row">
         <td colspan="5">
           <div class="defect-detail">
+            {{! Error sits at the failing step inside the trail. Only when there is no
+                identified failing step (capture.steps off, or no errored step) is it
+                shown here under the title as a fallback. }}
+            {{#unless execution.failingStep}}
             {{#if error.message}}
             <div class="defect-detail__error">{{error.message}}</div>
             {{/if}}
+            {{/unless}}
 
             {{#if execution.steps.length}}
             <div class="repro">
@@ -38,6 +43,7 @@
                   <span class="repro-step__title">{{title}}</span>
                   {{#if failed}}{{#if location}}<span class="repro-step__loc">{{location}}</span>{{/if}}{{/if}}
                   <span class="repro-step__dur">{{formatDuration durationMs}}</span>
+                  {{#if errorMessage}}<div class="repro-step__error">{{../error.message}}</div>{{/if}}
                 </li>
                 {{/each}}
               </ol>
@@ -64,8 +70,8 @@
         </td>
       </tr>
       {{/if}}
+      </tbody>
       {{/each}}
-    </tbody>
   </table>
 </div>
 {{/if}}

package/dist/templates/styles/base.css

@@ -409,6 +409,9 @@ code {
    design-system-consistent: a light bordered card (NO side rail, per the
    side-stripe ban in DESIGN.md), with hairline dividers between sub-blocks —
    mirroring the reworked .failure-detail and .retry-history patterns. */
+/* Each failure is its own <tbody> so its summary row and detail card stay together
+   across a page break — the detail no longer strands under a repeated header. */
+.defect-row-group { break-inside: avoid; page-break-inside: avoid; }
 .defect-detail-row td { padding: 0; border-top: none; }
 .defect-detail {
   border: 1px solid var(--border-2);
@@ -435,12 +438,19 @@ code {
 .repro { margin-top: 11px; padding-top: 10px; border-top: 1px solid var(--border); }
 .repro-steps { list-style: none; margin: 6px 0 0; padding: 0; }
 .repro-step {
-  display: flex; align-items: baseline; gap: 8px;
+  display: flex; align-items: baseline; flex-wrap: wrap; gap: 8px;
   /* indent by tree depth (capped) — the value comes from `--d` on the element */
   padding: 2px 0 2px min(60px, calc(var(--d, 0) * 12px));
   font-size: 9px; line-height: 1.5;
   page-break-inside: avoid; break-inside: avoid;
 }
+/* The error sits at the failing step — full width below its row, indented past the
+   category column so it reads as belonging to that step. */
+.repro-step__error {
+  flex-basis: 100%; margin: 3px 0 2px; padding-left: 60px;
+  font-family: 'JetBrains Mono', monospace; font-size: 9px; line-height: 1.55;
+  color: #8A2E1E; white-space: pre-wrap; word-break: break-word;
+}
 .repro-step__cat {
   flex: none; min-width: 52px;
   font-family: 'JetBrains Mono', monospace; font-size: 8px; color: var(--text-3);

package/package.json

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