dependency-cruiser 13.0.0-beta-1 → 13.0.0-beta-2

package/README.md

@@ -63,7 +63,7 @@ npx depcruise src --include-only "^src" --output-type dot | dot -T svg > depende
   options in the [command line interface](./doc/cli.md) documentation.
 - _[Real world samples](./doc/real-world-samples.md)_
   contains dependency cruises of some of the most used projects on npm.
-- If our grandma is more into formats like `mermaid`, `json`, `csv`, `html` or plain text
+- If your grandma is more into formats like `mermaid`, `json`, `csv`, `html` or plain text
   we've [got her covered](./doc/cli.md#--output-type-specify-the-output-format)
   as well.
 
@@ -107,7 +107,7 @@ Sample rule:
 #### Report them
 
 ```sh
-npx depcruise --config .dependency-cruiser.js src
+npx depcruise src
 ```
 
 This will validate against your rules and shows any violations in an eslint-like format:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "13.0.0-beta-1",
+  "version": "13.0.0-beta-2",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -46,24 +46,19 @@
   "main": "src/main/index.mjs",
   "exports": {
     ".": {
-      "import": "./src/main/index.mjs",
-      "require": "./dist/main/index.js"
+      "import": "./src/main/index.mjs"
     },
     "./config-utl/extract-babel-config": {
-      "import": "./src/config-utl/extract-babel-config.mjs",
-      "require": "./dist/config-utl/extract-babel-config.js"
+      "import": "./src/config-utl/extract-babel-config.mjs"
     },
     "./config-utl/extract-depcruise-config": {
-      "import": "./src/config-utl/extract-depcruise-config/index.mjs",
-      "require": "./dist/config-utl/extract-depcruise-config/index.js"
+      "import": "./src/config-utl/extract-depcruise-config/index.mjs"
     },
     "./config-utl/extract-ts-config": {
-      "import": "./src/config-utl/extract-ts-config.mjs",
-      "require": "./dist/config-utl/extract-ts-config.js"
+      "import": "./src/config-utl/extract-ts-config.mjs"
     },
     "./config-utl/extract-webpack-resolve-config": {
-      "import": "./src/config-utl/extract-webpack-resolve-config.mjs",
-      "require": "./dist/config-utl/extract-webpack-resolve-config.js"
+      "import": "./src/config-utl/extract-webpack-resolve-config.mjs"
     },
     "./sample-reporter-plugin": "./configs/plugins/stats-reporter-plugin.js",
     "./sample-3d-reporter-plugin": "./configs/plugins/3d-reporter-plugin.js",
@@ -153,11 +148,11 @@
     "acorn-walk": "8.2.0",
     "ajv": "8.12.0",
     "chalk": "5.2.0",
-    "commander": "10.0.0",
+    "commander": "10.0.1",
     "enhanced-resolve": "5.12.0",
     "figures": "5.0.0",
     "get-stream": "^6.0.1",
-    "glob": "10.0.0",
+    "glob": "10.1.0",
     "handlebars": "4.7.7",
     "ignore": "5.2.4",
     "indent-string": "5.0.0",
@@ -168,23 +163,23 @@
     "prompts": "2.4.2",
     "rechoir": "^0.8.0",
     "safe-regex": "2.1.1",
-    "semver": "^7.3.7",
+    "semver": "^7.4.0",
     "semver-try-require": "6.2.2",
     "teamcity-service-messages": "0.1.14",
     "tsconfig-paths-webpack-plugin": "4.0.1",
-    "watskeburt": "0.10.1",
+    "watskeburt": "0.10.2",
     "wrap-ansi": "8.1.0"
   },
   "devDependencies": {
     "@babel/core": "7.21.4",
     "@babel/plugin-transform-modules-commonjs": "7.21.2",
     "@babel/preset-typescript": "7.21.4",
-    "@swc/core": "1.3.49",
-    "@types/lodash": "4.14.192",
+    "@swc/core": "1.3.50",
+    "@types/lodash": "4.14.194",
     "@types/node": "18.15.11",
     "@types/prompts": "2.4.4",
-    "@typescript-eslint/eslint-plugin": "5.57.1",
-    "@typescript-eslint/parser": "5.57.1",
+    "@typescript-eslint/eslint-plugin": "5.58.0",
+    "@typescript-eslint/parser": "5.58.0",
     "@vue/compiler-sfc": "3.2.47",
     "c8": "7.13.0",
     "chai": "4.3.7",

package/src/cache/helpers.mjs

@@ -1,7 +1,7 @@
 /* eslint-disable import/exports-last */
 import { createHash } from "node:crypto";
 import { readFileSync } from "node:fs";
-import path from "node:path";
+import { extname } from "node:path";
 import memoize from "lodash/memoize.js";
 import { filenameMatchesPattern } from "../graph-utl/match-facade.mjs";
 
@@ -69,7 +69,7 @@ export function includeOnlyFilter(pIncludeOnlyFilter) {
  * @returns {(pFileName: string) => boolean}
  */
 export function hasInterestingExtension(pExtensions) {
-  return (pFileName) => pExtensions.has(path.extname(pFileName));
+  return (pFileName) => pExtensions.has(extname(pFileName));
 }
 
 /**

package/src/cli/index.mjs

@@ -23,13 +23,13 @@ import setUpPerformanceLogListener from "./listeners/performance-log/index.mjs";
 import setUpNDJSONListener from "./listeners/ndjson.mjs";
 import initConfig from "./init-config/index.mjs";
 
-function extractResolveOptions(pCruiseOptions) {
+async function extractResolveOptions(pCruiseOptions) {
   let lResolveOptions = {};
   const lWebPackConfigFileName =
     pCruiseOptions?.ruleSet?.options?.webpackConfig?.fileName ?? null;
 
   if (lWebPackConfigFileName) {
-    lResolveOptions = extractWebpackResolveConfig(
+    lResolveOptions = await extractWebpackResolveConfig(
       lWebPackConfigFileName,
       pCruiseOptions?.ruleSet?.options?.webpackConfig?.env ?? null,
       pCruiseOptions?.ruleSet?.options?.webpackConfig?.arguments ?? null
@@ -118,7 +118,7 @@ async function runCruise(pFileDirectoryArray, pCruiseOptions) {
   const lReportingResult = await cruise(
     pFileDirectoryArray,
     lCruiseOptions,
-    extractResolveOptions(lCruiseOptions),
+    await extractResolveOptions(lCruiseOptions),
     {
       tsConfig: extractTSConfigOptions(lCruiseOptions),
       babelConfig: await extractBabelConfigOptions(lCruiseOptions),

package/src/cli/init-config/validators.mjs

@@ -1,10 +1,10 @@
-import fs from "node:fs";
+import { statSync } from "node:fs";
 import { toSourceLocationArray } from "./environment-helpers.mjs";
 
 export function validateLocation(pLocations) {
   for (const lLocation of toSourceLocationArray(pLocations)) {
     try {
-      if (!fs.statSync(lLocation).isDirectory()) {
+      if (!statSync(lLocation).isDirectory()) {
         return `'${lLocation}' doesn't seem to be a folder - please try again`;
       }
     } catch (pError) {

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

@@ -1,5 +1,5 @@
-import fs from "node:fs";
-import path from "node:path";
+import { accessSync, R_OK } from "node:fs";
+import { isAbsolute } from "node:path";
 import set from "lodash/set.js";
 import get from "lodash/get.js";
 import has from "lodash/has.js";
@@ -45,7 +45,7 @@ function normalizeConfigFileName(pCliOptions, pConfigWrapperName, pDefault) {
 
 function fileExists(pFileName) {
   try {
-    fs.accessSync(pFileName, fs.R_OK);
+    accessSync(pFileName, R_OK);
     return true;
   } catch (pError) {
     return false;
@@ -129,7 +129,7 @@ async function normalizeValidationOption(pCliOptions) {
   return {
     rulesFile,
     ruleSet: await loadConfig(
-      path.isAbsolute(rulesFile) ? rulesFile : `./${rulesFile}`
+      isAbsolute(rulesFile) ? rulesFile : `./${rulesFile}`
     ),
     validate: true,
   };

package/src/cli/utl/io.mjs

@@ -32,8 +32,9 @@ function writeToStdOut(pString, pBufferSize = PIPE_BUFFER_SIZE) {
 
   /* eslint no-plusplus: 0 */
   for (lIndex = 0; lIndex < lNumberOfChunks; lIndex++) {
+    const lChunkStart = lIndex * pBufferSize;
     process.stdout.write(
-      pString.substr(lIndex * pBufferSize, pBufferSize),
+      pString.substring(lChunkStart, lChunkStart + pBufferSize),
       "utf8"
     );
   }

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

@@ -7,7 +7,7 @@ import tryImport from "semver-try-require";
 import meta from "../meta.js";
 import makeAbsolute from "./make-absolute.mjs";
 
-async function getCommonJSConfig(pBabelConfigFileName) {
+async function getJSConfig(pBabelConfigFileName) {
   let lReturnValue = {};
 
   try {
@@ -55,9 +55,9 @@ function getJSON5Config(pBabelConfigFileName) {
 
 async function getConfig(pBabelConfigFileName) {
   const lExtensionToParseFunction = {
-    ".js": getCommonJSConfig,
-    ".cjs": getCommonJSConfig,
-    ".mjs": getCommonJSConfig,
+    ".js": getJSConfig,
+    ".cjs": getJSConfig,
+    ".mjs": getJSConfig,
     "": getJSON5Config,
     ".json": getJSON5Config,
     ".json5": getJSON5Config,

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

@@ -1,4 +1,4 @@
-import path from "node:path";
+import { dirname } from "node:path";
 import cloneDeep from "lodash/cloneDeep.js";
 import has from "lodash/has.js";
 import { resolve } from "../../extract/resolve/resolve.mjs";
@@ -67,7 +67,7 @@ export default async function extractDepcruiseConfig(
     ),
     "cli"
   );
-  const lBaseDirectory = path.dirname(lResolvedFileName);
+  const lBaseDirectory = dirname(lResolvedFileName);
 
   if (pAlreadyVisited.has(lResolvedFileName)) {
     throw new Error(

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

@@ -1,5 +1,4 @@
 import { isDeepStrictEqual } from "node:util";
-import get from "lodash/get.js";
 import uniqBy from "lodash/uniqBy.js";
 import uniqWith from "lodash/uniqWith.js";
 
@@ -87,10 +86,8 @@ function mergeOptions(pOptionsExtended, pOptionsBase) {
  * @returns {string} - a string from the SeverityType value set
  */
 function mergeAllowedSeverities(pConfigExtended, pConfigBase) {
-  return get(
-    pConfigExtended,
-    "allowedSeverity",
-    get(pConfigBase, "allowedSeverity", "warn")
+  return (
+    pConfigExtended?.allowedSeverity ?? pConfigBase?.allowedSeverity ?? "warn"
   );
 }
 
@@ -108,18 +105,19 @@ function mergeAllowedSeverities(pConfigExtended, pConfigBase) {
  *
  * @returns {Object} - The merged rule set
  */
+// eslint-disable-next-line complexity
 export default (pConfigExtended, pConfigBase) => {
   const lForbidden = mergeRules(
-    get(pConfigExtended, "forbidden", []),
-    get(pConfigBase, "forbidden", [])
+    pConfigExtended?.forbidden ?? [],
+    pConfigBase?.forbidden ?? []
   );
   const lRequired = mergeRules(
-    get(pConfigExtended, "required", []),
-    get(pConfigBase, "required", [])
+    pConfigExtended?.required ?? [],
+    pConfigBase?.required ?? []
   );
   const lAllowed = mergeAllowedRules(
-    get(pConfigExtended, "allowed", []),
-    get(pConfigBase, "allowed", [])
+    pConfigExtended?.allowed ?? [],
+    pConfigBase?.allowed ?? []
   );
 
   return {
@@ -132,8 +130,8 @@ export default (pConfigExtended, pConfigBase) => {
         }
       : {}),
     options: mergeOptions(
-      get(pConfigExtended, "options", {}),
-      get(pConfigBase, "options", {})
+      pConfigExtended?.options ?? {},
+      pConfigBase?.options ?? {}
     ),
   };
 };

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

@@ -1,4 +1,4 @@
-import path from "node:path";
+import { dirname, resolve } from "node:path";
 import tryImport from "semver-try-require";
 import meta from "../meta.js";
 
@@ -55,7 +55,7 @@ export default function extractTSConfig(pTSConfigFileName) {
     lReturnValue = typescript.parseJsonConfigFileContent(
       lConfig.config,
       typescript.sys,
-      path.dirname(path.resolve(pTSConfigFileName)),
+      dirname(resolve(pTSConfigFileName)),
       {},
       pTSConfigFileName
     );

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

@@ -46,14 +46,6 @@ function suggestModules(pSuggestionList, pWebpackConfigFilename) {
 
 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 = require("interpret");
   const rechoir = require("rechoir");
 
@@ -71,6 +63,41 @@ function tryRegisterNonNative(pWebpackConfigFilename) {
   }
 }
 
+function isNativelySupported(pWebpackConfigFilename) {
+  const lNativelySupportedExtensions = [
+    ".js",
+    ".cjs",
+    ".mjs",
+    ".json",
+    ".node",
+  ];
+  const lWebpackConfigExtension = extname(pWebpackConfigFilename);
+  return lNativelySupportedExtensions.includes(lWebpackConfigExtension);
+}
+
+async function attemptImport(pAbsoluteWebpackConfigFileName) {
+  try {
+    if (isNativelySupported(pAbsoluteWebpackConfigFileName)) {
+      const lModule = await import(`file://${pAbsoluteWebpackConfigFileName}`);
+      return lModule.default;
+    } else {
+      tryRegisterNonNative(pAbsoluteWebpackConfigFileName);
+      /* we're using still using require instead of dynamic imports here because
+       * the modules webpack uses for non-native formats monkey-patch on the commonjs
+       * module system. If we'd use a dynamic import, these monkey-patches wouldn't
+       * be used.
+       */
+      /* eslint node/global-require:0, security/detect-non-literal-require:0, import/no-dynamic-require:0 */
+      return require(pAbsoluteWebpackConfigFileName);
+    }
+  } catch (pError) {
+    throw new Error(
+      `The webpack config '${pAbsoluteWebpackConfigFileName}' seems to be not quite valid for use:` +
+        `\n\n          "${pError}"\n`
+    );
+  }
+}
+
 /**
  * Reads the file with name `pWebpackConfigFilename` and (applying the
  * environment `pEnvironment` and the arguments `pArguments` (which can
@@ -84,42 +111,21 @@ function tryRegisterNonNative(pWebpackConfigFilename) {
  * @throws {Error} when the webpack config isn't usable (e.g. because it
  *                 doesn't exist, or because it's invalid)
  */
-
-export default function extractWebpackResolveConfig(
+export default async 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 {
-    /* we're using still using require instead of dynamic imports here because
-     * the modules webpack uses for non-native formats monkey-patch on the commonjs
-     * module system. If we'd use a dynamic import, these monkey-patches wouldn't
-     * be used.
-     */
-    /* eslint node/global-require:0, security/detect-non-literal-require:0, import/no-dynamic-require:0 */
-    const lWebpackConfigModule = require(lWebpackConfigFilename);
-    const lWebpackConfig = pryConfigFromTheConfig(
-      lWebpackConfigModule,
-      pEnvironment,
-      pArguments
-    );
-
-    if (lWebpackConfig.resolve) {
-      lReturnValue = lWebpackConfig.resolve;
-    }
-  } catch (pError) {
-    throw new Error(
-      `The webpack config '${pWebpackConfigFilename}' seems to be not quite valid for use:` +
-        `\n\n          "${pError}"\n`
-    );
+  const lAbsoluteConfigFilename = makeAbsolute(pWebpackConfigFilename);
+  const lWebpackConfig = pryConfigFromTheConfig(
+    await attemptImport(lAbsoluteConfigFilename),
+    pEnvironment,
+    pArguments
+  );
+
+  if (lWebpackConfig.resolve) {
+    lReturnValue = lWebpackConfig.resolve;
   }
 
   return lReturnValue;

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

@@ -1,5 +1,5 @@
 /* eslint-disable security/detect-object-injection */
-import path from "node:path/posix";
+import { dirname } from "node:path/posix";
 import { calculateInstability, metricsAreCalculable } from "../module-utl.mjs";
 import detectCycles from "../circular.mjs";
 import IndexedModuleGraph from "../../../graph-utl/indexed-module-graph.mjs";
@@ -42,7 +42,7 @@ function upsertFolderAttributes(pAllMetrics, pModule, pDirname) {
 }
 
 function aggregateToFolder(pAllFolders, pModule) {
-  getParentFolders(path.dirname(pModule.source)).forEach((pParentDirectory) =>
+  getParentFolders(dirname(pModule.source)).forEach((pParentDirectory) =>
     upsertFolderAttributes(pAllFolders, pModule, pParentDirectory)
   );
   return pAllFolders;
@@ -56,9 +56,9 @@ function getFolderLevelCouplings(pCouplingArray) {
   return Array.from(
     new Set(
       pCouplingArray.map((pCoupling) =>
-        path.dirname(pCoupling.name) === "."
+        dirname(pCoupling.name) === "."
           ? pCoupling.name
-          : path.dirname(pCoupling.name)
+          : dirname(pCoupling.name)
       )
     )
   ).map((pCoupling) => ({ name: pCoupling }));

package/src/enrich/derive/folders/utl.mjs

@@ -1,5 +1,5 @@
 /* eslint-disable security/detect-object-injection */
-import path from "node:path/posix";
+import { sep } from "node:path/posix";
 
 export function findFolderByName(pAllFolders, pName) {
   return pAllFolders.find((pFolder) => pFolder.name === pName);
@@ -7,13 +7,13 @@ export function findFolderByName(pAllFolders, pName) {
 
 export function getAfferentCouplings(pModule, pDirname) {
   return pModule.dependents.filter(
-    (pDependent) => !pDependent.startsWith(pDirname.concat(path.sep))
+    (pDependent) => !pDependent.startsWith(pDirname.concat(sep))
   );
 }
 
 export function getEfferentCouplings(pModule, pDirname) {
   return pModule.dependencies.filter(
-    (pDependency) => !pDependency.resolved.startsWith(pDirname.concat(path.sep))
+    (pDependency) => !pDependency.resolved.startsWith(pDirname.concat(sep))
   );
 }
 

package/src/extract/gather-initial-sources.mjs

@@ -1,7 +1,6 @@
 import { readdirSync, statSync } from "node:fs";
 import { join } from "node:path";
 import { glob } from "glob";
-import get from "lodash/get.js";
 import { filenameMatchesPattern } from "../graph-utl/match-facade.mjs";
 import getExtension from "../utl/get-extension.mjs";
 import pathToPosix from "../utl/path-to-posix.mjs";
@@ -22,16 +21,16 @@ function fileIsScannable(pOptions, pPathToFile) {
 
 function shouldBeIncluded(pFullPathToFile, pOptions) {
   return (
-    !get(pOptions, "includeOnly.path") ||
+    !pOptions?.includeOnly?.path ||
     filenameMatchesPattern(pFullPathToFile, pOptions.includeOnly.path)
   );
 }
 
 function shouldNotBeExcluded(pFullPathToFile, pOptions) {
   return (
-    (!get(pOptions, "exclude.path") ||
+    (!pOptions?.exclude?.path ||
       !filenameMatchesPattern(pFullPathToFile, pOptions.exclude.path)) &&
-    (!get(pOptions, "doNotFollow.path") ||
+    (!pOptions?.doNotFollow?.path ||
       !filenameMatchesPattern(pFullPathToFile, pOptions.doNotFollow.path))
   );
 }

package/src/extract/get-dependencies.mjs

@@ -1,5 +1,4 @@
 import { join, extname, dirname } from "node:path";
-import get from "lodash/get.js";
 import uniqBy from "lodash/uniqBy.js";
 import { intersects } from "../utl/array-util.mjs";
 import resolve from "./resolve/index.mjs";
@@ -235,9 +234,9 @@ export default function getDependencies(
       .map(addResolutionAttributes(pCruiseOptions, pFileName, pResolveOptions))
       .filter(
         ({ resolved }) =>
-          (!get(pCruiseOptions, "exclude.path") ||
+          (!pCruiseOptions?.exclude?.path ||
             !matchesPattern(resolved, pCruiseOptions.exclude.path)) &&
-          (!get(pCruiseOptions, "includeOnly.path") ||
+          (!pCruiseOptions?.includeOnly?.path ||
             matchesPattern(resolved, pCruiseOptions.includeOnly.path))
       );
   } catch (pError) {

package/src/main/helpers.mjs

@@ -39,7 +39,6 @@ export function normalizeREProperties(
       set(
         lPropertyContainer,
         lProperty,
-
         normalizeToREAsString(get(lPropertyContainer, lProperty))
       );
     }

package/src/main/resolve-options/normalize.mjs

@@ -1,5 +1,4 @@
 import fs from "node:fs";
-import get from "lodash/get.js";
 import has from "lodash/has.js";
 import omit from "lodash/omit.js";
 import enhancedResolve from "enhanced-resolve";
@@ -110,11 +109,8 @@ async function compileResolveOptions(
     ...omit(pResolveOptionsFromDCConfig, "cachedInputFileSystem"),
     ...pResolveOptions,
     ...getNonOverridableResolveOptions(
-      get(
-        pResolveOptionsFromDCConfig,
-        "cachedInputFileSystem.cacheDuration",
+      pResolveOptionsFromDCConfig?.cachedInputFileSystem?.cacheDuration ??
         DEFAULT_CACHE_DURATION
-      )
     ),
   };
 }
@@ -125,12 +121,13 @@ async function compileResolveOptions(
  * @param {import("typescript").ParsedTsconfig} pTSConfig
  * @returns
  */
+// eslint-disable-next-line complexity
 export default async function normalizeResolveOptions(
   pResolveOptions,
   pOptions,
   pTSConfig
 ) {
-  const lRuleSet = get(pOptions, "ruleSet", {});
+  const lRuleSet = pOptions?.ruleSet ?? {};
   // eslint-disable-next-line no-return-await
   return await compileResolveOptions(
     {
@@ -140,20 +137,20 @@ export default async function normalizeResolveOptions(
         symlink === !preserveSymlinks - but using it that way
         breaks backwards compatibility
       */
-      symlinks: get(pOptions, "preserveSymlinks", null),
-      tsConfig: get(pOptions, "ruleSet.options.tsConfig.fileName", null),
+      symlinks: pOptions?.preserveSymlinks ?? null,
+      tsConfig: pOptions?.ruleSet?.options?.tsConfig?.fileName ?? null,
 
       /* squirrel the externalModuleResolutionStrategy and combinedDependencies
          thing into the resolve options
          - they're not for enhanced resolve, but they are for what we consider
          resolve options ...
        */
-      combinedDependencies: get(pOptions, "combinedDependencies", false),
+      combinedDependencies: pOptions?.combinedDependencies ?? false,
       resolveLicenses: ruleSetHasLicenseRule(lRuleSet),
       resolveDeprecations: ruleSetHasDeprecationRule(lRuleSet),
       ...(pResolveOptions || {}),
     },
     pTSConfig || {},
-    get(pOptions, "enhancedResolveOptions", {})
+    pOptions?.enhancedResolveOptions ?? {}
   );
 }

package/src/meta.js

@@ -1,7 +1,7 @@
 /* generated - don't edit */
 
 module.exports = {
-  version: "13.0.0-beta-1",
+  version: "13.0.0-beta-2",
   engines: {
     node: "^16.14||>=18",
   },

package/src/report/csv.mjs

@@ -1,3 +1,4 @@
+import { EOL } from "node:os";
 import dependencyToIncidenceTransformer from "./utl/dependency-to-incidence-transformer.mjs";
 
 function renderHeader(pModules) {
@@ -9,15 +10,15 @@ function mapIncidences(pIncidences) {
 }
 
 function renderBody(pModules) {
-  return pModules.reduce(
-    (pAll, pModule) =>
-      `${pAll}\n"${pModule.source}",${mapIncidences(pModule.incidences)},""`,
-    ""
-  );
+  return pModules
+    .map(
+      (pModule) => `"${pModule.source}",${mapIncidences(pModule.incidences)},""`
+    )
+    .join(EOL);
 }
 
 function report(pModules) {
-  return `"",${renderHeader(pModules)},""${renderBody(pModules)}\n`;
+  return `"",${renderHeader(pModules)},""${EOL}${renderBody(pModules)}${EOL}`;
 }
 
 /**

package/src/report/error.mjs

@@ -1,3 +1,4 @@
+import { EOL } from "node:os";
 import chalk from "chalk";
 import figures from "figures";
 import { findRuleByName } from "../graph-utl/rule-set.mjs";
@@ -17,9 +18,9 @@ const SEVERITY2CHALK = {
 const EXTRA_PATH_INFORMATION_INDENT = 6;
 
 function formatExtraPathInformation(pExtra) {
-  return "\n".concat(
+  return EOL.concat(
     wrapAndIndent(
-      pExtra.join(` ${figures.arrowRight} \n`),
+      pExtra.join(` ${figures.arrowRight} ${EOL}`),
       EXTRA_PATH_INFORMATION_INDENT
     )
   );
@@ -48,7 +49,7 @@ function formatReachabilityViolation(pViolation) {
 }
 
 function formatInstabilityViolation(pViolation) {
-  return `${formatDependencyViolation(pViolation)}\n${wrapAndIndent(
+  return `${formatDependencyViolation(pViolation)}${EOL}${wrapAndIndent(
     chalk.dim(
       `instability: ${formatPercentage(pViolation.metrics.from.instability)} ${
         figures.arrowRight
@@ -78,7 +79,7 @@ function formatViolation(pViolation) {
     }: ${lFormattedViolators}` +
     `${
       pViolation.comment
-        ? `\n${wrapAndIndent(chalk.dim(pViolation.comment))}\n`
+        ? `${EOL}${wrapAndIndent(chalk.dim(pViolation.comment))}${EOL}`
         : ""
     }`
   );
@@ -93,11 +94,11 @@ function sumMeta(pMeta) {
 }
 
 function formatSummary(pSummary) {
-  let lMessage = `\n${figures.cross} ${sumMeta(
+  let lMessage = `${EOL}${figures.cross} ${sumMeta(
     pSummary
   )} dependency violations (${formatMeta(pSummary)}). ${
     pSummary.totalCruised
-  } modules, ${pSummary.totalDependenciesCruised} dependencies cruised.\n`;
+  } modules, ${pSummary.totalDependenciesCruised} dependencies cruised.${EOL}`;
 
   return pSummary.error > 0 ? chalk.red(lMessage) : lMessage;
 }
@@ -114,7 +115,7 @@ function addExplanation(pRuleSet, pLong) {
 function formatIgnoreWarning(pNumberOfIgnoredViolations) {
   if (pNumberOfIgnoredViolations > 0) {
     return chalk.yellow(
-      `${figures.warning} ${pNumberOfIgnoredViolations} known violations ignored. Run with --no-ignore-known to see them.\n`
+      `${figures.warning} ${pNumberOfIgnoredViolations} known violations ignored. Run with --no-ignore-known to see them.${EOL}`
     );
   }
   return "";
@@ -126,22 +127,24 @@ function report(pResults, pLong) {
   );
 
   if (lNonIgnorableViolations.length === 0) {
-    return `\n${chalk.green(figures.tick)} no dependency violations found (${
+    return `${EOL}${chalk.green(
+      figures.tick
+    )} no dependency violations found (${
       pResults.summary.totalCruised
     } modules, ${
       pResults.summary.totalDependenciesCruised
-    } dependencies cruised)\n${formatIgnoreWarning(
+    } dependencies cruised)${EOL}${formatIgnoreWarning(
       pResults.summary.ignore
-    )}\n\n`;
+    )}${EOL}`;
   }
 
   return lNonIgnorableViolations
     .reverse()
     .map(addExplanation(pResults.summary.ruleSetUsed, pLong))
-    .reduce((pAll, pThis) => `${pAll}  ${formatViolation(pThis)}\n`, "\n")
+    .reduce((pAll, pThis) => `${pAll}  ${formatViolation(pThis)}${EOL}`, EOL)
     .concat(formatSummary(pResults.summary))
     .concat(formatIgnoreWarning(pResults.summary.ignore))
-    .concat(`\n`);
+    .concat(EOL);
 }
 
 /**

package/src/report/text.mjs

@@ -1,3 +1,4 @@
+import { EOL } from "node:os";
 import figures from "figures";
 import chalk from "chalk";
 
@@ -72,7 +73,7 @@ function report(pResults, pOptions) {
     getModulesInFocus(pResults.modules),
     lOptions.highlightFocused === true
   ).reduce(
-    (pAll, pDependency) => pAll.concat(stringify(pDependency)).concat("\n"),
+    (pAll, pDependency) => pAll.concat(stringify(pDependency)).concat(EOL),
     ""
   );
 }