dependency-cruiser 16.6.0 → 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.6.0",
+  "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/cache/cache.mjs

@@ -13,10 +13,9 @@ import { scannableExtensions } from "#extract/transpile/meta.mjs";
 import { bus } from "#utl/bus.mjs";
 
 /**
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionData} IRevisionData
- * @typedef {import("../../types/strict-options.mjs").IStrictCruiseOptions} IStrictCruiseOptions
- * @typedef {import("../../types/dependency-cruiser.mjs").ICruiseResult} ICruiseResult
- * @typedef {import("../../types/cache-options.mjs").cacheStrategyType} cacheStrategyType
+ * @import { IRevisionData, ICruiseResult } from "../../types/dependency-cruiser.mjs";
+ * @import { IStrictCruiseOptions } from "../../types/strict-options.mjs";
+ * @import { cacheStrategyType } from "../../types/cache-options.mjs";
  */
 
 const CACHE_FILE_NAME = "cache.json";

package/src/cache/content-strategy.mjs

@@ -10,11 +10,9 @@ import {
 } from "./helpers.mjs";
 
 /**
- * @typedef {import("../../types/dependency-cruiser.mjs").IModule} IModule
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionChange} IRevisionChange
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionData} IRevisionData
- * @typedef {import("../../types/dependency-cruiser.mjs").ICruiseResult} ICruiseResult
- * @typedef {import("../../types/strict-options.mjs").IStrictCruiseOptions} IStrictCruiseOptions
+ * @import { IModule, IRevisionChange, IRevisionData, ICruiseResult } from "../../types/dependency-cruiser.mjs"
+ * @import { IStrictCruiseOptions } from "../../types/strict-options.mjs"
+ * @import { changeType, getSHA } from "watskeburt"
  */
 
 /**
@@ -56,10 +54,10 @@ export default class ContentStrategy {
    * @param {IStrictCruiseOptions} pCruiseOptions
    * @param {Object} pOptions
    * @param {Set<string>} pOptions.extensions
-   * @param {Set<import("watskeburt").changeType>=} pOptions.interestingChangeTypes?
+   * @param {Set<changeType>=} pOptions.interestingChangeTypes?
    * @param {string=} pOptions.baseDir
    * @param {typeof findContentChanges=} pOptions.diffListFn
-   * @param {typeof import('watskeburt').getSHA=} pOptions.checksumFn
+   * @param {typeof getSHA=} pOptions.checksumFn
    * @returns {IRevisionData}
    */
   getRevisionData(pDirectory, pCachedCruiseResult, pCruiseOptions, pOptions) {

package/src/cache/find-content-changes.mjs

@@ -11,10 +11,16 @@ import {
 import { bus } from "#utl/bus.mjs";
 import findAllFiles from "#utl/find-all-files.mjs";
 
+/**
+ * @import { IModule, IRevisionChange } from "../../types/dependency-cruiser.mjs"
+ * @import { ICruiseResult } from "../../types/cruise-result.mjs"
+ * @import { IStrictExcludeType, IStrictIncludeOnlyType } from "../../types/strict-filter-types.mjs"
+ */
+
 /**
  * @param {Set<string>} pFileSet
  * @param {typeof getFileHashSync} pFileHashFunction
- * @returns {(pModule:import("../..").IModule) => import('../..').IRevisionChange}
+ * @returns {(pModule:IModule) => IRevisionChange}
  */
 function diffCachedModuleAgainstFileSet(
   pFileSet,
@@ -63,13 +69,13 @@ function diffCachedModuleAgainstFileSet(
   - there is a cache, but it doesn't contain checksums => same as before, except
     all files will be marked as 'modified'
  * @param {string} pDirectory
- * @param {import("../..").ICruiseResult} pCachedCruiseResult
+ * @param {ICruiseResult} pCachedCruiseResult
  * @param {Object} pOptions
  * @param {Set<string>} pOptions.extensions
  * @param {string} pOptions.baseDir
- * @param {import("../../types/strict-filter-types.mjs").IStrictExcludeType} pOptions.exclude
- * @param {import("../../types/strict-filter-types.mjs").IStrictIncludeOnlyType=} pOptions.includeOnly
- * @returns {import("../..").IRevisionChange[]}
+ * @param {IStrictExcludeType} pOptions.exclude
+ * @param {IStrictIncludeOnlyType=} pOptions.includeOnly
+ * @returns {IRevisionChange[]}
  */
 export default function findContentChanges(
   pDirectory,

package/src/cache/helpers.mjs

@@ -6,10 +6,10 @@ import memoize from "memoize";
 import { filenameMatchesPattern } from "#graph-utl/match-facade.mjs";
 
 /**
- * @typedef {import("../..").IModule} IModule
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionChange} IRevisionChange
- * @typedef {import("../../types/filter-types.mjs").IExcludeType} IExcludeType
- * @typedef {import("../../types/strict-filter-types.mjs").IStrictIncludeOnlyType} IStrictIncludeOnlyType
+ * @import { IModule, IRevisionChange, } from "../../types/dependency-cruiser.mjs"
+ * @import { IExcludeType } from "../../types/filter-types.mjs"
+ * @import { IStrictIncludeOnlyType } from "../../types/strict-filter-types.mjs"
+ * @import { changeType, IChange } from "watskeburt"
  */
 
 /**
@@ -35,7 +35,7 @@ function _getFileHashSync(pFileName) {
 export const getFileHashSync = memoize(_getFileHashSync);
 
 /**
- * @param {import("watskeburt").IChange} pChange
+ * @param {IChange} pChange
  * @return {IRevisionChange}
  */
 export function addCheckSumToChangeSync(pChange) {
@@ -80,7 +80,7 @@ export function hasInterestingExtension(pExtensions) {
 
 /**
  * @param {Set<string>} pExtensions
- * @returns {(pChange: import("watskeburt").IChange) => boolean}
+ * @returns {(pChange: IChange) => boolean}
  */
 export function changeHasInterestingExtension(pExtensions) {
   return (pChange) => {
@@ -106,8 +106,8 @@ const DEFAULT_INTERESTING_CHANGE_TYPES = new Set([
 ]);
 
 /**
- * @param {Set<import("watskeburt").changeType>=} pInterestingChangeTypes
- * @returns {(pChange: import("watskeburt").IChange) => boolean}
+ * @param {Set<changeType>=} pInterestingChangeTypes
+ * @returns {(pChange: IChange) => boolean}
  */
 export function isInterestingChangeType(pInterestingChangeTypes) {
   return (pChange) =>

package/src/cache/metadata-strategy.mjs

@@ -12,20 +12,18 @@ import {
 import { bus } from "#utl/bus.mjs";
 
 /**
- * @typedef {import("../../types/dependency-cruiser.mjs").IModule} IModule
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionChange} IRevisionChange
- * @typedef {import("../../types/dependency-cruiser.mjs").IRevisionData} IRevisionData
- * @typedef {import("../../types/dependency-cruiser.mjs").ICruiseResult} ICruiseResult
- * @typedef {import("../../types/strict-options.mjs").IStrictCruiseOptions} IStrictCruiseOptions
+ * @import {IModule, IRevisionChange, IRevisionData, ICruiseResult } from "../../types/dependency-cruiser.mjs"
+ * @import { IStrictCruiseOptions } from "../../types/strict-options.mjs"
+ * @import { changeType, IChange } from "watskeburt"
  */
 
 export default class MetaDataStrategy {
   /**
    * @param {string} _pDirectory
-   * @param {import("../../types/cruise-result.mjs").ICruiseResult} _pCachedCruiseResult
+   * @param {ICruiseResult} _pCachedCruiseResult
    * @param {Object} pOptions
    * @param {Set<string>} pOptions.extensions
-   * @param {Set<import("watskeburt").changeType>=} pOptions.interestingChangeTypes
+   * @param {Set<changeType>=} pOptions.interestingChangeTypes
    * @param {typeof getSHA=} pOptions.shaRetrievalFn
    * @param {typeof list=} pOptions.diffListFn
    * @param {typeof addCheckSumToChangeSync=} pOptions.checksumFn
@@ -47,7 +45,7 @@ export default class MetaDataStrategy {
       bus.debug("cache: - getting sha");
       const lSHA = await lOptions.shaRetrievalFn();
       bus.debug("cache: - getting diff");
-      const lDiff = /** @type {import("watskeburt").IChange[]} */ (
+      const lDiff = /** @type {IChange[]} */ (
         await lOptions.diffListFn({ oldRevision: lSHA })
       );
       const lChanges = lDiff

package/src/cache/options-compatible.mjs

@@ -2,7 +2,7 @@
 import { isDeepStrictEqual } from "node:util";
 
 /**
- * @typedef {import("../../types/strict-options.mjs").IStrictCruiseOptions} IStrictCruiseOptions
+ * @import { IStrictCruiseOptions } from "../../types/strict-options.mjs"
  */
 
 /*
@@ -105,6 +105,7 @@ export function optionsAreCompatible(pOldOptions, pNewOptions) {
     pOldOptions.preserveSymlinks === pNewOptions.preserveSymlinks &&
     pOldOptions.combinedDependencies === pNewOptions.combinedDependencies &&
     pOldOptions.experimentalStats === pNewOptions.experimentalStats &&
+    pOldOptions.detectJSDocImports === pNewOptions.detectJSDocImports &&
     metricsIsCompatible(pOldOptions.metrics, pNewOptions.metrics) &&
     // includeOnly suffers from a backwards compatibility disease
     includeOnlyIsCompatible(pOldOptions.includeOnly, pNewOptions.includeOnly) &&

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/init-config/build-config.mjs

@@ -2,6 +2,10 @@
 import { folderNameArrayToRE } from "./utl.mjs";
 import configTemplate from "./config-template.mjs";
 
+/**
+ * @import { IInitConfig } from "./types.js";
+ */
+
 /**
  * @param {string} pString
  * @returns {string}
@@ -22,7 +26,7 @@ function extensionsToString(pExtensions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildNotToTestRule(pInitOptions) {
@@ -49,7 +53,7 @@ function buildNotToTestRule(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildTsPreCompilationDepsAttribute(pInitOptions) {
@@ -59,7 +63,7 @@ function buildTsPreCompilationDepsAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildCombinedDependenciesAttribute(pInitOptions) {
@@ -69,7 +73,7 @@ function buildCombinedDependenciesAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildTsOrJsConfigAttribute(pInitOptions) {
@@ -89,7 +93,7 @@ function buildTsOrJsConfigAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildWebpackConfigAttribute(pInitOptions) {
@@ -107,7 +111,7 @@ function buildWebpackConfigAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildBabelConfigAttribute(pInitOptions) {
@@ -121,7 +125,7 @@ function buildBabelConfigAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildExtensionsAttribute(pInitOptions) {
@@ -131,7 +135,7 @@ function buildExtensionsAttribute(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string}
  */
 function buildMainFieldsAttribute(pInitOptions) {
@@ -149,7 +153,7 @@ function buildMainFieldsAttribute(pInitOptions) {
  * Creates a .dependency-cruiser config with a set of basic validations
  * to the current directory.
  *
- * @param {import("./types.js").IInitConfig} pInitOptions ('Normalized') options that influence the shape of
+ * @param {IInitConfig} pInitOptions ('Normalized') options that influence the shape of
  *                  the configuration
  * @returns {string} the configuration as a string
  */

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

@@ -228,6 +228,17 @@ module.exports = {
     
     // moduleSystems: ['cjs', 'es6'],
 
+    /* 
+      false: don't look at JSDoc imports
+      true: dependency-cruiser will detect dependencies in JSDoc-style
+      import statements. Implies "parser": "tsc", so the dependency-cruiser
+      will use the typescript parser for JavaScript files.
+     
+      For this to work the typescript compiler will need to be installed in the
+      same spot as you're running dependency-cruiser from.
+     */
+    // detectJSDocImports: true,
+
     /* prefix for links in html and svg output (e.g. 'https://github.com/you/yourrepo/blob/main/'
        to open it on your online repo or \`vscode://file/$\{process.cwd()}/\` to 
        open it in visual studio code),

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

@@ -20,17 +20,21 @@ import {
 } from "./environment-helpers.mjs";
 import { writeRunScriptsToManifest } from "./write-run-scripts-to-manifest.mjs";
 
+/**
+ * @import { IInitConfig, IPartialInitConfig, OneShotConfigIDType } from "./types.js";
+ */
+
 const PACKAGE_MANIFEST = `./${_PACKAGE_MANIFEST}`;
 
 /**
  * Create a initialization configuration based on guessed defaults
  * (e.g. a tsconfig exists => use it and assume typescript is used)
  *
- * @param {import("./types").OneShotConfigIDType} pOneShotConfigId
- * @return {import("./types").IPartialInitConfig} an initialization configuration
+ * @param {OneShotConfigIDType} pOneShotConfigId
+ * @return {IPartialInitConfig} an initialization configuration
  */
 function getOneShotConfig(pOneShotConfigId) {
-  /** @type {import("./types").IPartialInitConfig} */
+  /** @type {IPartialInitConfig} */
   const lBaseConfig = {
     isMonoRepo: isLikelyMonoRepo(),
     isTypeModule: isTypeModule(),
@@ -46,7 +50,7 @@ function getOneShotConfig(pOneShotConfigId) {
     babelConfig: getBabelConfigCandidates().shift(),
     specifyResolutionExtensions: true,
   };
-  /** @type {Map<import("./types").OneShotConfigIDType, import("./types").IPartialInitConfig>} */
+  /** @type {Map<OneShotConfigIDType, IPartialInitConfig>} */
   const lOneShotConfigs = new Map([
     ["yes", lBaseConfig],
     [
@@ -63,7 +67,7 @@ function getOneShotConfig(pOneShotConfigId) {
 
 /**
  *
- * @param {import("./types").IInitConfig} pNormalizedInitConfig
+ * @param {IInitConfig} pNormalizedInitConfig
  */
 function manifestIsUpdatable(pNormalizedInitConfig) {
   return (
@@ -73,7 +77,7 @@ function manifestIsUpdatable(pNormalizedInitConfig) {
 }
 
 /**
- * @param {boolean|import("./types").OneShotConfigIDType} pInit
+ * @param {boolean|OneShotConfigIDType} pInit
  * @param {string=} pConfigFileName
  * @param {{stdout: NodeJS.WritableStream, stderr: NodeJS.WritableStream}=} pStreams
  */

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

@@ -8,7 +8,11 @@ import findExtensions from "./find-extensions.mjs";
 import meta from "#meta.cjs";
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @import { IInitConfig, IPartialInitConfig } from "./types.js";
+ */
+
+/**
+ * @param {IInitConfig} pInitOptions
  * @param {string[]} pExtensions
  * @returns {boolean}
  */
@@ -25,7 +29,7 @@ function usesTypeScript(pInitOptions, pExtensions) {
 }
 
 /**
- * @param {import("./types.js").IInitConfig} pInitOptions
+ * @param {IInitConfig} pInitOptions
  * @returns {string[]}
  */
 function getExtensions(pInitOptions) {
@@ -38,8 +42,8 @@ function getExtensions(pInitOptions) {
 }
 
 /**
- * @param {import("./types.js").IPartialInitConfig} pInitOptions
- * @return {import("./types.js").IPartialInitConfig}
+ * @param {IPartialInitConfig} pInitOptions
+ * @return {IPartialInitConfig}
  */
 function populate(pInitOptions) {
   const lReturnValue = {
@@ -66,8 +70,8 @@ function populate(pInitOptions) {
 
 /**
  *
- * @param {import("./types.js").IPartialInitConfig} pInitOptions
- * @return {import("./types.js").IInitConfig}
+ * @param {IPartialInitConfig} pInitOptions
+ * @return {IInitConfig}
  */
 export default function normalizeInitOptions(pInitOptions) {
   let lReturnValue = populate(pInitOptions);

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-depcruise-options.mjs

@@ -1,9 +1,13 @@
 import extractDepcruiseConfig from "./extract-depcruise-config/index.mjs";
 
 /**
- *
- * @param {import('../../../types/configuration.mjs').IConfiguration}} pConfiguration
- * @returns {import('../../../types/configuration.mjs').ICruiseOptions}
+ * @import { IConfiguration } from "../../types/configuration.mjs";
+ * @import { ICruiseOptions } from "../../types/options.mjs";
+ */
+
+/**
+ * @param {IConfiguration} pConfiguration
+ * @returns {ICruiseOptions}
  */
 function configuration2options(pConfiguration) {
   /* c8 ignore next 1 */
@@ -20,7 +24,7 @@ function configuration2options(pConfiguration) {
 /**
  *
  * @param {string} pConfigFileName
- * @returns {Promise<import('../../../types/configuration.mjs').ICruiseOptions>}
+ * @returns {Promise<ICruiseOptions>}
  */
 export default async function extractDepcruiseOptions(pConfigFileName) {
   const lReturnValue = await extractDepcruiseConfig(pConfigFileName);

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

@@ -3,8 +3,8 @@ import json5 from "json5";
 import makeAbsolute from "./make-absolute.mjs";
 
 /**
- * @typedef {import("../../types/violations.d.mjs").IViolation} IViolation
- * @typedef {import("../../types/shared-types.d.mts").DependencyType} DependencyType
+ * @import { IViolation } from "../../types/violations.mjs"
+ * @import { DependencyType } from "../../types/shared-types.mjs"
  *
  * @typedef {IViolation & {cycle: Array<{name: string, dependencyTypes: Array<DependencyType>}>} | {cycle: Array<DependencyType>}} IMaybeOldFormatViolation
  */
@@ -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/enrich/summarize/summarize-options.mjs

@@ -4,6 +4,7 @@ const SHAREABLE_OPTIONS = [
   "cache",
   "collapse",
   "combinedDependencies",
+  "detectJSDocImports",
   "doNotFollow",
   "enhancedResolveOptions",
   "exclude",
@@ -20,6 +21,7 @@ const SHAREABLE_OPTIONS = [
   "moduleSystems",
   "outputTo",
   "outputType",
+  "parser",
   "prefix",
   "preserveSymlinks",
   "reaches",