ace-review 0.56.0 → 0.56.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 92554e56ae82d2ecd841daf699eae3c77e6a97f7b01f0a6e2681804ab5c88109
-  data.tar.gz: 77450cdd27add8a7a239e4949ceec8a9b74a2e61824d0477f9e49c8a3bbeee84
+  metadata.gz: a3eb68d4c56fe426002098bd5d5dc10567ea5dcf92ba709069d526c9d8d4ca77
+  data.tar.gz: bdaafd88e24669a1c2beb50fd16b2daf606f56b7f07d0815a36ba28deda175a3
 SHA512:
-  metadata.gz: 40d4fb853eecf34a49a3aebcbee634409c82d76ca1e584c3218e5ca0fe3086a11e9a61f104e19ddae27508a51473ddf89f031eb3fab287d98f0998a7047c6353
-  data.tar.gz: f9d0635b9258e7395e56ffde24dc58977bc5dd031984a4ef05b6d779f5460f42c4090e818ce439748a9100498a8314920c8832f29d938a98dcebc31db4954899
+  metadata.gz: c1acb46ca00a76f48bd05e65f7b167e9bb395732a922bf848a8792fd342be10e85777e155eb6ba5675396a6021bd257fd9d3008dcc26a03656768f83931d9338
+  data.tar.gz: 23c76466b28055d2c02364e711373d2743b71684b3dcb611c2523e344c430546034af289d972c3af80d590bb221f61cef8b2d977c988d334fcd620398c1467c6

data/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.56.1] - 2026-09-29
+
+### Fixed
+- A full PR review round is no longer mistaken for a delta round. dry-cli yields
+  nil for an absent `--delta` and for a bare valueless `--delta`, and both were
+  normalized to auto-resolution — so the first-ever review of any new PR failed
+  with "No prior review session records a head". Nil now stays a full round;
+  auto-resolution is the explicit `--delta auto` value (explicit heads unchanged).
+- The released gemspec now declares `ace-git-github ~> 0.2`, matching the source
+  constraint: the previously published 0.56.0 metadata still pinned `~> 0.1.1`,
+  which kept published installations resolving ace-git-github 0.1.2 and made
+  the released 0.2.0 unreachable through dependency resolution.
+
 ## [0.56.0] - 2026-09-27
 
 ### Added

data/lib/ace/review/cli/commands/review.rb

@@ -64,7 +64,7 @@ module Ace
           option :gh_timeout, type: :integer, desc: "Timeout for gh CLI operations in seconds (default: 30)"
           option :evidence_session, type: :array, desc: "Include selected prior review sessions as context"
           option :delta, type: :string, optional_value: true,
-            desc: "Delta round for --pr: review only changes since a reference head (explicit head, or bare --delta to auto-resolve from the most recent prior session for this PR)"
+            desc: "Delta round for --pr: review only changes since a reference head (explicit head, or `auto` to auto-resolve from the most recent prior session for this PR); omit for a full round"
           option :prepare_goals_brief, type: :boolean, desc: "Generate or reuse the shared goals brief for this PR and preset"
 
           # Standard options
@@ -157,14 +157,23 @@ module Ace
             options
           end
 
-          # Normalize --delta: bare flag or empty value means auto-resolve the
-          # reference head from prior sessions (:auto); a value is the explicit head
+          # Normalize --delta: the explicit value "auto" resolves the reference
+          # head from prior sessions (:auto); any other value is the head itself.
+          # dry-cli cannot distinguish an absent --delta from a bare one (both
+          # arrive as nil), and a full round must never be mistaken for a delta
+          # round — so nil stays a full round and auto-resolution is spelled out.
           def process_delta(options)
             return unless options.key?(:delta)
 
             value = options[:delta]
             value = value.to_s.strip if value.is_a?(String)
-            options[:delta] = (value.nil? || value == "") ? :auto : value
+            options[:delta] = if value.nil? || value.empty?
+              nil
+            elsif value.casecmp("auto").zero?
+              :auto
+            else
+              value
+            end
           end
 
           def process_subjects(options)

data/lib/ace/review/molecules/review_evidence.rb

@@ -21,7 +21,14 @@ module Ace
             if metadata["pr_url"] && metadata["pr_url"] != pr_metadata["url"]
               return {success: false, error: "#{dir} belongs to a different PR"}
             end
-            reports = Dir.glob(File.join(dir, "review-report-*.md"))
+            # Reviewer reports are written as review-report-<model>.md (Ruby API
+            # path) or review-<model>.md (CLI provider path); prompt/context and
+            # dev-feedback files are not reports.
+            reports = Dir.glob(File.join(dir, "review-*.md")).select do |path|
+              base = File.basename(path)
+              !base.end_with?(".prompt.md", ".context.md") &&
+                !%w[review-dev-feedback.md].include?(base)
+            end
             return {success: false, error: "No review report in #{dir}"} if reports.empty?
 
             lines += ["", "## #{File.basename(dir)} — #{metadata["preset"]}",

data/lib/ace/review/version.rb

@@ -2,6 +2,6 @@
 
 module Ace
   module Review
-    VERSION = "0.56.0"
+    VERSION = "0.56.1"
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ace-review
 version: !ruby/object:Gem::Version
-  version: 0.56.0
+  version: 0.56.1
 platform: ruby
 authors:
 - Michal Czyz
 bindir: exe
 cert_chain: []
-date: 2026-09-27 00:00:00.000000000 Z
+date: 2026-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ace-support-cli
@@ -113,14 +113,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.1
+        version: '0.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.1
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: ace-support-nav
   requirement: !ruby/object:Gem::Requirement