cleo_quality_review 0.2.0 → 0.4.0

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cleo_quality_review
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Gavin Morrice
@@ -9,20 +9,6 @@ bindir: exe
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: debride
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: fasterer
   requirement: !ruby/object:Gem::Requirement
@@ -80,7 +66,6 @@ files:
 - lib/cleo_quality_review.rb
 - lib/cleo_quality_review/changes_diff.rb
 - lib/cleo_quality_review/checks.rb
-- lib/cleo_quality_review/checks/debride.rb
 - lib/cleo_quality_review/checks/fasterer.rb
 - lib/cleo_quality_review/checks/flog.rb
 - lib/cleo_quality_review/checks/quality_check.rb
@@ -89,12 +74,15 @@ files:
 - lib/cleo_quality_review/cli.rb
 - lib/cleo_quality_review/command_result.rb
 - lib/cleo_quality_review/command_runner.rb
+- lib/cleo_quality_review/concurrent_executor.rb
 - lib/cleo_quality_review/configuration.rb
 - lib/cleo_quality_review/diff_map.rb
 - lib/cleo_quality_review/formatter.rb
 - lib/cleo_quality_review/git_diff_base.rb
+- lib/cleo_quality_review/github_client.rb
 - lib/cleo_quality_review/github_review_builder.rb
 - lib/cleo_quality_review/github_review_publisher.rb
+- lib/cleo_quality_review/incremental_base_resolver.rb
 - lib/cleo_quality_review/llm_client.rb
 - lib/cleo_quality_review/llm_config.rb
 - lib/cleo_quality_review/llm_errors.rb
@@ -115,6 +103,7 @@ files:
 - lib/cleo_quality_review/target_resolver.rb
 - lib/cleo_quality_review/version.rb
 - prompts/agent.md
+- prompts/configuration.md
 - prompts/github.md
 - prompts/human.md
 - prompts/pr_review.md

data/lib/cleo_quality_review/checks/debride.rb

@@ -1,65 +0,0 @@
-# frozen_string_literal: true
-
-require "json"
-
-require_relative "quality_check"
-
-module CleoQualityReview
-  module Checks
-    ##
-    # Quality check implementation for Debride unused-code analyzer
-    class Debride < QualityCheck
-      self.check_name = "dead_code"
-      self.tool_name = "debride"
-      self.output_extension = "json"
-
-      private
-
-      def command(files)
-        [ruby_executable, gem_executable("debride", "debride"), "--json", "--rails", *files]
-      end
-
-      def parse(stdout, stderr)
-        findings = missing_methods(stdout).flat_map do |class_name, methods|
-          results_for_class(class_name, methods)
-        end
-        return findings unless findings.empty? && stderr.to_s.strip != ""
-
-        [result(check: "Execution error", message: stderr, filepath: nil)]
-      end
-
-      def missing_methods(stdout)
-        parsed = JSON.parse(stdout.to_s)
-        missing = parsed.fetch("missing", {})
-        return {} unless missing.is_a?(Hash)
-
-        missing
-      rescue JSON::ParserError
-        {}
-      end
-
-      def results_for_class(class_name, methods)
-        Array(methods).map { |entry| method_to_result(class_name, entry) }
-      end
-
-      def method_to_result(class_name, entry)
-        method_name, location = Array(entry)
-        filepath, line = parse_location(location)
-
-        result(
-          check: "PotentialDeadMethod",
-          message: "#{class_name}##{method_name} might not be called",
-          filepath: filepath,
-          line: line,
-        )
-      end
-
-      def parse_location(location)
-        match = location.to_s.match(/\A(?<filepath>.*):(?<line>\d+)(?:-\d+)?\z/)
-        return [nil, nil] unless match
-
-        match.values_at(:filepath, :line)
-      end
-    end
-  end
-end