@depup/webpack 5.111.1-depup.2 → 5.112.0-depup.1

package/lib/dependencies/{AMDRequireDependency.js → amd/AMDRequireDependency.js}

@@ -5,17 +5,17 @@
 
 "use strict";
 
-const RuntimeGlobals = require("../RuntimeGlobals");
-const makeSerializable = require("../util/makeSerializable");
-const NullDependency = require("./NullDependency");
+const RuntimeGlobals = require("../../runtime/RuntimeGlobals");
+const makeSerializable = require("../../util/makeSerializable");
+const NullDependency = require("../core/NullDependency");
 
 /** @import { ReplaceSource } from "webpack-sources" */
-/** @import AsyncDependenciesBlock from "../AsyncDependenciesBlock" */
-/** @import Dependency from "../Dependency" */
-/** @import { DependencyTemplateContext } from "../DependencyTemplate" */
-/** @import { Range } from "../javascript/JavascriptParser" */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[Range, Range, Range | null, Range | null, boolean, boolean]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[Range, Range, Range | null, Range | null, boolean, boolean]>} ObjectSerializerContext */
+/** @import AsyncDependenciesBlock from "../../graph/AsyncDependenciesBlock" */
+/** @import Dependency from "../../graph/Dependency" */
+/** @import { DependencyTemplateContext } from "../../template/DependencyTemplate" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[Range, Range, Range | null, Range | null, boolean, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[Range, Range, Range | null, Range | null, boolean, boolean]>} ObjectSerializerContext */
 
 class AMDRequireDependency extends NullDependency {
 	/**
@@ -79,10 +79,11 @@ class AMDRequireDependency extends NullDependency {
 	}
 }
 
-makeSerializable(
-	AMDRequireDependency,
+makeSerializable(AMDRequireDependency, [
+	"webpack/lib/dependencies/amd/AMDRequireDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/AMDRequireDependency"
-);
+]);
 
 AMDRequireDependency.Template = class AMDRequireDependencyTemplate extends (
 	NullDependency.Template

package/lib/dependencies/{AMDRequireItemDependency.js → amd/AMDRequireItemDependency.js}

@@ -5,11 +5,11 @@
 
 "use strict";
 
-const makeSerializable = require("../util/makeSerializable");
-const ModuleDependency = require("./ModuleDependency");
-const ModuleDependencyTemplateAsRequireId = require("./ModuleDependencyTemplateAsRequireId");
+const makeSerializable = require("../../util/makeSerializable");
+const ModuleDependency = require("../core/ModuleDependency");
+const ModuleDependencyTemplateAsRequireId = require("../core/ModuleDependencyTemplateAsRequireId");
 
-/** @import { Range } from "../javascript/JavascriptParser" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
 
 class AMDRequireItemDependency extends ModuleDependency {
 	/**
@@ -32,10 +32,11 @@ class AMDRequireItemDependency extends ModuleDependency {
 	}
 }
 
-makeSerializable(
-	AMDRequireItemDependency,
+makeSerializable(AMDRequireItemDependency, [
+	"webpack/lib/dependencies/amd/AMDRequireItemDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/AMDRequireItemDependency"
-);
+]);
 
 AMDRequireItemDependency.Template = ModuleDependencyTemplateAsRequireId;
 

package/lib/dependencies/{AMDRuntimeModules.js → amd/AMDRuntimeModules.js}

@@ -4,9 +4,9 @@
 
 "use strict";
 
-const RuntimeGlobals = require("../RuntimeGlobals");
-const RuntimeModule = require("../RuntimeModule");
-const Template = require("../Template");
+const RuntimeGlobals = require("../../runtime/RuntimeGlobals");
+const RuntimeModule = require("../../runtime/RuntimeModule");
+const Template = require("../../template/Template");
 
 /** @import { AmdOptions } from "./AMDPlugin" */
 

package/lib/dependencies/{LocalModule.js → amd/LocalModule.js}

@@ -5,10 +5,10 @@
 
 "use strict";
 
-const makeSerializable = require("../util/makeSerializable");
+const makeSerializable = require("../../util/makeSerializable");
 
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, number, boolean]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, number, boolean]>} ObjectSerializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, number, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[string, number, boolean]>} ObjectSerializerContext */
 
 class LocalModule {
 	/**
@@ -58,6 +58,10 @@ class LocalModule {
 	}
 }
 
-makeSerializable(LocalModule, "webpack/lib/dependencies/LocalModule");
+makeSerializable(LocalModule, [
+	"webpack/lib/dependencies/amd/LocalModule",
+	// TODO in the next major release: remove, they restore pre-move cache packs
+	"webpack/lib/dependencies/LocalModule"
+]);
 
 module.exports = LocalModule;

package/lib/dependencies/{LocalModuleDependency.js → amd/LocalModuleDependency.js}

@@ -5,15 +5,15 @@
 
 "use strict";
 
-const makeSerializable = require("../util/makeSerializable");
-const NullDependency = require("./NullDependency");
+const makeSerializable = require("../../util/makeSerializable");
+const NullDependency = require("../core/NullDependency");
 
 /** @import { ReplaceSource } from "webpack-sources" */
-/** @import Dependency from "../Dependency" */
-/** @import { DependencyTemplateContext } from "../DependencyTemplate" */
-/** @import { Range } from "../javascript/JavascriptParser" */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[LocalModule, Range | undefined, boolean]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[LocalModule, Range | undefined, boolean]>} ObjectSerializerContext */
+/** @import Dependency, { DependencyKinds } from "../../graph/Dependency" */
+/** @import { DependencyTemplateContext } from "../../template/DependencyTemplate" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[LocalModule, Range | undefined, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[LocalModule, Range | undefined, boolean]>} ObjectSerializerContext */
 /** @import LocalModule from "./LocalModule" */
 
 class LocalModuleDependency extends NullDependency {
@@ -58,10 +58,11 @@ class LocalModuleDependency extends NullDependency {
 	}
 }
 
