Skip to content

Add review depth modes (quick/default/thorough) - #7

Merged
lionuncle merged 2 commits into
mainfrom
feat/review-depth-modes
Apr 10, 2026
Merged

Add review depth modes (quick/default/thorough)#7
lionuncle merged 2 commits into
mainfrom
feat/review-depth-modes

Conversation

@lionuncle

Copy link
Copy Markdown
Contributor

Summary

  • Adds --depth/-d flag with three review modes:
    • quick — blockers only: production bugs, security vulnerabilities, data loss risks, major architectural violations. Skips nits, style, and minor edge cases. Ideal for fast approvals on low-risk PRs.
    • default — standard review behavior (unchanged from today).
    • thorough — deep analysis: architecture fit, edge cases, naming clarity, API design, testability, performance at scale. For critical code paths and complex changes.
  • Depth instructions are injected into the system prompt so they shape the entire review, not just a checklist.
  • Status line shows the active depth when non-default (e.g. 🤖 Reviewing with claude-opus-4-6 [quick]...).

Usage

rpr 42 --depth quick       # or -d quick
rpr 42 --depth thorough    # or -d thorough
rpr 42                     # default (unchanged)

Test plan

  • rpr <pr> -d quick --dry-run — verify review only flags critical issues, approves clean code
  • rpr <pr> -d thorough --dry-run — verify review covers architecture, naming, scale concerns
  • rpr <pr> --dry-run — verify default behavior is unchanged
  • rpr <pr> -d invalid — verify argparse rejects unknown depths

🤖 Generated with Claude Code

lionuncle and others added 2 commits April 10, 2026 17:08
Adds --depth/-d flag with three modes:
- quick: blockers only — bugs, security, data loss, major arch issues
- default: standard review (unchanged behavior)
- thorough: deep analysis including architecture, naming, API design,
  testability, and scale considerations

The depth instructions are injected into the system prompt so they
shape the entire review persona, not just the focus areas.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… vars

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lionuncle
lionuncle merged commit a34a769 into main Apr 10, 2026
8 checks passed
@lionuncle
lionuncle deleted the feat/review-depth-modes branch April 15, 2026 12:45
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