Merge pull request #3430 from Earlopain/parser-translator-mlhs-simplify

kddnewton · web-flow · commit 4afe194d5222 · 2025-01-20T13:02:43.000-05:00
Only unnest parser mlhs nodes when no rest argument is provided
diff --git a/lib/prism/translation/parser/compiler.rb b/lib/prism/translation/parser/compiler.rb
@@ -1320,7 +1320,7 @@ def visit_multi_target_node(node)
         def visit_multi_write_node(node)
           elements = multi_target_elements(node)
 
-          if elements.length == 1 && elements.first.is_a?(MultiTargetNode)
+          if elements.length == 1 && elements.first.is_a?(MultiTargetNode) && !node.rest
             elements = multi_target_elements(elements.first)
           end
 
diff --git a/test/prism/fixtures/variables.txt b/test/prism/fixtures/variables.txt
@@ -45,3 +45,5 @@ Foo = 1, 2
 (a; b; c)
 
 a, (b, c), d = []
+
+(a,), = []
diff --git a/test/prism/snapshots/variables.txt b/test/prism/snapshots/variables.txt
