JavaScriptExample
diff --git a/‎lib/optimize/SplitChunksPlugin.js‎
Lines changed: 24 additions & 15 deletions b/‎lib/optimize/SplitChunksPlugin.js‎
Lines changed: 24 additions & 15 deletions
diff --git a/‎test/statsCases/split-chunks/expected.txt‎
Lines changed: 1 addition & 1 deletion b/‎test/statsCases/split-chunks/expected.txt‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎test/statsCases/split-chunks/webpack.config.js‎
Lines changed: 2 additions & 2 deletions b/‎test/statsCases/split-chunks/webpack.config.js‎
Lines changed: 2 additions & 2 deletions
@@ -85,7 +85,9 @@ module.exports = class SplitChunksPlugin {
 
 	static normalizeOptions(options = {}) {
 		return {
-			chunks: options.chunks || "all",
+			chunksFilter: SplitChunksPlugin.normalizeChunksFilter(
+				options.chunks || "all"
+			),
 			minSize: options.minSize || 0,
 			minChunks: options.minChunks || 1,
 			maxAsyncRequests: options.maxAsyncRequests || 1,
@@ -135,6 +137,19 @@ module.exports = class SplitChunksPlugin {
 		if (typeof name === "function") return name;
 	}
 
+	static normalizeChunksFilter(chunks) {
+		if (chunks === "initial") {
+			return chunk => chunk.canBeInitial();
+		}
+		if (chunks === "async") {
+			return chunk => !chunk.canBeInitial();
+		}
+		if (chunks === "all") {
+			return () => true;
+		}
+		if (typeof chunks === "function") return chunks;
+	}
+
 	static normalizeCacheGroups({ cacheGroups, automaticNameDelimiter }) {
 		if (typeof cacheGroups === "function") {
 			return cacheGroups;
@@ -174,7 +189,9 @@ module.exports = class SplitChunksPlugin {
 								name: option.name,
 								automaticNameDelimiter
 							}),
-							chunks: option.chunks,
+							chunksFilter: SplitChunksPlugin.normalizeChunksFilter(
+								option.chunks
+							),
 							enforce: option.enforce,
 							minSize: option.minSize,
 							minChunks: option.minChunks,
@@ -264,7 +281,8 @@ module.exports = class SplitChunksPlugin {
 							const cacheGroup = {
 								key: cacheGroupSource.key,
 								priority: cacheGroupSource.priority || 0,
-								chunks: cacheGroupSource.chunks || this.options.chunks,
+								chunksFilter:
+									cacheGroupSource.chunksFilter || this.options.chunksFilter,
 								minSize:
 									cacheGroupSource.minSize !== undefined
 										? cacheGroupSource.minSize
@@ -304,18 +322,9 @@ module.exports = class SplitChunksPlugin {
 								// Break if minimum number of chunks is not reached
 								if (chunkIndices.length < cacheGroup.minChunks) continue;
 								// Select chunks by configuration
-								const selectedChunks =
-									typeof cacheGroup.chunks === "function"
-										? cacheGroup.chunks(Array.from(chunkCombination))
-										: cacheGroup.chunks === "initial"
-											? Array.from(chunkCombination).filter(chunk =>
-													chunk.canBeInitial()
-												)
-											: cacheGroup.chunks === "async"
-												? Array.from(chunkCombination).filter(
-														chunk => !chunk.canBeInitial()
-													)
-												: Array.from(chunkCombination);
+								const selectedChunks = Array.from(chunkCombination).filter(
+									cacheGroup.chunksFilter
+								);
 								// Break if minimum number of chunks is not reached
 								if (selectedChunks.length < cacheGroup.minChunks) continue;
 								// Determine name for split chunk
 
@@ -258,7 +258,7 @@ Child name-too-long:
         [0] ./d.js 20 bytes {1} {10} {11} {12} [built]
         [1] ./f.js 20 bytes {2} {11} {12} [built]
         [5] ./c.js 72 bytes {7} {12} [built]
-Child chunks-selector:
+Child custom-chunks-filter:
     Entrypoint main = default/main.js
     Entrypoint a = default/a.js
     Entrypoint b = default/vendors~async-a~async-b~async-c~b~c.js default/vendors~async-a~async-b~b.js default/b.js
 
@@ -101,7 +101,7 @@ module.exports = [
 	},
 
 	{
-		name: "chunks-selector",
+		name: "custom-chunks-filter",
 		mode: "production",
 		entry: {
 			main: "./",
@@ -115,7 +115,7 @@ module.exports = [
 		optimization: {
 			splitChunks: {
 				minSize: 0,
-				chunks: chunks => chunks.filter(chunk => chunk.name !== "a")
+				chunks: chunk => chunk.name !== "a"
 			}
 		},
 		stats