@mnci/cli 4.1.1 → 4.1.3

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}`
   };
 }
@@ -1414,10 +1419,13 @@ var ROLLUP_ARG_ONE_WITH_SOURCE_MAPS = [
   ""
 ].join("\n");
 function hasRollupSourceMaps(config) {
-  return config.includes("sourceMap: true");
+  return /sourceMap\s*:\s*true\b/.test(config);
+}
+function canRepairRollupConfig(config) {
+  return config.includes(ROLLUP_ARG_ONE_BOUNDARY);
 }
 function withRollupSourceMaps(config) {
-  if (hasRollupSourceMaps(config) || !config.includes(ROLLUP_ARG_ONE_BOUNDARY)) {
+  if (hasRollupSourceMaps(config) || !canRepairRollupConfig(config)) {
     return config;
   }
   const withCompiler = config.replace(GENERATED_COMPILER, () => SOURCE_MAP_CAPABLE_COMPILER);
@@ -1446,6 +1454,28 @@ function repairRollupSourceMaps(workspaceRoot) {
   }
   return changed;
 }
+function resolveRequireTarget(fromDir, specifier) {
+  const base = (0, import_node_path3.join)(fromDir, specifier);
+  return [base, `${base}.cjs`, `${base}.js`].find((candidate) => fileExists(candidate));
+}
+function resolveRollupConfigText(configPath, visited = /* @__PURE__ */ new Set()) {
+  if (visited.has(configPath)) {
+    return "";
+  }
+  visited.add(configPath);
+  let text;
+  try {
+    text = (0, import_node_fs3.readFileSync)(configPath, "utf8");
+  } catch {
+    return "";
+  }
+  const specifiers = Array.from(
+    text.matchAll(/require\((['"])(\.[^'"]*)\1\)/g),
+    (match) => match[2]
+  );
+  const requiredText = specifiers.map((specifier) => resolveRequireTarget((0, import_node_path3.dirname)(configPath), specifier)).filter((target) => target !== void 0).map((target) => resolveRollupConfigText(target, visited));
+  return [text, ...requiredText].join("\n");
+}
 var SOURCEMAP_PATH_TRANSFORM = [
   "    // Added by MoNecromanCI. rollup hands sourcemapPathTransform an OS-NATIVE",
   "    // path with one parent segment too many, so `sources` resolve to nothing",
@@ -2864,9 +2894,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 = [">=", "<=", "^", "~", ">", "<", "="];
@@ -3452,18 +3482,19 @@ function checkRollupSourceMaps(workspaceRoot) {
     cwd: workspaceRoot
   }).map((config) => toPosix2(config));
   return configs.flatMap((relativePath) => {
+    const configPath = (0, import_node_path14.join)(workspaceRoot, relativePath);
     let config;
     try {
-      config = (0, import_node_fs8.readFileSync)((0, import_node_path14.join)(workspaceRoot, relativePath), "utf8");
+      config = (0, import_node_fs8.readFileSync)(configPath, "utf8");
     } catch {
       return [];
     }
     return [
       {
         check: `source maps enabled in ${relativePath}`,
-        ok: hasRollupSourceMaps(config),
+        ok: hasRollupSourceMaps(resolveRollupConfigText(configPath)),
         detail: "rollup emits no .js.map without it, so a breakpoint in a .ts file can never bind",
-        remedy: "run `mnci upgrade`, which adds it to every rollup config"
+        remedy: canRepairRollupConfig(config) ? "run `mnci upgrade`, which adds it to every rollup config" : "this config delegates via require() to a shared base mnci does not own \u2014 add `sourceMap: true` to withNx's first argument there by hand; `mnci upgrade` cannot repair a config in this shape"
       }
     ];
   });