syntax_tree 2.3.0 → 2.4.1

data/lib/syntax_tree/node.rb

@@ -3,9 +3,21 @@
 module SyntaxTree
   # Represents the location of a node in the tree from the source code.
   class Location
-    attr_reader :start_line, :start_char, :start_column, :end_line, :end_char, :end_column
+    attr_reader :start_line,
+                :start_char,
+                :start_column,
+                :end_line,
+                :end_char,
+                :end_column
 
-    def initialize(start_line:, start_char:, start_column:, end_line:, end_char:, end_column:)
+    def initialize(
+      start_line:,
+      start_char:,
+      start_column:,
+      end_line:,
+      end_char:,
+      end_column:
+    )
       @start_line = start_line
       @start_char = start_char
       @start_column = start_column
@@ -39,7 +51,7 @@ module SyntaxTree
       [start_line, start_char, start_column, end_line, end_char, end_column]
     end
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         start_line: start_line,
         start_char: start_char,
@@ -62,7 +74,14 @@ module SyntaxTree
     end
 
     def self.fixed(line:, char:, column:)
-      new(start_line: line, start_char: char, start_column: column, end_line: line, end_char: char, end_column: column)
+      new(
+        start_line: line,
+        start_char: char,
+        start_column: column,
+        end_line: line,
+        end_char: char,
+        end_column: column
+      )
     end
   end
 
@@ -102,6 +121,10 @@ module SyntaxTree
       visitor = Visitor::JSONVisitor.new
       visitor.visit(self).to_json(*opts)
     end
+
+    def construct_keys
+      PP.format(+"") { |q| Visitor::MatchVisitor.new(q).visit(self) }
+    end
   end
 
   # BEGINBlock represents the use of the +BEGIN+ keyword, which hooks into the
@@ -140,7 +163,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         lbrace: lbrace,
         statements: statements,
@@ -192,7 +215,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -243,7 +266,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         lbrace: lbrace,
         statements: statements,
@@ -298,7 +321,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -323,6 +346,8 @@ module SyntaxTree
   # symbols (note that this includes dynamic symbols like
   # :"left-#{middle}-right").
   class Alias < Node
+    # Formats an argument to the alias keyword. For symbol literals it uses the
+    # value of the symbol directly to look like bare words.
     class AliasArgumentFormatter
       # [DynaSymbol | SymbolLiteral] the argument being passed to alias
       attr_reader :argument
@@ -374,7 +399,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { left: left, right: right, location: location, comments: comments }
     end
 
@@ -435,7 +460,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         collection: collection,
         index: index,
@@ -496,7 +521,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         collection: collection,
         index: index,
@@ -558,7 +583,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -606,7 +631,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location, comments: comments }
     end
 
@@ -642,7 +667,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -679,7 +704,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -729,7 +754,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -745,6 +770,7 @@ module SyntaxTree
   #     [one, two, three]
   #
   class ArrayLiteral < Node
+    # Formats an array of multiple simple string literals into the %w syntax.
     class QWordsFormatter
       # [Args] the contents of the array
       attr_reader :contents
@@ -761,7 +787,7 @@ module SyntaxTree
               if part.is_a?(StringLiteral)
                 q.format(part.parts.first)
               else
-                q.text(part.value[1..-1])
+                q.text(part.value[1..])
               end
             end
           end
@@ -770,6 +796,7 @@ module SyntaxTree
       end
     end
 
+    # Formats an array of multiple simple symbol literals into the %i syntax.
     class QSymbolsFormatter
       # [Args] the contents of the array
       attr_reader :contents
@@ -791,6 +818,28 @@ module SyntaxTree
       end
     end
 
+    # Formats an array that contains only a list of variable references. To make
+    # things simpler, if there are a bunch, we format them all using the "fill"
+    # algorithm as opposed to breaking them into a ton of lines. For example,
+    #
+    #     [foo, bar, baz]
+    #
+    # instead of becoming:
+    #
+    #     [
+    #       foo,
+    #       bar,
+    #       baz
+    #     ]
+    #
+    # would instead become:
+    #
+    #     [
+    #       foo, bar,
+    #       baz
+    #     ]
+    #
+    # provided the line length was hit between `bar` and `baz`.
     class VarRefsFormatter
       # [Args] the contents of the array
       attr_reader :contents
@@ -816,6 +865,9 @@ module SyntaxTree
       end
     end
 
+    # This is a special formatter used if the array literal contains no values
+    # but _does_ contain comments. In this case we do some special formatting to
+    # make sure the comments gets indented properly.
     class EmptyWithCommentsFormatter
       # [LBracket] the opening bracket
       attr_reader :lbracket
@@ -865,7 +917,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         lbracket: lbracket,
         contents: contents,
@@ -951,7 +1003,8 @@ module SyntaxTree
     # If we have an empty array that contains only comments, then we're going
     # to do some special printing to ensure they get indented correctly.
     def empty_with_comments?
-      contents.nil? && lbracket.comments.any? && lbracket.comments.none?(&:inline?)
+      contents.nil? && lbracket.comments.any? &&
+        lbracket.comments.none?(&:inline?)
     end
   end
 
@@ -973,6 +1026,7 @@ module SyntaxTree
   # and an optional array of positional matches that occur after the splat.
   # All of the in clauses above would create an AryPtn node.
   class AryPtn < Node
