Make sure record update arguments are grouped together by the formatter

Author: giacomocavalieri

CHANGELOG.md

@@ -5,6 +5,8 @@
 ### Formatter
 
 - Fixed a bug where the `@internal` annotation wouldn't be displayed.
+- Fixed a bug where a record update's arguments would be incorrectly split on
+  multiple lines.
 
 ## v1.1.0-rc2 - 2024-04-10
 

compiler-core/src/format.rs

@@ -1149,7 +1149,7 @@ impl<'comments> Formatter<'comments> {
         let constructor_doc = self.expr(constructor);
         let comments = self.pop_comments(spread.base.location().start);
         let spread_doc = commented("..".to_doc().append(self.expr(&spread.base)), comments);
-        let arg_docs = args.iter().map(|a| self.record_update_arg(a));
+        let arg_docs = args.iter().map(|a| self.record_update_arg(a).group());
         let all_arg_docs = once(spread_doc).chain(arg_docs);
         constructor_doc.append(wrap_args(all_arg_docs)).group()
     }

compiler-core/src/format/tests.rs

@@ -5840,6 +5840,21 @@ fn piped_blocks_are_not_needlessly_indented() {
     );
 }
 
+// https://github.com/gleam-lang/gleam/issues/2924
+#[test]
+fn record_update_fields_are_not_needlessly_broken() {
+    assert_format!(
+        r#"pub fn main() {
+  Model(
+    ..model,
+    wibble: wibble_wobble_wibble_wobble + 1,
+    wobble: Some(wibble_wobble_wibble_wobble),
+  )
+}
+"#
+    );
+}
+
 // https://github.com/gleam-lang/gleam/issues/2890
 #[test]
 fn piped_lists_are_not_needlessly_indented() {