dependency-cruiser 10.8.0-beta-3 → 10.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "10.8.0-beta-3",
+  "version": "10.8.0",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -131,13 +131,13 @@
     "version": "npm-run-all build depcruise:graph:doc scm:stage"
   },
   "dependencies": {
-    "acorn": "8.5.0",
+    "acorn": "8.6.0",
     "acorn-jsx": "5.3.2",
     "acorn-jsx-walk": "2.0.0",
     "acorn-loose": "8.2.1",
     "acorn-walk": "8.2.0",
-    "ajv": "8.6.3",
-    "chalk": "4.1.2",
+    "ajv": "8.8.2",
+    "chalk": "^4.1.2",
     "commander": "8.3.0",
     "enhanced-resolve": "5.8.3",
     "figures": "^3.2.0",
@@ -152,17 +152,17 @@
     "semver": "^7.3.5",
     "semver-try-require": "^5.0.1",
     "teamcity-service-messages": "0.1.11",
-    "tsconfig-paths-webpack-plugin": "3.5.1",
+    "tsconfig-paths-webpack-plugin": "3.5.2",
     "wrap-ansi": "^7.0.0"
   },
   "devDependencies": {
-    "@babel/core": "7.15.8",
-    "@babel/plugin-transform-modules-commonjs": "7.15.4",
-    "@babel/preset-typescript": "7.15.0",
-    "@swc/core": "1.2.105",
-    "@typescript-eslint/eslint-plugin": "5.2.0",
-    "@typescript-eslint/parser": "5.2.0",
-    "@vue/compiler-sfc": "3.2.20",
+    "@babel/core": "7.16.0",
+    "@babel/plugin-transform-modules-commonjs": "7.16.0",
+    "@babel/preset-typescript": "7.16.0",
+    "@swc/core": "1.2.113",
+    "@typescript-eslint/eslint-plugin": "5.4.0",
+    "@typescript-eslint/parser": "5.4.0",
+    "@vue/compiler-sfc": "3.2.23",
     "c8": "7.10.0",
     "chai": "4.3.4",
     "chai-json-schema": "1.5.1",
@@ -171,23 +171,23 @@
     "eslint-config-moving-meadow": "2.0.9",
     "eslint-config-prettier": "8.3.0",
     "eslint-plugin-budapestian": "3.0.1",
-    "eslint-plugin-import": "2.25.2",
+    "eslint-plugin-import": "2.25.3",
     "eslint-plugin-mocha": "9.0.0",
     "eslint-plugin-node": "11.1.0",
     "eslint-plugin-security": "1.4.0",
-    "eslint-plugin-unicorn": "37.0.1",
+    "eslint-plugin-unicorn": "39.0.0",
     "husky": "^4.3.8",
     "intercept-stdout": "0.1.2",
-    "lint-staged": "11.2.6",
+    "lint-staged": "12.1.2",
     "mocha": "9.1.3",
     "normalize-newline": "^3.0.0",
     "npm-run-all": "4.1.5",
-    "prettier": "2.4.1",
+    "prettier": "2.5.0",
     "proxyquire": "2.1.3",
     "shx": "0.3.3",
-    "svelte": "3.44.0",
+    "svelte": "3.44.2",
     "symlink-dir": "5.0.1",
-    "typescript": "4.4.4",
+    "typescript": "4.5.2",
     "upem": "^7.0.0",
     "vue-template-compiler": "2.6.14",
     "yarn": "1.22.17"
@@ -199,6 +199,11 @@
         "policy": "wanted",
         "because": "some eslint plugins (eslint-plugin-budapestian) are not compatible with eslint 8 yet "
       },
+      {
+        "package": "chalk",
+        "policy": "wanted",
+        "because": "version 5 only exports ejs - and we use cjs and don't transpile"
+      },
       {
         "package": "figures",
         "policy": "wanted",

package/src/enrich/derive/utl.js

@@ -1,14 +1,26 @@
-const _memoize = require("lodash/memoize");
+let gIndexedGraph = null;
 
-function bareFindModuleByName(pGraph, pSource) {
-  return pGraph.find((pNode) => pNode.source === pSource);
+/**
+ * Returns the module with attribute pSource, when it exists in the pModuleGraph.
+ * Returns undefined when it doesn't.
+ *
+ * This function uses an indexed cache for efficiency reasons. If you need to
+ * call this function consecutively for different module graphs, you can clear
+ * this cache with the clearCache function from this module.
+ *
+ * @param {import('../../../types/cruise-result').IModule[]} pModuleGraph
+ * @param {string} pSource
+ * @returns {import('../../../types/cruise-result').IModule | undefined}
+ */
+function findModuleByName(pModuleGraph, pSource) {
+  if (!gIndexedGraph) {
+    gIndexedGraph = new Map(
+      pModuleGraph.map((pModule) => [pModule.source, pModule])
+    );
+  }
+  return gIndexedGraph.get(pSource);
 }
 
-const findModuleByName = _memoize(
-  bareFindModuleByName,
-  (_pGraph, pSource) => pSource
-);
-
 function isDependent(pResolvedName) {
   return (pModule) =>
     pModule.dependencies.some(
@@ -17,7 +29,11 @@ function isDependent(pResolvedName) {
 }
 
 function clearCache() {
-  findModuleByName.cache.clear();
+  gIndexedGraph = null;
 }
 
-module.exports = { findModuleByName, clearCache, isDependent };
+module.exports = {
+  findModuleByName,
+  clearCache,
+  isDependent,
+};

package/src/enrich/soften-known-violations.js

@@ -78,7 +78,8 @@ function softenKnownViolation(pModule, pKnownViolations, pSoftenedSeverity) {
           pRule,
           pModule.source,
           pKnownViolations.filter(
-            (pKnownError) => pKnownError.from === pKnownError.to
+            (pKnownError) =>
+              pKnownError.from === pKnownError.to && !pKnownError.cycle
           ),
           pSoftenedSeverity
         )
@@ -93,7 +94,8 @@ function softenKnownViolation(pModule, pKnownViolations, pSoftenedSeverity) {
         pDependency,
         pModule.source,
         pKnownViolations.filter(
-          (pKnownError) => pKnownError.from !== pKnownError.to
+          (pKnownError) =>
+            pKnownError.from !== pKnownError.to || pKnownError.cycle
         ),
         pSoftenedSeverity
       )

package/src/extract/ast-extractors/swc-dependency-visitor.js

@@ -119,7 +119,13 @@ if (VisitorModule) {
     // also include the same method, but with the correct spelling.
     visitExportAllDeclration(pNode) {
       this.pushImportExportSource(pNode);
-      return super.visitExportAllDeclration(pNode);
+      /* c8 ignore start */
+      if (super.visitExportAllDeclration) {
+        return super.visitExportAllDeclration(pNode);
+      } else {
+        /* c8 ignore stop */
+        return super.visitExportAllDeclaration(pNode);
+      }
     }
 
     /* c8 ignore start */
@@ -131,7 +137,13 @@ if (VisitorModule) {
     // same spelling error as the above - same solution
     visitExportNamedDeclration(pNode) {
       this.pushImportExportSource(pNode);
-      return super.visitExportNamedDeclration(pNode);
+      /* c8 ignore start */
+      if (super.visitExportNamedDeclration) {
+        return super.visitExportNamedDeclration(pNode);
+      } else {
+        /* c8 ignore stop */
+        return super.visitExportNamedDeclaration(pNode);
+      }
     }
     /* c8 ignore start */
     visitExportNamedDeclaration(pNode) {

package/src/meta.js

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