+    # Formats the optional splat of an array pattern.
     class RestFormatter
       # [VarField] the identifier that represents the remaining positionals
       attr_reader :value
@@ -1035,7 +1089,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         constant: constant,
         requireds: requireds,
@@ -1067,6 +1121,8 @@ module SyntaxTree
         q.text("[")
         q.seplist(parts) { |part| q.format(part) }
         q.text("]")
+      elsif parts.empty?
+        q.text("[]")
       else
         q.group { q.seplist(parts) { |part| q.format(part) } }
       end
@@ -1077,7 +1133,8 @@ module SyntaxTree
   module AssignFormatting
     def self.skip_indent?(value)
       case value
-      in ArrayLiteral | HashLiteral | Heredoc | Lambda | QSymbols | QWords | Symbols | Words
+      in ArrayLiteral | HashLiteral | Heredoc | Lambda | QSymbols | QWords |
+           Symbols | Words
         true
       in Call[receiver:]
         skip_indent?(receiver)
@@ -1123,7 +1180,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { target: target, value: value, location: location, comments: comments }
     end
 
@@ -1185,12 +1242,12 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { key: key, value: value, location: location, comments: comments }
     end
 
     def format(q)
-      if value&.is_a?(HashLiteral)
+      if value.is_a?(HashLiteral)
         format_contents(q)
       else
         q.group { format_contents(q) }
@@ -1243,7 +1300,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -1281,7 +1338,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -1316,7 +1373,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -1329,21 +1386,28 @@ module SyntaxTree
   # hash or bare hash. It first determines if every key in the hash can use
   # labels. If it can, it uses labels. Otherwise it uses hash rockets.
   module HashKeyFormatter
+    # Formats the keys of a hash literal using labels.
     class Labels
+      LABEL = /^[@$_A-Za-z]([_A-Za-z0-9]*)?([!_=?A-Za-z0-9])?$/
+
       def format_key(q, key)
         case key
-        when Label
+        in Label
           q.format(key)
-        when SymbolLiteral
+        in SymbolLiteral
           q.format(key.value)
           q.text(":")
-        when DynaSymbol
+        in DynaSymbol[parts: [TStringContent[value: LABEL] => part]]
+          q.format(part)
+          q.text(":")
+        in DynaSymbol
           q.format(key)
           q.text(":")
         end
       end
     end
 
+    # Formats the keys of a hash literal using hash rockets.
     class Rockets
       def format_key(q, key)
         case key
@@ -1417,7 +1481,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { assocs: assocs, location: location, comments: comments }
     end
 
@@ -1459,7 +1523,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { bodystmt: bodystmt, location: location, comments: comments }
     end
 
@@ -1507,7 +1571,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { statement: statement, location: location, comments: comments }
     end
 
@@ -1567,7 +1631,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         left: left,
         operator: operator,
@@ -1682,7 +1746,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { params: params, locals: locals, location: location, comments: comments }
     end
 
@@ -1726,7 +1790,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { name: name, location: location, comments: comments }
     end
 
@@ -1822,7 +1886,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statements: statements,
         rescue_clause: rescue_clause,
@@ -1868,6 +1932,7 @@ module SyntaxTree
 
   # Responsible for formatting either a BraceBlock or a DoBlock.
   class BlockFormatter
+    # Formats the opening brace or keyword of a block.
     class BlockOpenFormatter
       # [String] the actual output that should be printed
       attr_reader :text
@@ -1933,9 +1998,9 @@ module SyntaxTree
       end
 
       q.group do
-        q.if_break { format_break(q, break_opening, break_closing) }.if_flat do
-          format_flat(q, flat_opening, flat_closing)
-        end
+        q
+          .if_break { format_break(q, break_opening, break_closing) }
+          .if_flat { format_flat(q, flat_opening, flat_closing) }
       end
     end
 
@@ -2060,7 +2125,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         lbrace: lbrace,
         block_var: block_var,
@@ -2099,7 +2164,11 @@ module SyntaxTree
           #
           #     break
           #
-        in [Paren[contents: { body: [ArrayLiteral[contents: { parts: [_, _, *] }] => array] }]]
+        in [Paren[
+             contents: {
+               body: [ArrayLiteral[contents: { parts: [_, _, *] }] => array]
+             }
+           ]]
           # Here we have a single argument that is a set of parentheses wrapping
           # an array literal that has at least 2 elements. We're going to print
           # the contents of the array directly. This would be like if we had:
@@ -2255,7 +2324,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -2287,6 +2356,20 @@ module SyntaxTree
     end
   end
 
+  # This is probably the most complicated formatter in this file. It's
+  # responsible for formatting chains of method calls, with or without arguments
+  # or blocks. In general, we want to go from something like
+  #
+  #     foo.bar.baz
+  #
+  # to
+  #
+  #     foo
+  #       .bar
+  #       .baz
+  #
+  # Of course there are a lot of caveats to that, including trailing operators
+  # when necessary, where comments are places, how blocks are aligned, etc.
   class CallChainFormatter
     # [Call | MethodAddBlock] the top of the call chain
     attr_reader :node
@@ -2301,7 +2384,7 @@ module SyntaxTree
 
       # First, walk down the chain until we get to the point where we're not
       # longer at a chainable node.
-      while true
+      loop do
         case children.last
         in Call[receiver: Call]
           children << children.last.receiver
