Merge pull request #15577 from webpack/support-node-prefix-in-build-deps

support node: prefix in esm build dependencies
webpack · Mar 28, 2022 · 84aa5cf · 84aa5cf
2 parents 86a8bd9 + 8df4d5c
commit 84aa5cf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/FileSystemInfo.js b/lib/FileSystemInfo.js
@@ -1673,6 +1673,10 @@ class FileSystemInfo {
 													// e.g. import.meta
 													continue;
 												}
+
+												// we should not track Node.js build dependencies
+												if (dependency.startsWith("node:")) continue;
+
 												push({
 													type: RBDT_RESOLVE_ESM_FILE,
 													context,

diff --git a/test/BuildDependencies.longtest.js b/test/BuildDependencies.longtest.js
@@ -133,7 +133,7 @@ describe("BuildDependencies", () => {
 		);
 		fs.writeFileSync(
 			path.resolve(inputDirectory, "esm-async-dependency.mjs"),
-			"export default 0;"
+			'import path from "node:path"; export default 0;'
 		);
 		await exec("0", {
 			invalidBuildDependencies: true,