aireview 0.3.0 → 2.1.0

data/lib/aireview/stages.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Aireview
+  # The two review stages: Generate finds the findings, Critique checks them.
+  # The single definition; inside the code a stage is a string.
+  STAGES = %w[generate critique].freeze
+end

data/lib/aireview/utils.rb

@@ -14,5 +14,34 @@ module Aireview
     def presence(value)
       present?(value) ? value.to_s.strip : nil
     end
+
+    # Merges settings by key: nested hashes merge, everything else (arrays
+    # included) is replaced by the right side as a whole.
+    def deep_merge(left, right)
+      left.merge(right) do |_, old_value, new_value|
+        if old_value.is_a?(Hash) && new_value.is_a?(Hash)
+          deep_merge(old_value, new_value)
+        else
+          new_value
+        end
+      end
+    end
+
+    # Hash keys are strings at any depth: YAML, env and LLM answers arrive in
+    # different shapes, everything downstream works with one.
+    def normalize_hash(value)
+      case value
+      when Hash
+        value.each_with_object({}) { |(key, inner), result| result[key.to_s] = normalize_hash(inner) }
+      when Array
+        value.map { |item| normalize_hash(item) }
+      else
+        value
+      end
+    end
+
+    def dig(data, *keys)
+      keys.reduce(data) { |accumulator, key| accumulator.is_a?(Hash) ? accumulator[key] : nil }
+    end
   end
 end

data/lib/aireview/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Aireview
-  VERSION = '0.3.0'
+  VERSION = '2.1.0'
 end

data/lib/aireview.rb

@@ -16,6 +16,7 @@ require_relative 'aireview/secret_scrubber'
 require_relative 'aireview/context_builder'
 require_relative 'aireview/reviewer'
 require_relative 'aireview/review_pipeline'
+require_relative 'aireview/model_checker'
 require_relative 'aireview/gitlab_client'
 require_relative 'aireview/jira_client'
 require_relative 'aireview/review_marker'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aireview
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Denis Levenko
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-14 00:00:00.000000000 Z
+date: 2026-09-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dotenv
@@ -31,6 +31,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.14'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.14.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -38,20 +41,23 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.14'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.14.4
 - !ruby/object:Gem::Dependency
   name: ruby_llm
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.16.0
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.16.0
+        version: 2.0.0
 description: 'Reviews self-hosted GitLab merge requests with a two-pass LLM pipeline:
   the first pass finds candidate findings, the second one critiques them and drops
   the weak ones. Supports Gemini and local Ollama, optional Jira context and posting
@@ -68,12 +74,15 @@ files:
 - README.md
 - bin/aireview
 - config/.aireview.yml.example
+- config/defaults.yml
 - lib/aireview.rb
 - lib/aireview/candidate_checker.rb
 - lib/aireview/cli.rb
 - lib/aireview/config.rb
 - lib/aireview/config_fallbacks.rb
+- lib/aireview/config_layers.rb
 - lib/aireview/config_limits.rb
+- lib/aireview/config_loader.rb
 - lib/aireview/context_budget.rb
 - lib/aireview/context_builder.rb
 - lib/aireview/diff_fetcher.rb
@@ -81,20 +90,27 @@ files:
 - lib/aireview/errors.rb
 - lib/aireview/gitlab_client.rb
 - lib/aireview/jira_client.rb
+- lib/aireview/llm_client.rb
 - lib/aireview/llm_failure.rb
 - lib/aireview/llm_router.rb
+- lib/aireview/model_candidate.rb
+- lib/aireview/model_checker.rb
+- lib/aireview/model_pool.rb
+- lib/aireview/model_state.rb
 - lib/aireview/mr_parser.rb
 - lib/aireview/output_schemas.rb
 - lib/aireview/prompts/critique.txt
 - lib/aireview/prompts/generate.txt
 - lib/aireview/publisher.rb
-- lib/aireview/result_validation.rb
+- lib/aireview/result_parser.rb
 - lib/aireview/review_marker.rb
 - lib/aireview/review_pipeline.rb
 - lib/aireview/review_renderer.rb
 - lib/aireview/review_schemas.rb
 - lib/aireview/reviewer.rb
 - lib/aireview/secret_scrubber.rb
+- lib/aireview/stage_chains.rb
+- lib/aireview/stages.rb
 - lib/aireview/utils.rb
 - lib/aireview/version.rb
 homepage: https://github.com/DenisDenis9331/aireview

data/lib/aireview/result_validation.rb

@@ -1,65 +0,0 @@
-# frozen_string_literal: true
-
-module Aireview
-  # Проверка формы ответов LLM после разбора JSON: кандидаты, вердикты, id.
-  # Ошибка формы — SchemaError, пайплайн решает, чинить ли ответ повторным
-  # запросом. Ожидает от включающего класса value, normalize_id и
-  # normalize_decision.
-  module ResultValidation
-    class SchemaError < StandardError
-    end
-
-    private
-
-    def validate_generate_result_shape!(parsed)
-      valid_shape = parsed.is_a?(Hash) && parsed['candidates'].is_a?(Array)
-      raise SchemaError, 'expected an object with summary and candidates array' unless valid_shape
-      raise SchemaError, 'each generate candidate must be an object' unless parsed['candidates'].all?(Hash)
-    end
-
-    def validate_critique_result_shape!(parsed)
-      valid_shape = parsed.is_a?(Hash) && parsed['verdicts'].is_a?(Array)
-      raise SchemaError, 'expected an object with verdicts array' unless valid_shape
-      raise SchemaError, 'each critique verdict must be an object' unless parsed['verdicts'].all?(Hash)
-    end
-
-    def validate_identifiers!(identifiers, missing_message:, duplicate_prefix:)
-      raise SchemaError, missing_message unless identifiers.all?
-
-      duplicate_ids = identifiers.group_by(&:itself).select { |_, ids| ids.size > 1 }.keys
-      return if duplicate_ids.empty?
-
-      raise SchemaError, "#{duplicate_prefix}: #{duplicate_ids.join(', ')}"
-    end
-
-    def validate_expected_verdict_ids!(verdict_ids, expected_ids)
-      return unless expected_ids
-
-      unknown_ids = verdict_ids - expected_ids
-      missing_ids = expected_ids - verdict_ids
-      raise SchemaError, "unknown verdict ids: #{unknown_ids.join(', ')}" unless unknown_ids.empty?
-      raise SchemaError, "missing verdict ids: #{missing_ids.join(', ')}" unless missing_ids.empty?
-    end
-
-    def validate_verdict!(verdict)
-      id = normalize_id(value(verdict, 'id'))
-      decision = normalize_decision(value(verdict, 'decision'))
-      raise SchemaError, "invalid verdict decision for #{id}" unless %w[keep reject].include?(decision)
-
-      refinement = value(verdict, 'refinement')
-      raise SchemaError, "reject verdict cannot include refinement for #{id}" if invalid_refinement?(verdict, decision)
-      return if refinement.nil?
-      return if refinement.is_a?(Hash)
-
-      raise SchemaError, "refinement must be an object for #{id}"
-    end
-
-    def invalid_refinement?(verdict, decision)
-      refinement_key?(verdict) && decision != 'keep'
-    end
-
-    def refinement_key?(verdict)
-      verdict.key?('refinement') || verdict.key?(:refinement)
-    end
-  end
-end