Allow for one-line CREATE TABLE statements

Fixes #46

Author: nene

src/PrintFn.ts

@@ -23,8 +23,8 @@ export type PrintFn<T, K = PrintableKey<T>> = {
   separated(separator: Doc, path: K[]): Doc[];
 
   // Returns either:
-  // - line: when the node contains a newline
-  // - hardline: when the original source contains no newlines
+  // - line: when the node contains no newlines
+  // - hardline: when the original source contains a newline
   dynamicLine(): Doc;
 };
 

src/syntax/expr.ts

@@ -57,7 +57,10 @@ export const exprMap: CstToDocMap<AllExprNodes> = {
       return print("expr");
     }
     const parent = path.getParentNode() as Node;
-    const lineStyle = isCreateTableStmt(parent) ? hardline : softline;
+    const lineStyle =
+      isCreateTableStmt(parent) && print.dynamicLine() === hardline
+        ? hardline
+        : softline;
     return group(["(", indent([lineStyle, print("expr")]), lineStyle, ")"]);
   },
   binary_expr: (print, node) => {

test/canonical_syntax.test.ts

@@ -21,9 +21,7 @@ describe("with sqlCanonicalSyntax enabled", () => {
         sqlCanonicalSyntax: true,
       }),
     ).toBe(dedent`
-      CREATE TEMPORARY TABLE foo (
-        id INT
-      )
+      CREATE TEMPORARY TABLE foo (id INT)
     `);
   });
 

test/ddl/create_table.test.ts

@@ -2,7 +2,13 @@ import dedent from "dedent-js";
 import { test, testBigquery, testMysql, testPostgresql } from "../test_utils";
 
 describe("create table", () => {
-  it(`formats CREATE TABLE always on multiple lines`, async () => {
+  it(`formats short CREATE TABLE on single line if it fits`, async () => {
+    await test(dedent`
+      CREATE TABLE client (id INT, name VARCHAR(100), org_id INT)
+    `);
+  });
+
+  it(`formats short CREATE TABLE on multiple lines if user prefers`, async () => {
     await test(dedent`
       CREATE TABLE client (
         id INT,