@depup/webpack 5.109.0-depup.0 → 5.110.1-depup.0

package/lib/config/target.js

@@ -7,6 +7,7 @@
 
 const memoize = require("../util/memoize");
 
+// browserslist pulls in caniuse-lite; only a browserslist target needs it
 const getBrowserslistTargetHandler = memoize(() =>
 	require("./browserslistTargetHandler")
 );
@@ -81,6 +82,8 @@ const getDefaultTarget = (context) => {
  * @property {boolean | null} templateLiteral template literal is available
  * @property {boolean | null} asyncFunction async functions and await are available
  * @property {boolean | null} generator generator functions and yield are available
+ * @property {(boolean | null)=} deferImport deferred evaluation (`import defer`, `import.defer()`) is available (optional; absent means "assume unavailable")
+ * @property {(boolean | null)=} sourceImport source phase imports (`import source`, `import.source()`) are available (optional; absent means "assume unavailable")
  */
 
 /**
@@ -116,12 +119,16 @@ const versionDependent = (major, minor) => {
 		nMajor > vMajor || (nMajor === vMajor && nMinor >= vMinor);
 };
 
+// A `browserslist` target, with an optional `:env` / `:query` / config path
+// after the keyword that `browserslistTargetHandler.load` reads.
+const BROWSERSLIST_TARGET = /^browserslist(?::(.+))?$/;
+
 /** @type {[string, string, RegExp, (...args: string[]) => Partial<TargetProperties>][]} */
 const TARGETS = [
 	[
 		"browserslist / browserslist:env / browserslist:query / browserslist:path-to-config / browserslist:path-to-config:env",
 		"Resolve features from browserslist. Will resolve browserslist config automatically. Only browser or node queries are supported (electron is not supported). Examples: 'browserslist:modern' to use 'modern' environment from browserslist config",
-		/^browserslist(?::(.+))?$/,
+		BROWSERSLIST_TARGET,
 		(rest, context) => {
 			const browserslistTargetHandler = getBrowserslistTargetHandler();
 			const browsers = browserslistTargetHandler.load(
@@ -267,7 +274,9 @@ You can also more options via the 'target' option: 'browserslist' / 'browserslis
 				bigIntLiteral: v(10, 4),
 				dynamicImport: v(12, 17),
 				dynamicImportInWorker: v(12, 17),
-				module: v(12, 17)
+				module: v(12, 17),
+				// v24.5.0, for WebAssembly only; `import defer` is not implemented
+				sourceImport: v(24, 5)
 			};
 		}
 	],
@@ -319,7 +328,11 @@ You can also more options via the 'target' option: 'browserslist' / 'browserslis
 				spread: true,
 				templateLiteral: true,
 				asyncFunction: true,
-				generator: true
+				generator: true,
+				// import defer - v2.8.0
+				deferImport: v(2, 8),
+				// import source - v2.6.0
+				sourceImport: v(2, 6)
 			};
 		}
 	],
@@ -536,6 +549,10 @@ const getTargetProperties = (target, context) => {
 	);
 };
 
+// Absence means "cannot read it" here, so a target omitting one votes against rather
+// than letting another's `true` stand — `["web", "deno2.8"]` reads neither phase.
+const UNSUPPORTED_WHEN_ABSENT = new Set(["deferImport", "sourceImport"]);
+
 /**
  * Merges target properties.
  * @param {TargetProperties[]} targetProperties array of target properties
@@ -563,6 +580,10 @@ const mergeTargetProperties = (targetProperties) => {
 				case false:
 					hasFalse = true;
 					break;
+				default:
+					if (value === undefined && UNSUPPORTED_WHEN_ABSENT.has(key)) {
+						hasFalse = true;
+					}
 			}
 		}
 		if (hasTrue || hasFalse) {
@@ -582,6 +603,28 @@ const mergeTargetProperties = (targetProperties) => {
 const getTargetsProperties = (targets, context) =>
 	mergeTargetProperties(targets.map((t) => getTargetProperties(t, context)));
 
+/**
+ * The raw browserslist selection behind the targets (`["chrome 100", …]`), merged
+ * across every `browserslist` target and deduplicated — undefined when none is
+ * one, and prefixing stays off with no browser list to prefix for.
+ * @param {string[]} targets the targets
+ * @param {string} context the context directory
+ * @returns {string[] | undefined} the selected browsers, or undefined
+ */
+const getTargetsBrowsers = (targets, context) => {
+	const { load } = getBrowserslistTargetHandler();
+	/** @type {Set<string>} */
+	const browsers = new Set();
+	for (const target of targets) {
+		const match = BROWSERSLIST_TARGET.exec(target);
+		if (!match) continue;
+		const selected = load(match[1] ? match[1].trim() : null, context);
+		if (selected) for (const browser of selected) browsers.add(browser);
+	}
+	return browsers.size === 0 ? undefined : [...browsers];
+};
+
 module.exports.getDefaultTarget = getDefaultTarget;
 module.exports.getTargetProperties = getTargetProperties;
