Merge pull request #319 from ruby-syntax-tree/keywords-bare-hashes

Handle keywords with bare hashes

Author: kddnewton

CHANGELOG.md

@@ -14,6 +14,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) a
 
 - The `nesting` field on the results of the indexing operation is no longer a single flat array. Instead it is an array of arrays, where each array is a single nesting level. This more accurately reflects the nesting of the nodes in the tree. For example, `class Foo::Bar::Baz; end` would result in `[Foo, Bar, Baz]`, but that incorrectly implies that you can see constants at each of those levels. Now this would result in `[[Foo, Bar, Baz]]` to indicate that it can see either the top level or constants within the scope of `Foo::Bar::Baz` only.
 - When formatting hashes that have omitted values and mixed hash rockets with labels, the formatting now maintains whichever delimiter was used in the source. This is because forcing the use of hash rockets with omitted values results in a syntax error.
+- Handle the case where a bare hash is used after the `break`, `next`, or `return` keywords. Previously this would result in hash labels which is not valid syntax. Now it maintains the delimiters used in the source.
 
 ## [6.0.0] - 2023-02-10
 

lib/syntax_tree/node.rb

@@ -1874,7 +1874,15 @@ def ===(other)
     end
 
     def format_key(q, key)
-      (@key_formatter ||= HashKeyFormatter.for(self)).format_key(q, key)
+      @key_formatter ||=
+        case q.parents.take(3).last
+        when Break, Next, ReturnNode
+          HashKeyFormatter::Identity.new
+        else
+          HashKeyFormatter.for(self)
+        end
+
+      @key_formatter.format_key(q, key)
     end
   end
 

test/fixtures/break.rb

@@ -33,3 +33,5 @@
     qux
   end
 )
+%
+break :foo => "bar"

test/fixtures/next.rb

@@ -72,3 +72,5 @@
   fun foo do
   end
 )
+%
+next :foo => "bar"

test/fixtures/return.rb

@@ -37,3 +37,5 @@
 return []
 %
 return [1]
+%
+return :foo => "bar"