@hublo/sentinel 1.0.1 → 1.0.2

package/dist/bin/sentinel.js

@@ -1,5 +1,8 @@
 #!/usr/bin/env node
 import {
+  FLAVOURS,
+  TARGETS,
+  VERBS,
   availableTargets,
   describeFramework,
   dispatch,
@@ -11,7 +14,7 @@ import {
   registerAdapters,
   resolve,
   resolveBin
-} from "../chunk-HKGCWPRT.js";
+} from "../chunk-SK6E5EM4.js";
 
 // bin/sentinel.ts
 import { program } from "commander";
@@ -106,20 +109,6 @@ function resolveContext(cwd2, opts2) {
   );
 }
 
-// src/core/domain.ts
-var VERBS = ["run", "inspect", "init", "migrate", "report", "status"];
-var TARGETS = [
-  "lint",
-  "format",
-  "typescript",
-  "build",
-  "test",
-  "static-analysis",
-  "runtime-analysis",
-  "arch"
-];
-var FLAVOURS = ["react", "nest", "svelte", "node"];
-
 // src/core/orchestrate.ts
 async function analyse(params) {
   const results = [];
@@ -141,14 +130,6 @@ async function analyse(params) {
         );
       }
     }
-    const ctx = {
-      module: module.name,
-      cwd: module.root,
-      flavour: flavour2,
-      ci: params.ci,
-      fix: params.fix,
-      maxDiagnostics: params.maxDiagnostics
-    };
     for (const target of params.targets) {
       let adapter;
       try {
@@ -167,13 +148,22 @@ async function analyse(params) {
         }
         continue;
       }
+      const effectiveFlavour = params.flavour ?? adapter.declaredFlavour?.(module.root) ?? flavour2;
+      const ctx = {
+        module: module.name,
+        cwd: module.root,
+        flavour: effectiveFlavour,
+        ci: params.ci,
+        fix: params.fix,
+        maxDiagnostics: params.maxDiagnostics
+      };
       try {
         if (params.verb === "run") {
           if (params.modules.length > 1) {
             const err = palette(process.stderr);
             process.stderr.write(
               `
-  ${err.accent("\u25B6")} ${err.strong(module.name)} ${err.dim(`(${flavour2})`)} ${target}
+  ${err.accent("\u25B6")} ${err.strong(module.name)} ${err.dim(`(${effectiveFlavour})`)} ${target}
 `
             );
           }
@@ -181,7 +171,7 @@ async function analyse(params) {
           results.push({
             project: module.name,
             target,
-            flavour: flavour2,
+            flavour: effectiveFlavour,
             ok: result.ok,
             status: result.ok ? "ok" : "failed",
             data: {}
@@ -192,7 +182,7 @@ async function analyse(params) {
           results.push({
             project: module.name,
             target,
-            flavour: flavour2,
+            flavour: effectiveFlavour,
             ok: result.ok,
             status: result.ok ? "ok" : "failed",
             data: result.metrics ?? {}
@@ -203,7 +193,7 @@ async function analyse(params) {
           results.push({
             project: module.name,
             target,
-            flavour: flavour2,
+            flavour: effectiveFlavour,
             ok: true,
             status: "ok",
             data: config
@@ -214,7 +204,7 @@ async function analyse(params) {
           results.push({
             project: module.name,
             target,
-            flavour: flavour2,
+            flavour: effectiveFlavour,
             ok,
             status: ok ? "ok" : "failed",
             data: status
@@ -226,7 +216,7 @@ async function analyse(params) {
         results.push({
           project: module.name,
           target,
-          flavour: flavour2,
+          flavour: effectiveFlavour,
           ok: false,
           status: "failed",
           data: { error: message }
@@ -529,8 +519,13 @@ async function runVerb() {
     maxDiagnostics,
     ci: Boolean(opts.ci),
     fix: Boolean(opts.fix),
-    onProgress: (done, total, name) => process.stderr.write(err.dim(`  [${done}/${total}] ${name}
-`))
+    // Per-module progress is a HUMAN affordance: it tells someone staring at a slow sweep
+    // that it is alive. On this monorepo's 441 projects it is 441 lines that bury the four
+    // that matter, and it is pure noise in a captured log. So emit it only for an
+    // interactive terminal; a pipe, CI, or a generated trace gets the results and the
+    // one-line summary, which is all any of them can use.
+    onProgress: process.stderr.isTTY ? (done, total, name) => process.stderr.write(err.dim(`  [${done}/${total}] ${name}
+`)) : void 0
   });
   const summary = generateSummaries(results, targets);
   if (opts.json && verb !== "run") {

package/dist/{chunk-HKGCWPRT.js → chunk-SK6E5EM4.js}

@@ -62,6 +62,20 @@ var BaseAdapter = class {
   }
 };
 
+// src/core/domain.ts
+var VERBS = ["run", "inspect", "init", "migrate", "report", "status"];
+var TARGETS = [
+  "lint",
+  "format",
+  "typescript",
+  "build",
+  "test",
+  "static-analysis",
+  "runtime-analysis",
+  "arch"
+];
+var FLAVOURS = ["react", "nest", "svelte", "node"];
+
 // src/shared/color.ts
 import { styleText } from "util";
 function palette(stream) {
@@ -294,6 +308,20 @@ var TscAdapter = class extends BaseAdapter {
   appliesTo(_flavour) {
     return true;
   }
+  /**
+   * The flavour read from the committed `extends` chain
+   * (`@hublo/sentinel/tsconfig/nest` -> `nest`), or undefined when the module has not
+   * adopted a preset, so the engine falls back to dependency detection.
+   *
+   * This is the same detection-free read `--status` uses, and it is why an adopted React app
+   * reports `react` even in a monorepo that hoists `react` to the root.
+   */
+  declaredFlavour(cwd) {
+    const { preset } = readTsconfigAdoption(cwd);
+    if (!preset) return void 0;
+    const name = preset.slice(preset.lastIndexOf("/") + 1);
+    return FLAVOURS.includes(name) ? name : void 0;
+  }
   /**
    * Plan `--init`: make the module extend the sentinel preset with a THIN,
    * conformant stub, route type-checking through the CLI, and pin the
@@ -745,8 +773,9 @@ async function dispatch(opts) {
   if (opts.verb !== "init") {
     throw new Error(`dispatch handles --init only; --${opts.verb} routes through analyse()`);
   }
-  const flavour = resolveFlavour(opts);
-  const adapter = resolve(opts.target, flavour, opts.runner);
+  const detected = resolveFlavour(opts);
+  const adapter = resolve(opts.target, detected, opts.runner);
+  const flavour = opts.flavour ?? adapter.declaredFlavour?.(opts.cwd) ?? detected;
   const context = { cwd: opts.cwd, flavour };
   const plan = await adapter.plan(context);
   if (plan.blocked) {
@@ -772,6 +801,9 @@ export {
   availableTargets,
   resolve,
   BaseAdapter,
+  VERBS,
+  TARGETS,
+  FLAVOURS,
   palette,
   readOwnPackage,
   readOwnVersion,
@@ -783,4 +815,4 @@ export {
   detectFramework,
   dispatch
 };
-//# sourceMappingURL=chunk-HKGCWPRT.js.map
+//# sourceMappingURL=chunk-SK6E5EM4.js.map

package/dist/index.d.ts

@@ -169,6 +169,17 @@ interface Adapter {
      * adapters can share a (target, runner) if they specialise different flavours.
      */
     appliesTo(flavour: Flavour): boolean;
+    /**
+     * The flavour this module has COMMITTED to, read from its own config, or undefined when
+     * it has not adopted a preset yet. Optional: an adapter implements it only when its tool's
+     * config states the flavour unambiguously.
+     *
+     * The engine prefers this over dependency detection, because in a monorepo dependencies are
+     * hoisted to the root: `host-admin` declares no `react`, so detection honestly answers
+     * `node` while the committed `extends` says `.../tsconfig/react`. Reporting the detected
+     * value there would show every adopted module as `node`, which is simply untrue.
+     */
+    declaredFlavour?(cwd: string): Flavour | undefined;
     /**
      * Plan what `--init` should write, as declarative operations (see
      * `FileOperation`). Given a normalized `UpdateContext`, the adapter may READ the

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-HKGCWPRT.js";
+} from "./chunk-SK6E5EM4.js";
 export {
   BaseAdapter,
   all,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "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",