rubocop 0.93.1 → 1.79.2

data/lib/rubocop/config_loader_resolver.rb

@@ -1,29 +1,47 @@
 # frozen_string_literal: true
 
-require 'yaml'
 require 'pathname'
+require 'yaml'
+require_relative 'plugin'
 
 module RuboCop
   # A help class for ConfigLoader that handles configuration resolution.
   # @api private
-  class ConfigLoaderResolver
+  class ConfigLoaderResolver # rubocop:disable Metrics/ClassLength
+    def resolve_plugins(rubocop_config, plugins)
+      plugins = Array(plugins) - ConfigLoader.loaded_plugins.map { |plugin| plugin.about.name }
+      return if plugins.empty?
+
+      Plugin.integrate_plugins(rubocop_config, plugins)
+    end
+
     def resolve_requires(path, hash)
       config_dir = File.dirname(path)
-      Array(hash.delete('require')).each do |r|
-        if r.start_with?('.')
-          require(File.join(config_dir, r))
-        else
-          require(r)
+      hash.delete('require').tap do |loaded_features|
+        Array(loaded_features).each do |feature|
+          if Plugin.plugin_capable?(feature)
+            # NOTE: Compatibility for before plugins style.
+            warn Rainbow(<<~MESSAGE).yellow
+              #{feature} extension supports plugin, specify `plugins: #{feature}` instead of `require: #{feature}` in #{path}.
+              For more information, see https://docs.rubocop.org/rubocop/plugin_migration_guide.html.
+            MESSAGE
+            rubocop_config = Config.create(hash, path, check: false)
+
+            resolve_plugins(rubocop_config, feature)
+          else
+            FeatureLoader.load(config_directory_path: config_dir, feature: feature)
+          end
         end
       end
     end
 
-    # rubocop:disable Metrics/MethodLength
-    def resolve_inheritance(path, hash, file, debug)
+    def resolve_inheritance(path, hash, file, debug) # rubocop:disable Metrics/MethodLength, Metrics/AbcSize
       inherited_files = Array(hash['inherit_from'])
       base_configs(path, inherited_files, file)
-        .reverse.each_with_index do |base_config, index|
+        .each_with_index.reverse_each do |base_config, index|
         override_department_setting_for_cops(base_config, hash)
+        override_enabled_for_disabled_departments(base_config, hash)
+
         base_config.each do |k, v|
           next unless v.is_a?(Hash)
 
@@ -34,17 +52,30 @@ module RuboCop
                       inherit_mode: determine_inherit_mode(hash, k))
           end
           hash[k] = v
+          fix_include_paths(base_config.loaded_path, hash, path, k, v) if v.key?('Include')
         end
       end
     end
-    # rubocop:enable Metrics/MethodLength
+
+    # When one .rubocop.yml file inherits from another .rubocop.yml file, the Include paths in the
+    # base configuration are relative to the directory where the base configuration file is. For the
+    # derived configuration, we need to make those paths relative to where the derived configuration
+    # file is.
+    def fix_include_paths(base_config_path, hash, path, key, value)
+      return unless File.basename(base_config_path).start_with?('.rubocop')
+
+      base_dir = File.dirname(base_config_path)
+      derived_dir = File.dirname(path)
+      hash[key]['Include'] = value['Include'].map do |include_path|
+        PathUtil.relative_path(File.join(base_dir, include_path), derived_dir)
+      end
+    end
 
     def resolve_inheritance_from_gems(hash)
       gems = hash.delete('inherit_gem')
       (gems || {}).each_pair do |gem_name, config_path|
         if gem_name == 'rubocop'
-          raise ArgumentError,
-                "can't inherit configuration from the rubocop gem"
+          raise ArgumentError, "can't inherit configuration from the rubocop gem"
         end
 
         hash['inherit_from'] = Array(hash['inherit_from'])
@@ -58,7 +89,7 @@ module RuboCop
     # Merges the given configuration with the default one. If
     # AllCops:DisabledByDefault is true, it changes the Enabled params so that
     # only cops from user configuration are enabled. If