@@ -2321,7 +2404,7 @@ module SyntaxTree
       # block. For more details, see
       # https://github.com/prettier/plugin-ruby/issues/863.
       parents = q.parents.take(4)
-      if parent = parents[2]
+      if (parent = parents[2])
         # If we're at a do_block, then we want to go one more level up. This is
         # because do blocks have BodyStmt nodes instead of just Statements
         # nodes.
@@ -2335,7 +2418,11 @@ module SyntaxTree
       end
 
       if children.length >= threshold
-        q.group { q.if_break { format_chain(q, children) }.if_flat { node.format_contents(q) } }
+        q.group do
+          q
+            .if_break { format_chain(q, children) }
+            .if_flat { node.format_contents(q) }
+        end
       else
         node.format_contents(q)
       end
@@ -2346,9 +2433,9 @@ module SyntaxTree
       # chain of calls without arguments except for the last one. This is common
       # enough in Ruby source code that it's worth the extra complexity here.
       empty_except_last =
-        children.drop(1).all? do |child|
-          child.is_a?(Call) && child.arguments.nil?
-        end
+        children
+          .drop(1)
+          .all? { |child| child.is_a?(Call) && child.arguments.nil? }
 
       # Here, we're going to add all of the children onto the stack of the
       # formatter so it's as if we had descending normally into them. This is
@@ -2368,9 +2455,12 @@ module SyntaxTree
           # and a trailing operator.
           skip_operator = false
 
-          while child = children.pop
+          while (child = children.pop)
             case child
-            in Call[receiver: Call[message: { value: "where" }], message: { value: "not" }]
+            in Call[
+                 receiver: Call[message: { value: "where" }],
+                 message: { value: "not" }
+               ]
               # This is very specialized behavior wherein we group
               # .where.not calls together because it looks better. For more
               # information, see
@@ -2432,16 +2522,25 @@ module SyntaxTree
     # want to indent the first call. So we'll pop off the first children and
     # format it separately here.
     def attach_directly?(node)
-      [ArrayLiteral, HashLiteral, Heredoc, If, Unless, XStringLiteral]
-        .include?(node.receiver.class)
+      [ArrayLiteral, HashLiteral, Heredoc, If, Unless, XStringLiteral].include?(
+        node.receiver.class
+      )
     end
 
-    def format_child(q, child, skip_comments: false, skip_operator: false, skip_attached: false)
+    def format_child(
+      q,
+      child,
+      skip_comments: false,
+      skip_operator: false,
+      skip_attached: false
+    )
       # First, format the actual contents of the child.
       case child
       in Call
         q.group do
-          q.format(CallOperatorFormatter.new(child.operator)) unless skip_operator
+          unless skip_operator
+            q.format(CallOperatorFormatter.new(child.operator))
+          end
           q.format(child.message) if child.message != :call
           child.format_arguments(q) unless skip_attached
         end
@@ -2513,7 +2612,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         receiver: receiver,
         operator: operator,
@@ -2528,8 +2627,13 @@ module SyntaxTree
       # If we're at the top of a call chain, then we're going to do some
       # specialized printing in case we can print it nicely. We _only_ do this
       # at the top of the chain to avoid weird recursion issues.
-      if !CallChainFormatter.chained?(q.parent) && CallChainFormatter.chained?(receiver)
-        q.group { q.if_break { CallChainFormatter.new(self).format(q) }.if_flat { format_contents(q) } }
+      if !CallChainFormatter.chained?(q.parent) &&
+           CallChainFormatter.chained?(receiver)
+        q.group do
+          q
+            .if_break { CallChainFormatter.new(self).format(q) }
+            .if_flat { format_contents(q) }
+        end
       else
         format_contents(q)
       end
@@ -2618,7 +2722,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         keyword: keyword,
         value: value,
@@ -2683,7 +2787,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         value: value,
         operator: operator,
@@ -2772,7 +2876,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         constant: constant,
         superclass: superclass,
@@ -2837,7 +2941,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -2875,7 +2979,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         message: message,
         arguments: arguments,
@@ -2899,6 +3003,7 @@ module SyntaxTree
         q.text(" ")
         yield
       in Args[parts: [IfOp]]
+        q.if_flat { q.text(" ") }
         yield
       in Args[parts: [Command => command]]
         align(q, command, &block)
@@ -2956,7 +3061,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         receiver: receiver,
         operator: operator,
@@ -3078,7 +3183,7 @@ module SyntaxTree
     end
 
     def ignore?
-      value[1..-1].strip == "stree-ignore"
+      value[1..].strip == "stree-ignore"
     end
 
     def comments
@@ -3095,7 +3200,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, inline: inline, location: location }
     end
 
@@ -3141,7 +3246,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -3183,7 +3288,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         parent: parent,
         constant: constant,
@@ -3230,7 +3335,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         parent: parent,
         constant: constant,
@@ -3275,7 +3380,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { constant: constant, location: location, comments: comments }
     end
 
@@ -3311,7 +3416,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -3355,7 +3460,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         name: name,
         params: params,
@@ -3440,7 +3545,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         target: target,
         operator: operator,
@@ -3508,7 +3613,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -3574,7 +3679,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         target: target,
         operator: operator,
@@ -3649,7 +3754,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         keyword: keyword,
         block_var: block_var,
