reuseExistingChunk is able to reuse an entry chunk when it's the only…

… selected chunk
webpack · Jan 19, 2021 · 2b3489d · 2b3489d
1 parent 8918ab1
commit 2b3489d
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 1 deletion.
diff --git a/lib/optimize/SplitChunksPlugin.js b/lib/optimize/SplitChunksPlugin.js
@@ -1298,7 +1298,10 @@ module.exports = class SplitChunksPlugin {
 								) {
 									continue;
 								}
-								if (chunkGraph.getNumberOfEntryModules(chunk) > 0) {
+								if (
+									item.chunks.size > 1 &&
+									chunkGraph.getNumberOfEntryModules(chunk) > 0
+								) {
 									continue;
 								}
 								for (const module of item.modules) {

diff --git a/test/configCases/split-chunks/vendor-only-entrypoint/node_modules/vendor.js b/test/configCases/split-chunks/vendor-only-entrypoint/node_modules/vendor.js
diff --git a/test/configCases/split-chunks/vendor-only-entrypoint/test.config.js b/test/configCases/split-chunks/vendor-only-entrypoint/test.config.js
@@ -0,0 +1,5 @@
+module.exports = {
+	findBundle: function (i, options) {
+		return ["runtime~vendor.js", "vendor.js"];
+	}
+};
diff --git a/test/configCases/split-chunks/vendor-only-entrypoint/webpack.config.js b/test/configCases/split-chunks/vendor-only-entrypoint/webpack.config.js
@@ -0,0 +1,19 @@
+/** @type {import("../../../../").Configuration} */
+module.exports = {
+	entry: {
+		vendor: "vendor"
+	},
+	output: {
+		filename: "[name].js"
+	},
+	target: "web",
+	optimization: {
+		chunkIds: "named",
+		runtimeChunk: true,
+		splitChunks: {
+			chunks: "all",
+			minSize: 10,
+			enforceSizeThreshold: 20
+		}
+	}
+};