pronto-rubocop 0.8.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3f3464874bf36e423208f529eb5325482e8b13f3
-  data.tar.gz: 183b57a9994cf7a70fb51c68ec9983f1fd01f22b
+SHA256:
+  metadata.gz: '0588bd2b9b7d1c7a702dba2d9a5cdfc74e4d5df1eaa893557aa13445ab82c9d7'
+  data.tar.gz: 6bd89ce4bcec0647adc3c62abe770c8f43aff21d77b96a6c9679f6e35ffc84c2
 SHA512:
-  metadata.gz: bf51ee17658b490a7fd98bcd8a556cb4bc9a917d8c3db14bfce218ad7944ac76eba1dfdbe9eded26c9af69e1d3f2525424d3b83046d46ba62ece28227f5ed53f
-  data.tar.gz: f842558e562ce4dbe4812e74c854aca016a89c244b325f1e4ae817f8d3c6796b912d0d73daadef02b61a535f2c8eb22e5d4445669483ba16bf32fa9ecfabb3c7
+  metadata.gz: 2177a1cdf31387028a4dac6aed07b0df8379c33e84d80ac3b1329187beb99ad6b3bb490d1c70b4c098345b5cfac538463beff2767fe0bb28c094e78cbdcd14f1
+  data.tar.gz: 48856034490369353ed28755decda13d3e0f225468c8600a2706257ec2a119b88516de1757e7b5a5969ebd76e00e16fd908ca2b0e642901c1f51389a6ce483fc

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,22 @@
+name: Checks
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  ruby:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ['2.3', '2.4', '2.5', '2.6', '2.7', '3.0']
+    steps:
+    - uses: actions/checkout@v2
+    - 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,40 @@
-# 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)

data/lib/pronto/rubocop.rb

@@ -1,71 +1,18 @@
 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 > 0 }
+        .flat_map { |patch| PatchCop.new(patch, self).messages }
     end
 
-    def level(severity)
-      case severity
-      when :refactor, :convention
-        :info
-      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,128 @@
+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 == 0
+        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)
+        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 < v0.85.0
+        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,73 @@
+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,5 @@
 module Pronto
   module RubocopVersion
-    VERSION = '0.8.0'.freeze
+    VERSION = '0.11.0'.freeze
   end
 end

data/pronto-rubocop.gemspec

@@ -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,9 +32,9 @@ 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_development_dependency('rake', '~> 11.0')
+  s.add_runtime_dependency('pronto', '~> 0.11.0')
+  s.add_runtime_dependency('rubocop', '>= 0.49.1', '< 1.0')
+  s.add_development_dependency('rake', '~> 12.0')
   s.add_development_dependency('rspec', '~> 3.4')
   s.add_development_dependency('rspec-its', '~> 1.2')
 end

metadata

@@ -1,63 +1,63 @@
 --- !ruby/object:Gem::Specification
 name: pronto-rubocop
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Mindaugas Mozūras
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-12 00:00:00.000000000 Z
+date: 2021-01-31 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.49.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.49.1
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   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