@@ -3729,7 +3834,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { left: left, right: right, location: location, comments: comments }
     end
 
@@ -3777,7 +3882,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { left: left, right: right, location: location, comments: comments }
     end
 
@@ -3799,7 +3904,7 @@ module SyntaxTree
     # quotes, then single quotes would deactivate it.)
     def self.locked?(node)
       node.parts.any? do |part|
-        part.is_a?(TStringContent) && part.value.match?(/#[@${]|[\\]/)
+        !part.is_a?(TStringContent) || part.value.match?(/\\|#[@${]/)
       end
     end
 
@@ -3864,7 +3969,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, quote: quote, location: location, comments: comments }
     end
 
@@ -3905,9 +4010,14 @@ module SyntaxTree
         matching = Quotes.matching(quote[2])
         pattern = /[\n#{Regexp.escape(matching)}'"]/
 
-        if parts.any? { |part|
-             part.is_a?(TStringContent) && part.value.match?(pattern)
-           }
+        # This check is to ensure we don't find a matching quote inside of the
+        # symbol that would be confusing.
+        matched =
+          parts.any? do |part|
+            part.is_a?(TStringContent) && part.value.match?(pattern)
+          end
+
+        if matched
           [quote, matching]
         elsif Quotes.locked?(self)
           ["#{":" unless hash_key}'", "'"]
@@ -3916,7 +4026,7 @@ module SyntaxTree
         end
       elsif Quotes.locked?(self)
         if quote.start_with?(":")
-          [hash_key ? quote[1..-1] : quote, quote[1..-1]]
+          [hash_key ? quote[1..] : quote, quote[1..]]
         else
           [hash_key ? quote : ":#{quote}", quote]
         end
@@ -3959,7 +4069,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         keyword: keyword,
         statements: statements,
@@ -4025,7 +4135,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         statements: statements,
@@ -4097,7 +4207,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
 
@@ -4132,7 +4242,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -4162,7 +4272,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -4194,7 +4304,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -4233,7 +4343,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         keyword: keyword,
         statements: statements,
@@ -4287,7 +4397,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -4330,7 +4440,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         value: value,
         arguments: arguments,
@@ -4342,7 +4452,8 @@ module SyntaxTree
     def format(q)
       q.format(value)
 
-      if arguments.is_a?(ArgParen) && arguments.arguments.nil? && !value.is_a?(Const)
+      if arguments.is_a?(ArgParen) && arguments.arguments.nil? &&
+           !value.is_a?(Const)
         # If you're using an explicit set of parentheses on something that looks
         # like a constant, then we need to match that in order to maintain valid
         # Ruby. For example, you could do something like Foo(), on which we
@@ -4389,7 +4500,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         parent: parent,
         operator: operator,
@@ -4435,7 +4546,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -4487,7 +4598,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         constant: constant,
         left: left,
@@ -4551,7 +4662,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         index: index,
         collection: collection,
@@ -4608,7 +4719,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -4622,6 +4733,9 @@ module SyntaxTree
   #     { key => value }
   #
   class HashLiteral < Node
+    # This is a special formatter used if the hash literal contains no values
+    # but _does_ contain comments. In this case we do some special formatting to
+    # make sure the comments gets indented properly.
     class EmptyWithCommentsFormatter
       # [LBrace] the opening brace
       attr_reader :lbrace
@@ -4671,7 +4785,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { lbrace: lbrace, assocs: assocs, location: location, comments: comments }
     end
 
@@ -4740,7 +4854,14 @@ module SyntaxTree
     # [Array[ Comment | EmbDoc ]] the comments attached to this node
     attr_reader :comments
 
-    def initialize(beginning:, ending: nil, dedent: 0, parts: [], location:, comments: [])
+    def initialize(
+      beginning:,
+      ending: nil,
+      dedent: 0,
+      parts: [],
+      location:,
+      comments: []
+    )
       @beginning = beginning
       @ending = ending
       @dedent = dedent
@@ -4759,7 +4880,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         location: location,
@@ -4774,8 +4895,12 @@ module SyntaxTree
       # prettyprint module. It's when you want to force a newline, but don't
       # want to force the break parent.
       breakable = -> do
-        q.target <<
-          PrettyPrint::Breakable.new(" ", 1, indent: false, force: true)
+        q.target << PrettyPrint::Breakable.new(
+          " ",
+          1,
+          indent: false,
+          force: true
+        )
       end
 
       q.group do
@@ -4831,7 +4956,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -4848,6 +4973,7 @@ module SyntaxTree
   #     end
   #
   class HshPtn < Node
+    # Formats a key-value pair in a hash pattern. The value is optional.
     class KeywordFormatter
       # [Label] the keyword being used
       attr_reader :key
@@ -4874,6 +5000,7 @@ module SyntaxTree
       end
     end
 
+    # Formats the optional double-splat from the pattern.
     class KeywordRestFormatter
       # [VarField] the parameter that matches the remaining keywords
       attr_reader :keyword_rest
@@ -4923,7 +5050,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         constant: constant,
         keywords: keywords,
@@ -4937,36 +5064,64 @@ module SyntaxTree
       parts = keywords.map { |(key, value)| KeywordFormatter.new(key, value) }
       parts << KeywordRestFormatter.new(keyword_rest) if keyword_rest
 
+      nested = PATTERNS.include?(q.parent.class)
       contents = -> do
         q.group { q.seplist(parts) { |part| q.format(part, stackable: false) } }
 
         # If there isn't a constant, and there's a blank keyword_rest, then we
         # have an plain ** that needs to have a `then` after it in order to
         # parse correctly on the next parse.
-        q.text(" then") if !constant && keyword_rest && keyword_rest.value.nil?
+        if !constant && keyword_rest && keyword_rest.value.nil? && !nested
+          q.text(" then")
+        end
       end
 
+      # If there is a constant, we're going to format to have the constant name
+      # first and then use brackets.
       if constant
-        q.format(constant)
-        q.group(0, "[", "]", &contents)
+        q.group do
+          q.format(constant)
+          q.text("[")
+          q.indent do
+            q.breakable("")
+            contents.call
+          end
+          q.breakable("")
+          q.text("]")
+        end
         return
       end
 
+      # If there's nothing at all, then we're going to use empty braces.
       if parts.empty?
         q.text("{}")
-      elsif PATTERNS.include?(q.parent.class)
-        q.text("{ ")
-        contents.call
-        q.text(" }")
-      else
+        return
+      end
+
+      # If there's only one pair, then we'll just print the contents provided
+      # we're not inside another pattern.
+      if !nested && parts.size == 1
         contents.call
+        return
+      end
+
+      # Otherwise, we're going to always use braces to make it clear it's a hash
+      # pattern.
+      q.group do
+        q.text("{")
+        q.indent do
+          q.breakable
+          contents.call
+        end
+        q.breakable
+        q.text("}")
       end
     end
   end
 
   # The list of nodes that represent patterns inside of pattern matching so that
   # when a pattern is being printed it knows if it's nested.
-  PATTERNS = [AryPtn, Binary, FndPtn, HshPtn, RAssign]
+  PATTERNS = [AryPtn, Binary, FndPtn, HshPtn, RAssign].freeze
 
   # Ident represents an identifier anywhere in code. It can represent a very
   # large number of things, depending on where it is in the syntax tree.
@@ -4996,7 +5151,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5013,7 +5168,7 @@ module SyntaxTree
     def self.call(parent)
       queue = [parent]
 
-      while node = queue.shift
+      while (node = queue.shift)
         return true if [Assign, MAssign, OpAssign].include?(node.class)
         queue += node.child_nodes
       end
@@ -5040,9 +5195,12 @@ module SyntaxTree
         else
           # Otherwise, we're going to check the conditional for certain cases.
           case node
-          in { predicate: Assign | Command | CommandCall | MAssign | OpAssign }
+          in predicate: Assign | Command | CommandCall | MAssign | OpAssign
             false
-          in { statements: { body: [truthy] }, consequent: Else[statements: { body: [falsy] }] }
+          in {
+               statements: { body: [truthy] },
+               consequent: Else[statements: { body: [falsy] }]
+             }
             ternaryable?(truthy) && ternaryable?(falsy)
           else
             false
@@ -5053,8 +5211,8 @@ module SyntaxTree
       private
 
       # Certain expressions cannot be reduced to a ternary without adding
-      # parentheses around them. In this case we say they cannot be ternaried and
-      # default instead to breaking them into multiple lines.
+      # parentheses around them. In this case we say they cannot be ternaried
+      # and default instead to breaking them into multiple lines.
       def ternaryable?(statement)
         # This is a list of nodes that should not be allowed to be a part of a
         # ternary clause.
@@ -5112,13 +5270,15 @@ module SyntaxTree
         q.group { format_break(q, force: true) }
       else
         q.group do
-          q.if_break { format_break(q, force: false) }.if_flat do
-            Parentheses.flat(q) do
-              q.format(node.statements)
-              q.text(" #{keyword} ")
-              q.format(node.predicate)
+          q
+            .if_break { format_break(q, force: false) }
+            .if_flat do
+              Parentheses.flat(q) do
+                q.format(node.statements)
+                q.text(" #{keyword} ")
+                q.format(node.predicate)
+              end
             end
-          end
         end
       end
     end
@@ -5147,51 +5307,53 @@ module SyntaxTree
 
     def format_ternary(q)
       q.group do
-        q.if_break do
-          q.text("#{keyword} ")
-          q.nest(keyword.length + 1) { q.format(node.predicate) }
-
-          q.indent do
-            q.breakable
-            q.format(node.statements)
-          end
+        q
+          .if_break do
+            q.text("#{keyword} ")
+            q.nest(keyword.length + 1) { q.format(node.predicate) }
 
-          q.breakable
-          q.group do
-            q.format(node.consequent.keyword)
             q.indent do
-              # This is a very special case of breakable where we want to force
-              # it into the output but we _don't_ want to explicitly break the
-              # parent. If a break-parent shows up in the tree, then it's going
-              # to force it all the way up to the tree, which is going to negate
-              # the ternary. Maybe this should be an option in prettyprint? As
-              # in force: :no_break_parent or something.
-              q.target << PrettyPrint::Breakable.new(" ", 1, force: true)
-              q.format(node.consequent.statements)
+              q.breakable
+              q.format(node.statements)
             end
-          end
 
-          q.breakable
-          q.text("end")
-        end.if_flat do
-          Parentheses.flat(q) do
-            q.format(node.predicate)
-            q.text(" ? ")
+            q.breakable
+            q.group do
+              q.format(node.consequent.keyword)
+              q.indent do
+                # This is a very special case of breakable where we want to
+                # force it into the output but we _don't_ want to explicitly
+                # break the parent. If a break-parent shows up in the tree, then
+                # it's going to force it all the way up to the tree, which is
+                # going to negate the ternary. Maybe this should be an option in
+                # prettyprint? As in force: :no_break_parent or something.
+                q.target << PrettyPrint::Breakable.new(" ", 1, force: true)
+                q.format(node.consequent.statements)
+              end
+            end
+
+            q.breakable
+            q.text("end")
+          end
+          .if_flat do
+            Parentheses.flat(q) do
+              q.format(node.predicate)
+              q.text(" ? ")
 
-            statements = [node.statements, node.consequent.statements]
-            statements.reverse! if keyword == "unless"
+              statements = [node.statements, node.consequent.statements]
+              statements.reverse! if keyword == "unless"
 
-            q.format(statements[0])
-            q.text(" : ")
-            q.format(statements[1])
+              q.format(statements[0])
+              q.text(" : ")
+              q.format(statements[1])
+            end
           end
-        end
       end
     end
 
     def contains_conditional?
       case node
-      in { statements: { body: [If | IfMod | IfOp | Unless | UnlessMod] } }
+      in statements: { body: [If | IfMod | IfOp | Unless | UnlessMod] }
         true
       else
         false
@@ -5241,7 +5403,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         statements: statements,
@@ -5291,7 +5453,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         truthy: truthy,
@@ -5309,7 +5471,8 @@ module SyntaxTree
         Yield0, ZSuper
       ]
 
-      if q.parent.is_a?(Paren) || force_flat.include?(truthy.class) || force_flat.include?(falsy.class)
+      if q.parent.is_a?(Paren) || force_flat.include?(truthy.class) ||
+           force_flat.include?(falsy.class)
         q.group { format_flat(q) }
         return
       end
@@ -5429,7 +5592,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         predicate: predicate,
@@ -5470,7 +5633,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5517,7 +5680,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         pattern: pattern,
         statements: statements,
@@ -5576,7 +5739,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5586,7 +5749,7 @@ module SyntaxTree
         # the values, then we're going to insert them every 3 characters
         # starting from the right.
         index = (value.length + 2) % 3
-        q.text("  #{value}"[index..-1].scan(/.../).join("_").strip)
+        q.text("  #{value}"[index..].scan(/.../).join("_").strip)
       else
         q.text(value)
       end
@@ -5620,7 +5783,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5665,7 +5828,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5702,7 +5865,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { name: name, location: location, comments: comments }
     end
 
@@ -5748,7 +5911,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5783,7 +5946,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -5819,7 +5982,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         params: params,
         statements: statements,
@@ -5841,25 +6004,27 @@ module SyntaxTree
         end
 
         q.text(" ")
-        q.if_break do
-          force_parens =
-            q.parents.any? do |node|
-              node.is_a?(Command) || node.is_a?(CommandCall)
+        q
+          .if_break do
+            force_parens =
+              q.parents.any? do |node|
+                node.is_a?(Command) || node.is_a?(CommandCall)
+              end
+
+            q.text(force_parens ? "{" : "do")
+            q.indent do
+              q.breakable
+              q.format(statements)
             end
 
-          q.text(force_parens ? "{" : "do")
-          q.indent do
             q.breakable
+            q.text(force_parens ? "}" : "end")
+          end
+          .if_flat do
+            q.text("{ ")
             q.format(statements)
+            q.text(" }")
           end
-
-          q.breakable
-          q.text(force_parens ? "}" : "end")
-        end.if_flat do
-          q.text("{ ")
-          q.format(statements)
-          q.text(" }")
-        end
       end
     end
   end
@@ -5888,7 +6053,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5921,7 +6086,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -5954,7 +6119,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -6004,7 +6169,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { target: target, value: value, location: location, comments: comments }
     end
 
@@ -6051,7 +6216,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { call: call, block: block, location: location, comments: comments }
     end
 
@@ -6059,8 +6224,13 @@ module SyntaxTree
       # If we're at the top of a call chain, then we're going to do some
       # specialized printing in case we can print it nicely. We _only_ do this
       # at the top of the chain to avoid weird recursion issues.
-      if !CallChainFormatter.chained?(q.parent) && CallChainFormatter.chained?(call)
-        q.group { q.if_break { CallChainFormatter.new(self).format(q) }.if_flat { format_contents(q) } }
+      if !CallChainFormatter.chained?(q.parent) &&
+           CallChainFormatter.chained?(call)
+        q.group do
+          q
+            .if_break { CallChainFormatter.new(self).format(q) }
+            .if_flat { format_contents(q) }
+        end
       else
         format_contents(q)
       end
@@ -6107,7 +6277,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location, comma: comma, comments: comments }
     end
 