+module.exports.getTargetsBrowsers = getTargetsBrowsers;
 module.exports.getTargetsProperties = getTargetsProperties;

package/lib/container/ContainerEntryDependency.js

@@ -8,7 +8,7 @@
 const Dependency = require("../Dependency");
 const makeSerializable = require("../util/makeSerializable");
 
-/** @typedef {import("./ContainerEntryModule").ExposesList} ExposesList */
+/** @import { ExposesList } from "./ContainerEntryModule" */
 
 class ContainerEntryDependency extends Dependency {
 	/**

package/lib/container/ContainerEntryModule.js

@@ -19,22 +19,30 @@ const StaticExportsDependency = require("../dependencies/StaticExportsDependency
 const makeSerializable = require("../util/makeSerializable");
 const ContainerExposedDependency = require("./ContainerExposedDependency");
 
-/** @typedef {import("../config/defaults").WebpackOptionsNormalizedWithDefaults} WebpackOptions */
-/** @typedef {import("../Compilation")} Compilation */
-/** @typedef {import("../Module").BuildCallback} BuildCallback */
-/** @typedef {import("../Module").CodeGenerationContext} CodeGenerationContext */
-/** @typedef {import("../Module").CodeGenerationResult} CodeGenerationResult */
-/** @typedef {import("../Module").LibIdentOptions} LibIdentOptions */
-/** @typedef {import("../Module").LibIdent} LibIdent */
-/** @typedef {import("../Module").NeedBuildCallback} NeedBuildCallback */
-/** @typedef {import("../Module").NeedBuildContext} NeedBuildContext */
-/** @typedef {import("../Module").Sources} Sources */
-/** @typedef {import("../Module").SourceTypes} SourceTypes */
-/** @typedef {import("../RequestShortener")} RequestShortener */
-/** @typedef {import("../ResolverFactory").ResolverWithOptions} ResolverWithOptions */
+/**
+ * @import {
+ * 	WebpackOptionsNormalizedWithDefaults as WebpackOptions
+ * } from "../config/defaults"
+ */
+/** @import Compilation from "../Compilation" */
+/**
+ * @import {
+ * 	BuildCallback,
+ * 	CodeGenerationContext,
+ * 	CodeGenerationResult,
+ * 	LibIdentOptions,
+ * 	LibIdent,
+ * 	NeedBuildCallback,
+ * 	NeedBuildContext,
+ * 	Sources,
+ * 	SourceTypes
+ * } from "../Module"
+ */
+/** @import RequestShortener from "../RequestShortener" */
+/** @import { ResolverWithOptions } from "../ResolverFactory" */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, ExposesList, string]>} ObjectDeserializerContext */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, ExposesList, string]>} ObjectSerializerContext */
-/** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
+/** @import { InputFileSystem } from "../util/fs" */
 
 /**
  * Defines the expose options type used by this module.
@@ -194,7 +202,8 @@ class ContainerEntryModule extends Module {
 					block,
 					message: "",
 					chunkGraph,
-					runtimeRequirements
+					runtimeRequirements,
+					originModule: this
 				})}.then(${runtimeTemplate.returningFunction(
 					runtimeTemplate.returningFunction(
 						`(${modules

package/lib/container/ContainerEntryModuleFactory.js

@@ -8,9 +8,13 @@
 const ModuleFactory = require("../ModuleFactory");
 const ContainerEntryModule = require("./ContainerEntryModule");
 
-/** @typedef {import("../ModuleFactory").ModuleFactoryCallback} ModuleFactoryCallback */
-/** @typedef {import("../ModuleFactory").ModuleFactoryCreateData} ModuleFactoryCreateData */
-/** @typedef {import("./ContainerEntryDependency")} ContainerEntryDependency */
+/**
+ * @import {
+ * 	ModuleFactoryCallback,
+ * 	ModuleFactoryCreateData
+ * } from "../ModuleFactory"
+ */
+/** @import ContainerEntryDependency from "./ContainerEntryDependency" */
 
 module.exports = class ContainerEntryModuleFactory extends ModuleFactory {
 	/**

package/lib/container/ContainerPlugin.js

@@ -5,19 +5,19 @@
 
 "use strict";
 
-const memoize = require("../util/memoize");
 const ContainerEntryDependency = require("./ContainerEntryDependency");
 const ContainerEntryModuleFactory = require("./ContainerEntryModuleFactory");
 const ContainerExposedDependency = require("./ContainerExposedDependency");
+const getModuleFederationCompilationHooks = require("./moduleFederationHooks");
 const { parseOptions } = require("./options");
 
-/** @typedef {import("../../declarations/plugins/container/ContainerPlugin").ContainerPluginOptions} ContainerPluginOptions */
-/** @typedef {import("../Compiler")} Compiler */
-/** @typedef {import("./ContainerEntryModule").ExposesList} ExposesList */
-
-const getModuleFederationPlugin = memoize(() =>
-	require("./ModuleFederationPlugin")
-);
+/**
+ * @import {
+ * 	ContainerPluginOptions
+ * } from "../../declarations/plugins/container/ContainerPlugin"
+ */
+/** @import Compiler from "../Compiler" */
+/** @import { ExposesList } from "./ContainerEntryModule" */
 
 const PLUGIN_NAME = "ContainerPlugin";
 
@@ -78,8 +78,7 @@ class ContainerPlugin {
 		const shareScope = this.options.shareScope || "default";
 
 		compiler.hooks.make.tapAsync(PLUGIN_NAME, (compilation, callback) => {
-			const hooks =
-				getModuleFederationPlugin().getCompilationHooks(compilation);
+			const hooks = getModuleFederationCompilationHooks(compilation);
 			const dep = new ContainerEntryDependency(
 				this.options.name,
 				exposes,

package/lib/container/ContainerReferencePlugin.js

@@ -16,8 +16,12 @@ const RemoteRuntimeModule = require("./RemoteRuntimeModule");
 const RemoteToExternalDependency = require("./RemoteToExternalDependency");
 const { parseOptions } = require("./options");
 
-/** @typedef {import("../../declarations/plugins/container/ContainerReferencePlugin").ContainerReferencePluginOptions} ContainerReferencePluginOptions */
-/** @typedef {import("../Compiler")} Compiler */
+/**
+ * @import {
+ * 	ContainerReferencePluginOptions
+ * } from "../../declarations/plugins/container/ContainerReferencePlugin"
+ */
+/** @import Compiler from "../Compiler" */
 
 const slashCode = "/".charCodeAt(0);
 const PLUGIN_NAME = "ContainerReferencePlugin";

package/lib/container/FallbackDependency.js

@@ -8,7 +8,7 @@
 const Dependency = require("../Dependency");
 const makeSerializable = require("../util/makeSerializable");
 
-/** @typedef {import("./RemoteModule").ExternalRequests} ExternalRequests */
+/** @import { ExternalRequests } from "./RemoteModule" */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[ExternalRequests]>} ObjectDeserializerContext */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[ExternalRequests]>} ObjectSerializerContext */
 

package/lib/container/FallbackModule.js

@@ -17,25 +17,32 @@ const Template = require("../Template");
 const makeSerializable = require("../util/makeSerializable");
 const FallbackItemDependency = require("./FallbackItemDependency");
 
-/** @typedef {import("../config/defaults").WebpackOptionsNormalizedWithDefaults} WebpackOptions */
-/** @typedef {import("../Chunk")} Chunk */
-/** @typedef {import("../Compilation")} Compilation */
-/** @typedef {import("../Module").BuildCallback} BuildCallback */
-/** @typedef {import("../Module").CodeGenerationContext} CodeGenerationContext */
-/** @typedef {import("../Module").CodeGenerationResult} CodeGenerationResult */
-/** @typedef {import("../Module").LibIdentOptions} LibIdentOptions */
-/** @typedef {import("../Module").LibIdent} LibIdent */
-/** @typedef {import("../Module").NameForCondition} NameForCondition */
-/** @typedef {import("../Module").NeedBuildCallback} NeedBuildCallback */
-/** @typedef {import("../Module").NeedBuildContext} NeedBuildContext */
-/** @typedef {import("../Module").Sources} Sources */
-/** @typedef {import("../Module").SourceTypes} SourceTypes */
-/** @typedef {import("../RequestShortener")} RequestShortener */
-/** @typedef {import("../ResolverFactory").ResolverWithOptions} ResolverWithOptions */
+/**
+ * @import {
+ * 	WebpackOptionsNormalizedWithDefaults as WebpackOptions
+ * } from "../config/defaults"
+ */
+/** @import Chunk from "../Chunk" */
+/** @import Compilation from "../Compilation" */
+/**
+ * @import {
+ * 	BuildCallback,
+ * 	CodeGenerationContext,
+ * 	CodeGenerationResult,
+ * 	LibIdentOptions,
+ * 	LibIdent,
+ * 	NeedBuildCallback,
+ * 	NeedBuildContext,
+ * 	Sources,
+ * 	SourceTypes
+ * } from "../Module"
+ */
+/** @import RequestShortener from "../RequestShortener" */
+/** @import { ResolverWithOptions } from "../ResolverFactory" */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[ExternalRequests]>} ObjectDeserializerContext */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[ExternalRequests]>} ObjectSerializerContext */
-/** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
-/** @typedef {import("./RemoteModule").ExternalRequests} ExternalRequests */
+/** @import { InputFileSystem } from "../util/fs" */
+/** @import { ExternalRequests } from "./RemoteModule" */
 
 const RUNTIME_REQUIREMENTS = new Set([RuntimeGlobals.module]);
 

package/lib/container/FallbackModuleFactory.js

@@ -8,9 +8,13 @@
 const ModuleFactory = require("../ModuleFactory");
 const FallbackModule = require("./FallbackModule");
 
-/** @typedef {import("../ModuleFactory").ModuleFactoryCallback} ModuleFactoryCallback */
-/** @typedef {import("../ModuleFactory").ModuleFactoryCreateData} ModuleFactoryCreateData */
-/** @typedef {import("./FallbackDependency")} FallbackDependency */
+/**
+ * @import {
+ * 	ModuleFactoryCallback,
+ * 	ModuleFactoryCreateData
+ * } from "../ModuleFactory"
+ */
+/** @import FallbackDependency from "./FallbackDependency" */
 
 module.exports = class FallbackModuleFactory extends ModuleFactory {
 	/**

package/lib/container/HoistContainerReferencesPlugin.js

@@ -8,17 +8,13 @@
 const AsyncDependenciesBlock = require("../AsyncDependenciesBlock");
 const ExternalModule = require("../ExternalModule");
 const { STAGE_ADVANCED } = require("../OptimizationStages");
-const memoize = require("../util/memoize");
 const { forEachRuntime } = require("../util/runtime");
+const getModuleFederationCompilationHooks = require("./moduleFederationHooks");
 
-/** @typedef {import("../Compilation")} Compilation */
-/** @typedef {import("../Compiler")} Compiler */
-/** @typedef {import("../Dependency")} Dependency */
-/** @typedef {import("../Module")} Module */
-
-const getModuleFederationPlugin = memoize(() =>
-	require("./ModuleFederationPlugin")
-);
+/** @import Compilation from "../Compilation" */
+/** @import Compiler from "../Compiler" */
+/** @import Dependency from "../Dependency" */
+/** @import Module from "../Module" */
 
 const PLUGIN_NAME = "HoistContainerReferences";
 
@@ -32,18 +28,18 @@ class HoistContainerReferences {
 	 */
 	apply(compiler) {
 		compiler.hooks.thisCompilation.tap(PLUGIN_NAME, (compilation) => {
-			const hooks =
-				getModuleFederationPlugin().getCompilationHooks(compilation);
+			const hooks = getModuleFederationCompilationHooks(compilation);
 			/** @type {Set<Dependency>} */
 			const depsToTrace = new Set();
 			/** @type {Set<Dependency>} */
 			const entryExternalsToHoist = new Set();
-			hooks.addContainerEntryDependency.tap(PLUGIN_NAME, (dep) => {
-				depsToTrace.add(dep);
-			});
-			hooks.addFederationRuntimeDependency.tap(PLUGIN_NAME, (dep) => {
+			// Both hooks feed the same trace set, so they share one callback.
+			/** @type {(dep: Dependency) => void} */
+			const traceDep = (dep) => {
 				depsToTrace.add(dep);
-			});
+			};
+			hooks.addContainerEntryDependency.tap(PLUGIN_NAME, traceDep);
+			hooks.addFederationRuntimeDependency.tap(PLUGIN_NAME, traceDep);
 
 			compilation.hooks.addEntry.tap(PLUGIN_NAME, (entryDep) => {
 				if (entryDep.type === "entry") {
@@ -76,46 +72,26 @@ class HoistContainerReferences {
 	 * @param {Set<Dependency>} entryExternalsToHoist Set of container entry dependencies to hoist.
 	 */
 	hoistModulesInChunks(compilation, depsToTrace, entryExternalsToHoist) {
-		const { chunkGraph, moduleGraph } = compilation;
+		const { moduleGraph } = compilation;
 
-		// loop over entry points
+		// Entry externals: hoist the external modules (e.g. RemoteModule) they reference.
 		for (const dep of entryExternalsToHoist) {
 			const entryModule = moduleGraph.getModule(dep);
 			if (!entryModule) continue;
-			// get all the external module types and hoist them to the runtime chunk, this will get RemoteModule externals
 			const allReferencedModules = getAllReferencedModules(
 				compilation,
 				entryModule,
 				"external",
 				false
 			);
-
-			const containerRuntimes = chunkGraph.getModuleRuntimes(entryModule);
-			/** @type {Set<string>} */
-			const runtimes = new Set();
-
-			for (const runtimeSpec of containerRuntimes) {
-				forEachRuntime(runtimeSpec, (runtimeKey) => {
-					if (runtimeKey) {
-						runtimes.add(runtimeKey);
-					}
-				});
-			}
-
-			for (const runtime of runtimes) {
-				const runtimeChunk = compilation.namedChunks.get(runtime);
-				if (!runtimeChunk) continue;
-
-				for (const module of allReferencedModules) {
-					if (!chunkGraph.isModuleInChunk(module, runtimeChunk)) {
-						chunkGraph.connectChunkAndModule(runtimeChunk, module);
-					}
-				}
-			}
-			this.cleanUpChunks(compilation, allReferencedModules);
+			this.hoistReferencedModules(
+				compilation,
+				entryModule,
+				allReferencedModules
+			);
 		}
 
-		// handle container entry specifically
+		// Container entries: hoist the initial graph plus its external references.
 		for (const dep of depsToTrace) {
 			const containerEntryModule = moduleGraph.getModule(dep);
 			if (!containerEntryModule) continue;
@@ -125,43 +101,54 @@ class HoistContainerReferences {
 				"initial",
 				false
 			);
-
 			const allRemoteReferences = getAllReferencedModules(
 				compilation,
 				containerEntryModule,
 				"external",
 				false
 			);
-
 			for (const remote of allRemoteReferences) {
 				allReferencedModules.add(remote);
 			}
+			this.hoistReferencedModules(
+				compilation,
+				containerEntryModule,
+				allReferencedModules
+			);
+		}
+	}
 
-			const containerRuntimes =
-				chunkGraph.getModuleRuntimes(containerEntryModule);
-			/** @type {Set<string>} */
-			const runtimes = new Set();
-
-			for (const runtimeSpec of containerRuntimes) {
-				forEachRuntime(runtimeSpec, (runtimeKey) => {
-					if (runtimeKey) {
-						runtimes.add(runtimeKey);
-					}
-				});
-			}
+	/**
+	 * Connect `referencedModules` into each runtime chunk of `entryModule`, then
+	 * prune the chunks they were hoisted out of. The two passes above build
+	 * `referencedModules` differently but hoist them identically.
+	 * @param {Compilation} compilation The webpack compilation instance.
+	 * @param {Module} entryModule The module whose runtimes receive the modules.
+	 * @param {Set<Module>} referencedModules The modules to hoist.
+	 */
+	hoistReferencedModules(compilation, entryModule, referencedModules) {
+		const { chunkGraph } = compilation;
+		/** @type {Set<string>} */
+		const runtimes = new Set();
+		for (const runtimeSpec of chunkGraph.getModuleRuntimes(entryModule)) {
+			forEachRuntime(runtimeSpec, (runtimeKey) => {
+				if (runtimeKey) {
+					runtimes.add(runtimeKey);
+				}
+			});
+		}
 
-			for (const runtime of runtimes) {
-				const runtimeChunk = compilation.namedChunks.get(runtime);
-				if (!runtimeChunk) continue;
+		for (const runtime of runtimes) {
+			const runtimeChunk = compilation.namedChunks.get(runtime);
+			if (!runtimeChunk) continue;
 
-				for (const module of allReferencedModules) {
-					if (!chunkGraph.isModuleInChunk(module, runtimeChunk)) {
-						chunkGraph.connectChunkAndModule(runtimeChunk, module);
-					}
+			for (const module of referencedModules) {
+				if (!chunkGraph.isModuleInChunk(module, runtimeChunk)) {
+					chunkGraph.connectChunkAndModule(runtimeChunk, module);
 				}
 			}
-			this.cleanUpChunks(compilation, allReferencedModules);
 		}
+		this.cleanUpChunks(compilation, referencedModules);
 	}
 
 	/**

package/lib/container/ModuleFederationPlugin.js

@@ -5,25 +5,20 @@
 
 "use strict";
 
-const { SyncHook } = require("tapable");
 const isValidExternalsType = require("../../schemas/plugins/container/ExternalsType.check");
 const SharePlugin = require("../sharing/SharePlugin");
-const createHooksRegistry = require("../util/createHooksRegistry");
 const ContainerPlugin = require("./ContainerPlugin");
 const ContainerReferencePlugin = require("./ContainerReferencePlugin");
 const HoistContainerReferences = require("./HoistContainerReferencesPlugin");
-
-/** @typedef {import("../../declarations/plugins/container/ModuleFederationPlugin").ExternalsType} ExternalsType */
-/** @typedef {import("../../declarations/plugins/container/ModuleFederationPlugin").ModuleFederationPluginOptions} ModuleFederationPluginOptions */
-/** @typedef {import("../Compiler")} Compiler */
-/** @typedef {import("../Dependency")} Dependency */
+const getCompilationHooks = require("./moduleFederationHooks");
 
 /**
- * Defines the compilation hooks type used by this module.
- * @typedef {object} CompilationHooks
- * @property {SyncHook<Dependency>} addContainerEntryDependency
- * @property {SyncHook<Dependency>} addFederationRuntimeDependency
+ * @import {
+ * 	ExternalsType,
+ * 	ModuleFederationPluginOptions
+ * } from "../../declarations/plugins/container/ModuleFederationPlugin"
  */
+/** @import Compiler from "../Compiler" */
 
 const PLUGIN_NAME = "ModuleFederationPlugin";
 
@@ -110,12 +105,6 @@ class ModuleFederationPlugin {
 	}
 }
 
-ModuleFederationPlugin.getCompilationHooks = createHooksRegistry(
-	() =>
-		/** @type {CompilationHooks} */ ({
-			addContainerEntryDependency: new SyncHook(["dependency"]),
-			addFederationRuntimeDependency: new SyncHook(["dependency"])
-		})
-);
+ModuleFederationPlugin.getCompilationHooks = getCompilationHooks;
 
 module.exports = ModuleFederationPlugin;

package/lib/container/RemoteModule.js

@@ -17,27 +17,35 @@ const makeSerializable = require("../util/makeSerializable");
 const FallbackDependency = require("./FallbackDependency");
 const RemoteToExternalDependency = require("./RemoteToExternalDependency");
 
-/** @typedef {import("../config/defaults").WebpackOptionsNormalizedWithDefaults} WebpackOptions */
-/** @typedef {import("../Compilation")} Compilation */
-/** @typedef {import("../Module").BuildCallback} BuildCallback */
-/** @typedef {import("../Module").CodeGenerationContext} CodeGenerationContext */
-/** @typedef {import("../Module").CodeGenerationResultData} CodeGenerationResultData */
-/** @typedef {import("../Module").CodeGenerationResult} CodeGenerationResult */
-/** @typedef {import("../Module").LibIdentOptions} LibIdentOptions */
-/** @typedef {import("../Module").LibIdent} LibIdent */
-/** @typedef {import("../Module").NameForCondition} NameForCondition */
-/** @typedef {import("../Module").NeedBuildCallback} NeedBuildCallback */
-/** @typedef {import("../Module").NeedBuildContext} NeedBuildContext */
-/** @typedef {import("../Module").Sources} Sources */
-/** @typedef {import("../Module").SourceTypes} SourceTypes */
-/** @typedef {import("../ModuleGraph")} ModuleGraph */
-/** @typedef {import("../Module").ExportsType} ExportsType */
-/** @typedef {import("../RequestShortener")} RequestShortener */
-/** @typedef {import("../ResolverFactory").ResolverWithOptions} ResolverWithOptions */
+/**
+ * @import {
+ * 	WebpackOptionsNormalizedWithDefaults as WebpackOptions
+ * } from "../config/defaults"
+ */
+/** @import Compilation from "../Compilation" */
+/**
+ * @import {
+ * 	BuildCallback,
+ * 	CodeGenerationContext,
+ * 	CodeGenerationResultData,
+ * 	CodeGenerationResult,
+ * 	LibIdentOptions,
+ * 	LibIdent,
+ * 	NameForCondition,
+ * 	NeedBuildCallback,
+ * 	NeedBuildContext,
+ * 	Sources,
+ * 	SourceTypes,
+ * 	ExportsType,
+ * 	BasicSourceTypes
+ * } from "../Module"
+ */
+/** @import ModuleGraph from "../ModuleGraph" */
+/** @import RequestShortener from "../RequestShortener" */
+/** @import { ResolverWithOptions } from "../ResolverFactory" */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, ExternalRequests, string, string]>} ObjectDeserializerContext */
 /** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, ExternalRequests, string, string]>} ObjectSerializerContext */
-/** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
-/** @typedef {import("../Module").BasicSourceTypes} BasicSourceTypes */
+/** @import { InputFileSystem } from "../util/fs" */
 
 const RUNTIME_REQUIREMENTS = new Set([RuntimeGlobals.module]);
 

package/lib/container/RemoteRuntimeModule.js

@@ -8,13 +8,12 @@
 const RuntimeGlobals = require("../RuntimeGlobals");
 const RuntimeModule = require("../RuntimeModule");
 const Template = require("../Template");
+const { compareModulesByIdentifier } = require("../util/comparators");
 
-/** @typedef {import("../Chunk")} Chunk */
-/** @typedef {import("../Chunk").ChunkId} ChunkId */
-/** @typedef {import("../ChunkGraph")} ChunkGraph */
-/** @typedef {import("../ChunkGraph").ModuleId} ModuleId */
-/** @typedef {import("../Compilation")} Compilation */
-/** @typedef {import("./RemoteModule")} RemoteModule */
+/** @import Chunk, { ChunkId } from "../Chunk" */
+/** @import ChunkGraph, { ModuleId } from "../ChunkGraph" */
+/** @import Compilation from "../Compilation" */
+/** @import RemoteModule from "./RemoteModule" */
 
 class RemoteRuntimeModule extends RuntimeModule {
 	constructor() {
@@ -36,9 +35,12 @@ class RemoteRuntimeModule extends RuntimeModule {
 		for (const chunk of /** @type {Chunk} */ (
 			this.chunk
 		).getAllReferencedChunks()) {
-			const modules = chunkGraph.getChunkModulesIterableBySourceType(
+			// ordered explicitly: an unordered read reflects whichever comparator
+			// last sorted the chunk's modules, which is not ours to depend on
+			const modules = chunkGraph.getOrderedChunkModulesIterableBySourceType(
 				chunk,
-				"remote"
+				"remote",
+				compareModulesByIdentifier
 			);
 			if (!modules) continue;
 			/** @type {ModuleId[]} */