test: add redirect-style-false style-inject

Author: SoonIter

packages/core/src/css/cssConfig.ts

@@ -116,16 +116,7 @@ const pluginLibCss = (rootDir: string): RsbuildPlugin => ({
   name: pluginName,
   setup(api) {
     api.modifyBundlerChain((config, { CHAIN_ID }) => {
-      const cssExtract = CHAIN_ID.PLUGIN.MINI_CSS_EXTRACT;
-      config.plugins.delete(cssExtract);
-      config
-        .plugin(RemoveCssExtractAssetPlugin.name)
-        .use(RemoveCssExtractAssetPlugin, [
-          {
-            include: new RegExp(`^${RSLIB_CSS_ENTRY_FLAG}`),
-          },
-        ]);
-
+      let isUsingCssExtract = false;
       for (const ruleId of [
         CHAIN_ID.RULE.CSS,
         CHAIN_ID.RULE.SASS,
@@ -134,6 +125,7 @@ const pluginLibCss = (rootDir: string): RsbuildPlugin => ({
       ]) {
         const rule = config.module.rule(ruleId);
         if (rule.uses.has(CHAIN_ID.USE.MINI_CSS_EXTRACT)) {
+          isUsingCssExtract = true;
           rule
             .use(CHAIN_ID.USE.MINI_CSS_EXTRACT)
             .loader(require.resolve('./libCssExtractLoader.js'))
@@ -142,6 +134,18 @@ const pluginLibCss = (rootDir: string): RsbuildPlugin => ({
             });
         }
       }
+
+      if (isUsingCssExtract) {
+        const cssExtract = CHAIN_ID.PLUGIN.MINI_CSS_EXTRACT;
+        config.plugins.delete(cssExtract);
+        config
+          .plugin(RemoveCssExtractAssetPlugin.name)
+          .use(RemoveCssExtractAssetPlugin, [
+            {
+              include: new RegExp(`^${RSLIB_CSS_ENTRY_FLAG}`),
+            },
+          ]);
+      }
     });
   },
 });

tests/integration/style/redirect-style-false/index.test.ts

@@ -0,0 +1,27 @@
+import { buildAndGetResults } from 'test-helper';
+import { expectFileContainContent } from 'test-helper/vitest';
+import { expect, test } from 'vitest';
+
+test('should extract css successfully when using redirect.style = false', async () => {
+  const fixturePath = __dirname;
+  const { contents } = await buildAndGetResults(fixturePath, 'js');
+  const esmFiles = Object.keys(contents.esm);
+  expect(esmFiles).toMatchInlineSnapshot(`
+    [
+      "<ROOT>/tests/integration/style/redirect-style-false/dist/esm/index.js",
+    ]
+  `);
+  expectFileContainContent(contents.esm, 'index.js', 'import "./index.less";');
+
+  const cjsFiles = Object.keys(contents.cjs);
+  expect(cjsFiles).toMatchInlineSnapshot(`
+    [
+      "<ROOT>/tests/integration/style/redirect-style-false/dist/cjs/index.cjs",
+    ]
+  `);
+  expectFileContainContent(
+    contents.cjs,
+    'index.cjs',
+    'require("./index.less")',
+  );
+});

tests/integration/style/redirect-style-false/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "css-bundle-false-redirect-style-false-test",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module"
+}

tests/integration/style/redirect-style-false/rslib.config.ts

@@ -0,0 +1,27 @@
+import { defineConfig } from '@rslib/core';
+import { generateBundleCjsConfig, generateBundleEsmConfig } from 'test-helper';
+
+export default defineConfig({
+  lib: [
+    generateBundleEsmConfig({
+      bundle: false,
+      redirect: {
+        style: false,
+      },
+    }),
+    generateBundleCjsConfig({
+      bundle: false,
+      redirect: {
+        style: false,
+      },
+    }),
+  ],
+  source: {
+    entry: {
+      index: ['./src/index.ts'],
+    },
+  },
+  output: {
+    copy: [{ from: './src/index.less' }, { from: './src/style.module.less' }],
+  },
+});

tests/integration/style/redirect-style-false/src/index.less

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

tests/integration/style/redirect-style-false/src/index.ts

@@ -0,0 +1,4 @@
+import './index.less';
+import styles from './style.module.less';
+
+styles;

tests/integration/style/redirect-style-false/src/style.module.less

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

tests/integration/style/style-inject/__fixtures__/basic/src/import.css

@@ -0,0 +1,8 @@
+@import 'https://cdnjs.cloudflare.com/ajax/libs/modern-normalize/1.1.0/modern-normalize.css';
+@import url('https://cdnjs.cloudflare.com/ajax/libs/modern-normalize/1.0.0/modern-normalize.css');
+@import url('lib1.css');
+@import 'lib2.css';
+
+.import {
+  background-image: url('https://cdnjs.cloudflare.com/ajax/libs/modern-normalize/1.1.0/modern-normalize.css');
+}

tests/integration/style/style-inject/__fixtures__/basic/src/index.ts

@@ -0,0 +1 @@
+import './import.css';

tests/integration/style/style-inject/__fixtures__/basic/src/lib1.css

@@ -0,0 +1,3 @@
+.lib1 {
+  color: red;
+}