@@ -6151,7 +6321,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { contents: contents, location: location, comments: comments }
     end
 
@@ -6207,7 +6377,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         constant: constant,
         bodystmt: bodystmt,
@@ -6274,7 +6444,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location, comments: comments }
     end
 
@@ -6323,7 +6493,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -6360,7 +6530,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -6406,7 +6576,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         target: target,
         operator: operator,
@@ -6474,7 +6644,16 @@ module SyntaxTree
   # This approach maintains the nice conciseness of the inline version, while
   # keeping the correct semantic meaning.
   module Parentheses
-    NODES = [Args, Assign, Assoc, Binary, Call, Defined, MAssign, OpAssign]
+    NODES = [
+      Args,
+      Assign,
+      Assoc,
+      Binary,
+      Call,
+      Defined,
+      MAssign,
+      OpAssign
+    ].freeze
 
     def self.flat(q)
       return yield unless NODES.include?(q.parent.class)
@@ -6506,6 +6685,8 @@ module SyntaxTree
   #     def method(param) end
   #
   class Params < Node
+    # Formats the optional position of the parameters. This includes the label,
+    # as well as the default value.
     class OptionalFormatter
       # [Ident] the name of the parameter
       attr_reader :name
@@ -6529,6 +6710,8 @@ module SyntaxTree
       end
     end
 
