ace-review 0.58.1 → 0.59.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (5) hide show
  1. checksums.yaml +4 -4
  2. data/CHANGELOG.md +8 -0
  3. data/handbook/workflow-instructions/review/pr.wf.md +3 -3
  4. data/lib/ace/review/version.rb +1 -1
  5. metadata +18 -4
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA256:
3
- metadata.gz: b74a181828465000d62fedc0062c118681144724a8beec940af5a335dade03b7
4
- data.tar.gz: 700918d82fb6f27833a1b0f054301912d1a99dd5609b6140d93ff155a272f0b5
3
+ metadata.gz: eeb8650e65f2751e795ea4db791eb9074f95b26ee0eb93d54e4db8040b826b47
4
+ data.tar.gz: 58a8a7d0a1d268e7282029a9e7805b975deba5d56d6afc24d7b88ab7cf8e8650
5
5
  SHA512:
6
- metadata.gz: 3747d044dc834fe44d7977dfa120052709e5db45885a0c4fa39f7bbb60d4b819e6ea4c0f1f70c7189229b580f565b8936e6d5c1821ca04d4eb787a4cc84add61
7
- data.tar.gz: 8ac3e30edd97356132095db4deca911ec62db8ba4c8ab0b92f7de8a11a8b6464cf5a9404180d8420d6eb18badb398ec158ee9e6f1b60a37f117d0dfbd4f6d6b6
6
+ metadata.gz: b0d0d7238c634039f0b40e3302bf7ba90ff6a84db3e4e6833e72632905590b15ff4df1c6e0da927633c15fa444e97e4fdaf2fbd7292adbe8fe9df3381f98a7f3
7
+ data.tar.gz: d63ca8120a95fb200115fc27ece39f49ba4d3b662587ea94b5532f978c7a27bb8ed722746e46c0d6c46388f39f84494bb48c4eba0bc38d9053c0668a7acec7d8
data/CHANGELOG.md CHANGED
@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
7
7
 
8
8
  ## [Unreleased]
9
9
 
10
+ ## [0.59.0] - 2026-10-05
11
+
12
+ ### Changed
13
+
14
+ - Declare the required direct dependencies and minimum producer versions for this coordinated release: `ace-git ~> 0.29`, `ace-support-fs ~> 0.3`.
15
+
16
+ - PR review instructions retain neutral forge selection and current candidate evidence instead of identifying a PR through GitHub CLI.
17
+
10
18
  ## [0.58.1] - 2026-10-02
11
19
 
12
20
  ### Fixed
data/handbook/workflow-instructions/review/pr.wf.md CHANGED
@@ -19,15 +19,15 @@ Use at least one available reviewer per needed scope. Prefer the project's confi
19
19
 
20
20
  ## 1. Choose and prepare the scope
21
21
 
22
- Identify the PR from the argument or `gh pr view`. Read its goal, current requirements, relevant ADRs and changed-file list. Use the smallest set of presets covering the change. Small coherent PRs can be reviewed whole; split large inputs by high-level module or functional lens, not arbitrary file chunks. Include needed contracts and code when reviewing tests and include integration behavior when modules interact.
22
+ Identify the PR from an explicit number, owner/repository#number or URL and read normalized evidence with `ace-git pr show IDENTIFIER --format json`. Use the assignment's retained forge selection: `--server NAME` or `--default-server`, mutually exclusive; otherwise resolve the repository remote. Keep that same selection on `ace-review --pr IDENTIFIER` in every round. Do not infer a forge or PR from a fixed hostname or caller identity. Read its goal, current requirements, relevant ADRs and changed-file list. Use the smallest set of presets covering the change. Small coherent PRs can be reviewed whole; split large inputs by high-level module or functional lens, not arbitrary file chunks. Include needed contracts and code when reviewing tests and include integration behavior when modules interact.
23
23
 
