fix(cli): css parsing arguments for ci command (#7840)

Co-authored-by: siketyan <[email protected]>

Author: ematipico

.changeset/afraid-meals-help.md

@@ -0,0 +1,5 @@
+---
+"@biomejs/biome": patch
+---
+
+Fixed [#7838](https://github.com/biomejs/biome/issues/7838), which caused the new `--css-parse-*` arguments not being recognised by the `ci` command.

crates/biome_cli/src/commands/ci.rs

@@ -78,14 +78,14 @@ impl CommandRunner for CiCommandPayload {
             .json
             .get_or_insert_with(JsonConfiguration::default);
         if self.json_parser.is_some() {
-            json.parser.clone_from(&self.json_parser)
+            json.parser.merge_with(self.json_parser.clone())
         }
 
         let css = configuration
             .css
             .get_or_insert_with(CssConfiguration::default);
         if self.css_parser.is_some() {
-            css.parser.clone_from(&self.css_parser);
+            css.parser.merge_with(self.css_parser.clone());
         }
 
         let assist = configuration

crates/biome_cli/tests/cases/css_parsing.rs

@@ -219,6 +219,60 @@ fn format_css_parse_tailwind_directives_true() {
     ));
 }
 
+#[test]
+fn ci_css_parse_css_modules_true() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Utf8Path::new("file.module.css");
+    // CSS Modules specific syntax
+    fs.insert(
+        file_path.into(),
+        ".className { composes: other from './other.module.css'; }".as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(["ci", "--css-parse-css-modules=true", file_path.as_str()].as_slice()),
+    );
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "ci_css_parse_css_modules_true",
+        fs,
+        console,
+        result,
+    ));
+}
+
+#[test]
+fn ci_css_parse_css_modules_false() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Utf8Path::new("file.module.css");
+    // CSS Modules specific syntax
+    fs.insert(
+        file_path.into(),
+        ".className { composes: other from './other.module.css'; }".as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(["ci", "--css-parse-css-modules=false", file_path.as_str()].as_slice()),
+    );
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "ci_css_parse_css_modules_false",
+        fs,
+        console,
+        result,
+    ));
+}
+
 #[test]
 fn ci_css_parse_tailwind_directives_true() {
     let fs = MemoryFileSystem::default();
@@ -253,6 +307,76 @@ fn ci_css_parse_tailwind_directives_true() {
     ));
 }
 
+#[test]
+fn ci_css_parse_tailwind_directives_false() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Utf8Path::new("file.css");
+    // Tailwind CSS 4.0 directive
+    fs.insert(
+        file_path.into(),
+        "@import 'tailwindcss';\n.foo { color: red; }".as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(
+            [
+                "ci",
+                "--css-parse-tailwind-directives=false",
+                file_path.as_str(),
+            ]
+            .as_slice(),
+        ),
+    );
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "ci_css_parse_tailwind_directives_false",
+        fs,
+        console,
+        result,
+    ));
+}
+
+#[test]
+fn ci_combined_css_parser_flags() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let file_path = Utf8Path::new("file.module.css");
+    // CSS Modules with Tailwind directives
+    fs.insert(
+        file_path.into(),
+        "@import 'tailwindcss';\n.className { composes: other from './other.module.css'; }"
+            .as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(
+            [
+                "ci",
+                "--css-parse-css-modules=true",
+                "--css-parse-tailwind-directives=true",
+                file_path.as_str(),
+            ]
+            .as_slice(),
+        ),
+    );
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "ci_combined_css_parser_flags",
+        fs,
+        console,
+        result,
+    ));
+}
+
 // Combined tests
 
 #[test]

crates/biome_cli/tests/snapshots/main_cases_css_parsing/ci_combined_css_parser_flags.snap

@@ -0,0 +1,44 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `file.module.css`
+
+```css
+@import 'tailwindcss';
+.className { composes: other from './other.module.css'; }
+```
+
+# Termination Message
+
+```block
+ci ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+  
+
+
+```
+
+# Emitted Messages
+
+```block
+file.module.css format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × File content differs from formatting output
+  
+    1   │ - @import·'tailwindcss';
+    2   │ - .className·{·composes:·other·from·'./other.module.css';·}
+      1 │ + @import·"tailwindcss";
+      2 │ + .className·{
+      3 │ + → composes:·other·from·"./other.module.css";
+      4 │ + }
+      5 │ + 
+  
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes applied.
+Found 1 error.
+```

crates/biome_cli/tests/snapshots/main_cases_css_parsing/ci_css_parse_css_modules_false.snap

@@ -0,0 +1,61 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `file.module.css`
+
+```css
+.className { composes: other from './other.module.css'; }
+```
+
+# Termination Message
+
+```block
+ci ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+  
+
+
+```
+
+# Emitted Messages
+
+```block
+file.module.css:1:14 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × `composes` declaration is not a standard CSS feature.
+  
+  > 1 │ .className { composes: other from './other.module.css'; }
+      │              ^^^^^^^^
+  
+  i You can enable `composes` declaration parsing by setting the `css.parser.cssModules` option to `true` in your configuration file.
+  
+
+```
+
+```block
+file.module.css:1:14 parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × `composes` declaration is not a standard CSS feature.
+  
+  > 1 │ .className { composes: other from './other.module.css'; }
+      │              ^^^^^^^^
+  
+  i You can enable `composes` declaration parsing by setting the `css.parser.cssModules` option to `true` in your configuration file.
+  
+
+```
+
+```block
+file.module.css format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Code formatting aborted due to parsing errors. To format code with errors, enable the 'formatter.formatWithErrors' option.
+  
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes applied.
+Found 3 errors.
+```

crates/biome_cli/tests/snapshots/main_cases_css_parsing/ci_css_parse_css_modules_true.snap

@@ -0,0 +1,41 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `file.module.css`
+
+```css
+.className { composes: other from './other.module.css'; }
+```
+
+# Termination Message
+
+```block
+ci ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+  
+
+
+```
+
+# Emitted Messages
+
+```block
+file.module.css format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × File content differs from formatting output
+  
+    1   │ - .className·{·composes:·other·from·'./other.module.css';·}
+      1 │ + .className·{
+      2 │ + → composes:·other·from·"./other.module.css";
+      3 │ + }
+      4 │ + 
+  
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes applied.
+Found 1 error.
+```

crates/biome_cli/tests/snapshots/main_cases_css_parsing/ci_css_parse_tailwind_directives_false.snap

@@ -0,0 +1,44 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `file.css`
+
+```css
+@import 'tailwindcss';
+.foo { color: red; }
+```
+
+# Termination Message
+
+```block
+ci ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × Some errors were emitted while running checks.
+  
+
+
+```
+
+# Emitted Messages
+
+```block
+file.css format ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  × File content differs from formatting output
+  
+    1   │ - @import·'tailwindcss';
+    2   │ - .foo·{·color:·red;·}
+      1 │ + @import·"tailwindcss";
+      2 │ + .foo·{
+      3 │ + → color:·red;
+      4 │ + }
+      5 │ + 
+  
+
+```
+
+```block
+Checked 1 file in <TIME>. No fixes applied.
+Found 1 error.
+```