Skip to content

Claude PR review: drop --max-turns, scope tools instead#198

Merged
bnaul merged 6 commits into
original-directionfrom
add-claude-pr-review
May 29, 2026
Merged

Claude PR review: drop --max-turns, scope tools instead#198
bnaul merged 6 commits into
original-directionfrom
add-claude-pr-review

Conversation

@bnaul

@bnaul bnaul commented May 29, 2026

Copy link
Copy Markdown

The review on #194 failed with error_max_turns (--max-turns 15 was exhausted at turn 16). Following the proven config in model/.github/workflows/claude-code-pr-review.yml, this removes the turn cap and instead scopes the agent to diff-focused review tools:

--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"

Must merge before #194's review will pass validation (head workflow must match default branch).

🤖 Generated with Claude Code

bnaul and others added 6 commits May 29, 2026 07:13
Reviews every PR (opened/synchronize/reopened) via
anthropics/claude-code-action@v1, authenticated with the org-level
ANTHROPIC_API_KEY secret. Prompt is tuned for this gRPC routing
service (concurrency, hot-path perf, correctness).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Only review on PR open/reopen, not on every push to the branch.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Lets marking a draft PR ready-for-review kick off a review, in
addition to opened/reopened.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…ude-pr-review

# Conflicts:
#	.github/workflows/claude-code-review.yml
15 turns was too low — reviews exhausted it (error_max_turns at 16).
Match the proven config from model/.github: no turn cap, bound the
agent to diff-focused review tools instead.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@bnaul bnaul merged commit 5eb4fbe into original-direction May 29, 2026
1 check passed
@bnaul bnaul deleted the add-claude-pr-review branch May 29, 2026 13:43
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