fix(html/astro): don't indent Astro frontmatter

Author: PFiS1737

.changeset/vast-donuts-rescue.md

@@ -0,0 +1,5 @@
+---
+"@biomejs/biome": patch
+---
+
+`indentScriptAndStyle` no longer indents the frontmatter in Astro files.

crates/biome_cli/tests/cases/handle_astro_files.rs

@@ -667,3 +667,51 @@ fn format_astro_with_typescript_script_tag() {
         result,
     ));
 }
+
+#[test]
+fn dont_indent_frontmatter() {
+    let fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    fs.insert(
+        "biome.json".into(),
+        r#"{ "html": { "formatter": {"enabled": true, "indentScriptAndStyle": true}, "linter": {"enabled": true}, "experimentalFullSupportEnabled": true } }"#.as_bytes(),
+    );
+
+    let astro_file_path = Utf8Path::new("file.astro");
+    fs.insert(
+        astro_file_path.into(),
+        r#"---
+import Foo from "./Foo.astro"
+const bar = 123
+if (bar>1) {console.log(bar+1)}
+---
+<Foo>{bar}</Foo>
+
+<style>
+#id { font-family: comic-sans } .class { background: red}
+</style>
+
+<script>
+function foo(){console.log("Hello")}
+</script>
+"#
+        .as_bytes(),
+    );
+
+    let (fs, result) = run_cli(
+        fs,
+        &mut console,
+        Args::from(["format", "--write", astro_file_path.as_str()].as_slice()),
+    );
+
+    assert!(result.is_ok(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "dont_indent_frontmatter",
+        fs,
+        console,
+        result,
+    ));
+}

crates/biome_cli/tests/snapshots/main_cases_handle_astro_files/dont_indent_frontmatter.snap

@@ -0,0 +1,50 @@
+---
+source: crates/biome_cli/tests/snap_test.rs
+expression: redactor(content)
+---
+## `biome.json`
+
+```json
+{
+  "html": {
+    "formatter": { "enabled": true, "indentScriptAndStyle": true },
+    "linter": { "enabled": true },
+    "experimentalFullSupportEnabled": true
+  }
+}
+```
+
+## `file.astro`
+
+```astro
+---
+import Foo from "./Foo.astro";
+const bar = 123;
+if (bar > 1) {
+	console.log(bar + 1);
+}
+---
+<Foo>{bar}</Foo>
+
+<style>
+	#id {
+		font-family: comic-sans;
+	}
+	.class {
+		background: red;
+	}
+</style>
+
+<script>
+	function foo() {
+		console.log("Hello");
+	}
+</script>
+
+```
+
+# Emitted Messages
+
+```block
+Formatted 1 file in <TIME>. Fixed 1 file.
+```

crates/biome_js_syntax/src/file_source.rs

@@ -121,6 +121,9 @@ impl Language {
 )]
 pub enum EmbeddingKind {
     Astro,
+    // This variant is currently only used to record whether it is an Astro frontmatter,
+    // and in that case, the `indentScriptAndStyle` setting is ignored.
+    AstroFrontmatter,
     Vue,
     Svelte,
     #[default]

crates/biome_service/src/file_handlers/html.rs

@@ -445,7 +445,7 @@ pub(crate) fn parse_astro_embedded_script(
     settings: &Settings,
 ) -> Option<(EmbeddedSnippet<JsLanguage>, DocumentFileSource)> {
     let content = element.content_token()?;
-    let file_source = JsFileSource::ts().with_embedding_kind(EmbeddingKind::Astro);
+    let file_source = JsFileSource::ts().with_embedding_kind(EmbeddingKind::AstroFrontmatter);
     let document_file_source = DocumentFileSource::Js(file_source);
     let options = settings.parse_options::<JsLanguage>(path, &document_file_source);
     let parse = parse_js_with_offset_and_cache(
@@ -672,8 +672,8 @@ fn format_embedded(
         let mut iter = embedded_nodes.iter();
         let node = iter.find(|node| node.range == range)?;
 
-        let wrap_document = |document: Document| {
-            if indent_script_and_style {
+        let wrap_document = |document: Document, is_astro_frontmatter: bool| {
+            if indent_script_and_style && !is_astro_frontmatter {
                 let elements = vec![
                     FormatElement::Line(LineMode::Hard),
                     FormatElement::Tag(Tag::StartIndent),
@@ -693,27 +693,31 @@ fn format_embedded(
         };
 
         match node.source {
-            DocumentFileSource::Js(_) => {
+            DocumentFileSource::Js(file_source) => {
                 let js_options = settings.format_options::<JsLanguage>(biome_path, &node.source);
                 let node = node.node.clone().embedded_syntax::<JsLanguage>().clone();
                 let formatted =
                     biome_js_formatter::format_node_with_offset(js_options, &node).ok()?;
-                Some(wrap_document(formatted.into_document()))
+
+                Some(wrap_document(
+                    formatted.into_document(),
+                    file_source.as_embedding_kind() == &EmbeddingKind::AstroFrontmatter,
+                ))
             }
             DocumentFileSource::Json(_) => {
                 let json_options =
                     settings.format_options::<JsonLanguage>(biome_path, &node.source);
                 let node = node.node.clone().embedded_syntax::<JsonLanguage>().clone();
                 let formatted =
                     biome_json_formatter::format_node_with_offset(json_options, &node).ok()?;
-                Some(wrap_document(formatted.into_document()))
+                Some(wrap_document(formatted.into_document(), false))
             }
             DocumentFileSource::Css(_) => {
                 let css_options = settings.format_options::<CssLanguage>(biome_path, &node.source);
                 let node = node.node.clone().embedded_syntax::<CssLanguage>();
                 let formatted =
                     biome_css_formatter::format_node_with_offset(css_options, &node).ok()?;
-                Some(wrap_document(formatted.into_document()))
+                Some(wrap_document(formatted.into_document(), false))
             }
             _ => None,
         }

crates/biome_service/src/file_handlers/mod.rs

@@ -814,7 +814,7 @@ impl Features {
         match language_hint {
             // TODO: remove match once we remove vue/astro/svelte handlers
             DocumentFileSource::Js(source) => match source.as_embedding_kind() {
-                EmbeddingKind::Astro => self.astro.capabilities(),
+                EmbeddingKind::Astro | EmbeddingKind::AstroFrontmatter => self.astro.capabilities(),
                 EmbeddingKind::Vue => self.vue.capabilities(),
                 EmbeddingKind::Svelte => self.svelte.capabilities(),
                 EmbeddingKind::None => self.js.capabilities(),