webpack 5.107.1 → 5.108.0

package/lib/dependencies/ImportMetaPlugin.js

@@ -26,6 +26,13 @@ const { propertyAccess } = require("../util/property");
 const ConstDependency = require("./ConstDependency");
 const ModuleInitFragmentDependency = require("./ModuleInitFragmentDependency");
 
+// Loaded at module scope, not inside the parser handler: the async resolver
+// cache tail can run that handler after a test tears down the module registry.
+const WEBPACK_VERSION = Number.parseInt(
+	require("../../package.json").version,
+	10
+);
+
 /** @typedef {import("estree").MemberExpression} MemberExpression */
 /** @typedef {import("estree").Identifier} Identifier */
 /** @typedef {import("../../declarations/WebpackOptions").JavascriptParserOptions} JavascriptParserOptions */
@@ -40,6 +47,29 @@ const ModuleInitFragmentDependency = require("./ModuleInitFragmentDependency");
 
 const PLUGIN_NAME = "ImportMetaPlugin";
 
+const IMPORT_META = "import.meta";
+const IMPORT_META_URL = "import.meta.url";
+const IMPORT_META_WEBPACK = "import.meta.webpack";
+const IMPORT_META_MAIN = "import.meta.main";
+const IMPORT_META_ENV = "import.meta.env";
+const IMPORT_META_DIRNAME = "import.meta.dirname";
+const IMPORT_META_FILENAME = "import.meta.filename";
+
+// Single source of truth for `import.meta` accesses that only exist in ES modules
+// and therefore mark a module as ESM (consumed by HarmonyDetectionParserPlugin).
+// `dirname`/`filename` are resolved by NodeStuffPlugin, the rest here. The webpack
+// pragmas `import.meta.webpackHot`/`webpackContext` are intentionally excluded:
+// they are usable in CommonJS modules and must not force ESM.
+const IMPORT_META_NAMES = [
+	IMPORT_META,
+	IMPORT_META_URL,
+	IMPORT_META_WEBPACK,
+	IMPORT_META_MAIN,
+	IMPORT_META_ENV,
+	IMPORT_META_DIRNAME,
+	IMPORT_META_FILENAME
+];
+
 /** @type {WeakMap<Compilation, { stringify: string, env: Record<string, string> }>} */
 const compilationMetaEnvMap = new WeakMap();
 
