rbhint 0.8.5.rc1

data/lib/rubocop/cop/layout/argument_alignment.rb

@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Layout
+      # Here we check if the arguments on a multi-line method
+      # definition are aligned.
+      #
+      # @example EnforcedStyle: with_first_argument (default)
+      #   # good
+      #
+      #   foo :bar,
+      #       :baz
+      #
+      #   foo(
+      #     :bar,
+      #     :baz
+      #   )
+      #
+      #   # bad
+      #
+      #   foo :bar,
+      #     :baz
+      #
+      #   foo(
+      #     :bar,
+      #       :baz
+      #   )
+      #
+      # @example EnforcedStyle: with_fixed_indentation
+      #   # good
+      #
+      #   foo :bar,
+      #     :baz
+      #
+      #   # bad
+      #
+      #   foo :bar,
+      #       :baz
+      class ArgumentAlignment < Cop
+        include Alignment
+
+        ALIGN_PARAMS_MSG = 'Align the arguments of a method call if ' \
+          'they span more than one line.'
+
+        FIXED_INDENT_MSG = 'Use one level of indentation for arguments ' \
+          'following the first line of a multi-line method call.'
+
+        def on_send(node)
+          return if node.arguments.size < 2 ||
+                    node.send_type? && node.method?(:[]=)
+
+          check_alignment(node.arguments, base_column(node, node.arguments))
+        end
+        alias on_csend on_send
+
+        def autocorrect(node)
+          AlignmentCorrector.correct(processed_source, node, column_delta)
+        end
+
+        private
+
+        def message(_node)
+          fixed_indentation? ? FIXED_INDENT_MSG : ALIGN_PARAMS_MSG
+        end
+
+        def fixed_indentation?
+          cop_config['EnforcedStyle'] == 'with_fixed_indentation'
+        end
+
+        def base_column(node, args)
+          if fixed_indentation?
+            lineno = target_method_lineno(node)
+            line = node.source_range.source_buffer.source_line(lineno)
+            indentation_of_line = /\S.*/.match(line).begin(0)
+            indentation_of_line + configured_indentation_width
+          else
+            display_column(args.first.source_range)
+          end
+        end
+
+        def target_method_lineno(node)
+          if node.loc.selector
+            node.loc.selector.line
+          else
+            # l.(1) has no selector, so we use the opening parenthesis instead
+            node.loc.begin.line
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/layout/array_alignment.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Layout
+      # Here we check if the elements of a multi-line array literal are
+      # aligned.
+      #
+      # @example EnforcedStyle: with_first_element (default)
+      #   # good
+      #
+      #   array = [1, 2, 3,
+      #            4, 5, 6]
+      #   array = ['run',
+      #            'forrest',
+      #            'run']
+      #
+      #   # bad
+      #
+      #   array = [1, 2, 3,
+      #     4, 5, 6]
+      #   array = ['run',
+      #        'forrest',
+      #        'run']
+      #
+      # @example EnforcedStyle: with_fixed_indentation
+      #   # good
+      #
+      #   array = [1, 2, 3,
+      #     4, 5, 6]
+      #
+      #   # bad
+      #
+      #   array = [1, 2, 3,
+      #            4, 5, 6]
+      class ArrayAlignment < Cop
+        include Alignment
+
+        ALIGN_ELEMENTS_MSG = 'Align the elements of an array literal ' \
+          'if they span more than one line.'
+
+        FIXED_INDENT_MSG = 'Use one level of indentation for elements ' \
+          'following the first line of a multi-line array.'
+
+        def on_array(node)
+          return if node.children.size < 2
+
+          check_alignment(node.children, base_column(node, node.children))
+        end
+
+        def autocorrect(node)
+          AlignmentCorrector.correct(processed_source, node, column_delta)
+        end
+
+        private
+
+        def message(_node)
+          fixed_indentation? ? FIXED_INDENT_MSG : ALIGN_ELEMENTS_MSG
+        end
+
+        def fixed_indentation?
+          cop_config['EnforcedStyle'] == 'with_fixed_indentation'
+        end
+
+        def base_column(node, args)
+          if fixed_indentation?
+            lineno = target_method_lineno(node)
+            line = node.source_range.source_buffer.source_line(lineno)
+            indentation_of_line = /\S.*/.match(line).begin(0)
+            indentation_of_line + configured_indentation_width
+          else
+            display_column(args.first.source_range)
+          end
+        end
+
+        def target_method_lineno(node)
+          node.loc.line
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/layout/assignment_indentation.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Layout
+      # This cop checks the indentation of the first line of the
+      # right-hand-side of a multi-line assignment.
+      #
+      # @example
+      #   # bad
+      #   value =
+      #   if foo
+      #     'bar'
+      #   end
+      #
+      #   # good
+      #   value =
+      #     if foo
+      #       'bar'
+      #     end
+      #
+      # The indentation of the remaining lines can be corrected with
+      # other cops such as `IndentationConsistency` and `EndAlignment`.
+      class AssignmentIndentation < Cop
+        include CheckAssignment
+        include Alignment
+
+        MSG = 'Indent the first line of the right-hand-side of a ' \
+              'multi-line assignment.'
+
+        def check_assignment(node, rhs)
+          return unless rhs
+          return unless node.loc.operator
+          return if node.loc.operator.line == rhs.first_line
+
+          base = display_column(leftmost_multiple_assignment(node).source_range)
+          check_alignment([rhs], base + configured_indentation_width)
+        end
+
+        def autocorrect(node)
+          AlignmentCorrector.correct(processed_source, node, column_delta)
+        end
+
+        def leftmost_multiple_assignment(node)
+          return node unless same_line?(node, node.parent) &&
+                             node.parent.assignment?
+
+          leftmost_multiple_assignment(node.parent)
+
+          node.parent
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/layout/block_alignment.rb

