fix: maxInitialRequests should exists in cache groups

crates/node_binding/binding.d.ts

@@ -979,6 +979,8 @@ export interface RawCacheGroupOptions {
   maxSize?: number | RawSplitChunkSizes
   maxAsyncSize?: number | RawSplitChunkSizes
   maxInitialSize?: number | RawSplitChunkSizes
+  maxAsyncRequests?: number
+  maxInitialRequests?: number
   name?: string | false | Function
   reuseExistingChunk?: boolean
   enforce?: boolean

crates/rspack_binding_options/src/options/raw_split_chunks/mod.rs

@@ -39,8 +39,8 @@ pub struct RawSplitChunksOptions {
   pub chunks: Option<Chunks>,
   pub used_exports: Option<bool>,
   pub automatic_name_delimiter: Option<String>,
-  pub max_async_requests: Option<u32>,
-  pub max_initial_requests: Option<u32>,
+  pub max_async_requests: Option<f64>,
+  pub max_initial_requests: Option<f64>,
   pub default_size_types: Vec<String>,
   pub min_chunks: Option<u32>,
   pub hide_path_info: Option<bool>,
@@ -90,6 +90,8 @@ pub struct RawCacheGroupOptions {
   pub max_size: Option<Either<f64, RawSplitChunkSizes>>,
   pub max_async_size: Option<Either<f64, RawSplitChunkSizes>>,
   pub max_initial_size: Option<Either<f64, RawSplitChunkSizes>>,
+  pub max_async_requests: Option<f64>,
+  pub max_initial_requests: Option<f64>,
   #[napi(ts_type = "string | false | Function")]
   #[derivative(Debug = "ignore")]
   pub name: Option<RawChunkOptionName>,
@@ -214,10 +216,8 @@ impl From<RawSplitChunksOptions> for rspack_plugin_split_chunks::PluginOptions {
               .unwrap_or(overall_automatic_name_delimiter.clone()),
             filename: v.filename.map(Filename::from),
             reuse_existing_chunk: v.reuse_existing_chunk.unwrap_or(false),
-            // TODO(hyf0): the non-enforced default value should be 30
-            // I would set align default value with Webpack when the options is exposed to users
-            max_async_requests: u32::MAX,
-            max_initial_requests: u32::MAX,
+            max_async_requests: v.max_async_requests.unwrap_or(f64::INFINITY),
+            max_initial_requests: v.max_initial_requests.unwrap_or(f64::INFINITY),
             max_async_size,
             max_initial_size,
             r#type,

crates/rspack_plugin_split_chunks/src/options/cache_group.rs

@@ -36,8 +36,8 @@ pub struct CacheGroup {
   /// number of referenced chunks
   pub min_chunks: u32,
   pub id_hint: String,
-  pub max_initial_requests: u32,
-  pub max_async_requests: u32,
+  pub max_initial_requests: f64, // f64 for compat js Infinity
+  pub max_async_requests: f64,   // f64 for compat js Infinity
   pub max_async_size: SplitChunkSizes,
   pub max_initial_size: SplitChunkSizes,
   pub filename: Option<Filename>,

crates/rspack_plugin_split_chunks/src/plugin/max_request.rs

@@ -24,7 +24,7 @@ impl SplitChunksPlugin {
         let allowed_max_request = if chunk.is_only_initial(chunk_group_db) {
           cache_group.max_initial_requests
         } else if chunk.can_be_initial(chunk_group_db) {
-          u32::max(
+          f64::max(
             cache_group.max_initial_requests,
             cache_group.max_async_requests,
           )
@@ -54,7 +54,7 @@ impl SplitChunksPlugin {
           .map(|requests| requests as u32)
           .unwrap_or_default();
 
-        if actually_requests >= allowed_max_request {
+        if actually_requests as f64 >= allowed_max_request {
           Some(chunk.ukey)
         } else {
           None

packages/rspack-test-tools/tests/configCases/split-chunks-common/rspack-issue-7509/index.js

@@ -0,0 +1,14 @@
+it("should not split chunk when maxInitialRequests set to only 1", function () {
+	const a = require("./node_modules/a");
+	expect(a).toBe("a");
+	const b = require("./node_modules/b");
+	expect(b).toBe("b");
+	const c = require("./node_modules/c");
+	expect(c).toBe("c");
+	const d = require("./node_modules/d");
+	expect(d).toBe("d");
+
+	const fs = require('fs')
+	const files = fs.readdirSync(__dirname)
+	expect(files.filter(file => file.endsWith('.js')).length).toBe(1)
+});

packages/rspack-test-tools/tests/configCases/split-chunks-common/rspack-issue-7509/rspack.config.js

@@ -0,0 +1,20 @@
+/** @type {import("@rspack/core").Configuration} */
+module.exports = {
+	optimization: {
+		chunkIds: 'named',
+		splitChunks: {
+			maxInitialRequests: 10,
+			cacheGroups: {
+				vendors: {
+					chunks: 'all',
+					test: /node_modules/,
+					minSize: 0,
+					filename: 'split-[name].js',
+
+					// should override the splitChunks.maxInitialRequests
+					maxInitialRequests: 1,
+				}
+			}
+		}
+	}
+}

packages/rspack/src/config/zod.ts

@@ -1244,6 +1244,8 @@ const sharedOptimizationSplitChunksCacheGroup = {
 	maxSize: optimizationSplitChunksSizes.optional(),
 	maxAsyncSize: optimizationSplitChunksSizes.optional(),
 	maxInitialSize: optimizationSplitChunksSizes.optional(),
+	maxAsyncRequests: z.number().optional(),
+	maxInitialRequests: z.number().optional(),
 	automaticNameDelimiter: z.string().optional()
 };
 const optimizationSplitChunksCacheGroup = z.strictObject({
@@ -1272,8 +1274,6 @@ const optimizationSplitChunksOptions = z.strictObject({
 	cacheGroups: z
 		.record(z.literal(false).or(optimizationSplitChunksCacheGroup))
 		.optional(),
-	maxAsyncRequests: z.number().optional(),
-	maxInitialRequests: z.number().optional(),
 	fallbackCacheGroup: z
 		.strictObject({
 			chunks: optimizationSplitChunksChunks.optional(),