Skip to content

Commit a45d6e3

Browse files
authored
Merge pull request openvanilla#710 from openvanilla/ci/concise_claude_review
ci: make Claude Code review concise and focused
2 parents dd35dd2 + 3934b75 commit a45d6e3

1 file changed

Lines changed: 25 additions & 10 deletions

File tree

.github/workflows/claude-code-review.yml

Lines changed: 25 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@ name: Claude Code Review
22

33
on:
44
pull_request:
5-
types: [opened, synchronize]
5+
types: [opened, synchronize, reopened]
6+
paths-ignore:
7+
- '.github/workflows/claude-code-review.yml'
68
# Optional: Only run on specific file changes
79
# paths:
810
# - "src/**/*.ts"
@@ -40,18 +42,31 @@ jobs:
4042
REPO: ${{ github.repository }}
4143
PR NUMBER: ${{ github.event.pull_request.number }}
4244
43-
Please review this pull request and provide feedback on:
44-
- Code quality and best practices
45-
- Potential bugs or issues
46-
- Performance considerations
47-
- Security concerns
48-
- Test coverage
45+
Please review this PR focusing ONLY on critical issues. Use the repository's CLAUDE.md for conventions.
4946
50-
Use the repository's CLAUDE.md for guidance on style and conventions. Be constructive and helpful in your feedback.
47+
Review for:
48+
- Potential bugs or logic errors
49+
- Security vulnerabilities
50+
- Performance problems
51+
- Correctness issues
52+
- Violations of repository coding standards
5153
52-
Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR.
54+
Keep your response concise:
55+
- Maximum 10 bullet points in summary
56+
- Use inline comments with code suggestions for specific fixes
57+
- Group similar issues together to avoid repetition
58+
- Skip style/formatting nitpicks unless they impact security or performance
59+
- No general observations, praise, or minor suggestions
60+
61+
Format:
62+
- Start with 1-2 sentence overview
63+
- List only critical/high-priority issues as bullet points
64+
- For each issue: what, where, why it matters, suggested fix
65+
- End with sign-off: APPROVED or ISSUES FOUND
66+
67+
Use `gh pr comment` with your Bash tool to leave your concise review as a comment on the PR.
5368
5469
# See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
5570
# or https://docs.claude.com/en/docs/claude-code/cli-reference for available options
56-
claude_args: '--allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'
71+
claude_args: '--allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)" --max-turns 5'
5772

0 commit comments

Comments
 (0)