refactor: dedups uniq

src/enrich/derive/folders/aggregate-to-folders.mjs

@@ -10,6 +10,7 @@ import {
   object2Array,
 } from "./utl.mjs";
 import IndexedModuleGraph from "#graph-utl/indexed-module-graph.mjs";
+import { uniq } from "#utl/array-util.mjs";
 
 function upsertCouplings(pAllDependents, pNewDependents) {
   pNewDependents.forEach((pNewDependent) => {
@@ -53,13 +54,11 @@ function sumCounts(pAll, pCurrent) {
 }
 
 function getFolderLevelCouplings(pCouplingArray) {
-  return Array.from(
-    new Set(
-      pCouplingArray.map((pCoupling) =>
-        dirname(pCoupling.name) === "."
-          ? pCoupling.name
-          : dirname(pCoupling.name),
-      ),
+  return uniq(
+    pCouplingArray.map((pCoupling) =>
+      dirname(pCoupling.name) === "."
+        ? pCoupling.name
+        : dirname(pCoupling.name),
     ),
   ).map((pCoupling) => ({ name: pCoupling }));
 }
@@ -95,9 +94,7 @@ function deNormalizeInstability(pFolder, _, pAllFolders) {
     }),
   };
 }
-function uniq(pArray) {
-  return [...new Set(pArray)];
-}
+
 function getSinks(pFolders) {
   const lKnownFolders = new Set(pFolders.map(({ name }) => name));
   const lAllFolders = uniq(

src/extract/resolve/merge-manifests.mjs

@@ -1,3 +1,5 @@
+import { uniq } from "#utl/array-util.mjs";
+
 /* eslint-disable security/detect-object-injection */
 function normalizeManifestKeys(pManifest) {
   let lReturnValue = pManifest;
@@ -19,9 +21,7 @@ function mergeDependencyKey(pClosestDependencyKey, pFurtherDependencyKey) {
 }
 
 function mergeDependencyArray(pClosestDependencyKey, pFurtherDependencyKey) {
-  return Array.from(
-    new Set(pClosestDependencyKey.concat(pFurtherDependencyKey)),
-  );
+  return uniq(pClosestDependencyKey.concat(pFurtherDependencyKey));
 }
 
 function isInterestingKey(pKey) {
@@ -35,11 +35,9 @@ function getDependencyKeys(pPackage) {
 }
 
 function getJointUniqueDependencyKeys(pClosestPackage, pFurtherPackage) {
-  return Array.from(
-    new Set(
-      getDependencyKeys(pClosestPackage).concat(
-        getDependencyKeys(pFurtherPackage),
-      ),
+  return uniq(
+    getDependencyKeys(pClosestPackage).concat(
+      getDependencyKeys(pFurtherPackage),
     ),
   );
 }

src/graph-utl/consolidate-module-dependencies.mjs

@@ -1,16 +1,13 @@
 import reject from "lodash/reject.js";
 import compare from "./compare.mjs";
+import { uniq } from "#utl/array-util.mjs";
 
 function mergeDependency(pLeftDependency, pRightDependency) {
   return {
     ...pLeftDependency,
     ...pRightDependency,
-    dependencyTypes: Array.from(
-      new Set(
-        pLeftDependency.dependencyTypes.concat(
-          pRightDependency.dependencyTypes,
-        ),
-      ),
+    dependencyTypes: uniq(
+      pLeftDependency.dependencyTypes.concat(pRightDependency.dependencyTypes),
     ),
     rules: pLeftDependency.rules
       .concat(pRightDependency?.rules ?? [])

src/main/options/normalize.mjs

@@ -1,14 +1,11 @@
 /* eslint-disable security/detect-object-injection */
 import { normalizeREProperties } from "../helpers.mjs";
 import defaults from "./defaults.mjs";
+import { uniq } from "#utl/array-util.mjs";
 
 const DEFAULT_CACHE_FOLDER = "node_modules/.cache/dependency-cruiser";
 const DEFAULT_CACHE_STRATEGY = "metadata";
 
-function uniq(pArray) {
-  return [...new Set(pArray)];
-}
-
 function normalizeFilterOption(pFilterOption) {
   let lReturnValue = pFilterOption || {};
 

src/utl/array-util.mjs

@@ -9,3 +9,7 @@
 export function intersects(pLeftArray, pRightArray) {
   return pLeftArray.some((pLeftItem) => pRightArray.includes(pLeftItem));
 }
+
+export function uniq(pArray) {
+  return [...new Set(pArray)];
+}