webpack 5.93.0 → 5.94.0

package/lib/dependencies/CommonJsExportsParserPlugin.js

@@ -43,7 +43,6 @@ const ModuleDecoratorDependency = require("./ModuleDecoratorDependency");
  * exports.foo = void 0;
  * exports.foo = "bar";
  * ```
- *
  * @param {TODO} expr expression
  * @returns {Expression | undefined} returns the value of property descriptor
  */
@@ -61,15 +60,13 @@ const getValueOfPropertyDescription = expr => {
  * The purpose of this function is to check whether an expression is a truthy literal or not. This is
  * useful when parsing CommonJS exports, because CommonJS modules can export any value, including falsy
  * values like `null` and `false`. However, exports should only be created if the exported value is truthy.
- *
  * @param {Expression} expr expression being checked
  * @returns {boolean} true, when the expression is a truthy literal
- *
  */
 const isTruthyLiteral = expr => {
 	switch (expr.type) {
 		case "Literal":
-			return !!expr.value;
+			return Boolean(expr.value);
 		case "UnaryExpression":
 			if (expr.operator === "!") return isFalsyLiteral(expr.argument);
 	}
@@ -80,7 +77,6 @@ const isTruthyLiteral = expr => {
  * The purpose of this function is to check whether an expression is a falsy literal or not. This is
  * useful when parsing CommonJS exports, because CommonJS modules can export any value, including falsy
  * values like `null` and `false`. However, exports should only be created if the exported value is truthy.
- *
  * @param {Expression} expr expression being checked
  * @returns {boolean} true, when the expression is a falsy literal
  */
@@ -215,7 +211,7 @@ class CommonJsExportsParserPlugin {
 					!parser.isStatementLevelExpression(expr)
 				);
 				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-				dep.optional = !!parser.scope.inTry;
+				dep.optional = Boolean(parser.scope.inTry);
 				parser.state.module.addDependency(dep);
 				return true;
 			}
@@ -241,9 +237,9 @@ class CommonJsExportsParserPlugin {
 		};
 		parser.hooks.assignMemberChain
 			.for("exports")
-			.tap("CommonJsExportsParserPlugin", (expr, members) => {
-				return handleAssignExport(expr, "exports", members);
-			});
+			.tap("CommonJsExportsParserPlugin", (expr, members) =>
+				handleAssignExport(expr, "exports", members)
+			);
 		parser.hooks.assignMemberChain
 			.for("this")
 			.tap("CommonJsExportsParserPlugin", (expr, members) => {
@@ -306,7 +302,7 @@ class CommonJsExportsParserPlugin {
 		 * @param {CallExpression=} call call expression
 		 * @returns {boolean | void} true, when the expression was handled
 		 */
-		const handleAccessExport = (expr, base, members, call = undefined) => {
+		const handleAccessExport = (expr, base, members, call) => {
 			if (HarmonyExports.isEnabled(parser.state)) return;
 			if (members.length === 0) {
 				bailout(
@@ -328,7 +324,7 @@ class CommonJsExportsParserPlugin {
 				/** @type {Range} */ (expr.range),
 				base,
 				members,
-				!!call
+				Boolean(call)
 			);
 			dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 			parser.state.module.addDependency(dep);
@@ -339,19 +335,19 @@ class CommonJsExportsParserPlugin {
 		};
 		parser.hooks.callMemberChain
 			.for("exports")
-			.tap("CommonJsExportsParserPlugin", (expr, members) => {
-				return handleAccessExport(expr.callee, "exports", members, expr);
-			});
+			.tap("CommonJsExportsParserPlugin", (expr, members) =>
+				handleAccessExport(expr.callee, "exports", members, expr)
+			);
 		parser.hooks.expressionMemberChain
 			.for("exports")
-			.tap("CommonJsExportsParserPlugin", (expr, members) => {
-				return handleAccessExport(expr, "exports", members);
-			});
+			.tap("CommonJsExportsParserPlugin", (expr, members) =>
+				handleAccessExport(expr, "exports", members)
+			);
 		parser.hooks.expression
 			.for("exports")
-			.tap("CommonJsExportsParserPlugin", expr => {
-				return handleAccessExport(expr, "exports", []);
-			});
+			.tap("CommonJsExportsParserPlugin", expr =>
+				handleAccessExport(expr, "exports", [])
+			);
 		parser.hooks.callMemberChain
 			.for("module")
 			.tap("CommonJsExportsParserPlugin", (expr, members) => {
@@ -371,9 +367,9 @@ class CommonJsExportsParserPlugin {
 			});
 		parser.hooks.expression
 			.for("module.exports")
-			.tap("CommonJsExportsParserPlugin", expr => {
-				return handleAccessExport(expr, "module.exports", []);
-			});
+			.tap("CommonJsExportsParserPlugin", expr =>
+				handleAccessExport(expr, "module.exports", [])
+			);
 		parser.hooks.callMemberChain
 			.for("this")
 			.tap("CommonJsExportsParserPlugin", (expr, members) => {

package/lib/dependencies/CommonJsFullRequireDependency.js

@@ -146,7 +146,7 @@ CommonJsFullRequireDependency.Template = class CommonJsFullRequireDependencyTemp
 			if (usedImported) {
 				const comment = equals(usedImported, trimmedIds)
 					? ""
-					: Template.toNormalComment(propertyAccess(trimmedIds)) + " ";
+					: `${Template.toNormalComment(propertyAccess(trimmedIds))} `;
 				const access = `${comment}${propertyAccess(usedImported)}`;
 				requireExpr =
 					dep.asiSafe === true

package/lib/dependencies/CommonJsImportsParserPlugin.js

@@ -63,7 +63,7 @@ class CommonJsImportsParserPlugin {
 			}
 		};
 
-		//#region metadata
+		// #region metadata
 		/**
 		 * @param {string} expression expression
 		 * @param {() => string[]} getMembers get members
@@ -102,7 +102,7 @@ class CommonJsImportsParserPlugin {
 		tapRequireExpression("require", () => []);
 		tapRequireExpression("require.resolve", () => ["resolve"]);
 		tapRequireExpression("require.resolveWeak", () => ["resolveWeak"]);
-		//#endregion
+		// #endregion
 
 		// Weird stuff //
 		parser.hooks.assign
@@ -115,7 +115,7 @@ class CommonJsImportsParserPlugin {
 				return true;
 			});
 
-		//#region Unsupported
+		// #region Unsupported
 		parser.hooks.expression
 			.for("require.main")
 			.tap(
@@ -152,9 +152,9 @@ class CommonJsImportsParserPlugin {
 					"module.parent.require is not supported by webpack."
 				)
 			);
-		//#endregion
+		// #endregion
 
-		//#region Renaming
+		// #region Renaming
 		/**
 		 * @param {Expression} expr expression
 		 * @returns {boolean} true when set undefined
@@ -175,9 +175,9 @@ class CommonJsImportsParserPlugin {
 		parser.hooks.rename
 			.for("require")
 			.tap("CommonJsImportsParserPlugin", defineUndefined);
-		//#endregion
+		// #endregion
 
-		//#region Inspection
+		// #region Inspection
 		const requireCache = toConstantDependency(
 			parser,
 			RuntimeGlobals.moduleCache,
@@ -191,9 +191,9 @@ class CommonJsImportsParserPlugin {
 		parser.hooks.expression
 			.for("require.cache")
 			.tap("CommonJsImportsParserPlugin", requireCache);
-		//#endregion
+		// #endregion
 
-		//#region Require as expression
+		// #region Require as expression
 		/**
 		 * @param {Expression} expr expression
 		 * @returns {boolean} true when handled
@@ -215,16 +215,16 @@ class CommonJsImportsParserPlugin {
 				options.unknownContextCritical &&
 				"require function is used in a way in which dependencies cannot be statically extracted";
 			dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-			dep.optional = !!parser.scope.inTry;
+			dep.optional = Boolean(parser.scope.inTry);
 			parser.state.current.addDependency(dep);
 			return true;
 		};
 		parser.hooks.expression
 			.for("require")
 			.tap("CommonJsImportsParserPlugin", requireAsExpressionHandler);
-		//#endregion
+		// #endregion
 
-		//#region Require
+		// #region Require
 		/**
 		 * @param {CallExpression | NewExpression} expr expression
 		 * @param {BasicEvaluatedExpression} param param
@@ -238,7 +238,7 @@ class CommonJsImportsParserPlugin {
 					getContext()
 				);
 				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-				dep.optional = !!parser.scope.inTry;
+				dep.optional = Boolean(parser.scope.inTry);
 				parser.state.current.addDependency(dep);
 				return true;
 			}
@@ -264,7 +264,7 @@ class CommonJsImportsParserPlugin {
 			);
 			if (!dep) return;
 			dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-			dep.optional = !!parser.scope.inTry;
+			dep.optional = Boolean(parser.scope.inTry);
 			parser.state.current.addDependency(dep);
 			return true;
 		};
@@ -288,21 +288,17 @@ class CommonJsImportsParserPlugin {
 						);
 					}
 				}
-				if (requireOptions) {
-					if (requireOptions.webpackIgnore !== undefined) {
-						if (typeof requireOptions.webpackIgnore !== "boolean") {
-							parser.state.module.addWarning(
-								new UnsupportedFeatureWarning(
-									`\`webpackIgnore\` expected a boolean, but received: ${requireOptions.webpackIgnore}.`,
-									/** @type {DependencyLocation} */ (expr.loc)
-								)
-							);
-						} else {
-							// Do not instrument `require()` if `webpackIgnore` is `true`
-							if (requireOptions.webpackIgnore) {
-								return true;
-							}
-						}
+				if (requireOptions && requireOptions.webpackIgnore !== undefined) {
+					if (typeof requireOptions.webpackIgnore !== "boolean") {
+						parser.state.module.addWarning(
+							new UnsupportedFeatureWarning(
+								`\`webpackIgnore\` expected a boolean, but received: ${requireOptions.webpackIgnore}.`,
+								/** @type {DependencyLocation} */ (expr.loc)
+							)
+						);
+					} else if (requireOptions.webpackIgnore) {
+						// Do not instrument `require()` if `webpackIgnore` is `true`
+						return true;
 					}
 				}
 			}
