pronto-rubocop 0.8.1 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 0f25431e077763d8bc08d97405887e0b2d9f8968
-  data.tar.gz: 3db87215b3a0ddb0dc2da910bfad3151e235bbb3
+SHA256:
+  metadata.gz: 9d5cd91a500a54c1184c0850685fd32ca72a832a82fd9c3a57b840d5b8eca6b2
+  data.tar.gz: 3b41e6777620b40b9d7e3d0d20946294c24acf8b081b32667e5adc69e2f03810
 SHA512:
-  metadata.gz: 8315b02d6e4fb04c4e63526a21654456b0c1123e7caf8a718da89a3e75dbc3f314b15c0195e9769946696e997d7838db5f810ed46531efa8dc42a5da6c8537c2
-  data.tar.gz: 0445514cbd4374be163a5499e9516494092bbcaa568a4002ab031622d59bea7de3bbd1f19333da96c8297037a127e6ff6a9e95160dacbec032c042a5923eb688
+  metadata.gz: 7e440e9d28f6ef2b46ba4b6556164dcf97298e7926a972f5997417275dc4b2611e434ecdca32ccf8dc4550daa2d3561ae66871404b1773a306b3d93a8eb90a90
+  data.tar.gz: 81f24db46df5d142535c545c3943514ebcb5b138f08171e17a5c8e72178ae014fc1d67b206a425e7e34d04fc6dad7f0839ba56b4a5ea9e3291434ebaf809e74f

data/.github/CODEOWNERS

@@ -0,0 +1,3 @@
+# Order is important. The last matching pattern takes the most precedence.
+# Default owners for everything in the repo.
+* @prontolabs/core

data/.github/workflows/checks.yml

@@ -0,0 +1,34 @@
+name: Checks
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  unit_tests:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ['2.4', '2.5', '2.6', '2.7', '3.0']
+        rubocop: ['0.63.1', '0.81.0', '0.84.0', '0.85.0', '0.86.0', '0.87.0', '< 1.0', '1.0.0', '< 2.0']
+        include:
+          - ruby: '2.3'
+            rubocop: '0.63.1'
+          - ruby: '2.3'
+            rubocop: '0.81.0'
+      fail-fast: false
+    steps:
+    - uses: actions/checkout@v2
+    - name: use specific rubocop version
+      run: echo "gem 'rubocop', '${{ matrix.rubocop }}'" > Gemfile.local
+    - name: use specific rubocop-ast version (if required)
+      if: matrix.rubocop == '0.84.0' || matrix.rubocop == '0.85.0'
+      run: echo "gem 'rubocop-ast', '< 0.7.0'" >> Gemfile.local
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+    - name: rake spec
+      run: bundle exec rake spec

data/README.md

@@ -1,13 +1,54 @@
-# Pronto runner for Rubocop
+# Pronto runner for RuboCop
 
