Claude / Claude Code Review
completed
Jul 28, 2026 in 16m 57s
Code review found 1 important issue
Found 5 candidates, confirmed 9. See review comments for details.
Details
| Severity | Count |
|---|---|
| π΄ Important | 1 |
| π‘ Nit | 7 |
| π£ Pre-existing | 1 |
| Severity | File:Line | Issue |
|---|---|---|
| π‘ Nit | RELEASE.md:82-86 |
Maintenance post-release check fails whenever the newest v2 tag is a pre-release |
| π£ Pre-existing | RELEASE.md:7-8 |
Stale dependency-bump step: uv lock --resolution lowest-direct would mass-downgrade the committed lockfile |
Annotations
Check warning on line 86 in RELEASE.md
claude / Claude Code Review
Maintenance post-release check fails whenever the newest v2 tag is a pre-release
The maintenance section's post-release check β "confirm `/releases/latest` still names the newest v2 tag" β is literally unsatisfiable whenever the newest v2 tag is a pre-release (e.g. `v2.1.0b1`), which is the normal cadence this doc documents: pre-releases are never Latest, so `/releases/latest` correctly names an older stable while the check reads as failed. One-word fix: "newest v2 tag" β "newest stable v2 release", matching the pre-release section's own wording.
Check notice on line 8 in RELEASE.md
claude / Claude Code Review
Stale dependency-bump step: uv lock --resolution lowest-direct would mass-downgrade the committed lockfile
Pre-existing issue (this line is unchanged by the PR): step 2 of "Bumping Dependencies" says "Upgrade lock with `uv lock --resolution lowest-direct`", but that command doesn't upgrade anything β it re-resolves every direct dependency down to its declared floor, mass-downgrading the committed `uv.lock` (pydantic 2.12.5 β 2.12.0, typer 0.17.4 β 0.16.0, β¦) and persisting `resolution-mode = "lowest-direct"` so later plain `uv lock` runs silently stay in that mode. Since this PR's purpose is making R
Loading