@hublo/sentinel 1.2.0-alpha.13 → 1.2.0-alpha.15

package/dist/bin/sentinel.d.ts

@@ -4,3 +4,4 @@ import '@vitejs/plugin-react';
 import 'nitro/vite';
 import 'vite-plugin-svgr';
 import 'vite';
+import 'vitest/config';

package/dist/bin/sentinel.js

@@ -22,7 +22,7 @@ import {
   registerAdapters,
   resolve,
   resolveBin
-} from "../chunk-X675OYUC.js";
+} from "../chunk-RV34GQVH.js";
 
 // bin/sentinel.ts
 import { program } from "commander";

package/dist/{chunk-X675OYUC.js → chunk-RV34GQVH.js}

@@ -333,11 +333,11 @@ var BUILD_SCRIPT_NAME = "build";
 var SENTINEL_BUILD_COMMAND = "sentinel --run --build --";
 var DEV_SCRIPT_NAME = "serve";
 var SENTINEL_DEV_COMMAND = "sentinel --run --dev";
-function buildTarget() {
+function buildTarget(configFiles = BUILD_CONFIG_FILES) {
   return {
     [BUILD_SCRIPT_NAME]: {
       cache: true,
-      inputs: ["default", "^default", ...BUILD_CONFIG_FILES.map((name) => `{projectRoot}/${name}`)]
+      inputs: ["default", "^default", ...configFiles.map((name) => `{projectRoot}/${name}`)]
     }
   };
 }
@@ -572,6 +572,11 @@ function existingCommand(cwd, target) {
 // src/core/config/nx-target.ts
 import { existsSync as existsSync7 } from "fs";
 import { join as join7 } from "path";
+function nxTargetMetadataOperations(options) {
+  const { targets } = options;
+  if (Object.keys(targets).length === 0) return [];
+  return [{ kind: "merge-json", path: "package.json", value: { nx: { targets } } }];
+}
 function nxTargetOperations(options) {
   const { cwd, targets } = options;
   const names = Object.keys(targets);
@@ -584,11 +589,7 @@ function nxTargetOperations(options) {
       keys: names.map((name) => ["targets", name])
     });
   }
-  operations.push({
-    kind: "merge-json",
-    path: "package.json",
-    value: { nx: { targets } }
-  });
+  operations.push(...nxTargetMetadataOperations(options));
   return operations;
 }
 
@@ -730,32 +731,31 @@ function describeUnmet(preset, unmet) {
   return `the build role does not apply to this module yet: ${list}. ${entry?.why ?? ""}. Nothing was written. Raise those versions, check the module still builds and its tests still pass, then re-run \`sentinel --init --build\`.`;
 }
 
-// src/roles/build/rewrite-imports.ts
+// src/core/config/rewrite-imports.ts
 import { parseSync } from "oxc-parser";
-var MOVED_SPECIFIERS = new Set(OWNED_PACKAGES.map((entry) => entry.specifier));
+var movedSpecifiers = (toolchain) => new Set(toolchain.owned.map((entry) => entry.specifier));
 function esmBlocker(fileName, packageType) {
   if (/\.(mts|mjs)$/.test(fileName)) return void 0;
   if (packageType === "module") return void 0;
   return `sentinel is ESM only, and this module does not declare \`"type": "module"\`, so Vite would load ${fileName} as CommonJS and fail to require it. Add \`"type": "module"\` to this module's package.json, or rename the config to \`${fileName.replace(/\.(ts|js)$/, ".m$1")}\`, then run this again.`;
 }
