standardrb --fix

lib/dead_end/block_node.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 module DeadEnd
-
   # A core data structure
   #
   # A block node keeps a reference to the block above it
@@ -157,7 +156,7 @@ def fork_invalid
     def handle_multiple
       @diagnose.next.first
     end
-    alias :remove_pseudo_pair :handle_multiple
+    alias_method :remove_pseudo_pair, :handle_multiple
 
     def split_leaning
       @diagnose.next.first

lib/dead_end/block_recorder.rb

@@ -5,7 +5,6 @@ module DeadEnd
   #
   # This class allows for tracing the algorithm
   class BlockRecorder
-
     # Convienece constructor for building a BlockRecorder given
     # a directory object.
     #
@@ -14,7 +13,7 @@ class BlockRecorder
     #
     # Multiple different processes may be logging to the same
     # directory, so writing to a subdir is recommended
-    def self.from_dir(dir, subdir: , code_lines: )
+    def self.from_dir(dir, subdir:, code_lines:)
       if dir == DEFAULT_VALUE
         dir = ENV["DEAD_END_RECORD_DIR"] || ENV["DEBUG"] ? DeadEnd.record_dir("tmp") : nil
       end

lib/dead_end/diagnose_node.rb

@@ -62,10 +62,10 @@ def call
       invalid = get_invalid
       return self if invalid.empty?
 
-      if @problem == :multiple_invalid_parents
-        @next = invalid.map {|b| BlockNode.from_blocks([b]) }
+      @next = if @problem == :multiple_invalid_parents
+        invalid.map { |b| BlockNode.from_blocks([b]) }
       else
-        @next = [ BlockNode.from_blocks(invalid) ]
+        [BlockNode.from_blocks(invalid)]
       end
 
       self
@@ -111,7 +111,7 @@ def call
       if left && right && invalid.length >= 3 && BlockNode.from_blocks([left, right]).valid?
         @problem = :invalid_inside_split_pair
 
-        invalid.reject! {|x| x == left || x == right }
+        invalid.reject! { |x| x == left || x == right }
 
         # If the left/right was not mapped properly or we've accidentally got a :multiple_invalid_parents
         # we can get a false positive, double check the invalid lines fully capture the problem
@@ -120,12 +120,11 @@ def call
           without_lines: invalid.flat_map(&:lines)
         )
 
-          return invalid
+          invalid
         end
       end
     end
 
-
     # ## (:remove_pseudo_pair) Handle else/ensure case
     #
     # Example:
@@ -168,19 +167,19 @@ def call
       return false if above.nil? || below.nil?
 
       if invalid.reject! { |block|
-          b = BlockNode.from_blocks([above, block, below])
-          b.leaning == :equal && b.valid?
-        }
+           b = BlockNode.from_blocks([above, block, below])
+           b.leaning == :equal && b.valid?
+         }
 
         if invalid.any?
           @problem = :remove_pseudo_pair
-          return invalid
+          invalid
         else
 
           invalid = block.parents.select(&:invalid?)
-          if (b = invalid.detect { |b| BlockNode.from_blocks([above, invalid - [b] , below].flatten).valid? })
+          if (b = invalid.detect { |b| BlockNode.from_blocks([above, invalid - [b], below].flatten).valid? })
             @problem = :extract_from_multiple
-            return [b]
+            [b]
           end
         end
       end
@@ -189,10 +188,10 @@ def call
     # We couldn't detect any special cases, either return 1 or N invalid nodes
     private def diagnose_one_or_more_parents
       invalid = block.parents.select(&:invalid?)
-      if invalid.length > 1
-        @problem = :multiple_invalid_parents
+      @problem = if invalid.length > 1
+        :multiple_invalid_parents
       else
-        @problem = :one_invalid_parent
+        :one_invalid_parent
       end
 
       invalid
@@ -201,7 +200,7 @@ def call
     private def diagnose_self
       if block.parents.empty?
         @problem = :self
-        return []
+        []
       end
     end
   end

lib/dead_end/indent_search.rb

@@ -28,7 +28,7 @@ module DeadEnd
   class IndentSearch
     attr_reader :finished
 
-    def initialize(tree: , record_dir: DEFAULT_VALUE)
+    def initialize(tree:, record_dir: DEFAULT_VALUE)
       @tree = tree
       @root = tree.root
       @finished = []
@@ -57,7 +57,7 @@ def call
         end
       end
 
-      @finished.sort_by! {|j| j.node.starts_at }
+      @finished.sort_by! { |j| j.node.starts_at }
 
       self
     end

lib/dead_end/indent_tree.rb

@@ -25,7 +25,6 @@ def initialize(document:, record_dir: DEFAULT_VALUE)
       @code_lines = document.code_lines
       @last_length = Float::INFINITY
 
-
       @recorder = BlockRecorder.from_dir(record_dir, subdir: "build_tree", code_lines: @code_lines)
     end
 

spec/unit/indent_search_spec.rb

@@ -328,7 +328,7 @@ def initialize(arguments:, block:, location:)
       search = IndentSearch.new(tree: tree).call
 
       expect(search.finished.first.node.to_s).to eq(<<~'EOM')
-          def on_args_add(arguments, argument)
+        def on_args_add(arguments, argument)
       EOM
     end
 

spec/unit/indent_tree_spec.rb

@@ -24,15 +24,14 @@ class Cow
       node = tree.root
 
       expect(node.diagnose).to eq(:one_invalid_parent)
-      node =  node.next_invalid
+      node = node.next_invalid
 
       expect(node.diagnose).to eq(:self)
       expect(node.to_s).to eq(<<~'EOM')
         end # two
       EOM
     end
 
-
     it "ambiguous kw" do
       source = <<~'EOM'
         class Cow
@@ -46,7 +45,7 @@ def speak
       node = tree.root
       expect(node.parents.length).to eq(2)
       expect(node.diagnose).to eq(:one_invalid_parent)
-      node =  node.next_invalid
+      node = node.next_invalid
 
       expect(node.diagnose).to eq(:self)
       expect(node.to_s).to eq(<<~'EOM')
@@ -74,7 +73,7 @@ class Buffalo
       # expect(node.parents.length).to eq(2)
 
       expect(node.diagnose).to eq(:multiple_invalid_parents)
-      forks =  node.fork_invalid
+      forks = node.fork_invalid
 
       node = forks.first
 
@@ -120,9 +119,9 @@ def speak
       expect(node.diagnose).to eq(:invalid_inside_split_pair)
       node = node.split_leaning
       expect(node.to_s).to eq(<<~'EOM')
-        puts (
-      else
-        puts }
+          puts (
+        else
+          puts }
       EOM
 
       expect(node.diagnose).to eq(:remove_pseudo_pair)