@@ -370,9 +366,9 @@ class CommonJsImportsParserPlugin {
 		parser.hooks.new
 			.for("module.require")
 			.tap("CommonJsImportsParserPlugin", createRequireHandler(true));
-		//#endregion
+		// #endregion
 
-		//#region Require with property access
+		// #region Require with property access
 		/**
 		 * @param {Expression} expr expression
 		 * @param {string[]} calleeMembers callee members
@@ -403,7 +399,7 @@ class CommonJsImportsParserPlugin {
 				dep.asiSafe = !parser.isAsiPosition(
 					/** @type {Range} */ (expr.range)[0]
 				);
-				dep.optional = !!parser.scope.inTry;
+				dep.optional = Boolean(parser.scope.inTry);
 				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 				parser.state.current.addDependency(dep);
 				return true;
@@ -440,7 +436,7 @@ class CommonJsImportsParserPlugin {
 				dep.asiSafe = !parser.isAsiPosition(
 					/** @type {Range} */ (expr.range)[0]
 				);
-				dep.optional = !!parser.scope.inTry;
+				dep.optional = Boolean(parser.scope.inTry);
 				dep.loc = /** @type {DependencyLocation} */ (expr.callee.loc);
 				parser.state.current.addDependency(dep);
 				parser.walkExpressions(expr.arguments);
@@ -459,9 +455,9 @@ class CommonJsImportsParserPlugin {
 		parser.hooks.callMemberChainOfCallMemberChain
 			.for("module.require")
 			.tap("CommonJsImportsParserPlugin", callChainHandler);
-		//#endregion
+		// #endregion
 
-		//#region Require.resolve
+		// #region Require.resolve
 		/**
 		 * @param {CallExpression} expr call expression
 		 * @param {boolean} weak weak
@@ -485,18 +481,17 @@ class CommonJsImportsParserPlugin {
 				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 				parser.state.module.addPresentationalDependency(dep);
 				return true;
-			} else {
-				const result = processResolveItem(expr, param, weak);
-				if (result === undefined) {
-					processResolveContext(expr, param, weak);
-				}
-				const dep = new RequireResolveHeaderDependency(
-					/** @type {Range} */ (expr.callee.range)
-				);
-				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-				parser.state.module.addPresentationalDependency(dep);
-				return true;
 			}
