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

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-6DBTDMSN.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-6DBTDMSN.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 }
   };
 }
 
@@ -465,7 +469,7 @@ var BUILD_CONFIG_FILES = [
 ];
 var BUILD_PRESET_SPECIFIER = "@hublo/sentinel/build/react";
 var BUILD_SCRIPT_NAME = "build";
-var SENTINEL_BUILD_COMMAND = "sentinel --run --build";
+var SENTINEL_BUILD_COMMAND = "sentinel --run --build --";
 var DEV_SCRIPT_NAME = "serve";
 var SENTINEL_DEV_COMMAND = "sentinel --run --dev";
 function buildTarget() {
@@ -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,10 +723,12 @@ 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}`
+      // Trailing `--` for the same reason as the build script: whatever the caller appends
+      // reaches Vite, not sentinel's parser. It goes after `--module`, which is sentinel's own.
+      `${selfCommand(cwd, "--run --dev") ?? SENTINEL_DEV_COMMAND} --module ${own2} --`
     );
   }
   return scripts;
@@ -929,6 +936,26 @@ var ViteDevAdapter = class extends ViteRoleAdapter {
   get label() {
     return "dev";
   }
+  /**
+   * `--init --dev` does the same thing as `--init --build`, and now says so.
+   *
+   * The shared plan is deliberate (see `ViteRoleAdapter.plan`): one plan writes both scripts,
+   * because adopting one without the other leaves the module half-migrated. What was missing
+   * was only the telling: the run printed a message about the build and nothing about `serve`,
+   * so a developer could believe the dev server had been adopted on its own. Refusing was the
+   * wrong correction, it added friction to a command that already produced the right result.
+   */
+  plan(context) {
+    const planned = super.plan(context);
+    if (planned.operations.length === 0) return planned;
+    return {
+      ...planned,
+      notes: [
+        ...planned.notes ?? [],
+        "--init --dev adopts the build too: one plan writes both `build` and `serve`."
+      ]
+    };
+  }
   /**
    * 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 +6249,7 @@ export {
   resolveBin,
   readOwnVersion,
   readProjectPackageJson,
-  readNxProjectName,
+  moduleName,
   VERBS,
   TARGETS,
   LONG_RUNNING_TARGETS,
@@ -6238,4 +6265,4 @@ export {
   detectFramework,
   dispatch
 };
-//# sourceMappingURL=chunk-SA2RMTVI.js.map
+//# sourceMappingURL=chunk-6DBTDMSN.js.map

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-SA2RMTVI.js";
+} from "./chunk-6DBTDMSN.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.4",
   "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",