fablicop 1.10.2 → 1.10.3

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/next.rb

@@ -0,0 +1,233 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Use `next` to skip iteration instead of a condition at the end.
+      #
+      # @example EnforcedStyle: skip_modifier_ifs (default)
+      #   # bad
+      #   [1, 2].each do |a|
+      #     if a == 1
+      #       puts a
+      #     end
+      #   end
+      #
+      #   # good
+      #   [1, 2].each do |a|
+      #     next unless a == 1
+      #     puts a
+      #   end
+      #
+      #   # good
+      #   [1, 2].each do |a|
+      #     puts a if a == 1
+      #   end
+      #
+      # @example EnforcedStyle: always
+      #   # With `always` all conditions at the end of an iteration needs to be
+      #   # replaced by next - with `skip_modifier_ifs` the modifier if like
+      #   # this one are ignored: `[1, 2].each { |a| puts a if a == 1 }`
+      #
+      #   # bad
+      #   [1, 2].each do |a|
+      #     puts a if a == 1
+      #   end
+      #
+      #   # bad
+      #   [1, 2].each do |a|
+      #     if a == 1
+      #       puts a
+      #     end
+      #   end
+      #
+      #   # good
+      #   [1, 2].each do |a|
+      #     next unless a == 1
+      #     puts a
+      #   end
+      class Next < Base
+        include ConfigurableEnforcedStyle
+        include MinBodyLength
+        include RangeHelp
+        extend AutoCorrector
+
+        MSG = 'Use `next` to skip iteration.'
+        EXIT_TYPES = %i[break return].freeze
+
+        def self.autocorrect_incompatible_with
+          [Style::SafeNavigation]
+        end
+
+        def on_new_investigation
+          # When correcting nested offenses, we need to keep track of how much
+          # we have adjusted the indentation of each line
+          @reindented_lines = Hash.new(0)
+        end
+
+        def on_block(node)
+          return unless node.send_node.send_type? && node.send_node.enumerator_method?
+
+          check(node)
+        end
+
+        alias on_numblock on_block
+
+        def on_while(node)
+          check(node)
+        end
+        alias on_until on_while
+        alias on_for on_while
+
+        private
+
+        def check(node)
+          return unless node.body && ends_with_condition?(node.body)
+
+          offending_node = offense_node(node.body)
+
+          add_offense(offense_location(offending_node)) do |corrector|
+            if offending_node.modifier_form?
+              autocorrect_modifier(corrector, offending_node)
+            else
+              autocorrect_block(corrector, offending_node)
+            end
+          end
+        end
+
+        def ends_with_condition?(body)
+          return true if simple_if_without_break?(body)
+
+          body.begin_type? && simple_if_without_break?(body.children.last)
+        end
+
+        def simple_if_without_break?(node)
+          return false unless if_without_else?(node)
+          return false if if_else_children?(node)
+          return false if allowed_modifier_if?(node)
+
+          !exit_body_type?(node)
+        end
+
+        def allowed_modifier_if?(node)
+          if node.modifier_form?
+            style == :skip_modifier_ifs
+          else
+            !min_body_length?(node)
+          end
+        end
+
+        def if_else_children?(node)
+          node.each_child_node(:if).any?(&:else?)
+        end
+
+        def if_without_else?(node)
+          node&.if_type? && !node.ternary? && !node.else?
+        end
+
+        def exit_body_type?(node)
+          return false unless node.if_branch
+
+          EXIT_TYPES.include?(node.if_branch.type)
+        end
+
+        def offense_node(body)
+          *_, condition = *body
+
+          condition&.if_type? ? condition : body
+        end
+
+        def offense_location(offense_node)
+          offense_begin_pos = offense_node.source_range.begin
+          offense_begin_pos.join(offense_node.condition.source_range)
+        end
+
+        def autocorrect_modifier(corrector, node)
+          body = node.if_branch || node.else_branch
+
+          replacement =
+            "next #{node.inverse_keyword} #{node.condition.source}\n" \
+            "#{' ' * node.source_range.column}#{body.source}"
+
+          corrector.replace(node, replacement)
+        end
+
+        def autocorrect_block(corrector, node)
+          next_code = "next #{node.inverse_keyword} #{node.condition.source}"
+
+          corrector.insert_before(node, next_code)
+
+          corrector.remove(cond_range(node, node.condition))
+          corrector.remove(end_range(node))
+
+          lines = reindentable_lines(node)
+
+          return if lines.empty?
+
+          reindent(lines, node.condition, corrector)
+        end
+
+        def cond_range(node, cond)
+          end_pos = if node.loc.begin
+                      node.loc.begin.end_pos # after "then"
+                    else
+                      cond.source_range.end_pos
+                    end
+
+          range_between(node.source_range.begin_pos, end_pos)
+        end
+
+        def end_range(node)
+          source_buffer = node.source_range.source_buffer
+          end_pos = node.loc.end.end_pos
+          begin_pos = node.loc.end.begin_pos - node.loc.end.column
+          begin_pos -= 1 if end_followed_by_whitespace_only?(source_buffer, end_pos)
+
+          range_between(begin_pos, end_pos)
+        end
+
+        def end_followed_by_whitespace_only?(source_buffer, end_pos)
+          /\A\s*$/.match?(source_buffer.source[end_pos..])
+        end
+
+        def reindentable_lines(node)
+          buffer = node.source_range.source_buffer
+
+          # end_range starts with the final newline of the if body
+          lines = (node.source_range.line + 1)...node.loc.end.line
+          lines = lines.to_a - heredoc_lines(node)
+          # Skip blank lines
+          lines.reject { |lineno| /\A\s*\z/.match?(buffer.source_line(lineno)) }
+        end
+
+        # Adjust indentation of `lines` to match `node`
+        def reindent(lines, node, corrector)
+          range  = node.source_range
+          buffer = range.source_buffer
+
+          target_indent = range.source_line =~ /\S/
+          delta = actual_indent(lines, buffer) - target_indent
+          lines.each { |lineno| reindent_line(corrector, lineno, delta, buffer) }
+        end
+
+        def actual_indent(lines, buffer)
+          lines.map { |lineno| buffer.source_line(lineno) =~ /\S/ }.min
+        end
+
+        def heredoc_lines(node)
+          node.each_node(:dstr)
+              .select(&:heredoc?)
+              .map { |n| n.loc.heredoc_body }
+              .flat_map { |b| (b.line...b.last_line).to_a }
+        end
+
+        def reindent_line(corrector, lineno, delta, buffer)
+          adjustment = delta + @reindented_lines[lineno]
+          @reindented_lines[lineno] = adjustment
+
+          corrector.remove_leading(buffer.line_range(lineno), adjustment) if adjustment.positive?
+        end
+      end
+    end
+  end
+end

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/nil_comparison.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Checks for comparison of something with nil using `==` and
+      # `nil?`.
+      #
+      # Supported styles are: predicate, comparison.
+      #
+      # @example EnforcedStyle: predicate (default)
+      #
+      #   # bad
+      #   if x == nil
+      #   end
+      #
+      #   # good
+      #   if x.nil?
+      #   end
+      #
+      # @example EnforcedStyle: comparison
+      #
+      #   # bad
+      #   if x.nil?
+      #   end
+      #
+      #   # good
+      #   if x == nil
+      #   end
+      #
+      class NilComparison < Base
+        include ConfigurableEnforcedStyle
+        extend AutoCorrector
+
+        PREDICATE_MSG = 'Prefer the use of the `nil?` predicate.'
+        EXPLICIT_MSG = 'Prefer the use of the `==` comparison.'
+
+        RESTRICT_ON_SEND = %i[== === nil?].freeze
+
+        # @!method nil_comparison?(node)
+        def_node_matcher :nil_comparison?, '(send _ {:== :===} nil)'
+
+        # @!method nil_check?(node)
+        def_node_matcher :nil_check?, '(send _ :nil?)'
+
+        def on_send(node)
+          style_check?(node) do
+            add_offense(node.loc.selector) do |corrector|
+              new_code = if prefer_comparison?
+                           node.source.sub('.nil?', ' == nil')
+                         else
+                           node.source.sub(/\s*={2,3}\s*nil/, '.nil?')
+                         end
+
+              corrector.replace(node, new_code)
+
+              parent = node.parent
+              corrector.wrap(node, '(', ')') if parent.respond_to?(:method?) && parent.method?(:!)
+            end
+          end
+        end
+
+        private
+
+        def message(_node)
+          prefer_comparison? ? EXPLICIT_MSG : PREDICATE_MSG
+        end
+
+        def style_check?(node, &block)
+          if prefer_comparison?
+            nil_check?(node, &block)
+          else
+            nil_comparison?(node, &block)
+          end
+        end
+
+        def prefer_comparison?
+          style == :comparison
+        end
+      end
+    end
+  end
+end

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/nil_lambda.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Checks for lambdas and procs that always return nil,
+      # which can be replaced with an empty lambda or proc instead.
+      #
+      # @example
+      #   # bad
+      #   -> { nil }
+      #
+      #   lambda do
+      #     next nil
+      #   end
+      #
+      #   proc { nil }
+      #
+      #   Proc.new do
+      #     break nil
+      #   end
+      #
+      #   # good
+      #   -> {}
+      #
+      #   lambda do
+      #   end
+      #
+      #   -> (x) { nil if x }
+      #
+      #   proc {}
+      #
+      #   Proc.new { nil if x }
+      #
+      class NilLambda < Base
+        extend AutoCorrector
+        include RangeHelp
+
+        MSG = 'Use an empty %<type>s instead of always returning nil.'
+
+        # @!method nil_return?(node)
+        def_node_matcher :nil_return?, <<~PATTERN
+          { ({return next break} nil) (nil) }
+        PATTERN
+
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
+          return unless node.lambda_or_proc?
+          return unless nil_return?(node.body)
+
+          message = format(MSG, type: node.lambda? ? 'lambda' : 'proc')
+          add_offense(node, message: message) do |corrector|
+            autocorrect(corrector, node)
+          end
+        end
+
+        private
+
+        def autocorrect(corrector, node)
+          range = if node.single_line?
+                    range_with_surrounding_space(node.body.source_range)
+                  else
+                    range_by_whole_lines(node.body.source_range, include_final_newline: true)
+                  end
+
+          corrector.remove(range)
+        end
+      end
+    end
+  end
+end

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/non_nil_check.rb

