Skip to content

Commit 9a82ddd

Browse files
committed
chore: update Claude code review workflow configuration
Enhance the Claude code review workflow by allowing any bot to trigger reviews and adding accessibility to the review criteria. Adjust the feedback prompt to emphasize conciseness and constructiveness. Clean up unnecessary whitespace for better readability.
1 parent 2571196 commit 9a82ddd

File tree

1 file changed

+14
-11
lines changed

1 file changed

+14
-11
lines changed

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

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,14 @@ jobs:
1717
# github.event.pull_request.user.login == 'external-contributor' ||
1818
# github.event.pull_request.user.login == 'new-developer' ||
1919
# github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'
20-
20+
2121
runs-on: ubuntu-latest
2222
permissions:
2323
contents: read
2424
pull-requests: read
2525
issues: read
2626
id-token: write
27-
27+
2828
steps:
2929
- name: Checkout repository
3030
uses: actions/checkout@v4
@@ -39,40 +39,43 @@ jobs:
3939

4040
# Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
4141
# model: "claude-opus-4-20250514"
42-
42+
43+
# Allow any bot to trigger the review
44+
allowed_bots: '*'
45+
4346
# Direct prompt for automated review (no @claude mention needed)
4447
direct_prompt: |
4548
Please review this pull request and provide feedback on:
4649
- Code quality and best practices
50+
- Accessibility
4751
- Potential bugs or issues
4852
- Performance considerations
4953
- Security concerns
5054
- Test coverage
51-
52-
Be constructive and helpful in your feedback.
55+
56+
Be concise and constructive in your feedback.
5357
5458
# Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
5559
# use_sticky_comment: true
56-
60+
5761
# Optional: Customize review based on file types
5862
# direct_prompt: |
5963
# Review this PR focusing on:
6064
# - For TypeScript files: Type safety and proper interface usage
6165
# - For API endpoints: Security, input validation, and error handling
6266
# - For React components: Performance, accessibility, and best practices
6367
# - For tests: Coverage, edge cases, and test quality
64-
68+
6569
# Optional: Different prompts for different authors
6670
# direct_prompt: |
67-
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
71+
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
6872
# 'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
6973
# 'Please provide a thorough code review focusing on our coding standards and best practices.' }}
70-
74+
7175
# Optional: Add specific tools for running tests or linting
7276
# allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"
73-
77+
7478
# Optional: Skip review for certain conditions
7579
# if: |
7680
# !contains(github.event.pull_request.title, '[skip-review]') &&
7781
# !contains(github.event.pull_request.title, '[WIP]')
78-

0 commit comments

Comments
 (0)