Merge branch 'main' into allow-formatting-pipelines-inline

ascandone · Aug 25, 2024 · f1c37cd · f1c37cd
2 parents ec29b1d + 03321b3
commit f1c37cd
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/formatter.test.ts b/src/formatter.test.ts
@@ -912,6 +912,13 @@ test("long arg as last arg", () => {
   expect(src).toBeFormatted();
 });
 
+test("long arg as last arg", () => {
+  const src = `let x = a_super_long_function_that_wraps_but_does_not_have_space_to_wrap_a_super_long()
+`;
+
+  expect(src).toBeFormatted();
+});
+
 test("let inside list", () => {
   const src = `let x = [
   {

diff --git a/src/formatter.ts b/src/formatter.ts
@@ -323,7 +323,11 @@ function exprToDoc(ast: UntypedExpr, block: boolean): Doc {
         ast.caller.type === "identifier" &&
         isTupleN(ast.caller.namespace, ast.caller.name);
 
-      return nextBreakFits(
+      if (ast.args.length === 0) {
+        return concat(exprToDoc(ast.caller, false), text("()"));
+      }
+
+      return broken(
         group(
           isTuple ? nil : exprToDoc(ast.caller, false),
           text("("),
@@ -345,7 +349,6 @@ function exprToDoc(ast: UntypedExpr, block: boolean): Doc {
           break_("", ","),
           text(")"),
         ),
-        false,
       );
     }