fix: when sourcemap is false, the css sourcemap also is true in development (#7696)

Author: yifancong

.changeset/fancy-ravens-attack.md

@@ -0,0 +1,5 @@
+---
+'@modern-js/uni-builder': patch
+---
+
+fix: when sourcemap is false, the css sourcemap also is true in development

packages/cli/uni-builder/src/shared/parseCommonConfig.ts

@@ -197,9 +197,12 @@ export async function parseCommonConfig(
     output.cssModules.localIdentName = cssModuleLocalIdentName;
   }
 
-  if (isUseCssSourceMap(disableSourceMap) && output.sourceMap !== true) {
+  if (
+    isUseCssSourceMap(disableSourceMap) &&
+    typeof output.sourceMap !== 'boolean'
+  ) {
     output.sourceMap ||= {};
-    output.sourceMap.css = true;
+    output.sourceMap.css ??= true;
   }
 
   const { server: _server, worker, ...rsbuildDistPath } = distPath;

packages/cli/uni-builder/tests/parseConfig.test.ts

@@ -362,4 +362,70 @@ describe('parseCommonConfig', () => {
       });
     }
   });
+
+  describe('CSS source map configuration', () => {
+    test('should not set css source map when output.sourceMap is true', async () => {
+      const config = await parseCommonConfig({
+        output: {
+          sourceMap: true,
+        },
+      });
+
+      expect(config.rsbuildConfig.output?.sourceMap).toBe(true);
+      expect(config.rsbuildConfig.output?.sourceMap).not.toHaveProperty('css');
+    });
+
+    test('should set css source map to false when output.sourceMap is false', async () => {
+      const config = await parseCommonConfig({
+        output: {
+          sourceMap: false,
+        },
+      });
+
+      expect(config.rsbuildConfig.output?.sourceMap).toEqual(false);
+    });
+
+    test('should set css source map to false when output.sourceMap.css is explicitly false', async () => {
+      const config = await parseCommonConfig({
+        output: {
+          sourceMap: {
+            css: false,
+          },
+        },
+      });
+
+      expect(config.rsbuildConfig.output?.sourceMap).toEqual({
+        css: false,
+      });
+    });
+
+    test('should set css source map to true when output.sourceMap is undefined in development', async () => {
+      process.env.NODE_ENV = 'development';
+
+      const config = await parseCommonConfig({
+        output: {},
+      });
+
+      expect(config.rsbuildConfig.output?.sourceMap).toEqual({
+        css: true,
+      });
+    });
+
+    test('should respect explicitly set css source map in development', async () => {
+      process.env.NODE_ENV = 'development';
+
+      const config = await parseCommonConfig({
+        output: {
+          sourceMap: {
+            css: false,
+          },
+        },
+      });
+
+      // Even in development, if explicitly set to false, it should remain false
+      expect(config.rsbuildConfig.output?.sourceMap).toEqual({
+        css: false,
+      });
+    });
+  });
 });