Goal
Adopt the shared CodeRabbit review capability from acgetchell/research-repo-tools#5 through thin just review [base] and just review-uncommitted recipes in this repository. This is a separate review-adoption issue; general maintenance and toolchain migration remains in #254.
Prerequisite and rollout
The shared implementation in acgetchell/research-repo-tools#5 must be ready, and a release containing the review commands must be available on PyPI before production adoption. Pin that exact version in the tooling dependency group included by dev and update the lockfile. Do not assume the first package release contains this capability. A local wheel may be used for pilot comparison, but committed dependencies and CI must use the published package.
Coordinate with #253: preferably deliver its requested recipes directly through the shared implementation when available. If local recipes land first, replace their helper during this adoption. Keep #253 independently actionable; count shared adoption toward its requirements rather than duplicating the work.
Scope
- Use the shared public CLI for branch and uncommitted scopes; keep all common scope, freshness, instruction discovery, subprocess, and error handling upstream.
- Default branch review to
origin/main, verified against the live remote at invocation. Include committed branch changes, staged/unstaged changes, and non-ignored untracked files. Preserve explicit local-base overrides and uncommitted-only review without remote freshness checks.
- Use the shared structured output and configuration discovery contract with this repository's instructions. Preserve streamed output, interruption, and failures; unavailable reviews must not appear clean.
- Keep CodeRabbit externally installed and authenticated. Review remains opt-in, outside
check/ci, with explicit maintainer authorization for agent invocation. No implicit installation, authentication, paid-credit enabling, retries, or Git mutations.
- Document prerequisites, scopes, freshness diagnostics, and dependency updates; expose both recipes in help using the repository's command conventions.
- Port applicable common regressions upstream before deleting duplicated helpers or tests. Retain focused consumer tests for recipe wiring, configuration discovery, argument forwarding, and failure propagation using local stubs. Keep helpers still needed by other callers.
Acceptance criteria
A live CodeRabbit smoke review is separate, explicitly authorized work; it is not an automatic validation step.
Goal
Adopt the shared CodeRabbit review capability from acgetchell/research-repo-tools#5 through thin
just review [base]andjust review-uncommittedrecipes in this repository. This is a separate review-adoption issue; general maintenance and toolchain migration remains in #254.Prerequisite and rollout
The shared implementation in acgetchell/research-repo-tools#5 must be ready, and a release containing the review commands must be available on PyPI before production adoption. Pin that exact version in the tooling dependency group included by dev and update the lockfile. Do not assume the first package release contains this capability. A local wheel may be used for pilot comparison, but committed dependencies and CI must use the published package.
Coordinate with #253: preferably deliver its requested recipes directly through the shared implementation when available. If local recipes land first, replace their helper during this adoption. Keep #253 independently actionable; count shared adoption toward its requirements rather than duplicating the work.
Scope
origin/main, verified against the live remote at invocation. Include committed branch changes, staged/unstaged changes, and non-ignored untracked files. Preserve explicit local-base overrides and uncommitted-only review without remote freshness checks.check/ci, with explicit maintainer authorization for agent invocation. No implicit installation, authentication, paid-credit enabling, retries, or Git mutations.Acceptance criteria
A live CodeRabbit smoke review is separate, explicitly authorized work; it is not an automatic validation step.