better parse errors for records and arrays (#1975)

malloydata · Oct 22, 2024 · 614b418 · 614b418
1 parent 1b20b15
commit 614b418
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/packages/malloy/src/lang/malloy-to-ast.ts b/packages/malloy/src/lang/malloy-to-ast.ts
@@ -1964,6 +1964,11 @@ export class MalloyToAST
   }
 
   visitExprLiteralRecord(pcx: parse.ExprLiteralRecordContext) {
+    this.contextError(
+      pcx,
+      'not-yet-implemented',
+      'Record data is not yet implemented'
+    );
     const els = this.only<ast.RecordElement>(
       pcx.recordElement().map(elCx => this.astAt(this.visit(elCx), elCx)),
       visited => visited instanceof ast.RecordElement && visited,
@@ -1972,6 +1977,15 @@ export class MalloyToAST
     return new ast.RecordLiteral(els);
   }
 
+  visitExprArrayLiteral(pcx: parse.ExprArrayLiteralContext): ast.Unimplemented {
+    this.contextError(
+      pcx,
+      'not-yet-implemented',
+      'Array data is not yet implemented'
+    );
+    return new ast.Unimplemented();
+  }
+
   visitExprWarnLike(pcx: parse.ExprWarnLikeContext): ast.ExprCompare {
     let op: ast.CompareMalloyOperator = '~';
     const left = pcx.fieldExpr(0);

diff --git a/packages/malloy/src/lang/parse-log.ts b/packages/malloy/src/lang/parse-log.ts
@@ -350,6 +350,7 @@ type MessageParameterTypes = {
   'sql-is-not-null': string;
   'sql-is-null': string;
   'illegal-record-property-type': string;
+  'not-yet-implemented': string;
   'sql-case': string;
   'case-then-type-does-not-match': {
     thenType: FieldValueType;