Support ":" as struct key-value separator

Instead of the array-slice operator (which for now doesn't quite work.

Author: nene

src/languages/duckdb/duckdb.formatter.ts

@@ -209,7 +209,7 @@ export const duckdb: DialectOptions = {
     ],
   },
   formatOptions: {
-    alwaysDenseOperators: ['::', ':'],
+    alwaysDenseOperators: ['::'],
     onelineClauses: [...standardOnelineClauses, ...tabularOnelineClauses],
     tabularOnelineClauses,
   },

test/behavesLikePostgresqlFormatter.ts

@@ -64,17 +64,6 @@ export default function behavesLikePostgresqlFormatter(format: FormatFn) {
     `);
   });
 
-  // Regression test for issue #624
-  it('supports array slice operator', () => {
-    expect(format('SELECT foo[:5], bar[1:], baz[1:5], zap[:];')).toBe(dedent`
-      SELECT
-        foo[:5],
-        bar[1:],
-        baz[1:5],
-        zap[:];
-    `);
-  });
-
   it('formats ALTER TABLE ... ALTER COLUMN', () => {
     expect(
       format(

test/duckdb.test.ts

@@ -101,6 +101,65 @@ describe('DuckDBFormatter', () => {
   supportsParams(format, { positional: true, numbered: ['$'], quoted: ['$""'] });
   supportsLimiting(format, { limit: true, offset: true });
 
+  it('formats {} struct literal (string keys)', () => {
+    expect(format("SELECT {'id':1,'type':'Tarzan'} AS obj;")).toBe(dedent`
+      SELECT
+        {'id': 1, 'type': 'Tarzan'} AS obj;
+    `);
+  });
+
+  it('formats {} struct literal (identifier keys)', () => {
+    expect(format("SELECT {id:1,type:'Tarzan'} AS obj;")).toBe(dedent`
+      SELECT
+        {id: 1, type: 'Tarzan'} AS obj;
+    `);
+  });
+
+  it('formats {} struct literal (quoted identifier keys)', () => {
+    expect(format(`SELECT {"id":1,"type":'Tarzan'} AS obj;`)).toBe(dedent`
+      SELECT
+        {"id": 1, "type": 'Tarzan'} AS obj;
+    `);
+  });
+
+  it('formats large struct and list literals', () => {
+    const result = format(`
+      INSERT INTO heroes (KEY, VALUE) VALUES ('123', {'id': 1, 'type': 'Tarzan',
+      'array': [123456789, 123456789, 123456789, 123456789, 123456789], 'hello': 'world'});
+    `);
+    expect(result).toBe(dedent`
+      INSERT INTO
+        heroes (KEY, VALUE)
+      VALUES
+        (
+          '123',
+          {
+            'id': 1,
+            'type': 'Tarzan',
+            'array': [
+              123456789,
+              123456789,
+              123456789,
+              123456789,
+              123456789
+            ],
+            'hello': 'world'
+          }
+        );
+    `);
+  });
+
+  // TODO: This currently conflicts with ":"-operator in struct literals
+  it.skip('supports array slice operator', () => {
+    expect(format('SELECT foo[:5], bar[1:], baz[1:5], zap[:];')).toBe(dedent`
+      SELECT
+        foo[:5],
+        bar[1:],
+        baz[1:5],
+        zap[:];
+    `);
+  });
+
   it('formats TIMESTAMP WITH TIME ZONE syntax', () => {
     expect(
       format(`

test/postgresql.test.ts

@@ -139,6 +139,17 @@ describe('PostgreSqlFormatter', () => {
   supportsLimiting(format, { limit: true, offset: true, fetchFirst: true, fetchNext: true });
   supportsDataTypeCase(format);
 
+  // Regression test for issue #624
+  it('supports array slice operator', () => {
+    expect(format('SELECT foo[:5], bar[1:], baz[1:5], zap[:];')).toBe(dedent`
+      SELECT
+        foo[:5],
+        bar[1:],
+        baz[1:5],
+        zap[:];
+    `);
+  });
+
   // Regression test for issue #447
   it('formats empty SELECT', () => {
     expect(format('SELECT;')).toBe(dedent`