ace-review 0.49.0

data/lib/ace/review/version.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Ace
+  module Review
+    VERSION = "0.49.0"
+  end
+end

data/lib/ace/review.rb

@@ -0,0 +1,135 @@
+# frozen_string_literal: true
+
+# Load ace-config for configuration cascade management
+require "ace/support/config"
+
+# Try to load ace-bundle if available (required for full functionality)
+begin
+  require "ace/bundle"
+rescue LoadError
+  # ace-bundle is required for context processing
+  # Will raise MissingDependencyError when needed
+end
+
+# Load ace-git (required dependency for git operations)
+require "ace/git"
+
+require_relative "review/version"
+require_relative "review/errors"
+
+# Require atoms
+require_relative "review/atoms/context_limit_resolver"
+require_relative "review/atoms/diff_boundary_finder"
+require_relative "review/atoms/feedback_id_generator"
+require_relative "review/atoms/feedback_slug_generator"
+require_relative "review/atoms/feedback_state_validator"
+require_relative "review/atoms/pr_comment_formatter"
+require_relative "review/atoms/preset_validator"
+require_relative "review/atoms/priority_filter"
+require_relative "review/atoms/slug_generator"
+require_relative "review/atoms/token_estimator"
+
+# Require all necessary components explicitly
+require_relative "review/molecules/context_composer"
+require_relative "review/molecules/context_extractor"
+require_relative "review/molecules/llm_executor"
+require_relative "review/molecules/preset_manager"
+require_relative "review/molecules/prompt_composer"
+require_relative "review/molecules/nav_prompt_resolver"
+require_relative "review/molecules/subject_extractor"
+require_relative "review/molecules/gh_cli_executor"
+require_relative "review/molecules/gh_pr_fetcher"
+require_relative "review/molecules/gh_pr_comment_fetcher"
+require_relative "review/molecules/gh_comment_poster"
+require_relative "review/molecules/gh_comment_resolver"
+require_relative "review/molecules/multi_model_executor"
+require_relative "review/molecules/feedback_file_writer"
+require_relative "review/molecules/feedback_file_reader"
+require_relative "review/molecules/feedback_directory_manager"
+require_relative "review/molecules/feedback_synthesizer"
+require_relative "review/molecules/subject_filter"
+require_relative "review/molecules/subject_strategy"
+require_relative "review/molecules/pr_task_spec_resolver"
+
+require_relative "review/organisms/review_manager"
+require_relative "review/organisms/feedback_manager"
+
+require_relative "review/models/review_options"
+require_relative "review/models/feedback_item"
+require_relative "review/models/reviewer"
+
+require_relative "review/cli"
+require_relative "review/cli/feedback_cli"
+
+module Ace
+  module Review
+    # Define module namespaces
+    module Atoms; end
+    module Molecules; end
+    module Organisms; end
+    module Models; end
+    module Commands; end
+
+    class << self
+      # Configuration accessor
+      # Follows ADR-022: Configuration Default and Override Pattern
+      # Uses Ace::Support::Config.create() for configuration cascade resolution
+      def config
+        @config ||= begin
+          gem_root = Gem.loaded_specs["ace-review"]&.gem_dir ||
+            File.expand_path("../..", __dir__)
+
+          resolver = Ace::Support::Config.create(
+            config_dir: ".ace",
+            defaults_dir: ".ace-defaults",
+            gem_path: gem_root
+          )
+
+          # Resolve config for review namespace
+          config = resolver.resolve_namespace("review")
+          config.data
+        rescue => e
+          warn "Warning: Could not load ace-review config: #{e.message}" if debug?
+          # Fall back to gem defaults instead of empty hash to prevent silent config erasure
+          load_gem_defaults_fallback
+        end
+      end
+
+      private
+
+      # Load gem defaults directly as fallback when cascade resolution fails
+      # This ensures configuration is never silently erased due to YAML errors
+      # or user config issues
+      def load_gem_defaults_fallback
+        gem_root = Gem.loaded_specs["ace-review"]&.gem_dir ||
+          File.expand_path("../..", __dir__)
+        defaults_path = File.join(gem_root, ".ace-defaults", "review", "config.yml")
+
+        return {} unless File.exist?(defaults_path)
+
+        YAML.safe_load_file(defaults_path, permitted_classes: [Date], aliases: true) || {}
+      rescue Psych::SyntaxError, Errno::ENOENT, Errno::EACCES
+        {} # Only return empty hash if even defaults fail to load
+      end
+
+      public
+
+      # Get configuration value with dot notation
+      def get(*keys)
+        keys.reduce(config) do |hash, key|
+          hash.is_a?(Hash) ? hash[key.to_s] : nil
+        end
+      end
+
+      # Reset cached configuration (useful for testing)
+      def reset_config!
+        @config = nil
+      end
+
+      # Check if running in debug mode
+      def debug?
+        ENV["ACE_DEBUG"] == "1" || ENV["DEBUG"] == "1"
+      end
+    end
+  end
+end

