Skip to content

Commit 98597c3

Browse files
authored
ci: Update claude review workflow (#91)
1 parent 489f3a0 commit 98597c3

File tree

1 file changed

+15
-13
lines changed

1 file changed

+15
-13
lines changed

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

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -42,19 +42,27 @@ jobs:
4242

4343
# Direct prompt for automated review (no @claude mention needed)
4444
direct_prompt: |
45-
Please review this pull request and provide feedback on:
46-
- Code quality and best practices
47-
- Potential bugs or issues
48-
- Performance considerations
49-
- Security concerns
50-
- Test coverage
45+
Important, ensure:
46+
- PR includes changeset when updating package source
47+
- PR includes documentation updates when API surface changes
48+
- Code updates are clean and well architected
49+
- Tests cover relevant code changes
50+
51+
Provide feedback on:
52+
- Code quality and best practices - readability, maintainability, elegance
53+
- Potential bugs, performance considerations, security concerns
5154
5255
Be constructive and helpful in your feedback.
5356
57+
${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
58+
'
59+
60+
Welcome! This is from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' || '' }}
61+
5462
allowed_bots: "renovate"
5563

5664
# Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
57-
# use_sticky_comment: true
65+
use_sticky_comment: true
5866

5967
# Optional: Customize review based on file types
6068
# direct_prompt: |
@@ -64,12 +72,6 @@ jobs:
6472
# - For React components: Performance, accessibility, and best practices
6573
# - For tests: Coverage, edge cases, and test quality
6674

67-
# Optional: Different prompts for different authors
68-
# direct_prompt: |
69-
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
70-
# 'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
71-
# 'Please provide a thorough code review focusing on our coding standards and best practices.' }}
72-
7375
# Optional: Add specific tools for running tests or linting
7476
# allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"
7577

0 commit comments

Comments
 (0)