dependency-cruiser 11.9.0-beta-2 → 11.10.0

package/configs/plugins/mermaid-reporter-plugin.js

@@ -1,111 +1 @@
-const ACORN_DUMMY_VALUE = "✖";
-
-/* eslint-disable security/detect-object-injection */
-const mermaidNode = (pNode, pText) => {
-  const lNode = pNode
-    .replace(ACORN_DUMMY_VALUE, "__unknown__")
-    .replace(/^\.$|^\.\//g, "__currentPath__")
-    .replace(/^\.{2}$|^\.{2}\//g, "__prevPath__")
-    .replace(/[[\]/.@]/g, "_");
-  const lText = pText ? `["${pText}"]` : "";
-  return `${lNode}${lText}`;
-};
-
-const mermaidEdge = (pFrom, pTo) => {
-  const lFromNode = mermaidNode(pFrom.node);
-  const lToNode = mermaidNode(pTo.node);
-  return `${lFromNode} --> ${lToNode}`;
-};
-
-const mermaidEdges = (pEdges) => {
-  return pEdges.map((pEdge) => mermaidEdge(pEdge.from, pEdge.to)).join("\n");
-};
-
-const convertedEdgeSources = (pCruiseResult) => {
-  return pCruiseResult.modules.flatMap((pModule) => {
-    const lFrom = {
-      node: pModule.source,
-      text: pModule.source.split("/").slice(-1)[0],
-    };
-
-    return pModule.dependencies.map((pDep) => {
-      return {
-        from: lFrom,
-        to: {
-          node: pDep.resolved,
-          text: pDep.resolved.split("/").slice(-1)[0],
-        },
-      };
-    });
-  });
-};
-
-const indent = (pDepth = 0) => {
-  return "  ".repeat(pDepth);
-};
-
-const mermaidSubgraph = (pNode, pText, pChildren, pDepth) => {
-  return `${indent(pDepth)}subgraph ${mermaidNode(pNode, pText)}
-${pChildren}
-${indent(pDepth)}end`;
-};
-
-const mermaidSubgraphs = (pSource, pDepth = 0) => {
-  return Object.keys(pSource)
-    .map((pName) => {
-      const source = pSource[pName];
-      const children = mermaidSubgraphs(source.children, pDepth + 1);
-      if (children === "")
-        return `${indent(pDepth)}${mermaidNode(source.node, source.text)}`;
-
-      return mermaidSubgraph(source.node, source.text, children, pDepth);
-    })
-    .join("\n");
-};
-
-const convertedSubgraphSources = (pCruiseResult) => {
-  let lTree = {};
-
-  pCruiseResult.modules.forEach((pModule) => {
-    const paths = pModule.source.split("/");
-
-    paths.reduce((pChildren, pCurrentPath, pIndex) => {
-      if (!pChildren[pCurrentPath]) {
-        pChildren[pCurrentPath] = {
-          node: paths.slice(0, pIndex + 1).join("/"),
-          text: pCurrentPath,
-          children: {},
-        };
-      }
-      return pChildren[pCurrentPath].children;
-    }, lTree);
-  });
-
-  return lTree;
-};
-
-const renderMermaidThing = (pCruiseResult) => {
-  const subgraphs = convertedSubgraphSources(pCruiseResult);
-  const edges = convertedEdgeSources(pCruiseResult);
-
-  return `flowchart LR
-
-${mermaidSubgraphs(subgraphs)}
-${mermaidEdges(edges)}
-`;
-};
-
-/**
- * mermaid reporter plugin
- *
- * @param {import('../../types/dependency-cruiser').ICruiseResult} pCruiseResult -
- *      the output of a dependency-cruise adhering to dependency-cruiser's
- *      cruise result schema
- * @return {import('../../types/dependency-cruiser').IReporterOutput} -
- *      output: a string
- *      exitCode: 0
- */
-module.exports = (pCruiseResult) => ({
-  output: renderMermaidThing(pCruiseResult),
-  exitCode: 0,
-});
+module.exports = require("../../src/report/mermaid");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "11.9.0-beta-2",
+  "version": "11.10.0",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -117,7 +117,7 @@
     "test:i": "mocha --timeout 4000 \"test/**/*.spec.{js,mjs,cjs}\" -g \"^\\[[I]\\]\"",
     "test:u": "mocha --timeout 4000 \"test/**/*.spec.{js,mjs,cjs}\" --grep \"^\\[[U]\\]\"",
     "test:e": "mocha --timeout 4000 \"test/**/*.spec.{js,mjs,cjs}\" --grep \"^\\[[E]\\]\"",
-    "test:cover": "c8 --check-coverage --statements 99.9 --branches 99.7 --functions 100 --lines 99.9 --exclude \"{bin,configs,doc,docs,coverage,test,tools,webpack.conf.js,tmp*,src/**/*.template.js,src/cli/tools/svg-in-html-snippets/script.snippet.js,src/cli/init-config/get-user-input.js,src/cli/listeners/*/index.js}\" --reporter text-summary --reporter html --reporter json-summary npm test",
+    "test:cover": "c8 --check-coverage --statements 99.9 --branches 99.7 --functions 100 --lines 99.9 --exclude \"{bin,configs/*.js,configs/rules,configs/plugins/3d-*.js,configs/plugins/stats-*.js,doc,docs,coverage,test,tools,webpack.conf.js,tmp*,src/**/*.template.js,src/cli/tools/svg-in-html-snippets/script.snippet.js,src/cli/init-config/get-user-input.js,src/cli/listeners/*/index.js}\" --reporter text-summary --reporter html --reporter json-summary npm test",
     "test:glob": "set -f && test \"`bin/dependency-cruise.js test/extract/__mocks__/gather-globbing/packages/**/src/**/*.js | grep \"no dependency violations found\"`\" = \"✔ no dependency violations found (6 modules, 0 dependencies cruised)\"",
     "test:yarn-pnp": "npm-run-all test:yarn-pnp:cleanup test:yarn-pnp:pack test:yarn-pnp:copy test:yarn-pnp:install test:yarn-pnp:version test:yarn-pnp:run test:yarn-pnp:test test:yarn-pnp:cleanup",
     "test:yarn-pnp:pack": "npm pack",
@@ -164,17 +164,17 @@
     "@babel/core": "7.18.2",
     "@babel/plugin-transform-modules-commonjs": "7.18.2",
     "@babel/preset-typescript": "7.17.12",
-    "@swc/core": "1.2.194",
+    "@swc/core": "1.2.197",
     "@types/lodash": "4.14.182",
-    "@types/node": "17.0.36",
-    "@typescript-eslint/eslint-plugin": "5.26.0",
-    "@typescript-eslint/parser": "5.26.0",
-    "@vue/compiler-sfc": "3.2.36",
+    "@types/node": "17.0.40",
+    "@typescript-eslint/eslint-plugin": "5.27.1",
+    "@typescript-eslint/parser": "5.27.1",
+    "@vue/compiler-sfc": "3.2.37",
     "c8": "7.11.3",
     "chai": "4.3.6",
     "chai-json-schema": "1.5.1",
     "coffeescript": "2.7.0",
-    "eslint": "^8.16.0",
+    "eslint": "^8.17.0",
     "eslint-config-moving-meadow": "3.0.0",
     "eslint-config-prettier": "8.5.0",
     "eslint-plugin-budapestian": "3.0.2",
@@ -194,7 +194,7 @@
     "shx": "0.3.4",
     "svelte": "3.48.0",
     "symlink-dir": "5.0.1",
-    "typescript": "4.7.2",
+    "typescript": "4.7.3",
     "upem": "^7.0.0",
     "vue-template-compiler": "2.6.14",
     "yarn": "1.22.18"

package/src/cli/index.js

@@ -1,7 +1,7 @@
 const glob = require("glob");
-const _get = require("lodash/get");
-const _clone = require("lodash/clone");
-const _set = require("lodash/set");
+const get = require("lodash/get");
+const clone = require("lodash/clone");
+const set = require("lodash/set");
 
 const main = require("../main");
 const bus = require("../utl/bus");
@@ -19,7 +19,7 @@ const setUpPerformanceLogListener = require("./listeners/performance-log");
 
 function extractResolveOptions(pCruiseOptions) {
   let lResolveOptions = {};
-  const lWebPackConfigFileName = _get(
+  const lWebPackConfigFileName = get(
     pCruiseOptions,
     "ruleSet.options.webpackConfig.fileName",
     null
@@ -28,8 +28,8 @@ function extractResolveOptions(pCruiseOptions) {
   if (lWebPackConfigFileName) {
     lResolveOptions = extractWebpackResolveConfig(
       lWebPackConfigFileName,
-      _get(pCruiseOptions, "ruleSet.options.webpackConfig.env", null),
-      _get(pCruiseOptions, "ruleSet.options.webpackConfig.arguments", null)
+      get(pCruiseOptions, "ruleSet.options.webpackConfig.env", null),
+      get(pCruiseOptions, "ruleSet.options.webpackConfig.arguments", null)
     );
   }
   return lResolveOptions;
@@ -43,8 +43,8 @@ function addKnownViolations(pCruiseOptions) {
 
     // Check against json schema is already done in src/main/options/validate
     // so here we can just concentrate on the io
-    let lCruiseOptions = _clone(pCruiseOptions);
-    _set(lCruiseOptions, "ruleSet.options.knownViolations", lKnownViolations);
+    let lCruiseOptions = clone(pCruiseOptions);
+    set(lCruiseOptions, "ruleSet.options.knownViolations", lKnownViolations);
     return lCruiseOptions;
   }
   return pCruiseOptions;
@@ -52,7 +52,7 @@ function addKnownViolations(pCruiseOptions) {
 
 function extractTSConfigOptions(pCruiseOptions) {
   let lReturnValue = {};
-  const lTSConfigFileName = _get(
+  const lTSConfigFileName = get(
     pCruiseOptions,
     "ruleSet.options.tsConfig.fileName",
     null
@@ -67,7 +67,7 @@ function extractTSConfigOptions(pCruiseOptions) {
 
 function extractBabelConfigOptions(pCruiseOptions) {
   let lReturnValue = {};
-  const lBabelConfigFileName = _get(
+  const lBabelConfigFileName = get(
     pCruiseOptions,
     "ruleSet.options.babelConfig.fileName",
     null
@@ -85,12 +85,12 @@ function setUpListener(pCruiseOptions) {
     "cli-feedback": setUpCliFeedbackListener,
     "performance-log": setUpPerformanceLogListener,
   };
-  const lListenerID = _get(
+  const lListenerID = get(
     pCruiseOptions,
     "progress",
-    _get(pCruiseOptions, "ruleSet.options.progress.type")
+    get(pCruiseOptions, "ruleSet.options.progress.type")
   );
-  const lListenerFunction = _get(lString2Listener, lListenerID);
+  const lListenerFunction = get(lString2Listener, lListenerID);
   /* c8 ignore next 3 */
   if (Boolean(lListenerFunction)) {
     lListenerFunction(bus);

package/src/cli/init-config/environment-helpers.js

@@ -1,7 +1,7 @@
 const fs = require("fs");
 const path = require("path");
-const _has = require("lodash/has");
-const _get = require("lodash/get");
+const has = require("lodash/has");
+const get = require("lodash/get");
 const { DEFAULT_CONFIG_FILE_NAME } = require("../defaults");
 
 const LIKELY_SOURCE_FOLDERS = ["src", "lib", "app", "bin", "sources"];
@@ -41,7 +41,7 @@ function babelIsConfiguredInManifest() {
   let lReturnValue = false;
 
   try {
-    lReturnValue = _has(readManifest(), "babel");
+    lReturnValue = has(readManifest(), "babel");
   } catch (pError) {
     // silently ignore - we'll return false anyway then
   }
@@ -52,7 +52,7 @@ function isTypeModule() {
   let lReturnValue = false;
 
   try {
-    lReturnValue = _get(readManifest(), "type", "commonjs") === "module";
+    lReturnValue = get(readManifest(), "type", "commonjs") === "module";
   } catch (pError) {
     // silently ignore - we'll return false anyway then
   }

package/src/cli/init-config/normalize-init-options.js

@@ -1,4 +1,4 @@
-const _has = require("lodash/has");
+const has = require("lodash/has");
 const { version } = require("../../../src/meta.js");
 const {
   getSourceFolderCandidates,
@@ -40,7 +40,7 @@ module.exports = function normalizeInitOptions(pInitOptions) {
   if (lReturnValue.configType === "preset" && !lReturnValue.preset) {
     lReturnValue.preset = "dependency-cruiser/configs/recommended-warn-only";
   }
-  if (!_has(lReturnValue, "hasTestsOutsideSource")) {
+  if (!has(lReturnValue, "hasTestsOutsideSource")) {
     lReturnValue.hasTestsOutsideSource =
       !pInitOptions.isMonoRepo &&
       !hasTestsWithinSource(

package/src/cli/normalize-cli-options.js

@@ -1,9 +1,9 @@
 const fs = require("fs");
 const path = require("path");
-const _set = require("lodash/set");
-const _get = require("lodash/get");
-const _has = require("lodash/has");
-const _clone = require("lodash/clone");
+const set = require("lodash/set");
+const get = require("lodash/get");
+const has = require("lodash/has");
+const clone = require("lodash/clone");
 const loadConfig = require("../config-utl/extract-depcruise-config");
 const defaults = require("./defaults");
 
@@ -68,10 +68,10 @@ function ejectNonCLIOptions(pCliOptions, pKnownCliOptions) {
 }
 
 function normalizeConfigFileName(pCliOptions, pConfigWrapperName, pDefault) {
-  let lOptions = _clone(pCliOptions);
+  let lOptions = clone(pCliOptions);
 
-  if (_has(lOptions, pConfigWrapperName)) {
-    _set(
+  if (has(lOptions, pConfigWrapperName)) {
+    set(
       lOptions,
       `ruleSet.options.${pConfigWrapperName}.fileName`,
       getOptionValue(pDefault)(lOptions[pConfigWrapperName])
@@ -81,10 +81,10 @@ function normalizeConfigFileName(pCliOptions, pConfigWrapperName, pDefault) {
   }
 
   if (
-    _get(lOptions, `ruleSet.options.${pConfigWrapperName}`, null) &&
-    !_get(lOptions, `ruleSet.options.${pConfigWrapperName}.fileName`, null)
+    get(lOptions, `ruleSet.options.${pConfigWrapperName}`, null) &&
+    !get(lOptions, `ruleSet.options.${pConfigWrapperName}.fileName`, null)
   ) {
-    _set(lOptions, `ruleSet.options.${pConfigWrapperName}.fileName`, pDefault);
+    set(lOptions, `ruleSet.options.${pConfigWrapperName}.fileName`, pDefault);
   }
 
   return lOptions;
@@ -153,7 +153,7 @@ function validateAndGetKnownViolationsFileName(pKnownViolations) {
 }
 
 function normalizeKnownViolationsOption(pCliOptions) {
-  if (_has(pCliOptions, "ignoreKnown")) {
+  if (has(pCliOptions, "ignoreKnown")) {
     const lReturnValue = {
       knownViolationsFile: validateAndGetKnownViolationsFileName(
         pCliOptions.ignoreKnown
@@ -165,7 +165,7 @@ function normalizeKnownViolationsOption(pCliOptions) {
 }
 
 function normalizeValidationOption(pCliOptions) {
-  if (_has(pCliOptions, "validate")) {
+  if (has(pCliOptions, "validate")) {
     const rulesFile = validateAndNormalizeRulesFileName(pCliOptions.validate);
     return {
       rulesFile,
@@ -184,8 +184,8 @@ function normalizeValidationOption(pCliOptions) {
 function normalizeProgress(pCliOptions) {
   let lProgress = null;
 
-  if (_has(pCliOptions, "progress")) {
-    lProgress = _get(pCliOptions, "progress");
+  if (has(pCliOptions, "progress")) {
+    lProgress = get(pCliOptions, "progress");
     if (lProgress === true) {
       lProgress = "cli-feedback";
     }
@@ -211,13 +211,13 @@ module.exports = function normalizeOptions(
     ...ejectNonCLIOptions(pOptionsAsPassedFromCommander, pKnownCliOptions),
   };
 
-  if (_has(lOptions, "moduleSystems")) {
+  if (has(lOptions, "moduleSystems")) {
     lOptions.moduleSystems = lOptions.moduleSystems
       .split(",")
       .map((pString) => pString.trim());
   }
 
-  if (_has(lOptions, "config")) {
+  if (has(lOptions, "config")) {
     lOptions.validate = lOptions.config;
   }
 

package/src/config-utl/extract-babel-config.js

@@ -4,8 +4,8 @@
 const fs = require("fs");
 const path = require("path");
 const json5 = require("json5");
-const _get = require("lodash/get");
-const _has = require("lodash/has");
+const get = require("lodash/get");
+const has = require("lodash/has");
 const tryRequire = require("semver-try-require");
 const { supportedTranspilers } = require("../../src/meta.js");
 const makeAbsolute = require("./make-absolute");
@@ -48,7 +48,7 @@ function getJSON5Config(pBabelConfigFileName) {
   }
 
   if (pBabelConfigFileName.endsWith("package.json")) {
-    lReturnValue = _get(lReturnValue, "babel", {});
+    lReturnValue = get(lReturnValue, "babel", {});
   }
   return lReturnValue;
 }
@@ -63,7 +63,7 @@ function getConfig(pBabelConfigFileName) {
   };
   const lExtension = path.extname(pBabelConfigFileName);
 
-  if (!_has(lExtensionToParseFunction, lExtension)) {
+  if (!has(lExtensionToParseFunction, lExtension)) {
     throw new Error(
       `The babel config '${pBabelConfigFileName}' is in a format ('${lExtension}')\n` +
         "         dependency-cruiser doesn't support yet.\n"

package/src/config-utl/extract-depcruise-config/index.js

@@ -1,5 +1,5 @@
 const path = require("path");
-const _has = require("lodash/has");
+const has = require("lodash/has");
 const resolve = require("../../extract/resolve/resolve");
 const normalizeResolveOptions = require("../../main/resolve-options/normalize");
 const readConfig = require("./read-config");
@@ -78,7 +78,7 @@ function extractDepcruiseConfig(
 
   let lReturnValue = readConfig(lResolvedFileName, pBaseDirectory);
 
-  if (_has(lReturnValue, "extends")) {
+  if (has(lReturnValue, "extends")) {
     lReturnValue = processExtends(
       lReturnValue,
       pAlreadyVisited,

package/src/config-utl/extract-depcruise-config/merge-configs.js

@@ -1,7 +1,7 @@
 const util = require("util");
-const _get = require("lodash/get");
-const _uniqBy = require("lodash/uniqBy");
-const _uniqWith = require("lodash/uniqWith");
+const get = require("lodash/get");
+const uniqBy = require("lodash/uniqBy");
+const uniqWith = require("lodash/uniqWith");
 
 function extendNamedRule(pExtendedRule, pForbiddenArrayBase) {
   return pForbiddenArrayBase
@@ -31,7 +31,7 @@ function extendNamedRule(pExtendedRule, pForbiddenArrayBase) {
  */
 function mergeRules(pRuleArrayExtended, pRuleArrayBase) {
   // merge anonymous on 100% equality
-  let lAnonymousRules = _uniqWith(
+  let lAnonymousRules = uniqWith(
     pRuleArrayExtended.concat(pRuleArrayBase).filter((pRule) => !pRule.name),
     util.isDeepStrictEqual
   );
@@ -41,7 +41,7 @@ function mergeRules(pRuleArrayExtended, pRuleArrayBase) {
     .map((pNamedRule) => extendNamedRule(pNamedRule, pRuleArrayBase));
 
   // merge named rules based on unique name
-  lNamedRules = _uniqBy(
+  lNamedRules = uniqBy(
     // ordered extended => base because the uniqBy picks the
     // first it encounters and we want the ones from the
     // extended in case of a conflict
@@ -66,7 +66,7 @@ function mergeRules(pRuleArrayExtended, pRuleArrayBase) {
  * @return {Array} - the merged array
  */
 function mergeAllowedRules(pAllowedArrayExtended, pAllowedArrayBase) {
-  return _uniqWith(
+  return uniqWith(
     pAllowedArrayExtended.concat(pAllowedArrayBase),
     util.isDeepStrictEqual
   );
@@ -87,10 +87,10 @@ function mergeOptions(pOptionsExtended, pOptionsBase) {
  * @returns {string} - a string from the SeverityType value set
  */
 function mergeAllowedSeverities(pConfigExtended, pConfigBase) {
-  return _get(
+  return get(
     pConfigExtended,
     "allowedSeverity",
-    _get(pConfigBase, "allowedSeverity", "warn")
+    get(pConfigBase, "allowedSeverity", "warn")
   );
 }
 
@@ -110,16 +110,16 @@ function mergeAllowedSeverities(pConfigExtended, pConfigBase) {
  */
 module.exports = (pConfigExtended, pConfigBase) => {
   const lForbidden = mergeRules(
-    _get(pConfigExtended, "forbidden", []),
-    _get(pConfigBase, "forbidden", [])
+    get(pConfigExtended, "forbidden", []),
+    get(pConfigBase, "forbidden", [])
   );
   const lRequired = mergeRules(
-    _get(pConfigExtended, "required", []),
-    _get(pConfigBase, "required", [])
+    get(pConfigExtended, "required", []),
+    get(pConfigBase, "required", [])
   );
   const lAllowed = mergeAllowedRules(
-    _get(pConfigExtended, "allowed", []),
-    _get(pConfigBase, "allowed", [])
+    get(pConfigExtended, "allowed", []),
+    get(pConfigBase, "allowed", [])
   );
 
   return {
@@ -132,8 +132,8 @@ module.exports = (pConfigExtended, pConfigBase) => {
         }
       : {}),
     options: mergeOptions(
-      _get(pConfigExtended, "options", {}),
-      _get(pConfigBase, "options", {})
+      get(pConfigExtended, "options", {}),
+      get(pConfigBase, "options", {})
     ),
   };
 };

package/src/config-utl/extract-ts-config.js

@@ -1,6 +1,6 @@
 const path = require("path");
 const tryRequire = require("semver-try-require");
-const _get = require("lodash/get");
+const get = require("lodash/get");
 const { supportedTranspilers } = require("../../src/meta.js");
 
 const typescript = tryRequire("typescript", supportedTranspilers.typescript);
@@ -11,7 +11,7 @@ const FORMAT_DIAGNOSTICS_HOST = {
 
     // depends on the platform which branch is taken, hence the c8 ignore
     /* c8 ignore start */
-    if (_get(typescript, "sys.useCaseSensitiveFileNames", false)) {
+    if (get(typescript, "sys.useCaseSensitiveFileNames", false)) {
       lReturnValue = pFileName;
     }
     /* c8 ignore stop */

package/src/enrich/derive/reachable/index.js

@@ -1,19 +1,17 @@
 /* eslint-disable security/detect-object-injection, no-inline-comments */
 
-const _clone = require("lodash/clone");
-const _get = require("lodash/get");
-const _has = require("lodash/has");
+const clone = require("lodash/clone");
+const get = require("lodash/get");
+const has = require("lodash/has");
 const matchers = require("../../../validate/matchers");
 const { extractGroups } = require("../../../utl/regex-util");
 const getPath = require("./get-path");
 
 function getReachableRules(pRuleSet) {
-  return _get(pRuleSet, "forbidden", [])
-    .filter((pRule) => _has(pRule.to, "reachable"))
+  return get(pRuleSet, "forbidden", [])
+    .filter((pRule) => has(pRule.to, "reachable"))
     .concat(
-      _get(pRuleSet, "allowed", []).filter((pRule) =>
-        _has(pRule.to, "reachable")
-      )
+      get(pRuleSet, "allowed", []).filter((pRule) => has(pRule.to, "reachable"))
     );
 }
 
@@ -87,8 +85,8 @@ function hasCapturingGroups(pRule) {
   const lCapturingGroupPlaceholderRe = "\\$[0-9]+";
 
   return Boolean(
-    _get(pRule, "to.path", "").match(lCapturingGroupPlaceholderRe) ||
-      _get(pRule, "to.pathNot", "").match(lCapturingGroupPlaceholderRe)
+    get(pRule, "to.path", "").match(lCapturingGroupPlaceholderRe) ||
+      get(pRule, "to.pathNot", "").match(lCapturingGroupPlaceholderRe)
   );
 }
 function shouldAddReachable(pRule, pModuleTo, pGraph) {
@@ -156,7 +154,7 @@ function addReachableToModule(pModule, pGraph, pReachableRule) {
 
 function addReachabilityToGraph(pGraph, pReachableRule) {
   return pGraph.map((pModule) => {
-    let lClonedModule = _clone(pModule);
+    let lClonedModule = clone(pModule);
 
     if (shouldAddReaches(pReachableRule, lClonedModule)) {
       lClonedModule = addReachesToModule(lClonedModule, pGraph, pReachableRule);
@@ -177,6 +175,6 @@ module.exports = (pGraph, pRuleSet) => {
 
   return lReachableRules.reduce(
     (pReturnGraph, pRule) => addReachabilityToGraph(pReturnGraph, pRule),
-    _clone(pGraph)
+    clone(pGraph)
   );
 };

package/src/enrich/enrich-modules.js

@@ -1,4 +1,4 @@
-const _get = require("lodash/get");
+const get = require("lodash/get");
 const bus = require("../utl/bus");
 const busLogLevels = require("../utl/bus-log-levels");
 const addFocus = require("../../src/graph-utl/add-focus");
@@ -31,7 +31,7 @@ module.exports = function enrichModules(pModules, pOptions) {
   bus.emit("progress", "analyzing: add focus (if any)", {
     level: busLogLevels.INFO,
   });
-  lModules = addFocus(lModules, _get(pOptions, "focus"));
+  lModules = addFocus(lModules, get(pOptions, "focus"));
 
   // when validate === false we might want to skip the addValidations.
   // We don't at this time, however, as "valid" is a mandatory

package/src/enrich/summarize/add-rule-set-used.js

@@ -1,20 +1,20 @@
-const _get = require("lodash/get");
-const _clone = require("lodash/clone");
+const get = require("lodash/get");
+const clone = require("lodash/clone");
 
 // the fixed name for allowed rules served a purpose during the extraction
 // process - but it's not necessary to reflect it in the output.
 function removeNames(pRule) {
-  const lReturnValue = _clone(pRule);
+  const lReturnValue = clone(pRule);
 
   Reflect.deleteProperty(lReturnValue, "name");
   return lReturnValue;
 }
 
 module.exports = function addRuleSetUsed(pOptions) {
-  const lForbidden = _get(pOptions, "ruleSet.forbidden");
-  const lAllowed = _get(pOptions, "ruleSet.allowed");
-  const lAllowedSeverity = _get(pOptions, "ruleSet.allowedSeverity");
-  const lRequired = _get(pOptions, "ruleSet.required");
+  const lForbidden = get(pOptions, "ruleSet.forbidden");
+  const lAllowed = get(pOptions, "ruleSet.allowed");
+  const lAllowedSeverity = get(pOptions, "ruleSet.allowedSeverity");
+  const lRequired = get(pOptions, "ruleSet.required");
 
   return Object.assign(
     lForbidden ? { forbidden: lForbidden } : {},

package/src/enrich/summarize/summarize-modules.js

@@ -1,7 +1,7 @@
 const _flattenDeep = require("lodash/flattenDeep");
-const _get = require("lodash/get");
-const _has = require("lodash/has");
-const _uniqWith = require("lodash/uniqWith");
+const get = require("lodash/get");
+const has = require("lodash/has");
+const uniqWith = require("lodash/uniqWith");
 const { findRuleByName } = require("../../graph-utl/rule-set");
 const compare = require("../../graph-utl/compare");
 const isSameViolation = require("./is-same-violation");
@@ -24,8 +24,8 @@ function toDependencyViolationSummary(pRule, pModule, pDependency, pRuleSet) {
   };
 
   if (
-    _has(pDependency, "cycle") &&
-    _get(findRuleByName(pRuleSet, pRule.name), "to.circular")
+    has(pDependency, "cycle") &&
+    get(findRuleByName(pRuleSet, pRule.name), "to.circular")
   ) {
     lReturnValue = {
       ...lReturnValue,
@@ -35,9 +35,9 @@ function toDependencyViolationSummary(pRule, pModule, pDependency, pRuleSet) {
   }
 
   if (
-    _has(pModule, "instability") &&
-    _has(pDependency, "instability") &&
-    _has(findRuleByName(pRuleSet, pRule.name), "to.moreUnstable")
+    has(pModule, "instability") &&
+    has(pDependency, "instability") &&
+    has(findRuleByName(pRuleSet, pRule.name), "to.moreUnstable")
   ) {
     lReturnValue = {
       ...lReturnValue,
@@ -90,7 +90,7 @@ function toModuleViolationSummary(pRule, pModule, pRuleSet) {
   ];
   if (
     pModule.reaches &&
-    _get(findRuleByName(pRuleSet, pRule.name), "to.reachable")
+    get(findRuleByName(pRuleSet, pRule.name), "to.reachable")
   ) {
     lReturnValue = pModule.reaches
       .filter((pReachable) => pReachable.asDefinedInRule === pRule.name)
@@ -135,7 +135,7 @@ function extractModuleViolations(pModules, pRuleSet) {
 }
 
 module.exports = function summarizeModules(pModules, pRuleSet) {
-  return _uniqWith(
+  return uniqWith(
     extractDependencyViolations(pModules, pRuleSet)
       .concat(extractModuleViolations(pModules, pRuleSet))
       .sort(compare.violations),