refactor module_pack_expression

Author: aspeddro

grammar.js

@@ -716,15 +716,13 @@ module.exports = grammar({
     )),
 
     module_pack_expression: $ => seq(
-      'module', '(',
+      'module',
+      '(',
       choice(
-        seq(
-          field('name', choice($.module_identifier, $.module_identifier_path)),
-          optional(field('signature', seq(':', $.module_identifier))),
-          optional(field('definition', seq('(', $.module_unpack, ')')))
-        ),
-        seq(field('definition', $.block), ':', field('signature', $.module_identifier))
+        $._module_definition,
+        seq(choice($.module_identifier, $.module_identifier_path), '(', $.module_unpack, ')')
       ),
+      optional(seq(':', $.module_expression)),
       ')'
     ),
 

test/corpus/modules.txt

@@ -155,6 +155,7 @@ module(
     let foo = "Hello"
   }: X
 )
+module(SomeFunctor(unpack(x)))
 
 ---
 
@@ -170,7 +171,12 @@ module(
         (let_binding
           (value_identifier)
           (string (string_fragment))))
-      (module_identifier))))
+      (module_identifier)))
+  (expression_statement
+    (module_pack_expression
+      (module_identifier)
+      (module_unpack
+        (call_arguments (value_identifier))))))
 
 
 ===========================================