fix(solidity): bump submodule

lang/semgrep-grammars/src/semgrep-solidity/grammar.js

@@ -24,11 +24,10 @@ module.exports = grammar(base_grammar, {
         source_file: ($, previous) => {
           return choice(
             previous,
-            repeat1($._statement),
-            $._expression,
+            repeat1($.statement),
+            $.expression,
             $.constructor_definition,
             $.modifier_definition,
-            $.event_definition,
           );
         },
 
@@ -47,9 +46,9 @@ module.exports = grammar(base_grammar, {
                 ),
             )
         },
-      
+
       // Ellipsis
-        _expression: ($, previous) => {
+        expression: ($, previous) => {
             return choice(
                 previous,
                 $.ellipsis,
@@ -70,7 +69,7 @@ module.exports = grammar(base_grammar, {
        // TODO: how to use PREC.MEMBER from original grammar instead of hardcoded value?
        member_ellipsis_expression : $ => prec(1, seq(
             field('object', choice(
-                $._expression,
+                $.expression,
                 $.identifier,
             )),
             '.',
@@ -97,8 +96,7 @@ module.exports = grammar(base_grammar, {
             );
         },
 
-        // typo on name in the original grammar so we must copy the typo
-        event_paramater: ($, previous) => {
+        event_parameter: ($, previous) => {
             return choice(
                previous,
                $.ellipsis
@@ -108,7 +106,7 @@ module.exports = grammar(base_grammar, {
         for_statement: ($, previous) => {
             return choice(
                previous,
-               seq('for', '(', $.ellipsis, ')', $._statement)
+               seq('for', '(', $.ellipsis, ')', $.statement)
             );
         },
 
@@ -119,23 +117,20 @@ module.exports = grammar(base_grammar, {
             );
         },
 
-      //TODO? it would be better to refactor the original grammar with
-      // a enum_member so we don't have to copy-paste the original rule
-      enum_declaration: $ =>  seq(
-            'enum',
-            field("enum_type_name", $.identifier),
-            '{',
-            commaSep($._enum_member),
-            '}',
-      ),
-      _enum_member: $ => choice(
-          alias($.identifier, $.enum_value),
-          $.ellipsis
-      ),
+        enum_body: $ => choice(
+          seq(
+              '{',
+              choice(
+                commaSep(alias($.identifier, $.enum_value)),
+                $.ellipsis
+              ),
+              '}',
+          ),
+        ),
 
       // The actual ellipsis rules
         deep_ellipsis: $ => seq(
-            '<...', $._expression, '...>'
+            '<...', $.expression, '...>'
         ),
 
         ellipsis: $ => '...',

lang/semgrep-grammars/src/semgrep-solidity/test/corpus/semgrep.txt

@@ -3,6 +3,7 @@ Metavariable for version
 =====================================
 pragma solidity ^ $VER;
 ---
+
 (source_file
   (pragma_directive
     (solidity_pragma_token
@@ -14,6 +15,7 @@ Metavariable for versions range
 =====================================
 pragma solidity >= $VER_FROM <= $VER_TO;
 ---
+
 (source_file
   (pragma_directive
     (solidity_pragma_token
@@ -29,9 +31,12 @@ contract Foo {
   ...
 }
 ---
+
 (source_file
-	(contract_declaration (identifier)
-			      (contract_body (ellipsis))))
+  (contract_declaration
+    (identifier)
+    (contract_body
+      (ellipsis))))
 
 =====================================
 Ellipsis for 'for'
@@ -43,7 +48,11 @@ for(...) {
 ---
 
 (source_file
-  (for_statement (ellipsis) (block_statement)))
+  (statement
+    (for_statement
+      (ellipsis)
+      (statement
+        (block_statement)))))
 
 =====================================
 Constructor pattern
@@ -52,13 +61,15 @@ constructor (...) {
    ...
 }
 ---
- (source_file
-      (constructor_definition
-        (parameter
-          (ellipsis))
-        (function_body
-          (expression_statement
-            (ellipsis)))))
+
+(source_file
+  (constructor_definition
+    (parameter
+      (ellipsis))
+    (function_body
+      (statement
+        (expression_statement
+          (ellipsis))))))
 
 =====================================
 Modifier pattern
@@ -68,20 +79,23 @@ modifier $M(...) {
    ...
 }
 ---
+
 (source_file
-      (modifier_definition
-        (identifier)
-        (parameter
-          (ellipsis))
-        (function_body
-          (expression_statement
-            (ellipsis)))))
+  (modifier_definition
+    (identifier)
+    (parameter
+      (ellipsis))
+    (function_body
+      (statement
+        (expression_statement
+          (ellipsis))))))
 
 =====================================
 Inheritance ellipsis pattern
 =====================================
 contract Example is ..., BoringBatchable, ... {}
 ---
+
 (source_file
   (contract_declaration
     (identifier)
@@ -92,8 +106,7 @@ contract Example is ..., BoringBatchable, ... {}
         (identifier)))
     (inheritance_specifier
       (ellipsis))
-  (contract_body)))
-
+    (contract_body)))
 
 =====================================
 Enum pattern
@@ -105,9 +118,10 @@ enum $X {
 ---
 
 (source_file
-      (enum_declaration
-        (identifier)
-        (ellipsis)))
+  (enum_declaration
+    (identifier)
+    (enum_body
+      (ellipsis))))
 
 =====================================
 Event pattern
@@ -117,7 +131,7 @@ event $EV(...);
 ---
 
 (source_file
-      (event_definition
-        (identifier)
-        (event_paramater
-          (ellipsis))))
+  (event_definition
+    (identifier)
+    (event_parameter
+      (ellipsis))))