Skip to content

Improve PR review: exhaustive multi-turn review with unified submission #62

Description

@mega-putin

Problem

The current pr-review job in .github/workflows/claude.yml runs Claude with no --max-turns, so it typically does a single-pass review and submits findings immediately. This means:

  • Claude may miss issues that a deeper multi-pass review would catch
  • Comments are submitted incrementally rather than as one cohesive review
  • The 15-minute timeout is tight for thorough review

Proposed Changes

1. Add --max-turns for multi-turn agentic loop

--max-turns 20

This lets Claude do multiple passes (scope → checklist → content quality → thread resolution) before submitting.

2. Update prompt to enforce exhaustive review + unified submission

Replace the current prompt with a phased review process:

  1. First pass — understand full diff scope
  2. Second pass — check each file against REVIEW.md checklist
  3. Third pass — content quality against layer-specific AGENTS.md
  4. Fourth pass — review previous threads, resolve addressed ones
  5. Submit — ONE unified review with gh pr comment + inline comments

3. Increase timeout

timeout-minutes: 25

To accommodate the multi-pass review.

Note

/loop (interactive slash command) is not available in CI — --max-turns is the correct print-mode equivalent.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions