dependency-cruiser 18.1.0 → 18.1.1

package/configs/plugins/3d-reporter-plugin.mjs

@@ -114,7 +114,7 @@ function render3DThing(pCruiseResult) {
  * @param {import('../../types/dependency-cruiser').ICruiseResult} pCruiseResult -
  *      the output of a dependency-cruise adhering to dependency-cruiser's
  *      cruise result schema
- * @return {import('../../types/dependency-cruiser').IReporterOutput} -
+ * @returns {import('../../types/dependency-cruiser').IReporterOutput} -
  *      output: a string
  *      exitCode: 0
  */

package/configs/plugins/stats-reporter-plugin.mjs

@@ -34,7 +34,7 @@ function doMagic(pCruiseResult) {
  * got passed
  *
  * @param {import('../../types/dependency-cruiser').ICruiseResult} pCruiseResult - a result from a cruise.
- * @return {string} an object with some stats
+ * @returns {string} an object with some stats
  */
 function samplePluginReporter(pCruiseResult) {
 	const lDependencyCounts = pCruiseResult.modules
@@ -68,7 +68,7 @@ function samplePluginReporter(pCruiseResult) {
  * @param {import('../../types/dependency-cruiser').ICruiseResult} pCruiseResult -
  *      the output of a dependency-cruise adhering to dependency-cruiser's
  *      cruise result schema
- * @return {import('../../types/dependency-cruiser').IReporterOutput} -
+ * @returns {import('../../types/dependency-cruiser').IReporterOutput} -
  *      output: some stats on modules and dependencies in json format
  *      exitCode: 0
  */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "dependency-cruiser",
-	"version": "18.1.0",
+	"version": "18.1.1",
 	"description": "Validate and visualize dependencies. With your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.",
 	"keywords": [
 		"static analysis",
@@ -143,13 +143,13 @@
 		"README.md"
 	],
 	"dependencies": {
-		"acorn": "8.17.0",
+		"acorn": "8.18.0",
 		"acorn-jsx": "5.3.2",
 		"acorn-jsx-walk": "2.0.0",
 		"acorn-loose": "8.5.2",
 		"acorn-walk": "8.3.5",
 		"commander": "15.0.0",
-		"enhanced-resolve": "5.24.2",
+		"enhanced-resolve": "5.24.5",
 		"ignore": "7.0.6",
 		"interpret": "3.1.1",
 		"is-installed-globally": "1.0.0",

package/src/analyze/add-validations.mjs

@@ -17,7 +17,7 @@ function addDependencyViolations(pModule, pDependency, pRuleSet, pValidate) {
  * @param  {Partial<import("../../types/cruise-result.mjs").IModule>[]} pModules array of modules
  * @param  {import("../../types/rule-set.mjs").IFlattenedRuleSet} pRuleSet normalized & validated rule set
  * @param {boolean} pValidate - whether or not to validate (typically you want to pass 'true' here)
- * @return {import("../../types/cruise-result.mjs").IModule[]} the same array of modules, with for each
+ * @returns {import("../../types/cruise-result.mjs").IModule[]} the same array of modules, with for each
  *                  of them added whether or not it is
  *                  valid and if not which rules were violated
  */

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

@@ -23,8 +23,8 @@ export function getEfferentCouplings(pModule, pDirname) {
  * @returns string[]
  */
 export function getParentFolders(pPath) {
-	let lFragments = pPath.split("/");
-	let lReturnValue = [];
+	const lFragments = pPath.split("/");
+	const lReturnValue = [];
 
 	while (lFragments.length > 0) {
 		lReturnValue.push(lFragments.join("/"));

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

@@ -69,7 +69,7 @@ function toDependencyViolationSummary(pRule, pModule, pDependency, pRuleSet) {
  *
  * @param {any} pModules an array of modules
  * @param {any} pRuleSet? a rule set
- * @return {any} an array of violations
+ * @returns {any} an array of violations
  */
 function extractDependencyViolations(pModules, pRuleSet) {
 	return pModules

package/src/cache/cache.mjs

@@ -115,7 +115,7 @@ export default class Cache {
 				lPayload = await readFile(join(pCacheFolder, CACHE_FILE_NAME), "utf8");
 			}
 			return JSON.parse(lPayload);
-		} catch (pError) {
+		} catch {
 			return EMPTY_CACHE;
 		}
 	}
@@ -123,7 +123,7 @@ export default class Cache {
 	/**
 	 * @param {string} pPayload
 	 * @param {boolean} pCompress
-	 * @return {Buffer|string}
+	 * @returns {Buffer|string}
 	 */
 	#compact(pPayload, pCompress) {
 		if (pCompress) {
@@ -144,7 +144,7 @@ export default class Cache {
 	 * @param {IRevisionData=} pRevisionData
 	 */
 	async write(pCacheFolder, pCruiseResult, pRevisionData) {
-		let lRevisionData = pRevisionData ?? this.#revisionData;
+		const lRevisionData = pRevisionData ?? this.#revisionData;
 
 		await mkdir(pCacheFolder, { recursive: true });
 		const lUncompressedPayload = JSON.stringify(
@@ -153,7 +153,7 @@ export default class Cache {
 				lRevisionData,
 			),
 		);
-		let lPayload = this.#compact(lUncompressedPayload, this.#compress);
+		const lPayload = this.#compact(lUncompressedPayload, this.#compress);
 
 		// relying on writeFile defaults to 'do the right thing' (i.e. utf8
 		// when the payload is a string; raw buffer otherwise)

package/src/cache/helpers.mjs

@@ -37,7 +37,7 @@ export function getFileHashSync(pFileName) {
 	let lHashedFileName = "file not found";
 	try {
 		lHashedFileName = hash(readFileSync(pFileName, "utf8"));
-	} catch (pError) {
+	} catch {
 		// will return "file not found" as per default
 	}
 	HASHES_CACHE.set(pFileName, lHashedFileName);
@@ -46,7 +46,7 @@ export function getFileHashSync(pFileName) {
 
 /**
  * @param {IChange} pChange
- * @return {IRevisionChange}
+ * @returns {IRevisionChange}
  */
 export function addCheckSumToChangeSync(pChange) {
 	return {

package/src/cli/format-meta-info.mjs

@@ -10,12 +10,12 @@ const MAX_TRANSPILER_NAME_LENGTH = 22;
 const MAX_VERSION_STRING_LENGTH = 24;
 
 function formatTranspilers(pAvailableTranspilers) {
-	let lTranspilerTableHeader = styleText(
+	const lTranspilerTableHeader = styleText(
 		"bold",
 		`    ✔ ${"transpiler".padEnd(MAX_TRANSPILER_NAME_LENGTH)} ${"versions supported".padEnd(MAX_VERSION_RANGE_STRING_LENGTH)} version found`,
 	);
-	let lTranspilerTableDivider = `    - ${"-".repeat(MAX_TRANSPILER_NAME_LENGTH)} ${"-".repeat(MAX_VERSION_RANGE_STRING_LENGTH)} ${"-".repeat(MAX_VERSION_STRING_LENGTH)}`;
-	let lTranspilerTable = pAvailableTranspilers
+	const lTranspilerTableDivider = `    - ${"-".repeat(MAX_TRANSPILER_NAME_LENGTH)} ${"-".repeat(MAX_VERSION_RANGE_STRING_LENGTH)} ${"-".repeat(MAX_VERSION_STRING_LENGTH)}`;
+	const lTranspilerTable = pAvailableTranspilers
 		.map(
 			(pTranspiler) =>
 				`    ${bool2Symbol(pTranspiler.available)} ${pTranspiler.name.padEnd(MAX_TRANSPILER_NAME_LENGTH)} ${pTranspiler.version.padEnd(MAX_VERSION_RANGE_STRING_LENGTH)} ${pTranspiler.currentVersion}`,

package/src/cli/index.mjs

@@ -41,7 +41,7 @@ async function addKnownViolations(pCruiseOptions) {
 
 		// Check against json schema is already done in src/main/options/validate
 		// so here we can just concentrate on the io
-		let lCruiseOptions = structuredClone(pCruiseOptions);
+		const lCruiseOptions = structuredClone(pCruiseOptions);
 		set(lCruiseOptions, "ruleSet.options.knownViolations", lKnownViolations);
 		return lCruiseOptions;
 	}
@@ -150,7 +150,7 @@ export default async function executeCli(
 		stderr: process.stderr,
 		...(pStreams || {}),
 	};
-	let lCruiseOptions = pCruiseOptions || {};
+	const lCruiseOptions = pCruiseOptions || {};
 	let lExitCode = 0;
 
 	try {

package/src/cli/init-config/check-and-warn-inconsistencies.mjs

@@ -16,7 +16,7 @@ import { getAvailableTranspilers } from "#extract/transpile/meta.mjs";
  *
  * @param {IInitConfig} pInitOptions
  * @param {IAvailableTranspiler[]} pAvailableTranspilers
- * @return {boolean}
+ * @returns {boolean}
  */
 function typescriptIsConsistent(pInitOptions, pAvailableTranspilers) {
 	if (pInitOptions.usesTypeScript) {
@@ -37,7 +37,7 @@ function typescriptIsConsistent(pInitOptions, pAvailableTranspilers) {
  *
  * @param {IInitConfig} pInitOptions
  * @param {IAvailableTranspiler[]} pAvailableTranspilers
- * @return {boolean}
+ * @returns {boolean}
  */
 function babelIsConsistent(pInitOptions, pAvailableTranspilers) {
 	if (pInitOptions.babelConfig) {
@@ -57,7 +57,7 @@ function babelIsConsistent(pInitOptions, pAvailableTranspilers) {
  * @param {IInitConfig} pInitOptions
  * @param {NodeJS.WritableStream} [pErrorStream]
  * @param {typeof getAvailableTranspilers} [pAvailableTranspilerFunction]
- * @return {IInitConfig} pInitOptions
+ * @returns {IInitConfig} pInitOptions
  */
 export function checkAndWarnInconsistencies(
 	pInitOptions,

package/src/cli/init-config/environment-helpers.mjs

@@ -38,7 +38,7 @@ export function readManifest(pManifestFileName = "./package.json") {
 export function fileExists(pFile) {
 	try {
 		accessSync(pFile, constants.R_OK);
-	} catch (pError) {
+	} catch {
 		return false;
 	}
 	return true;
@@ -53,7 +53,7 @@ function babelIsConfiguredInManifest() {
 	try {
 		// @ts-expect-error defaultly tsc doesn't know about newfangled stuff like hasOwn
 		lReturnValue = Object.hasOwn(readManifest(), "babel");
-	} catch (pError) {
+	} catch {
 		// silently ignore - we'll return false anyway then
 	}
 	return lReturnValue;
@@ -67,7 +67,7 @@ export function isTypeModule() {
 
 	try {
 		lReturnValue = (readManifest()?.type ?? "commonjs") === "module";
-	} catch (pError) {
+	} catch {
 		// silently ignore - we'll return false anyway then
 	}
 
@@ -112,7 +112,7 @@ function getPackageManager(pManifest) {
 		if (Object.hasOwn(lManifest, "packageManager")) {
 			lReturnValue = lManifest.packageManager;
 		}
-	} catch (pError) {
+	} catch {
 		// silently ignore - we'll return the empty string anyway then
 	}
 	return lReturnValue;

package/src/cli/init-config/get-user-input.mjs

@@ -163,7 +163,7 @@ const QUESTIONS = [
 ];
 
 /**
- * @return {Promise<import("./types.d.mts").IPartialInitConfig>}
+ * @returns {Promise<import("./types.d.mts").IPartialInitConfig>}
  */
 export default function getUserInput() {
 	return prompts(QUESTIONS);

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

@@ -33,7 +33,7 @@ const PACKAGE_MANIFEST = `./${_PACKAGE_MANIFEST}`;
  * (e.g. a tsconfig exists => use it and assume typescript is used)
  *
  * @param {OneShotConfigIDType} pOneShotConfigId
- * @return {IPartialInitConfig} an initialization configuration
+ * @returns {IPartialInitConfig} an initialization configuration
  */
 function getOneShotConfig(pOneShotConfigId) {
 	/** @type {IPartialInitConfig} */

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

@@ -43,7 +43,7 @@ function getExtensions(pInitOptions) {
 
 /**
  * @param {IPartialInitConfig} pInitOptions
- * @return {IPartialInitConfig}
+ * @returns {IPartialInitConfig}
  */
 function populate(pInitOptions) {
 	const lReturnValue = {
@@ -71,10 +71,10 @@ function populate(pInitOptions) {
 /**
  *
  * @param {IPartialInitConfig} pInitOptions
- * @return {IInitConfig}
+ * @returns {IInitConfig}
  */
 export default function normalizeInitOptions(pInitOptions) {
-	let lReturnValue = populate(pInitOptions);
+	const lReturnValue = populate(pInitOptions);
 	if (!Object.hasOwn(lReturnValue, "hasTestsOutsideSource")) {
 		lReturnValue.hasTestsOutsideSource =
 			!pInitOptions.isMonoRepo &&

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

@@ -7,7 +7,7 @@ export function validateLocation(pLocations) {
 			if (!statSync(lLocation).isDirectory()) {
 				return `'${lLocation}' doesn't seem to be a folder - please try again`;
 			}
-		} catch (pError) {
+		} catch {
 			return `'${lLocation}' doesn't seem to exist - please try again`;
 		}
 	}

package/src/cli/init-config/write-run-scripts-to-manifest.mjs

@@ -60,7 +60,7 @@ const EXPERIMENTAL_SCRIPT_DOC = [
 
 /**
  * @param {import("./types.d.mts").IInitConfig} pInitOptions
- * @return {any} an bunch of key value pairs that can be plonked into a `scripts` attribute in a package.json
+ * @returns {any} an bunch of key value pairs that can be plonked into a `scripts` attribute in a package.json
  */
 export function compileRunScripts(pInitOptions) {
 	let lReturnValue = {};
@@ -91,7 +91,7 @@ export function compileRunScripts(pInitOptions) {
  * @param {any} pAdditionalRunScripts
  * @param {any} pExistingRunScripts
  *
- * @return {any} the scripts in pAdditionalRunScripts that don't yet exist in
+ * @returns {any} the scripts in pAdditionalRunScripts that don't yet exist in
  * pExistingRunScripts
  */
 function filterNewScriptEntries(pExistingRunScripts, pAdditionalRunScripts) {

package/src/cli/listeners/ndjson.mjs

@@ -88,7 +88,7 @@ export default function setUpNDJSONListener(
 	pMaxLevel = INFO,
 	pStream = process.stderr,
 ) {
-	let lState = {
+	const lState = {
 		runStartTime: new Date(Date.now()).toISOString(),
 		runId: randomInt(MIN_RUN_ID, MAX_RUN_ID),
 		previousMessage: "startup: nodejs loading",

package/src/cli/listeners/performance-log/index.mjs

@@ -34,7 +34,7 @@ export default function setUpPerformanceLogListener(
 	pMaxLevel = INFO,
 	pStream = process.stderr,
 ) {
-	let lState = {
+	const lState = {
 		previousMessage: "startup: nodejs loading",
 		previousTime: 0,
 		previousUserUsage: 0,

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

@@ -19,7 +19,7 @@ function getOptionValue(pDefault) {
 
 // eslint-disable-next-line complexity
 function normalizeConfigFileName(pCliOptions, pConfigWrapperName, pDefault) {
-	let lOptions = structuredClone(pCliOptions);
+	const lOptions = structuredClone(pCliOptions);
 
 	if (Object.hasOwn(lOptions, pConfigWrapperName)) {
 		set(
@@ -45,12 +45,13 @@ function fileExists(pFileName) {
 	try {
 		accessSync(pFileName, constants.R_OK);
 		return true;
-	} catch (pError) {
+	} catch {
 		return false;
 	}
 }
 
 function validateAndGetCustomRulesFileName(pValidate) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = "";
 
 	if (fileExists(pValidate)) {
@@ -66,7 +67,7 @@ function validateAndGetCustomRulesFileName(pValidate) {
 }
 
 function validateAndGetDefaultRulesFileName() {
-	let lReturnValue = RULES_FILE_NAME_SEARCH_ARRAY.find(fileExists);
+	const lReturnValue = RULES_FILE_NAME_SEARCH_ARRAY.find(fileExists);
 
 	if (typeof lReturnValue === "undefined") {
 		throw new TypeError(
@@ -79,6 +80,7 @@ function validateAndGetDefaultRulesFileName() {
 }
 
 function validateAndNormalizeRulesFileName(pValidate) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = "";
 
 	if (typeof pValidate === "string") {
@@ -196,7 +198,7 @@ function normalizeCache(pCliOptions) {
  *
  * @param  {object} pOptionsAsPassedFromCommander [description]
  * @param {any} pKnownCliOptions [description]
- * @return {import("../../types/options.mjs").ICruiseOptions}          [description]
+ * @returns {import("../../types/options.mjs").ICruiseOptions}          [description]
  */
 export default async function normalizeOptions(pOptionsAsPassedFromCommander) {
 	let lOptions = {

package/src/cli/utl/assert-file-existence.mjs

@@ -3,7 +3,7 @@ import { accessSync, constants } from "node:fs";
 export default function assertFileExistence(pDirectoryOrFile) {
 	try {
 		accessSync(pDirectoryOrFile, constants.R_OK);
-	} catch (pError) {
+	} catch {
 		throw new Error(
 			`Can't open '${pDirectoryOrFile}' for reading. Does it exist?\n`,
 		);

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

@@ -6,6 +6,7 @@ import tryImport from "#utl/try-import.mjs";
 import meta from "#meta.cjs";
 
 async function getJSConfig(pBabelConfigFileName) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = {};
 
 	try {
@@ -34,6 +35,7 @@ async function getJSConfig(pBabelConfigFileName) {
 }
 
 async function getJSON5Config(pBabelConfigFileName) {
+	/* oxlint-disable-next-line no-useless-assignment */
 	let lReturnValue = {};
 	const { default: json5 } = await import("json5");
 
@@ -78,7 +80,7 @@ async function getConfig(pBabelConfigFileName) {
  * Silently fails if a supported @babel/core version can't be found
  *
  * @param {string} pBabelConfigFileName
- * @return {object} babel config as an object
+ * @returns {object} babel config as an object
  * @throws {Error} when the babel config has an unknown extension OR
  *                 when the babel config is invalid OR
  *                 when dependency-cruiser can't yet process it

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

@@ -47,7 +47,7 @@ async function processExtends(pReturnValue, pAlreadyVisited, pBaseDirectory) {
  * @param {string} pConfigFileName
  * @param {Set?} pAlreadyVisited
  * @param {string?} pBaseDirectory
- * @return {import('../../../types/configuration.mjs').IConfiguration} dependency-cruiser options
+ * @returns {import('../../../types/configuration.mjs').IConfiguration} dependency-cruiser options
  * @throws {Error} when the config is not valid (/ does not exist/ isn't readable)
  */
 export default async function extractDepcruiseConfig(

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/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(