-    # AllCops::EnabledByDefault is true, it changes the Enabled params so that
+    # AllCops:EnabledByDefault is true, it changes the Enabled params so that
     # only cops explicitly disabled in user configuration are disabled.
     def merge_with_default(config, config_file, unset_nil:)
       default_configuration = ConfigLoader.default_configuration
@@ -73,9 +104,9 @@ module RuboCop
       end
 
       config = handle_disabled_by_default(config, default_configuration) if disabled_by_default
+      override_enabled_for_disabled_departments(default_configuration, config)
 
-      opts = { inherit_mode: config['inherit_mode'] || {},
-               unset_nil: unset_nil }
+      opts = { inherit_mode: config['inherit_mode'] || {}, unset_nil: unset_nil }
       Config.new(merge(default_configuration, config, **opts), config_file)
     end
 
@@ -90,10 +121,10 @@ module RuboCop
       keys_appearing_in_both.each do |key|
         if opts[:unset_nil] && derived_hash[key].nil?
           result.delete(key)
-        elsif base_hash[key].is_a?(Hash)
+        elsif merge_hashes?(base_hash, derived_hash, key)
           result[key] = merge(base_hash[key], derived_hash[key], **opts)
-        elsif should_union?(base_hash, key, opts[:inherit_mode])
-          result[key] = base_hash[key] | derived_hash[key]
+        elsif should_union?(derived_hash, base_hash, opts[:inherit_mode], key)
+          result[key] = Array(base_hash[key]) | Array(derived_hash[key])
         elsif opts[:debug]
           warn_on_duplicate_setting(base_hash, derived_hash, key, **opts)
         end
@@ -109,8 +140,7 @@ module RuboCop
         next unless key =~ %r{(.*)/.*}
 
         department = Regexp.last_match(1)
-        next unless disabled?(derived_hash, department) ||
-                    disabled?(base_hash, department)
+        next unless disabled?(derived_hash, department) || disabled?(base_hash, department)
 
         # The `override_department` setting for the `Enabled` parameter is an
         # internal setting that's not documented in the manual. It will cause a
@@ -120,50 +150,93 @@ module RuboCop
       end
     end
 
+    # If a cop was previously explicitly enabled, but then superseded by the
+    # department being disabled, disable it.
+    def override_enabled_for_disabled_departments(base_hash, derived_hash)
+      cops_to_disable = derived_hash.each_key.with_object([]) do |key, cops|
+        next unless disabled?(derived_hash, key)
+
+        cops.concat(base_hash.keys.grep(Regexp.new("^#{key}/")))
+      end
+
+      cops_to_disable.each do |cop_name|
+        next unless base_hash.dig(cop_name, 'Enabled') == true
+
+        derived_hash.replace(merge({ cop_name => { 'Enabled' => false } }, derived_hash))
+      end
+    end
+
     private
 
     def disabled?(hash, department)
-      hash[department] && hash[department]['Enabled'] == false
+      hash[department].is_a?(Hash) && hash[department]['Enabled'] == false
     end
 
     def duplicate_setting?(base_hash, derived_hash, key, inherited_file)
       return false if inherited_file.nil? # Not inheritance resolving merge
       return false if inherited_file.start_with?('..') # Legitimate override
       return false if base_hash[key] == derived_hash[key] # Same value
-      return false if remote_file?(inherited_file) # Can't change
+      return false if PathUtil.remote_file?(inherited_file) # Can't change
 
       Gem.path.none? { |dir| inherited_file.start_with?(dir) } # Can change?
     end
 
     def warn_on_duplicate_setting(base_hash, derived_hash, key, **opts)
-      return unless duplicate_setting?(base_hash, derived_hash,
-                                       key, opts[:inherited_file])
+      # If the file being considered is remote, don't bother checking for duplicates
+      return if remote_config?(opts[:file])
+
+      return unless duplicate_setting?(base_hash, derived_hash, key, opts[:inherited_file])
 
