coverme-security-scanner 3.7.3 → 3.7.4

package/dist/pdf/generator.js

@@ -402,6 +402,9 @@ export class PDFGenerator {
             this.y = this.doc.y + spacing.paragraph;
         }
         if (report.architecture?.components?.length) {
+            // Start components table on new page to keep it together
+            this.newPage();
+            this.y = spacing.page.top;
             this.subTitle('Components');
             this.renderSimpleTable(['Component', 'Technology', 'Description'], report.architecture.components.map(c => [c.name, c.technology, c.description]), [100, 150, 245]);
         }
@@ -434,7 +437,6 @@ export class PDFGenerator {
                 this.y += 18;
             });
         }
-        this.checkPageBreak();
     }
     // ─────────────────────────────────────────────────────────────────
     // Network
@@ -614,15 +616,20 @@ export class PDFGenerator {
         const boxPadding = 12;
         const boxInnerWidth = layout.content.width - (boxPadding * 2);
         const style = colors.severity[finding.severity];
-        // ID in severity color, title in black
+        // ID in severity color, title in black - allow wrapping for long titles
+        const titleText = `[${finding.id}] ${finding.title}`;
+        const titleHeight = this.doc
+            .font(fonts.weights.bold)
+            .fontSize(fonts.sizes.body)
+            .heightOfString(titleText, { width: layout.content.width });
         this.doc
             .font(fonts.weights.bold)
             .fontSize(fonts.sizes.body)
             .fillColor(style.text)
             .text(`[${finding.id}]`, spacing.page.margin, this.y, { continued: true })
             .fillColor(colors.text.primary)
-            .text(` ${finding.title}`);
-        this.y += 18;
+            .text(` ${finding.title}`, { width: layout.content.width - 60 });
+        this.y = this.doc.y + 4;
         // Cross-references and DREAD score line
         const hasRefs = finding.relatedFindings?.length || finding.dreadScore || finding.cwe;
         if (hasRefs) {
@@ -1141,53 +1148,73 @@ export class PDFGenerator {
         this.y += 20;
     }
     renderSimpleTable(headers, rows, colWidths) {
-        const rowHeight = 24;
+        const minRowHeight = 24;
         const x = spacing.page.margin;
-        // Header
-        this.doc
-            .rect(x, this.y, layout.content.width, rowHeight)
-            .fill(colors.table.header);
-        let colX = x;
-        headers.forEach((header, i) => {
+        const cellPadding = 8;
+        // Helper to render table header
+        const renderHeader = () => {
             this.doc
-                .font(fonts.weights.bold)
-                .fontSize(fonts.sizes.small)
-                .fillColor(colors.table.headerText)
-                .text(header, colX + 8, this.y + 7, {
-                width: colWidths[i] - 16,
+                .rect(x, this.y, layout.content.width, minRowHeight)
+                .fill(colors.table.header);
+            let colX = x;
+            headers.forEach((header, i) => {
+                this.doc
+                    .font(fonts.weights.bold)
+                    .fontSize(fonts.sizes.small)
+                    .fillColor(colors.table.headerText)
+                    .text(header, colX + cellPadding, this.y + 7, {
+                    width: colWidths[i] - (cellPadding * 2),
+                });
+                colX += colWidths[i];
             });
-            colX += colWidths[i];
-        });
-        this.y += rowHeight;
-        // Rows
+            this.y += minRowHeight;
+        };
+        // Initial header
+        renderHeader();
+        // Rows - calculate dynamic height based on content
         rows.forEach((row, rowIndex) => {
-            this.checkPageBreak(rowHeight + 20);
+            // Calculate the height needed for this row based on longest cell
+            let maxCellHeight = minRowHeight;
+            row.forEach((cell, i) => {
+                const cellWidth = colWidths[i] - (cellPadding * 2);
+                const textHeight = this.doc
+                    .font(fonts.weights.normal)
+                    .fontSize(fonts.sizes.small)
+                    .heightOfString(cell, { width: cellWidth });
+                const cellHeight = Math.max(minRowHeight, textHeight + (cellPadding * 2));
+                maxCellHeight = Math.max(maxCellHeight, cellHeight);
+            });
+            // Check if we need a page break - if so, re-render header on new page
+            const needsPageBreak = this.y > layout.page.height - spacing.page.bottom - maxCellHeight - 20;
+            if (needsPageBreak) {
+                this.newPage();
+                renderHeader();
+            }
             // Alternating background
             if (rowIndex % 2 === 1) {
                 this.doc
-                    .rect(x, this.y, layout.content.width, rowHeight)
+                    .rect(x, this.y, layout.content.width, maxCellHeight)
                     .fill(colors.table.altRow);
             }
             // Border
             this.doc
-                .moveTo(x, this.y + rowHeight)
-                .lineTo(x + layout.content.width, this.y + rowHeight)
+                .moveTo(x, this.y + maxCellHeight)
+                .lineTo(x + layout.content.width, this.y + maxCellHeight)
                 .strokeColor(colors.table.border)
                 .lineWidth(0.5)
                 .stroke();
-            colX = x;
+            let colX = x;
             row.forEach((cell, i) => {
                 this.doc
                     .font(fonts.weights.normal)
                     .fontSize(fonts.sizes.small)
                     .fillColor(colors.text.primary)
-                    .text(cell, colX + 8, this.y + 7, {
-                    width: colWidths[i] - 16,
-                    ellipsis: true,
+                    .text(cell, colX + cellPadding, this.y + 7, {
+                    width: colWidths[i] - (cellPadding * 2),
                 });
                 colX += colWidths[i];
             });
-            this.y += rowHeight;
+            this.y += maxCellHeight;
         });
         this.y += spacing.paragraph;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "coverme-security-scanner",
-  "version": "3.7.3",
+  "version": "3.7.4",
   "description": "AI-powered security assessment reports with beautiful PDF output",
   "type": "module",
   "bin": {