+    # Formats the keyword position of the parameters. This includes the label,
+    # as well as an optional default value.
     class KeywordFormatter
       # [Ident] the name of the parameter
       attr_reader :name
@@ -6555,6 +6738,8 @@ module SyntaxTree
       end
     end
 
+    # Formats the keyword_rest position of the parameters. This can be the **nil
+    # syntax, the ... syntax, or the ** syntax.
     class KeywordRestFormatter
       # [:nil | ArgsForward | KwRestParam] the value of the parameter
       attr_reader :value
@@ -6568,11 +6753,7 @@ module SyntaxTree
       end
 
       def format(q)
-        if value == :nil
-          q.text("**nil")
-        else
-          q.format(value)
-        end
+        value == :nil ? q.text("**nil") : q.format(value)
       end
     end
 
@@ -6653,7 +6834,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         location: location,
         requireds: requireds,
@@ -6674,18 +6855,17 @@ module SyntaxTree
       ]
 
       parts << rest if rest && !rest.is_a?(ExcessedComma)
-      parts +=
-        [
-          *posts,
-          *keywords.map { |(name, value)| KeywordFormatter.new(name, value) }
-        ]
+      parts += [
+        *posts,
+        *keywords.map { |(name, value)| KeywordFormatter.new(name, value) }
+      ]
 
       parts << KeywordRestFormatter.new(keyword_rest) if keyword_rest
       parts << block if block
 
       contents = -> do
         q.seplist(parts) { |part| q.format(part) }