-      inherit_mode = opts[:inherit_mode]['merge'] ||
-                     opts[:inherit_mode]['override']
-      return if base_hash[key].is_a?(Array) &&
-                inherit_mode && inherit_mode.include?(key)
+      inherit_mode = opts[:inherit_mode]['merge'] || opts[:inherit_mode]['override']
+      return if base_hash[key].is_a?(Array) && inherit_mode&.include?(key)
+
+      puts duplicate_setting_warning(opts, key)
+    end
 
-      puts "#{PathUtil.smart_path(opts[:file])}: " \
-           "#{opts[:cop_name]}:#{key} overrides " \
-           "the same parameter in #{opts[:inherited_file]}"
+    def duplicate_setting_warning(opts, key)
+      "#{PathUtil.smart_path(opts[:file])}: " \
+        "#{opts[:cop_name]}:#{key} overrides " \
+        "the same parameter in #{opts[:inherited_file]}"
     end
 
     def determine_inherit_mode(hash, key)
       cop_cfg = hash[key]
-      local_inherit = cop_cfg.delete('inherit_mode') if cop_cfg.is_a?(Hash)
+      local_inherit = cop_cfg['inherit_mode'] if cop_cfg.is_a?(Hash)
       local_inherit || hash['inherit_mode'] || {}
     end
 
-    def should_union?(base_hash, key, inherit_mode)
-      base_hash[key].is_a?(Array) &&
-        inherit_mode &&
-        inherit_mode['merge'] &&
-        inherit_mode['merge'].include?(key)
+    def should_union?(derived_hash, base_hash, root_mode, key)
+      return false unless base_hash[key].is_a?(Array) || derived_hash[key].is_a?(Array)
+
+      derived_mode = derived_hash['inherit_mode']
+      return false if should_override?(derived_mode, key)
+      return true if should_merge?(derived_mode, key)
+
+      base_mode = base_hash['inherit_mode']
+      return false if should_override?(base_mode, key)
+      return true if should_merge?(base_mode, key)
+
+      should_merge?(root_mode, key)
+    end
+
+    def should_merge?(mode, key)
+      mode && mode['merge']&.include?(key)
+    end
+
+    def should_override?(mode, key)
+      mode && mode['override']&.include?(key)
+    end
+
+    def merge_hashes?(base_hash, derived_hash, key)
+      base_hash[key].is_a?(Hash) && derived_hash[key].is_a?(Hash)
     end
 
     def base_configs(path, inherit_from, file)
-      configs = Array(inherit_from).compact.map do |f|
+      inherit_froms = Array(inherit_from).compact.flat_map do |f|
+        PathUtil.glob?(f) ? Dir.glob(f) : f
+      end
+
+      configs = inherit_froms.map do |f|
         ConfigLoader.load_file(inherited_file(path, f, file))
       end
 
@@ -171,7 +244,7 @@ module RuboCop
     end
 
     def inherited_file(path, inherit_from, file)
-      if remote_file?(inherit_from)
+      if PathUtil.remote_file?(inherit_from)
         # A remote configuration, e.g. `inherit_from: http://example.com/rubocop.yml`.
         RemoteConfig.new(inherit_from, File.dirname(path))
       elsif Pathname.new(inherit_from).absolute?
@@ -191,9 +264,8 @@ module RuboCop
       end
     end
 
-    def remote_file?(uri)
-      regex = URI::DEFAULT_PARSER.make_regexp(%w[http https])
-      /\A#{regex}\z/.match?(uri)
+    def remote_config?(file)
+      file.is_a?(RemoteConfig)
     end
 
     def handle_disabled_by_default(config, new_default_configuration)
@@ -220,16 +292,21 @@ module RuboCop
 
     def gem_config_path(gem_name, relative_config_path)
       if defined?(Bundler)
-        gem = Bundler.load.specs[gem_name].first
-        gem_path = gem.full_gem_path if gem
+        begin
+          gem = Bundler.load.specs[gem_name].first
+          gem_path = gem.full_gem_path if gem
+        rescue Bundler::GemfileNotFound
+          # No Gemfile found. Bundler may be loaded manually
+        rescue Bundler::GitError
+          # The Gemfile exists but contains an uninstalled git source
+        end
       end
 
       gem_path ||= Gem::Specification.find_by_name(gem_name).gem_dir
 
       File.join(gem_path, relative_config_path)
     rescue Gem::LoadError => e
