@@ -3,48 +3,55 @@ name: Claude Code Review
33on :
44 pull_request :
55 types : [opened, synchronize]
6+ # Optional: Only run on specific file changes
7+ # paths:
8+ # - "src/**/*.ts"
9+ # - "src/**/*.tsx"
10+ # - "src/**/*.js"
11+ # - "src/**/*.jsx"
612
713jobs :
814 claude-review :
15+ # Optional: Filter by PR author
16+ # if: |
17+ # github.event.pull_request.user.login == 'external-contributor' ||
18+ # github.event.pull_request.user.login == 'new-developer' ||
19+ # github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'
20+
921 runs-on : ubuntu-latest
1022 permissions :
1123 contents : read
12- pull-requests : write
24+ pull-requests : read
1325 issues : read
1426 id-token : write
1527
1628 steps :
1729 - name : Checkout repository
1830 uses : actions/checkout@v4
1931 with :
20- fetch-depth : 0
32+ fetch-depth : 1
2133
2234 - name : Run Claude Code Review
2335 id : claude-review
2436 uses : anthropics/claude-code-action@v1
2537 with :
2638 claude_code_oauth_token : ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
2739 prompt : |
28- Review PR #${{ github.event.pull_request.number }} in ${{ github.repository }}.
29-
30- This is an MCP (Model Context Protocol) server for the Coolify API. Focus on:
31-
32- **Critical Issues:**
33- - Security vulnerabilities (exposed tokens, injection attacks, unsafe data handling)
34- - Breaking changes to the MCP tool interface
35- - Bugs that would cause runtime errors
40+ REPO: ${{ github.repository }}
41+ PR NUMBER: ${{ github.event.pull_request.number }}
3642
37- **Code Quality:**
38- - TypeScript type safety - ensure proper typing, avoid `any`
39- - Error handling - all API calls should have proper error handling
40- - Consistency with existing patterns in coolify-client.ts and mcp-server.ts
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
4149
42- **Testing:**
43- - New functionality should have corresponding tests
44- - Tests should cover edge cases and error conditions
50+ Use the repository's CLAUDE.md for guidance on style and conventions. Be constructive and helpful in your feedback.
4551
46- Keep feedback concise and actionable. Skip minor style issues that Prettier/ESLint will catch .
52+ Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR .
4753
48- Use `gh pr comment` to post your review. If no critical issues, say "✅ LGTM - No critical issues found."
54+ # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
55+ # or https://code.claude.com/docs/en/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:*)"'
4957
50- claude_args : ' --allowed-tools "Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Read,Glob,Grep"'
0 commit comments