+			const result = processResolveItem(expr, param, weak);
+			if (result === undefined) {
+				processResolveContext(expr, param, weak);
+			}
+			const dep = new RequireResolveHeaderDependency(
+				/** @type {Range} */ (expr.callee.range)
+			);
+			dep.loc = /** @type {DependencyLocation} */ (expr.loc);
+			parser.state.module.addPresentationalDependency(dep);
+			return true;
 		};
 		/**
 		 * @param {CallExpression} expr call expression
@@ -512,7 +507,7 @@ class CommonJsImportsParserPlugin {
 					getContext()
 				);
 				dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-				dep.optional = !!parser.scope.inTry;
+				dep.optional = Boolean(parser.scope.inTry);
 				dep.weak = weak;
 				parser.state.current.addDependency(dep);
 				return true;
@@ -540,24 +535,20 @@ class CommonJsImportsParserPlugin {
 			);
 			if (!dep) return;
 			dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-			dep.optional = !!parser.scope.inTry;
+			dep.optional = Boolean(parser.scope.inTry);
 			parser.state.current.addDependency(dep);
 			return true;
 		};
 
 		parser.hooks.call
 			.for("require.resolve")
-			.tap("CommonJsImportsParserPlugin", expr => {
-				return processResolve(expr, false);
-			});
+			.tap("CommonJsImportsParserPlugin", expr => processResolve(expr, false));
 		parser.hooks.call
 			.for("require.resolveWeak")
-			.tap("CommonJsImportsParserPlugin", expr => {
-				return processResolve(expr, true);
-			});
-		//#endregion
+			.tap("CommonJsImportsParserPlugin", expr => processResolve(expr, true));
+		// #endregion
 
-		//#region Create require
+		// #region Create require
 
 		if (!options.createRequire) return;
 
@@ -610,12 +601,12 @@ class CommonJsImportsParserPlugin {
 			});
 		parser.hooks.unhandledExpressionMemberChain
 			.for(createdRequireIdentifierTag)
-			.tap("CommonJsImportsParserPlugin", (expr, members) => {
-				return expressionIsUnsupported(
+			.tap("CommonJsImportsParserPlugin", (expr, members) =>
+				expressionIsUnsupported(
 					parser,
 					`createRequire().${members.join(".")} is not supported by webpack.`
-				)(expr);
-			});
+				)(expr)
+			);
 		parser.hooks.canRename
 			.for(createdRequireIdentifierTag)
 			.tap("CommonJsImportsParserPlugin", () => true);
@@ -789,7 +780,7 @@ class CommonJsImportsParserPlugin {
 				parser.state.module.addPresentationalDependency(clearDep);
 				return true;
 			});
-		//#endregion
+		// #endregion
 	}
 }
 module.exports = CommonJsImportsParserPlugin;

package/lib/dependencies/CommonJsSelfReferenceDependency.js

@@ -50,7 +50,7 @@ class CommonJsSelfReferenceDependency extends NullDependency {
 	 * @returns {string | null} an identifier to merge equal requests
 	 */
 	getResourceIdentifier() {
-		return `self`;
+		return "self";
 	}
 
 	/**
@@ -108,19 +108,17 @@ CommonJsSelfReferenceDependency.Template = class CommonJsSelfReferenceDependency
 		{ module, moduleGraph, runtime, runtimeRequirements }
 	) {
 		const dep = /** @type {CommonJsSelfReferenceDependency} */ (dependency);
