@hublo/sentinel 1.5.0-alpha.10 → 1.5.0-alpha.12

package/dist/bin/sentinel.js

@@ -15,7 +15,7 @@ import {
   palette,
   registerAdapters,
   resolve
-} from "../chunk-FWGU7QLH.js";
+} from "../chunk-RH7TOWSN.js";
 import {
   resolveContext
 } from "../chunk-KLEXJFJ6.js";

package/dist/{chunk-FWGU7QLH.js → chunk-RH7TOWSN.js}

@@ -4864,6 +4864,16 @@ function helpFor(key) {
 }
 
 // src/roles/static-analysis/report.ts
+var SCRIPTS = { json: "static-analysis:json", html: "static-analysis:html" };
+function secondSide(finding) {
+  if (!finding.other) return "";
+  const file = (at) => at.split(":")[0] ?? at;
+  const sameFile = file(finding.other) === file(finding.path);
+  const label = sameFile ? "and line" : "and";
+  const shownSide = sameFile ? finding.other.split(":")[1] ?? finding.other : finding.other;
+  return ` <span class="unit">${label}</span> <code>${escape(shownSide)}</code>`;
+}
+var HTML_GROUP_LIMIT = 50;
 function shown(row) {
   return row.unit === "%" && typeof row.value === "number" ? row.value.toFixed(2) : String(row.value);
 }
@@ -4900,6 +4910,12 @@ function renderConsole(report) {
     }
     lines.push("    a language is absent when no file of it was long enough to scan");
   }
+  if (report.findings.length > 0) {
+    lines.push(
+      "",
+      `  ${report.findings.length} place(s) to look, listed by \`${SCRIPTS.json}\` and \`${SCRIPTS.html}\``
+    );
+  }
   const caveats = report.rows.map((row) => ({ row, help: helpFor(row.metric) })).filter(
     (entry) => Boolean(entry.help)
   );
@@ -4937,6 +4953,36 @@ function renderHtml(report) {
         <td>${setting.moved ? '<span class="moved">overridden in this module</span>' : ""}</td>
       </tr>`
   ).join("\n");
+  const groups = /* @__PURE__ */ new Map();
+  for (const finding of report.findings) {
+    const bucket = groups.get(finding.metric) ?? [];
+    bucket.push(finding);
+    groups.set(finding.metric, bucket);
+  }
+  const findings = [...groups.entries()].map(([metric, all2]) => {
+    const label = helpFor(metric)?.label ?? metric;
+    const sorted = [...all2].sort((a, b) => (b.lines ?? 0) - (a.lines ?? 0));
+    const shown2 = sorted.slice(0, HTML_GROUP_LIMIT);
+    const rest = sorted.length - shown2.length;
+    const rows2 = shown2.map(
+      (finding) => `      <tr>
+        <td><code>${escape(finding.path)}</code>${secondSide(finding)}</td>
+        <td>${finding.name ? `<code>${escape(finding.name)}</code>` : ""}</td>
+        <td class="num">${finding.lines ?? ""}</td>
+        <td class="num">${finding.confidence ?? ""}</td>
+      </tr>`
+    ).join("\n");
+    return `  <h2>${escape(label)}: where to look</h2>
+  <table>
+    <thead>
+      <tr><th scope="col">File</th><th scope="col">Name</th><th scope="col">Lines</th><th scope="col">Confidence</th></tr>
+    </thead>
+    <tbody>
+${rows2}
+    </tbody>
+  </table>
+${rest > 0 ? `  <p class="sub">The ${HTML_GROUP_LIMIT} longest are listed; ${rest} more are in <code>${SCRIPTS.json}</code>.</p>` : ""}`;
+  }).join("\n");
   return `<!doctype html>
 <html lang="en">
 <head>
@@ -4985,6 +5031,7 @@ ${rows}
 ${settings}
     </tbody>
   </table>
+${findings}
 </main>
 </body>
 </html>
@@ -5403,7 +5450,7 @@ ${result.stderr ?? ""}`
       const category = finding.category ?? "unknown";
       byCategory[category] = (byCategory[category] ?? 0) + 1;
     }
-    return { kind: "figures", statistics, byCategory };
+    return { kind: "figures", statistics, byCategory, findings: parsed.findings ?? [] };
   } finally {
     dispose();
   }
@@ -5437,7 +5484,12 @@ ${result.stderr ?? ""}`
       );
       return { kind: "failed", code: 1 };
     }
-    return { kind: "figures", total, formats: parsed?.statistics?.formats ?? {} };
+    return {
+      kind: "figures",
+      total,
+      formats: parsed?.statistics?.formats ?? {},
+      duplicates: parsed?.duplicates ?? []
+    };
   } finally {
     dispose();
   }
@@ -5449,6 +5501,32 @@ function safeParse(text) {
     return void 0;
   }
 }
+function placesToLook(product, tests, dead) {
+  const at = (side) => `${side?.name ?? "(unknown)"}${side?.start === void 0 ? "" : `:${side.start}`}`;
+  const clones = (outcome, metric) => outcome.kind === "figures" ? outcome.duplicates.map((entry) => ({
+    metric,
+    path: at(entry.firstFile),
+    other: at(entry.secondFile),
+    ...entry.lines === void 0 ? {} : { lines: entry.lines }
+  })) : [];
+  const deadMetric = {
+    "unused-file": "unusedFiles",
+    "unused-export": "unusedExports",
+    "unused-symbol": "unusedSymbols",
+    "unused-import": "unusedImports"
+  };
+  return [
+    ...clones(product, "clones"),
+    ...clones(tests, "clonesTests"),
+    ...dead.kind === "figures" ? dead.findings.map((finding) => ({
+      metric: deadMetric[finding.category ?? ""] ?? "deadLinesPercent",
+      path: finding.path ?? "(unknown)",
+      ...finding.name ? { name: finding.name } : {},
+      ...finding.lines === void 0 ? {} : { lines: finding.lines },
+      ...finding.confidence === void 0 ? {} : { confidence: finding.confidence }
+    })) : []
+  ];
+}
 function assemble(ctx, config, product, tests, dead) {
   const share = (total) => Number((total.percentage ?? 0).toFixed(2));
   const rows = [
@@ -5508,6 +5586,7 @@ function assemble(ctx, config, product, tests, dead) {
     rows,
     notes,
     settings: duplicationSettings(config),
+    findings: placesToLook(product, tests, dead),
     byLanguage
   };
 }

package/dist/index.js

@@ -6,7 +6,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-FWGU7QLH.js";
+} from "./chunk-RH7TOWSN.js";
 import {
   BaseAdapter
 } from "./chunk-KLEXJFJ6.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.5.0-alpha.10",
+  "version": "1.5.0-alpha.12",
   "description": "One CLI that guards code health across Hublo repos: shared lint/typescript/build/test presets, static & dynamic analysis, and architecture checks.",
   "type": "module",
   "license": "MIT",