@@ -0,0 +1,158 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Checks for non-nil checks, which are usually redundant.
+      #
+      # With `IncludeSemanticChanges` set to `false` by default, this cop
+      # does not report offenses for `!x.nil?` and does no changes that might
+      # change behavior.
+      # Also `IncludeSemanticChanges` set to `false` with `EnforcedStyle: comparison` of
+      # `Style/NilComparison` cop, this cop does not report offenses for `x != nil` and
+      # does no changes to `!x.nil?` style.
+      #
+      # With `IncludeSemanticChanges` set to `true`, this cop reports offenses
+      # for `!x.nil?` and autocorrects that and `x != nil` to solely `x`, which
+      # is *usually* OK, but might change behavior.
+      #
+      # @example
+      #   # bad
+      #   if x != nil
+      #   end
+      #
+      #   # good
+      #   if x
+      #   end
+      #
+      #   # Non-nil checks are allowed if they are the final nodes of predicate.
+      #   # good
+      #   def signed_in?
+      #     !current_user.nil?
+      #   end
+      #
+      # @example IncludeSemanticChanges: false (default)
+      #   # good
+      #   if !x.nil?
+      #   end
+      #
+      # @example IncludeSemanticChanges: true
+      #   # bad
+      #   if !x.nil?
+      #   end
+      #
+      class NonNilCheck < Base
+        extend AutoCorrector
+
+        MSG_FOR_REPLACEMENT = 'Prefer `%<prefer>s` over `%<current>s`.'
+        MSG_FOR_REDUNDANCY = 'Explicit non-nil checks are usually redundant.'
+
+        RESTRICT_ON_SEND = %i[!= nil? !].freeze
+
+        # @!method not_equal_to_nil?(node)
+        def_node_matcher :not_equal_to_nil?, '(send _ :!= nil)'
+
+        # @!method unless_check?(node)
+        def_node_matcher :unless_check?, '(if (send _ :nil?) ...)'
+
+        # @!method nil_check?(node)
+        def_node_matcher :nil_check?, '(send _ :nil?)'
+
+        # @!method not_and_nil_check?(node)
+        def_node_matcher :not_and_nil_check?, '(send (send _ :nil?) :!)'
+
+        def on_send(node)
+          return if ignored_node?(node) ||
+                    (!include_semantic_changes? && nil_comparison_style == 'comparison')
+          return unless register_offense?(node)
+
+          message = message(node)
+          add_offense(node, message: message) { |corrector| autocorrect(corrector, node) }
+        end
+
+        def on_def(node)
+          body = node.body
+
+          return unless node.predicate_method? && body
+
+          if body.begin_type?
+            ignore_node(body.children.last)
+          else
+            ignore_node(body)
+          end
+        end
+        alias on_defs on_def
+
+        private
+
+        def register_offense?(node)
+          not_equal_to_nil?(node) ||
+            (include_semantic_changes? && (not_and_nil_check?(node) || unless_and_nil_check?(node)))
+        end
+
+        def autocorrect(corrector, node)
+          case node.method_name
+          when :!=
+            autocorrect_comparison(corrector, node)
+          when :!
+            autocorrect_non_nil(corrector, node, node.receiver)
+          when :nil?
+            autocorrect_unless_nil(corrector, node, node.receiver)
+          end
+        end
+
+        def unless_and_nil_check?(send_node)
+          parent = send_node.parent
+
+          nil_check?(send_node) && unless_check?(parent) && !parent.ternary? && parent.unless?
+        end
+
+        def message(node)
+          if node.method?(:!=) && !include_semantic_changes?
+            prefer = "!#{node.receiver.source}.nil?"
+            format(MSG_FOR_REPLACEMENT, prefer: prefer, current: node.source)
+          else
+            MSG_FOR_REDUNDANCY
+          end
+        end
+
+        def include_semantic_changes?
+          cop_config['IncludeSemanticChanges']
+        end
+
+        def autocorrect_comparison(corrector, node)
+          expr = node.source
+
+          new_code = if include_semantic_changes?
+                       expr.sub(/\s*!=\s*nil/, '')
+                     else
+                       expr.sub(/^(\S*)\s*!=\s*nil/, '!\1.nil?')
+                     end
+
+          return if expr == new_code
+
+          corrector.replace(node, new_code)
+        end
+
+        def autocorrect_non_nil(corrector, node, inner_node)
+          if inner_node.receiver
+            corrector.replace(node, inner_node.receiver.source)
+          else
+            corrector.replace(node, 'self')
+          end
+        end
+
+        def autocorrect_unless_nil(corrector, node, receiver)
+          corrector.replace(node.parent.loc.keyword, 'if')
+          corrector.replace(node, receiver.source)
+        end
+
+        def nil_comparison_style
+          nil_comparison_conf = config.for_cop('Style/NilComparison')
+
+          nil_comparison_conf['Enabled'] && nil_comparison_conf['EnforcedStyle']
+        end
+      end
+    end
+  end
+end

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/not.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Checks for uses of the keyword `not` instead of `!`.
+      #
+      # @example
+      #
+      #   # bad - parentheses are required because of op precedence
+      #   x = (not something)
+      #
+      #   # good
+      #   x = !something
+      #
+      class Not < Base
+        include RangeHelp
+        extend AutoCorrector
+
+        MSG = 'Use `!` instead of `not`.'
+        RESTRICT_ON_SEND = %i[!].freeze
+
+        OPPOSITE_METHODS = {
+          :== => :!=,
+          :!= => :==,
+          :<= => :>,
+          :> => :<=,
+          :< => :>=,
+          :>= => :<
+        }.freeze
+
+        def on_send(node)
+          return unless node.prefix_not?
+
+          add_offense(node.loc.selector) do |corrector|
+            range = range_with_surrounding_space(node.loc.selector, side: :right)
+
+            if opposite_method?(node.receiver)
+              correct_opposite_method(corrector, range, node.receiver)
+            elsif requires_parens?(node.receiver)
+              correct_with_parens(corrector, range, node)
+            else
+              correct_without_parens(corrector, range)
+            end
+          end
+        end
+
+        private
+
+        def opposite_method?(child)
+          child.send_type? && OPPOSITE_METHODS.key?(child.method_name)
+        end
+
+        def requires_parens?(child)
+          child.and_type? || child.or_type? ||
+            (child.send_type? && child.binary_operation?) ||
+            (child.if_type? && child.ternary?)
+        end
+
+        def correct_opposite_method(corrector, range, child)
+          corrector.remove(range)
+          corrector.replace(child.loc.selector, OPPOSITE_METHODS[child.method_name].to_s)
+        end
+
+        def correct_with_parens(corrector, range, node)
+          corrector.replace(range, '!(')
+          corrector.insert_after(node, ')')
+        end
+
+        def correct_without_parens(corrector, range)
+          corrector.replace(range, '!')
+        end
+      end
+    end
+  end
+end

