Skip to content

Commit 30f68a3

Browse files
committed
fix(ci): simplify PR review workflow to use default tools
Removed custom --allowedTools that were causing errors. The MCP tools like mcp__github_inline_comment__create_inline_comment are not available in the claude-code-action context.
1 parent 2e07ee7 commit 30f68a3

1 file changed

Lines changed: 13 additions & 61 deletions

File tree

.github/workflows/pr-review-comprehensive.yml

Lines changed: 13 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -21,85 +21,37 @@ jobs:
2121
uses: anthropics/claude-code-action@v1
2222
with:
2323
anthropic_api_key: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
24-
2524
track_progress: true
2625

2726
prompt: |
2827
REPO: ${{ github.repository }}
2928
PR NUMBER: ${{ github.event.pull_request.number }}
3029
31-
Perform a comprehensive code review with the following focus areas:
30+
Perform a code review with the following focus areas:
3231
3332
## 1. Release Workflow Compliance (CRITICAL)
3433
35-
This project uses Release Please with Conventional Commits. Check:
34+
This project uses Release Please with Conventional Commits.
3635
37-
**Commit Message Format:**
38-
- All commits MUST follow conventional commit format: `type(scope): description`
36+
**Check commit messages follow this format:** `type(scope): description`
3937
- Valid types: `feat`, `fix`, `perf`, `refactor`, `docs`, `test`, `ci`, `build`, `chore`, `style`
4038
- Type must be lowercase
41-
- Description must be lowercase and not end with a period
42-
- Breaking changes must include `BREAKING CHANGE:` in the commit footer
4339
44-
**Version Bump Rules (explain to contributor):**
40+
**Version Bump Rules:**
4541
- `feat:` → Minor version bump (0.4.0 → 0.5.0)
46-
- `fix:`, `perf:`, `refactor:` → Patch version bump (0.4.0 → 0.4.1)
42+
- `fix:`, `perf:`, `refactor:` → Patch version bump
4743
- `docs:`, `test:`, `ci:`, `build:`, `chore:`, `style:` → No version bump
48-
- `BREAKING CHANGE:` footer → Major version bump
49-
50-
**If commits don't follow the format:**
51-
- List each non-compliant commit
52-
- Show the correct format it should use
53-
- Suggest squashing/rewording before merge
54-
55-
Run this command to check commits:
56-
```
57-
git log --oneline origin/main..HEAD
58-
```
5944
6045
## 2. Code Quality
61-
- Clean code principles and best practices
62-
- Proper error handling and edge cases
63-
- Code readability and maintainability
46+
- Clean code principles
47+
- Proper error handling
48+
- Code readability
6449
6550
## 3. Security
66-
- Check for potential security vulnerabilities
67-
- Validate input sanitization
68-
- Review authentication/authorization logic
69-
- Check for secrets or credentials in code
70-
71-
## 4. Performance
72-
- Identify potential performance bottlenecks
73-
- Check for memory leaks or resource issues
74-
75-
## 5. Testing
76-
- Verify adequate test coverage for new features
77-
- Review test quality and edge cases
78-
79-
## 6. Documentation
80-
- New features should update README.md if user-facing
81-
- Breaking changes should be clearly documented
82-
83-
## Review Output Format
84-
85-
Start your review with a **Release Compliance Summary**:
86-
87-
```
88-
## Release Compliance Check
89-
90-
| Commit | Format | Type | Valid |
91-
|--------|--------|------|-------|
92-
| abc123 | feat: add feature | feat | ✅ |
93-
| def456 | Update readme | INVALID | ❌ |
94-
95-
**Action Required:** [Yes/No]
96-
```
97-
98-
If any commits are invalid, provide clear instructions on how to fix them
99-
(e.g., interactive rebase commands).
51+
- Check for vulnerabilities
52+
- No secrets in code
10053
101-
Then proceed with the rest of the code review using inline comments
102-
for specific issues and top-level comments for general observations.
54+
## Review Format
10355
104-
claude_args: |
105-
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(git log:*),Bash(git show:*)"
56+
Start with a **Release Compliance Summary** table showing each commit's validity.
57+
Then provide code review feedback.

0 commit comments

Comments
 (0)