-[![Code Climate](https://codeclimate.com/github/mmozuras/pronto-rubocop.png)](https://codeclimate.com/github/mmozuras/pronto-rubocop)
-[![Build Status](https://travis-ci.org/mmozuras/pronto-rubocop.png)](https://travis-ci.org/mmozuras/pronto-rubocop)
+[![Code Climate](https://codeclimate.com/github/prontolabs/pronto-rubocop.png)](https://codeclimate.com/github/prontolabs/pronto-rubocop)
+[![Build Status](https://travis-ci.org/prontolabs/pronto-rubocop.svg?branch=master)](https://travis-ci.org/prontolabs/pronto-rubocop)
 [![Gem Version](https://badge.fury.io/rb/pronto-rubocop.png)](http://badge.fury.io/rb/pronto-rubocop)
-[![Dependency Status](https://gemnasium.com/mmozuras/pronto-rubocop.png)](https://gemnasium.com/mmozuras/pronto-rubocop)
 
-Pronto runner for [Rubocop](https://github.com/bbatsov/rubocop), ruby code analyzer. [What is Pronto?](https://github.com/mmozuras/pronto)
+Pronto runner for [RuboCop](https://github.com/bbatsov/rubocop), ruby code
+analyzer. [What is Pronto?](https://github.com/prontolabs/pronto)
 
 ## Configuration
 
-Configuring Rubocop via .rubocop.yml will work just fine with pronto-rubocop.
-You can also specify a custom `.rubocop.yml` location with the environment variable `RUBOCOP_CONFIG`
+Configuring RuboCop via `.rubocop.yml` will work just fine with
+`pronto-rubocop`.
+
+You can also specify a custom `.rubocop.yml` location with the environment
+variable `RUBOCOP_CONFIG`.
+
+You can also provide additional configuration via `.pronto.yml`:
+
+```yml
+rubocop:
+  # Map of RuboCop severity level to Pronto severity level
+  severities:
+    refactor: info
+    warning: error
+
+  # Enable suggestions
+  suggestions: true
+```
+
+## Suggestions
+
+When suggestions are enabled, the messages will include a line suggesting
+what to change, using [GitHub's](https://twitter.com/wa7son/status/1052326282900443137)
+syntax on Pull Request reviews, that can be approved in one click right from
+the Pull Request.
+
+For example:
+
+![GitHub screenshot with suggestion](https://user-images.githubusercontent.com/132/50402757-1bd75b80-0799-11e9-809f-8b8a23ed33f6.png)
+
+## RuboCop versions
+
+If you need to use RuboCop v0.84.0 or v0.85.x, you'll need to ensure that
+you've also need to add `gem 'rubocop-ast', '< 0.7.0'` to your Gemfile as
+these were the first versions to use rubocop-ast, and unfortunately the
+dependency was loose enough that rubocop-ast versions >= 0.7.0 were allowed,
+which causes `require 'rubocop'` to fail with
+```
+  NoMethodError:
+    undefined method `join' for #<Set: {:==, :===, :!=, :<=, :>=, :>, :<}>
+  in rubocop-0.84.0/lib/rubocop/cop/style/redundant_conditional.rb:57:in `<class:RedundantConditional>'
+```
+This is due to https://github.com/rubocop-hq/rubocop-ast/issues/22

data/lib/pronto/rubocop.rb

@@ -1,71 +1,20 @@
+# frozen_string_literal: true
+
 require 'pronto'
 require 'rubocop'
+require 'pronto/rubocop/patch_cop'
+require 'pronto/rubocop/offense_line'
 
 module Pronto
   class Rubocop < Runner
-    def initialize(_, _ = nil)
-      super
-
-      @config_store = ::RuboCop::ConfigStore.new
-      @config_store.options_config = ENV['RUBOCOP_CONFIG'] if ENV['RUBOCOP_CONFIG']
-      @inspector = ::RuboCop::Runner.new({}, @config_store)
-    end
-
     def run
-      return [] unless @patches
-
-      @patches.select { |patch| valid_patch?(patch) }
-        .map { |patch| inspect(patch) }
-        .flatten.compact
-    end
-
-    def valid_patch?(patch)
-      return false if patch.additions < 1
-
-      config_store = config_store_for(patch)
-      path = patch.new_file_full_path
-
-      return false if config_store.file_to_exclude?(path.to_s)
-      return true if config_store.file_to_include?(path.to_s)
-
-      ruby_file?(path)
-    end
-
-    def inspect(patch)
-      processed_source = processed_source_for(patch)
-      offences = @inspector.send(:inspect_file, processed_source).first
-
-      offences.sort.reject(&:disabled?).map do |offence|
-        patch.added_lines
-          .select { |line| line.new_lineno == offence.line }
-          .map { |line| new_message(offence, line) }
-      end
-    end
-
-    def new_message(offence, line)
-      path = line.patch.delta.new_file[:path]
-      level = level(offence.severity.name)
-
-      Message.new(path, line, level, offence.message, nil, self.class)
-    end
-
-    def config_store_for(patch)
-      path = patch.new_file_full_path.to_s
-      @config_store.for(path)
-    end
-
-    def processed_source_for(patch)
-      path = patch.new_file_full_path.to_s
-      ::RuboCop::ProcessedSource.from_file(path, RUBY_VERSION[0..2].to_f)
+      ruby_patches
+        .select { |patch| patch.additions.positive? }
+        .flat_map { |patch| PatchCop.new(patch, self).messages }
     end
 
-    def level(severity)
-      case severity
-      when :refactor, :convention
-        :warning
-      when :warning, :error, :fatal
-        severity
-      end
+    def pronto_rubocop_config
+      @pronto_rubocop_config ||= Pronto::ConfigFile.new.to_h['rubocop'] || {}
     end
   end
 end

data/lib/pronto/rubocop/offense_line.rb

@@ -0,0 +1,130 @@
+# frozen_string_literal: true
+
+module Pronto
+  class Rubocop < Runner
+    class OffenseLine
+      def initialize(patch_cop, offense, line)
+        @patch_cop = patch_cop
+        @offense = offense
+        @line = line
+      end
+
+      def message
+        Message.new(path, line, level, message_text, nil, Pronto::Rubocop)
+      end
+
+      private
+
+      attr_reader :patch_cop, :offense, :line
+
+      def path
+        line.patch.delta.new_file[:path]
+      end
+
+      def processed_source
+        patch_cop.processed_source
+      end
+
+      def message_text
+        return offense.message unless suggestion_text
+
+        "#{offense.message}\n\n```suggestion\n#{suggestion_text}```"
+      end
+
+      def suggestion_text
+        return unless patch_cop.runner.pronto_rubocop_config['suggestions']
+        return if corrections_count.zero?
+        return if differing_lines_count != corrections_count
+
+        @suggestion_text ||= corrected_lines[offense.line - 1]
+      end
+
+      def corrected_lines
+        @corrected_lines ||= corrector.rewrite.lines
+      end
+
+      def differing_lines_count
+        original_lines.each_with_index.count do |line, index|
+          line != corrected_lines[index]
+        end
+      end
+
+      def original_lines
+        processed_source.lines.join("\n").lines
+      end
+
+      if ::RuboCop::Cop::Team.respond_to?(:mobilize) && ::RuboCop::Cop::Team.public_method_defined?(:investigate)
+        # rubocop >= 0.87.0 has both mobilize and public investigate method
+        MOBILIZE = :mobilize
+
+        def report
+          @report ||= autocorrect_team.investigate(processed_source).cop_reports.first
+        end
+
+        def corrector
+          report.corrector
+        end
+
+        def corrections_count
+          report.offenses.size
+        end
+      else
+        # rubocop 0.85.x and 0.86.0 have mobilize, older versions don't
+        MOBILIZE = ::RuboCop::Cop::Team.respond_to?(:mobilize) ? :mobilize : :new
+
+        def corrector
+          @corrector ||= begin
+            autocorrect_team.inspect_file(processed_source)
+            corrector = RuboCop::Cop::Corrector.new(processed_source.buffer)
+            corrector.corrections.concat(autocorrect_team.cops.first.corrections)
+            corrector
+          end
+        end
+
+        def corrections_count
+          @corrections_count ||= corrector.corrections.count
+        end
+      end
+
+      def autocorrect_team
+        @autocorrect_team ||=
+          ::RuboCop::Cop::Team.send(MOBILIZE,
+                                    ::RuboCop::Cop::Registry.new([cop_class]),
+                                    patch_cop.rubocop_config,
+                                    auto_correct: true,
+                                    stdin: true)
+      end
+
+      def cop_class
+        patch_cop.registry.find_by_cop_name(offense.cop_name)
+      end
+
+      def level
+        severities.fetch(offense.severity.name)
+      end
+
+      def severities
+        @severities ||= DEFAULT_SEVERITIES.merge(config_severities)
+      end
+
+      def config_severities
+        patch_cop
+          .runner
+          .pronto_rubocop_config
+          .fetch('severities', {})
+          .map { |k, v| [k.to_sym, v.to_sym] }
+          .to_h
+      end
+
+      DEFAULT_SEVERITIES = {
+        refactor: :warning,
+        convention: :warning,
+        warning: :warning,
+        error: :error,
+        fatal: :fatal
+      }.freeze
+
+      private_constant :DEFAULT_SEVERITIES
+    end
+  end
+end

data/lib/pronto/rubocop/patch_cop.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+module Pronto
+  class Rubocop < Runner
+    class PatchCop
+      attr_reader :runner
+
+      def initialize(patch, runner)
+        @patch = patch
+        @runner = runner
+      end
+
+      def messages
+        return [] unless valid?
+
+        offenses.flat_map do |offense|
+          patch
+            .added_lines
+            .select { |line| line.new_lineno == offense.line }
+            .map { |line| OffenseLine.new(self, offense, line).message }
+        end
+      end
+
+      def processed_source
+        @processed_source ||= ::RuboCop::ProcessedSource.from_file(
+          path,
+          rubocop_config.target_ruby_version
+        )
+      end
+
+      def registry
+        @registry ||= ::RuboCop::Cop::Registry.new(RuboCop::Cop::Cop.all)
+      end
+
+      def rubocop_config
+        @rubocop_config ||= begin
+          store = ::RuboCop::ConfigStore.new
+          store.for(path)
+        end
+      end
+
+      private
+
+      attr_reader :patch
+
+      def valid?
+        return false if rubocop_config.file_to_exclude?(path)
+        return true if rubocop_config.file_to_include?(path)
+
+        true
+      end
+
+      def path
+        @path ||= patch.new_file_full_path.to_s
+      end
+
+      def offenses
+        team
+          .inspect_file(processed_source)
+          .sort
+          .reject(&:disabled?)
+      end
+
+      def team
+        @team ||=
+          if ::RuboCop::Cop::Team.respond_to?(:mobilize)
+            # rubocop v0.85.0 and later
+            ::RuboCop::Cop::Team.mobilize(registry, rubocop_config)
+          else
+            ::RuboCop::Cop::Team.new(registry, rubocop_config)
+          end
+      end
+    end
+  end
+end

data/lib/pronto/rubocop/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module Pronto
   module RubocopVersion
-    VERSION = '0.8.1'.freeze
+    VERSION = '0.11.1'
   end
 end

data/pronto-rubocop.gemspec

@@ -1,6 +1,6 @@
-# -*- encoding: utf-8 -*-
+# frozen_string_literal: true
 
-$LOAD_PATH.push File.expand_path('../lib', __FILE__)
+$LOAD_PATH.push File.expand_path('lib', __dir__)
 require 'pronto/rubocop/version'
 require 'English'
 
@@ -10,11 +10,11 @@ Gem::Specification.new do |s|
   s.platform = Gem::Platform::RUBY
   s.author = 'Mindaugas Mozūras'
   s.email = 'mindaugas.mozuras@gmail.com'
-  s.homepage = 'http://github.org/mmozuras/pronto-rubocop'
+  s.homepage = 'http://github.com/mmozuras/pronto-rubocop'
   s.summary = 'Pronto runner for Rubocop, ruby code analyzer'
 
   s.licenses = ['MIT']
-  s.required_ruby_version = '>= 2.0.0'
+  s.required_ruby_version = '>= 2.3.0'
   s.rubygems_version = '1.8.23'
 
   s.files = `git ls-files`.split($RS).reject do |file|
@@ -32,8 +32,8 @@ Gem::Specification.new do |s|
   s.extra_rdoc_files = ['LICENSE', 'README.md']
   s.require_paths = ['lib']
 
-  s.add_runtime_dependency('rubocop', '~> 0.38', '>= 0.35.0')
-  s.add_runtime_dependency('pronto', '~> 0.8.0')
+  s.add_runtime_dependency('pronto', '~> 0.11.0')
+  s.add_runtime_dependency('rubocop', '>= 0.63.1', '< 2.0')
   s.add_development_dependency('rake', '~> 12.0')
   s.add_development_dependency('rspec', '~> 3.4')
   s.add_development_dependency('rspec-its', '~> 1.2')

metadata

@@ -1,49 +1,49 @@
 --- !ruby/object:Gem::Specification
 name: pronto-rubocop
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.11.1
 platform: ruby
 authors:
 - Mindaugas Mozūras
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-05 00:00:00.000000000 Z
+date: 2021-02-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: pronto
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.38'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.35.0
+        version: 0.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.38'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.35.0
+        version: 0.11.0
 - !ruby/object:Gem::Dependency
-  name: pronto
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.63.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.63.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -94,12 +94,16 @@ extra_rdoc_files:
 - LICENSE
 - README.md
 files:
+- ".github/CODEOWNERS"
+- ".github/workflows/checks.yml"
 - LICENSE
 - README.md
 - lib/pronto/rubocop.rb
+- lib/pronto/rubocop/offense_line.rb
+- lib/pronto/rubocop/patch_cop.rb
 - lib/pronto/rubocop/version.rb
 - pronto-rubocop.gemspec
-homepage: http://github.org/mmozuras/pronto-rubocop
+homepage: http://github.com/mmozuras/pronto-rubocop
 licenses:
 - MIT
 metadata: {}
@@ -111,15 +115,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Pronto runner for Rubocop, ruby code analyzer