Skip to content

Run Claude review automatically on every PR - #47

Merged
jlocke2 merged 2 commits into
masterfrom
agentic-readiness/auto-review
Jul 20, 2026
Merged

Run Claude review automatically on every PR#47
jlocke2 merged 2 commits into
masterfrom
agentic-readiness/auto-review

Conversation

@jlocke2

@jlocke2 jlocke2 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Adds .github/workflows/claude-review.yml so Claude reviews every PR automatically.

  • Auto-review triggers when a PR is opened or marked ready for review; draft PRs are excluded
  • Mention @claude in a PR comment to trigger a re-review
  • Uses the org-level ANTHROPIC_API_KEY secret (already visible to this repo)
  • Review prompt is adapted for an API client gem: bugs, breaking public-API changes, HTTP-in-loops, CLAUDE.md conventions, test coverage (tenant-isolation/Sidekiq checks dropped since they do not apply)
  • Each run is capped at 12 turns / $0.75

Part of the agentic-readiness work across Pipeline CRM Ruby repos.

🤖 Generated with Claude Code

Update: review runs pinned to Sonnet — cheaper per run, and the reviewing model differs from the model that authors most changes.

jlocke2 and others added 2 commits July 17, 2026 09:28
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jlocke2
jlocke2 requested a review from sevgibson July 17, 2026 15:14

@sevgibson sevgibson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — uses the safe pull_request trigger (not pull_request_target, so fork PRs get no secrets), contents: read, and budget/turn caps.

@jlocke2
jlocke2 merged commit 3c0b04f into master Jul 20, 2026
2 checks passed
@jlocke2
jlocke2 deleted the agentic-readiness/auto-review branch July 20, 2026 01:06
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.

2 participants