-      raise Gem::LoadError,
-            "Unable to find gem #{gem_name}; is the gem installed? #{e}"
+      raise Gem::LoadError, "Unable to find gem #{gem_name}; is the gem installed? #{e}"
     end
   end
 end

data/lib/rubocop/config_obsoletion/changed_enforced_styles.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for changing a parameter
+    # @api private
+    class ChangedEnforcedStyles < ParameterRule
+      BASE_MESSAGE = 'obsolete `%<parameter>s: %<value>s` (for `%<cop>s`) found in %<path>s'
+
+      def violated?
+        super && config[cop][parameter] == value
+      end
+
+      def message
+        base = format(BASE_MESSAGE,
+                      parameter: parameter, value: value, cop: cop, path: smart_loaded_path)
+
+        if alternative
+          "#{base}\n`#{parameter}: #{value}` has been renamed to " \
+            "`#{parameter}: #{alternative.chomp}`."
+        else
+          "#{base}\n#{reason.chomp}"
+        end
+      end
+
+      private
+
+      def value
+        metadata['value']
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/changed_parameter.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for changing a parameter
+    # @api private
+    class ChangedParameter < ParameterRule
+      BASE_MESSAGE = 'obsolete parameter `%<parameter>s` (for `%<cop>s`) found in %<path>s'
+
+      def message
+        base = format(BASE_MESSAGE, parameter: parameter, cop: cop, path: smart_loaded_path)
+
+        if alternative
+          "#{base}\n`#{parameter}` has been renamed to `#{alternative.chomp}`."
+        elsif alternatives
+          "#{base}\n`#{parameter}` has been renamed to #{to_sentence(alternatives.map do |item|
+                                                                       "`#{item}`"
+                                                                     end,
+                                                                     connector: 'and/or')}."
+        else
+          "#{base}\n#{reason.chomp}"
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/cop_rule.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Base class for ConfigObsoletion rules relating to cops
+    # @api private
+    class CopRule < Rule
+      attr_reader :old_name
+
+      def initialize(config, old_name)
+        super(config)
+        @old_name = old_name
+      end
+
+      def cop_rule?
+        true
+      end
+
+      def message
+        rule_message + "\n(obsolete configuration found in #{smart_loaded_path}, please update it)"
+      end
+
+      # Cop rules currently can only be failures, not warnings
+      def warning?
+        false
+      end
+
+      def violated?
+        config.key?(old_name) || config.key?(Cop::Badge.parse(old_name).cop_name)
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/extracted_cop.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for splitting a cop's
+    # functionality into multiple new cops.
+    # @api private
+    class ExtractedCop < CopRule
+      attr_reader :gem, :department
+
+      def initialize(config, old_name, gem)
+        super(config, old_name)
+        @department, * = old_name.rpartition('/')
+        @gem = gem
+      end
+
+      def violated?
+        return false if plugin_loaded?
+
+        affected_cops.any?
+      end
+
+      def rule_message
+        msg = '%<name>s been extracted to the `%<gem>s` gem.'
+        format(msg,
+               name: affected_cops.size > 1 ? "`#{department}` cops have" : "`#{old_name}` has",
+               gem: gem)
+      end
+
+      private
+
+      def affected_cops
+        return old_name unless old_name.end_with?('*')
+
+        # Handle whole departments (expressed as `Department/*`)
+        config.keys.select do |key|
+          key == department || key.start_with?("#{department}/")
+        end
+      end
+
+      def plugin_loaded?
+        # Plugins loaded via `require` are included in `loaded_features`.
+        config.loaded_plugins.include?(gem) || config.loaded_features.include?(gem)
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/parameter_rule.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Base class for ConfigObsoletion rules relating to parameters
+    # @api private
+    class ParameterRule < Rule
+      attr_reader :cop, :parameter, :metadata
+
+      def initialize(config, cop, parameter, metadata)
+        super(config)
+        @cop = cop
+        @parameter = parameter
+        @metadata = metadata
+      end
+
+      def parameter_rule?
+        true
+      end
+
+      def violated?
+        applies_to_current_ruby_version? && config[cop]&.key?(parameter)
+      end
+
+      def warning?
+        severity == 'warning'
+      end
+
+      private
+
+      def applies_to_current_ruby_version?
+        minimum_ruby_version = metadata['minimum_ruby_version']
+
+        return true unless minimum_ruby_version
+
+        config.target_ruby_version >= minimum_ruby_version
+      end
+
+      def alternative
+        metadata['alternative']
+      end
+
+      def alternatives
+        metadata['alternatives']
+      end
+
+      def reason
+        metadata['reason']
+      end
+
+      def severity
+        metadata['severity']
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/removed_cop.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for removing
+    # a previously defined cop.
+    # @api private
+    class RemovedCop < CopRule
+      attr_reader :old_name, :metadata
+
+      BASE_MESSAGE = 'The `%<old_name>s` cop has been removed'
+
+      def initialize(config, old_name, metadata)
+        super(config, old_name)
+        @metadata = metadata.is_a?(Hash) ? metadata : {}
+      end
+
+      def rule_message
+        base = format(BASE_MESSAGE, old_name: old_name)
+
+        if reason
+          "#{base} since #{reason.chomp}."
+        elsif alternatives
+          "#{base}. Please use #{to_sentence(alternatives, connector: 'and/or')} instead."
+        else
+          "#{base}."
+        end
+      end
+
+      private
+
+      def reason
+        metadata['reason']
+      end
+
+      def alternatives
+        Array(metadata['alternatives']).map { |name| "`#{name}`" }
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/renamed_cop.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for renaming
+    # a cop or moving it to a new department.
+    # @api private
+    class RenamedCop < CopRule
+      attr_reader :new_name, :metadata
+
+      def initialize(config, old_name, name_or_hash)
+        super(config, old_name)
+
+        if name_or_hash.is_a?(Hash)
+          @metadata = name_or_hash
+          @new_name = name_or_hash['new_name']
+        else
+          @metadata = {}
+          @new_name = name_or_hash
+        end
+      end
+
+      def rule_message
+        "The `#{old_name}` cop has been #{verb} to `#{new_name}`."
+      end
+
+      def warning?
+        severity == 'warning'
+      end
+
+      private
+
+      def moved?
+        old_badge = Cop::Badge.parse(old_name)
+        new_badge = Cop::Badge.parse(new_name)
+
+        old_badge.department != new_badge.department && old_badge.cop_name == new_badge.cop_name
+      end
+
+      def verb
+        moved? ? 'moved' : 'renamed'
+      end
+
+      def severity
+        metadata['severity']
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/rule.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Abstract base class for ConfigObsoletion rules
+    # @api private
+    class Rule
+      def initialize(config)
+        @config = config
+      end
+
+      # Does this rule relate to cops?
+      def cop_rule?
+        false
+      end
+
+      # Does this rule relate to parameters?
+      def parameter_rule?
+        false
+      end
+
+      def violated?
+        raise NotImplementedError
+      end
+
+      private
+
+      attr_reader :config
+
+      def to_sentence(collection, connector: 'and')
+        return collection.first if collection.size == 1
+
+        [collection[0..-2].join(', '), collection[-1]].join(" #{connector} ")
+      end
+
+      def smart_loaded_path
+        PathUtil.smart_path(config.loaded_path)
+      end
+    end
+  end
+end

data/lib/rubocop/config_obsoletion/split_cop.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module RuboCop
+  class ConfigObsoletion
+    # Encapsulation of a ConfigObsoletion rule for splitting a cop's
+    # functionality into multiple new cops.
+    # @api private
+    class SplitCop < CopRule
+      attr_reader :metadata
+
+      def initialize(config, old_name, metadata)
+        super(config, old_name)
+        @metadata = metadata
+      end
+
+      def rule_message
+        "The `#{old_name}` cop has been split into #{to_sentence(alternatives)}."
+      end
+
+      private
+
+      def alternatives
+        Array(metadata['alternatives']).map { |name| "`#{name}`" }
+      end
+    end
+  end
+end