data/vendor/bundle/ruby/3.2.0/gems/rubocop-1.54.1/lib/rubocop/cop/style/numbered_parameters.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Style
+      # Checks for numbered parameters.
+      #
+      # It can either restrict the use of numbered parameters to
+      # single-lined blocks, or disallow completely numbered parameters.
+      #
+      # @example EnforcedStyle: allow_single_line (default)
+      #   # bad
+      #   collection.each do
+      #     puts _1
+      #   end
+      #
+      #   # good
+      #   collection.each { puts _1 }
+      #
+      # @example EnforcedStyle: disallow
+      #   # bad
+      #   collection.each { puts _1 }
+      #
+      #   # good
+      #   collection.each { |item| puts item }
+      #
+      class NumberedParameters < Base
+        include ConfigurableEnforcedStyle
+        extend TargetRubyVersion
+
+        MSG_DISALLOW = 'Avoid using numbered parameters.'
+        MSG_MULTI_LINE = 'Avoid using numbered parameters for multi-line blocks.'
+
+        minimum_target_ruby_version 2.7
+
+        def on_numblock(node)
+          if style == :disallow
+            add_offense(node, message: MSG_DISALLOW)
+          elsif node.multiline?
+            add_offense(node, message: MSG_MULTI_LINE)
+          end
+        end
+      end
+    end
+  end
+end