webpack 5.91.0 → 5.92.0

package/lib/javascript/JavascriptParser.js

@@ -6,7 +6,7 @@
 "use strict";
 
 const { Parser: AcornParser } = require("acorn");
-const { importAssertions } = require("acorn-import-assertions");
+const { importAttributesOrAssertions } = require("acorn-import-attributes");
 const { SyncBailHook, HookMap } = require("tapable");
 const vm = require("vm");
 const Parser = require("../Parser");
@@ -88,7 +88,6 @@ const BasicEvaluatedExpression = require("./BasicEvaluatedExpression");
 /** @typedef {import("estree").TaggedTemplateExpression} TaggedTemplateExpression */
 /** @typedef {import("estree").TemplateLiteral} TemplateLiteral */
 /** @typedef {import("estree").AssignmentProperty} AssignmentProperty */
-/** @typedef {Record<string, any>} Attributes */
 /**
  * @template T
  * @typedef {import("tapable").AsArray<T>} AsArray<T>
@@ -100,6 +99,8 @@ const BasicEvaluatedExpression = require("./BasicEvaluatedExpression");
 /** @typedef {Statement | ModuleDeclaration | Expression} StatementPathItem */
 /** @typedef {TODO} OnIdent */
 
+/** @typedef {Record<string, string> & { _isLegacyAssert?: boolean }} ImportAttributes */
+
 /** @type {string[]} */
 const EMPTY_ARRAY = [];
 const ALLOWED_MEMBER_TYPES_CALL_EXPRESSION = 0b01;
@@ -108,7 +109,7 @@ const ALLOWED_MEMBER_TYPES_ALL = 0b11;
 
 // Syntax: https://developer.mozilla.org/en/SpiderMonkey/Parser_API
 
