@hublo/sentinel 1.5.0-alpha.20 → 1.5.0-alpha.21

package/dist/bin/sentinel.js

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

package/dist/{chunk-ZN32JGFP.js → chunk-AFM4AJHA.js}

@@ -5433,10 +5433,10 @@ var metricsHelp = {
   },
   unusedFiles: {
     label: "Unreachable files",
-    what: "Files no entry point reaches, each with a confidence the tool assigns.",
-    read: "The most actionable of the four, and the one to start from: a whole file nothing reaches is a decision, not a cleanup. Open the ones with the highest confidence first.",
-    caveat: "Three classes hide in this count and only one of them is a deletion, so read the reason the tool gives before acting: a file nothing reaches at all, a file only a spec or a story reaches, which is alive and breaks if removed, and an entry point of a framework this role has not declared. The tool labels the second itself as `used-only-by-tests`, and that is not a corner: measured over 1 231 findings on this repository, 659 of them carry it, 53.5%. A finding with no reason given is the one to open first.",
-    source: "jscpd dead-code findings of category `unused-file`, with the engine's own `reasons` and `message` carried through. The 53.5% was counted over every dead-code report produced on this repository; the third class was separated by hand on `apps/front/console`, where three of five findings were its TanStack bootstrap."
+    what: "Files of this module that nothing reaches: not its own code, not another module, not a spec.",
+    read: "A candidate for deletion, and the only one of the dead-code rows that names whole files. Zero is the ordinary value for an application whose entry points are declared.",
+    caveat: "\u26A0\uFE0F Two classes remain in it and only one is a deletion: a file nothing reaches at all, and an entry point of a framework this role has not been told about, which is a missing declaration rather than dead code. A THIRD class used to sit here and no longer does: a file reached only by a spec, which the tool labels `used-only-by-tests` itself, is now counted on its own row instead. Measured on `libs/nest/auth`, that moved 5 files out of 20, and the figure now agrees with the reason printed beside each finding instead of contradicting it.",
+    source: "jscpd's dead-code walk, category `unused-file`, minus the findings it labels `used-only-by-tests`. The walk starts from the module's declared entry points PLUS the files another module imports, which come from the workspace index: without those seeds a library has no starting point at all and reports 87 unreachable files where 12 are, measured on `libs/nest/auth`."
   },
   unusedExports: {
     label: "Unused exports",
@@ -6701,6 +6701,11 @@ function assembleReport(ctx, config, product, tests, dead, complexity, graph, co
   if (unreadable) {
     notes.push(unreadableNote(coverage));
   }
+  const toolTestOnly = new Set(
+    dead.kind === "figures" ? dead.findings.filter(
+      (finding) => finding.category === "unused-file" && finding.reasons?.includes("used-only-by-tests") === true
+    ).map((finding) => finding.path ?? "").filter(Boolean) : []
+  );
   if (dead.kind === "figures" && !unreadable) {
     const stats = dead.statistics;
     rows.push(
@@ -6710,14 +6715,23 @@ function assembleReport(ctx, config, product, tests, dead, complexity, graph, co
         unit: `of ${stats.files ?? 0} files`
       },
       { metric: "deadLinesPercent", value: share(stats), unit: "%" },
-      { metric: "unusedFiles", value: dead.byCategory["unused-file"] ?? 0, unit: "files" },
+      // ⛔ Minus the files the TOOL itself labels `used-only-by-tests`. They were counted as
+      // unreachable while the tool knew better: measured on `libs/nest/auth`, 5 of the 20. A file a
+      // spec reaches is not unreachable, and leaving it in the headline count meant the figure
+      // disagreed with the reason printed beside it.
+      {
+        metric: "unusedFiles",
+        value: Math.max((dead.byCategory["unused-file"] ?? 0) - toolTestOnly.size, 0),
+        unit: "files"
+      },
       { metric: "unusedExports", value: dead.byCategory["unused-export"] ?? 0, unit: "exports" },
       { metric: "unusedSymbols", value: dead.byCategory["unused-symbol"] ?? 0, unit: "symbols" },
       { metric: "unusedImports", value: dead.byCategory["unused-import"] ?? 0, unit: "imports" }
     );
   }
-  if (outside.testOnly.size > 0 && !unreadable) {
-    rows.push({ metric: "usedOnlyByTests", value: outside.testOnly.size, unit: "files" });
+  const testOnly = /* @__PURE__ */ new Set([...toolTestOnly, ...outside.testOnly]);
+  if (testOnly.size > 0 && !unreadable) {
+    rows.push({ metric: "usedOnlyByTests", value: testOnly.size, unit: "files" });
   }
   if (complexity.functions > 0) {
     rows.push({ metric: "complexityMax", value: complexity.max, unit: "max per function" });

package/dist/index.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.5.0-alpha.20",
+  "version": "1.5.0-alpha.21",
   "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",