-        q.format(rest) if rest && rest.is_a?(ExcessedComma)
+        q.format(rest) if rest.is_a?(ExcessedComma)
       end
 
       if ![Def, Defs, DefEndless].include?(q.parent.class) || parts.empty?
@@ -6735,7 +6915,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         lparen: lparen,
         contents: contents,
@@ -6786,7 +6966,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -6819,7 +6999,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { statements: statements, location: location, comments: comments }
     end
 
@@ -6864,7 +7044,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         elements: elements,
@@ -6919,18 +7099,9 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
-
-    def pretty_print(q)
-      q.group(2, "(", ")") do
-        q.text("qsymbols_beg")
-
-        q.breakable
-        q.pp(value)
-      end
-    end
   end
 
   # QWords represents a string literal array without interpolation.
@@ -6964,7 +7135,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         elements: elements,
@@ -7019,7 +7190,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7051,7 +7222,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -7080,7 +7251,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7105,7 +7276,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7137,7 +7308,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -7176,7 +7347,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { beginning: beginning, parts: parts, location: location }
     end
   end
@@ -7209,7 +7380,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7243,7 +7414,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7284,10 +7455,11 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         ending: ending,
+        options: options,
         parts: parts,
         location: location,
         comments: comments
@@ -7295,7 +7467,7 @@ module SyntaxTree
     end
 
     def format(q)
-      braces = ambiguous?(q) || include?(%r{\/})
+      braces = ambiguous?(q) || include?(%r{/})
 
       if braces && include?(/[{}]/)
         q.group do
@@ -7322,18 +7494,22 @@ module SyntaxTree
           end
 
           q.text("}")
-          q.text(ending[1..-1])
+          q.text(options)
         end
       else
         q.group do
           q.text("/")
           q.format_each(parts)
           q.text("/")
-          q.text(ending[1..-1])
+          q.text(options)
         end
       end
     end
 
+    def options
+      ending[1..]
+    end
+
     private
 
     def include?(pattern)
@@ -7389,7 +7565,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         exceptions: exceptions,
         variable: variable,
@@ -7464,7 +7640,10 @@ module SyntaxTree
 
       if consequent
         consequent.bind_end(end_char, end_column)
-        statements.bind_end(consequent.location.start_char, consequent.location.start_column)
+        statements.bind_end(
+          consequent.location.start_char,
+          consequent.location.start_column
+        )
       else
         statements.bind_end(end_char, end_column)
       end
@@ -7480,7 +7659,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         keyword: keyword,
         exception: exception,
@@ -7547,7 +7726,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         value: value,
@@ -7601,7 +7780,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { name: name, location: location, comments: comments }
     end
 
@@ -7638,7 +7817,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -7674,7 +7853,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -7710,7 +7889,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -7739,7 +7918,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -7778,7 +7957,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         target: target,
         bodystmt: bodystmt,
@@ -7880,7 +8059,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parser: parser, body: body, location: location, comments: comments }
     end
 
@@ -7950,12 +8129,19 @@ module SyntaxTree
         comment = parser_comments[comment_index]
         location = comment.location
 