-const parser = AcornParser.extend(importAssertions);
+const parser = AcornParser.extend(importAttributesOrAssertions);
 
 class VariableInfo {
 	/**
@@ -128,14 +129,14 @@ class VariableInfo {
 /** @typedef {Omit<AcornOptions, "sourceType" | "ecmaVersion"> & { sourceType: "module" | "script" | "auto", ecmaVersion?: AcornOptions["ecmaVersion"] }} ParseOptions */
 
 /**
- * @typedef {Object} TagInfo
+ * @typedef {object} TagInfo
  * @property {any} tag
  * @property {any} data
  * @property {TagInfo | undefined} next
  */
 
 /**
- * @typedef {Object} ScopeInfo
+ * @typedef {object} ScopeInfo
  * @property {StackedMap<string, VariableInfo | ScopeInfo>} definitions
  * @property {boolean | "arrow"} topLevelScope
  * @property {boolean | string} inShorthand
@@ -147,6 +148,13 @@ class VariableInfo {
 
 /** @typedef {[number, number]} Range */
 
+/**
+ * @typedef {object} DestructuringAssignmentProperty
+ * @property {string} id
+ * @property {Range | undefined=} range
+ * @property {boolean | string} shorthand
+ */
+
 /**
  * Helper function for joining two ranges into a single range. This is useful
  * when working with AST nodes, as it allows you to combine the ranges of child nodes
@@ -437,7 +445,7 @@ class JavascriptParser extends Parser {
 		this.statementPath = undefined;
 		/** @type {Statement | ModuleDeclaration | Expression | undefined} */
 		this.prevStatement = undefined;
-		/** @type {WeakMap<Expression, Set<string>> | undefined} */
+		/** @type {WeakMap<Expression, Set<DestructuringAssignmentProperty>> | undefined} */
 		this.destructuringAssignmentProperties = undefined;
 		this.currentTagData = undefined;
 		this.magicCommentContext = vm.createContext(undefined, {
@@ -605,7 +613,7 @@ class JavascriptParser extends Parser {
 		 *
 		 * import("./" + foo + bar); // webpack will auto evaluate this into import("./foobar")
 		 * ```
-		 * @param {boolean | number | BigInt | string} value the value to convert to an expression
+		 * @param {boolean | number | bigint | string} value the value to convert to an expression
 		 * @param {BinaryExpression | UnaryExpression} expr the expression being evaluated
 		 * @param {boolean} sideEffects whether the expression has side effects
 		 * @returns {BasicEvaluatedExpression | undefined} the evaluated expression
@@ -663,7 +671,7 @@ class JavascriptParser extends Parser {
 				 * Evaluates a binary expression if and only if it is a const operation (e.g. 1 + 2, "a" + "b", etc.).
 				 *
 				 * @template T
-				 * @param {(leftOperand: T, rightOperand: T) => boolean | number | BigInt | string} operandHandler the handler for the operation (e.g. (a, b) => a + b)
+				 * @param {(leftOperand: T, rightOperand: T) => boolean | number | bigint | string} operandHandler the handler for the operation (e.g. (a, b) => a + b)
 				 * @returns {BasicEvaluatedExpression | undefined} the evaluated expression
 				 */
 				const handleConstOperation = operandHandler => {
@@ -1076,7 +1084,7 @@ class JavascriptParser extends Parser {
 				 * Evaluates a UnaryExpression if and only if it is a basic const operator (e.g. +a, -a, ~a).
 				 *
 				 * @template T
-				 * @param {(operand: T) => boolean | number | BigInt | string} operandHandler handler for the operand
+				 * @param {(operand: T) => boolean | number | bigint | string} operandHandler handler for the operand
 				 * @returns {BasicEvaluatedExpression | undefined} evaluated expression
 				 */
 				const handleConstOperation = operandHandler => {
@@ -1701,7 +1709,7 @@ class JavascriptParser extends Parser {
 
 	/**
 	 * @param {Expression} node node
-	 * @returns {Set<string>|undefined} destructured identifiers
+	 * @returns {Set<DestructuringAssignmentProperty> | undefined} destructured identifiers
 	 */
 	destructuringAssignmentPropertiesFor(node) {
 		if (!this.destructuringAssignmentProperties) return undefined;
@@ -2332,7 +2340,7 @@ class JavascriptParser extends Parser {
 		// check multiple assignments
 		if (this.destructuringAssignmentProperties.has(expression)) {
 			const set =
-				/** @type {Set<string>} */
+				/** @type {Set<DestructuringAssignmentProperty>} */
 				(this.destructuringAssignmentProperties.get(expression));
 			this.destructuringAssignmentProperties.delete(expression);
 			for (const id of set) keys.add(id);
@@ -2627,30 +2635,43 @@ class JavascriptParser extends Parser {
 
 	/**
 	 * @param {ObjectPattern} objectPattern object pattern
-	 * @returns {Set<string> | undefined} set of names or undefined if not all keys are identifiers
+	 * @returns {Set<DestructuringAssignmentProperty> | undefined} set of names or undefined if not all keys are identifiers
 	 */
 	_preWalkObjectPattern(objectPattern) {
-		const ids = new Set();
+		/** @type {Set<DestructuringAssignmentProperty>} */
+		const props = new Set();
 		const properties = objectPattern.properties;
 		for (let i = 0; i < properties.length; i++) {
 			const property = properties[i];
 			if (property.type !== "Property") return;
+			if (property.shorthand && property.value.type === "Identifier") {
+				this.scope.inShorthand = property.value.name;
+			}
 			const key = property.key;
 			if (key.type === "Identifier") {
-				ids.add(key.name);
+				props.add({
+					id: key.name,
+					range: key.range,
+					shorthand: this.scope.inShorthand
+				});
 			} else {
 				const id = this.evaluateExpression(/** @type {TODO} */ (key));
 				const str = id.asString();
 				if (str) {
-					ids.add(str);
+					props.add({
+						id: str,
+						range: key.range,
+						shorthand: this.scope.inShorthand
+					});
 				} else {
 					// could not evaluate key
 					return;
 				}
 			}
+			this.scope.inShorthand = false;
 		}
 
-		return ids;
+		return props;
 	}
 
 	/**

package/lib/library/AbstractLibraryPlugin.js

@@ -26,7 +26,7 @@ const COMMON_LIBRARY_NAME_MESSAGE =
 
 /**
  * @template T
- * @typedef {Object} LibraryContext
+ * @typedef {object} LibraryContext
  * @property {Compilation} compilation
  * @property {ChunkGraph} chunkGraph
  * @property {T} options
@@ -37,7 +37,7 @@ const COMMON_LIBRARY_NAME_MESSAGE =
  */
 class AbstractLibraryPlugin {
 	/**
-	 * @param {Object} options options
+	 * @param {object} options options
 	 * @param {string} options.pluginName name of the plugin
 	 * @param {LibraryType} options.type used library type
 	 */

package/lib/library/AmdLibraryPlugin.js

@@ -21,13 +21,13 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
 
 /**
- * @typedef {Object} AmdLibraryPluginOptions
+ * @typedef {object} AmdLibraryPluginOptions
  * @property {LibraryType} type
  * @property {boolean=} requireAsWrapper
  */
 
 /**
- * @typedef {Object} AmdLibraryPluginParsed
+ * @typedef {object} AmdLibraryPluginParsed
  * @property {string} name
  * @property {string} amdContainer
  */

package/lib/library/AssignLibraryPlugin.js

@@ -91,7 +91,7 @@ const accessWithInit = (accessor, existingLength, initLast = false) => {
 };
 
 /**
- * @typedef {Object} AssignLibraryPluginOptions
+ * @typedef {object} AssignLibraryPluginOptions
  * @property {LibraryType} type
  * @property {string[] | "global"} prefix name prefix
  * @property {string | false} declare declare name as variable
@@ -100,7 +100,7 @@ const accessWithInit = (accessor, existingLength, initLast = false) => {
  */
 
 /**
- * @typedef {Object} AssignLibraryPluginParsed
+ * @typedef {object} AssignLibraryPluginParsed
  * @property {string | string[]} name
  * @property {string | string[] | undefined} export
  */
@@ -323,7 +323,7 @@ class AssignLibraryPlugin extends AbstractLibraryPlugin {
 					true
 				)};\n`
 			);
-			/** @type {String} */
+			/** @type {string} */
 			let exports = RuntimeGlobals.exports;
 			if (exportAccess) {
 				result.add(

package/lib/library/ExportPropertyLibraryPlugin.js

@@ -22,12 +22,12 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
 
 /**
- * @typedef {Object} ExportPropertyLibraryPluginParsed
+ * @typedef {object} ExportPropertyLibraryPluginParsed
  * @property {string | string[]} export
  */
 
 /**
- * @typedef {Object} ExportPropertyLibraryPluginOptions
+ * @typedef {object} ExportPropertyLibraryPluginOptions
  * @property {LibraryType} type
  * @property {boolean} nsObjectUsed the namespace object is used
  */

package/lib/library/JsonpLibraryPlugin.js

@@ -19,12 +19,12 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
 
 /**
- * @typedef {Object} JsonpLibraryPluginOptions
+ * @typedef {object} JsonpLibraryPluginOptions
  * @property {LibraryType} type
  */
 
 /**
- * @typedef {Object} JsonpLibraryPluginParsed
+ * @typedef {object} JsonpLibraryPluginParsed
  * @property {string} name
  */
 

package/lib/library/ModuleLibraryPlugin.js

@@ -23,12 +23,12 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
 
 /**
- * @typedef {Object} ModuleLibraryPluginOptions
+ * @typedef {object} ModuleLibraryPluginOptions
  * @property {LibraryType} type
  */
 
 /**
- * @typedef {Object} ModuleLibraryPluginParsed
+ * @typedef {object} ModuleLibraryPluginParsed
  * @property {string} name
  */
 

package/lib/library/SystemLibraryPlugin.js

@@ -23,12 +23,12 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
 
 /**
- * @typedef {Object} SystemLibraryPluginOptions
+ * @typedef {object} SystemLibraryPluginOptions
  * @property {LibraryType} type
  */
 
 /**
- * @typedef {Object} SystemLibraryPluginParsed
+ * @typedef {object} SystemLibraryPluginParsed
  * @property {string} name
  */
 

package/lib/library/UmdLibraryPlugin.js

@@ -18,8 +18,13 @@ const AbstractLibraryPlugin = require("./AbstractLibraryPlugin");
 /** @typedef {import("../../declarations/WebpackOptions").LibraryType} LibraryType */
 /** @typedef {import("../Compiler")} Compiler */
 /** @typedef {import("../javascript/JavascriptModulesPlugin").RenderContext} RenderContext */
+/** @typedef {import("../ExternalModule").RequestRecord} RequestRecord */
 /** @typedef {import("../util/Hash")} Hash */
-/** @template T @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>} LibraryContext<T> */
+/**
+ * @template T
+ * @typedef {import("./AbstractLibraryPlugin").LibraryContext<T>}
+ * LibraryContext<T>
+ */
 
 /**
  * @param {string[]} accessor the accessor to convert to path
@@ -53,13 +58,13 @@ const accessorAccess = (base, accessor, joinWith = ", ") => {
 /** @typedef {string | string[] | LibraryCustomUmdObject} UmdLibraryPluginName */
 
 /**
- * @typedef {Object} UmdLibraryPluginOptions
+ * @typedef {object} UmdLibraryPluginOptions
  * @property {LibraryType} type
  * @property {boolean=} optionalAmdExternalAsGlobal
  */
 
 /**
- * @typedef {Object} UmdLibraryPluginParsed
+ * @typedef {object} UmdLibraryPluginParsed
  * @property {string | string[]} name
  * @property {LibraryCustomUmdObject} names
  * @property {string | LibraryCustomUmdCommentObject} auxiliaryComment
@@ -158,24 +163,38 @@ class UmdLibraryPlugin extends AbstractLibraryPlugin {
 			});
 		};
 
+		/**
+		 * @param {ExternalModule[]} modules external modules
+		 * @returns {string} result
+		 */
 		const externalsDepsArray = modules => {
 			return `[${replaceKeys(
 				modules
 					.map(m =>
 						JSON.stringify(
-							typeof m.request === "object" ? m.request.amd : m.request
+							typeof m.request === "object"
+								? /** @type {RequestRecord} */
+									(m.request).amd
+								: m.request
 						)
 					)
 					.join(", ")
 			)}]`;
 		};
 
+		/**
+		 * @param {ExternalModule[]} modules external modules
+		 * @returns {string} result
+		 */
 		const externalsRootArray = modules => {
 			return replaceKeys(
 				modules
 					.map(m => {
 						let request = m.request;
-						if (typeof request === "object") request = request.root;
+						if (typeof request === "object")
+							request =
+								/** @type {RequestRecord} */
+								(request).root;
 						return `root${accessorToObjectAccess([].concat(request))}`;
 					})
 					.join(", ")
@@ -194,7 +213,7 @@ class UmdLibraryPlugin extends AbstractLibraryPlugin {
 						let request = m.request;
 						if (typeof request === "object") {
 							request =
-								/** @type {Record<string, string | string[]>} */
+								/** @type {RequestRecord} */
 								(request)[type];
 						}
 						if (request === undefined) {
@@ -233,8 +252,14 @@ class UmdLibraryPlugin extends AbstractLibraryPlugin {
 				.join(", ");
 		};
 
+		/**
+		 * @param {string| string[]} library library name
+		 * @returns {string} stringified library name
+		 */
 		const libraryName = library => {
-			return JSON.stringify(replaceKeys([].concat(library).pop()));
+			return JSON.stringify(
+				replaceKeys(/** @type {string[]} */ ([]).concat(library).pop())
+			);
 		};
 
 		let amdFactory;
@@ -313,11 +338,7 @@ class UmdLibraryPlugin extends AbstractLibraryPlugin {
 							"	else\n" +
 							"		" +
 							replaceKeys(
-								accessorAccess(
-									"root",
-									/** @type {string | string[]} */ (names.root) ||
-										/** @type {string} */ (names.commonjs)
-								)
+								accessorAccess("root", names.root || names.commonjs)
 							) +
 							" = factory(" +
 							externalsRootArray(externals) +

package/lib/logging/createConsoleLogger.js

@@ -15,7 +15,7 @@ const { LogType } = require("./Logger");
 /** @typedef {function(string, LogTypeEnum, any[]): void} LoggingFunction */
 
 /**
- * @typedef {Object} LoggerConsole
+ * @typedef {object} LoggerConsole
  * @property {function(): void} clear
  * @property {function(): void} trace
  * @property {(...args: any[]) => void} info
@@ -33,7 +33,7 @@ const { LogType } = require("./Logger");
  */
 
 /**
- * @typedef {Object} LoggerOptions
+ * @typedef {object} LoggerOptions
  * @property {false|true|"none"|"error"|"warn"|"info"|"log"|"verbose"} level loglevel
  * @property {FilterTypes|boolean} debug filter for debug logging
  * @property {LoggerConsole} console the console to log to

package/lib/node/CommonJsChunkLoadingPlugin.js

@@ -11,7 +11,8 @@ const StartupChunkDependenciesPlugin = require("../runtime/StartupChunkDependenc
 /** @typedef {import("../Chunk")} Chunk */
 /** @typedef {import("../Compiler")} Compiler */
 
-/** @typedef {Object} CommonJsChunkLoadingPluginOptions
+/**
+ * @typedef {object} CommonJsChunkLoadingPluginOptions
  * @property {boolean} [asyncChunkLoading] enable async chunk loading
  */
 

package/lib/node/NodeEnvironmentPlugin.js

@@ -17,7 +17,7 @@ const nodeConsole = require("./nodeConsole");
 
 class NodeEnvironmentPlugin {
 	/**
-	 * @param {Object} options options
+	 * @param {object} options options
 	 * @param {InfrastructureLogging} options.infrastructureLogging infrastructure logging options
 	 */
 	constructor(options) {

package/lib/node/NodeTemplatePlugin.js

@@ -11,7 +11,7 @@ const EnableChunkLoadingPlugin = require("../javascript/EnableChunkLoadingPlugin
 /** @typedef {import("../Compiler")} Compiler */
 
 /**
- * @typedef {Object} NodeTemplatePluginOptions
+ * @typedef {object} NodeTemplatePluginOptions
  * @property {boolean} [asyncChunkLoading] enable async chunk loading
  */
 

package/lib/node/ReadFileCompileWasmPlugin.js

@@ -14,7 +14,7 @@ const WasmChunkLoadingRuntimeModule = require("../wasm-sync/WasmChunkLoadingRunt
 /** @typedef {import("../Compiler")} Compiler */
 
 /**
- * @typedef {Object} ReadFileCompileWasmPluginOptions
+ * @typedef {object} ReadFileCompileWasmPluginOptions
  * @property {boolean} [mangleImports] mangle imports
  */
 

package/lib/node/nodeConsole.js

@@ -11,7 +11,7 @@ const truncateArgs = require("../logging/truncateArgs");
 /** @typedef {import("../logging/createConsoleLogger").LoggerConsole} LoggerConsole */
 
 /**
- * @param {Object} options options
+ * @param {object} options options
  * @param {boolean=} options.colors colors
  * @param {boolean=} options.appendOnly append only
  * @param {NodeJS.WritableStream} options.stream stream

package/lib/optimize/AggressiveMergingPlugin.js

@@ -11,7 +11,7 @@ const { STAGE_ADVANCED } = require("../OptimizationStages");
 /** @typedef {import("../Compiler")} Compiler */
 
 /**
- * @typedef {Object} AggressiveMergingPluginOptions
+ * @typedef {object} AggressiveMergingPluginOptions
  * @property {number=} minSizeReduce minimal size reduction to trigger merging
  */
 

package/lib/optimize/ConcatenatedModule.js

@@ -26,6 +26,7 @@ const { concatComparators } = require("../util/comparators");
 const createHash = require("../util/createHash");
 const { makePathsRelative } = require("../util/identifier");
 const makeSerializable = require("../util/makeSerializable");
+const { getAllReferences, getPathInAst } = require("../util/mergeScope");
 const propertyAccess = require("../util/propertyAccess");
 const { propertyName } = require("../util/propertyName");
 const {
@@ -92,7 +93,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
 }
 
 /**
- * @typedef {Object} ReexportInfo
+ * @typedef {object} ReexportInfo
  * @property {Module} module
  * @property {string[]} export
  */
@@ -100,7 +101,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
 /** @typedef {RawBinding | SymbolBinding} Binding */
 
 /**
- * @typedef {Object} RawBinding
+ * @typedef {object} RawBinding
  * @property {ModuleInfo} info
  * @property {string} rawName
  * @property {string=} comment
@@ -109,7 +110,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
  */
 
 /**
- * @typedef {Object} SymbolBinding
+ * @typedef {object} SymbolBinding
  * @property {ConcatenatedModuleInfo} info
  * @property {string} name
  * @property {string=} comment
@@ -121,7 +122,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
 /** @typedef {ConcatenatedModuleInfo | ExternalModuleInfo | ReferenceToModuleInfo } ModuleInfoOrReference */
 
 /**
- * @typedef {Object} ConcatenatedModuleInfo
+ * @typedef {object} ConcatenatedModuleInfo
  * @property {"concatenated"} type
  * @property {Module} module
  * @property {number} index
@@ -146,7 +147,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
  */
 
 /**
- * @typedef {Object} ExternalModuleInfo
+ * @typedef {object} ExternalModuleInfo
  * @property {"external"} type
  * @property {Module} module
  * @property {RuntimeSpec | boolean} runtimeCondition
@@ -161,7 +162,7 @@ if (!ReferencerClass.prototype.PropertyDefinition) {
  */
 
 /**
- * @typedef {Object} ReferenceToModuleInfo
+ * @typedef {object} ReferenceToModuleInfo
  * @property {"reference"} type
  * @property {RuntimeSpec | boolean} runtimeCondition
  * @property {ConcatenatedModuleInfo | ExternalModuleInfo} target
@@ -254,7 +255,7 @@ const joinIterableWithComma = iterable => {
 };
 
 /**
- * @typedef {Object} ConcatenationEntry
+ * @typedef {object} ConcatenationEntry
  * @property {"concatenated" | "external"} type
  * @property {Module} module
  * @property {RuntimeSpec | boolean} runtimeCondition
@@ -625,72 +626,6 @@ const addScopeSymbols = (s, nameSet, scopeSet1, scopeSet2) => {
 	}
 };
 
-/**
- * @param {Variable} variable variable
- * @returns {Reference[]} references
- */
-const getAllReferences = variable => {
-	let set = variable.references;
-	// Look for inner scope variables too (like in class Foo { t() { Foo } })
-	const identifiers = new Set(variable.identifiers);
-	for (const scope of variable.scope.childScopes) {
-		for (const innerVar of scope.variables) {
-			if (innerVar.identifiers.some(id => identifiers.has(id))) {
-				set = set.concat(innerVar.references);
-				break;
-			}
-		}
-	}
-	return set;
-};
-
-/**
- * @param {TODO} ast ast
- * @param {TODO} node node
- * @returns {TODO} result
- */
-const getPathInAst = (ast, node) => {
-	if (ast === node) {
-		return [];
-	}
-
-	const nr = node.range;
-
-	const enterNode = n => {
-		if (!n) return undefined;
-		const r = n.range;
-		if (r) {
-			if (r[0] <= nr[0] && r[1] >= nr[1]) {
-				const path = getPathInAst(n, node);
-				if (path) {
-					path.push(n);
-					return path;
-				}
-			}
-		}
-		return undefined;
-	};
-
-	if (Array.isArray(ast)) {
-		for (let i = 0; i < ast.length; i++) {
-			const enterResult = enterNode(ast[i]);
-			if (enterResult !== undefined) return enterResult;
-		}
-	} else if (ast && typeof ast === "object") {
-		const keys = Object.keys(ast);
-		for (let i = 0; i < keys.length; i++) {
-			const value = ast[keys[i]];
-			if (Array.isArray(value)) {
-				const pathResult = getPathInAst(value, node);
-				if (pathResult !== undefined) return pathResult;
-			} else if (value && typeof value === "object") {
-				const enterResult = enterNode(value);
-				if (enterResult !== undefined) return enterResult;
-			}
-		}
-	}
-};
-
 const TYPES = new Set(["javascript"]);
 
 class ConcatenatedModule extends Module {
@@ -698,7 +633,7 @@ class ConcatenatedModule extends Module {
 	 * @param {Module} rootModule the root module of the concatenation
 	 * @param {Set<Module>} modules all modules in the concatenation (including the root module)
 	 * @param {RuntimeSpec} runtime the runtime
-	 * @param {Object=} associatedObjectForCache object for caching
+	 * @param {object=} associatedObjectForCache object for caching
 	 * @param {string | HashConstructor=} hashFunction hash function to use
 	 * @returns {ConcatenatedModule} the module
 	 */
@@ -724,7 +659,7 @@ class ConcatenatedModule extends Module {
 	}
 
 	/**
-	 * @param {Object} options options
+	 * @param {object} options options
 	 * @param {string} options.identifier the identifier of the module
 	 * @param {Module=} options.rootModule the root module of the concatenation
 	 * @param {RuntimeSpec} options.runtime the selected runtime
@@ -1098,7 +1033,7 @@ class ConcatenatedModule extends Module {
 	/**
 	 * @param {Module} rootModule the root module of the concatenation
 	 * @param {Set<Module>} modules all modules in the concatenation (including the root module)
-	 * @param {Object=} associatedObjectForCache object for caching
+	 * @param {object=} associatedObjectForCache object for caching
 	 * @param {string | HashConstructor=} hashFunction hash function to use
 	 * @returns {string} the identifier
 	 */

package/lib/optimize/InnerGraph.js

@@ -21,7 +21,7 @@ const { UsageState } = require("../ExportsInfo");
 /** @typedef {function(boolean | Set<string> | undefined): void} UsageCallback */
 
 /**
- * @typedef {Object} StateObject
+ * @typedef {object} StateObject
  * @property {InnerGraph} innerGraph
  * @property {TopLevelSymbol=} currentTopLevelSymbol
  * @property {Map<TopLevelSymbol, Set<UsageCallback>>} usageCallbackMap