fix(formatter): do not break properties of type annotation in functio…

…n parameters

Signed-off-by: Naoki Ikeguchi <[email protected]>

.changeset/cyan-beans-yawn.md

@@ -0,0 +1,5 @@
+---
+"@biomejs/biome": patch
+---
+
+Fixed properties of object type annotation in function parameters are expanded, which is inconsistent with Prettier.

crates/biome_js_formatter/src/utils/object_like.rs

@@ -2,7 +2,9 @@ use crate::prelude::*;
 use crate::JsFormatContext;
 use biome_formatter::write;
 use biome_formatter::{Format, FormatResult};
-use biome_js_syntax::{JsObjectExpression, JsSyntaxToken, TsObjectType};
+use biome_js_syntax::{
+    JsFormalParameter, JsObjectExpression, JsSyntaxToken, TsObjectType, TsTypeAnnotation,
+};
 use biome_rowan::{declare_node_union, AstNode, AstNodeList, AstSeparatedList, SyntaxResult};
 
 declare_node_union! {
@@ -61,8 +63,13 @@ impl Format<JsFormatContext> for JsObjectLike {
             )?;
         } else {
             let should_insert_space_around_brackets = f.options().bracket_spacing().value();
-            let should_expand =
-                f.options().object_wrap().is_preserve() && self.members_have_leading_newline();
+            let should_expand = f.options().object_wrap().is_preserve()
+                && self.members_have_leading_newline()
+                // const fn = ({ foo }: { foo: string }) => { ... };
+                //                      ^ do not break properties here
+                && self
+                    .parent::<TsTypeAnnotation>()
+                    .is_none_or(|node| node.parent::<JsFormalParameter>().is_none());
 
             write!(
                 f,

crates/biome_js_formatter/tests/specs/ts/object/objects.ts

@@ -0,0 +1,19 @@
+let foo: {
+	bar: string;
+};
+
+const fn = ({
+	foo,
+	bar,
+}: {
+	foo: boolean;
+	bar: string;
+}) => {
+	console.log(foo, bar);
+}
+
+function fn2(foo: {
+	bar: string;
+}) {
+	console.log(foo);
+}

crates/biome_js_formatter/tests/specs/ts/object/objects.ts.snap

@@ -0,0 +1,66 @@
+---
+source: crates/biome_formatter_test/src/snapshot_builder.rs
+info: ts/object/objects.ts
+---
+# Input
+
+```ts
+let foo: {
+	bar: string;
+};
+
+const fn = ({
+	foo,
+	bar,
+}: {
+	foo: boolean;
+	bar: string;
+}) => {
+	console.log(foo, bar);
+}
+
+function fn2(foo: {
+	bar: string;
+}) {
+	console.log(foo);
+}
+
+```
+
+
+=============================
+
+# Outputs
+
+## Output 1
+
+-----
+Indent style: Tab
+Indent width: 2
+Line ending: LF
+Line width: 80
+Quote style: Double Quotes
+JSX quote style: Double Quotes
+Quote properties: As needed
+Trailing commas: All
+Semicolons: Always
+Arrow parentheses: Always
+Bracket spacing: true
+Bracket same line: false
+Attribute Position: Auto
+Object wrap: Preserve
+-----
+
+```ts
+let foo: {
+	bar: string;
+};
+
+const fn = ({ foo, bar }: { foo: boolean; bar: string }) => {
+	console.log(foo, bar);
+};
+
+function fn2(foo: { bar: string }) {
+	console.log(foo);
+}
+```