@@ -137,7 +167,7 @@ class ImportMetaPlugin {
 						if (importMetaName === "import.meta") return;
 
 						parser.hooks.expression
-							.for("import.meta")
+							.for(IMPORT_META)
 							.tap(PLUGIN_NAME, (metaProperty) => {
 								const dep = new ConstDependency(
 									/** @type {string} */ (importMetaName),
@@ -151,13 +181,10 @@ class ImportMetaPlugin {
 					}
 
 					// import.meta direct
-					const webpackVersion = Number.parseInt(
-						require("../../package.json").version,
-						10
-					);
 					const importMetaUrl = () =>
 						JSON.stringify(getUrl(parser.state.module));
-					const importMetaWebpackVersion = () => JSON.stringify(webpackVersion);
+					const importMetaWebpackVersion = () =>
+						JSON.stringify(WEBPACK_VERSION);
 					/**
 					 * Import meta unknown property.
 					 * @param {Members} members members
@@ -173,7 +200,7 @@ class ImportMetaPlugin {
 					};
 
 					parser.hooks.typeof
-						.for("import.meta")
+						.for(IMPORT_META)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, JSON.stringify("object"))
@@ -185,7 +212,7 @@ class ImportMetaPlugin {
 						}
 					);
 					parser.hooks.expression
-						.for("import.meta")
+						.for(IMPORT_META)
 						.tap(PLUGIN_NAME, (metaProperty) => {
 							/** @type {RawRuntimeRequirements} */
 							const runtimeRequirements = [];
@@ -254,7 +281,9 @@ class ImportMetaPlugin {
 										);
 										break;
 									case "env":
-										str += `env: ${collectImportMetaEnvDefinitions(compilation).stringify},`;
+										str += `env: ${
+											collectImportMetaEnvDefinitions(compilation).stringify
+										},`;
 										break;
 									default:
 										str += `[${JSON.stringify(
@@ -273,22 +302,22 @@ class ImportMetaPlugin {
 							return true;
 						});
 					parser.hooks.evaluateTypeof
-						.for("import.meta")
+						.for(IMPORT_META)
 						.tap(PLUGIN_NAME, evaluateToString("object"));
-					parser.hooks.evaluateIdentifier.for("import.meta").tap(
+					parser.hooks.evaluateIdentifier.for(IMPORT_META).tap(
 						PLUGIN_NAME,
-						evaluateToIdentifier("import.meta", "import.meta", () => [], true)
+						evaluateToIdentifier(IMPORT_META, IMPORT_META, () => [], true)
 					);
 
 					// import.meta.url
 					parser.hooks.typeof
-						.for("import.meta.url")
+						.for(IMPORT_META_URL)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, JSON.stringify("string"))
 						);
 					parser.hooks.expression
-						.for("import.meta.url")
+						.for(IMPORT_META_URL)
 						.tap(PLUGIN_NAME, (expr) => {
 							const dep = new ConstDependency(
 								importMetaUrl(),
@@ -299,10 +328,10 @@ class ImportMetaPlugin {
 							return true;
 						});
 					parser.hooks.evaluateTypeof
-						.for("import.meta.url")
+						.for(IMPORT_META_URL)
 						.tap(PLUGIN_NAME, evaluateToString("string"));
 					parser.hooks.evaluateIdentifier
-						.for("import.meta.url")
+						.for(IMPORT_META_URL)
 						.tap(PLUGIN_NAME, (expr) =>
 							new BasicEvaluatedExpression()
 								.setString(getUrl(parser.state.module))
@@ -311,26 +340,26 @@ class ImportMetaPlugin {
 
 					// import.meta.webpack
 					parser.hooks.expression
-						.for("import.meta.webpack")
+						.for(IMPORT_META_WEBPACK)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, importMetaWebpackVersion())
 						);
 					parser.hooks.typeof
-						.for("import.meta.webpack")
+						.for(IMPORT_META_WEBPACK)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, JSON.stringify("number"))
 						);
 					parser.hooks.evaluateTypeof
-						.for("import.meta.webpack")
+						.for(IMPORT_META_WEBPACK)
 						.tap(PLUGIN_NAME, evaluateToString("number"));
 					parser.hooks.evaluateIdentifier
-						.for("import.meta.webpack")
-						.tap(PLUGIN_NAME, evaluateToNumber(webpackVersion));
+						.for(IMPORT_META_WEBPACK)
+						.tap(PLUGIN_NAME, evaluateToNumber(WEBPACK_VERSION));
 
 					parser.hooks.expression
-						.for("import.meta.main")
+						.for(IMPORT_META_MAIN)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(
@@ -344,24 +373,24 @@ class ImportMetaPlugin {
 							)
 						);
 					parser.hooks.typeof
-						.for("import.meta.main")
+						.for(IMPORT_META_MAIN)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, JSON.stringify("boolean"))
 						);
 					parser.hooks.evaluateTypeof
-						.for("import.meta.main")
+						.for(IMPORT_META_MAIN)
 						.tap(PLUGIN_NAME, evaluateToString("boolean"));
 
 					// import.meta.env
 					parser.hooks.typeof
-						.for("import.meta.env")
+						.for(IMPORT_META_ENV)
 						.tap(
 							PLUGIN_NAME,
 							toConstantDependency(parser, JSON.stringify("object"))
 						);
 					parser.hooks.expressionMemberChain
-						.for("import.meta")
+						.for(IMPORT_META)
 						.tap(PLUGIN_NAME, (expr, members) => {
 							if (members[0] === "env" && members[1]) {
 								const name = members[1];
@@ -378,7 +407,7 @@ class ImportMetaPlugin {
 							}
 						});
 					parser.hooks.expression
-						.for("import.meta.env")
+						.for(IMPORT_META_ENV)
 						.tap(PLUGIN_NAME, (expr) => {
 							const { stringify } =
 								collectImportMetaEnvDefinitions(compilation);
@@ -392,10 +421,10 @@ class ImportMetaPlugin {
 							return true;
 						});
 					parser.hooks.evaluateTypeof
-						.for("import.meta.env")
+						.for(IMPORT_META_ENV)
 						.tap(PLUGIN_NAME, evaluateToString("object"));
 					parser.hooks.evaluateIdentifier
-						.for("import.meta.env")
+						.for(IMPORT_META_ENV)
 						.tap(PLUGIN_NAME, (expr) =>
 							new BasicEvaluatedExpression()
 								.setTruthy()
@@ -405,7 +434,7 @@ class ImportMetaPlugin {
 
 					// Unknown properties
 					parser.hooks.unhandledExpressionMemberChain
-						.for("import.meta")
+						.for(IMPORT_META)
 						.tap(PLUGIN_NAME, (expr, members) => {
 							// unknown import.meta properties should be determined at runtime
 							if (importMeta === "preserve-unknown") {
@@ -442,6 +471,18 @@ class ImportMetaPlugin {
 								expr.property.type ===
 									(expr.computed ? "Literal" : "Identifier")
 							) {
+								const prop = expr.computed
+									? /** @type {import("estree").Literal} */ (expr.property)
+											.value
+									: /** @type {Identifier} */ (expr.property).name;
+								// dirname/filename are resolved by NodeStuffPlugin; main is a runtime value
+								if (
+									prop === "dirname" ||
+									prop === "filename" ||
+									prop === "main"
+								) {
+									return;
+								}
 								return new BasicEvaluatedExpression()
 									.setUndefined()
 									.setRange(/** @type {Range} */ (expr.range));
@@ -461,3 +502,6 @@ class ImportMetaPlugin {
 }
 
 module.exports = ImportMetaPlugin;
+module.exports.IMPORT_META_DIRNAME = IMPORT_META_DIRNAME;
+module.exports.IMPORT_META_FILENAME = IMPORT_META_FILENAME;
+module.exports.IMPORT_META_NAMES = IMPORT_META_NAMES;

package/lib/dependencies/ImportParserPlugin.js

@@ -12,6 +12,7 @@ const {
 	VariableInfo,
 	getImportAttributes
 } = require("../javascript/JavascriptParser");
+const memoize = require("../util/memoize");
 const traverseDestructuringAssignmentProperties = require("../util/traverseDestructuringAssignmentProperties");
 const ContextDependencyHelpers = require("./ContextDependencyHelpers");
 const { getNonOptionalPart } = require("./HarmonyImportDependency");
@@ -42,6 +43,8 @@ const ImportWeakDependency = require("./ImportWeakDependency");
 /** @typedef {{ references: RawReferencedExports, expression: ImportExpression }} ImportSettings */
 /** @typedef {WeakMap<ImportExpression, RawReferencedExports>} State */
 
+const getHarmonyImportGuard = memoize(() => require("./HarmonyImportGuard"));
+
 /** @type {WeakMap<JavascriptParserState, State>} */
 const parserStateMap = new WeakMap();
 const dynamicImportTag = Symbol("import()");
@@ -177,6 +180,7 @@ class ImportParserPlugin {
 	 * @param {JavascriptParserOptions} options options
 	 */
 	constructor(options) {
+		/** @type {JavascriptParserOptions} */
 		this.options = options;
 	}
 
@@ -452,6 +456,20 @@ class ImportParserPlugin {
 						exports = exportsFromEnumerable(importOptions.webpackExports);
 					}
 				}
+				// `worker` is an internal entry option set only for workers
+				if (
+					importOptions.webpackEntryOptions !== undefined &&
+					typeof importOptions.webpackEntryOptions === "object" &&
+					importOptions.webpackEntryOptions !== null &&
+					importOptions.webpackEntryOptions.worker !== undefined
+				) {
+					parser.state.module.addWarning(
+						new UnsupportedFeatureWarning(
+							"`worker` entry option is not supported in `import()`, it only applies to workers (e.g. `new Worker(new URL(...))`).",
+							/** @type {DependencyLocation} */ (expr.loc)
+						)
+					);
+				}
 			}
 
 			if (
@@ -550,6 +568,7 @@ class ImportParserPlugin {
 					dep.optional = Boolean(parser.scope.inTry);
 					depBlock.addDependency(dep);
 					parser.state.current.addBlock(depBlock);
+					getHarmonyImportGuard().attachDependencyGuards(parser, dep);
 				}
 			} else {
 				if (mode === "weak") {

package/lib/dependencies/ImportWeakDependency.js

@@ -29,6 +29,7 @@ class ImportWeakDependency extends ImportDependency {
 	 */
 	constructor(request, range, referencedExports, phase, attributes) {
 		super(request, range, referencedExports, phase, attributes);
+		/** @type {boolean} */
 		this.weak = true;
 	}
 

package/lib/dependencies/JsonExportsDependency.js

@@ -14,8 +14,8 @@ const NullDependency = require("./NullDependency");
 /** @typedef {import("../ModuleGraph")} ModuleGraph */
 /** @typedef {import("../json/JsonData")} JsonData */
 /** @typedef {import("../json/JsonData").JsonValue} JsonValue */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[JsonData, number]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[JsonData, number]>} ObjectSerializerContext */
 /** @typedef {import("../util/Hash")} Hash */
 
 /**
@@ -79,7 +79,9 @@ class JsonExportsDependency extends NullDependency {
 	 */
 	constructor(data, exportsDepth) {
 		super();
+		/** @type {JsonData} */
 		this.data = data;
+		/** @type {number} */
 		this.exportsDepth = exportsDepth;
 	}
 
@@ -116,9 +118,7 @@ class JsonExportsDependency extends NullDependency {
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.data);
-		write(this.exportsDepth);
+		context.write(this.data).write(this.exportsDepth);
 		super.serialize(context);
 	}
 
@@ -127,10 +127,10 @@ class JsonExportsDependency extends NullDependency {
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {
-		const { read } = context;
-		this.data = read();
-		this.exportsDepth = read();
-		super.deserialize(context);
+		this.data = context.read();
+		const c1 = context.rest;
+		this.exportsDepth = c1.read();
+		super.deserialize(c1.rest);
 	}
 }
 

package/lib/dependencies/LoaderImportDependency.js

@@ -17,6 +17,7 @@ class LoaderImportDependency extends ModuleDependency {
 	 */
 	constructor(request) {
 		super(request);
+		/** @type {boolean} */
 		this.weak = true;
 	}
 

package/lib/dependencies/LocalModule.js

@@ -7,8 +7,8 @@
 
 const makeSerializable = require("../util/makeSerializable");
 
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, number, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, number, boolean]>} ObjectSerializerContext */
 
 class LocalModule {
 	/**
@@ -17,8 +17,11 @@ class LocalModule {
 	 * @param {number} idx index
 	 */
 	constructor(name, idx) {
+		/** @type {string} */
 		this.name = name;
+		/** @type {number} */
 		this.idx = idx;
+		/** @type {boolean} */
 		this.used = false;
 	}
 
@@ -39,11 +42,7 @@ class LocalModule {
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-
-		write(this.name);
-		write(this.idx);
-		write(this.used);
+		context.write(this.name).write(this.idx).write(this.used);
 	}
 
 	/**
@@ -51,11 +50,11 @@ class LocalModule {
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {
-		const { read } = context;
-
-		this.name = read();
-		this.idx = read();
-		this.used = read();
+		this.name = context.read();
+		const c1 = context.rest;
+		this.idx = c1.read();
+		const c2 = c1.rest;
+		this.used = c2.read();
 	}
 }
 

package/lib/dependencies/LocalModuleDependency.js

@@ -12,8 +12,8 @@ const NullDependency = require("./NullDependency");
 /** @typedef {import("../Dependency")} Dependency */
 /** @typedef {import("../DependencyTemplate").DependencyTemplateContext} DependencyTemplateContext */
 /** @typedef {import("../javascript/JavascriptParser").Range} Range */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[LocalModule, Range | undefined, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[LocalModule, Range | undefined, boolean]>} ObjectSerializerContext */
 /** @typedef {import("./LocalModule")} LocalModule */
 
 class LocalModuleDependency extends NullDependency {
@@ -26,8 +26,10 @@ class LocalModuleDependency extends NullDependency {
 	constructor(localModule, range, callNew) {
 		super();
 
+		/** @type {LocalModule} */
 		this.localModule = localModule;
 		this.range = range;
+		/** @type {boolean} */
 		this.callNew = callNew;
 	}
 
@@ -36,11 +38,7 @@ class LocalModuleDependency extends NullDependency {
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-
-		write(this.localModule);
-		write(this.range);
-		write(this.callNew);
+		context.write(this.localModule).write(this.range).write(this.callNew);
 
 		super.serialize(context);
 	}
@@ -50,13 +48,13 @@ class LocalModuleDependency extends NullDependency {
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {
-		const { read } = context;
-
-		this.localModule = read();
-		this.range = read();
-		this.callNew = read();
+		this.localModule = context.read();
+		const c1 = context.rest;
+		this.range = c1.read();
+		const c2 = c1.rest;
+		this.callNew = c2.read();
 
-		super.deserialize(context);
+		super.deserialize(c2.rest);
 	}
 }
 

package/lib/dependencies/ModuleDecoratorDependency.js

@@ -16,8 +16,8 @@ const NullDependency = require("./NullDependency");
 /** @typedef {import("../Dependency").UpdateHashContext} UpdateHashContext */
 /** @typedef {import("../DependencyTemplate").DependencyTemplateContext} DependencyTemplateContext */
 /** @typedef {import("../ModuleGraph")} ModuleGraph */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, boolean]>} ObjectSerializerContext */
 /** @typedef {import("../util/Hash")} Hash */
 /** @typedef {import("../util/runtime").RuntimeSpec} RuntimeSpec */
 
@@ -29,7 +29,9 @@ class ModuleDecoratorDependency extends NullDependency {
 	 */
 	constructor(decorator, allowExportsAccess) {
 		super();
+		/** @type {string} */
 		this.decorator = decorator;
+		/** @type {boolean} */
 		this.allowExportsAccess = allowExportsAccess;
 		/** @type {undefined | string} */
 		this._hashUpdate = undefined;
@@ -85,9 +87,7 @@ class ModuleDecoratorDependency extends NullDependency {
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.decorator);
-		write(this.allowExportsAccess);
+		context.write(this.decorator).write(this.allowExportsAccess);
 		super.serialize(context);
 	}
 
@@ -96,10 +96,10 @@ class ModuleDecoratorDependency extends NullDependency {
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {
-		const { read } = context;
-		this.decorator = read();
-		this.allowExportsAccess = read();
-		super.deserialize(context);
+		this.decorator = context.read();
+		const c1 = context.rest;
+		this.allowExportsAccess = c1.read();
+		super.deserialize(c1.rest);
 	}
 }
 

package/lib/dependencies/ModuleDependency.js

@@ -23,13 +23,18 @@ class ModuleDependency extends Dependency {
 	 */
 	constructor(request, sourceOrder) {
 		super();
+		/** @type {string} */
 		this.request = request;
+		/** @type {string} */
 		this.userRequest = request;
+		/** @type {number | undefined} */
 		this.sourceOrder = sourceOrder;
 		/** @type {Range | undefined} */
 		this.range = undefined;
 		/** @type {undefined | string} */
 		this._context = undefined;
+		/** @type {boolean} */
+		this.weak = false;
 	}
 
 	/**
@@ -84,6 +89,7 @@ class ModuleDependency extends Dependency {
 		write(this._context);
 		write(this.range);
 		write(this.sourceOrder);
+		write(this.weak);
 		super.serialize(context);
 	}
 
@@ -98,6 +104,7 @@ class ModuleDependency extends Dependency {
 		this._context = read();
 		this.range = read();
 		this.sourceOrder = read();
+		this.weak = read();
 		super.deserialize(context);
 	}
 }

package/lib/dependencies/ModuleHotAcceptDependency.js

@@ -20,6 +20,7 @@ class ModuleHotAcceptDependency extends ModuleDependency {
 	constructor(request, range) {
 		super(request, Infinity);
 		this.range = range;
+		/** @type {boolean} */
 		this.weak = true;
 	}
 

package/lib/dependencies/ModuleHotDeclineDependency.js

@@ -21,6 +21,7 @@ class ModuleHotDeclineDependency extends ModuleDependency {
 		super(request);
 
 		this.range = range;
+		/** @type {boolean} */
 		this.weak = true;
 	}
 

package/lib/dependencies/ModuleInitFragmentDependency.js

@@ -12,8 +12,8 @@ const NullDependency = require("./NullDependency");
 /** @typedef {import("webpack-sources").ReplaceSource} ReplaceSource */
 /** @typedef {import("../Dependency")} Dependency */
 /** @typedef {import("../DependencyTemplate").DependencyTemplateContext} DependencyTemplateContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, string[], string | undefined]>} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, string[], string | undefined]>} ObjectDeserializerContext */
 /**
  * A dependency that adds an init fragment to the module
  */
@@ -26,8 +26,11 @@ class ModuleInitFragmentDependency extends NullDependency {
 	 */
 	constructor(initCode, runtimeRequirements, key) {
 		super();
+		/** @type {string} */
 		this.initCode = initCode;
+		/** @type {string[]} */
 		this.runtimeRequirements = runtimeRequirements;
+		/** @type {string | undefined} */
 		this.key = key;
 	}
 
@@ -36,10 +39,10 @@ class ModuleInitFragmentDependency extends NullDependency {
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.initCode);
-		write(this.runtimeRequirements);
-		write(this.key);
+		context
+			.write(this.initCode)
+			.write(this.runtimeRequirements)
+			.write(this.key);
 		super.serialize(context);
 	}
 
@@ -48,11 +51,12 @@ class ModuleInitFragmentDependency extends NullDependency {
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {
-		const { read } = context;
-		this.initCode = read();
-		this.runtimeRequirements = read();
-		this.key = read();
-		super.deserialize(context);
+		this.initCode = context.read();
+		const c1 = context.rest;
+		this.runtimeRequirements = c1.read();
+		const c2 = c1.rest;
+		this.key = c2.read();
+		super.deserialize(c2.rest);
 	}
 }