@vibecheckai/cli 3.1.2 → 3.1.5

package/bin/runners/runReport.js

@@ -1,63 +1,93 @@
 /**
- * vibecheck report - World-Class Professional Reports
+ * vibecheck report - World-Class Enterprise Reports
  * 
  * TIER ENFORCEMENT:
  * - FREE: HTML, MD formats only
  * - STARTER: + SARIF, CSV formats
- * - PRO: + compliance packs, redaction templates
+ * - PRO: + compliance packs, PDF export, redaction templates
  * 
  * Enterprise-grade report generation with:
  * - Beautiful interactive HTML with modern design
- * - Multiple export formats (HTML, MD, JSON, SARIF, CSV)
+ * - Multiple export formats (HTML, MD, JSON, SARIF, CSV, PDF)
  * - Executive, Technical, Compliance report types
  * - Historical trends and fix time estimates
  * - Print-optimized layouts
+ * - White-label customization
  */
 
 const path = require("path");
 const fs = require("fs");
 
 // Entitlements enforcement
-const entitlements = require("./lib/entitlements-v2");
+let entitlements;
+try {
+  entitlements = require("./lib/entitlements-v2");
+} catch {
+  // Fallback: allow all features if entitlements not available
+  entitlements = {
+    getLimits: () => ({ reportFormats: ["html", "md", "json", "sarif", "csv", "pdf"] }),
+    getTier: async () => "pro",
+    enforce: async () => ({ allowed: true }),
+    EXIT_FEATURE_NOT_ALLOWED: 1,
+  };
+}
 
-// Report engine modules
-let reportEngine, reportHtml;
+// Report modules
+let reportEngine, reportHtml, reportTemplates;
 try {
   reportEngine = require("./lib/report-engine");
+} catch (e) {
+  console.error("Warning: report-engine not found:", e.message);
+}
+try {
   reportHtml = require("./lib/report-html");
 } catch (e) {
-  // Fallback to basic templates if enhanced modules unavailable
-  reportEngine = null;
-  reportHtml = null;
+  console.error("Warning: report-html not found:", e.message);
+}
+try {
+  reportTemplates = require("./lib/report-templates");
+} catch (e) {
+  console.error("Warning: report-templates not found:", e.message);
 }
 
