@hublo/sentinel 1.4.0-alpha.21 → 1.4.0-alpha.23

package/dist/bin/sentinel.js

@@ -16,7 +16,7 @@ import {
   registerAdapters,
   resolve,
   resolveBin
-} from "../chunk-7LE72AFG.js";
+} from "../chunk-7ADFV7P4.js";
 import {
   WORKSPACE_ROOT_MARKER,
   ensureWorkspacePrep,

package/dist/{chunk-7LE72AFG.js → chunk-7ADFV7P4.js}

@@ -5406,9 +5406,7 @@ function registerLint() {
 
 // src/roles/test/adapters/vitest/vitest.adapter.ts
 import { spawnSync as spawnSync5 } from "node:child_process";
-import { mkdtempSync as mkdtempSync2, readFileSync as readFileSync31 } from "node:fs";
-import { tmpdir as tmpdir2 } from "node:os";
-import { isAbsolute as isAbsolute3, join as join39, resolve as resolve6 } from "node:path";
+import { isAbsolute as isAbsolute3, resolve as resolve6 } from "node:path";
 
 // src/roles/test/presets/toolchain.json
 var toolchain_default2 = {
@@ -7848,11 +7846,13 @@ function concurrencyBlock(cwd, reading, target) {
   if (declared !== void 0) {
     return declared === "1" ? `        fileParallelism: false,` : `        maxWorkers: ${declared},`;
   }
-  const ci = projectTargets(cwd)?.[target]?.configurations?.ci;
-  if (ci === void 0) return "";
-  if (ci.runInBand === true) return `        fileParallelism: false,`;
-  if (typeof ci.maxWorkers === "number") {
-    return ci.maxWorkers === 1 ? `        fileParallelism: false,` : `        maxWorkers: ${ci.maxWorkers},`;
+  const nxTarget = projectTargets(cwd)?.[target];
+  for (const level of [nxTarget?.options, nxTarget?.configurations?.ci]) {
+    if (level === void 0) continue;
+    if (level.runInBand === true) return `        fileParallelism: false,`;
+    if (typeof level.maxWorkers === "number") {
+      return level.maxWorkers === 1 ? `        fileParallelism: false,` : `        maxWorkers: ${level.maxWorkers},`;
+    }
   }
   return "";
 }
@@ -8703,166 +8703,6 @@ function readSnapshot(report) {
     claimedFailing: typeof claimed === "number" ? claimed : failing.length
   };
 }
-function difference(left, right) {
-  const remaining = /* @__PURE__ */ new Map();
-  for (const name of right) remaining.set(name, (remaining.get(name) ?? 0) + 1);
-  const missing = [];
-  for (const name of left) {
-    const left_ = remaining.get(name) ?? 0;
-    if (left_ === 0) {
-      missing.push(name);
-      continue;
-    }
-    remaining.set(name, left_ - 1);
-  }
-  return missing.sort();
-}
-var withoutRenderingQuotes = (name) => name.replaceAll(String.raw`\'`, "'").replaceAll(String.raw`\"`, '"').replaceAll(/['"`]\s*\+\s*['"`]/g, "").replaceAll(/(^|[\s:,[{(])(''|"")($|[\s,)\]}])/g, "$1<empty-string>$3").replaceAll(/['"`]/g, "");
-function withoutRenderingNoise(name) {
-  const spelled = withSortedKeys(
-    withoutUndefinedEntries(
-      withoutErrorRendering(
-        withoutNaN(
-          withoutTimestamps(withoutRenderingQuotes(withoutClassBodies(withoutStackFrames(name))))
-        )
-      )
-    )
-  );
-  return withoutArrayBrackets(
-    spelled.replaceAll(/\[\s+/g, "[").replaceAll(/\s+\]/g, "]").replaceAll(/\{\s+/g, "{").replaceAll(/\s+\}/g, "}").replaceAll(/(?<=[[{,\s])\+0\b/g, "0").replaceAll(/:\s+/g, ":").replaceAll(/,\s+/g, ",")
-  );
-}
-function withoutArrayBrackets(name) {
-  return name.replaceAll(
-    /\[([^[\]]*)\]/g,
-    (match, inner) => (
-      // `[Object]` and `[Array]` are the formatter saying it STOPPED, not an array literal.
-      // Stripping their brackets would erase the marker `collapsedOf` pairs on.
-      /^(?:Object|Array)$/.test(inner) ? match : inner.replaceAll(/,\s+/g, ",")
-    )
-  );
-}
-var withoutClassBodies = (name) => name.replaceAll(
-  /class\s+([A-Za-z_$][\w$]*)\s*\{\s*\}/g,
-  (_match, className) => `class ${className}`
-).replaceAll(
-  /class\s+([A-Za-z_$][\w$]*)[\s\S]*?\n\}/g,
-  (_match, className) => `class ${className}`
-);
-var withoutTimestamps = (name) => name.replaceAll(/\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d+)?Z/g, "<instant>");
-var withoutNaN = (name) => name.replaceAll(/\bNaN\b/g, "null");
-function entriesOf(inner) {
-  const parts = [];
-  let depth = 0;
-  let start = 0;
-  for (let index = 0; index < inner.length; index += 1) {
-    const character = inner[index];
-    if (character === "[" || character === "(") depth += 1;
-    else if (character === "]" || character === ")") depth -= 1;
-    else if (character === "," && depth === 0) {
-      parts.push(inner.slice(start, index));
-      start = index + 1;
-    }
-    if (depth < 0) return void 0;
-  }
-  if (depth !== 0) return void 0;
-  parts.push(inner.slice(start));
-  return parts.map((part) => part.trim()).filter(Boolean);
-}
-var sortedGroup = (match, inner) => {
-  if (!inner.includes(":")) return match;
-  const parts = entriesOf(inner);
-  if (parts === void 0) return match;
-  if (parts.length < 2 || !parts.every((part) => part.includes(":"))) return match;
-  return `{${[...parts].sort().join(",")}}`;
-};
-var HELD = "\u241F";
-var withSortedKeys = (name) => {
-  const held = [];
-  let current = name;
-  for (let depth = 0; depth < 24; depth += 1) {
-    const next = current.replaceAll(/\{([^{}]*)\}/g, (match, inner) => {
-      held.push(sortedGroup(match, inner));
-      return `${HELD}${held.length - 1}${HELD}`;
-    });
-    if (next === current) break;
-    current = next;
-  }
-  for (let depth = 0; depth < 24 && current.includes(HELD); depth += 1) {
-    current = current.replaceAll(
-      new RegExp(`${HELD}(\\d+)${HELD}`, "g"),
-      (_match, index) => held[Number(index)]
-    );
-  }
-  return current;
-};
-var withoutStackFrames = (name) => {
-  const frame = name.search(/\n\s+at\s/);
-  return frame === -1 ? name : name.slice(0, frame);
-};
-var withoutErrorRendering = (name) => name.replaceAll(/\[[A-Za-z]*(?:Error|Exception):[^\][]*\]/g, "{}");
-var withoutUndefinedEntries = (name) => name.replaceAll(/([{,])\s*[A-Za-z_$][\w$]*\s*:\s*undefined\s*(?=[,}])/g, "$1").replaceAll(/\{\s*,/g, "{").replaceAll(/,\s*\}/g, "}").replaceAll(/\{\s*\}/g, "{}");
-function collapsedOf(candidate, name) {
-  if (!/\[(?:Object|Array)\]/.test(candidate)) return false;
-  const pattern = candidate.split(/\[(?:Object|Array)\]/).map((part) => part.replaceAll(/[.*+?^${}()|[\]\\]/g, "\\$&")).join(".*");
-  return new RegExp(`^${pattern}$`, "s").test(name);
-}
-function truncationOf(candidate, name) {
-  const cut = candidate.indexOf("\u2026");
-  if (cut === -1) return false;
-  const head = candidate.slice(0, cut);
-  const tail = candidate.slice(cut + 1);
-  return name.startsWith(head) && name.endsWith(tail);
-}
-function pairRenamed(missing, added) {
-  const remaining = [...added];
-  const renamed = [];
-  const unmatched = [];
-  for (const name of missing) {
-    const normalised = withoutRenderingNoise(name);
-    const at = remaining.findIndex((candidate) => withoutRenderingNoise(candidate) === normalised);
-    if (at === -1) {
-      unmatched.push(name);
-      continue;
-    }
-    renamed.push({ before: name, after: remaining[at] });
-    remaining.splice(at, 1);
-  }
-  const lost = [];
-  for (const name of unmatched) {
-    const normalised = withoutRenderingNoise(name);
-    const at = remaining.findIndex(
-      (candidate) => truncationOf(withoutRenderingNoise(candidate), normalised) || collapsedOf(normalised, withoutRenderingNoise(candidate))
-    );
-    if (at === -1) {
-      lost.push(name);
-      continue;
-    }
-    renamed.push({ before: name, after: remaining[at] });
-    remaining.splice(at, 1);
-  }
-  return { renamed, missing: lost, added: remaining };
-}
-function compareSnapshots(before, after) {
-  const beforeAll = [...before.passing, ...before.failing];
-  const afterAll = [...after.passing, ...after.failing];
-  const paired = pairRenamed(difference(beforeAll, afterAll), difference(afterAll, beforeAll));
-  const renamedBefore = new Set(paired.renamed.map((pair) => pair.before));
-  const regressed = difference(before.passing, after.passing).filter(
-    (name) => !paired.missing.includes(name) && !renamedBefore.has(name)
-  );
-  const failedBefore = new Set(before.failing);
-  const recovered = after.passing.filter((name) => failedBefore.has(name));
-  return {
-    identical: paired.missing.length === 0 && paired.added.length === 0 && regressed.length === 0,
-    missing: paired.missing,
-    added: paired.added,
-    regressed,
-    renamed: paired.renamed,
-    recovered,
-    alreadyFailing: before.failing.length
-  };
-}
 function unusableAsBaseline(snapshot) {
   if (snapshot.unloadable.length > 0) {
     return `${snapshot.unloadable.length} file(s) in the reference run did not load, so they reported no test at all and the run is short by however many they hold: ${snapshot.unloadable.slice(0, 3).join(", ")}. A reference that quietly lost tests cannot prove anything about the run after it`;
@@ -8875,45 +8715,6 @@ function unusableAsBaseline(snapshot) {
   }
   return void 0;
 }
-function renamedExample(before, after) {
-  let shared = 0;
-  while (shared < before.length && shared < after.length && before[shared] === after[shared]) {
-    shared += 1;
-  }
-  const boundary = before.lastIndexOf(" ", shared);
-  const from = boundary > 0 ? boundary + 1 : 0;
-  const ellipsis = from > 0 ? "\u2026" : "";
-  return `"${ellipsis}${before.slice(from)}" -> "${ellipsis}${after.slice(from)}"`;
-}
-function describeComparison(comparison) {
-  const firstRename = comparison.renamed[0];
-  const renamedNote = firstRename === void 0 ? "" : ` ${comparison.renamed.length} test(s) kept their identity and changed name, because the runners render an each title's interpolated value differently (e.g. ${renamedExample(firstRename.before, firstRename.after)}).`;
-  const alreadyRedNote = comparison.alreadyFailing > 0 ? ` ${comparison.alreadyFailing} test(s) were already failing before the migration and are outside this comparison, which covers only what passed.` : "";
-  const recoveredNote = comparison.recovered.length > 0 ? ` ${comparison.recovered.length} test(s) were failing before and pass now.` : "";
-  if (comparison.identical) {
-    return `every test that ran before runs and passes now.${recoveredNote}${renamedNote}${alreadyRedNote}`;
-  }
-  const parts = [];
-  if (comparison.recovered.length > 0) {
-    parts.push(`${comparison.recovered.length} test(s) were failing before and pass now`);
-  }
-  if (comparison.missing.length > 0) {
-    parts.push(
-      `${comparison.missing.length} test(s) no longer exist: ${comparison.missing.slice(0, 3).join(", ")}`
-    );
-  }
-  if (comparison.added.length > 0) {
-    parts.push(
-      `${comparison.added.length} test(s) appeared, which a migration should not do: ${comparison.added.slice(0, 3).join(", ")}`
-    );
-  }
-  if (comparison.regressed.length > 0) {
-    parts.push(
-      `${comparison.regressed.length} test(s) passed before and do not now: ${comparison.regressed.slice(0, 3).join(", ")}`
-    );
-  }
-  return `${parts.join("; ")}.${renamedNote}${alreadyRedNote}`;
-}
 
 // src/roles/test/reference.ts
 function referencePath(cwd) {
@@ -8971,26 +8772,6 @@ function captureReference(cwd, config) {
     message: `recorded ${snapshot.passing.length} passing test name(s) from ${config} as the reference. The next \`sentinel --run --test\` compares against it.`
   };
 }
-function hasReference(cwd) {
-  return existsSync31(referencePath(cwd));
-}
-function compareAgainstReference(cwd, report) {
-  const path = referencePath(cwd);
-  let stored;
-  try {
-    stored = JSON.parse(readFileSync30(path, "utf8"));
-  } catch {
-    rmSync3(path, { force: true });
-    return { ok: true, message: `the recorded reference could not be read, so nothing was proved.` };
-  }
-  const comparison = compareSnapshots(stored.snapshot, readSnapshot(report));
-  if (comparison.identical) rmSync3(path, { force: true });
-  const kept = comparison.identical ? "" : ` The reference is KEPT until a run passes it, so this does not go green by running again. Fix the suite and re-run, or delete ${path} to abandon the proof on purpose.`;
-  return {
-    ok: comparison.identical,
-    message: `${describeComparison(comparison)} (reference: ${stored.config}, ${stored.takenAt})${kept}`
-  };
-}
 
 // src/roles/test/adapters/vitest/vitest.adapter.ts
 function resolveVitest(cwd) {
@@ -9144,31 +8925,13 @@ var VitestAdapter = class extends BaseAdapter {
     });
     const { cwd: runFrom, options, paths } = fromWorkspaceRoot(ctx.cwd, typedOptions, typedPaths);
     const watching = options.includes("--watch") || options.includes("-w");
-    const proving = !watching && hasReference(ctx.cwd);
-    const reportFile = proving ? join39(mkdtempSync2(join39(tmpdir2(), "sentinel-run-")), "vitest.json") : void 0;
-    const reporters = reportFile === void 0 ? [] : ["--reporter=default", "--reporter=json", `--outputFile=${reportFile}`];
-    const argv = watching ? [...options, ...paths] : ["run", ...reporters, ...options, ...paths];
+    const argv = watching ? [...options, ...paths] : ["run", ...options, ...paths];
     const result = spawnSync5(bin, argv, { cwd: runFrom, stdio: "inherit" });
     if (result.error) {
       this.say(`could not run vitest (${result.error.message})`);
       return Promise.resolve({ ok: false, code: 1 });
     }
     const code = result.status ?? 1;
-    if (reportFile !== void 0) {
-      let report;
-      try {
-        report = JSON.parse(readFileSync31(reportFile, "utf8"));
-      } catch {
-        report = void 0;
-      }
-      if (report === void 0) {
-        this.say(`vitest wrote no report, so this run proved nothing against the reference.`);
-      } else {
-        const outcome = compareAgainstReference(ctx.cwd, report);
-        this.say(outcome.message);
-        if (!outcome.ok) return Promise.resolve({ ok: false, code: code === 0 ? 1 : code });
-      }
-    }
     return Promise.resolve({ ok: code === 0, code });
   }
   say(message) {
@@ -9210,9 +8973,9 @@ function registerTest() {
 
 // src/roles/typescript/adapters/tsc/tsc.adapter.ts
 import { spawnSync as spawnSync6 } from "node:child_process";
-import { existsSync as existsSync32, readFileSync as readFileSync33 } from "node:fs";
+import { existsSync as existsSync32, readFileSync as readFileSync32 } from "node:fs";
 import { createRequire as createRequire7 } from "node:module";
-import { join as join41 } from "node:path";
+import { join as join40 } from "node:path";
 
 // src/roles/typescript/presets/nest.json
 var nest_default2 = {
@@ -9393,8 +9156,8 @@ function readTsconfigAdoption(cwd) {
 }
 
 // src/roles/typescript/adapters/tsc/plan.ts
-import { readFileSync as readFileSync32 } from "node:fs";
-import { join as join40 } from "node:path";
+import { readFileSync as readFileSync31 } from "node:fs";
+import { join as join39 } from "node:path";
 
 // src/roles/typescript/typecheck-script.ts
 var SENTINEL_TYPECHECK_COMMAND = "sentinel --run --typescript";
@@ -9487,7 +9250,7 @@ function planAdoption(context) {
     };
   }
   const existing = parseJsonc(
-    readFileSync32(join40(context.cwd, target.path), "utf8"),
+    readFileSync31(join39(context.cwd, target.path), "utf8"),
     target.path
   );
   const extendsChain = composeExtends(existing.extends, preset);
@@ -9625,7 +9388,7 @@ var TscAdapter = class _TscAdapter extends BaseAdapter {
     let chain;
     try {
       const parsed = parseJsonc(
-        readFileSync33(join41(cwd, target.path), "utf8"),
+        readFileSync32(join40(cwd, target.path), "utf8"),
         target.path
       );
       chain = parsed.extends;
@@ -9638,7 +9401,7 @@ var TscAdapter = class _TscAdapter extends BaseAdapter {
     );
     if (preset === void 0) return void 0;
     try {
-      createRequire7(join41(cwd, "noop.js")).resolve(preset);
+      createRequire7(join40(cwd, "noop.js")).resolve(preset);
       return void 0;
     } catch {
       return preset;
@@ -9734,7 +9497,7 @@ var TscAdapter = class _TscAdapter extends BaseAdapter {
   referencedProjects(cwd, config) {
     try {
       const parsed = parseJsonc(
-        readFileSync33(join41(cwd, config), "utf8"),
+        readFileSync32(join40(cwd, config), "utf8"),
         config
       );
       const referenced = (parsed.references ?? []).map((reference) => reference.path).filter((path) => typeof path === "string" && path.length > 0);
@@ -9750,7 +9513,7 @@ var TscAdapter = class _TscAdapter extends BaseAdapter {
    * check.
    */
   typecheckTarget(cwd) {
-    if (existsSync32(join41(cwd, "tsconfig.json"))) return "tsconfig.json";
+    if (existsSync32(join40(cwd, "tsconfig.json"))) return "tsconfig.json";
     const target = resolveTsconfigTarget(cwd);
     return target.reason === "none" ? null : target.path;
   }
@@ -9955,7 +9718,7 @@ function replaceLines(current, replacements) {
 }
 
 // src/core/apply-plan.ts
-import { existsSync as existsSync33, readFileSync as readFileSync34, renameSync, rmSync as rmSync4, writeFileSync as writeFileSync4 } from "node:fs";
+import { existsSync as existsSync33, readFileSync as readFileSync33, renameSync, rmSync as rmSync4, writeFileSync as writeFileSync4 } from "node:fs";
 import { resolve as resolve7, sep } from "node:path";
 import { applyEdits as applyEdits3, findNodeAtLocation, modify, parseTree } from "jsonc-parser";
 
@@ -9974,7 +9737,7 @@ function resolveWithinRoot(cwd, relativePath) {
   return absolutePath;
 }
 function readIfExists(absolutePath) {
-  return existsSync33(absolutePath) ? readFileSync34(absolutePath, "utf8") : void 0;
+  return existsSync33(absolutePath) ? readFileSync33(absolutePath, "utf8") : void 0;
 }
 function* leaves(value, prefix = []) {
   for (const [key, keyValue] of Object.entries(value)) {
@@ -10092,8 +9855,8 @@ function applyPlan(cwd, plan4) {
 }
 
 // src/core/config/preset-evidence.ts
-import { existsSync as existsSync34, readdirSync as readdirSync9, readFileSync as readFileSync35 } from "node:fs";
-import { join as join42 } from "node:path";
+import { existsSync as existsSync34, readdirSync as readdirSync9, readFileSync as readFileSync34 } from "node:fs";
+import { join as join41 } from "node:path";
 var PATH_SIGNALS = [
   {
     preset: "nest",
@@ -10127,12 +9890,12 @@ function sourceImports(cwd, pattern) {
     }
     for (const entry of entries) {
       if (entry.isDirectory()) {
-        if (!NOT_SCANNED.has(entry.name)) stack.push(join42(dir, entry.name));
+        if (!NOT_SCANNED.has(entry.name)) stack.push(join41(dir, entry.name));
         continue;
       }
       if (!SOURCE_FILE2.test(entry.name)) continue;
       try {
-        if (pattern.test(readFileSync35(join42(dir, entry.name), "utf8"))) return true;
+        if (pattern.test(readFileSync34(join41(dir, entry.name), "utf8"))) return true;
       } catch {
         continue;
       }
@@ -10141,11 +9904,11 @@ function sourceImports(cwd, pattern) {
   return false;
 }
 function dependencyNames(cwd) {
-  const path = join42(cwd, "package.json");
+  const path = join41(cwd, "package.json");
   if (!existsSync34(path)) return [];
   try {
     const manifest = parseJsonc(
-      readFileSync35(path, "utf8"),
+      readFileSync34(path, "utf8"),
       path
     );
     return [
@@ -10168,7 +9931,7 @@ function declaresJsx(cwd) {
   for (const name of entries) {
     try {
       const config = parseJsonc(
-        readFileSync35(join42(cwd, name), "utf8"),
+        readFileSync34(join41(cwd, name), "utf8"),
         name
       );
       if (config.compilerOptions?.jsx !== void 0) return true;

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-7LE72AFG.js";
+} from "./chunk-7ADFV7P4.js";
 import "./chunk-NW6UHNYX.js";
 export {
   BaseAdapter,

package/package.json

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