fix: chunk groups' order when building mpa

crates/mako/src/config.rs

@@ -28,7 +28,7 @@ mod tree_shaking;
 mod umd;
 mod watch;
 
-use std::collections::HashMap;
+use std::collections::{BTreeMap, HashMap};
 use std::fmt;
 use std::path::{Path, PathBuf};
 
@@ -128,7 +128,7 @@ pub enum Platform {
 #[derive(Deserialize, Serialize, Debug)]
 #[serde(rename_all = "camelCase")]
 pub struct Config {
-    pub entry: HashMap<String, PathBuf>,
+    pub entry: BTreeMap<String, PathBuf>,
     pub output: OutputConfig,
     pub resolve: ResolveConfig,
     #[serde(deserialize_with = "deserialize_manifest", default)]

crates/mako/src/module_graph.rs

@@ -1,4 +1,4 @@
-use std::collections::{HashMap, HashSet};
+use std::collections::{BTreeSet, HashMap, HashSet};
 use std::fmt;
 
 use fixedbitset::FixedBitSet;
@@ -15,15 +15,15 @@ use crate::module::{Dependencies, Dependency, Module, ModuleId};
 pub struct ModuleGraph {
     pub id_index_map: HashMap<ModuleId, NodeIndex<DefaultIx>>,
     pub graph: StableDiGraph<Module, Dependencies>,
-    entries: HashSet<ModuleId>,
+    entries: BTreeSet<ModuleId>,
 }
 
 impl ModuleGraph {
     pub fn new() -> Self {
         Self {
             id_index_map: HashMap::new(),
             graph: StableDiGraph::new(),
-            entries: HashSet::new(),
+            entries: BTreeSet::new(),
         }
     }
 

e2e/fixtures/css.css-merge-multi-entries/expect.js

@@ -0,0 +1,18 @@
+const assert = require("assert");
+const { parseBuildResult } = require("../../../scripts/test-utils");
+const { files } = parseBuildResult(__dirname);
+
+assert(
+  files['common.css'].includes(`
+body::after {
+  content: "common_1";
+}
+body::after {
+  content: "common_2";
+}
+body::after {
+  content: "common_3";
+}
+`.trim()),
+  "css merge in mpa should works"
+);

e2e/fixtures/css.css-merge-multi-entries/mako.config.json

@@ -0,0 +1,21 @@
+{
+  "entry": {
+    "a": "./src/a.ts",
+    "b": "./src/b.ts",
+    "c": "./src/c.ts"
+  },
+  "codeSplitting": {
+    "strategy": "advanced",
+    "options": {
+      "minSize": 1,
+      "groups": [
+        {
+          "name": "common",
+          "allowChunks": "entry",
+          "minChunks": 2,
+          "minSize": 1
+        }
+      ]
+    }
+  }
+}

e2e/fixtures/css.css-merge-multi-entries/src/a.css

@@ -0,0 +1,3 @@
+body {
+  color: azure;
+}

e2e/fixtures/css.css-merge-multi-entries/src/a.ts

@@ -0,0 +1,6 @@
+import "./common_1.css";
+import "./common_2.css";
+import "./common_3.css";
+import "./a.css";
+console.log('a');
+

e2e/fixtures/css.css-merge-multi-entries/src/b.css

@@ -0,0 +1,3 @@
+body {
+  color: blue;
+}

e2e/fixtures/css.css-merge-multi-entries/src/b.ts

@@ -0,0 +1,5 @@
+import "./common_2.css";
+import "./common_1.css";
+import "./common_3.css";
+import "./b.css";
+console.log('b');

e2e/fixtures/css.css-merge-multi-entries/src/c.css

@@ -0,0 +1,3 @@
+body {
+  color: cyan;
+}

e2e/fixtures/css.css-merge-multi-entries/src/c.ts

@@ -0,0 +1,4 @@
+import "./common_2.css";
+import "./common_3.css";
+import "./common_1.css";
+console.log("c");

e2e/fixtures/css.css-merge-multi-entries/src/common_1.css

@@ -0,0 +1,3 @@
+body::after {
+  content: "common_1";
+}

e2e/fixtures/css.css-merge-multi-entries/src/common_2.css

@@ -0,0 +1,3 @@
+body::after {
+  content: "common_2";
+}

e2e/fixtures/css.css-merge-multi-entries/src/common_3.css

@@ -0,0 +1,3 @@
+body::after {
+  content: "common_3";
+}