dependency-cruiser 18.0.0 → 18.1.1

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

@@ -6,7 +6,9 @@ function extendNamedRule(pExtendedRule, pForbiddenArrayBase) {
 		.filter(({ name }) => name === pExtendedRule.name)
 		.reduce(
 			(pAll, pBaseRule) => ({
+				/* oxlint-disable-next-line no-accumulating-spread */
 				...pBaseRule,
+				/* oxlint-disable-next-line no-accumulating-spread */
 				...pAll,
 			}),
 			pExtendedRule,
@@ -25,11 +27,11 @@ function extendNamedRule(pExtendedRule, pForbiddenArrayBase) {
  * @param {*} pRuleArrayExtended - array of 'forbidden' rules that extend the ...
  * @param {*} pRuleArrayBase - array of 'forbidden' rules to extend
  *
- * @return {Array} - the merged array
+ * @returns {Array} - the merged array
  */
 function mergeRules(pRuleArrayExtended, pRuleArrayBase) {
 	// merge anonymous on 100% equality
-	let lAnonymousRules = uniqWith(
+	const lAnonymousRules = uniqWith(
 		pRuleArrayExtended.concat(pRuleArrayBase).filter(({ name }) => !name),
 		isDeepStrictEqual,
 	);
@@ -61,7 +63,7 @@ function mergeRules(pRuleArrayExtended, pRuleArrayBase) {
  * @param {*} pAllowedArrayExtended - array of 'allowed' rules that extend the ///
  * @param {*} pAllowedArrayBase - array of 'allowed' rules to extend
  *
- * @return {Array} - the merged array
+ * @returns {Array} - the merged array
  */
 function mergeAllowedRules(pAllowedArrayExtended, pAllowedArrayBase) {
 	return uniqWith(

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

@@ -34,7 +34,7 @@ const FORMAT_DIAGNOSTICS_HOST = {
  * Silently fails if a supported version of the typescript compiler isn't available
  *
  * @param {string} pTSConfigFileName
- * @return {import("typescript").ParsedCommandLine} tsconfig as an object
+ * @returns {import("typescript").ParsedCommandLine} tsconfig as an object
  * @throws {Error} when the tsconfig is invalid/ has errors
  * @throws {TypeError} when the tsconfig is unreadable
  */

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 n/global-require:0, security/detect-non-literal-require:0, import/no-dynamic-require:0 */
+			/* oxlint-disable-next-line no-dynamic-require */
 			return require(pAbsoluteWebpackConfigFileName);
 		}
 	} catch (pError) {
@@ -112,7 +112,7 @@ async function attemptImport(pAbsoluteWebpackConfigFileName) {
  * @param {string} pWebpackConfigFilename
  * @param {{ [key: string]: any }=} pEnvironment
  * @param {webpackArgumentsType=} pArguments
- * @return {Promise<import("enhanced-resolve").ResolveOptions|{}>} webpack resolve config as an object
+ * @returns {Promise<import("enhanced-resolve").ResolveOptions|{}>} webpack resolve config as an object
  * @throws {Error} when the webpack config isn't usable (e.g. because it
  *                 doesn't exist, or because it's invalid)
  */

package/src/environment.mjs

@@ -0,0 +1,17 @@
+import { release, platform, arch } from "node:os";
+import meta from "#meta.cjs";
+import {
+	getAvailableTranspilers,
+	allExtensions,
+} from "#extract/transpile/meta.mjs";
+
+export function getEnvironmentInfo() {
+	return {
+		version: meta.version,
+		nodeVersionSupported: meta.engines.node,
+		nodeVersionFound: process.version,
+		osVersionFound: `${arch()} ${platform()}@${release()}`,
+		transpilersFound: getAvailableTranspilers(),
+		extensionsFound: structuredClone(allExtensions),
+	};
+}

package/src/extract/acorn/extract.mjs

@@ -15,7 +15,7 @@ export function extract(
 	pFileName,
 	pTranspileOptions,
 ) {
-	let lDependencies = [];
+	const lDependencies = [];
 	const lAST = getASTCached(join(baseDir, pFileName), pTranspileOptions);
 
 	if (moduleSystems.includes("cjs")) {

package/src/extract/acorn/parse.mjs

@@ -46,7 +46,7 @@ export function getASTFromSource(pFileRecord, pTranspileOptions) {
 		// ecmaVersion 11 necessary for acorn to understand dynamic imports
 		// explicitly passing ecmaVersion is recommended from acorn 8
 		return acornParse(lJavaScriptSource, ACORN_OPTIONS);
-	} catch (pError) {
+	} catch {
 		return acornLooseParse(lJavaScriptSource, ACORN_OPTIONS);
 	}
 }
@@ -62,7 +62,7 @@ export function getASTFromSource(pFileRecord, pTranspileOptions) {
  *
  * @param {string} pFileName - the name of the file to compile
  * @param {any} pTranspileOptions - options for the transpiler(s) - typically a tsconfig or a babel config
- * @return {acorn.Node} - a (javascript) AST
+ * @returns {acorn.Node} - a (javascript) AST
  */
 export function getASTCached(pFileName, pTranspileOptions) {
 	// taking the transpile options into account of the cache key seems like

package/src/extract/extract-dependencies.mjs

@@ -154,7 +154,7 @@ function compareDeps(pLeft, pRight) {
  * @param  {ITranspileOptions} pTranspileOptions       an object with tsconfig ('typescript project') options
  *                               ('flattened' so there's no need for file access on any
  *                               'extends' option in there)
- * @return {IDependency[]} an array of dependency objects (see above)
+ * @returns {IDependency[]} an array of dependency objects (see above)
  */
 export default function getDependencies(
 	pFileName,

package/src/extract/extract-stats.mjs

@@ -32,7 +32,7 @@ function determineExtractionFunction(pCruiseOptions, pFileName) {
  * @param  {ITranspileOptions} pTranspileOptions       an object with tsconfig ('typescript project') options
  *                               ('flattened' so there's no need for file access on any
  *                               'extends' option in there)
- * @return {IDependency[]} an array of dependency objects (see above)
+ * @returns {IDependency[]} an array of dependency objects (see above)
  */
 export default function extractStats(
 	pFileName,

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

@@ -100,7 +100,7 @@ function expandGlob(pBaseDirectory, pScannedGlob) {
  *                               notably useful attributes:
  *                               - exclude - regexp of what to exclude
  *                               - includeOnly - regexp what to include
- * @return {string[]}            paths to files to be gathered.
+ * @returns {string[]}            paths to files to be gathered.
  */
 export default function gatherInitialSources(
 	pFileDirectoryAndGlobArray,

package/src/extract/helpers.mjs

@@ -75,7 +75,7 @@ function getCanonicalModuleName(pModuleName, pProtocol) {
  */
 // eslint-disable-next-line complexity
 export function extractModuleAttributes(pString) {
-	let lReturnValue = { module: pString };
+	const lReturnValue = { module: pString };
 	const lModuleAttributes = pString.match(
 		// eslint-disable-next-line security/detect-unsafe-regex
 		/^(?<protocol>node:|file:|data:|bun:)(?:(?<mimeType>[^,]+),)?(?<module>.+)$/,

package/src/extract/index.mjs

@@ -68,7 +68,7 @@ function extractFileDirectoryArray(
 	pResolveOptions,
 	pTranspileOptions,
 ) {
-	let lVisited = new Set();
+	const lVisited = new Set();
 
 	bus.debug("extract: gather initial sources");
 	const lInitialSources = gatherInitialSources(

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

@@ -115,7 +115,7 @@ function determineNodeModuleDependencyTypes(
 	pResolveOptions,
 ) {
 	/** @type { DependencyType[] } */
-	let lReturnValue = determineManifestDependencyTypes(
+	const lReturnValue = determineManifestDependencyTypes(
 		getPackageRoot(pModuleName),
 		pPackageDeps,
 		pResolveOptions.modules,
@@ -151,6 +151,7 @@ function determineExternalModuleDependencyTypes(
 	pBaseDirectory,
 ) {
 	/** @type { DependencyType[] } */
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = [];
 
 	if (
@@ -179,7 +180,7 @@ function determineExternalModuleDependencyTypes(
  * @param {string} pBaseDirectory the base directory dependency cruise is run on
  * @param {ITranspileOptions} pTranspileOptions
  *
- * @return { DependencyType[] }an array of dependency types for the dependency
+ * @returns { DependencyType[] }an array of dependency types for the dependency
  */
 // eslint-disable-next-line max-lines-per-function
 export default function determineDependencyTypes(

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

@@ -30,14 +30,14 @@ const PACKAGE_JSON_CACHE = new Map();
  * 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
+ * @returns {string}         the module name root
  */
 export function getPackageRoot(pModule) {
 	if (!pModule || isRelativeModuleName(pModule)) {
 		return pModule;
 	}
 
-	let lPathElements = pModule.split("/");
+	const lPathElements = pModule.split("/");
 
 	if (isScoped(pModule)) {
 		// '@imdoingweirdvoodoo'
@@ -69,7 +69,7 @@ export function getPackageRoot(pModule) {
  * @param  {string} pModuleName  The name of the module to get the package.json of
  * @param  {string} pFileDirectory The folder the module resides in. Defaults to the current working directory
  * @param  {any} pResolveOptions options for the resolver
- * @return {Record<string, any>} The package.json as a javascript object, or
+ * @returns {Record<string, any>} The package.json as a javascript object, or
  *                           null if either module or package.json could
  *                           not be found
  */
@@ -104,7 +104,7 @@ export function getPackageJson(pModuleName, pFileDirectory, pResolveOptions) {
 			"manifest-resolution",
 		);
 		lReturnValue = JSON.parse(readFileSync(lPackageJsonFilename, "utf8"));
-	} catch (pError) {
+	} catch {
 		// left empty on purpose
 	}
 	PACKAGE_JSON_CACHE.set(lCacheKey, lReturnValue);
@@ -117,7 +117,7 @@ export function getPackageJson(pModuleName, pFileDirectory, pResolveOptions) {
  * @param  {string} pModuleName  The name of the module to get the deprecation status of
  * @param  {string} pFileDirectory The folder the module resides in.
  * @param  {any} pResolveOptions options for the resolver
- * @return {boolean}         true when deprecated, false in all other cases
+ * @returns {boolean}         true when deprecated, false in all other cases
  */
 export function dependencyIsDeprecated(
 	pModuleName,
@@ -125,7 +125,7 @@ export function dependencyIsDeprecated(
 	pResolveOptions,
 ) {
 	let lReturnValue = false;
-	let lPackageJson = getPackageJson(
+	const lPackageJson = getPackageJson(
 		pModuleName,
 		pFileDirectory,
 		pResolveOptions,
@@ -144,12 +144,12 @@ export function dependencyIsDeprecated(
  * @param  {string} pModuleName  The name of the module to get the license of
  * @param  {string} pFileDirectory The folder the module resides in.
  * @param  {any} pResolveOptions options for the resolver
- * @return {string}          The module's license string, or '' in case
+ * @returns {string}          The module's license string, or '' in case
  *                           there is no package.json or no license field
  */
 export function getLicense(pModuleName, pFileDirectory, pResolveOptions) {
 	let lReturnValue = "";
-	let lPackageJson = getPackageJson(
+	const lPackageJson = getPackageJson(
 		pModuleName,
 		pFileDirectory,
 		pResolveOptions,

package/src/extract/resolve/get-manifest.mjs

@@ -14,7 +14,7 @@ const COMBINED_MANIFEST_CACHE = new Map();
  *
  * @param {string} pFileDirectory the folder relative to which to find
  *                          the package.json
- * @return {any} the contents of the package.json as a javascript
+ * @returns {any} the contents of the package.json as a javascript
  *               object or null if the package.json could not be
  *               found or is invalid
  */
@@ -34,10 +34,10 @@ function getSingleManifest(pFileDirectory) {
 
 		try {
 			lReturnValue = JSON.parse(lPackageContent);
-		} catch (pError) {
+		} catch {
 			// left empty on purpose
 		}
-	} catch (pError) {
+	} catch {
 		const lNextDirectory = dirname(pFileDirectory);
 
 		if (lNextDirectory !== pFileDirectory) {
@@ -60,17 +60,17 @@ function maybeReadPackage(pFileDirectory) {
 
 		try {
 			lReturnValue = JSON.parse(lPackageContent);
-		} catch (pError) {
+		} catch {
 			// left empty on purpose
 		}
-	} catch (pError) {
+	} catch {
 		// left empty on purpose
 	}
 	return lReturnValue;
 }
 
 function getIntermediatePaths(pFileDirectory, pBaseDirectory) {
-	let lReturnValue = [];
+	const lReturnValue = [];
 	let lIntermediate = pFileDirectory;
 
 	while (
@@ -133,7 +133,7 @@ function getCombinedManifests(pFileDirectory, pBaseDirectory) {
  * @param  {string}  pBaseDirectory              the directory to consider as base (or 'root')
  * @param  {Boolean} pCombinedDependencies whether to stop (false) or continue
  *                                         searching until the 'root'
- * @return {any}                           the contents of a package.json as a javascript
+ * @returns {any}                           the contents of a package.json as a javascript
  *                                         object or null if a package.json could not be
  *                                        found or is invalid
  */

package/src/extract/resolve/index.mjs

@@ -30,6 +30,7 @@ function resolveModule(
 	pFileDirectory,
 	pResolveOptions,
 ) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = null;
 
 	const lStrippedModuleName = stripToModuleName(pModule.module);
@@ -178,7 +179,7 @@ function resolveWithRetry(
  *                              detected in
  * @param  {IResolveOptions} pResolveOptions
  * @param  {any} pTranspileOptions
- * @return {Partial <IDependency>}
+ * @returns {Partial <IDependency>}
  *
  *
  */

package/src/extract/resolve/merge-manifests.mjs

@@ -2,7 +2,7 @@ import { uniq } from "#utl/array-util.mjs";
 
 /* eslint-disable security/detect-object-injection */
 function normalizeManifestKeys(pManifest) {
-	let lReturnValue = pManifest;
+	const lReturnValue = pManifest;
 
 	if (pManifest.bundleDependencies) {
 		pManifest.bundledDependencies = structuredClone(
@@ -60,7 +60,7 @@ function isAnArrayKey(pKey) {
  *                               source being perused (e.g. ./packages/sub/package.json)
  * @param  {any} pFurtherManifest the contents of a package.json further away
  *                               (e.g. ./package.json)
- * @return {any}                 the combined dependency-keys within those manifests
+ * @returns {any}                 the combined dependency-keys within those manifests
  */
 export default function mergeManifests(pClosestManifest, pFurtherManifest) {
 	return getJointUniqueDependencyKeys(

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

@@ -54,7 +54,7 @@ export function isExternalModule(
 }
 
 function determineFollowableExtensions(pResolveOptions) {
-	let lReturnValue = new Set(pResolveOptions.extensions);
+	const lReturnValue = new Set(pResolveOptions.extensions);
 
 	// we could include things like pictures, movies, html, xml etc in
 	// lKnownUnfollowables as well. Ijavascript-like sources you don't

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

@@ -11,7 +11,7 @@ const fileExists = (pFile) => {
 
 	try {
 		accessSync(pFile, constants.R_OK);
-	} catch (pError) {
+	} catch {
 		FILE_EXISTENCE_CACHE.set(pFile, false);
 		return false;
 	}

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

@@ -10,7 +10,7 @@ function addResolutionAttributes(
 	pFileDirectory,
 	pResolveOptions,
 ) {
-	let lReturnValue = {};
+	const lReturnValue = {};
 
 	if (isBuiltin(pModuleName, pResolveOptions)) {
 		lReturnValue.coreModule = true;
@@ -26,7 +26,7 @@ function addResolutionAttributes(
 				lReturnValue.resolved,
 				pResolveOptions,
 			);
-		} catch (pError) {
+		} catch {
 			lReturnValue.couldNotResolve = true;
 		}
 	}

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

@@ -7,7 +7,7 @@ export function addLicenseAttribute(
 	{ baseDirectory, fileDirectory },
 	pResolveOptions,
 ) {
-	let lReturnValue = {};
+	const lReturnValue = {};
 	if (
 		pResolveOptions.resolveLicenses &&
 		isExternalModule(

package/src/extract/resolve/resolve.mjs

@@ -5,9 +5,9 @@ import pathToPosix from "#utl/path-to-posix.mjs";
 /** @import {IResolveOptions} from "../../../types/resolve-options.mjs" */
 
 /** @type {Map<string, enhancedResolve.Resolver>} */
-let gResolvers = new Map();
+const gResolvers = new Map();
 /** @type {Set<string>} */
-let gInitialized = new Set();
+const gInitialized = new Set();
 
 /**
  * Initializes a resolver for the given caching context if not already done

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

@@ -273,5 +273,6 @@ export default Visitor
 			}
 		}
 	: /* c8 ignore start */
+		/* oxlint-disable-next-line typescript/no-extraneous-class */
 		class Empty {};
 /* c8 ignore stop */

package/src/extract/swc/parse.mjs

@@ -32,7 +32,7 @@ export function getASTFromSource(pSource) {
  * return the result from a cache
  *
  * @param {string} pFileName - the name of the file to compile
- * @return {ModuleItem[]} - an (swc) AST
+ * @returns {ModuleItem[]} - an (swc) AST
  */
 export function getASTCached(pFileName) {
 	if (CACHE.has(pFileName)) {
@@ -49,7 +49,7 @@ export function clearCache() {
 }
 
 /**
- * @return {boolean} - true if the swc compiler is available,
+ * @returns {boolean} - true if the swc compiler is available,
  *                     false in all other cases
  */
 // @ts-expect-error dfdfd

package/src/extract/transpile/index.mjs

@@ -89,7 +89,7 @@ export function getWrapper(pExtension, pTranspilerOptions) {
  * @param  {{ extension:string; source:string; filename:string; }} pFileRecord      Record with source code, an extension and a filename
  * @param  {any} pTranspilerOptions (optional) object with options influencing
  *                                the underlying transpiler behavior.
- * @return {string}               the transpiled version of the file (or the file
+ * @returns {string}               the transpiled version of the file (or the file
  *                                itself when the function could not find a
  *                                transpiler matching pExtension
  */

package/src/extract/transpile/meta.mjs

@@ -155,7 +155,7 @@ export const scannableExtensions = Array.from(
  * - the version (range) supported
  * - whether or not it is available in the current environment
  *
- * @return {IAvailableTranspiler[]} an array of supported transpilers
+ * @returns {IAvailableTranspiler[]} an array of supported transpilers
  */
 export function getAvailableTranspilers() {
 	return ["javascript"]

package/src/extract/transpile/try-import-available.mjs

@@ -38,7 +38,7 @@ export default function tryImportAvailable(pModuleName, pSemanticVersion) {
 		// of course we'd love to use something like import.meta.resolve, but
 		// that's _experimental_, so ¯\_(ツ)_/¯
 		return Boolean(require.resolve(pModuleName));
-	} catch (pError) {
+	} catch {
 		return false;
 	}
 }

package/src/extract/transpile/typescript-wrap.mjs

@@ -7,7 +7,7 @@ const typescript = await tryImport(
 );
 
 function getCompilerOptions(pFlavor, pTSConfig) {
-	let lCompilerOptions = {};
+	const lCompilerOptions = {};
 
 	if (pFlavor === "tsx") {
 		lCompilerOptions.jsx = "react";

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

@@ -540,7 +540,7 @@ function extractNestedDependencies(
 	pDetectJSDocImports,
 	pDetectProcessBuiltinModuleCalls,
 ) {
-	let lResult = [];
+	const lResult = [];
 
 	walk(
 		lResult,

package/src/extract/tsc/parse.mjs

@@ -18,7 +18,7 @@ const CACHE = new Map();
  * @param {object} pFileRecord Record with source code, an extension and a filename
  * @param {any} [pTranspileOptions] options for the transpiler(s) - a tsconfig or
  *                                a babel config
- * @return {object} - a (typescript) AST
+ * @returns {object} - a (typescript) AST
  */
 export function getASTFromSource(pFileRecord, pTranspileOptions) {
 	let lSource = pFileRecord.source;
@@ -42,7 +42,7 @@ export function getASTFromSource(pFileRecord, pTranspileOptions) {
  * @param {string} pFileName - the name of the file to compile
  * @param {any} [pTranspileOptions] options for the transpiler(s) - a tsconfig or
  *                                a babel config
- * @return {object} - a (typescript) AST
+ * @returns {object} - a (typescript) AST
  */
 export function getASTCached(pFileName, pTranspileOptions) {
 	if (CACHE.has(pFileName)) {
@@ -61,7 +61,7 @@ export function getASTCached(pFileName, pTranspileOptions) {
 }
 
 /**
- * @return {boolean} - true if the typescript compiler is available,
+ * @returns {boolean} - true if the typescript compiler is available,
  *                     false in all other cases
  */
 export const isAvailable = () => typescript !== false;

package/src/graph-utl/add-focus.mjs

@@ -34,7 +34,7 @@ export default function addFocus(pModules, pFilter) {
 			({ source }) => source,
 		);
 		let lReachableModuleNamesArray = [];
-		let lIndexedModules = new IndexedModuleGraph(pModules);
+		const lIndexedModules = new IndexedModuleGraph(pModules);
 
 		for (const lFocusedModule of lFocusedModuleNames) {
 			lReachableModuleNamesArray = lReachableModuleNamesArray

package/src/graph-utl/consolidate-to-pattern.mjs

@@ -16,6 +16,7 @@ function squashDependencyToPattern(pCollapsePattern) {
 }
 
 function determineConsolidatedness(pConsolidated, pCollapseMatch, pSource) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = false;
 
 	// if it was  already established it's consolidated (e.g. from an earlier

package/src/graph-utl/indexed-module-graph.mjs

@@ -43,7 +43,7 @@ export default class IndexedModuleGraph {
 
 	/**
 	 * @param {string} pName
-	 * @return {IVertex}
+	 * @returns {IVertex}
 	 */
 	findVertexByName(pName) {
 		return this.#indexedGraph.get(pName);
@@ -71,7 +71,7 @@ export default class IndexedModuleGraph {
 		const lModule = this.#indexedGraph.get(pName);
 
 		if (lModule && (!pMaxDepth || pDepth <= pMaxDepth)) {
-			let lDependents = lModule.dependents || [];
+			const lDependents = lModule.dependents || [];
 			const lVisited = pVisited.add(pName);
 
 			// @TODO this works for modules, but not for folders yet
@@ -115,7 +115,7 @@ export default class IndexedModuleGraph {
 		const lModule = this.#indexedGraph.get(pName);
 
 		if (lModule && (!pMaxDepth || pDepth <= pMaxDepth)) {
-			let lDependencies = lModule.dependencies;
+			const lDependencies = lModule.dependencies;
 			const lVisited = pVisited.add(pName);
 
 			if (lDependencies.length > 0) {
@@ -169,7 +169,7 @@ export default class IndexedModuleGraph {
 				if (lDependency.name === pTo) {
 					return [this.#geldEdge(lDependency)];
 				}
-				let lCandidatePath = this.getPath(lDependency.name, pTo, pVisited);
+				const lCandidatePath = this.getPath(lDependency.name, pTo, pVisited);
 
 				if (lCandidatePath.length > 0) {
 					return [this.#geldEdge(lDependency)].concat(lCandidatePath);
@@ -187,7 +187,7 @@ export default class IndexedModuleGraph {
 	 * @param {string} pInitialSource The 'source' attribute of the node to be tested (source uniquely identifying a node)
 	 * @param {IEdge} pCurrentDependency The 'to' node to be traversed as a dependency object of the previous 'from' traversed
 	 * @param {Set<string>=} pVisited Technical parameter - best to leave out of direct calls
-	 * @return {Array<IMiniDependency>} see description above
+	 * @returns {Array<IMiniDependency>} see description above
 	 */
 	#getCycle(pInitialSource, pCurrentDependency, pVisited) {
 		const lVisited = pVisited || new Set();
@@ -239,7 +239,7 @@ export default class IndexedModuleGraph {
 	 *                                (source uniquely identifying a node)
 	 * @param {string} pCurrentSource The 'source' attribute of the 'to' node to
 	 *                                be traversed
-	 * @return {Array<IMiniDependency>}   see description above
+	 * @returns {Array<IMiniDependency>}   see description above
 	 */
 	getCycle(pInitialSource, pCurrentSource) {
 		/** @type {IVertex} */

package/src/graph-utl/module-graph-with-dependency-set.mjs

@@ -3,7 +3,7 @@ export default class ModuleGraphWithDependencySet {
 	#modulesWithDependencySet;
 	/**
 	 * Creates a module graph optimized for querying dependents
-	 * @constructor
+	 * @class
 	 * @param {import("../../types/dependency-cruiser.mjs").IModule[]} pModules
 	 */
 	constructor(pModules) {

package/src/graph-utl/rule-set.mjs

@@ -11,7 +11,7 @@
  *
  * @param {IFlattenedRuleSet} pRuleSet - The rule set to search in
  * @param {string} pName - The rule name to look for
- * @return {IForbiddenRuleType|undefined} - a rule (or 'undefined' if nothing found)
+ * @returns {IForbiddenRuleType|undefined} - a rule (or 'undefined' if nothing found)
  */
 export function findRuleByName(pRuleSet, pName) {
 	const lNamedRules = (pRuleSet?.forbidden ?? []).concat(
@@ -34,7 +34,7 @@ function ruleHasALicenseLikeAttribute(pRule) {
  * Returns false in all other cases
  *
  * @param {IFlattenedRuleSet} pRuleSet
- * @return {boolean}
+ * @returns {boolean}
  */
 export function ruleSetHasLicenseRule(pRuleSet) {
 	return (
@@ -45,7 +45,7 @@ export function ruleSetHasLicenseRule(pRuleSet) {
 
 /**
  * @param {IFlattenedRuleSet} pRuleSet
- * @return {boolean}
+ * @returns {boolean}
  */
 export function ruleSetHasDeprecationRule(pRuleSet) {
 	return (

package/src/main/cruise.mjs

@@ -22,7 +22,7 @@ export default async function cruise(
 	bus.summary("startup: parse options", c(1));
 	const lCruiseOptionsValid = assertCruiseOptionsValid(pCruiseOptions);
 	/** @type {import("../../types/strict-options.mjs").IStrictCruiseOptions} */
-	let lCruiseOptions = normalizeCruiseOptions(
+	const lCruiseOptions = normalizeCruiseOptions(
 		lCruiseOptionsValid,
 		pFileAndDirectoryArray,
 	);

package/src/main/helpers.mjs

@@ -25,7 +25,7 @@ export function normalizeREProperties(
 	pPropertyContainer,
 	pREProperties = RE_PROPERTIES,
 ) {
-	let lPropertyContainer = structuredClone(pPropertyContainer);
+	const lPropertyContainer = structuredClone(pPropertyContainer);
 
 	for (const lProperty of pREProperties) {
 		// lProperty can be nested properties, so we use _.has and _.get