fix: disallow empty groupings, alternations

Something like `()` or `[]` is a syntax error and it would be nice if the parser reflected that (even if just with a "missing" node rather than an error node).
tree-sitter-grammars · Nov 9, 2024 · d0d9126 · d0d9126
1 parent 3c61bfc
commit d0d9126
Show file tree

Hide file tree

Showing 4 changed files with 2,739 additions and 3,280 deletions.
diff --git a/grammar.js b/grammar.js
@@ -75,11 +75,11 @@ module.exports = grammar({
     string_content: $ => repeat1(choice(token.immediate(prec(PREC.STRING, /[^"\\]+/)), $.escape_sequence)),
     parameters: $ => repeat1(choice($.capture, $.string, $._node_identifier)),
     comment: _ => token(prec(PREC.COMMENT, seq(";", /.*/))),
-    list: $ => seq("[", repeat($.definition), "]", quantifier($), captures($)),
+    list: $ => seq("[", repeat1($.definition), "]", quantifier($), captures($)),
 
     grouping: $ => seq(
       "(",
-      repeat(seq($._group_expression, optional("."))),
+      repeat1(seq($._group_expression, optional("."))),
       ")",
       quantifier($),
       captures($),

diff --git a/src/grammar.json b/src/grammar.json
diff --git a/src/node-types.json b/src/node-types.json