24
- Use `ace-review --pr <number> --preset <preset> --dry-run` to inspect the selected/omitted files, complete prompt and budget. Context and instructions each have a 30k ceiling; total input has a 128k ceiling. Change scope if too large; do not silently truncate or claim omitted code was reviewed. Generated files may use an appropriate deterministic check, recorded in the round summary, without a separate certification system.
24
+ Use `ace-review --pr IDENTIFIER --server NAME --preset PRESET --dry-run` (or the retained default/remote selection) to inspect the selected/omitted files, complete prompt and budget. Context and instructions each have a 30k ceiling; total input has a 128k ceiling. Change scope if too large; do not silently truncate or claim omitted code was reviewed. Generated files may use an appropriate deterministic check, recorded in the round summary, without a separate certification system.
25
25
 
26
26
  For presets with a goals brief, `--prepare-goals-brief` generates or reuses the shared summary. Its cache follows source content and instructions, not review-round state. Architectural foundations should receive design review before implementation.
27
27
 
28
28
  ## 2. Run a round and verify findings
29
29
 
30
- Run `ace-review --pr <number> --preset <preset> --auto-execute` for the needed scopes. Pass selected previous sessions with `--evidence-session <path>` when useful. Earlier commit SHAs are expected: prior reports provide context and dispositions, not current-code certificates. Supply a short summary of fixes and outstanding issues through the preset context when it is more useful than full reports.
30
+ Run `ace-review --pr IDENTIFIER --server NAME --preset PRESET --auto-execute` with the retained default/remote selection when applicable. Pass selected previous sessions with `--evidence-session <path>` when useful. Earlier commit SHAs are expected: prior reports provide context and dispositions, not current-code certificates. Supply a short summary of fixes and outstanding issues through the preset context when it is more useful than full reports. Every final accepted review and test receipt must bind the current candidate SHA; journal commits are separate from candidate_head. A committed task edit changes the candidate and requires renewed evidence. CI remains visible and advisory; failed execution, unresolved findings or stale receipts block acceptance.
31
31
 
32
32
  Round N+1 does not need to re-review unchanged code. `--delta` scopes the round to the diff since the reference head recorded by the most recent prior session of that PR (`--delta <head>` names an explicit reference), carries its findings forward as evidence, and completes as a zero-model no-op session when the delta is empty. When the project config declares `exempt_paths` and a delta touches only those paths, the round is recorded as a review-exempt no-op; mixed deltas review only the non-exempt part. Full rounds are never converted into no-ops by exempt paths.
33
33
 
data/lib/ace/review/version.rb CHANGED
@@ -2,6 +2,6 @@
2
2
 
3
3
  module Ace
4
4
  module Review
5
- VERSION = "0.58.1"
5
+ VERSION = "0.59.0"
6
6
  end
7
7
  end
metadata CHANGED
@@ -1,14 +1,28 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: ace-review
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.58.1
4
+ version: 0.59.0
5
5
  platform: ruby
6
6
  authors:
7
7
  - Michal Czyz
8
8
  bindir: exe
9
9
  cert_chain: []
10
- date: 2026-10-03 00:00:00.000000000 Z
10
+ date: 2026-10-05 00:00:00.000000000 Z
11
11
  dependencies:
12
+ - !ruby/object:Gem::Dependency
13
+ name: ace-support-fs
14
+ requirement: !ruby/object:Gem::Requirement
15
+ requirements:
16
+ - - "~>"
17
+ - !ruby/object:Gem::Version
18
+ version: '0.3'
19
+ type: :runtime
20
+ prerelease: false
21
+ version_requirements: !ruby/object:Gem::Requirement
22
+ requirements:
23
+ - - "~>"
24
+ - !ruby/object:Gem::Version
25
+ version: '0.3'
12
26
  - !ruby/object:Gem::Dependency
13
27
  name: ace-support-cli
14
28
  requirement: !ruby/object:Gem::Requirement
@@ -99,14 +113,14 @@ dependencies:
99
113
  requirements:
100
114
  - - "~>"
101
115
  - !ruby/object:Gem::Version
102
- version: '0.26'
116
+ version: '0.29'
103
117
  type: :runtime
104
118
  prerelease: false
105
119
  version_requirements: !ruby/object:Gem::Requirement
106
120
  requirements:
107
121
  - - "~>"
108
122
  - !ruby/object:Gem::Version
109
- version: '0.26'
123
+ version: '0.29'
110
124
  - !ruby/object:Gem::Dependency
111
125
  name: ace-git-github
112
126
  requirement: !ruby/object:Gem::Requirement