fix(html/formatter): style tag with lang

ematipico · ematipico · commit 08898c902905 · 2025-12-04T08:53:39.000Z
diff --git a/.changeset/sad-boxes-lead.md b/.changeset/sad-boxes-lead.md
@@ -0,0 +1,5 @@
+---
+"@biomejs/biome": patch
+---
+
+Fixed [#7917](https://github.com/biomejs/biome/issues/7917), where Biome removed the styles contained in a `<style lang="scss">`, when `experimentalFullSupportEnabled` is enabled.
diff --git a/crates/biome_cli/tests/cases/handle_vue_files.rs b/crates/biome_cli/tests/cases/handle_vue_files.rs
@@ -568,6 +568,45 @@ const props: Props = { title: "Hello" };
     ));
 }
 
+#[test]
+fn full_support_enabled_and_scss_is_skipped() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    fs.insert(
+        "biome.json".into(),
+        r#"{ "html": { "formatter": {"enabled": true}, "linter": {"enabled": true}, "experimentalFullSupportEnabled": true } }"#.as_bytes(),
+    );
+
+    let astro_file_path = Utf8Path::new("file.vue");
+    fs.insert(
+        astro_file_path.into(),
+        r#"<html><head><title>Svelte</title></head><body></body></html>
+
+<style lang="scss">
+#id { font-family: comic-sans } .class { background: red}
+</style>
+"#
+        .as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(["check", "--write", "--unsafe", astro_file_path.as_str()].as_slice()),
+    );
+
+    assert!(result.is_ok(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "full_support_enabled_and_scss_is_skipped",
+        fs,
+        console,
+        result,
+    ));
+}
+
 #[test]
 fn full_support_tsx() {
     let fs = MemoryFileSystem::default();
@@ -915,3 +954,26 @@ fn vue_compiler_macros_as_globals() {
         result,
     ));
 }
+
+#[test]
+fn format_vue_when_() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let vue_file_path = Utf8Path::new("file.vue");
+    fs.insert(vue_file_path.into(), VUE_TS_FILE_SETUP_GLOBALS.as_bytes());
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(["lint", vue_file_path.as_str()].as_slice()),
+    );
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "vue_compiler_macros_as_globals",
+        fs,
+        console,
+        result,
+    ));
+}
diff --git a/crates/biome_cli/tests/snapshots/main_cases_handle_vue_files/full_support_enabled_and_scss_is_skipped.snap b/crates/biome_cli/tests/snapshots/main_cases_handle_vue_files/full_support_enabled_and_scss_is_skipped.snap
@@ -0,0 +1,37 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `biome.json`
+
+```json
+{
+  "html": {
+    "formatter": { "enabled": true },
+    "linter": { "enabled": true },
+    "experimentalFullSupportEnabled": true
+  }
+}
+```
+
+## `file.vue`
+
+```vue
+<html>
+	<head>
+		<title>Svelte</title>
+	</head>
+	<body></body>
+</html>
+
+<style lang="scss">
+#id { font-family: comic-sans } .class { background: red}
+</style>
+
+```
+
+# Emitted Messages
+
+```block
+Checked 1 file in <TIME>. Fixed 1 file.
+```
diff --git a/crates/biome_html_formatter/src/html/auxiliary/embedded_content.rs b/crates/biome_html_formatter/src/html/auxiliary/embedded_content.rs
@@ -21,7 +21,7 @@ impl FormatNodeRule<HtmlEmbeddedContent> for FormatHtmlEmbeddedContent {
             .ancestors()
             .skip(1)
             .find_map(HtmlElement::cast)?;
-        if element.is_supported_script_tag() || element.is_style_tag() {
+        if element.is_supported_script_tag() || element.is_supported_style_tag() {
             Some(node.range())
         } else {
             None
diff --git a/crates/biome_html_syntax/src/element_ext.rs b/crates/biome_html_syntax/src/element_ext.rs
@@ -118,6 +118,11 @@ impl HtmlElement {
         self.get_script_type().is_some_and(ScriptType::is_supported)
     }
 
+    /// It's a style tag, and it doesn't contain `scss` as `lang`
+    pub fn is_supported_style_tag(&self) -> bool {
+        self.is_style_tag() && !self.is_sass_lang()
+    }
+
     /// Returns the type of script for a `<script>` tag.
     ///
     /// Returns `Some` [`ScriptType`] if this is a `<script>` tag, even if it

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"@biomejs/biome": patch
 +---
++
 +Fixed [#7917](https://github.com/biomejs/biome/issues/7917), where Biome removed the styles contained in a `<style lang="scss">`, when `experimentalFullSupportEnabled` is enabled.