@hublo/sentinel 1.4.0-alpha.10 → 1.4.0-alpha.11

package/dist/bin/sentinel.js

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

package/dist/{chunk-L7WS36XV.js → chunk-2S2VYRUM.js}

@@ -7272,7 +7272,7 @@ function rewriteSpyInstanceType(source, program) {
     if (args === void 0 || args.length === 0) return;
     const text = (argument) => source.slice(argument.start, argument.end);
     const returns = text(args[0]);
-    const parameters = args.length > 1 ? `...args: ${text(args[1])}` : "...args: any[]";
+    const parameters = args.length > 1 ? `...args: ${text(args[1])}` : "...args: never[]";
     edits.push({
       start: node.start,
       end: node.end,
@@ -8340,7 +8340,7 @@ function planJestMigration(context, adoption) {
     );
   }
   notes.push(
-    `run this module's own formatter (or lint --fix) on what this wrote, after installing. Two things need it: the Vitest type imports are written beside the existing ones, and a rewritten call is often SHORTER than the one it replaced, so the line breaks around it are no longer the ones your formatter would pick. Neither is a defect, and both fail a format check. Layout is a repository setting, so sentinel leaves it to the tool that owns it.`
+    `the files this rewrote were run through this module's own formatter afterwards. Two things needed it: the Vitest type imports land beside the existing ones, and a rewritten call is often SHORTER than the one it replaced, so the line breaks around it are no longer the ones your config would pick. Neither is a defect, and both would have failed your format check.`
   );
   notes.push(
     `then run this module's TYPECHECK, not only its tests. The migration rewrites types as well as calls, and a wrong type is invisible to a green suite: measured on a real module, \`--run --test\` passed while \`--run --typescript\` reported ten errors it had not had before.`
@@ -8837,6 +8837,44 @@ var VitestAdapter = class extends BaseAdapter {
     this.say(captureReference(ctx.cwd, config).message);
     return Promise.resolve();
   }
+  /**
+   * Hand what was just written to the module's OWN formatter.
+   *
+   * ⚠️ Not a style sentinel imposes: the module adopted `.oxfmtrc.json`, and this runs that
+   * config over the files this role created. Without it, `--init --test` leaves a module whose
+   * own `--run --format` fails on a file the module never wrote.
+   *
+   * Measured on `apps/nest/microservices/agency`, in this repository's CI:
+   *
+   *     agency:format   12 files, including vitest.config.mts and
+   *                     vitest.integration.config.mts, both generated here
+   *
+   * The generator cannot avoid it by wrapping at a fixed width, because the width is the
+   * MODULE's decision: `printWidth` lives in its own config, and 12 modules already set their
+   * own. Guessing it is how a tool starts owning a setting it does not own.
+   *
+   * The same delegation the build role asks a human to do by hand, done. A module that has not
+   * adopted the format role has no formatter to invoke, and this says nothing rather than
+   * choosing one for it.
+   */
+  async afterInit(ctx) {
+    const oxfmt = resolveOxfmt(ctx.cwd);
+    if (oxfmt === void 0) {
+      this.say(
+        `no formatter adopted here, so what was written keeps the shape it was written in. Run your own formatter over it before committing.`
+      );
+      return Promise.resolve();
+    }
+    const pass = spawnSync5(oxfmt, ["--write", "."], { cwd: ctx.cwd, encoding: "utf8" });
+    this.say(
+      pass.status === 0 ? `formatted what this wrote with the module's own oxfmt config, so its \`--run --format\` passes on files it did not write itself.` : (
+        // Adoption SUCCEEDED; only the tidy-up did not. Failing here would revert a migration
+        // over a formatting pass, which is the wrong trade.
+        `the module's formatter could not run over what this wrote (${pass.status ?? "no exit code"}). Adoption itself is applied; run \`sentinel --run --format --fix\` before committing.`
+      )
+    );
+    return Promise.resolve();
+  }
   plan(context) {
     return plan3(context);
   }

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-L7WS36XV.js";
+} from "./chunk-2S2VYRUM.js";
 import "./chunk-WLFE5RUU.js";
 export {
   BaseAdapter,

package/package.json

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