metadata

@@ -0,0 +1,351 @@
+--- !ruby/object:Gem::Specification
+name: ace-review
+version: !ruby/object:Gem::Version
+  version: 0.49.0
+platform: ruby
+authors:
+- Michal Czyz
+bindir: exe
+cert_chain: []
+date: 1980-01-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: ace-support-cli
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
+- !ruby/object:Gem::Dependency
+  name: ace-support-config
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: ace-b36ts
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: ace-support-core
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.24'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.24'
+- !ruby/object:Gem::Dependency
+  name: ace-bundle
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.31'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.31'
+- !ruby/object:Gem::Dependency
+  name: ace-git
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.11'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.11'
+- !ruby/object:Gem::Dependency
+  name: ace-support-nav
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
+- !ruby/object:Gem::Dependency
+  name: ace-llm
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.23'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.23'
+- !ruby/object:Gem::Dependency
+  name: ace-task
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.11'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.11'
+- !ruby/object:Gem::Dependency
+  name: ace-support-test-helpers
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.12'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.12'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.21'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.21'
+description: Run focused, repeatable code reviews using configurable presets and multiple
+  LLM models in parallel. Findings become trackable feedback items with a verify-apply-resolve
+  lifecycle.
+email:
+- mc@cs3b.com
+executables:
+- ace-review
+- ace-review-feedback
+extensions: []
+extra_rdoc_files: []
+files:
+- ".ace-defaults/nav/protocols/guide-sources/ace-review.yml"
+- ".ace-defaults/nav/protocols/prompt-sources/ace-review.yml"
+- ".ace-defaults/nav/protocols/tmpl-sources/ace-review.yml"
+- ".ace-defaults/nav/protocols/wfi-sources/ace-review.yml"
+- ".ace-defaults/review/config.yml"
+- ".ace-defaults/review/presets/code-fit.yml"
+- ".ace-defaults/review/presets/code-shine.yml"
+- ".ace-defaults/review/presets/code-valid.yml"
+- ".ace-defaults/review/presets/docs.yml"
+- ".ace-defaults/review/presets/spec.yml"
+- CHANGELOG.md
+- LICENSE
+- README.md
+- Rakefile
+- exe/ace-review
+- exe/ace-review-feedback
+- handbook/guides/code-review-process.g.md
+- handbook/prompts/base/sections.md
+- handbook/prompts/base/system.md
+- handbook/prompts/focus/architecture/atom.md
+- handbook/prompts/focus/architecture/reflection.md
+- handbook/prompts/focus/frameworks/rails.md
+- handbook/prompts/focus/frameworks/vue-firebase.md
+- handbook/prompts/focus/languages/ruby.md
+- handbook/prompts/focus/phase/correctness.md
+- handbook/prompts/focus/phase/polish.md
+- handbook/prompts/focus/phase/quality.md
+- handbook/prompts/focus/quality/performance.md
+- handbook/prompts/focus/quality/security.md
+- handbook/prompts/focus/scope/docs.md
+- handbook/prompts/focus/scope/spec.md
+- handbook/prompts/focus/scope/tests.md
+- handbook/prompts/format/compact.md
+- handbook/prompts/format/detailed.md
+- handbook/prompts/format/standard.md
+- handbook/prompts/guidelines/icons.md
+- handbook/prompts/guidelines/tone.md
+- handbook/prompts/synthesis-review-reports.system.md
+- handbook/prompts/synthesize-feedback.system.md
+- handbook/skills/as-review-apply-feedback/SKILL.md
+- handbook/skills/as-review-package/SKILL.md
+- handbook/skills/as-review-pr/SKILL.md
+- handbook/skills/as-review-run/SKILL.md
+- handbook/skills/as-review-verify-feedback/SKILL.md
+- handbook/templates/review-tasks/task-review-summary.template.md
+- handbook/workflow-instructions/review/apply-feedback.wf.md
+- handbook/workflow-instructions/review/package.wf.md
+- handbook/workflow-instructions/review/pr.wf.md
+- handbook/workflow-instructions/review/run.wf.md
+- handbook/workflow-instructions/review/verify-feedback.wf.md
+- lib/ace/review.rb
+- lib/ace/review/atoms/context_limit_resolver.rb
+- lib/ace/review/atoms/diff_boundary_finder.rb
+- lib/ace/review/atoms/feedback_id_generator.rb
+- lib/ace/review/atoms/feedback_slug_generator.rb
+- lib/ace/review/atoms/feedback_state_validator.rb
+- lib/ace/review/atoms/pr_comment_formatter.rb
+- lib/ace/review/atoms/preset_validator.rb
+- lib/ace/review/atoms/priority_filter.rb
+- lib/ace/review/atoms/retry_with_backoff.rb
+- lib/ace/review/atoms/slug_generator.rb
+- lib/ace/review/atoms/token_estimator.rb
+- lib/ace/review/cli.rb
+- lib/ace/review/cli/commands/feedback.rb
+- lib/ace/review/cli/commands/feedback/create.rb
+- lib/ace/review/cli/commands/feedback/list.rb
+- lib/ace/review/cli/commands/feedback/resolve.rb
+- lib/ace/review/cli/commands/feedback/session_discovery.rb
+- lib/ace/review/cli/commands/feedback/show.rb
+- lib/ace/review/cli/commands/feedback/skip.rb
+- lib/ace/review/cli/commands/feedback/verify.rb
+- lib/ace/review/cli/commands/review.rb
+- lib/ace/review/cli/feedback_cli.rb
+- lib/ace/review/errors.rb
+- lib/ace/review/models/feedback_item.rb
+- lib/ace/review/models/review_options.rb
+- lib/ace/review/models/reviewer.rb
+- lib/ace/review/molecules/context_composer.rb
+- lib/ace/review/molecules/context_extractor.rb
+- lib/ace/review/molecules/feedback_directory_manager.rb
+- lib/ace/review/molecules/feedback_file_reader.rb
+- lib/ace/review/molecules/feedback_file_writer.rb
+- lib/ace/review/molecules/feedback_synthesizer.rb
+- lib/ace/review/molecules/gh_cli_executor.rb
+- lib/ace/review/molecules/gh_comment_poster.rb
+- lib/ace/review/molecules/gh_comment_resolver.rb
+- lib/ace/review/molecules/gh_pr_comment_fetcher.rb
+- lib/ace/review/molecules/gh_pr_fetcher.rb
+- lib/ace/review/molecules/llm_executor.rb
+- lib/ace/review/molecules/multi_model_executor.rb
+- lib/ace/review/molecules/nav_prompt_resolver.rb
+- lib/ace/review/molecules/pr_task_spec_resolver.rb
+- lib/ace/review/molecules/preset_manager.rb
+- lib/ace/review/molecules/prompt_composer.rb
+- lib/ace/review/molecules/prompt_resolver.rb
+- lib/ace/review/molecules/strategies/adaptive_strategy.rb
+- lib/ace/review/molecules/strategies/chunked_strategy.rb
+- lib/ace/review/molecules/strategies/full_strategy.rb
+- lib/ace/review/molecules/subject_extractor.rb
+- lib/ace/review/molecules/subject_filter.rb
+- lib/ace/review/molecules/subject_strategy.rb
+- lib/ace/review/molecules/task_report_saver.rb
+- lib/ace/review/molecules/task_resolver.rb
+- lib/ace/review/organisms/feedback_manager.rb
+- lib/ace/review/organisms/review_manager.rb
+- lib/ace/review/version.rb
+homepage: https://github.com/cs3b/ace
+licenses:
+- MIT
+metadata:
+  allowed_push_host: https://rubygems.org
+  homepage_uri: https://github.com/cs3b/ace
+  source_code_uri: https://github.com/cs3b/ace
+  changelog_uri: https://github.com/cs3b/ace/blob/main/ace-review/CHANGELOG.md
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: 3.2.0
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubygems_version: 3.6.9
+specification_version: 4
+summary: Multi-model code review with preset-based analysis — for PRs, tasks, and
+  packages
+test_files: []