roast-my-codebase 1.3.1 → 1.3.2

package/dist/index.js

@@ -8733,6 +8733,34 @@ function renderReport(report, options) {
     `  ${chalk4.cyan("Dev Dependencies")}   ${report.stats.devDependencies}`
   );
   sections.push("");
+  const FINDING_TO_ROAST_CATEGORY = {
+    "unused-deps": "dependencies",
+    "nextjs-metadata": "framework",
+    "nextjs-client-server": "framework",
+    "react-error-boundary": "framework",
+    "secrets": "security",
+    "env-in-git": "security",
+    "eval-usage": "security",
+    "npm-audit": "npm-audit",
+    "dep-outdated": "dep-outdated",
+    "ruby-style": "ruby-issues",
+    "php-smell": "php-issues",
+    "swift-async": "swift-issues",
+    "kotlin-coroutine": "kotlin-issues",
+    "db-n-plus-one": "database",
+    "db-sql-injection": "database",
+    "db-over-fetch": "database",
+    "db-destructive": "database"
+  };
+  const roastByFile = /* @__PURE__ */ new Map();
+  const roastByCategory = /* @__PURE__ */ new Map();
+  for (const roast of report.roasts) {
+    if (roast.target === "codebase") continue;
+    roastByCategory.set(roast.category, roast.message);
+    if (roast.target.includes("/") || roast.target.includes(".") && !roast.target.includes(" ")) {
+      roastByFile.set(roast.target, roast.message);
+    }
+  }
   const warnings = report.findings.filter((f) => f.severity === "warning");
   const criticals = report.findings.filter((f) => f.severity === "critical");
   const infos = report.findings.filter((f) => f.severity === "info");
@@ -8753,27 +8781,37 @@ function renderReport(report, options) {
       sections.push(`  ${chalk4.blue("\u25CF")} ${chalk4.blue(`${infos.length} info`)}`);
     }
     sections.push("");
+    const shownRoastKeys = /* @__PURE__ */ new Set();
     const keyFindings = [...criticals, ...warnings].slice(0, 8);
     for (const finding of keyFindings) {
       const icon = finding.severity === "critical" ? chalk4.red("\u2717") : chalk4.yellow("\u26A0");
       sections.push(`  ${icon} ${chalk4.white(finding.message)}`);
+      let roastMsg;
+      let roastKey;
+      if (finding.file && roastByFile.has(finding.file)) {
+        roastKey = `file:${finding.file}`;
+        if (!shownRoastKeys.has(roastKey)) {
+          roastMsg = roastByFile.get(finding.file);
+        }
+      }
+      if (!roastMsg) {
+        const resolvedCategory = FINDING_TO_ROAST_CATEGORY[finding.category] ?? finding.category;
+        roastKey = `cat:${resolvedCategory}`;
+        if (!shownRoastKeys.has(roastKey)) {
+          roastMsg = roastByCategory.get(resolvedCategory);
+        }
+      }
+      if (roastMsg && roastKey) {
+        sections.push(`     ${chalk4.italic.dim(roastMsg)}`);
+        shownRoastKeys.add(roastKey);
+      }
     }
     sections.push("");
   }
-  if (report.roasts.length > 0) {
-    sections.push(
-      boxen(chalk4.bold.yellow(" Roast Time ") + " " + flame(), {
-        padding: { top: 0, bottom: 0, left: 1, right: 1 },
-        borderStyle: "round",
-        borderColor: "yellow"
-      })
-    );
+  const comboRoast = report.roasts.find((r) => r.target === "codebase");
+  if (comboRoast) {
+    sections.push(chalk4.italic.yellow(`  ${comboRoast.message}`));
     sections.push("");
-    for (const roast of report.roasts) {
-      sections.push(`  ${chalk4.bold.white(roast.target)}`);
-      sections.push(`  ${chalk4.yellow(roast.message)}`);
-      sections.push("");
-    }
   }
   if (report.fixes && report.fixes.length > 0) {
     sections.push(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roast-my-codebase",
-  "version": "1.3.1",
+  "version": "1.3.2",
   "description": "Get roasted. Get better. Ship faster. A CLI that analyzes your codebase and delivers brutally honest (but funny) feedback.",
   "type": "module",
   "main": "dist/index.js",