dependency-cruiser 16.7.0-beta-1 → 16.7.0-beta-2

package/bin/depcruise-baseline.mjs

@@ -28,7 +28,7 @@ try {
     .arguments("<files-or-directories>")
     .parse(process.argv);
 
-  if (Boolean(program.args[0])) {
+  if (program.args[0]) {
     process.exitCode = await cli(program.args, {
       ...program.opts(),
       cache: false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-cruiser",
-  "version": "16.7.0-beta-1",
+  "version": "16.7.0-beta-2",
   "description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
   "keywords": [
     "static analysis",
@@ -159,19 +159,19 @@
     "safe-regex": "^2.1.1",
     "semver": "^7.6.3",
     "teamcity-service-messages": "^0.1.14",
-    "tsconfig-paths-webpack-plugin": "^4.1.0",
-    "watskeburt": "^4.1.0"
+    "tsconfig-paths-webpack-plugin": "^4.2.0",
+    "watskeburt": "^4.1.1"
   },
   "overrides": {
-    "semver": "^7.6.3"
+    "cross-spawn": ">=6.0.6"
   },
   "resolutions": {
-    "semver": "^7.6.3"
+    "cross-spawn": ">=6.0.6"
   },
   "engines": {
     "node": "^18.17||>=20"
   },
   "scripts": {
-    "test": "echo for test, build and static analysis scripts: see the github repository"
+    "test": "echo see github for test, build and analysis scripts"
   }
 }

package/src/cli/index.mjs

@@ -13,6 +13,7 @@ import { set } from "#utl/object-util.mjs";
 import cruise from "#main/cruise.mjs";
 import { INFO, bus } from "#utl/bus.mjs";
 
+// eslint-disable-next-line complexity
 async function extractResolveOptions(pCruiseOptions) {
   let lResolveOptions = {};
   const lWebPackConfigFileName =
@@ -78,6 +79,7 @@ async function extractBabelConfigOptions(pCruiseOptions) {
   return lReturnValue;
 }
 
+// eslint-disable-next-line complexity
 function setUpListener(pCruiseOptions) {
   const lString2Listener = new Map([
     ["cli-feedback", setUpCliFeedbackListener],
@@ -90,7 +92,7 @@ function setUpListener(pCruiseOptions) {
 
   const lListenerFunction = lString2Listener.get(lListenerID);
   /* c8 ignore next 6 */
-  if (Boolean(lListenerFunction)) {
+  if (lListenerFunction) {
     lListenerFunction(
       bus,
       pCruiseOptions?.ruleSet?.options?.progress?.maximumLevel ?? INFO,
@@ -141,7 +143,7 @@ async function runCruise(pFileDirectoryArray, pCruiseOptions) {
  * @param {{stdout: NodeJS.WritableStream, stderr: NodeJS.WritableStream}=} pStreams
  * @returns {number}
  */
-// eslint-disable-next-line complexity
+
 export default async function executeCli(
   pFileDirectoryArray,
   pCruiseOptions,

package/src/cli/listeners/performance-log/format-helpers.mjs

@@ -65,7 +65,7 @@ function formatMessage(pMessage, pLevel) {
 export function formatTime(
   pNumber,
   pConversionMultiplier = MS_PER_SECOND,
-  pLevel,
+  pLevel = INFO,
 ) {
   return formatMessage(
     gTimeFormat(pConversionMultiplier * pNumber)

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

@@ -17,6 +17,7 @@ function getOptionValue(pDefault) {
   return (pValue) => (typeof pValue === "string" ? pValue : pDefault);
 }
 
+// eslint-disable-next-line complexity
 function normalizeConfigFileName(pCliOptions, pConfigWrapperName, pDefault) {
   let lOptions = structuredClone(pCliOptions);
 

package/src/cli/utl/io.mjs

@@ -28,10 +28,9 @@ function writeToFile(pOutputTo, pDependencyString) {
  */
 function writeToStdOut(pString, pBufferSize = PIPE_BUFFER_SIZE) {
   const lNumberOfChunks = Math.ceil(pString.length / pBufferSize);
-  let lIndex = 0;
 
   /* eslint no-plusplus: 0 */
-  for (lIndex = 0; lIndex < lNumberOfChunks; lIndex++) {
+  for (let lIndex = 0; lIndex < lNumberOfChunks; lIndex++) {
     const lChunkStart = lIndex * pBufferSize;
     process.stdout.write(
       pString.substring(lChunkStart, lChunkStart + pBufferSize),

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

@@ -68,7 +68,6 @@ async function getConfig(pBabelConfigFileName) {
       `${`The babel config '${pBabelConfigFileName}' is in a format ('${lExtension}')\n`}         dependency-cruiser doesn't support yet.\n`,
     );
   }
-  // eslint-disable-next-line no-return-await
   return await lExtensionToParseFunction.get(lExtension)(pBabelConfigFileName);
 }
 

package/src/config-utl/extract-known-violations.mjs

@@ -21,11 +21,11 @@ import makeAbsolute from "./make-absolute.mjs";
  */
 function makeForwardCompatible(pKnownViolation) {
   let lReturnValue = pKnownViolation;
-  if (Boolean(pKnownViolation.cycle)) {
+  if (pKnownViolation.cycle) {
     lReturnValue = {
       ...pKnownViolation,
       cycle: pKnownViolation.cycle.map((pModule) => {
-        if (Boolean(pModule.name)) {
+        if (pModule.name) {
           return pModule;
         }
         return {
@@ -35,11 +35,11 @@ function makeForwardCompatible(pKnownViolation) {
       }),
     };
   }
-  if (Boolean(pKnownViolation.via)) {
+  if (pKnownViolation.via) {
     lReturnValue = {
       ...pKnownViolation,
       via: pKnownViolation.via.map((pModule) => {
-        if (Boolean(pModule.name)) {
+        if (pModule.name) {
           return pModule;
         }
         return {

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

@@ -90,7 +90,7 @@ async function attemptImport(pAbsoluteWebpackConfigFileName) {
        * 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 */
+      /* eslint n/global-require:0, security/detect-non-literal-require:0, import/no-dynamic-require:0 */
       return require(pAbsoluteWebpackConfigFileName);
     }
   } catch (pError) {

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

@@ -116,7 +116,7 @@ export default function softenKnownViolations(
   pKnownViolations,
   pSoftenedSeverity = "ignore",
 ) {
-  if (Boolean(pKnownViolations)) {
+  if (pKnownViolations) {
     bus.info("analyzing: comparing against known errors");
     return pModules.map((pModule) =>
       softenKnownViolation(pModule, pKnownViolations, pSoftenedSeverity),

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

@@ -7,6 +7,7 @@ function removeNames(pRule) {
   return lReturnValue;
 }
 
+// eslint-disable-next-line complexity
 export default function addRuleSetUsed(pOptions) {
   const lForbidden = pOptions?.ruleSet?.forbidden;
   const lAllowed = pOptions?.ruleSet?.allowed;

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

@@ -12,7 +12,6 @@ function cutNonTransgressions(pModule) {
   };
 }
 
-// eslint-disable-next-line complexity
 function toDependencyViolationSummary(pRule, pModule, pDependency, pRuleSet) {
   let lReturnValue = {
     type: "dependency",

package/src/extract/extract-dependencies.mjs

@@ -78,10 +78,10 @@ function extractDependencies(pCruiseOptions, pFileName, pTranspileOptions) {
 }
 
 function matchesDoNotFollow({ resolved, dependencyTypes }, pDoNotFollow) {
-  const lMatchesPath = Boolean(pDoNotFollow.path)
+  const lMatchesPath = pDoNotFollow.path
     ? RegExp(pDoNotFollow.path, "g").test(resolved)
     : false;
-  const lMatchesDependencyTypes = Boolean(pDoNotFollow.dependencyTypes)
+  const lMatchesDependencyTypes = pDoNotFollow.dependencyTypes
     ? intersects(dependencyTypes, pDoNotFollow.dependencyTypes)
     : false;
 

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

@@ -1,5 +1,6 @@
 import { readdirSync, statSync } from "node:fs";
 import { join, normalize } from "node:path";
+// eslint-disable-next-line n/prefer-global/process
 import { platform } from "node:process";
 import picomatch from "picomatch";
 import { scannableExtensions } from "./transpile/meta.mjs";

package/src/extract/resolve/determine-dependency-types.mjs

@@ -59,7 +59,7 @@ function determineManifestDependencyTypes(
   /** @type {DependencyType[]} */
   let lReturnValue = ["npm-unknown"];
 
-  if (Boolean(pPackageDependencies)) {
+  if (pPackageDependencies) {
     lReturnValue = findModuleInPackageDependencies(
       pPackageDependencies,
       pModuleName,
@@ -89,7 +89,7 @@ function determineManifestDependencyTypes(
 function dependencyIsBundled(pModule, pPackageDeps) {
   let lReturnValue = false;
 
-  if (Boolean(pPackageDeps)) {
+  if (pPackageDeps) {
     const lBundledDependencies =
       pPackageDeps.bundledDependencies || pPackageDeps.bundleDependencies;
 

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

@@ -32,7 +32,7 @@ import { isScoped, isRelativeModuleName } from "./module-classifiers.mjs";
  * @return {string}         the module name root
  */
 export function getPackageRoot(pModule) {
-  if (!Boolean(pModule) || isRelativeModuleName(pModule)) {
+  if (!pModule || isRelativeModuleName(pModule)) {
     return pModule;
   }
 
@@ -78,7 +78,7 @@ function bareGetPackageJson(pModuleName, pFileDirectory, pResolveOptions) {
   try {
     const lPackageJsonFilename = resolve(
       join(getPackageRoot(pModuleName), "package.json"),
-      pFileDirectory ? pFileDirectory : process.cwd(),
+      pFileDirectory ?? process.cwd(),
       {
         ...pResolveOptions,
         // if a module has exports fields _and_ does not expose package.json
@@ -129,7 +129,7 @@ export function dependencyIsDeprecated(
     pResolveOptions,
   );
 
-  if (Boolean(lPackageJson)) {
+  if (lPackageJson) {
     lReturnValue =
       Object.hasOwn(lPackageJson, "deprecated") && lPackageJson.deprecated;
   }
@@ -154,7 +154,7 @@ export function getLicense(pModuleName, pFileDirectory, pResolveOptions) {
   );
 
   if (
-    Boolean(lPackageJson) &&
+    lPackageJson &&
     Object.hasOwn(lPackageJson, "license") &&
     typeof lPackageJson.license === "string"
   ) {

package/src/extract/resolve/module-classifiers.mjs

@@ -150,7 +150,7 @@ function isWorkspaceAliased(pModuleName, pResolvedModuleName, pManifest) {
   // an object. To prevent the code from borking we check whether it's an array
   // see https://github.com/sverweij/dependency-cruiser/issues/919
   const lWorkspaces = getWorkspacesArray(pManifest?.workspaces);
-  if (lWorkspaces.length >= 0) {
+  if (lWorkspaces.length > 0) {
     // workspaces are an array of globs that match the (sub) workspace
     // folder itself only.
     //
@@ -193,7 +193,7 @@ function isWorkspaceAliased(pModuleName, pResolvedModuleName, pManifest) {
     // of the workspace, not the path of the workspace itself. So if it's
     // in node_modules we need to check against the unresolved modulename.
     //
-    // Other then the detection for when symlinks are resolved to their realpath
+    // Other than the detection for when symlinks are resolved to their realpath
     // (the if above), this is a 'best effort' detection only for now; there's
     // guaranteed to be scenarios where this will fail. How often is the
     // --preserve-symlinks flag used in practice, though?

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

@@ -5,7 +5,7 @@ export function addLicenseAttribute(
   pModuleName,
   pResolvedModuleName,
   { baseDirectory, fileDirectory },
-  pResolveOptions
+  pResolveOptions,
 ) {
   let lReturnValue = {};
   if (
@@ -13,12 +13,12 @@ export function addLicenseAttribute(
     isExternalModule(
       pResolvedModuleName,
       pResolveOptions.modules,
-      baseDirectory
+      baseDirectory,
     )
   ) {
     const lLicense = getLicense(pModuleName, fileDirectory, pResolveOptions);
 
-    if (Boolean(lLicense)) {
+    if (lLicense) {
       lReturnValue.license = lLicense;
     }
   }

package/src/extract/swc/dependency-visitor.mjs

@@ -41,7 +41,7 @@ function argumentsAreUsable(pArguments) {
     ["StringLiteral", "TemplateLiteral"].includes(
       pArguments[0].expression.type,
     ) &&
-    (!(pArguments[0].expression.type === "TemplateLiteral") ||
+    (pArguments[0].expression.type !== "TemplateLiteral" ||
       isPlaceholderlessTemplateLiteral(pArguments[0]))
   );
 }
@@ -250,11 +250,7 @@ export default Visitor
         // as visitors for some shapes of type annotations aren't completely
         // implemented yet (1.2.51) pNode can come in as null (also see
         // comments in accompanying unit test)
-        if (
-          Boolean(pNode) &&
-          Boolean(pNode.typeAnnotation) &&
-          Boolean(pNode.typeAnnotation.argument)
-        )
+        if (pNode && pNode.typeAnnotation && pNode.typeAnnotation.argument)
           this.lResult.push({
             module: pNode.typeAnnotation.argument.value,
             moduleSystem: "es6",

package/src/extract/tsc/extract-typescript-deps.mjs

@@ -60,7 +60,7 @@ function extractImports(pAST) {
     .filter(
       (pStatement) =>
         typescript.SyntaxKind[pStatement.kind] === "ImportDeclaration" &&
-        Boolean(pStatement.moduleSpecifier),
+        pStatement.moduleSpecifier,
     )
     .map((pStatement) => ({
       module: pStatement.moduleSpecifier.text,
@@ -84,7 +84,7 @@ function extractExports(pAST) {
     .filter(
       (pStatement) =>
         typescript.SyntaxKind[pStatement.kind] === "ExportDeclaration" &&
-        Boolean(pStatement.moduleSpecifier),
+        pStatement.moduleSpecifier,
     )
     .map((pStatement) => ({
       module: pStatement.moduleSpecifier.text,
@@ -265,7 +265,7 @@ function extractJSDocImportTags(pJSDocTags) {
         pTag.tagName.escapedText === "import" &&
         pTag.moduleSpecifier?.kind &&
         typescript.SyntaxKind[pTag.moduleSpecifier.kind] === "StringLiteral" &&
-        Boolean(pTag.moduleSpecifier.text),
+        pTag.moduleSpecifier.text,
     )
     .map((pTag) => ({
       module: pTag.moduleSpecifier.text,
@@ -275,10 +275,43 @@ function extractJSDocImportTags(pJSDocTags) {
     }));
 }
 
+function extractJSDocBracketImports(pJSDocTags) {
+  return pJSDocTags
+    .filter(
+      (pTag) =>
+        pTag.tagName.escapedText !== "import" &&
+        /* c8 ignore start */
+        typescript.SyntaxKind[pTag.typeExpression?.kind ?? -1] ===
+          "FirstJSDocNode" &&
+        typescript.SyntaxKind[pTag.typeExpression.type?.kind ?? -1] ===
+          "LastTypeNode" &&
+        typescript.SyntaxKind[pTag.typeExpression.type.argument?.kind ?? -1] ===
+          "LiteralType" &&
+        typescript.SyntaxKind[
+          pTag.typeExpression.type.argument?.literal?.kind ?? -1
+        ] === "StringLiteral" &&
+        /* c8 ignore stop*/
+        pTag.typeExpression.type.argument.literal.text,
+    )
+    .map((pTag) => ({
+      module: pTag.typeExpression.type.argument.literal.text,
+      moduleSystem: "es6",
+      exoticallyRequired: false,
+      dependencyTypes: ["type-only", "import", "jsdoc", "jsdoc-bracket-import"],
+    }));
+}
+
 function extractJSDocImports(pJSDocNodes) {
-  return pJSDocNodes
-    .filter((pJSDocLine) => Boolean(pJSDocLine.tags))
-    .flatMap((pJSDocLine) => extractJSDocImportTags(pJSDocLine.tags));
+  const lJSDocNodesWithTags = pJSDocNodes.filter(
+    (pJSDocLine) => pJSDocLine.tags,
+  );
+  const lJSDocImportTags = lJSDocNodesWithTags.flatMap((pJSDocLine) =>
+    extractJSDocImportTags(pJSDocLine.tags),
+  );
+  const lJSDocBracketImports = lJSDocNodesWithTags.flatMap((pJSDocLine) =>
+    extractJSDocBracketImports(pJSDocLine.tags),
+  );
+  return lJSDocImportTags.concat(lJSDocBracketImports);
 }
 
 /**
@@ -346,7 +379,7 @@ function walk(pResult, pExoticRequireStrings, pDetectJSDocImports) {
     // TODO: all the kinds of tags that can have import statements as type declarations
     //      (e.g. @type, @param, @returns, @typedef, @property, @prop, @arg, ...)
     // https://www.typescriptlang.org/docs/handbook/jsdoc-supported-types.html
-    if (pDetectJSDocImports && Boolean(pASTNode.jsDoc)) {
+    if (pDetectJSDocImports && pASTNode.jsDoc) {
       const lJSDocImports = extractJSDocImports(pASTNode.jsDoc);
 
       // pResult = pResult.concat(lJSDocImports) looks like the more obvious
@@ -394,7 +427,7 @@ export default function extractTypeScriptDependencies(
   pDetectJSDocImports,
 ) {
   // console.dir(pTypeScriptAST, { depth: 100 });
-  return Boolean(typescript)
+  return typescript
     ? extractImports(pTypeScriptAST)
         .concat(extractExports(pTypeScriptAST))
         .concat(extractImportEquals(pTypeScriptAST))

package/src/graph-utl/filter-bank.mjs

@@ -74,7 +74,6 @@ function tagHighlight(pModules, pHighlightFilter) {
   }));
 }
 
-// eslint-disable-next-line complexity
 export function applyFilters(pModules, pFilters) {
   if (pFilters) {
     let lReturnValue = structuredClone(pModules);

package/src/main/cruise.mjs

@@ -1,4 +1,4 @@
-/* eslint-disable no-return-await, no-magic-numbers */
+/* eslint-disable no-magic-numbers */
 import { assertCruiseOptionsValid } from "./options/assert-validity.mjs";
 import { normalizeCruiseOptions } from "./options/normalize.mjs";
 import reportWrap from "./report-wrap.mjs";
@@ -65,7 +65,7 @@ export default async function cruise(
     import("#enrich/index.mjs"),
   ]);
 
-  if (Boolean(lCruiseOptions.ruleSet)) {
+  if (lCruiseOptions.ruleSet) {
     bus.summary("parsing rule set", c(4));
     lCruiseOptions.ruleSet = normalizeRuleSet(
       assertRuleSetValid(lCruiseOptions.ruleSet),

package/src/main/format.mjs

@@ -21,6 +21,5 @@ export default async function format(pResult, pFormatOptions = {}) {
 
   validateResultAgainstSchema(pResult);
 
-  // eslint-disable-next-line no-return-await
   return await reportWrap(pResult, lFormatOptions);
 }

package/src/main/options/assert-validity.mjs

@@ -34,10 +34,10 @@ function deepMerge(pTarget, pSource) {
 
 function assertModuleSystemsValid(pModuleSystems) {
   if (
-    Boolean(pModuleSystems) &&
+    pModuleSystems &&
     Array.isArray(pModuleSystems) &&
     !pModuleSystems.every((pModuleSystem) =>
-      Boolean(pModuleSystem.match(MODULE_SYSTEM_LIST_RE)),
+      pModuleSystem.match(MODULE_SYSTEM_LIST_RE),
     )
   ) {
     throw new Error(
@@ -47,7 +47,7 @@ function assertModuleSystemsValid(pModuleSystems) {
 }
 
 function assertRegExpSafety(pPattern) {
-  if (Boolean(pPattern) && !safeRegex(pPattern)) {
+  if (pPattern && !safeRegex(pPattern)) {
     throw new Error(
       `The pattern '${pPattern}' will probably run very slowly - cowardly refusing to run.\n`,
     );
@@ -56,7 +56,7 @@ function assertRegExpSafety(pPattern) {
 
 function assertOutputTypeValid(pOutputType) {
   if (
-    Boolean(pOutputType) &&
+    pOutputType &&
     !getAvailableReporters().includes(pOutputType) &&
     !pOutputType.startsWith("plugin:")
   ) {
@@ -65,7 +65,7 @@ function assertOutputTypeValid(pOutputType) {
 }
 
 function assertMaxDepthValid(pDepth) {
-  if (Boolean(pDepth) && !pDepth.toString().match(VALID_DEPTH_RE)) {
+  if (pDepth && !pDepth.toString().match(VALID_DEPTH_RE)) {
     throw new Error(
       `'${pDepth}' is not a valid depth - use an integer between 0 and 99`,
     );
@@ -77,7 +77,7 @@ function assertFocusDepthValid(pFocusDepth) {
   const lMaxFocusDepth = 99;
 
   if (
-    Boolean(pFocusDepth) &&
+    pFocusDepth &&
     (Number.isNaN(lFocusDepth) ||
       lFocusDepth < 0 ||
       lFocusDepth > lMaxFocusDepth)
@@ -105,7 +105,7 @@ function assertPathsSafety(pFilterOption) {
 export function assertCruiseOptionsValid(pOptions) {
   let lReturnValue = {};
 
-  if (Boolean(pOptions)) {
+  if (pOptions) {
     // necessary because can slip through the cracks when passed as a cli parameter
     assertModuleSystemsValid(pOptions.moduleSystems);
 

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

@@ -146,7 +146,7 @@ export default async function normalizeResolveOptions(
   pTSConfig,
 ) {
   const lRuleSet = pOptions?.ruleSet ?? {};
-  // eslint-disable-next-line no-return-await
+
   return await compileResolveOptions(
     {
       // EnhancedResolve's symlinks:

package/src/main/rule-set/normalize.mjs

@@ -16,7 +16,7 @@ const DEFAULT_RULE = "unnamed";
 const DEFAULT_SCOPE = "module";
 
 function normalizeSeverity(pSeverity) {
-  const lSeverity = pSeverity ? pSeverity : DEFAULT_SEVERITY;
+  const lSeverity = pSeverity ?? DEFAULT_SEVERITY;
 
   return VALID_SEVERITIES.test(lSeverity) ? lSeverity : DEFAULT_SEVERITY;
 }
@@ -26,7 +26,7 @@ function normalizeSeverity(pSeverity) {
  * @returns {string}
  */
 function normalizeName(pRuleName) {
-  return pRuleName ? pRuleName : DEFAULT_RULE;
+  return pRuleName ?? DEFAULT_RULE;
 }
 
 /**
@@ -34,7 +34,7 @@ function normalizeName(pRuleName) {
  * @returns {RuleScopeType}
  */
 function normalizeScope(pScope) {
-  return pScope ? pScope : DEFAULT_SCOPE;
+  return pScope ?? DEFAULT_SCOPE;
 }
 
 /**

package/src/meta.cjs

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

package/src/report/anon/anonymize-path.mjs

@@ -2,7 +2,7 @@ import { anonymizePathElement } from "./anonymize-path-element.mjs";
 
 export const WHITELIST_RE =
   // eslint-disable-next-line security/detect-unsafe-regex
-  /^(?:|[.]+|~|bin|apps?|cli|src|libs?|configs?|components?|fixtures?|helpers?|i18n|index\.(?:jsx?|[mc]js|d\.ts|tsx?|vue|coffee|ls)|_?_?mocks?_?_?|node_modules|packages?|package\.json|scripts?|services?|sources?|specs?|_?_?tests?_?_?|types?|uti?ls?|tools)$/;
+  /^(?:[.]+|~|bin|apps?|cli|src|libs?|configs?|components?|fixtures?|helpers?|i18n|index\.(?:jsx?|[mc]js|d\.ts|tsx?|vue|coffee|ls)|_?_?mocks?_?_?|node_modules|packages?|package\.json|scripts?|services?|sources?|specs?|_?_?tests?_?_?|types?|uti?ls?|tools)$/;
 
 /**
  * Kind of smartly anonymizes paths, by

package/src/report/d2.mjs

@@ -51,7 +51,7 @@ function getVertexName(pSource) {
  * @param {import("../../types/cruise-result").IOptions} pOptions
  * @returns {string}
  */
-// eslint-disable-next-line complexity
+
 function getModuleAttributes(pModule, pOptions) {
   let lReturnValue = "class: module";
   if (pModule.consolidated) {
@@ -90,7 +90,7 @@ function getModuleAttributes(pModule, pOptions) {
  * @param {import("../../types/cruise-result").IDependency} pDependency
  * @returns {string}
  */
-// eslint-disable-next-line complexity
+
 function getDependencyAttributes(pDependency) {
   let lThing = "";
   if (pDependency.valid === false) {

package/src/report/dot/index.mjs

@@ -25,15 +25,15 @@ const GRANULARITY2REPORTER_OPTIONS = new Map([
 ]);
 
 function buildGraphAttributes(pGraph) {
-  return Boolean(pGraph) ? `    ${attributizeObject(pGraph || {})}` : "";
+  return pGraph ? `    ${attributizeObject(pGraph || {})}` : "";
 }
 
 function buildNodeAttributes(pNode) {
-  return Boolean(pNode) ? `    node [${attributizeObject(pNode || {})}]` : "";
+  return pNode ? `    node [${attributizeObject(pNode || {})}]` : "";
 }
 
 function buildEdgeAttributes(pEdge) {
-  return Boolean(pEdge) ? `    edge [${attributizeObject(pEdge || {})}]` : "";
+  return pEdge ? `    edge [${attributizeObject(pEdge || {})}]` : "";
 }
 
 function buildGeneralAttributes(pTheme) {

package/src/report/dot-webpage/svg-in-html-snippets/script.cjs

@@ -18,7 +18,7 @@ function getHoverHandler(pTitle2ElementMap) {
     var closestTitleText = getTitleText(closestNodeOrEdge);
 
     if (
-      !(currentHighlightedTitle === closestTitleText) &&
+      currentHighlightedTitle !== closestTitleText &&
       gMode.get() === gMode.HOVER
     ) {
       resetNodesAndEdges();
@@ -40,12 +40,12 @@ function getSelectHandler(pTitle2ElementMap) {
     var closestNodeOrEdge = pMouseEvent.target.closest(".edge, .node");
     var closestTitleText = getTitleText(closestNodeOrEdge);
 
-    if (!!closestNodeOrEdge) {
+    if (closestNodeOrEdge) {
       gMode.setToSelect();
     } else {
       gMode.setToHover();
     }
-    if (!(currentHighlightedTitle === closestTitleText)) {
+    if (currentHighlightedTitle !== closestTitleText) {
       resetNodesAndEdges();
       addHighlight(closestNodeOrEdge);
       pTitle2ElementMap.get(closestTitleText).forEach(addHighlight);
@@ -216,7 +216,7 @@ function addHighlight(pGroup) {
 var gHints = {
   HIDDEN: 1,
   SHOWN: 2,
-  state: this.HIDDEN,
+  state: 1, // === HIDDEN
   show: function () {
     document.getElementById("hints").removeAttribute("style");
     gHints.state = gHints.SHOWN;

package/src/report/markdown.mjs

@@ -105,13 +105,50 @@ function formatViolations(pViolations, pIncludeIgnoredInDetails) {
     }, lTableHead);
 }
 
+function details(pResults, pOptions) {
+  let lReturnValue = "";
+  if (pResults.summary.violations.length > 0) {
+    if (pOptions.showDetailsHeader) {
+      lReturnValue += `${pOptions.detailsHeader}\n\n`;
+    }
+    if (pOptions.collapseDetails) {
+      lReturnValue += `<details><summary>${pOptions.collapsedMessage}</summary>\n\n`;
+    }
+    lReturnValue += `${formatViolations(
+      pResults.summary.violations,
+      pOptions.includeIgnoredInDetails,
+    )}\n\n`;
+    if (pOptions.collapseDetails) {
+      lReturnValue += "</details>\n\n";
+    }
+  } else {
+    lReturnValue += `${pOptions.noViolationsMessage}\n\n`;
+  }
+  return lReturnValue;
+}
+
+function summary(pResults, pOptions) {
+  let lReturnValue = "";
+
+  if (pOptions.showSummaryHeader) {
+    lReturnValue += `${pOptions.summaryHeader}\n\n`;
+  }
+  lReturnValue += `${formatStatsSummary(pResults.summary)}\n\n`;
+
+  if (pResults.summary.violations.length > 0 && pOptions.showRulesSummary) {
+    lReturnValue += `${formatRulesSummary(
+      pResults,
+      pOptions.includeIgnoredInSummary,
+    )}\n\n`;
+  }
+  return lReturnValue;
+}
+
 /**
- *
  * @param {import("../../types/cruise-result.mjs").ICruiseResult} pResults
  * @param {import("../../types/reporter-options.js").IMarkdownReporterOptions} pOptions
  * @returns {string}
  */
-// eslint-disable-next-line complexity, max-statements
 function report(pResults, pOptions) {
   const lOptions = { ...REPORT_DEFAULTS, ...(pOptions || {}) };
   let lReturnValue = "";
@@ -121,37 +158,11 @@ function report(pResults, pOptions) {
   }
 
   if (lOptions.showSummary) {
-    if (lOptions.showSummaryHeader) {
-      lReturnValue += `${lOptions.summaryHeader}\n\n`;
-    }
-    lReturnValue += `${formatStatsSummary(pResults.summary)}\n\n`;
-
-    if (pResults.summary.violations.length > 0 && lOptions.showRulesSummary) {
-      lReturnValue += `${formatRulesSummary(
-        pResults,
-        lOptions.includeIgnoredInSummary,
-      )}\n\n`;
-    }
+    lReturnValue += summary(pResults, lOptions);
   }
 
   if (lOptions.showDetails) {
-    if (pResults.summary.violations.length > 0) {
-      if (lOptions.showDetailsHeader) {
-        lReturnValue += `${lOptions.detailsHeader}\n\n`;
-      }
-      if (lOptions.collapseDetails) {
-        lReturnValue += `<details><summary>${lOptions.collapsedMessage}</summary>\n\n`;
-      }
-      lReturnValue += `${formatViolations(
-        pResults.summary.violations,
-        lOptions.includeIgnoredInDetails,
-      )}\n\n`;
-      if (lOptions.collapseDetails) {
-        lReturnValue += "</details>\n\n";
-      }
-    } else {
-      lReturnValue += `${lOptions.noViolationsMessage}\n\n`;
-    }
+    lReturnValue += details(pResults, lOptions);
   }
 
   if (lOptions.showFooter) {

package/src/report/mermaid.mjs

@@ -37,7 +37,7 @@ function convertEdgeSources(pCruiseResult, pNamesHashMap) {
   });
 }
 
-const indent = (pDepth = 0, pMinify) => (pMinify ? "" : "  ".repeat(pDepth));
+const indent = (pDepth, pMinify) => (pMinify ? "" : "  ".repeat(pDepth));
 
 const renderSubgraph = (pNode, pText, pChildren, pIndent) =>
   `${pIndent}subgraph ${renderNode(pNode, pText)}
@@ -96,7 +96,7 @@ function focusHighlights(pModules, pNamesHashMap) {
       (pModule) =>
         pModule.matchesFocus ||
         pModule.matchesReaches ||
-        pModule.matchesHighlight
+        pModule.matchesHighlight,
     )
     .reduce((pAll, pModule) => {
       const lSource = pNamesHashMap.get(pModule.source);

package/src/report/teamcity.mjs

@@ -144,6 +144,7 @@ function reportViolations(pViolations, pIgnoredCount) {
  * @param {import("../../types/dependency-cruiser.js").ICruiseResult} pResults
  * @returns {import("../../types/dependency-cruiser.js").IReporterOutput}
  */
+// eslint-disable-next-line complexity
 export default function teamcity(pResults) {
   // this is the documented way to get tsm to emit strings
   // Alternatively we could've used the 'low level API', which

package/src/utl/get-extension.mjs

@@ -1,7 +1,7 @@
 // @ts-check
 import { extname } from "node:path";
 
-const EXTENSION_RE = /(?<extension>(?:(?:\.d\.(?:c|m)?ts)|\.coffee\.md)$)/;
+const EXTENSION_RE = /(?<extension>(?:(?:\.d\.(?:[cm])?ts)|\.coffee\.md)$)/;
 
 /**
  * Returns the extension of the given file name path.

package/src/utl/try-import.mjs

@@ -45,7 +45,6 @@ function getVersion(pModuleName) {
  * @returns {Promise<NodeModule | false>} - The imported module or false if the import fails or the version does not satisfy the provided semantic version.
  */
 
-// eslint-disable-next-line complexity
 export default async function tryImport(pModuleName, pSemanticVersion) {
   try {
     if (pSemanticVersion) {

package/src/utl/try-require.cjs

@@ -12,7 +12,7 @@ function getVersion(pModuleName) {
   // @ts-expect-error TS2345 extractRootModuleName can return either a string or
   // undefined. If undefined this function will throw. Which is _fine_, even
   // _expected_ in the context it's currently used
-  // eslint-disable-next-line import/no-dynamic-require, node/global-require, security/detect-non-literal-require
+  // eslint-disable-next-line import/no-dynamic-require, n/global-require, security/detect-non-literal-require
   return require(join(extractRootModuleName(pModuleName), "package.json"))
     .version;
 }
@@ -41,7 +41,7 @@ function tryRequire(pModuleName, pSemanticVersion) {
         return false;
       }
     }
-    // eslint-disable-next-line import/no-dynamic-require, node/global-require, security/detect-non-literal-require
+    // eslint-disable-next-line import/no-dynamic-require, n/global-require, security/detect-non-literal-require
     return require(pModuleName);
   } catch (pError) {
     return false;

package/src/validate/match-module-rule-helpers.mjs

@@ -115,7 +115,7 @@ function dependentsCountsMatch(pRule, pDependents) {
  * @param {IModule} pModule
  * @returns {boolean}
  */
-// eslint-disable-next-line complexity
+
 export function matchesDependentsRule(pRule, pModule) {
   if (
     (Object.hasOwn(pModule, "dependents") &&

package/src/validate/matchers.mjs

@@ -85,7 +85,7 @@ export function matchToModulePath(pRule, pModule, pGroups) {
 
 function _toPathNot(pRule, pString, pGroups = []) {
   return (
-    !Boolean(pRule.to.pathNot) ||
+    !pRule.to.pathNot ||
     !pString.match(replaceGroupPlaceholders(pRule.to.pathNot, pGroups))
   );
 }
@@ -112,7 +112,6 @@ export function matchesToDependencyTypesNot(pRule, pDependency) {
   );
 }
 
-// eslint-disable-next-line complexity
 export function matchesToVia(pRule, pDependency, pGroups) {
   let lReturnValue = true;
   if (pRule.to.via && pDependency.cycle) {
@@ -144,7 +143,6 @@ export function matchesToVia(pRule, pDependency, pGroups) {
   return lReturnValue;
 }
 
-// eslint-disable-next-line complexity
 export function matchesToViaOnly(pRule, pDependency, pGroups) {
   let lReturnValue = true;
   if (pRule.to.viaOnly && pDependency.cycle) {