-        if !comment.inline? && (start_char <= location.start_char) && (end_char >= location.end_char) && !comment.ignore?
-          while (node = body[body_index]) && (node.is_a?(VoidStmt) || node.location.start_char < location.start_char)
+        if !comment.inline? && (start_char <= location.start_char) &&
+             (end_char >= location.end_char) && !comment.ignore?
+          while (node = body[body_index]) &&
+                  (
+                    node.is_a?(VoidStmt) ||
+                      node.location.start_char < location.start_char
+                  )
             body_index += 1
           end
 
-          if body_index != 0 && body[body_index - 1].location.start_char < location.start_char && body[body_index - 1].location.end_char > location.start_char
+          if body_index != 0 &&
+               body[body_index - 1].location.start_char < location.start_char &&
+               body[body_index - 1].location.end_char > location.start_char
             # The previous node entirely encapsules the comment, so we don't
             # want to attach it here since it will get attached normally. This
             # is mostly in the case of hash and array literals.
@@ -7995,7 +8181,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location }
     end
   end
@@ -8033,14 +8219,14 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { left: left, right: right, location: location, comments: comments }
     end
 
     def format(q)
       q.group do
         q.format(left)
-        q.text(' \\')
+        q.text(" \\")
         q.indent do
           q.breakable(force: true)
           q.format(right)
@@ -8078,7 +8264,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { variable: variable, location: location, comments: comments }
     end
 
@@ -8118,7 +8304,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { statements: statements, location: location, comments: comments }
     end
 
@@ -8176,7 +8362,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, quote: quote, location: location, comments: comments }
     end
 
@@ -8239,7 +8425,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -8292,7 +8478,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8322,7 +8508,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8356,7 +8542,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -8397,7 +8583,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         elements: elements,
@@ -8453,7 +8639,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8482,7 +8668,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8512,7 +8698,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8546,7 +8732,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { constant: constant, location: location, comments: comments }
     end
 
@@ -8584,7 +8770,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { constant: constant, location: location, comments: comments }
     end
 
@@ -8623,7 +8809,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8663,7 +8849,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -8701,7 +8887,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -8737,7 +8923,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         parentheses: parentheses,
@@ -8748,7 +8934,9 @@ module SyntaxTree
 
     def format(q)
       parent = q.parents.take(2)[1]
-      ternary = (parent.is_a?(If) || parent.is_a?(Unless)) && Ternaryable.call(q, parent)
+      ternary =
+        (parent.is_a?(If) || parent.is_a?(Unless)) &&
+          Ternaryable.call(q, parent)
 
       q.text("not")
 
@@ -8802,7 +8990,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         operator: operator,
         statement: statement,
@@ -8822,6 +9010,9 @@ module SyntaxTree
   #     undef method
   #
   class Undef < Node
+    # Undef accepts a variable number of arguments that can be either DynaSymbol
+    # or SymbolLiteral objects. For SymbolLiteral objects we descend directly
+    # into the value in order to have it come out as bare words.
     class UndefArgumentFormatter
       # [DynaSymbol | SymbolLiteral] the symbol to undefine
       attr_reader :node
@@ -8865,7 +9056,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { symbols: symbols, location: location, comments: comments }
     end
 
@@ -8924,7 +9115,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         statements: statements,
@@ -8970,7 +9161,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         predicate: predicate,
@@ -9009,13 +9200,15 @@ module SyntaxTree
       end
 
       q.group do
-        q.if_break { format_break(q) }.if_flat do
-          Parentheses.flat(q) do
-            q.format(statements)
-            q.text(" #{keyword} ")
-            q.format(node.predicate)
+        q
+          .if_break { format_break(q) }
+          .if_flat do
+            Parentheses.flat(q) do
+              q.format(statements)
+              q.text(" #{keyword} ")
+              q.format(node.predicate)
+            end
           end
-        end
       end
     end
 
@@ -9065,7 +9258,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         statements: statements,
@@ -9121,7 +9314,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         predicate: predicate,
@@ -9187,7 +9380,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { left: left, right: right, location: location, comments: comments }
     end
 
@@ -9230,7 +9423,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -9274,7 +9467,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -9315,7 +9508,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -9355,7 +9548,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -9390,7 +9583,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { location: location, comments: comments }
     end
 
@@ -9441,7 +9634,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         arguments: arguments,
         statements: statements,
@@ -9522,7 +9715,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         predicate: predicate,
         statements: statements,
@@ -9578,7 +9771,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         statement: statement,
         predicate: predicate,
@@ -9647,7 +9840,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location, comments: comments }
     end
 
@@ -9687,7 +9880,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       {
         beginning: beginning,
         elements: elements,
@@ -9743,7 +9936,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location }
     end
   end
@@ -9772,7 +9965,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location }
     end
   end
@@ -9805,7 +9998,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { parts: parts, location: location, comments: comments }
     end
 
@@ -9843,7 +10036,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { arguments: arguments, location: location, comments: comments }
     end
 
@@ -9893,7 +10086,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end
 
@@ -9929,7 +10122,7 @@ module SyntaxTree
 
     alias deconstruct child_nodes
 
-    def deconstruct_keys(keys)
+    def deconstruct_keys(_keys)
       { value: value, location: location, comments: comments }
     end