Fix hash emitter on kwrestargs

mbj · Oct 8, 2023 · a281ec2 · a281ec2
1 parent c996bba
commit a281ec2
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 5 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,3 +1,9 @@
+# v0.6.8 2023-10-08
+
+[#348](https://github.com/mbj/unparser/pull/348)
+
+* Fix crash on kwrestarg hash member
+
 # v0.6.8 2023-06-14
 
 [#347](https://github.com/mbj/unparser/pull/347)

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    unparser (0.6.8)
+    unparser (0.6.9)
       diff-lcs (~> 1.3)
       parser (>= 3.2.0)
 
@@ -26,7 +26,7 @@ GEM
       mutant (= 0.11.20)
       rspec-core (>= 3.8.0, < 4.0.0)
     parallel (1.23.0)
-    parser (3.2.2.3)
+    parser (3.2.2.4)
       ast (~> 2.4.1)
       racc
     racc (1.7.1)
@@ -64,7 +64,7 @@ GEM
     rubocop-packaging (0.5.2)
       rubocop (>= 1.33, < 2.0)
     ruby-progressbar (1.13.0)
-    sorbet-runtime (0.5.10878)
+    sorbet-runtime (0.5.11064)
     unicode-display_width (2.4.2)
 
 PLATFORMS

diff --git a/config/mutant.yml b/config/mutant.yml
@@ -25,6 +25,7 @@ matcher:
   - 'Unparser::Emitter::Class#local_variable_scope'
   - 'Unparser::Emitter::Def#local_variable_scope'
   - 'Unparser::Emitter::FindPattern#dispatch' # 3.0+ specific
+  - 'Unparser::Emitter::Hash#emit_heredoc_reminder_member' # 3.2+ specific
   - 'Unparser::Emitter::HashPattern#write_symbol_body'
   - 'Unparser::Emitter::LocalVariableRoot*'
   - 'Unparser::Emitter::LocalVariableRoot.included'

diff --git a/lib/unparser/emitter/hash.rb b/lib/unparser/emitter/hash.rb
@@ -25,7 +25,7 @@ def dispatch
       end
 
       def emit_heredoc_reminder_member(node)
-        emitter(node.children.last).emit_heredoc_reminders
+        emitter(node.children.last).emit_heredoc_reminders if n_pair?(node)
       end
 
       def emit_hash_body

diff --git a/test/corpus/literal/since/32.rb b/test/corpus/literal/since/32.rb
@@ -5,3 +5,7 @@ def foo(argument, **)
 def foo(argument, *)
   bar(argument, *)
 end
+
+def foo(**)
+  { default: 1, ** }
+end
diff --git a/unparser.gemspec b/unparser.gemspec
@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name        = 'unparser'
-  gem.version     = '0.6.8'
+  gem.version     = '0.6.9'
 
   gem.authors     = ['Markus Schirp']
   gem.email       = 'mbj@schirp-dso.com'