Skip to content

ci: adopt ai-review-prompts #90 cost gates + ready_for_review trigger - #218

Merged
heskew merged 3 commits into
mainfrom
bump-ai-review-prompts-pin
Aug 26, 2026
Merged

heskew merged 3 commits into
mainfrom
bump-ai-review-prompts-pin

Conversation

@heskew

@heskew heskew commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Advances the pins #216 landed (4632c5d → be549ad) to pick up ai-review-prompts#90's cost gates — draft PRs skip review until flipped ready (the claude-review/gemini-review label opts a draft in), mechanical diffs (lockfile-only, version bumps) skip before the agent runs, reasoning effort scales with diff size (≤60 lines → high, ≤1500 → xhigh, larger → max), and synchronize runs debounce 120s. Adds ready_for_review to both review callers' trigger types so flipping ready fires the review.

Merge before #217 (the v1.x sync now matches this content).

🤖 Generated with Claude Code

https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt

heskew and others added 2 commits August 25, 2026 16:40
Bump every ai-review-prompts pin to 4632c5d: claude-sonnet-5 model,
--effort xhigh, 96-turn review budget (issue-to-pr 100), 30m timeouts
(#89), the sibling-implementations and cannot-fail-test lenses (#88),
and the week-of-08-10/08-17 calibration prompt edits (#85/#87).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt
…review trigger

Draft PRs skip review until flipped ready (label still opts one in),
mechanical diffs skip pre-run, reasoning effort scales with diff size
(60/high, 1500/xhigh, else max), synchronize runs debounce 120s.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S94XethbGXpAb4DRKMD4kt
@heskew
heskew requested a review from a team as a code owner August 26, 2026 02:07
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions

Copy link
Copy Markdown
Contributor

Reviewed; no blockers found.

@heskew
heskew merged commit 1933e5e into main Aug 26, 2026
11 of 12 checks passed
@heskew
heskew deleted the bump-ai-review-prompts-pin branch August 26, 2026 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant