dependency-cruiser 11.14.1 → 11.15.0-beta-1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "11.14.1",
+  "version": "11.15.0-beta-1",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -188,12 +188,14 @@
     "eslint-plugin-unicorn": "^43.0.1",
     "husky": "^4.3.8",
     "intercept-stdout": "0.1.2",
+    "interpret": "^2.2.0",
     "lint-staged": "12.3.4",
     "mocha": "10.0.0",
     "normalize-newline": "^3.0.0",
     "npm-run-all": "4.1.5",
     "prettier": "2.7.1",
     "proxyquire": "2.1.3",
+    "rechoir": "^0.8.0",
     "shx": "0.3.4",
     "svelte": "3.49.0",
     "symlink-dir": "5.0.1",
@@ -234,6 +236,11 @@
         "policy": "wanted",
         "because": "version 5 only exports ejs - and we use cjs and don't transpile"
       },
+      {
+        "package": "interpret",
+        "policy": "wanted",
+        "because": "we want to keep interpret ~similar to what webpack-cli is using (which is ^2.2.0)"
+      },
       {
         "package": "lint-staged",
         "policy": "pin",
@@ -244,6 +251,11 @@
         "policy": "wanted",
         "because": "version 4 only exports ejs - and we use cjs and don't transpile (this only used in unit tests - but one of'em is a cjs one ...)"
       },
+      {
+        "package": "rechoir",
+        "policy": "wanted",
+        "because": "we want to keep rechoir ~similar to what webpack-cli is using (which is ^0.7.0, but ^0.8.0 is similar enough and more recent anyway)"
+      },
       {
         "package": "wrap-ansi",
         "policy": "wanted",

package/src/config-utl/extract-webpack-resolve-config.js

@@ -1,3 +1,5 @@
+const { extname } = require("path");
+const tryRequire = require("semver-try-require");
 const makeAbsolute = require("./make-absolute");
 
 function pryConfigFromTheConfig(
@@ -22,6 +24,61 @@ function pryConfigFromTheConfig(
   return lReturnValue;
 }
 
+function suggestModules(pSuggestionList, pWebpackConfigFilename) {
+  let lReturnValue = "";
+  let lSuggestionList = pSuggestionList;
+
+  if (pSuggestionList && typeof pSuggestionList === "string") {
+    lSuggestionList = [pSuggestionList];
+  }
+
+  if (Array.isArray(lSuggestionList) && lSuggestionList.length > 0) {
+    lReturnValue = lSuggestionList.reduce(
+      (pAll, pCurrent) =>
+        (pAll += `         - ${pCurrent.module || pCurrent}\n`),
+      `\n         Some npm modules that might fix that problem (one of which you'll` +
+        `\n         need so '${pWebpackConfigFilename}' works with webpack anyway):\n`
+    );
+  }
+  return lReturnValue;
+}
+
+function tryRegisterNonNative(pWebpackConfigFilename) {
+  const lConfigExtension = extname(pWebpackConfigFilename);
+
+  if (lConfigExtension === ".mjs") {
+    throw new Error(
+      `dependency-cruiser currently does not support webpack configurations in` +
+        `\n         ES Module format (like '${pWebpackConfigFilename}').\n`
+    );
+  }
+
+  const interpret = tryRequire("interpret", "^2.2.0");
+  const rechoir = tryRequire("rechoir", "^0.7.0||^0.8.0");
+
+  if (interpret.extensions && rechoir) {
+    try {
+      rechoir.prepare(interpret.extensions, pWebpackConfigFilename);
+    } catch (pError) {
+      throw new Error(
+        `${pError.message}` +
+          `\n${suggestModules(
+            // eslint-disable-next-line security/detect-object-injection
+            interpret.extensions[lConfigExtension],
+            pWebpackConfigFilename
+          )}`
+      );
+    }
+  } else {
+    throw new Error(
+      `The webpack config '${pWebpackConfigFilename}' needs to be transpiled before\n` +
+        `         dependency-cruiser can pass it to webpack.\n\n` +
+        `         To fix that:\n\n` +
+        `           npm install --development interpret@^2.2.0 rechoir@^0.7.0\n\n`
+    );
+  }
+}
+
 /**
  * Reads the file with name `pWebpackConfigFilename` and (applying the
  * environment `pEnvironment` and the arguments `pArguments` (which can
@@ -35,16 +92,23 @@ function pryConfigFromTheConfig(
  * @throws {Error} when the webpack config isn't usable (e.g. because it
  *                 doesn't exist, or because it's invalid)
  */
+// eslint-disable-next-line max-lines-per-function, complexity
 module.exports = function extractWebpackResolveConfig(
   pWebpackConfigFilename,
   pEnvironment,
   pArguments
 ) {
   let lReturnValue = {};
+  const lNativelySupportedExtensions = [".js", ".cjs", ".json", ".node"];
+  const lWebpackConfigFilename = makeAbsolute(pWebpackConfigFilename);
+
+  if (!lNativelySupportedExtensions.includes(extname(pWebpackConfigFilename))) {
+    tryRegisterNonNative(pWebpackConfigFilename);
+  }
 
   try {
     /* eslint node/global-require:0, security/detect-non-literal-require:0, import/no-dynamic-require:0 */
-    const lWebpackConfigModule = require(makeAbsolute(pWebpackConfigFilename));
+    const lWebpackConfigModule = require(lWebpackConfigFilename);
     const lWebpackConfig = pryConfigFromTheConfig(
       lWebpackConfigModule,
       pEnvironment,

package/src/meta.js

@@ -1,7 +1,7 @@
 /* generated - don't edit */
 
 module.exports = {
-  version: "11.14.1",
+  version: "11.15.0-beta-1",
   engines: {
     node: "^12.20||^14||>=16",
   },