@hublo/sentinel 1.5.0-alpha.31 → 1.5.0-alpha.33

package/dist/bin/sentinel.js

@@ -15,7 +15,7 @@ import {
   palette,
   registerAdapters,
   resolve
-} from "../chunk-PWHE527G.js";
+} from "../chunk-XQRL555B.js";
 import {
   resolveContext
 } from "../chunk-ZFNBW6S3.js";

package/dist/{chunk-PWHE527G.js → chunk-XQRL555B.js}

@@ -4824,7 +4824,7 @@ function findBuildOutput(cwd) {
   const builds = typeof pkg.scripts?.["build"] === "string" || pkg.nx?.targets?.["build"] !== void 0;
   const declaresOutputs = Array.isArray(pkg.nx?.targets?.["build"]?.["outputs"]);
   return {
-    reason: builds ? `no bundle figure: this module declares a build and wrote no entry manifest, so ${declaresOutputs ? "build it before measuring" : "declare that build's outputs on its nx target the way apps/front/career does"}. A build target with no declared \`outputs\` is still cached, and on a cache hit nx reports the task successful, restores no file and leaves the directory empty, so every job that reads the build sees nothing. Measured on this monorepo: 5 of the 48 projects with a build target are in that state.` : "no bundle figure: this module declares no build, so it writes no entry manifest and there is nothing to measure here. A component library and a back-end service are in that case by construction, and no action follows.",
+    reason: builds ? `this module declares a build and wrote no entry manifest, so ${declaresOutputs ? "build it before measuring" : "declare that build's outputs on its nx target the way apps/front/career does"}. A build target with no declared \`outputs\` is still cached, and on a cache hit nx reports the task successful, restores no file and leaves the directory empty, so every job that reads the build sees nothing. Measured on this monorepo: 5 of the 48 projects with a build target are in that state.` : "this module declares no build, so it writes no entry manifest and there is nothing to measure here. A component library and a back-end service are in that case by construction, and no action follows.",
     lookedIn
   };
 }
@@ -6084,11 +6084,26 @@ function renderMarkdown(report) {
     ...notes,
     "",
     ...families,
-    "> Measured on this branch and **not compared with `main`**. No figure here refuses anything.",
-    "> **ok**, **near** or **exceeds** is where a figure sits against a PUBLISHED reference; a",
-    "> figure with no verdict has none, which is a decision and not a gap. A file breaching several",
-    "> rules counts once per rule. The code behind every place, and the drawing of the import",
-    "> graph, are in the `static-analysis.html` the artifact carries.",
+    /*
+     * ⛔ The reserve says only what a reader of THIS comment can use, and the rest was cut.
+     *
+     * It used to spend 470 characters explaining `ok`, `near` and `exceeds` and defending the
+     * absence of a verdict as "a decision and not a gap". Measured on a five-module pull request:
+     * **not one of the 150 figures carried a verdict**, so three quarters of the reserve explained
+     * a notation that appeared nowhere in the comment, and argued with an objection a developer
+     * had not made. The legend now appears only when a verdict does.
+     *
+     * ⚠️ The non-comparison stays, and it is the one sentence here that prevents a wrong reading
+     * rather than answering a reviewer: a figure in a pull-request comment is read as a delta by
+     * default, so a module already at 6.81% looks identical to one this branch pushed there.
+     */
+    "> Measured on this branch, **not compared with `main`**: every figure is a value, not a",
+    "> change. Nothing here blocks the merge. The code behind every place, and the import graph,",
+    "> are in the `static-analysis.html` the run keeps as an artifact.",
+    ...report.rows.some((row) => row.status !== void 0) ? [
+      "> **ok**, **near** and **exceeds** place a figure against a published reference. A figure",
+      "> without one carries no verdict."
+    ] : [],
     // ⭐ The version that produced these numbers, because the thresholds a module cannot override
     // live in the tool: nine gauge limits, the complexity bands, the bundle budgets. Two recaps
     // from two versions are not comparable, and without this nothing on the comment would say so.
@@ -6285,10 +6300,11 @@ function summaryParts(report, bold) {
   const findings = report.findings ?? [];
   const inTests = findings.filter((finding) => finding.metric === "clonesTests").length;
   const inProduct = findings.length - inTests;
+  const thresholds = withReference.length === 0 ? [] : above === 0 ? [`${bold("No threshold exceeded")}: the measured levels are within the recommended norm`] : [`${bold(`${above} threshold${above === 1 ? "" : "s"} exceeded`)}`];
   return [
-    withReference.length === 0 ? "No figure here has a published reference" : `${bold(`${above} of ${withReference.length}`)} figures with a published reference are above it`,
-    `${bold(grouped(inProduct))} places in product code`,
-    ...inTests === 0 ? [] : [`${bold(grouped(inTests))} clones in tests`]
+    ...thresholds,
+    `${bold(grouped(inProduct))} place${inProduct === 1 ? "" : "s"} in product code`,
+    ...inTests === 0 ? [] : [`${bold(grouped(inTests))} clone${inTests === 1 ? "" : "s"} in tests`]
   ];
 }
 function summaryLine(report) {

package/dist/index.js

@@ -6,7 +6,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-PWHE527G.js";
+} from "./chunk-XQRL555B.js";
 import {
   BaseAdapter
 } from "./chunk-ZFNBW6S3.js";

package/package.json

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