@@ -0,0 +1,244 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Layout
+      # This cop checks whether the end keywords are aligned properly for do
+      # end blocks.
+      #
+      # Three modes are supported through the `EnforcedStyleAlignWith`
+      # configuration parameter:
+      #
+      # `start_of_block` : the `end` shall be aligned with the
+      # start of the line where the `do` appeared.
+      #
+      # `start_of_line` : the `end` shall be aligned with the
+      # start of the line where the expression started.
+      #
+      # `either` (which is the default) : the `end` is allowed to be in either
+      # location. The autofixer will default to `start_of_line`.
+      #
+      # @example EnforcedStyleAlignWith: either (default)
+      #   # bad
+      #
+      #   foo.bar
+      #      .each do
+      #        baz
+      #          end
+      #
+      #   # good
+      #
+      #   variable = lambda do |i|
+      #     i
+      #   end
+      #
+      # @example EnforcedStyleAlignWith: start_of_block
+      #   # bad
+      #
+      #   foo.bar
+      #      .each do
+      #        baz
+      #          end
+      #
+      #   # good
+      #
+      #   foo.bar
+      #     .each do
+      #       baz
+      #     end
+      #
+      # @example EnforcedStyleAlignWith: start_of_line
+      #   # bad
+      #
+      #   foo.bar
+      #      .each do
+      #        baz
+      #          end
+      #
+      #   # good
+      #
+      #   foo.bar
+      #     .each do
+      #        baz
+      #   end
+      class BlockAlignment < Cop
+        include ConfigurableEnforcedStyle
+        include RangeHelp
+
+        MSG = '%<current>s is not aligned with %<prefer>s%<alt_prefer>s.'
+
+        def_node_matcher :block_end_align_target?, <<~PATTERN
+          {assignment?
+           splat
+           and
+           or
+           (send _ :<<  ...)
+           (send equal?(%1) !:[] ...)}
+        PATTERN
+
+        def on_block(node)
+          check_block_alignment(start_for_block_node(node), node)
+        end
+
+        def style_parameter_name
+          'EnforcedStyleAlignWith'
+        end
+
+        def autocorrect(node)
+          ancestor_node = start_for_block_node(node)
+          start_col = compute_start_col(ancestor_node, node)
+          loc_end = node.loc.end
+          delta = start_col - loc_end.column
+
+          if delta.positive?
+            add_space_before(loc_end, delta)
+          elsif delta.negative?
+            remove_space_before(loc_end.begin_pos, -delta)
+          end
+        end
+
+        private
+
+        def start_for_block_node(block_node)
+          # Which node should we align the 'end' with?
+          result = block_end_align_target(block_node)
+
+          # In offense message, we want to show the assignment LHS rather than
+          # the entire assignment
+          result, = *result while result.op_asgn_type? || result.masgn_type?
+          result
+        end
+
+        def block_end_align_target(node)
+          lineage = [node, *node.ancestors]
+
+          target = lineage.each_cons(2) do |current, parent|
+            break current if end_align_target?(current, parent)
+          end
+
+          target || lineage.last
+        end
+
+        def end_align_target?(node, parent)
+          disqualified_parent?(parent, node) ||
+            !block_end_align_target?(parent, node)
+        end
+
+        def disqualified_parent?(parent, node)
+          parent&.loc && parent.first_line != node.first_line &&
+            !parent.masgn_type?
+        end
+
+        def check_block_alignment(start_node, block_node)
+          end_loc = block_node.loc.end
+          return unless begins_its_line?(end_loc)
+
+          start_loc = start_node.source_range
+          return unless start_loc.column != end_loc.column ||
+                        style == :start_of_block
+
+          do_source_line_column =
+            compute_do_source_line_column(block_node, end_loc)
+          return unless do_source_line_column
+
+          register_offense(
+            block_node,
+            start_loc,
+            end_loc,
+            do_source_line_column
+          )
+        end
+
+        def register_offense(block_node,
+                             start_loc,
+                             end_loc,
+                             do_source_line_column)
+
+          error_source_line_column = if style == :start_of_block
+                                       do_source_line_column
+                                     else
+                                       loc_to_source_line_column(start_loc)
+                                     end
+
+          message = format_message(start_loc, end_loc, do_source_line_column,
+                                   error_source_line_column)
+
+          add_offense(block_node, location: end_loc, message: message)
+        end
+
+        def format_message(start_loc, end_loc, do_source_line_column,
+                           error_source_line_column)
+          format(
+            MSG,
+            current: format_source_line_column(
+              loc_to_source_line_column(end_loc)
+            ),
+            prefer: format_source_line_column(error_source_line_column),
+            alt_prefer: alt_start_msg(start_loc, do_source_line_column)
+          )
+        end
+
+        def compute_do_source_line_column(node, end_loc)
+          do_loc = node.loc.begin # Actually it's either do or {.
+
+          # We've found that "end" is not aligned with the start node (which
+          # can be a block, a variable assignment, etc). But we also allow
+          # the "end" to be aligned with the start of the line where the "do"
+          # is, which is a style some people use in multi-line chains of
+          # blocks.
+          match = /\S.*/.match(do_loc.source_line)
+          indentation_of_do_line = match.begin(0)
+          return unless end_loc.column != indentation_of_do_line ||
+                        style == :start_of_line
+
+          {
+            source: match[0],
+            line: do_loc.line,
+            column: indentation_of_do_line
+          }
+        end
+
+        def loc_to_source_line_column(loc)
+          {
+            source: loc.source.lines.to_a.first.chomp,
+            line: loc.line,
+            column: loc.column
+          }
+        end
+
+        def alt_start_msg(start_loc, source_line_column)
+          if style != :either
+            ''
+          elsif start_loc.line == source_line_column[:line] &&
+                start_loc.column == source_line_column[:column]
+            ''
+          else
+            ' or ' + format_source_line_column(source_line_column)
+          end
+        end
+
+        def format_source_line_column(source_line_column)
+          "`#{source_line_column[:source]}` at #{source_line_column[:line]}, " \
+          "#{source_line_column[:column]}"
+        end
+
+        def compute_start_col(ancestor_node, node)
+          if style == :start_of_block
+            do_loc = node.loc.begin
+            return do_loc.source_line =~ /\S/
+          end
+          (ancestor_node || node).source_range.column
+        end
+
+        def add_space_before(loc, delta)
+          ->(corrector) { corrector.insert_before(loc, ' ' * delta) }
+        end
+
+        def remove_space_before(end_pos, delta)
+          range = range_between(end_pos - delta, end_pos)
+          ->(corrector) { corrector.remove(range) }
+        end
+      end
+    end
+  end
+end