fix: correct importLoaders count for node target (#6554)

Author: chenjiahan

packages/core/src/plugins/css.ts

@@ -227,19 +227,16 @@ const getPostcssLoaderOptions = async ({
 
 const getCSSLoaderOptions = ({
   config,
-  importLoaders,
   localIdentName,
   emitCss,
 }: {
   config: NormalizedEnvironmentConfig;
-  importLoaders: number;
   localIdentName: string;
   emitCss: boolean;
 }) => {
   const { cssModules } = config.output;
 
   const defaultOptions: CSSLoaderOptions = {
-    importLoaders,
     modules: {
       ...cssModules,
       localIdentName,
@@ -328,7 +325,10 @@ export const pluginCss = (): RsbuildPlugin => ({
         }
 
         // Number of loaders applied before css-loader for `@import` at-rules
-        let importLoaders = 0;
+        const importLoaders = {
+          normal: 0,
+          inline: 0,
+        };
 
         // Update the normal CSS rule and the inline CSS rule
         const updateRules = (
@@ -351,7 +351,10 @@ export const pluginCss = (): RsbuildPlugin => ({
           api.context.bundlerType === 'rspack' &&
           config.tools.lightningcssLoader !== false
         ) {
-          importLoaders++;
+          if (emitCss) {
+            importLoaders.normal++;
+          }
+          importLoaders.inline++;
 
           const { minifyCss } = parseMinifyOptions(config);
 
@@ -390,7 +393,11 @@ export const pluginCss = (): RsbuildPlugin => ({
           typeof postcssLoaderOptions.postcssOptions === 'function' ||
           postcssLoaderOptions.postcssOptions?.plugins?.length
         ) {
-          importLoaders++;
+          if (emitCss) {
+            importLoaders.normal++;
+          }
+          importLoaders.inline++;
+
           const postcssLoaderPath = getCompiledPath('postcss-loader');
 
           updateRules(
@@ -408,21 +415,27 @@ export const pluginCss = (): RsbuildPlugin => ({
         const localIdentName = getCSSModulesLocalIdentName(config, isProd);
         const cssLoaderOptions = getCSSLoaderOptions({
           config,
-          importLoaders,
           localIdentName,
           emitCss,
         });
 
         updateRules((rule, type) => {
-          rule.use(CHAIN_ID.USE.CSS).options(
-            type === 'inline'
-              ? ({
-                  ...cssLoaderOptions,
-                  exportType: 'string',
-                  modules: false,
-                } satisfies CSSLoaderOptions)
-              : cssLoaderOptions,
-          );
+          let finalOptions = cssLoaderOptions;
+
+          if (type === 'inline') {
+            finalOptions = {
+              ...cssLoaderOptions,
+              exportType: 'string',
+              modules: false,
+              importLoaders: importLoaders.inline,
+            };
+          } else {
+            finalOptions = {
+              ...cssLoaderOptions,
+              importLoaders: importLoaders.normal,
+            };
+          }
+          rule.use(CHAIN_ID.USE.CSS).options(finalOptions);
 
           // CSS imports should always be treated as sideEffects
           rule.sideEffects(true);

packages/core/tests/__snapshots__/css.test.ts.snap

@@ -133,7 +133,7 @@ exports[`plugin-css injectStyles > should apply ignoreCssLoader when injectStyle
           {
             "loader": "<ROOT>/packages/core/compiled/css-loader/index.js",
             "options": {
-              "importLoaders": 1,
+              "importLoaders": 0,
               "modules": {
                 "auto": true,
                 "exportGlobals": false,

packages/core/tests/__snapshots__/default.test.ts.snap

@@ -1136,7 +1136,7 @@ exports[`applyDefaultPlugins > should apply default plugins correctly when targe
           {
             "loader": "<ROOT>/packages/core/compiled/css-loader/index.js",
             "options": {
-              "importLoaders": 1,
+              "importLoaders": 0,
               "modules": {
                 "auto": true,
                 "exportGlobals": false,

packages/core/tests/__snapshots__/environments.test.ts.snap

@@ -1966,7 +1966,7 @@ exports[`environment config > tools.rspack / bundlerChain can be configured in e
             {
               "loader": "<ROOT>/packages/core/compiled/css-loader/index.js",
               "options": {
-                "importLoaders": 1,
+                "importLoaders": 0,
                 "modules": {
                   "auto": true,
                   "exportGlobals": false,