-		let used;
-		if (dep.names.length === 0) {
-			used = dep.names;
-		} else {
-			used = moduleGraph.getExportsInfo(module).getUsedName(dep.names, runtime);
-		}
+		const used =
+			dep.names.length === 0
+				? dep.names
+				: moduleGraph.getExportsInfo(module).getUsedName(dep.names, runtime);
 		if (!used) {
 			throw new Error(
 				"Self-reference dependency has unused export name: This should not happen"
 			);
 		}
 
-		let base = undefined;
+		let base;
 		switch (dep.base) {
 			case "exports":
 				runtimeRequirements.add(RuntimeGlobals.exports);

package/lib/dependencies/ConstDependency.js

@@ -44,7 +44,7 @@ class ConstDependency extends NullDependency {
 	 */
 	updateHash(hash, context) {
 		if (this._hashUpdate === undefined) {
-			let hashUpdate = "" + this.range + "|" + this.expression;
+			let hashUpdate = `${this.range}|${this.expression}`;
 			if (this.runtimeRequirements) {
 				for (const item of this.runtimeRequirements) {
 					hashUpdate += "|";

package/lib/dependencies/ContextDependency.js

@@ -27,7 +27,7 @@ const getCriticalDependencyWarning = memoize(() =>
  * @param {RegExp | null | undefined} r regexp
  * @returns {string} stringified regexp
  */
-const regExpToString = r => (r ? r + "" : "");
+const regExpToString = r => (r ? String(r) : "");
 
 class ContextDependency extends Dependency {
 	/**

package/lib/dependencies/ContextDependencyHelpers.js

@@ -22,9 +22,7 @@ const { parseResource } = require("../util/identifier");
  * @param {string} str String to quote
  * @returns {string} Escaped string
  */
-const quoteMeta = str => {
-	return str.replace(/[-[\]\\/{}()*+?.^$|]/g, "\\$&");
-};
+const quoteMeta = str => str.replace(/[-[\]\\/{}()*+?.^$|]/g, "\\$&");
 
 /**
  * @param {string} prefix prefix
@@ -57,7 +55,7 @@ const splitContextFromPrefix = prefix => {
  * @param {...any} depArgs depArgs
  * @returns {ContextDependency} the created Dependency
  */
-exports.create = (
+module.exports.create = (
 	Dep,
 	range,
 	param,
@@ -69,8 +67,8 @@ exports.create = (
 ) => {
 	if (param.isTemplateString()) {
 		const quasis = /** @type {BasicEvaluatedExpression[]} */ (param.quasis);
-		let prefixRaw = /** @type {string} */ (quasis[0].string);
-		let postfixRaw =
+		const prefixRaw = /** @type {string} */ (quasis[0].string);
+		const postfixRaw =
 			/** @type {string} */
 			(quasis.length > 1 ? quasis[quasis.length - 1].string : "");
 
@@ -84,7 +82,7 @@ exports.create = (
 
 		// When there are more than two quasis, the generated RegExp can be more precise
 		// We join the quasis with the expression regexp
-		const innerQuasis = quasis.slice(1, quasis.length - 1);
+		const innerQuasis = quasis.slice(1, -1);
 		const innerRegExp =
 			/** @type {RegExp} */ (options.wrappedContextRegExp).source +
 			innerQuasis
@@ -125,14 +123,14 @@ exports.create = (
 		const replaces = [];
 		const parts = /** @type {BasicEvaluatedExpression[]} */ (param.parts);
 
-		parts.forEach((part, i) => {
+		for (const [i, part] of parts.entries()) {
 			if (i % 2 === 0) {
 				// Quasis or merged quasi
 				let range = /** @type {Range} */ (part.range);
 				let value = /** @type {string} */ (part.string);
 				if (param.templateStringKind === "cooked") {
 					value = JSON.stringify(value);
-					value = value.slice(1, value.length - 1);
+					value = value.slice(1, -1);
 				}
 				if (i === 0) {
 					// prefix
@@ -151,14 +149,14 @@ exports.create = (
 						/** @type {Range} */ (part.range)[0],
 						/** @type {Range} */ (param.range)[1]
 					];
-					value = value + "`";
+					value = `${value}\``;
 				} else if (
 					part.expression &&
 					part.expression.type === "TemplateElement" &&
 					part.expression.value.raw === value
 				) {
 					// Shortcut when it's a single quasi and doesn't need to be replaced
-					return;
+					continue;
 				}
 				replaces.push({
 					range,
@@ -168,7 +166,7 @@ exports.create = (
 				// Expression
 				parser.walkExpression(part.expression);
 			}
-		});
+		}
 
 		dep.replaces = replaces;
 		dep.critical =
@@ -180,10 +178,10 @@ exports.create = (
 		((param.prefix && param.prefix.isString()) ||
 			(param.postfix && param.postfix.isString()))
 	) {
-		let prefixRaw =
+		const prefixRaw =
 			/** @type {string} */
 			(param.prefix && param.prefix.isString() ? param.prefix.string : "");
-		let postfixRaw =
+		const postfixRaw =
 			/** @type {string} */
 			(param.postfix && param.postfix.isString() ? param.postfix.string : "");
 		const prefixRange =
@@ -240,26 +238,25 @@ exports.create = (
 		}
 
 		return dep;
-	} else {
-		const dep = new Dep(
-			{
-				request: /** @type {string} */ (options.exprContextRequest),
-				recursive: /** @type {boolean} */ (options.exprContextRecursive),
-				regExp: /** @type {RegExp} */ (options.exprContextRegExp),
-				mode: "sync",
-				...contextOptions
-			},
-			range,
-			/** @type {Range} */ (param.range),
-			...depArgs
-		);
-		dep.loc = /** @type {DependencyLocation} */ (expr.loc);
-		dep.critical =
-			options.exprContextCritical &&
-			"the request of a dependency is an expression";
+	}
+	const dep = new Dep(
+		{
+			request: /** @type {string} */ (options.exprContextRequest),
+			recursive: /** @type {boolean} */ (options.exprContextRecursive),
+			regExp: /** @type {RegExp} */ (options.exprContextRegExp),
+			mode: "sync",
+			...contextOptions
+		},
+		range,
+		/** @type {Range} */ (param.range),
+		...depArgs
+	);
+	dep.loc = /** @type {DependencyLocation} */ (expr.loc);
+	dep.critical =
+		options.exprContextCritical &&
+		"the request of a dependency is an expression";
 
-		parser.walkExpression(param.expression);
+	parser.walkExpression(param.expression);
 
-		return dep;
-	}
+	return dep;
 };

package/lib/dependencies/ContextElementDependency.js

@@ -20,9 +20,9 @@ class ContextElementDependency extends ModuleDependency {
 	/**
 	 * @param {string} request request
 	 * @param {string|undefined} userRequest user request
-	 * @param {string} typePrefix type prefix
+	 * @param {string | undefined} typePrefix type prefix
 	 * @param {string} category category
-	 * @param {string[][]=} referencedExports referenced exports
+	 * @param {(string[][] | null)=} referencedExports referenced exports
 	 * @param {string=} context context
 	 * @param {ImportAttributes=} attributes import assertions
 	 */

package/lib/dependencies/CriticalDependencyWarning.js

@@ -16,7 +16,7 @@ class CriticalDependencyWarning extends WebpackError {
 		super();
 
 		this.name = "CriticalDependencyWarning";
-		this.message = "Critical dependency: " + message;
+		this.message = `Critical dependency: ${message}`;
 	}
 }
 

package/lib/dependencies/CssExportDependency.js

@@ -58,9 +58,9 @@ class CssExportDependency extends NullDependency {
 	 */
 	getExports(moduleGraph) {
 		const module = /** @type {CssModule} */ (moduleGraph.getParentModule(this));
-		const convention = /** @type {CssGenerator | CssExportsGenerator} */ (
-			module.generator
-		).convention;
+		const convention =
+			/** @type {CssGenerator | CssExportsGenerator} */
+			(module.generator).convention;
 		const names = this.getExportsConventionNames(this.name, convention);
 		return {
 			exports: names.map(name => ({
@@ -81,14 +81,14 @@ class CssExportDependency extends NullDependency {
 		const module = /** @type {CssModule} */ (
 			chunkGraph.moduleGraph.getParentModule(this)
 		);
-		const generator = /** @type {CssGenerator | CssExportsGenerator} */ (
-			module.generator
-		);
+		const generator =
+			/** @type {CssGenerator | CssExportsGenerator} */
+			(module.generator);
 		const names = this.getExportsConventionNames(
 			this.name,
 			generator.convention
 		);
-		hash.update(`exportsConvention`);
+		hash.update("exportsConvention");
 		hash.update(JSON.stringify(names));
 	}
 

package/lib/dependencies/CssImportDependency.js

@@ -26,9 +26,7 @@ const ModuleDependency = require("./ModuleDependency");
 class CssImportDependency extends ModuleDependency {
 	/**
 	 * Example of dependency:
-	 *
 	 * \@import url("landscape.css") layer(forms) screen and (orientation: landscape) screen and (orientation: landscape);
-	 *
 	 * @param {string} request request
 	 * @param {Range} range range of the argument
 	 * @param {string | undefined} layer layer

package/lib/dependencies/CssLocalIdentifierDependency.js

@@ -141,9 +141,9 @@ class CssLocalIdentifierDependency extends NullDependency {
 			this.name,
 			generator.convention
 		);
-		hash.update(`exportsConvention`);
+		hash.update("exportsConvention");
 		hash.update(JSON.stringify(names));
-		hash.update(`localIdentName`);
+		hash.update("localIdentName");
 		hash.update(generator.localIdentName);
 	}
 
@@ -178,7 +178,7 @@ class CssLocalIdentifierDependency extends NullDependency {
 const escapeCssIdentifier = (str, omitUnderscore) => {
 	const escaped = `${str}`.replace(
 		// cspell:word uffff
-		/[^a-zA-Z0-9_\u0081-\uffff-]/g,
+		/[^a-zA-Z0-9_\u0081-\uFFFF-]/g,
 		s => `\\${s}`
 	);
 	return !omitUnderscore && /^(?!--)[0-9-]/.test(escaped)