-var DEFAULT_EXPORT_NAMES = Object.fromEntries(
-  OWNED_PACKAGES.filter((entry) => entry.default !== void 0).map((entry) => [
-    entry.specifier,
-    entry.default
-  ])
+var defaultExportNames = (toolchain) => Object.fromEntries(
+  toolchain.owned.filter((entry) => entry.default !== void 0).map((entry) => [entry.specifier, entry.default])
 );
-function renderImport(bindings) {
+function renderImport(bindings, presetSpecifier) {
   const render = (list) => list.map((b) => b.exported === b.local ? b.exported : `${b.exported} as ${b.local}`).sort((left, right) => left.localeCompare(right)).join(", ");
   const values = bindings.filter((b) => !b.typeOnly);
   const types = bindings.filter((b) => b.typeOnly);
   const lines = [];
-  if (values.length > 0) lines.push(`import { ${render(values)} } from '${BUILD_PRESET_SPECIFIER}'`);
+  if (values.length > 0) lines.push(`import { ${render(values)} } from '${presetSpecifier}'`);
   if (types.length > 0) {
-    lines.push(`import type { ${render(types)} } from '${BUILD_PRESET_SPECIFIER}'`);
+    lines.push(`import type { ${render(types)} } from '${presetSpecifier}'`);
   }
   return lines.join("\n");
 }
-function rewriteBuildImports(source, fileName = "vite.config.ts") {
+function rewriteToolchainImports(toolchain, source, fileName) {
+  const moved = movedSpecifiers(toolchain);
+  const defaults = defaultExportNames(toolchain);
   const { program, errors } = parseSync(fileName, source, { sourceType: "module" });
   if (errors.length > 0) {
     return {
@@ -766,10 +766,10 @@ function rewriteBuildImports(source, fileName = "vite.config.ts") {
   const imports = program.body.filter(
     (node) => node.type === "ImportDeclaration"
   );
-  if (imports.some((node) => node.source.value === BUILD_PRESET_SPECIFIER)) {
+  if (imports.some((node) => node.source.value === toolchain.presetSpecifier)) {
     return { kind: "unchanged", why: "this config already imports its toolchain from sentinel" };
   }
-  const moving = imports.filter((node) => MOVED_SPECIFIERS.has(node.source.value));
+  const moving = imports.filter((node) => moved.has(node.source.value));
   if (moving.length === 0) {
     return {
       kind: "unchanged",
@@ -786,7 +786,7 @@ function rewriteBuildImports(source, fileName = "vite.config.ts") {
     for (const spec of specifiers) {
       const typeOnly = node.importKind === "type" || spec.importKind === "type";
       if (spec.type === "ImportDefaultSpecifier") {
-        const exported = DEFAULT_EXPORT_NAMES[from];
+        const exported = defaults[from];
         if (!exported) {
           return {
             kind: "blocked",
@@ -818,7 +818,7 @@ function rewriteBuildImports(source, fileName = "vite.config.ts") {
     const after = out.slice(span.end).match(/^\r?\n/)?.[0].length ?? 0;
     out = out.slice(0, span.start) + out.slice(span.end + after);
   }
-  out = out.slice(0, first.start) + renderImport(bindings) + out.slice(first.end);
+  out = out.slice(0, first.start) + renderImport(bindings, toolchain.presetSpecifier) + out.slice(first.end);
   return {
     kind: "rewritten",
     source: out,
@@ -826,6 +826,15 @@ function rewriteBuildImports(source, fileName = "vite.config.ts") {
   };
 }
 
+// src/roles/build/rewrite-imports.ts
+var BUILD_TOOLCHAIN = {
+  presetSpecifier: BUILD_PRESET_SPECIFIER,
+  owned: OWNED_PACKAGES
+};
+function rewriteBuildImports(source, fileName = "vite.config.ts") {
+  return rewriteToolchainImports(BUILD_TOOLCHAIN, source, fileName);
+}
+
 // src/roles/build/plan.ts
 function ownedBuildDependencies(cwd) {
   const manifest = readProjectPackageJson(cwd);
@@ -889,7 +898,7 @@ function plan(context) {
   if (configFile === void 0) {
     return {
       operations: [],
-      skipped: `no Vite config in this module, so there is nothing to point at sentinel. This role moves an existing config's imports; it does not introduce a bundler. If this module builds another way (36 services here build through \`@nx/webpack\`), the build role does not apply to it.`
+      skipped: `no Vite config in this module, so there is nothing to point at sentinel. This role moves an existing config's imports; it does not introduce a bundler. If this module builds another way (38 services here build through \`@nx/webpack\`), the build role does not apply to it.`
     };
   }
   return planReactApp(context, configFile);
@@ -949,7 +958,16 @@ function planNestService(context, configFile, webpackTarget) {
   if (refusal !== void 0) return { operations: [], blocked: refusal };
   const dropped = Object.keys(webpackTarget.configurations ?? {});
   return {
-    operations: nestAdoptionOperations(context.cwd, webpackTarget),
+    operations: [
+      ...nestAdoptionOperations(context.cwd, webpackTarget),
+      // The same nx metadata the React path writes, and for the same reason: declaring `inputs`
+      // at all REPLACES nx's default `['default','^default']`, so a target that declares none
+      // runs on defaults while every other target this module has declares them. The Nest path
+      // wrote none, which left its build the one target that could replay a stale bundle after a
+      // dependency changed. One name, because a Nest service's config can only be `.mts`.
+      ...nxTargetMetadataOperations({ cwd: context.cwd, targets: buildTarget([NEST_CONFIG_FILE]) }),
+      manifestOperation(context.cwd, { [BUILD_SCRIPT_NAME]: SENTINEL_BUILD_COMMAND })
+    ],
     notes: [
       `wrote ${NEST_CONFIG_FILE} and pointed the nx build target at sentinel. The target stays \`nx:run-commands\` with \`forwardAllArgs: false\`, so the image's \`--generatePackageJson\` still works and the Dockerfile needs no change.`,
       ...dropped.length > 0 ? [
@@ -1003,7 +1021,7 @@ var ViteRoleAdapter = class extends BaseAdapter {
    * which tells nobody what to do about it.
    *
    * Nest it now serves too, through `@hublo/sentinel/build/nest`. It used to be declined on the
-   * grounds that "there is nothing to bundle", which was never true: those 36 services bundle
+   * grounds that "there is nothing to bundle", which was never true: those 38 services bundle
    * with webpack. What changed is that Nx v24 removes the `@nx/webpack:webpack` executor and
    * the `composePlugins` / `withNx` helpers their shared config is built on, and nx's own
    * migration generator refuses every one of them because they use `@nx/js:node`
@@ -6439,4 +6457,4 @@ export {
   detectFramework,
   dispatch
 };
-//# sourceMappingURL=chunk-X675OYUC.js.map
+//# sourceMappingURL=chunk-RV34GQVH.js.map

package/dist/index.js

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

package/dist/roles/test/react/toolchain.d.ts

@@ -0,0 +1,2 @@
+export { ConfigEnv, TestUserConfig, ViteUserConfig, defineConfig, mergeConfig } from 'vitest/config';
+export { loadEnv } from 'vite';

package/dist/roles/test/react/toolchain.js

@@ -0,0 +1,9 @@
+// src/roles/test/react/toolchain.ts
+import { defineConfig, mergeConfig } from "vitest/config";
+import { loadEnv } from "vite";
+export {
+  defineConfig,
+  loadEnv,
+  mergeConfig
+};
+//# sourceMappingURL=toolchain.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.2.0-alpha.13",
+  "version": "1.2.0-alpha.15",
   "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",
@@ -45,6 +45,10 @@
     "./build/nest": {
       "types": "./dist/roles/build/nest/toolchain.d.ts",
       "import": "./dist/roles/build/nest/toolchain.js"
+    },
+    "./test/react": {
+      "types": "./dist/roles/test/react/toolchain.d.ts",
+      "import": "./dist/roles/test/react/toolchain.js"
     }
   },
   "files": [
@@ -72,7 +76,8 @@
     "oxlint-tsgolint": "7.0.2001",
     "typescript": "5.9.3",
     "vite": "8.0.8",
-    "vite-plugin-svgr": "5.2.0"
+    "vite-plugin-svgr": "5.2.0",
+    "vitest": "4.1.4"
   },
   "devDependencies": {
     "@hublo/sentinel": "link:.",