-makeSerializable(
-	LocalModuleDependency,
+makeSerializable(LocalModuleDependency, [
+	"webpack/lib/dependencies/amd/LocalModuleDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/LocalModuleDependency"
-);
+]);
 
 LocalModuleDependency.Template = class LocalModuleDependencyTemplate extends (
 	NullDependency.Template
@@ -83,4 +84,10 @@ LocalModuleDependency.Template = class LocalModuleDependencyTemplate extends (
 	}
 };
 
+/** @type {Set<keyof DependencyKinds>} */
+LocalModuleDependency.KINDS = new Set([
+	...NullDependency.KINDS,
+	"local-module"
+]);
+
 module.exports = LocalModuleDependency;

package/lib/dependencies/{LocalModulesHelpers.js → amd/LocalModulesHelpers.js}

@@ -7,7 +7,7 @@
 
 const LocalModule = require("./LocalModule");
 
-/** @import { JavascriptParserState } from "../javascript/JavascriptParser" */
+/** @import { JavascriptParserState } from "../../javascript/JavascriptParser" */
 
 /**
  * Returns resolved module.

package/lib/dependencies/{RequireJsStuffPlugin.js → amd/RequireJsStuffPlugin.js}

@@ -5,23 +5,23 @@
 
 "use strict";
 
+const {
+	toConstantDependency
+} = require("../../javascript/JavascriptParserHelpers");
 const {
 	JAVASCRIPT_MODULE_TYPE_AUTO,
 	JAVASCRIPT_MODULE_TYPE_DYNAMIC
-} = require("../ModuleTypeConstants");
-const RuntimeGlobals = require("../RuntimeGlobals");
-const {
-	toConstantDependency
-} = require("../javascript/JavascriptParserHelpers");
-const ConstDependency = require("./ConstDependency");
+} = require("../../module/ModuleTypeConstants");
+const RuntimeGlobals = require("../../runtime/RuntimeGlobals");
+const ConstDependency = require("../core/ConstDependency");
 
 /**
  * @import {
  * 	JavascriptParserOptions
- * } from "../../declarations/WebpackOptions"
+ * } from "../../../declarations/WebpackOptions"
  */
