fix(rsc): skip client entry mismatch without injections (#14018)

Author: SyMind

crates/rspack_plugin_rsc/src/client_plugin.rs

@@ -638,6 +638,13 @@ async fn make(&self, compilation: &mut Compilation) -> Result<()> {
 
   for (entry_name, entry_state) in &plugin_state.entries {
     let client_modules = &entry_state.injected_client_entries;
+    if entry_state.injected_client_entries.is_empty()
+      && !entry_state.has_css_imports_by_server_entry()
+      && entry_state.root_css_imports.is_empty()
+    {
+      continue;
+    }
+
     if compilation.entries.get(entry_name.as_ref()).is_none() {
       compilation.push_diagnostic(Diagnostic::error(
         "RSC Client Entry Mismatch".to_string(),
@@ -688,12 +695,12 @@ async fn make(&self, compilation: &mut Compilation) -> Result<()> {
         .add_dependency(dependency);
     }
 
-    #[allow(clippy::unwrap_used)]
-    let entry_data = compilation.entries.get_mut(entry_name.as_ref()).unwrap();
-    entry_data.dependencies.append(&mut entry_dependencies);
-    entry_data
-      .include_dependencies
-      .append(&mut include_dependencies);
+    if let Some(entry_data) = compilation.entries.get_mut(entry_name.as_ref()) {
+      entry_data.dependencies.append(&mut entry_dependencies);
+      entry_data
+        .include_dependencies
+        .append(&mut include_dependencies);
+    }
   }
 
   Ok(())

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/rspack.config.js

@@ -0,0 +1,82 @@
+const { experiments } = require('@rspack/core');
+
+const { createPlugins, Layers } = experiments.rsc;
+const { ServerPlugin, ClientPlugin } = createPlugins();
+
+const swcLoaderRule = {
+  test: /\.jsx?$/,
+  use: [
+    {
+      loader: 'builtin:swc-loader',
+      options: {
+        detectSyntax: 'auto',
+        rspackExperiments: {
+          reactServerComponents: true,
+        },
+      },
+    },
+  ],
+};
+
+module.exports = [
+  {
+    mode: 'production',
+    target: 'node',
+    output: {
+      filename: 'server-[name].js',
+    },
+    entry: {
+      main: {
+        import: './src/framework/entry.ssr.js',
+      },
+      rscService: {
+        import: './src/framework/rsc-service.rsc.js',
+      },
+    },
+    resolve: {
+      extensions: ['...', '.jsx'],
+    },
+    module: {
+      rules: [
+        swcLoaderRule,
+        {
+          resource: /[\\/]framework[\\/].*\.ssr\.js$/,
+          layer: Layers.ssr,
+        },
+        {
+          resource: /[\\/]framework[\\/].*\.rsc\.js$/,
+          layer: Layers.rsc,
+          resolve: {
+            conditionNames: ['react-server', '...'],
+          },
+        },
+        {
+          issuerLayer: Layers.rsc,
+          resolve: {
+            conditionNames: ['react-server', '...'],
+          },
+        },
+      ],
+    },
+    plugins: [new ServerPlugin()],
+  },
+  {
+    mode: 'production',
+    target: 'web',
+    output: {
+      filename: 'client-[name].js',
+    },
+    entry: {
+      main: {
+        import: './src/framework/entry.client.js',
+      },
+    },
+    resolve: {
+      extensions: ['...', '.jsx'],
+    },
+    module: {
+      rules: [swcLoaderRule],
+    },
+    plugins: [new ClientPlugin()],
+  },
+];

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/App.js

@@ -0,0 +1,3 @@
+"use server-entry";
+
+export const App = () => "main";

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/RscService.js

@@ -0,0 +1,5 @@
+"use server-entry";
+
+const rspackRsc = import.meta.rspackRsc;
+
+export const RscService = () => rspackRsc.loadCss().length;

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/framework/entry.client.js

@@ -0,0 +1 @@
+export {};

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/framework/entry.rsc.js

@@ -0,0 +1,7 @@
+import { App } from '../App';
+
+export const value = App();
+
+it('should build the main RSC entry', () => {
+  expect(value).toBe('main');
+});

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/framework/entry.ssr.js

@@ -0,0 +1,3 @@
+import { value } from './entry.rsc';
+
+export default value;

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/src/framework/rsc-service.rsc.js

@@ -0,0 +1,3 @@
+import { RscService } from '../RscService';
+
+export const value = RscService();

tests/rspack-test/configCases/rsc-plugin/rsc-service-no-client-injection/test.config.js

@@ -0,0 +1,6 @@
+/** @type {import("../../../..").TConfigCaseConfig} */
+module.exports = {
+  findBundle: function () {
+    return ['server-main.js'];
+  },
+};