Merge pull request #13567 from webpack/bugfix/initial-chunk-ids-with-…

…depend-on

getAllInitialChunks really returns all of them

lib/Chunk.js

@@ -598,8 +598,12 @@ class Chunk {
 	 */
 	getAllInitialChunks() {
 		const chunks = new Set();
-		for (const group of this.groupsIterable) {
-			for (const c of group.chunks) chunks.add(c);
+		const queue = new Set(this.groupsIterable);
+		for (const group of queue) {
+			if (group.isInitial()) {
+				for (const c of group.chunks) chunks.add(c);
+				for (const g of group.childrenIterable) queue.add(g);
+			}
 		}
 		return chunks;
 	}

test/configCases/entry/depend-on-non-js/a.css

@@ -0,0 +1 @@
+module.exports = [[module.id, "body { background-color: green; }"]];

test/configCases/entry/depend-on-non-js/a.js

@@ -0,0 +1 @@
+if (Math.random() < 0) require("./a.css");

test/configCases/entry/depend-on-non-js/b.css

@@ -0,0 +1 @@
+module.exports = [[module.id, "body { color: red; }"]];

test/configCases/entry/depend-on-non-js/b.js

@@ -0,0 +1,3 @@
+if (Math.random() < 0) require("./b.css");
+
+it("should run the test", () => {});

test/configCases/entry/depend-on-non-js/test.config.js

@@ -0,0 +1,5 @@
+module.exports = {
+	findBundle: function () {
+		return ["./runtime.js", "./a.js", "./b.js"];
+	}
+};

test/configCases/entry/depend-on-non-js/webpack.config.js

@@ -0,0 +1,38 @@
+const MiniCssPlugin = require("mini-css-extract-plugin");
+
+/** @type {import("../../../../").Configuration} */
+module.exports = {
+	entry: {
+		a: "./a.js",
+		b: { import: "./b.js", dependOn: "a" }
+	},
+	module: {
+		rules: [
+			{
+				test: /\.css$/,
+				loader: MiniCssPlugin.loader
+			}
+		]
+	},
+	output: {
+		filename: "[name].js"
+	},
+	optimization: {
+		runtimeChunk: "single",
+		splitChunks: {
+			chunks: "all",
+			cacheGroups: {
+				styles: {
+					type: "css/mini-extract",
+					enforce: true
+				}
+			}
+		}
+	},
+	target: "web",
+	plugins: [
+		new MiniCssPlugin({
+			experimentalUseImportModule: true
+		})
+	]
+};