-/** @import Compiler from "../Compiler" */
-/** @import JavascriptParser from "../javascript/JavascriptParser" */
+/** @import Compiler from "../../Compiler" */
+/** @import JavascriptParser from "../../javascript/JavascriptParser" */
 
 const PLUGIN_NAME = "RequireJsStuffPlugin";
 

package/lib/dependencies/{UnsupportedDependency.js → amd/UnsupportedDependency.js}

@@ -5,15 +5,15 @@
 
 "use strict";
 
-const makeSerializable = require("../util/makeSerializable");
-const NullDependency = require("./NullDependency");
+const makeSerializable = require("../../util/makeSerializable");
+const NullDependency = require("../core/NullDependency");
 
 /** @import { ReplaceSource } from "webpack-sources" */
-/** @import Dependency from "../Dependency" */
-/** @import { DependencyTemplateContext } from "../DependencyTemplate" */
-/** @import { Range } from "../javascript/JavascriptParser" */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, Range]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, Range]>} ObjectSerializerContext */
+/** @import Dependency from "../../graph/Dependency" */
+/** @import { DependencyTemplateContext } from "../../template/DependencyTemplate" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, Range]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[string, Range]>} ObjectSerializerContext */
 
 class UnsupportedDependency extends NullDependency {
 	/**
@@ -50,10 +50,11 @@ class UnsupportedDependency extends NullDependency {
 	}
 }
 
-makeSerializable(
-	UnsupportedDependency,
+makeSerializable(UnsupportedDependency, [
+	"webpack/lib/dependencies/amd/UnsupportedDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/UnsupportedDependency"
-);
+]);
 
 UnsupportedDependency.Template = class UnsupportedDependencyTemplate extends (
 	NullDependency.Template

package/lib/dependencies/{CommonJsDependencyHelpers.js → commonjs/CommonJsDependencyHelpers.js}

@@ -5,15 +5,14 @@
 
 "use strict";
 
-const Entrypoint = require("../Entrypoint");
-const RuntimeGlobals = require("../RuntimeGlobals");
-const { propertyAccess } = require("../util/property");
+const RuntimeGlobals = require("../../runtime/RuntimeGlobals");
+const { propertyAccess } = require("../../util/property");
 
-/** @import ChunkGraph from "../ChunkGraph" */
-/** @import Module, { RuntimeRequirements } from "../Module" */
-/** @import RuntimeTemplate from "../RuntimeTemplate" */
-/** @import ModuleGraph from "../ModuleGraph" */
-/** @import { RuntimeSpec } from "../util/runtime" */
+/** @import ChunkGraph from "../../graph/ChunkGraph" */
+/** @import Module, { RuntimeRequirements } from "../../module/Module" */
+/** @import RuntimeTemplate from "../../template/RuntimeTemplate" */
+/** @import ModuleGraph from "../../graph/ModuleGraph" */
+/** @import { RuntimeSpec } from "../../util/runtime" */
 /** @typedef {"exports" | "module.exports" | "this" | "Object.defineProperty(exports)" | "Object.defineProperty(module.exports)" | "Object.defineProperty(this)"} CommonJSDependencyBaseKeywords */
 
 /**
@@ -108,7 +107,7 @@ const isWorkerEntryThis = (module, chunkGraph, runtime, runtimeTemplate) => {
 	for (const chunk of chunkGraph.getModuleChunks(module)) {
 		if (chunk.runtime !== runtime) continue;
 		for (const group of chunk.groupsIterable) {
-			if (!(group instanceof Entrypoint)) continue;
+			if (!group.isEntrypoint()) continue;
 			const { options } = group;
 			// an entry may pin a worker to a non-worker entry's runtime, leaving one
 			// code generation for both roles: keep exports, the only safe answer

package/lib/dependencies/{CommonJsExportRequireDependency.js → commonjs/CommonJsExportRequireDependency.js}

@@ -5,20 +5,20 @@
 
 "use strict";
 
-const Dependency = require("../Dependency");
-const { UsageState } = require("../ExportsInfo");
-const Template = require("../Template");
-const { equals } = require("../util/ArrayHelpers");
-const makeSerializable = require("../util/makeSerializable");
-const { propertyAccess } = require("../util/property");
+const Dependency = require("../../graph/Dependency");
+const { UsageState } = require("../../graph/ExportsInfo");
+const Template = require("../../template/Template");
+const { equals } = require("../../util/ArrayHelpers");
+const makeSerializable = require("../../util/makeSerializable");
+const { propertyAccess } = require("../../util/property");
+const ModuleDependency = require("../core/ModuleDependency");
+const processExportInfo = require("../helpers/processExportInfo");
 const {
 	ESM_MODULE_EXPORTS_NAME,
 	getRequireEsmModuleExportsAccess,
 	handleDependencyBase,
 	isRequireEsmModuleExportsModule
 } = require("./CommonJsDependencyHelpers");
-const ModuleDependency = require("./ModuleDependency");
-const processExportInfo = require("./processExportInfo");
 
 /** @import { ReplaceSource } from "webpack-sources" */
 /**
@@ -28,23 +28,25 @@ const processExportInfo = require("./processExportInfo");
  * 	RawReferencedExports,
  * 	ReferencedExports,
  * 	TRANSITIVE,
- * 	ExportInfoName
- * } from "../Dependency"
+ * 	ExportInfoName,
+ * 	DependencyKinds
+ * } from "../../graph/Dependency"
  */
-/** @import { DependencyTemplateContext } from "../DependencyTemplate" */
-/** @import ExportsInfo, { ExportInfo } from "../ExportsInfo" */
-/** @import Module from "../Module" */
-/** @import ModuleGraph from "../ModuleGraph" */
-/** @import ModuleGraphConnection from "../ModuleGraphConnection" */
-/** @import { Range } from "../javascript/JavascriptParser" */
-/** @import { RuntimeSpec } from "../util/runtime" */
+/** @import { DependencyTemplateContext } from "../../template/DependencyTemplate" */
+/** @import ExportsInfo, { ExportInfo } from "../../graph/ExportsInfo" */
+/** @import Module from "../../module/Module" */
+/** @import ModuleGraph from "../../graph/ModuleGraph" */
+/** @import ModuleGraphConnection from "../../graph/ModuleGraphConnection" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
+/** @import { RuntimeSpec } from "../../util/runtime" */
+/** @import CommonJsRequireDependency from "./CommonJsRequireDependency" */
 /**
  * @import {
  * 	CommonJSDependencyBaseKeywords
  * } from "./CommonJsDependencyHelpers"
  */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[undefined | boolean, Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[], ExportInfoName[], boolean, boolean]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[undefined | boolean, Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[], ExportInfoName[], boolean, boolean]>} ObjectSerializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[undefined | boolean, Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[], ExportInfoName[], boolean, boolean, number | null, Range | null, CommonJsRequireDependency | null]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[undefined | boolean, Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[], ExportInfoName[], boolean, boolean, number | null, Range | null, CommonJsRequireDependency | null]>} ObjectSerializerContext */
 
 const idsSymbol = /** @type {symbol} */ (
 	Symbol("CommonJsExportRequireDependency.ids")
@@ -84,6 +86,14 @@ class CommonJsExportRequireDependency extends ModuleDependency {
 		// (as produced by `Object.defineProperty`) rather than an eager value.
 		/** @type {boolean} */
 		this.getter = false;
+		// start of the `Object.defineProperty` descriptor, kept verbatim but for its value
+		/** @type {number | null} */
+		this.descriptorStart = null;
+		// the `const` binding of the same `require()` the value reads, and its dependency
+		/** @type {Range | null} */
+		this.bindingRange = null;
+		/** @type {CommonJsRequireDependency | null} */
+		this.bindingDependency = null;
 	}
 
 	get type() {
@@ -235,18 +245,36 @@ class CommonJsExportRequireDependency extends ModuleDependency {
 			const ids = this.getIds(moduleGraph);
 			const name = this.names[0];
 			const from = moduleGraph.getConnection(this);
-			if (!from) return;
+			// a missing target may come back without this module changing, so
+			// what it exports now must not be cached
+			if (!from) return { exports: [], dependencies: [] };
 			const exportChain = esmUnwrap
 				? [ESM_MODULE_EXPORTS_NAME, ...ids]
 				: ids.length === 0
 					? null
 					: ids;
+
+			// `require()` of a namespace module returns an object carrying `__esModule`
+			let namespaceMarker = false;
+			if (
+				!esmUnwrap &&
+				ids.length === 0 &&
+				importedModule &&
+				importedModule.getExportsType(moduleGraph, false) === "namespace"
+			) {
+				const importedExportsInfo = moduleGraph.getExportsInfo(importedModule);
+				namespaceMarker = !(
+					importedExportsInfo.otherExportsInfo.provided === false &&
+					importedExportsInfo.ownedExports[Symbol.iterator]().next().done
+				);
+			}
 			return {
 				exports: [
 					{
 						name,
 						from,
 						export: exportChain,
+						exports: namespaceMarker ? ["__esModule"] : undefined,
 						// we can't mangle names that are in an empty object
 						// because one could access the prototype property
 						// when export isn't set yet
@@ -271,7 +299,7 @@ class CommonJsExportRequireDependency extends ModuleDependency {
 			};
 		}
 		const from = moduleGraph.getConnection(this);
-		if (!from) return;
+		if (!from) return { exports: [], dependencies: [] };
 		if (esmUnwrap) {
 			// In `module.exports = require("./esm")` of a module carrying a
 			// `"module.exports"` export, this module's own exports become the
@@ -430,7 +458,10 @@ class CommonJsExportRequireDependency extends ModuleDependency {
 			.write(this.names)
 			.write(this.ids)
 			.write(this.resultUsed)
-			.write(this.getter);
+			.write(this.getter)
+			.write(this.descriptorStart)
+			.write(this.bindingRange)
+			.write(this.bindingDependency);
 		super.serialize(context);
 	}
 
@@ -454,14 +485,67 @@ class CommonJsExportRequireDependency extends ModuleDependency {
 		this.resultUsed = c6.read();
 		const c7 = c6.rest;
 		this.getter = c7.read();
-		super.deserialize(c7.rest);
+		const c8 = c7.rest;
+		this.descriptorStart = c8.read();
+		const c9 = c8.rest;
+		this.bindingRange = c9.read();
+		const c10 = c9.rest;
+		this.bindingDependency = c10.read();
+		super.deserialize(c10.rest);
 	}
 }
 
-makeSerializable(
-	CommonJsExportRequireDependency,
+makeSerializable(CommonJsExportRequireDependency, [
+	"webpack/lib/dependencies/commonjs/CommonJsExportRequireDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/CommonJsExportRequireDependency"
-);
+]);
+
+/**
+ * Reads the re-exported value off the `const` binding already holding the same exports.
+ * Only when that binding renders the module, which it doesn't when its own require is inactive.
+ * @param {CommonJsExportRequireDependency} dep the dependency
+ * @param {Module | null} importedModule the re-exported module
+ * @param {ModuleGraph} moduleGraph the module graph
+ * @param {RuntimeSpec} runtime the runtime
+ * @param {boolean} used whether the re-export is used, so its value must be read
+ * @returns {string | undefined} the access to append to the binding, undefined when it can't be read
+ */
+const getBindingAccess = (dep, importedModule, moduleGraph, runtime, used) => {
+	if (!dep.bindingRange || !dep.bindingDependency || !importedModule) return;
+	const connection = moduleGraph.getConnection(dep.bindingDependency);
+	if (
+		!connection ||
+		connection.module !== importedModule ||
+		!connection.isTargetActive(runtime)
+	) {
+		return;
+	}
+	const ids = dep.getIds(moduleGraph);
+	if (ids.length === 0) return "";
+	// the binding holds the unwrapped value, whose properties aren't tracked
+	if (isRequireEsmModuleExportsModule(importedModule, moduleGraph)) return;
+	const usedImported = /** @type {string | string[] | false} */ (
+		moduleGraph.getExportsInfo(importedModule).getUsedName(ids, runtime)
+	);
+	if (usedImported) {
+		return propertyAccess(/** @type {string[]} */ (usedImported));
+	}
+	// an unused re-export of an unused member keeps only the load the binding did
+	if (!used) return "";
+};
+
+/**
+ * Replaces what follows the binding up to `end` (its member chain, a closing `})`).
+ * Nothing follows a bare binding, whose access is then empty too.
+ * @param {ReplaceSource} source the source
+ * @param {Range} bindingRange range of the binding identifier
+ * @param {number} end end of the replaced code
+ * @param {string} content replacement
+ */
+const replaceAfterBinding = (source, bindingRange, end, content) => {
+	if (end > bindingRange[1]) source.replace(bindingRange[1], end - 1, content);
+};
 
 CommonJsExportRequireDependency.Template = class CommonJsExportRequireDependencyTemplate extends (
 	ModuleDependency.Template
@@ -509,6 +593,51 @@ CommonJsExportRequireDependency.Template = class CommonJsExportRequireDependency
 		);
 
 		const importedModule = moduleGraph.getModule(dep);
+		const bindingAccess = getBindingAccess(
+			dep,
+			importedModule,
+			moduleGraph,
+			runtime,
+			Boolean(used)
+		);
+		if (bindingAccess !== undefined) {
+			const bindingRange = /** @type {Range} */ (dep.bindingRange);
+			if (!used) {
+				source.replace(
+					dep.range[0],
+					bindingRange[0] - 1,
+					"/* unused reexport */ "
+				);
+				replaceAfterBinding(source, bindingRange, dep.range[1], bindingAccess);
+				return;
+			}
+			if (type === "expression") {
+				source.replace(
+					dep.range[0],
+					bindingRange[0] - 1,
+					`${base}${propertyAccess(/** @type {string[]} */ (used))} = `
+				);
+			} else {
+				source.replace(
+					dep.range[0],
+					/** @type {number} */ (dep.descriptorStart) - 1,
+					`Object.defineProperty(${base}${propertyAccess(
+						/** @type {string[]} */ (used).slice(0, -1)
+					)}, ${JSON.stringify(used[used.length - 1])}, `
+				);
+			}
+			let valueEnd = dep.range[1];
+			if (type !== "expression") {
+				const valueRange = /** @type {Range} */ (dep.valueRange);
+				// `(ns).a`: the value opens before the binding; drop that with what follows it
+				if (valueRange[0] < bindingRange[0]) {
+					source.replace(valueRange[0], bindingRange[0] - 1, "");
+				}
+				valueEnd = valueRange[1];
+			}
+			replaceAfterBinding(source, bindingRange, valueEnd, bindingAccess);
+			return;
+		}
 		/** @type {string} */
 		let requireExpr;
 		if (
@@ -589,18 +718,16 @@ CommonJsExportRequireDependency.Template = class CommonJsExportRequireDependency
 					);
 					return;
 				}
-				const descriptor = dep.getter
-					? "enumerable: true, get: () => ("
-					: "value: (";
+				// Only the target and the value change, so the descriptor keeps its
+				// attributes and the getter its own function.
 				source.replace(
 					dep.range[0],
-					valueRange[0] - 1,
+					/** @type {number} */ (dep.descriptorStart) - 1,
 					`Object.defineProperty(${base}${propertyAccess(
 						/** @type {string[]} */ (used).slice(0, -1)
-					)}, ${JSON.stringify(used[used.length - 1])}, { ${descriptor}`
+					)}, ${JSON.stringify(used[used.length - 1])}, `
 				);
-				source.replace(valueRange[0], valueRange[1] - 1, requireExpr);
-				source.replace(valueRange[1], dep.range[1] - 1, ") })");
+				source.replace(valueRange[0], valueRange[1] - 1, `(${requireExpr})`);
 				return;
 			}
 			default:
@@ -611,4 +738,10 @@ CommonJsExportRequireDependency.Template = class CommonJsExportRequireDependency
 
 CommonJsExportRequireDependency.idsSymbol = idsSymbol;
 
+/** @type {Set<keyof DependencyKinds>} */
+CommonJsExportRequireDependency.KINDS = new Set([
+	...ModuleDependency.KINDS,
+	"commonjs-export-require"
+]);
+
 module.exports = CommonJsExportRequireDependency;

package/lib/dependencies/{CommonJsExportsDependency.js → commonjs/CommonJsExportsDependency.js}

@@ -5,15 +5,15 @@
 
 "use strict";
 
-const InitFragment = require("../InitFragment");
-const makeSerializable = require("../util/makeSerializable");
-const { propertyAccess } = require("../util/property");
+const InitFragment = require("../../template/InitFragment");
+const makeSerializable = require("../../util/makeSerializable");
+const { propertyAccess } = require("../../util/property");
+const NullDependency = require("../core/NullDependency");
 const {
 	handleDependencyBase,
 	isThisBase,
 	isWorkerEntryThis
 } = require("./CommonJsDependencyHelpers");
-const NullDependency = require("./NullDependency");
 
 /** @import { ReplaceSource } from "webpack-sources" */
 /**
@@ -21,19 +21,19 @@ const NullDependency = require("./NullDependency");
  * 	ExportsSpec,
  * 	UpdateHashContext,
  * 	ExportInfoName
- * } from "../Dependency"
+ * } from "../../graph/Dependency"
  */
-/** @import Hash from "../util/Hash" */
-/** @import { DependencyTemplateContext } from "../DependencyTemplate" */
-/** @import ModuleGraph from "../ModuleGraph" */
-/** @import { Range } from "../javascript/JavascriptParser" */
+/** @import Hash from "../../util/Hash" */
+/** @import { DependencyTemplateContext } from "../../template/DependencyTemplate" */
+/** @import ModuleGraph from "../../graph/ModuleGraph" */
+/** @import { Range } from "../../javascript/JavascriptParser" */
 /**
  * @import {
  * 	CommonJSDependencyBaseKeywords
  * } from "./CommonJsDependencyHelpers"
  */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[]]>} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[]]>} ObjectSerializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectDeserializerContext<[Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[]]>} ObjectDeserializerContext */
+/** @typedef {import("../../serialization/ObjectMiddleware").ObjectSerializerContext<[Range, Range | null, CommonJSDependencyBaseKeywords, ExportInfoName[]]>} ObjectSerializerContext */
 
 const EMPTY_OBJECT = {};
 
@@ -129,10 +129,11 @@ class CommonJsExportsDependency extends NullDependency {
 	}
 }
 
-makeSerializable(
-	CommonJsExportsDependency,
+makeSerializable(CommonJsExportsDependency, [
+	"webpack/lib/dependencies/commonjs/CommonJsExportsDependency",
+	// TODO in the next major release: remove, they restore pre-move cache packs
 	"webpack/lib/dependencies/CommonJsExportsDependency"
-);
+]);
 
 CommonJsExportsDependency.Template = class CommonJsExportsDependencyTemplate extends (
 	NullDependency.Template