-// ANSI color codes
-const c = {
-  reset: "\x1b[0m",
-  bold: "\x1b[1m",
-  dim: "\x1b[2m",
-  red: "\x1b[31m",
-  green: "\x1b[32m",
-  yellow: "\x1b[33m",
-  blue: "\x1b[34m",
-  magenta: "\x1b[35m",
-  cyan: "\x1b[36m",
-};
+// ═══════════════════════════════════════════════════════════════════════════════
+// ENHANCED TERMINAL UI & OUTPUT MODULES
+// ═══════════════════════════════════════════════════════════════════════════════
+
+const {
+  ansi,
+  colors,
+  icons,
+  Spinner,
+  renderSection,
+  formatDuration,
+} = require("./lib/terminal-ui");
+
+const {
+  BANNER,
+  formatHelp,
+  renderReportInfo,
+  renderSuccess,
+  renderError,
+} = require("./lib/report-output");
 
 function parseArgs(args) {
   const opts = {
-    type: "executive",
+    type: "technical",
     format: "html",
     path: ".",
     output: null,
     logo: null,
     company: null,
     theme: "dark",
+    framework: "SOC2",
     includeVerify: false,
     includeTrends: false,
     redactPaths: false,
     maxFindings: 50,
+    open: false,
     help: false,
+    quiet: false,
   };
 
   for (let i = 0; i < args.length; i++) {
@@ -68,6 +98,7 @@ function parseArgs(args) {
     if (a === "--logo") opts.logo = args[++i];
     if (a === "--company") opts.company = args[++i];
     if (a === "--theme") opts.theme = args[++i];
+    if (a === "--framework") opts.framework = args[++i];
     if (a === "--light") opts.theme = "light";
     if (a === "--dark") opts.theme = "dark";
     if (a === "--include-verify") opts.includeVerify = true;
@@ -76,6 +107,8 @@ function parseArgs(args) {
     if (a === "--max-findings") opts.maxFindings = parseInt(args[++i]) || 50;
     if (a.startsWith("--path=")) opts.path = a.split("=")[1];
     if (a === "--path" || a === "-p") opts.path = args[++i];
+    if (a === "--open") opts.open = true;
+    if (a === "--quiet" || a === "-q") opts.quiet = true;
     if (a === "--help" || a === "-h") opts.help = true;
   }
 
@@ -83,60 +116,8 @@ function parseArgs(args) {
 }
 
 function printHelp() {
-  console.log(`
-${c.bold}${c.cyan}vibecheck report${c.reset} — World-Class Professional Reports
-
-${c.dim}Generate beautiful, professional reports for stakeholders.${c.reset}
-
-${c.bold}USAGE${c.reset}
-  vibecheck report                          Generate executive HTML report
-  vibecheck report --format=md              Generate Markdown report
-  vibecheck report --format=sarif           Generate SARIF for security tools
-  vibecheck report --type=compliance        Generate compliance report
-
-${c.bold}REPORT TYPES${c.reset}
-  ${c.cyan}executive${c.reset}    One-page overview for stakeholders (default)
-  ${c.cyan}technical${c.reset}    Detailed findings for developers/CTOs
-  ${c.cyan}compliance${c.reset}   SOC2/HIPAA-ready language for regulated industries
-  ${c.cyan}trend${c.reset}        Historical score analysis over time
-
-${c.bold}OUTPUT FORMATS${c.reset}
-  ${c.green}html${c.reset}         Beautiful interactive report (default)
-  ${c.green}md${c.reset}           Markdown for documentation/GitHub
-  ${c.green}json${c.reset}         Machine-readable JSON
-  ${c.green}sarif${c.reset}        SARIF for security tool integration
-  ${c.green}csv${c.reset}          CSV for spreadsheet analysis
-
-${c.bold}OPTIONS${c.reset}
-  --type, -t <type>       Report type: executive, technical, compliance, trend
-  --format, -f <format>   Output format: html, md, json, sarif, csv
-  --output, -o <path>     Output file path
-  --theme <dark|light>    HTML theme (default: dark)
-  --company <name>        Company name for branding
-  --logo <path>           Custom logo path/URL
-  --include-trends        Include historical trend data
-  --redact-paths          Hide file paths for client reports
-  --max-findings <n>      Max findings to show (default: 50)
-  --path, -p <dir>        Project path (default: current directory)
-  --help, -h              Show this help
-
-${c.bold}EXAMPLES${c.reset}
-  vibecheck report                                    # Interactive HTML
-  vibecheck report --format=md --output=REPORT.md    # Markdown file
-  vibecheck report --type=compliance --company=Acme  # Compliance report
-  vibecheck report --format=sarif -o report.sarif    # Security tooling
-  vibecheck report --theme=light --redact            # Client-safe HTML
-  vibecheck report --trends                          # Include history
-
-${c.bold}OUTPUT${c.reset}
-  HTML reports include:
-  • Animated score visualization
-  • Interactive severity charts
-  • Category breakdown bars
-  • Fix time estimates
-  • Dark/light mode toggle
-  • Print-optimized layout
-`);
+  console.log(BANNER);
+  console.log(formatHelp());
 }
 
 async function runReport(args) {
@@ -151,26 +132,39 @@ async function runReport(args) {
   const outputDir = path.join(projectPath, ".vibecheck");
   const projectName = path.basename(projectPath);
 
-  // TIER ENFORCEMENT: Check format access
+  // TIER ENFORCEMENT
   const format = opts.format.toLowerCase();
-  const limits = entitlements.getLimits(await entitlements.getTier({ projectPath }));
+  const tier = await entitlements.getTier({ projectPath });
+  const limits = entitlements.getLimits(tier);
   const allowedFormats = limits.reportFormats || ["html", "md"];
-  
-  // Check if requested format is allowed
+
+  // Check format access
   if (!allowedFormats.includes(format) && format !== "json") {
-    // SARIF/CSV require STARTER+
     if (format === "sarif" || format === "csv") {
       const access = await entitlements.enforce("report.sarif_csv", {
         projectPath,
         silent: false,
       });
       if (!access.allowed) {
-        console.log(`\n${c.yellow}Tip:${c.reset} HTML and MD formats are available on FREE tier`);
+        console.log(`\n  ${colors.warning}${icons.warning}${ansi.reset} ${ansi.dim}HTML and MD formats are available on FREE tier${ansi.reset}`);
+        console.log(`  ${ansi.dim}Upgrade to STARTER for SARIF/CSV export${ansi.reset}\n`);
+        return entitlements.EXIT_FEATURE_NOT_ALLOWED;
+      }
+    }
+    
+    if (format === "pdf") {
+      const access = await entitlements.enforce("report.pdf_export", {
+        projectPath,
+        silent: false,
+      });
+      if (!access.allowed) {
+        console.log(`\n  ${colors.warning}${icons.warning}${ansi.reset} ${ansi.dim}PDF export requires PRO tier${ansi.reset}`);
+        console.log(`  ${ansi.dim}HTML reports can be printed to PDF from browser${ansi.reset}\n`);
         return entitlements.EXIT_FEATURE_NOT_ALLOWED;
       }
     }
   }
-  
+
   // Compliance reports require PRO
   if (opts.type === "compliance") {
     const access = await entitlements.enforce("report.compliance_packs", {
@@ -178,23 +172,35 @@ async function runReport(args) {
       silent: false,
     });
     if (!access.allowed) {
-      console.log(`\n${c.yellow}Tip:${c.reset} Executive and technical reports are available on lower tiers`);
+      console.log(`\n  ${colors.warning}${icons.warning}${ansi.reset} ${ansi.dim}Executive and technical reports available on lower tiers${ansi.reset}`);
       return entitlements.EXIT_FEATURE_NOT_ALLOWED;
     }
   }
 
-  console.log(`\n${c.bold}${c.cyan}📄 Generating ${opts.type} report...${c.reset}\n`);
+  // Display banner and report info
+  if (!opts.quiet) {
+    console.log(BANNER);
+    console.log(renderReportInfo(opts.type, opts.format, null));
+    console.log('');
+  }
+
+  // Progress spinner
+  const spinner = new Spinner({ color: colors.accent });
+  spinner.start(`Generating ${opts.type} report`);
 
-  // Load ship results from multiple possible sources
+  // Load ship results
   let shipResults = loadShipResults(projectPath, outputDir);
 
   if (!shipResults) {
-    console.log(`${c.yellow}⚠${c.reset} No scan results found. Using demo data for preview.\n`);
-    console.log(`${c.dim}  Tip: Run 'vibecheck ship' first for real results.${c.reset}\n`);
+    spinner.warn("No scan results found - using demo data");
+    if (!opts.quiet) {
+      console.log(`  ${ansi.dim}Tip: Run 'vibecheck ship' first for real results.${ansi.reset}\n`);
+    }
     shipResults = getDemoData();
   }
 
-  // Build comprehensive report data using the new engine
+  // Build comprehensive report data
+  spinner.update("Processing findings");
   let reportData;
   if (reportEngine) {
     reportData = reportEngine.buildReportData(shipResults, {
@@ -203,56 +209,92 @@ async function runReport(args) {
       includeTrends: opts.includeTrends,
     });
   } else {
-    // Fallback to basic report data
     reportData = buildBasicReportData(shipResults, projectName);
   }
 
-  // Generate report content based on format
+  // Generate report content
+  spinner.update(`Rendering ${format.toUpperCase()} output`);
   let reportContent = "";
-  let fileExtension = opts.format;
+  let fileExtension = format;
 
-  switch (opts.format) {
-    case "html":
-      reportContent = generateHTMLReport(reportData, opts);
-      break;
-    case "md":
-    case "markdown":
-      reportContent = generateMarkdownReport(reportData, opts);
-      fileExtension = "md";
-      break;
-    case "json":
-      reportContent = generateJSONReport(reportData, opts);
-      break;
-    case "sarif":
-      reportContent = generateSARIFReport(reportData, opts);
-      break;
-    case "csv":
-      reportContent = generateCSVReport(reportData, opts);
-      break;
-    default:
-      console.error(`${c.red}✗${c.reset} Unknown format: ${opts.format}`);
-      console.log(`${c.dim}  Supported: html, md, json, sarif, csv${c.reset}`);
-      return 1;
+  try {
+    switch (format) {
+      case "html":
+        // Debug: Log which HTML generator will be used
+        if (opts.verbose || opts.type === "technical") {
+          if (reportHtml && typeof reportHtml.generateWorldClassHTML === 'function') {
+            console.log(`  ${ansi.dim}Using world-class HTML generator${ansi.reset}`);
+          } else {
+            console.log(`  ${ansi.dim}Using basic HTML generator (report-html module not available)${ansi.reset}`);
+          }
+        }
+        reportContent = generateHTMLReport(reportData, opts);
+        break;
+      case "md":
+      case "markdown":
+        reportContent = generateMarkdownReport(reportData, opts);
+        fileExtension = "md";
+        break;
+      case "json":
+        reportContent = generateJSONReport(reportData, opts);
+        break;
+      case "sarif":
+        reportContent = generateSARIFReport(reportData, opts);
+        break;
+      case "csv":
+        reportContent = generateCSVReport(reportData, opts);
+        break;
+      case "pdf":
+        // PDF requires HTML first, then conversion
+        reportContent = await generatePDFReport(reportData, opts, outputDir);
+        break;
+      default:
+        spinner.fail(`Unknown format: ${format}`);
+        console.log(`  ${ansi.dim}Supported: html, md, json, sarif, csv, pdf${ansi.reset}`);
+        return 1;
+    }
+  } catch (err) {
+    spinner.fail(`Failed to generate report: ${err.message}`);
+    return 1;
   }
 
   // Determine output path
   const outputFileName = opts.output || path.join(outputDir, `report.${fileExtension}`);
+  const outputDirPath = path.dirname(outputFileName);
 
   // Ensure output directory exists
-  const outputDirPath = path.dirname(outputFileName);
   if (!fs.existsSync(outputDirPath)) {
     fs.mkdirSync(outputDirPath, { recursive: true });
   }
 
-  // Write report
-  fs.writeFileSync(outputFileName, reportContent);
+  // Write report (always overwrite to ensure fresh content)
+  spinner.update("Writing report");
+  fs.writeFileSync(outputFileName, reportContent, 'utf8');
+  
+  // If this is the default report.html, also update the timestamp to help with cache busting
+  if (!opts.output && fileExtension === 'html') {
+    try {
+      // Touch the file to update its modification time
+      const now = new Date();
+      fs.utimesSync(outputFileName, now, now);
+    } catch {
+      // Ignore errors on utimes
+    }
+  }
 
   // Save report history for trend analysis
   saveReportHistory(outputDir, reportData);
 
-  // Print success message
-  console.log(`${c.green}✓${c.reset} Report generated successfully!\n`);
-  printReportSummary(reportData, opts, outputFileName);
+  spinner.succeed("Report generated");
+  console.log("");
+  
+  // Print success message using design system
+  console.log(renderSuccess(outputFileName, opts.format));
+
+  // Open in browser if requested
+  if (opts.open && format === "html") {
+    openInBrowser(outputFileName);
+  }
 
   return 0;
 }
@@ -262,7 +304,6 @@ async function runReport(args) {
 // ============================================================================
 
 function loadShipResults(projectPath, outputDir) {
-  // Try multiple sources for ship results
   const sources = [
     path.join(outputDir, "ship_report.json"),
     path.join(outputDir, "report.json"),
@@ -286,7 +327,7 @@ function loadShipResults(projectPath, outputDir) {
     path.join(outputDir, "reality", "last_reality.json"),
     path.join(outputDir, "reality_report.json"),
   ];
-  
+
   for (const src of realitySources) {
     if (fs.existsSync(src)) {
       try {
@@ -311,29 +352,21 @@ function getDemoData() {
     score: 72,
     verdict: "WARN",
     findings: [
-      { id: "SEC001", severity: "BLOCK", type: "secret", message: "API key exposed in source code", file: "src/config.ts", line: 42, fix: "Move to environment variable" },
-      { id: "AUTH001", severity: "BLOCK", type: "auth", message: "Authentication bypass in admin route", file: "src/routes/admin.ts", line: 15, fix: "Add auth middleware" },
-      { id: "MOCK001", severity: "WARN", type: "mock", message: "Mock data used in production path", file: "src/api/users.ts", line: 88, fix: "Remove mock data fallback" },
-      { id: "ERR001", severity: "WARN", type: "error", message: "Empty catch block in payment flow", file: "src/billing/checkout.ts", line: 156, fix: "Add error handling" },
-      { id: "CFG001", severity: "INFO", type: "config", message: "Hardcoded timeout value", file: "src/utils/http.ts", line: 23, fix: "Move to config" },
-      { id: "DBG001", severity: "INFO", type: "quality", message: "Console.log statement", file: "src/debug.ts", line: 5, fix: "Remove or use logger" },
+      { id: "SEC001", severity: "BLOCK", type: "secret", message: "API key exposed in source code", title: "Exposed API Key", file: "src/config.ts", line: 42, fix: "Move to environment variable" },
+      { id: "AUTH001", severity: "BLOCK", type: "auth", message: "Authentication bypass in admin route", title: "Auth Bypass Vulnerability", file: "src/routes/admin.ts", line: 15, fix: "Add auth middleware" },
+      { id: "MOCK001", severity: "WARN", type: "mock", message: "Mock data used in production path", title: "Mock Data in Production", file: "src/api/users.ts", line: 88, fix: "Remove mock data fallback" },
+      { id: "ERR001", severity: "WARN", type: "error", message: "Empty catch block in payment flow", title: "Swallowed Exception", file: "src/billing/checkout.ts", line: 156, fix: "Add error handling" },
+      { id: "CFG001", severity: "INFO", type: "config", message: "Hardcoded timeout value", title: "Hardcoded Configuration", file: "src/utils/http.ts", line: 23, fix: "Move to config" },
+      { id: "DBG001", severity: "INFO", type: "quality", message: "Console.log statement", title: "Debug Statement", file: "src/debug.ts", line: 5, fix: "Remove or use logger" },
     ],
+    categoryScores: { security: 65, auth: 50, billing: 90, quality: 75 },
     truthpack: {
       routes: { server: Array(12).fill({}) },
       env: { vars: Array(8).fill("") },
-      auth: { nextMiddleware: [{}] },
-      billing: { webhooks: [{}] },
     },
     reality: {
-      coverage: {
-        clientCallsMapped: 87,
-        runtimeRequests: 72,
-        uiActionsVerified: 95,
-        authRoutes: 100,
-      },
-      requestsOverTime: [3, 5, 2, 8, 12, 6, 4, 9, 15, 7, 3, 10],
+      coverage: { clientCallsMapped: 87, uiActionsVerified: 95, authRoutes: 100 },
       latencyP95: 412,
-      latencySparkline: [120, 150, 180, 200, 220, 280, 320, 380, 400, 412],
       brokenFlows: [
         {
           title: "User cannot complete checkout flow",
@@ -346,21 +379,6 @@ function getDemoData() {
             { type: "error", label: "500 Internal Server Error" },
           ],
         },
-        {
-          title: "Password reset email never sent",
-          severity: "BLOCK",
-          steps: [
-            { type: "ui", label: "Click 'Forgot Password'" },
-            { type: "form", label: "Enter email" },
-            { type: "api", label: "POST /api/auth/reset" },
-            { type: "error", label: "Missing SMTP config" },
-          ],
-        },
-      ],
-      unmappedRequests: [
-        { method: "GET", path: "/api/analytics", count: 3 },
-        { method: "POST", path: "/api/tracking", count: 5 },
-        { method: "GET", path: "/api/health", count: 12 },
       ],
     },
   };
@@ -369,7 +387,7 @@ function getDemoData() {
 function buildBasicReportData(shipResults, projectName) {
   const findings = shipResults?.findings || [];
   const reality = shipResults?.reality || null;
-  
+
   return {
     meta: {
       projectName,
@@ -387,11 +405,11 @@ function buildBasicReportData(shipResults, projectName) {
         medium: findings.filter(f => f.severity === "WARN" || f.severity === "medium").length,
         low: findings.filter(f => f.severity === "low" || f.severity === "INFO").length,
       },
-      categoryScores: shipResults?.categoryScores || { security: 80, auth: 70, billing: 90, quality: 60 },
+      categoryScores: shipResults?.categoryScores || {},
     },
-    findings: findings.map((f, i) => ({ ...f, id: f.id || `F${String(i+1).padStart(3,"0")}` })),
-    fixEstimates: { humanReadable: "~2h", totalMinutes: 120, bySeverity: {} },
-    truthpack: { routes: 0, envVars: 0, hasAuth: false, hasBilling: false },
+    findings: findings.map((f, i) => ({ ...f, id: f.id || `F${String(i + 1).padStart(3, "0")}` })),
+    fixEstimates: { humanReadable: calculateFixTime(findings), totalMinutes: 120 },
+    truthpack: shipResults?.truthpack || null,
     reality: reality,
   };
 }
@@ -401,43 +419,76 @@ function buildBasicReportData(shipResults, projectName) {
 // ============================================================================
 
 function generateHTMLReport(reportData, opts) {
-  // Use world-class HTML generator if available
-  if (reportHtml && reportHtml.generateWorldClassHTML) {
-    return reportHtml.generateWorldClassHTML(reportData, opts);
-  }
-  
-  // Fallback to enhanced templates
-  try {
-    const templates = require("./lib/report-templates");
-    switch (opts.type) {
-      case "technical":
-        return templates.generateEnhancedTechnicalReport(convertToLegacyFormat(reportData), opts);
-      case "compliance":
-        return templates.generateEnhancedComplianceReport(convertToLegacyFormat(reportData), opts);
-      default:
-        return templates.generateEnhancedExecutiveReport(convertToLegacyFormat(reportData), opts);
-    }
-  } catch {
-    return generateBasicHTML(reportData, opts);
+  // Route to appropriate template based on type
+  switch (opts.type) {
+    case "executive":
+      if (reportTemplates?.generateEnhancedExecutiveReport) {
+        return reportTemplates.generateEnhancedExecutiveReport(
+          convertToLegacyFormat(reportData),
+          opts
+        );
+      }
+      // Fall through if template not available
+      break;
+    case "compliance":
+      if (reportTemplates?.generateEnhancedComplianceReport) {
+        return reportTemplates.generateEnhancedComplianceReport(
+          convertToLegacyFormat(reportData),
+          opts
+        );
+      }
+      // Fall through if template not available
+      break;
+    default:
+      // Technical report - ALWAYS use world-class HTML generator
+      if (reportHtml && typeof reportHtml.generateWorldClassHTML === 'function') {
+        try {
+          // Ensure reportData has required structure
+          if (!reportData.meta) {
+            reportData.meta = { projectName: 'Unknown', generatedAt: new Date().toISOString(), version: '2.0.0' };
+          }
+          if (!reportData.summary) {
+            reportData.summary = { score: 0, verdict: 'WARN', totalFindings: 0, severityCounts: {} };
+          }
+          if (!reportData.findings) {
+            reportData.findings = [];
+          }
+          
+          return reportHtml.generateWorldClassHTML(reportData, opts);
+        } catch (err) {
+          console.error(`\n  ${colors.error}${icons.error}${ansi.reset} Error generating world-class HTML: ${err.message}`);
+          if (opts.verbose) {
+            console.error(err.stack);
+          }
+          // Fall through to basic HTML only on error
+        }
+      } else {
+        console.warn(`\n  ${colors.warning}${icons.warning}${ansi.reset} report-html module not available, using basic HTML template`);
+      }
+      break;
   }
+
+  // Fallback to basic HTML only if world-class generator failed or not available
+  console.warn(`  ${ansi.dim}Using basic HTML fallback${ansi.reset}`);
+  return generateBasicHTML(reportData, opts);
 }
 
 function generateMarkdownReport(reportData, opts) {
-  if (reportEngine && reportEngine.exportToMarkdown) {
+  if (reportEngine?.exportToMarkdown) {
     return reportEngine.exportToMarkdown(reportData, opts);
   }
   return generateBasicMarkdown(reportData, opts);
 }
 
 function generateJSONReport(reportData, opts) {
-  if (reportEngine && reportEngine.exportToJSON) {
+  if (reportEngine?.exportToJSON) {
     return reportEngine.exportToJSON(reportData);
   }
   return JSON.stringify(reportData, null, 2);
 }
 
 function generateSARIFReport(reportData, opts) {
-  if (reportEngine && reportEngine.exportToSARIF) {
+  if (reportEngine?.exportToSARIF) {
     return JSON.stringify(reportEngine.exportToSARIF(reportData), null, 2);
   }
   // Basic SARIF fallback
@@ -456,29 +507,66 @@ function generateSARIFReport(reportData, opts) {
 }
 
 function generateCSVReport(reportData, opts) {
-  if (reportEngine && reportEngine.exportToCSV) {
+  if (reportEngine?.exportToCSV) {
     return reportEngine.exportToCSV(reportData);
   }
   // Basic CSV fallback
-  const headers = ["ID", "Severity", "Title", "File", "Line"];
+  const headers = ["ID", "Severity", "Title", "File", "Line", "Fix"];
   const rows = reportData.findings.map(f => [
     f.id || "",
     f.severity || "",
     `"${(f.title || f.message || "").replace(/"/g, '""')}"`,
     f.file || "",
     f.line || "",
+    `"${(f.fix || "").replace(/"/g, '""')}"`,
   ]);
   return [headers.join(","), ...rows.map(r => r.join(","))].join("\n");
 }
 
+async function generatePDFReport(reportData, opts, outputDir) {
+  // Generate HTML first
+  const htmlContent = generateHTMLReport(reportData, { ...opts, format: "html" });
+  const tempHtmlPath = path.join(outputDir, "temp_report.html");
+  fs.writeFileSync(tempHtmlPath, htmlContent);
+
+  // Try to use puppeteer for PDF generation
+  try {
+    const puppeteer = require("puppeteer");
+    const browser = await puppeteer.launch({ headless: "new" });
+    const page = await browser.newPage();
+    await page.setContent(htmlContent, { waitUntil: "networkidle0" });
+    const pdf = await page.pdf({
+      format: "Letter",
+      printBackground: true,
+      margin: { top: "0.5in", right: "0.5in", bottom: "0.5in", left: "0.5in" },
+    });
+    await browser.close();
+    
+    // Clean up temp file
+    try { fs.unlinkSync(tempHtmlPath); } catch {}
+    
+    return pdf;
+  } catch (e) {
+    // Clean up temp file
+    try { fs.unlinkSync(tempHtmlPath); } catch {}
+    
+    throw new Error(
+      `PDF generation requires puppeteer. Install with: npm install puppeteer\n` +
+      `Or use --format=html and print to PDF from your browser.`
+    );
+  }
+}
+
 function convertToLegacyFormat(reportData) {
   return {
     projectName: reportData.meta.projectName,
     generatedAt: reportData.meta.generatedAt,
+    reportId: reportData.meta.reportId,
     score: reportData.summary.score,
     verdict: reportData.summary.verdict,
     findings: reportData.findings,
     categoryScores: reportData.summary.categoryScores,
+    reality: reportData.reality,
   };
 }
 
@@ -488,18 +576,18 @@ function convertToLegacyFormat(reportData) {
 
 function generateBasicHTML(reportData, opts) {
   const { meta, summary } = reportData;
-  const verdictColor = summary.verdict === "SHIP" ? "#22c55e" : summary.verdict === "WARN" ? "#f59e0b" : "#ef4444";
-  
+  const verdictColor = summary.verdict === "SHIP" ? "#10b981" : summary.verdict === "WARN" ? "#f59e0b" : "#ef4444";
+
   return `<!DOCTYPE html>
 <html lang="en">
 <head>
   <meta charset="UTF-8">
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
-  <title>Vibecheck Report - ${meta.projectName}</title>
+  <title>VibeCheck Report - ${meta.projectName}</title>
   <style>
     body { font-family: -apple-system, sans-serif; max-width: 800px; margin: 0 auto; padding: 40px; background: #0f172a; color: #f8fafc; }
     .header { border-bottom: 2px solid #334155; padding-bottom: 20px; margin-bottom: 30px; }
-    .score { font-size: 4rem; font-weight: 800; text-align: center; margin: 40px 0; }
+    .score { font-size: 4rem; font-weight: 800; text-align: center; margin: 40px 0; color: ${verdictColor}; }
     .verdict { display: inline-block; padding: 12px 24px; border-radius: 50px; font-weight: 600; background: ${verdictColor}22; color: ${verdictColor}; }
     .section { margin: 30px 0; }
     h2 { color: #94a3b8; font-size: 0.875rem; text-transform: uppercase; letter-spacing: 0.1em; }
@@ -518,19 +606,19 @@ function generateBasicHTML(reportData, opts) {
     <h2>Findings (${summary.totalFindings})</h2>
     ${reportData.findings.slice(0, 10).map(f => `
       <div class="finding">
-        <strong>${f.severity?.toUpperCase()}</strong>: ${f.title || f.message}
+        <strong>${(f.severity || 'WARN').toUpperCase()}</strong>: ${f.title || f.message}
         ${f.file ? `<br><code>${f.file}${f.line ? `:${f.line}` : ""}</code>` : ""}
       </div>
     `).join("")}
   </div>
-  <div class="footer">Generated by Vibecheck · ${meta.reportId}</div>
+  <div class="footer">Generated by VibeCheck · ${meta.reportId}</div>
 </body>
 </html>`;
 }
 
 function generateBasicMarkdown(reportData, opts) {
   const { meta, summary, findings } = reportData;
-  return `# Vibecheck Report
+  return `# VibeCheck Report
 
 **Project:** ${meta.projectName}  
 **Generated:** ${new Date(meta.generatedAt).toLocaleString()}  
@@ -539,10 +627,10 @@ function generateBasicMarkdown(reportData, opts) {
 
 ## Findings (${summary.totalFindings})
 
-${findings.slice(0, 20).map(f => `- **${f.severity?.toUpperCase()}**: ${f.title || f.message}${f.file ? ` (\`${f.file}\`)` : ""}`).join("\n")}
+${findings.slice(0, 20).map(f => `- **${(f.severity || 'WARN').toUpperCase()}**: ${f.title || f.message}${f.file ? ` (\`${f.file}\`)` : ""}`).join("\n")}
 
 ---
-*Generated by Vibecheck · ${meta.reportId}*
+*Generated by VibeCheck · ${meta.reportId}*
 `;
 }
 
@@ -550,16 +638,30 @@ ${findings.slice(0, 20).map(f => `- **${f.severity?.toUpperCase()}**: ${f.title
 // UTILITIES
 // ============================================================================
 
+function calculateFixTime(findings) {
+  if (!findings || findings.length === 0) return "0h";
+
+  const minutes = findings.reduce((total, f) => {
+    const sev = (f.severity || "").toLowerCase();
+    const time = { block: 60, critical: 60, high: 45, warn: 20, medium: 20, info: 10, low: 10 };
+    return total + (time[sev] || 15);
+  }, 0);
+
+  if (minutes < 60) return `${minutes}m`;
+  const hours = Math.round(minutes / 60 * 10) / 10;
+  return hours > 8 ? `${Math.ceil(hours / 8)}d` : `${hours}h`;
+}
+
 function saveReportHistory(outputDir, reportData) {
   try {
     const historyDir = path.join(outputDir, "history");
     if (!fs.existsSync(historyDir)) {
       fs.mkdirSync(historyDir, { recursive: true });
     }
-    
+
     const timestamp = new Date().toISOString().split("T")[0];
     const historyFile = path.join(historyDir, `${timestamp}.json`);
-    
+
     fs.writeFileSync(historyFile, JSON.stringify({
       meta: reportData.meta,
       summary: reportData.summary,
@@ -569,16 +671,11 @@ function saveReportHistory(outputDir, reportData) {
   }
 }
 
-function printReportSummary(reportData, opts, outputFile) {
-  const { summary } = reportData;
-  const verdictColor = summary.verdict === "SHIP" ? c.green : summary.verdict === "WARN" ? c.yellow : c.red;
-  
-  console.log(`  ${c.dim}Type:${c.reset}     ${c.bold}${opts.type}${c.reset}`);
-  console.log(`  ${c.dim}Format:${c.reset}   ${c.bold}${opts.format}${c.reset}`);
-  console.log(`  ${c.dim}Score:${c.reset}    ${verdictColor}${c.bold}${summary.score}${c.reset}/100`);
-  console.log(`  ${c.dim}Verdict:${c.reset}  ${verdictColor}${c.bold}${summary.verdict}${c.reset}`);
-  console.log(`  ${c.dim}Findings:${c.reset} ${summary.totalFindings}`);
-  console.log(`\n  ${c.dim}Output:${c.reset}   ${c.cyan}${outputFile}${c.reset}\n`);
+function openInBrowser(filePath) {
+  const { exec } = require("child_process");
+  const cmd = process.platform === "darwin" ? "open" :
+              process.platform === "win32" ? "start" : "xdg-open";
+  exec(`${cmd} "${filePath}"`);
 }
 
 module.exports = { runReport };