dependency-cruiser 16.3.9 → 16.3.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "16.3.9",
+  "version": "16.3.10",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -49,6 +49,7 @@
     "Jessica Kerr (https://jessitron.com)",
     "Creative Ataraxia (https://github.com/Creative-Ataraxia)",
     "0xflotus (https://github.com/0xflotus)",
+    "KenjiroKubota (https://github.com/kubotak-is)",
     "Daniel Edholm Ignat (https://github.com/dignite)",
     "Daniel Rodríguez Rivero (https://danielorodriguez.com)",
     "Nick Ribal (https://github.com/elektronik2k5)",
@@ -145,12 +146,11 @@
     "acorn-walk": "8.3.3",
     "ajv": "8.17.1",
     "commander": "12.1.0",
-    "enhanced-resolve": "5.17.0",
+    "enhanced-resolve": "5.17.1",
     "ignore": "5.3.1",
     "interpret": "^3.1.1",
     "is-installed-globally": "1.0.0",
     "json5": "2.2.3",
-    "lodash": "4.17.21",
     "memoize": "10.0.0",
     "picocolors": "1.0.1",
     "picomatch": "4.0.2",

package/src/cli/index.mjs

@@ -1,6 +1,5 @@
 import { join } from "node:path";
 import picomatch from "picomatch";
-import set from "lodash/set.js";
 import isInstalledGlobally from "is-installed-globally";
 import pc from "picocolors";
 
@@ -10,6 +9,7 @@ import { write } from "./utl/io.mjs";
 import setUpCliFeedbackListener from "./listeners/cli-feedback.mjs";
 import setUpPerformanceLogListener from "./listeners/performance-log/index.mjs";
 import setUpNDJSONListener from "./listeners/ndjson.mjs";
+import { set } from "#utl/object-util.mjs";
 import cruise from "#main/cruise.mjs";
 import { INFO, bus } from "#utl/bus.mjs";
 

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

@@ -1,6 +1,5 @@
 import { accessSync, R_OK } from "node:fs";
 import { isAbsolute } from "node:path";
