@mnci/cli 4.1.0 → 4.1.2

package/dist/cli.js

@@ -250,7 +250,7 @@ var import_node_path2 = require("path");
 var DEFAULT_STACK = { testRunner: "jest" };
 var TS_COMPILER_DEPENDENCIES = {
   "@typescript/native": "npm:typescript@^7.0.2",
-  typescript: "npm:@typescript/typescript6@^6.0.2"
+  "typescript": "npm:@typescript/typescript6@^6.0.2"
 };
 function registryUrl(registry) {
   if (registry.kind === "azure-artifacts") {
@@ -447,7 +447,7 @@ var ESLINT_PEER_OVERRIDES = {
   // both — and forcing those to v5 breaks them. So the blast radius is one
   // dependency edge per named parent, and a test asserts there is no top-level
   // entry.
-  nx: { "brace-expansion": "^5.0.9" },
+  "nx": { "brace-expansion": "^5.0.9" },
   "@nx/js": { "brace-expansion": "^5.0.9" },
   "@nx/eslint": { "brace-expansion": "^5.0.9" },
   "@nx/eslint-plugin": { "brace-expansion": "^5.0.9" },
@@ -456,14 +456,14 @@ var ESLINT_PEER_OVERRIDES = {
   // which every publishable `npm-lib` gets. GHSA-2v37-7h3g-55p8 is high, fixed in
   // 3.3.18, and the fix is a patch — so unlike the entry above this one is not
   // even a trade, just a version nobody had bumped yet.
-  postcss: { nanoid: "^3.3.18" }
+  "postcss": { nanoid: "^3.3.18" }
 };
 var SECURITY_OVERRIDES = {
   "brace-expansion@4.0.0 - 5.0.8": "^5.0.9"
 };
 function eslintToolchainDependencies(nxVersion) {
   return {
-    eslint: ESLINT_VERSION,
+    "eslint": ESLINT_VERSION,
     "@nx/eslint": nxVersion,
     "@nx/eslint-plugin": nxVersion,
     "@mnci/eslint-config": eslintConfigSpec()
@@ -513,8 +513,13 @@ var ESLINT_BLOCK_INVENTORY = `// WHAT IS IN HERE. Each line is one config block,
 //                                 (Vitest's globals too; the two stacks share them)
 //   mnci/nx-dependency-checks     @nx/eslint-plugin, on publishable packages' manifests
 //   mnci/standard                 JavaScript Standard Style as ESLint rules \u2014 the
-//                                 whole formatting opinion. Composed LAST, on
-//                                 purpose: nothing may follow that disables it.
+//                                 whole formatting opinion, a faithful port of
+//                                 neostandard
+//   mnci/house-style              the deliberate departures from Standard:
+//                                 trailing commas, aligned object values,
+//                                 consistent-as-needed quote-props, a blank line
+//                                 before return. Composed LAST, on purpose:
+//                                 nothing may follow that disables it.
 //
 // To list them as ESLint actually resolves them:  npx eslint --inspect-config
 `;
@@ -657,14 +662,14 @@ function vscodeWorkspace(workspaceName, existingTasks, existingLaunch, existingS
   );
 }
 var ROOT_SCRIPTS = {
-  build: "nx run-many -t build",
-  lint: "nx run-many -t lint",
-  test: "nx run-many -t test",
-  typecheck: "nx run-many -t typecheck",
-  affected: "nx affected -t lint,typecheck,test,build",
-  graph: "nx graph",
+  "build": "nx run-many -t build",
+  "lint": "nx run-many -t lint",
+  "test": "nx run-many -t test",
+  "typecheck": "nx run-many -t typecheck",
+  "affected": "nx affected -t lint,typecheck,test,build",
+  "graph": "nx graph",
   "release:preview": "nx release --dry-run",
-  prepare: "husky"
+  "prepare": "husky"
 };
 function rootScripts() {
   return {
@@ -673,7 +678,7 @@ function rootScripts() {
     // `format:check`: `lint` already reports formatting as ordinary errors.
     // Carrying a second script that ran the same tool twice would just make a
     // CI run slower for no extra coverage.
-    format: "eslint . --fix --cache",
+    "format": "eslint . --fix --cache",
     "python:install": `${PYTHON_INSTALL_GUARD} && ${PYTHON_WORKSPACE_INSTALL_GUARD}`
   };
 }
@@ -1669,6 +1674,50 @@ function registerProjectCommands(workspaceRoot, name, commands) {
     })
   );
 }
+var PROJECT_MANIFEST_ROOTS = ["apps", "packages", "libs"];
+function rootRuntimeDependencies(workspaceRoot) {
+  try {
+    return readJson((0, import_node_path3.join)(workspaceRoot, "package.json")).dependencies ?? {};
+  } catch {
+    return {};
+  }
+}
+function relocateRootRuntimeDependencies(workspaceRoot, projectName, before) {
+  const after = rootRuntimeDependencies(workspaceRoot);
+  const added = Object.keys(after).filter((name) => !Object.hasOwn(before, name));
+  if (added.length === 0) {
+    return;
+  }
+  const manifestPath = PROJECT_MANIFEST_ROOTS.map(
+    (root) => (0, import_node_path3.join)(workspaceRoot, root, projectName, "package.json")
+  ).find((candidate) => fileExists(candidate));
+  if (manifestPath === void 0) {
+    return;
+  }
+  const manifest = readJson(manifestPath);
+  const owned = manifest.dependencies ?? {};
+  const moved = Object.fromEntries(added.map((name) => [name, after[name]]));
+  writeFileEnsured(manifestPath, toJson({ ...manifest, dependencies: { ...moved, ...owned } }));
+  const remaining = Object.fromEntries(
+    Object.entries(after).filter(([name]) => !added.includes(name))
+  );
+  const rootManifest = readJson((0, import_node_path3.join)(workspaceRoot, "package.json"));
+  const { dependencies: _dropped, ...rest } = rootManifest;
+  writeFileEnsured(
+    (0, import_node_path3.join)(workspaceRoot, "package.json"),
+    toJson(Object.keys(remaining).length > 0 ? { ...rest, dependencies: remaining } : rest)
+  );
+  logger.step(`Moved ${added.join(", ")} into ${projectName}'s own manifest`);
+  if (runShell(
+    "npm",
+    ["install", "--package-lock-only", "--no-audit", "--no-fund"],
+    workspaceRoot
+  ) !== 0) {
+    logger.warn(
+      `Could not refresh package-lock.json after moving ${added.join(", ")}. Run 'npm install'.`
+    );
+  }
+}
 
 // src/commands/add/flutter.ts
 function addProjectJsonTargets(projectJsonPath, newTargets) {
@@ -2820,9 +2869,9 @@ function resolvedVersion(workspaceRoot, ecosystem, name) {
 // src/deps/semver.ts
 var UPDATE_KINDS = ["patch", "minor", "major", "non-semver"];
 var UPDATE_KIND_LABELS = {
-  patch: { title: "Patch Update", blurb: "Backwards-compatible bug fixes." },
-  minor: { title: "Minor Update", blurb: "New backwards-compatible features." },
-  major: { title: "Major Update", blurb: "Potentially breaking API changes. Use caution." },
+  "patch": { title: "Patch Update", blurb: "Backwards-compatible bug fixes." },
+  "minor": { title: "Minor Update", blurb: "New backwards-compatible features." },
+  "major": { title: "Major Update", blurb: "Potentially breaking API changes. Use caution." },
   "non-semver": { title: "Non-Semver", blurb: "Versions less than 1.0.0, caution." }
 };
 var RANGE_OPERATORS = [">=", "<=", "^", "~", ">", "<", "="];
@@ -3076,6 +3125,7 @@ async function runAdd(kind, name, options) {
   });
   const resolvedName = name ?? await promptText("Project name");
   assertValidProjectName(resolvedName, "Project name");
+  const rootDependenciesBefore = rootRuntimeDependencies(workspaceRoot);
   switch (resolvedKind) {
     case "react-app": {
       addReactApp(workspaceRoot, resolvedName, stack);
@@ -3176,6 +3226,7 @@ async function runAdd(kind, name, options) {
       );
     }
   }
+  relocateRootRuntimeDependencies(workspaceRoot, resolvedName, rootDependenciesBefore);
   syncProjectReferences(workspaceRoot);
   runFormatter(workspaceRoot);
   logger.success(`Added ${resolvedKind} '${resolvedName}'.`);