@hublo/sentinel 1.2.0-alpha.2 → 1.2.0-alpha.3

package/dist/bin/sentinel.js

@@ -14,21 +14,21 @@ import {
   ensureWorkspacePrep,
   findWorkspaceRoot,
   inspectWorkspacePrep,
+  moduleName,
   palette,
-  readNxProjectName,
   readOwnVersion,
   readProjectPackageJson,
   registerAdapters,
   resolve,
   resolveBin
-} from "../chunk-SA2RMTVI.js";
+} from "../chunk-YZ4EKBK7.js";
 
 // bin/sentinel.ts
 import { program } from "commander";
 
 // src/core/context.ts
 import { existsSync } from "fs";
-import { basename, join as join2 } from "path";
+import { join as join2 } from "path";
 
 // src/core/discover-modules.ts
 import { execFileSync } from "child_process";
@@ -96,7 +96,7 @@ function resolveContext(cwd2, opts2) {
     if (opts2.ci) {
       throw new Error("--ci selects the affected set from the workspace root; run it there.");
     }
-    const name = readNxProjectName(cwd2) ?? basename(cwd2);
+    const name = moduleName(cwd2);
     return { modules: [{ name, root: cwd2 }], scope: "cwd-module" };
   }
   if (atRoot) {

package/dist/{chunk-SA2RMTVI.js → chunk-YZ4EKBK7.js}

@@ -82,7 +82,7 @@ var BaseAdapter = class {
 
 // src/shared/package-json.ts
 import { existsSync, readFileSync } from "fs";
-import { dirname, join } from "path";
+import { basename, dirname, join } from "path";
 import { fileURLToPath } from "url";
 function readOwnPackage() {
   let dir = dirname(fileURLToPath(import.meta.url));
@@ -127,6 +127,10 @@ function readNxProjectName(dir) {
     return void 0;
   }
 }
+function moduleName(dir) {
+  const declared = readProjectPackageJson(dir).name;
+  return readNxProjectName(dir) ?? (declared === void 0 ? basename(dir) : declared);
+}
 
 // src/core/domain.ts
 var VERBS = ["run", "inspect", "init", "migrate", "report", "status"];
@@ -217,7 +221,7 @@ function binSearchPath(cwd) {
 
 // src/roles/build/plan.ts
 import { existsSync as existsSync7, readFileSync as readFileSync5 } from "fs";
-import { basename as basename2, join as join7 } from "path";
+import { join as join7 } from "path";
 
 // src/core/config/existing-command.ts
 import { readFileSync as readFileSync3 } from "fs";
@@ -237,7 +241,7 @@ function parseJsonc(text, source = "config") {
 
 // src/core/config/manifest.ts
 import { existsSync as existsSync4, readFileSync as readFileSync2 } from "fs";
-import { basename, join as join3 } from "path";
+import { basename as basename2, join as join3 } from "path";
 function moduleScripts(cwd) {
   try {
     const pkg = JSON.parse(readFileSync2(join3(cwd, "package.json"), "utf8"));
@@ -246,7 +250,7 @@ function moduleScripts(cwd) {
     return {};
   }
 }
-function moduleName(cwd) {
+function moduleName2(cwd) {
   try {
     const pkg = JSON.parse(readFileSync2(join3(cwd, "package.json"), "utf8"));
     return pkg.name;
@@ -255,7 +259,7 @@ function moduleName(cwd) {
   }
 }
 function isSelfAdoption(cwd) {
-  return moduleName(cwd) === readOwnPackage().name;
+  return moduleName2(cwd) === readOwnPackage().name;
 }
 function selfCommand(cwd, flags) {
   const own = readOwnPackage();
@@ -274,7 +278,7 @@ function manifestOperation(cwd, scripts) {
   return {
     kind: "merge-json",
     path: "package.json",
-    value: { name: readNxProjectName(cwd) ?? basename(cwd), private: true, ...value }
+    value: { name: readNxProjectName(cwd) ?? basename2(cwd), private: true, ...value }
   };
 }
 
@@ -526,6 +530,7 @@ var REACT_PLUGIN_SPECIFIERS = ["@vitejs/plugin-react", "@tanstack/react-start"];
 function declaredBuildPreset(cwd) {
   const source = readBuildConfigSource(cwd);
   if (source === void 0) return void 0;
+  if (importsPreset(source)) return "react";
   return REACT_PLUGIN_SPECIFIERS.some((specifier) => importsSpecifier(source, specifier)) ? "react" : void 0;
 }
 function readBuildConfigSource(cwd) {
@@ -718,7 +723,7 @@ function buildScripts(cwd) {
   };
   const existingDev = existingCommand(cwd, DEV_SCRIPT_NAME);
   if (existingDev !== void 0) {
-    const own2 = basename2(cwd);
+    const own2 = moduleName(cwd);
     scripts[DEV_SCRIPT_NAME] = composeDevScript(
       existingDev,
       `${selfCommand(cwd, "--run --dev") ?? SENTINEL_DEV_COMMAND} --module ${own2}`
@@ -929,6 +934,24 @@ var ViteDevAdapter = class extends ViteRoleAdapter {
   get label() {
     return "dev";
   }
+  /**
+   * There is no such thing as adopting the dev server on its own.
+   *
+   * `ViteRoleAdapter.plan()` is shared with `--build` because everything else about the two
+   * targets is shared, and inheriting it here made `--init --dev` perform the FULL build
+   * adoption, silently, under a message that only ever mentioned the build. Exiting 0 on that
+   * is the worst of the options: it teaches that dev can be adopted by itself, and that belief
+   * is what produces the half-migrated module this role exists to prevent (see the class
+   * comment above, and `registerBuild`).
+   *
+   * A single plan writes both scripts, so the honest answer is to name the command that does
+   * it rather than to do it under a different name.
+   */
+  plan() {
+    throw new Error(
+      "the dev server is adopted together with the build, never on its own. Run --init --build: one plan writes both the `build` and the `serve` scripts."
+    );
+  }
   /**
    * Start the dev server. It does not return until stopped, so there is no verdict to report
    * beyond the exit code the developer's own Ctrl-C produces.
@@ -6222,7 +6245,7 @@ export {
   resolveBin,
   readOwnVersion,
   readProjectPackageJson,
-  readNxProjectName,
+  moduleName,
   VERBS,
   TARGETS,
   LONG_RUNNING_TARGETS,
@@ -6238,4 +6261,4 @@ export {
   detectFramework,
   dispatch
 };
-//# sourceMappingURL=chunk-SA2RMTVI.js.map
+//# sourceMappingURL=chunk-YZ4EKBK7.js.map

package/dist/index.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.2.0-alpha.2",
+  "version": "1.2.0-alpha.3",
   "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",