@reportforge/playwright-pdf 0.11.2 → 0.13.0

package/README.md

@@ -55,11 +55,11 @@ Three templates, one reporter. Every PDF is fully offline — fonts, charts, and
 - **CI/CD snippets** — GitHub Actions, GitLab CI, Bitbucket Pipelines, Jenkins, Azure DevOps
 - **Live runs** — opt-in `live` streams per-test progress to an unguessable watch link printed at run start; all shards of one CI run converge on a single live dashboard while tests execute. The PDF is still produced at the end, unchanged
 - **Shard merging** — combine N Playwright JSON shard reports into one PDF via `shardResults`; accepts glob patterns or explicit paths
-- **Notifications** — post pass/fail summaries to Slack, Teams, Discord, or email after each run; configurable trigger (`always` / `failure` / `success`) per channel; email supports optional PDF attachment via `attachPdf`
+- **Notifications** — post pass/fail summaries to Slack, Teams, Discord, or email after each run; configurable trigger (`always` / `failure` / `success`) per channel; email and Discord support optional PDF attachment via `attachPdf`
 - **Flakiness trend** — top-N flakiest tests table with per-test dot sparkline across stored history runs in the `detailed` template; `flakinessTopN` option (default 5, `0` disables)
 - **Offline failure analysis** — sorts each failure into one of 7 root-cause buckets and renders a per-test root-cause chip inline under the failed test in the `detailed` PDF (dominant-cause one-liner in `executive`). No network, no AI service — a small embedded classifier that updates itself safely over your existing license refresh; pin it any time. Full details at [reportforge.org/docs/advanced/failure-analysis](https://reportforge.org/docs/advanced/failure-analysis).
 - `npx @reportforge/playwright-pdf export-feedback` — exports locally-collected unrecognized failures (tokenized, local-only) for optional manual sharing.
-- **Rich execution capture** — opt-in repro steps, 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` / `console` / `evidence`); off by default.
+- **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 action 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`.
 - **Hybrid licensing** — one short key unlocks it; reports keep rendering offline via a cached JWT between refreshes
 
@@ -194,9 +194,9 @@ All options are the second element of the reporter tuple.
 | `maxInlineFailures` | `number` | derived from `compressionLevel` | Cap on failure entries rendered inline in the PDF. Overflow is written to a sibling `{basename}-failures.json` sidecar file. |
 | `maxFileSizeMb` | `number` | `8` | Soft cap on the final PDF size in MB. If exceeded, the report is re-rendered once with the `'max'` compression preset. |
 | `shardResults` | `string \| string[]` | — | Glob or path array of Playwright JSON shard report files to merge into one PDF. See the Shard Merging docs. |
-| `notify` | `object` | — | Notification channels: `slack`, `teams`, `discord` (each `{ url, enabled, on }`), `email` (`{ to, enabled, on, attachPdf }`). See the Notifications docs. |
+| `notify` | `object` | — | Notification channels: `slack`, `teams` (each `{ url, enabled, on }`), `discord` (`{ url, enabled, on, attachPdf }`), `email` (`{ to, enabled, on, attachPdf }`). See the Notifications docs. |
 | `failureAnalysis` | `object` | `{ enabled: true }` | Offline failure root-cause analysis (embedded Naive Bayes; no runtime network). Fields: `enabled` (default `true`), `maxClusters` (default `10`), `minStrength` (`weak`\|`moderate`\|`strong`, default `weak`), `maxFailuresToAnalyse` (default `500`), `collectUnclassified` (default `true`), `autoUpdateModel` (default `true`). See the Failure Analysis docs. |
-| `capture` | `object` | `{}` (off) | Opt-in rich execution capture for the defect log (reporter-side; no fixtures, no test-code changes). Fields: `steps` (compacted repro-steps trail from the Playwright step tree, default `false`), `apiSteps` (include granular `pw:api` click/fill/goto actions in the trail; off by default the trail shows only `test.step` intent + `expect` assertions + the failing action, default `false`), `console` (Node stdout/stderr tail, default `false`), `evidence` (trace/video file links, default `false`), `maxSteps` (default `50`), `maxConsoleLines` (default `50`). Renders in the Defect Log section of the `detailed` template. |
+| `capture` | `object` | `{}` (off) | Opt-in rich execution capture for the defect log (reporter-side; no fixtures, no test-code changes). Fields: `steps` (a copy-pasteable "Steps to Reproduce" outline built from the Playwright step tree — each row a Markdown list line indented under its parent `test.step`, default `false`), `apiSteps` (include every top-level `pw:api` action (click/fill/check/goto) in the outline for a full action trail; off by default the outline shows only `test.step` intent + `expect` assertions, default `false`), `console` (Node stdout/stderr tail, default `false`), `evidence` (trace/video file links, default `false`), `maxSteps` (default `50`), `maxConsoleLines` (default `50`). Renders in the Defect Log section of the `detailed` template. |
 | `live` | `object` | `{}` (off) | Opt-in live test-execution streaming. When `enabled`, the reporter prints an unguessable watch link to the CI logs at run start and streams per-test progress to the hosted dashboard while tests run — all shards of one CI run converge on a single live view. Fields: `enabled` (default `false`), `runId` (override the auto-derived run id), `serverUrl` (override the streaming server), `steps` (`none`\|`failed`\|`intent`\|`all`, default `failed`; `intent` shows only your `test.step` names plus any failing step), `console` (stream stdout/stderr tails, default `false`), `flushMs` (batch debounce 500–10000, default `2000`). The PDF at the end of the run is unaffected. Requires an active subscription with the `live` entitlement. See the Live Runs docs. |
 | `historyFile` | `string` | `~/.reportforge/{key}/history.json` | Path to the history JSON file. Relative paths resolve from `cwd`. Enables pass-rate trending charts in the `detailed` template. |
 | `historySize` | `number` | `10` | Maximum number of test runs to keep in the history file (integer ≥ 2). Older runs are pruned on append. |
@@ -282,6 +282,7 @@ notify: {
     url: process.env.DISCORD_WEBHOOK_URL,
     enabled: true,
     on: 'failure',
+    attachPdf: true,             // upload the PDF with the message (default: false)
   },
   email: {
     to: ['team@company.com', 'qa@company.com'],
@@ -304,6 +305,8 @@ The `on` trigger controls when the message fires:
 
 **Email** requires `RESEND_API_KEY` in the environment (get one at [resend.com](https://resend.com)). Sender defaults to `noreply@reportforge.org`; override with `RESEND_FROM`. Store webhook URLs and API keys as CI secrets — never commit them.
 
+**PDF attachment** (`attachPdf: true`) is available on `email` and `discord` — Slack and Teams webhooks cannot carry file uploads. On Discord the PDF is uploaded with the message; if the upload fails or the PDF exceeds Discord's file-size cap, the summary still posts without the attachment. With multiple templates, the first PDF is attached.
+
 The summary includes pass rate, test counts, duration, and the report filename. Notifications require a valid license and fire after PDF generation (or after a PDF failure — you still get the ping).
 
 ### Test History Trending

package/dist/cli/index.js

@@ -139,7 +139,37 @@ function buildDemoFailures() {
         { attempt: 2, status: "failed", duration: 4900 }
       ],
       durationMs: 4900,
-      severity: "critical"
+      severity: "critical",
+      // A test with no test.step → a flat repro outline (every step depth 0).
+      execution: {
+        source: "reporter",
+        steps: [
+          { title: "Navigate to '/login'", category: "pw:api", durationMs: 420, depth: 0, failed: false },
+          { title: "Fill getByLabel('Email')", category: "pw:api", durationMs: 60, depth: 0, failed: false },
+          { title: "Fill getByLabel('Password')", category: "pw:api", durationMs: 55, depth: 0, failed: false },
+          { title: "Click getByRole('button', { name: 'Sign in' }) \u2014 repeated 5\xD7", category: "pw:api", durationMs: 2100, depth: 0, failed: false },
+          {
+            title: "Expect response status toBe 429",
+            category: "expect",
+            durationMs: 80,
+            depth: 0,
+            failed: true,
+            failingLeaf: true,
+            errorMessage: "expect(received).toBe(expected) \u2014 Expected: 429  Received: 200",
+            location: "tests/auth.spec.ts:79"
+          }
+        ],
+        failingStep: {
+          title: "Expect response status toBe 429",
+          category: "expect",
+          durationMs: 80,
+          depth: 0,
+          failed: true,
+          failingLeaf: true,
+          errorMessage: "expect(received).toBe(expected) \u2014 Expected: 429  Received: 200",
+          location: "tests/auth.spec.ts:79"
+        }
+      }
     },
     {
       testId: "co-3",
@@ -155,7 +185,38 @@ function buildDemoFailures() {
         { attempt: 2, status: "failed", duration: 4900 }
       ],
       durationMs: 4900,
-      severity: "high"
+      severity: "high",
+      // Nested outline — child actions/assertions indent under their parent test.step.
+      execution: {
+        source: "reporter",
+        steps: [
+          { title: "Add product to cart", category: "test.step", durationMs: 1200, depth: 0, failed: false },
+          { title: "Click getByRole('button', { name: 'Add to cart' })", category: "pw:api", durationMs: 340, depth: 1, failed: false },
+          { title: "Apply expired promo code", category: "test.step", durationMs: 5600, depth: 0, failed: false },
+          { title: "Fill getByLabel('Promo code')", category: "pw:api", durationMs: 120, depth: 1, failed: false },
+          { title: "Click getByRole('button', { name: 'Apply' })", category: "pw:api", durationMs: 90, depth: 1, failed: false },
+          {
+            title: "Expect getByText('Promo code is invalid') to be visible",
+            category: "expect",
+            durationMs: 5001,
+            depth: 1,
+            failed: true,
+            failingLeaf: true,
+            errorMessage: "Timed out 5000ms waiting for expect(locator).toBeVisible()",
+            location: "tests/checkout.spec.ts:58"
+          }
+        ],
+        failingStep: {
+          title: "Expect getByText('Promo code is invalid') to be visible",
+          category: "expect",
+          durationMs: 5001,
+          depth: 1,
+          failed: true,
+          failingLeaf: true,
+          errorMessage: "Timed out 5000ms waiting for expect(locator).toBeVisible()",
+          location: "tests/checkout.spec.ts:58"
+        }
+      }
     },
     {
       testId: "cat-7",
@@ -171,7 +232,35 @@ function buildDemoFailures() {
         { attempt: 2, status: "failed", duration: 3900 }
       ],
       durationMs: 3900,
-      severity: "medium"
+      severity: "medium",
+      execution: {
+        source: "reporter",
+        steps: [
+          { title: "Scroll to lazy-loaded section", category: "test.step", durationMs: 900, depth: 0, failed: false },
+          { title: "Mouse wheel by (0, 2000)", category: "pw:api", durationMs: 300, depth: 1, failed: false },
+          { title: "Verify lazy image renders", category: "test.step", durationMs: 5e3, depth: 0, failed: false },
+          {
+            title: "Expect getByTestId('lazy-img-12') to be visible",
+            category: "expect",
+            durationMs: 5e3,
+            depth: 1,
+            failed: true,
+            failingLeaf: true,
+            errorMessage: "expect(locator).toBeVisible() \u2014 Expected: visible  Received: hidden",
+            location: "tests/catalog.spec.ts:112"
+          }
+        ],
+        failingStep: {
+          title: "Expect getByTestId('lazy-img-12') to be visible",
+          category: "expect",
+          durationMs: 5e3,
+          depth: 1,
+          failed: true,
+          failingLeaf: true,
+          errorMessage: "expect(locator).toBeVisible() \u2014 Expected: visible  Received: hidden",
+          location: "tests/catalog.spec.ts:112"
+        }
+      }
     }
   ];
 }
@@ -6670,6 +6759,7 @@ var init_engine = __esm({
       }
       registerHelpers() {
         this.handlebars.registerHelper("formatDuration", (ms) => formatDuration(ms ?? 0));
+        this.handlebars.registerHelper("repeat", (s, n) => typeof s === "string" ? s.repeat(Math.max(0, n | 0)) : "");
         this.handlebars.registerHelper("formatDate", (d) => {
           const date = d instanceof Date ? d : new Date(d);
           return date.toLocaleString("en-US", {
@@ -11831,7 +11921,7 @@ ${issues}`);
   }
   return result.data;
 }
-var hexColor, channelConfigSchema, emailChannelConfigSchema, notifyConfigSchema, failureAnalysisConfigSchema, captureConfigSchema, liveConfigSchema, sectionFlagsShape, sectionFlagsSchema, sectionsSchema, ReporterOptionsSchema;
+var hexColor, channelConfigSchema, discordChannelConfigSchema, emailChannelConfigSchema, notifyConfigSchema, failureAnalysisConfigSchema, captureConfigSchema, liveConfigSchema, sectionFlagsShape, sectionFlagsSchema, sectionsSchema, ReporterOptionsSchema;
 var init_schema = __esm({
   "src/config/schema.ts"() {
     "use strict";
@@ -11848,6 +11938,9 @@ var init_schema = __esm({
       on: external_exports.enum(["always", "failure", "success"]).default("always"),
       enabled: external_exports.boolean().default(false)
     });
+    discordChannelConfigSchema = channelConfigSchema.extend({
+      attachPdf: external_exports.boolean().default(false)
+    });
     emailChannelConfigSchema = external_exports.object({
       to: external_exports.array(external_exports.string().email('Each "to" entry must be a valid email')).min(1, '"to" must have at least one address'),
       on: external_exports.enum(["always", "failure", "success"]).default("always"),
@@ -11857,7 +11950,7 @@ var init_schema = __esm({
     notifyConfigSchema = external_exports.object({
       slack: channelConfigSchema.optional(),
       teams: channelConfigSchema.optional(),
-      discord: channelConfigSchema.optional(),
+      discord: discordChannelConfigSchema.optional(),
       email: emailChannelConfigSchema.optional()
     }).optional();
     failureAnalysisConfigSchema = external_exports.object({

package/dist/index.d.mts

@@ -158,7 +158,12 @@ interface ReporterOptions {
      * `on` controls the trigger: `'always'` (default), `'failure'`, or `'success'`.
      *
      * - `slack` / `teams` / `discord`: webhook URL channels
+     * - `discord`: `attachPdf: true` uploads the PDF with the message (falls back
+     *   to the summary alone if the upload fails or the PDF exceeds Discord's cap)
      * - `email`: sends via Resend; `to` is required; `attachPdf: true` attaches the PDF
+     *
+     * Slack and Teams webhooks cannot carry file uploads, so `attachPdf` does not
+     * exist on those channels.
      */
     notify?: {
         slack?: {
@@ -175,6 +180,7 @@ interface ReporterOptions {
             url: string;
             on?: 'always' | 'failure' | 'success';
             enabled?: boolean;
+            attachPdf?: boolean;
         };
         email?: {
             to: string[];
@@ -223,9 +229,10 @@ interface ReporterOptions {
         /** Capture a compacted repro-steps trail from the Playwright step tree. Default: false */
         steps?: boolean;
         /**
-         * Include granular `pw:api` actions (click/fill/goto) in the repro trail. Off by
-         * default — the trail shows only `test.step` intent + `expect` assertions + the
-         * failing action. Default: false
+         * Include every top-level `pw:api` action (click/fill/check/goto) in the repro
+         * outline — a full, copy-pasteable action trail across the whole test, nested under
+         * its parent `test.step`. Off by default, so the outline shows only `test.step`
+         * intent + `expect` assertions. Default: false
          */
         apiSteps?: boolean;
         /** Capture the tail of the test's Node stdout/stderr. Default: false */

package/dist/index.d.ts

@@ -158,7 +158,12 @@ interface ReporterOptions {
      * `on` controls the trigger: `'always'` (default), `'failure'`, or `'success'`.
      *
      * - `slack` / `teams` / `discord`: webhook URL channels
+     * - `discord`: `attachPdf: true` uploads the PDF with the message (falls back
+     *   to the summary alone if the upload fails or the PDF exceeds Discord's cap)
      * - `email`: sends via Resend; `to` is required; `attachPdf: true` attaches the PDF
+     *
+     * Slack and Teams webhooks cannot carry file uploads, so `attachPdf` does not
+     * exist on those channels.
      */
     notify?: {
         slack?: {
@@ -175,6 +180,7 @@ interface ReporterOptions {
             url: string;
             on?: 'always' | 'failure' | 'success';
             enabled?: boolean;
+            attachPdf?: boolean;
         };
         email?: {
             to: string[];
@@ -223,9 +229,10 @@ interface ReporterOptions {
         /** Capture a compacted repro-steps trail from the Playwright step tree. Default: false */
         steps?: boolean;
         /**
-         * Include granular `pw:api` actions (click/fill/goto) in the repro trail. Off by
-         * default — the trail shows only `test.step` intent + `expect` assertions + the
-         * failing action. Default: false
+         * Include every top-level `pw:api` action (click/fill/check/goto) in the repro
+         * outline — a full, copy-pasteable action trail across the whole test, nested under
+         * its parent `test.step`. Off by default, so the outline shows only `test.step`
+         * intent + `expect` assertions. Default: false
          */
         apiSteps?: boolean;
         /** Capture the tail of the test's Node stdout/stderr. Default: false */

package/dist/index.js

@@ -10454,6 +10454,9 @@ var channelConfigSchema = external_exports.object({
   on: external_exports.enum(["always", "failure", "success"]).default("always"),
   enabled: external_exports.boolean().default(false)
 });
+var discordChannelConfigSchema = channelConfigSchema.extend({
+  attachPdf: external_exports.boolean().default(false)
+});
 var emailChannelConfigSchema = external_exports.object({
   to: external_exports.array(external_exports.string().email('Each "to" entry must be a valid email')).min(1, '"to" must have at least one address'),
   on: external_exports.enum(["always", "failure", "success"]).default("always"),
@@ -10463,7 +10466,7 @@ var emailChannelConfigSchema = external_exports.object({
 var notifyConfigSchema = external_exports.object({
   slack: channelConfigSchema.optional(),
   teams: channelConfigSchema.optional(),
-  discord: channelConfigSchema.optional(),
+  discord: discordChannelConfigSchema.optional(),
   email: emailChannelConfigSchema.optional()
 }).optional();
 var failureAnalysisConfigSchema = external_exports.object({
@@ -11435,9 +11438,13 @@ init_cjs_shims();
 var MAX_TITLE_CHARS = 200;
 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));
+var isTopLevel = (flat, f) => !f.ancestors.some((a) => {
+  const c = flat[a].category;
+  return c === "pw:api" || c === "expect";
+});
+var testStepDepth = (flat, f) => f.ancestors.reduce((n, a) => n + (flat[a].category === "test.step" ? 1 : 0), 0);
 function flatten(steps) {
   const out = [];
   const walk = (nodes, depth, ancestors) => {
@@ -11456,15 +11463,19 @@ function flatten(steps) {
   walk(steps ?? [], 0, []);
   return out;
 }
-function materialize(n, isFailing) {
+function materialize(n, isFailing, depth) {
   const loc = n.raw.location;
+  const firstLine = isFailing && n.raw.error?.message ? stripAnsi(n.raw.error.message).split("\n").map((s) => s.trim()).find((s) => s.length) ?? "" : "";
   return {
     title: clampWithEllipsis(n.raw.title ?? "", MAX_TITLE_CHARS),
     category: n.category,
     durationMs: Math.max(0, Math.round(n.raw.duration ?? 0)),
-    depth: n.depth,
+    depth,
     failed: n.hasError,
-    errorMessage: isFailing && n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
+    // Marks the single failing leaf so the template renders the error there once —
+    // independent of whether a step-level message survives (falls back to the record error).
+    failingLeaf: isFailing || void 0,
+    errorMessage: firstLine ? clampWithEllipsis(firstLine, MAX_TITLE_CHARS) : void 0,
     location: loc?.file != null ? `${loc.file}:${loc.line ?? 0}` : void 0
   };
 }
@@ -11488,21 +11499,19 @@ function compactSteps(rawSteps, opts = {}) {
   const failingIdx = pickFailingIdx(flat);
   const keep = /* @__PURE__ */ new Set();
   flat.forEach((f, i) => {
-    if (f.category === "test.step" || f.category === "expect") keep.add(i);
+    if (inHookSubtree(flat, f)) return;
+    if (f.category === "test.step") keep.add(i);
+    else if (f.category === "expect" && isTopLevel(flat, f)) keep.add(i);
+    else if (opts.apiSteps && f.category === "pw:api" && isTopLevel(flat, f)) keep.add(i);
   });
   if (failingIdx >= 0) {
     keep.add(failingIdx);
     for (const a of flat[failingIdx].ancestors) keep.add(a);
-    if (opts.apiSteps) {
-      const pwApiBefore = [];
-      for (let i = 0; i < failingIdx; 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);
-    }
   }
   let keepArr = [...keep].sort((a, b) => a - b);
+  let omitted = 0;
   if (keepArr.length > maxSteps) {
+    omitted = keepArr.length - maxSteps;
     keepArr = keepArr.slice(-maxSteps);
     if (failingIdx >= 0 && !keepArr.includes(failingIdx)) {
       keepArr = [failingIdx, ...keepArr.slice(1)];
@@ -11512,13 +11521,23 @@ function compactSteps(rawSteps, opts = {}) {
   const mat = (i) => {
     let es = cache.get(i);
     if (!es) {
-      es = materialize(flat[i], i === failingIdx);
+      es = materialize(flat[i], i === failingIdx, testStepDepth(flat, flat[i]));
       cache.set(i, es);
     }
     return es;
   };
+  const steps = keepArr.map(mat);
+  if (omitted > 0) {
+    steps.unshift({
+      title: `\u2026 ${omitted} earlier step${omitted === 1 ? "" : "s"} omitted`,
+      category: "",
+      durationMs: 0,
+      depth: 0,
+      failed: false
+    });
+  }
   return {
-    steps: keepArr.map(mat),
+    steps,
     failingStep: failingIdx >= 0 ? mat(failingIdx) : void 0
   };
 }
@@ -12500,6 +12519,7 @@ var TemplateEngine = class {
   }
   registerHelpers() {
     this.handlebars.registerHelper("formatDuration", (ms) => formatDuration(ms ?? 0));
+    this.handlebars.registerHelper("repeat", (s, n) => typeof s === "string" ? s.repeat(Math.max(0, n | 0)) : "");
     this.handlebars.registerHelper("formatDate", (d) => {
       const date = d instanceof Date ? d : new Date(d);
       return date.toLocaleString("en-US", {
@@ -13182,7 +13202,7 @@ function fmtMb(bytes) {
 }
 
 // src/reporter.ts
-var import_path12 = __toESM(require("path"));
+var import_path13 = __toESM(require("path"));
 
 // src/history/HistoryManager.ts
 init_cjs_shims();
@@ -13263,6 +13283,8 @@ init_cjs_shims();
 
 // src/notify/NotificationSender.ts
 init_cjs_shims();
+var import_promises2 = require("fs/promises");
+var import_path12 = require("path");
 
 // src/notify/formatters/slack.ts
 init_cjs_shims();
@@ -13512,6 +13534,7 @@ var FORMATTERS = {
   teams: buildTeamsPayload,
   discord: buildDiscordPayload
 };
+var DISCORD_MAX_UPLOAD_BYTES = 25 * 1024 * 1024;
 var NotificationSender = class {
   async sendAll(reportData, notifyConfig, pdfPaths) {
     const result = { sent: [], skipped: [], failed: [] };
@@ -13524,6 +13547,9 @@ var NotificationSender = class {
           return Promise.resolve();
         }
         const payload = FORMATTERS[name](reportData.stats, pdfPaths, reportTitle);
+        if (name === "discord" && notifyConfig.discord?.attachPdf && pdfPaths.length > 0) {
+          return this.postDiscordWithPdf(channel.url, payload, pdfPaths[0], result);
+        }
         return this.post(name, channel.url, payload, result);
       })
     );
@@ -13565,6 +13591,47 @@ var NotificationSender = class {
       })
     );
   }
+  /**
+   * Posts the summary to a Discord webhook with the PDF as a multipart file
+   * upload (`payload_json` + `files[0]`). Any problem — unreadable file, PDF
+   * over Discord's upload cap, or a failed upload — falls back to the plain
+   * JSON summary post so the notification still lands; only the fallback's
+   * outcome is recorded in the result. Attaches the first PDF only (matches
+   * the email channel's multi-template behaviour).
+   */
+  async postDiscordWithPdf(url, payload, pdfPath, result) {
+    let file;
+    try {
+      file = await (0, import_promises2.readFile)(pdfPath);
+      if (file.byteLength > DISCORD_MAX_UPLOAD_BYTES) {
+        const mb = (file.byteLength / 1024 / 1024).toFixed(1);
+        throw new Error(`PDF is ${mb}MB \u2014 over Discord's upload limit`);
+      }
+    } catch (err) {
+      logger.warn(
+        `[notify] discord: ${err instanceof Error ? err.message : String(err)} \u2014 sending summary without the PDF`
+      );
+      return this.post("discord", url, payload, result);
+    }
+    const form = new FormData();
+    form.append("payload_json", JSON.stringify(payload));
+    form.append("files[0]", new Blob([new Uint8Array(file)], { type: "application/pdf" }), (0, import_path12.basename)(pdfPath));
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 3e4);
+    try {
+      const res = await fetch(url, { method: "POST", body: form, signal: controller.signal });
+      if (!res.ok) throw new Error(`HTTP ${res.status}`);
+      logger.info("[notify] discord: sent (PDF attached)");
+      result.sent.push("discord");
+    } catch (err) {
+      logger.warn(
+        `[notify] discord: upload failed (${err instanceof Error ? err.message : String(err)}) \u2014 retrying without the PDF`
+      );
+      return this.post("discord", url, payload, result);
+    } finally {
+      clearTimeout(timeoutId);
+    }
+  }
   async post(name, url, payload, result) {
     const controller = new AbortController();
     const timeoutId = setTimeout(() => controller.abort(), 1e4);
@@ -13887,7 +13954,7 @@ function mapCurrentStep(testId, step, shardKey) {
     currentStep: clampWithEllipsis(step.title ?? "", MAX_STEP_TITLE_CHARS)
   };
 }
-function testStepDepth(step) {
+function testStepDepth2(step) {
   let depth = 0;
   for (let p = step.parent; p; p = p.parent) {
     if (p.category === "test.step") depth++;
@@ -13907,7 +13974,7 @@ function mapStep(testId, step, phase, shardKey) {
     startedAt: step.startTime?.getTime(),
     // Nesting depth (test.step ancestors) so the page indents children under their
     // parent step. 0 → flat, so a test without test.steps renders as a flat trail.
-    depth: testStepDepth(step)
+    depth: testStepDepth2(step)
   };
 }
 function mapConsole(kind, chunk, testId, shardKey) {
@@ -13934,7 +14001,7 @@ var PdfReporter = class {
     this.liveConsole = false;
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.11.2" : "0.x";
+    const version = true ? "0.13.0" : "0.x";
     logger.info(`@reportforge/playwright-pdf v${version} initialised`);
     this.licenseClient = new LicenseClient({
       licenseKey: this.options.licenseKey,
@@ -14202,7 +14269,7 @@ var PdfReporter = class {
   }
   resolveProjectName() {
     try {
-      const pkgPath = import_path12.default.resolve(process.cwd(), "package.json");
+      const pkgPath = import_path13.default.resolve(process.cwd(), "package.json");
       const pkg = require(pkgPath);
       return pkg.name ?? "Playwright Tests";
     } catch {
@@ -14252,9 +14319,9 @@ function deriveVerdict(stats) {
 }
 function resolveHistoryPath(options, cwd) {
   if (options.historyFile) {
-    return import_path12.default.isAbsolute(options.historyFile) ? options.historyFile : import_path12.default.resolve(cwd, options.historyFile);
+    return import_path13.default.isAbsolute(options.historyFile) ? options.historyFile : import_path13.default.resolve(cwd, options.historyFile);
   }
-  return import_path12.default.join(resolveProjectDir(options, cwd), "history.json");
+  return import_path13.default.join(resolveProjectDir(options, cwd), "history.json");
 }
 function flattenSuiteNodes(suites) {
   return suites.flatMap((s) => [s, ...flattenSuiteNodes(s.suites)]);

package/dist/index.mjs

@@ -10455,6 +10455,9 @@ var channelConfigSchema = external_exports.object({
   on: external_exports.enum(["always", "failure", "success"]).default("always"),
   enabled: external_exports.boolean().default(false)
 });
+var discordChannelConfigSchema = channelConfigSchema.extend({
+  attachPdf: external_exports.boolean().default(false)
+});
 var emailChannelConfigSchema = external_exports.object({
   to: external_exports.array(external_exports.string().email('Each "to" entry must be a valid email')).min(1, '"to" must have at least one address'),
   on: external_exports.enum(["always", "failure", "success"]).default("always"),
@@ -10464,7 +10467,7 @@ var emailChannelConfigSchema = external_exports.object({
 var notifyConfigSchema = external_exports.object({
   slack: channelConfigSchema.optional(),
   teams: channelConfigSchema.optional(),
-  discord: channelConfigSchema.optional(),
+  discord: discordChannelConfigSchema.optional(),
   email: emailChannelConfigSchema.optional()
 }).optional();
 var failureAnalysisConfigSchema = external_exports.object({
@@ -11436,9 +11439,13 @@ init_esm_shims();
 var MAX_TITLE_CHARS = 200;
 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));
+var isTopLevel = (flat, f) => !f.ancestors.some((a) => {
+  const c = flat[a].category;
+  return c === "pw:api" || c === "expect";
+});
+var testStepDepth = (flat, f) => f.ancestors.reduce((n, a) => n + (flat[a].category === "test.step" ? 1 : 0), 0);
 function flatten(steps) {
   const out = [];
   const walk = (nodes, depth, ancestors) => {
@@ -11457,15 +11464,19 @@ function flatten(steps) {
   walk(steps ?? [], 0, []);
   return out;
 }
-function materialize(n, isFailing) {
+function materialize(n, isFailing, depth) {
   const loc = n.raw.location;
+  const firstLine = isFailing && n.raw.error?.message ? stripAnsi(n.raw.error.message).split("\n").map((s) => s.trim()).find((s) => s.length) ?? "" : "";
   return {
     title: clampWithEllipsis(n.raw.title ?? "", MAX_TITLE_CHARS),
     category: n.category,
     durationMs: Math.max(0, Math.round(n.raw.duration ?? 0)),
-    depth: n.depth,
+    depth,
     failed: n.hasError,
-    errorMessage: isFailing && n.raw.error?.message ? clampWithEllipsis(stripAnsi(n.raw.error.message).split("\n")[0], MAX_TITLE_CHARS) : void 0,
+    // Marks the single failing leaf so the template renders the error there once —
+    // independent of whether a step-level message survives (falls back to the record error).
+    failingLeaf: isFailing || void 0,
+    errorMessage: firstLine ? clampWithEllipsis(firstLine, MAX_TITLE_CHARS) : void 0,
     location: loc?.file != null ? `${loc.file}:${loc.line ?? 0}` : void 0
   };
 }
@@ -11489,21 +11500,19 @@ function compactSteps(rawSteps, opts = {}) {
   const failingIdx = pickFailingIdx(flat);
   const keep = /* @__PURE__ */ new Set();
   flat.forEach((f, i) => {
-    if (f.category === "test.step" || f.category === "expect") keep.add(i);
+    if (inHookSubtree(flat, f)) return;
+    if (f.category === "test.step") keep.add(i);
+    else if (f.category === "expect" && isTopLevel(flat, f)) keep.add(i);
+    else if (opts.apiSteps && f.category === "pw:api" && isTopLevel(flat, f)) keep.add(i);
   });
   if (failingIdx >= 0) {
     keep.add(failingIdx);
     for (const a of flat[failingIdx].ancestors) keep.add(a);
-    if (opts.apiSteps) {
-      const pwApiBefore = [];
-      for (let i = 0; i < failingIdx; 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);
-    }
   }
   let keepArr = [...keep].sort((a, b) => a - b);
+  let omitted = 0;
   if (keepArr.length > maxSteps) {
+    omitted = keepArr.length - maxSteps;
     keepArr = keepArr.slice(-maxSteps);
     if (failingIdx >= 0 && !keepArr.includes(failingIdx)) {
       keepArr = [failingIdx, ...keepArr.slice(1)];
@@ -11513,13 +11522,23 @@ function compactSteps(rawSteps, opts = {}) {
   const mat = (i) => {
     let es = cache.get(i);
     if (!es) {
-      es = materialize(flat[i], i === failingIdx);
+      es = materialize(flat[i], i === failingIdx, testStepDepth(flat, flat[i]));
       cache.set(i, es);
     }
     return es;
   };
+  const steps = keepArr.map(mat);
+  if (omitted > 0) {
+    steps.unshift({
+      title: `\u2026 ${omitted} earlier step${omitted === 1 ? "" : "s"} omitted`,
+      category: "",
+      durationMs: 0,
+      depth: 0,
+      failed: false
+    });
+  }
   return {
-    steps: keepArr.map(mat),
+    steps,
     failingStep: failingIdx >= 0 ? mat(failingIdx) : void 0
   };
 }
@@ -12501,6 +12520,7 @@ var TemplateEngine = class {
   }
   registerHelpers() {
     this.handlebars.registerHelper("formatDuration", (ms) => formatDuration(ms ?? 0));
+    this.handlebars.registerHelper("repeat", (s, n) => typeof s === "string" ? s.repeat(Math.max(0, n | 0)) : "");
     this.handlebars.registerHelper("formatDate", (d) => {
       const date = d instanceof Date ? d : new Date(d);
       return date.toLocaleString("en-US", {
@@ -13264,6 +13284,8 @@ init_esm_shims();
 
 // src/notify/NotificationSender.ts
 init_esm_shims();
+import { readFile as readFile2 } from "fs/promises";
+import { basename as basename3 } from "path";
 
 // src/notify/formatters/slack.ts
 init_esm_shims();
@@ -13513,6 +13535,7 @@ var FORMATTERS = {
   teams: buildTeamsPayload,
   discord: buildDiscordPayload
 };
+var DISCORD_MAX_UPLOAD_BYTES = 25 * 1024 * 1024;
 var NotificationSender = class {
   async sendAll(reportData, notifyConfig, pdfPaths) {
     const result = { sent: [], skipped: [], failed: [] };
@@ -13525,6 +13548,9 @@ var NotificationSender = class {
           return Promise.resolve();
         }
         const payload = FORMATTERS[name](reportData.stats, pdfPaths, reportTitle);
+        if (name === "discord" && notifyConfig.discord?.attachPdf && pdfPaths.length > 0) {
+          return this.postDiscordWithPdf(channel.url, payload, pdfPaths[0], result);
+        }
         return this.post(name, channel.url, payload, result);
       })
     );
@@ -13566,6 +13592,47 @@ var NotificationSender = class {
       })
     );
   }
+  /**
+   * Posts the summary to a Discord webhook with the PDF as a multipart file
+   * upload (`payload_json` + `files[0]`). Any problem — unreadable file, PDF
+   * over Discord's upload cap, or a failed upload — falls back to the plain
+   * JSON summary post so the notification still lands; only the fallback's
+   * outcome is recorded in the result. Attaches the first PDF only (matches
+   * the email channel's multi-template behaviour).
+   */
+  async postDiscordWithPdf(url, payload, pdfPath, result) {
+    let file;
+    try {
+      file = await readFile2(pdfPath);
+      if (file.byteLength > DISCORD_MAX_UPLOAD_BYTES) {
+        const mb = (file.byteLength / 1024 / 1024).toFixed(1);
+        throw new Error(`PDF is ${mb}MB \u2014 over Discord's upload limit`);
+      }
+    } catch (err) {
+      logger.warn(
+        `[notify] discord: ${err instanceof Error ? err.message : String(err)} \u2014 sending summary without the PDF`
+      );
+      return this.post("discord", url, payload, result);
+    }
+    const form = new FormData();
+    form.append("payload_json", JSON.stringify(payload));
+    form.append("files[0]", new Blob([new Uint8Array(file)], { type: "application/pdf" }), basename3(pdfPath));
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 3e4);
+    try {
+      const res = await fetch(url, { method: "POST", body: form, signal: controller.signal });
+      if (!res.ok) throw new Error(`HTTP ${res.status}`);
+      logger.info("[notify] discord: sent (PDF attached)");
+      result.sent.push("discord");
+    } catch (err) {
+      logger.warn(
+        `[notify] discord: upload failed (${err instanceof Error ? err.message : String(err)}) \u2014 retrying without the PDF`
+      );
+      return this.post("discord", url, payload, result);
+    } finally {
+      clearTimeout(timeoutId);
+    }
+  }
   async post(name, url, payload, result) {
     const controller = new AbortController();
     const timeoutId = setTimeout(() => controller.abort(), 1e4);
@@ -13888,7 +13955,7 @@ function mapCurrentStep(testId, step, shardKey) {
     currentStep: clampWithEllipsis(step.title ?? "", MAX_STEP_TITLE_CHARS)
   };
 }
-function testStepDepth(step) {
+function testStepDepth2(step) {
   let depth = 0;
   for (let p = step.parent; p; p = p.parent) {
     if (p.category === "test.step") depth++;
@@ -13908,7 +13975,7 @@ function mapStep(testId, step, phase, shardKey) {
     startedAt: step.startTime?.getTime(),
     // Nesting depth (test.step ancestors) so the page indents children under their
     // parent step. 0 → flat, so a test without test.steps renders as a flat trail.
-    depth: testStepDepth(step)
+    depth: testStepDepth2(step)
   };
 }
 function mapConsole(kind, chunk, testId, shardKey) {
@@ -13935,7 +14002,7 @@ var PdfReporter = class {
     this.liveConsole = false;
     this.options = parseOptions(rawOptions);
     this.dataCollector = new DataCollector(this.options.capture);
-    const version = true ? "0.11.2" : "0.x";
+    const version = true ? "0.13.0" : "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

@@ -36,17 +36,19 @@
             {{#if execution.steps.length}}
             <div class="repro">
               <div class="repro__tag">Steps to Reproduce</div>
-              <ol class="repro-steps">
+              {{! Each row is a real Markdown list line — `'  '×depth + '- ' + title` rendered
+                  pre-wrap — so the outline both displays indented and survives a copy-paste
+                  into a bug tracker as a nested list. }}
+              <div class="repro-steps">
                 {{#each execution.steps}}
-                <li class="repro-step{{#if failed}} repro-step--failed{{/if}}" style="--d:{{depth}};">
-                  <span class="repro-step__cat">{{category}}</span>
-                  <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>
+                <div class="repro-step{{#if failed}} repro-step--failed{{/if}}">
+                  <span class="repro-step__line">{{repeat "  " depth}}- {{title}}{{#if failed}} [FAILED]{{#if location}}  ({{location}}){{/if}}{{/if}}</span>
+                  {{! Error renders once, at the failing leaf: the step's own message, else the
+                      authoritative record error so an empty step message never hides it. }}
+                  {{#if failingLeaf}}<div class="repro-step__error">{{#if errorMessage}}{{errorMessage}}{{else}}{{../error.message}}{{/if}}</div>{{/if}}
+                </div>
                 {{/each}}
-              </ol>
+              </div>
             </div>
             {{/if}}
 

package/dist/templates/styles/base.css

@@ -436,32 +436,24 @@ code {
   font-family: 'JetBrains Mono', monospace; font-size: 8.5px; color: var(--text-3); margin-left: 6px;
 }
 .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; 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;
+.repro-steps { margin: 6px 0 0; padding: 0; }
+.repro-step { page-break-inside: avoid; break-inside: avoid; }
+/* One monospace, pre-wrap line per step. The leading `'  '×depth + '- '` (real
+   characters, not CSS padding) renders the outline indentation AND survives a PDF
+   text-copy — pasting yields a valid nested Markdown list. */
+.repro-step__line {
+  display: block;
+  font-family: 'JetBrains Mono', monospace; font-size: 9px; line-height: 1.55;
+  color: var(--text-2); white-space: pre-wrap; word-break: break-word;
 }
-/* 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--failed .repro-step__line { color: #8A2E1E; font-weight: 700; }
+/* The error sits at the failing step — below its line, slightly indented so it reads
+   as belonging to that step. */
 .repro-step__error {
-  flex-basis: 100%; margin: 3px 0 2px; padding-left: 60px;
+  margin: 2px 0 3px; padding-left: 18px;
   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);
-}
-.repro-step__title { flex: 1; color: var(--text-2); word-break: break-word; }
-.repro-step__loc {
-  flex: none; font-family: 'JetBrains Mono', monospace; font-size: 8px; color: var(--text-3);
-}
-.repro-step__dur { flex: none; color: var(--text-3); font-size: 8.5px; }
-.repro-step--failed .repro-step__title { color: #8A2E1E; font-weight: 700; }
-.repro-step--failed .repro-step__cat { color: var(--fail); }
 .console-tail { margin-top: 11px; padding-top: 10px; border-top: 1px solid var(--border); }
 .console-tail__line {
   font-family: 'JetBrains Mono', monospace; font-size: 8.5px; line-height: 1.55;

package/package.json

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