@mnci/cli 4.10.3 → 4.10.4

package/dist/cli.js

@@ -573,13 +573,9 @@ var ROOT_LINT_TARGET = {
     cwd: "."
   }
 };
-function reactExpressPeerOverride(manifest) {
-  const declared = {
-    ...manifest.dependencies,
-    ...manifest.devDependencies
-  };
-  return declared.express === void 0 ? {} : { "@nx/react": { express: "$express" } };
-}
+var NX_PEER_OVERRIDES = {
+  "@nx/react": { express: ">=4.0.0 <6.0.0" }
+};
 var ESLINT_PEER_OVERRIDES = {
   "eslint-plugin-jsx-a11y": { eslint: "$eslint" },
   // Scoped to EACH named parent, not top-level and not `nx` alone.
@@ -1861,11 +1857,9 @@ function applyOverlay(workspaceRoot, options, onProgress = () => {
     ...manifest.overrides,
     ...ESLINT_PEER_OVERRIDES,
     ...SECURITY_OVERRIDES,
-    // Last, and conditional — see reactExpressPeerOverride. At `mnci new` there
-    // is no express yet, so this is a no-op; `mnci add node-app --framework
-    // express` is what puts express in the root manifest, and syncs the override
-    // itself. This line is what carries it across an `mnci upgrade`.
-    ...reactExpressPeerOverride(manifest)
+    // Static, so it is present before any generator can run an install — see
+    // NX_PEER_OVERRIDES for why the conditional form could never be.
+    ...NX_PEER_OVERRIDES
   };
   const existingNx = manifest.nx;
   const existingTargets = withoutKey(
@@ -2093,8 +2087,22 @@ function projectTask(name, kind) {
   const base = { label: `${name}: ${kind}`, type: "npm", script, problemMatcher: [] };
   return kind === "start" ? { ...base, isBackground: true } : { ...base, group: kind };
 }
+function ensureNxPeerOverrides(workspaceRoot) {
+  const manifestPath = (0, import_node_path3.join)(workspaceRoot, "package.json");
+  if (!(0, import_node_fs3.existsSync)(manifestPath)) {
+    return;
+  }
+  const manifest = readJson(manifestPath);
+  const existing = manifest.overrides ?? {};
+  const overrides = { ...existing, ...NX_PEER_OVERRIDES };
+  if (JSON.stringify(existing) === JSON.stringify(overrides)) {
+    return;
+  }
+  writeFileEnsured(manifestPath, toJson({ ...manifest, overrides }));
+}
 function registerProjectCommands(workspaceRoot, name, commands) {
   removeStaleGitkeeps(workspaceRoot);
+  ensurePythonArtefactsIgnored(workspaceRoot);
   const scripts = {
     [`${name}:qa`]: `nx run ${name}:lint && nx run ${name}:test`
   };
@@ -2111,16 +2119,11 @@ function registerProjectCommands(workspaceRoot, name, commands) {
   const manifestPath = (0, import_node_path3.join)(workspaceRoot, "package.json");
   const manifest = readJson(manifestPath);
   const existingScripts = manifest.scripts ?? {};
-  const overrides = {
-    ...manifest.overrides,
-    ...reactExpressPeerOverride(manifest)
-  };
   writeFileEnsured(
     manifestPath,
     toJson({
       ...manifest,
-      scripts: { ...existingScripts, ...scripts },
-      ...Object.keys(overrides).length > 0 && { overrides }
+      scripts: { ...existingScripts, ...scripts }
     })
   );
   const codeWorkspacePath = findCodeWorkspaceFile(workspaceRoot);
@@ -4674,6 +4677,7 @@ async function runAdd(kind, name, options) {
   const resolvedName = name ?? await promptText("Project name");
   assertValidProjectName(resolvedName, "Project name");
   const rootDependenciesBefore = rootRuntimeDependencies(workspaceRoot);
+  ensureNxPeerOverrides(workspaceRoot);
   switch (resolvedKind) {
     case "react-app": {
       addReactApp(workspaceRoot, resolvedName, stack);