-import set from "lodash/set.js";
 import {
   RULES_FILE_NAME_SEARCH_ARRAY,
   DEFAULT_BASELINE_FILE_NAME,
@@ -11,6 +10,7 @@ import {
   BABEL_CONFIG,
   OLD_DEFAULT_RULES_FILE_NAME,
 } from "./defaults.mjs";
+import { set } from "#utl/object-util.mjs";
 import loadConfig from "#config-utl/extract-depcruise-config/index.mjs";
 
 function getOptionValue(pDefault) {

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

@@ -1,6 +1,5 @@
 import { isDeepStrictEqual } from "node:util";
-import uniqBy from "lodash/uniqBy.js";
-import uniqWith from "lodash/uniqWith.js";
+import { uniqBy, uniqWith } from "#utl/array-util.mjs";
 
 function extendNamedRule(pExtendedRule, pForbiddenArrayBase) {
   return pForbiddenArrayBase

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

@@ -1,7 +1,7 @@
-import uniqWith from "lodash/uniqWith.js";
 import isSameViolation from "./is-same-violation.mjs";
 import { findRuleByName } from "#graph-utl/rule-set.mjs";
 import compare from "#graph-utl/compare.mjs";
+import { uniqWith } from "#utl/array-util.mjs";
 
 function cutNonTransgressions(pModule) {
   return {

package/src/extract/extract-dependencies.mjs

@@ -1,5 +1,4 @@
 import { join, extname, dirname } from "node:path";
-import uniqBy from "lodash/uniqBy.js";
 import { extract as acornExtract } from "./acorn/extract.mjs";
 import {
   extract as tscExtract,
@@ -14,7 +13,7 @@ import {
   detectPreCompilationNess,
   extractModuleAttributes,
 } from "./helpers.mjs";
-import { intersects } from "#utl/array-util.mjs";
+import { uniqBy, intersects } from "#utl/array-util.mjs";
 
 function extractWithTsc(pCruiseOptions, pFileName, pTranspileOptions) {
   let lDependencies = tscExtract(pCruiseOptions, pFileName, pTranspileOptions);

package/src/extract/resolve/external-module-helpers.mjs

@@ -26,7 +26,7 @@ import { isScoped, isRelativeModuleName } from "./module-classifiers.mjs";
  *
  * At this time we don't take situations into account where the caller includes
  * a node module through a local path (which could make sense if you're on
- * non-commonJS and are still using node_modules) e.g. '../node_modules/lodash/fp'
+ * non-commonJS and are still using node_modules) e.g. '../node_modules/oldash/fp'
  *
  * @param  {string} pModule a module name
  * @return {string}         the module name root
@@ -58,8 +58,8 @@ export function getPackageRoot(pModule) {
  * returns the contents of the package.json of the given pModule as it would
  * resolve from pBaseDirectory
  *
- * e.g. to get the package.json of `lodash` that is required bya
- * `src/report/something.js` use `getPackageJSON('lodash', 'src/report/');`
+ * e.g. to get the package.json of `oldash` that is required bya
+ * `src/report/something.js` use `getPackageJSON('oldash', 'src/report/');`
  *
  * The pBaseDirectory parameter is necessary because dependency-cruiser/ this module
  * will have a different base dir, and will hence resolve either to the

package/src/graph-utl/consolidate-modules.mjs

@@ -1,5 +1,5 @@
-import uniqBy from "lodash/uniqBy.js";
 import compare from "./compare.mjs";
+import { uniqBy } from "#utl/array-util.mjs";
 
 function mergeModule(pLeftModule, pRightModule) {
   return {

package/src/main/helpers.mjs

@@ -1,6 +1,4 @@
-import get from "lodash/get.js";
-import has from "lodash/has.js";
-import set from "lodash/set.js";
+import { has, get, set } from "#utl/object-util.mjs";
 
 const RE_PROPERTIES = [
   "path",
@@ -30,7 +28,7 @@ export function normalizeREProperties(
   let lPropertyContainer = structuredClone(pPropertyContainer);
 
   for (const lProperty of pREProperties) {
-    // lProperty can be nested properties, so we use lodash.has and lodash.get
+    // lProperty can be nested properties, so we use _.has and _.get
     // instead of elvis operators
     if (has(lPropertyContainer, lProperty)) {
       set(

package/src/main/rule-set/assert-validity.mjs

@@ -1,10 +1,9 @@
 import Ajv from "ajv";
 import safeRegex from "safe-regex";
-import has from "lodash/has.js";
-import get from "lodash/get.js";
 import { assertCruiseOptionsValid } from "../options/assert-validity.mjs";
 import { normalizeToREAsString } from "../helpers.mjs";
 import configurationSchema from "#configuration-schema";
+import { has, get } from "#utl/object-util.mjs";
 
 const ajv = new Ajv();
 // the default for this is 25 - as noted in the safe-regex source code already,
@@ -29,8 +28,8 @@ function assertSchemaCompliance(pSchema, pConfiguration) {
 }
 
 function hasPath(pObject, pSection, pCondition) {
-  // pCondition can be nested properties, so we use lodash.has instead
-  // of elvis operators
+  // pCondition can be nested properties, so we use a bespoke
+  // 'has' function instead of simple elvis operators
   return has(pObject, pSection) && has(pObject[pSection], pCondition);
 }
 

package/src/meta.cjs

@@ -1,7 +1,7 @@
 /* generated - don't edit */
 
 module.exports = {
-  version: "16.3.9",
+  version: "16.3.10",
   engines: {
     node: "^18.17||>=20",
   },

package/src/report/dot/index.mjs

@@ -1,11 +1,11 @@
 /* eslint-disable prefer-template */
-import get from "lodash/get.js";
 import theming from "./theming.mjs";
 import moduleUtl from "./module-utl.mjs";
 import prepareFolderLevel from "./prepare-folder-level.mjs";
 import prepareCustomLevel from "./prepare-custom-level.mjs";
 import prepareFlatLevel from "./prepare-flat-level.mjs";
 import { applyFilters } from "#graph-utl/filter-bank.mjs";
+import { get } from "#utl/object-util.mjs";
 
 // not importing EOL from "node:os" so output is the same on windows and unices
 const EOL = "\n";
@@ -151,8 +151,8 @@ function pryReporterOptionsFromResults(pGranularity, pResults) {
   const lFallbackReporterOptions =
     pResults?.summary?.optionsUsed?.reporterOptions?.dot;
 
-  // using lodash.get here because the reporter options will contain nested
-  // properties, which it handles for us
+  // using a bespoke 'get' function here because the reporter options will
+  // contain nested properties, which it handles for us
   return get(
     pResults,
     GRANULARITY2REPORTER_OPTIONS.get(pGranularity),

package/src/report/dot/theming.mjs

@@ -1,6 +1,5 @@
-import get from "lodash/get.js";
-import has from "lodash/has.js";
 import DEFAULT_THEME from "./default-theme.mjs";
+import { has, get } from "#utl/object-util.mjs";
 
 function matchesRE(pValue, pRE) {
   const lMatchResult = pValue.match && pValue.match(pRE);
@@ -14,10 +13,13 @@ function matchesCriterion(pModuleKey, pCriterion) {
 
 function moduleOrDependencyMatchesCriteria(pSchemeEntry, pModule) {
   return Object.keys(pSchemeEntry.criteria).every((pKey) => {
-    // we use lodash.get here because in the criteria you can enter
-    // nested keys like "rules[0].severity" : "error", and lodash.get handles
-    // that for us
-    const lCriterion = get(pSchemeEntry.criteria, pKey);
+    // the keys can have paths in them like {"rules[0].severity": "info"}
+    // To get the criterion treat that key as a string and not as a path
+    // eslint-disable-next-line security/detect-object-injection
+    const lCriterion = pSchemeEntry.criteria[pKey];
+    // we use a bespoke 'get' here because in the criteria you can enter
+    // nested keys like "rules[0].severity" : "error", and that function
+    // handles those for us
     const lModuleKey = get(pModule, pKey);
 
     if (!(lModuleKey || has(pModule, pKey))) {

package/src/utl/array-util.mjs

@@ -13,3 +13,27 @@ export function intersects(pLeftArray, pRightArray) {
 export function uniq(pArray) {
   return [...new Set(pArray)];
 }
+
+/**
+ * @param {any[]} pArray
+ * @param {function} pIteratee
+ * @returns {any[]}
+ */
+export function uniqBy(pArray, pIteratee) {
+  return pArray.filter(
+    (pElement, pIndex, pSelf) =>
+      pIndex === pSelf.findIndex((pY) => pIteratee(pElement) === pIteratee(pY)),
+  );
+}
+
+/**
+ * @param {any[]} pArray
+ * @param {function} pComparator
+ * @returns {any[]}
+ */
+export function uniqWith(pArray, pComparator) {
+  return pArray.filter(
+    (pElement, pIndex, pSelf) =>
+      pIndex === pSelf.findIndex((pY) => pComparator(pElement, pY)),
+  );
+}

package/src/utl/object-util.mjs

@@ -0,0 +1,40 @@
+/* eslint-disable security/detect-object-injection */
+
+export function get(pObject, pPath, pDefault) {
+  if (!pObject || !pPath) {
+    return pDefault;
+  }
+  // Regex explained: https://regexr.com/58j0k
+  const lPathArray = pPath.match(/([^[.\]])+/g);
+
+  const lReturnValue = lPathArray.reduce((pPreviousObject, pKey) => {
+    return pPreviousObject && pPreviousObject[pKey];
+  }, pObject);
+
+  if (!lReturnValue) {
+    return pDefault;
+  }
+  return lReturnValue;
+}
+
+export function set(pObject, pPath, pValue) {
+  const lPathArray = pPath.match(/([^[.\]])+/g);
+
+  lPathArray.reduce((pPreviousObject, pKey, pIndex) => {
+    if (pIndex === lPathArray.length - 1) {
+      pPreviousObject[pKey] = pValue;
+    } else if (!pPreviousObject[pKey]) {
+      pPreviousObject[pKey] = {};
+    }
+    return pPreviousObject[pKey];
+  }, pObject);
+}
+
+/**
+ * @param {any} pObject
+ * @param {string} pPath
+ * @returns {boolean}
+ */
+export function has(pObject, pPath) {
+  return Boolean(get(pObject, pPath));
+}