Skip to content

Commit 5e3016a

Browse files
committed
improve claude yml
1 parent e628395 commit 5e3016a

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

.github/workflows/claude.yml

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,18 @@ jobs:
5151
- Example: To get the first 100 lines, call `mcp__github__get_pull_request_diff` with `per_page=100` and `page=1`. For the next 100 lines, use `page=2`, and so on.
5252
- Continue fetching pages until all changes are retrieved.
5353
3. Add inline comments: Use `mcp__github__add_comment_to_pending_review` to provide feedback on specific lines of code snippets.
54+
- **When suggesting code changes, format them as GitHub suggestion blocks so they can be committed directly:**
55+
- For single-line suggestions, use:
56+
```suggestion
57+
suggested code here
58+
```
59+
- For multi-line suggestions, use:
60+
```suggestion:-0+1
61+
suggested code here
62+
```
63+
(Adjust the numbers to match how many lines to remove and add)
64+
- Only provide suggestion blocks when you have a concrete, actionable fix
65+
- Include explanation text before or after the suggestion block
5466
4. Submit for review: Use `mcp__github__submit_pending_pull_request_review` with the event type set to "COMMENT" (not "REQUEST_CHANGES") to publish all comments for non-blocking review.
5567
5668
Focus on the following focus areas:
@@ -59,16 +71,19 @@ jobs:
5971
- Clean code principles and best practices
6072
- Proper error handling and edge cases
6173
- Code readability and maintainability
74+
- **Provide suggestion blocks for improvements when possible**
6275
6376
2. **Security**
6477
- Check for potential security vulnerabilities
6578
- Validate input sanitization
6679
- Review authentication/authorization logic
80+
- **Suggest secure alternatives with suggestion blocks**
6781
6882
3. **Performance**
6983
- Identify potential performance bottlenecks
7084
- Review database queries for efficiency
7185
- Check for memory leaks or resource issues
86+
- **Offer optimized code via suggestion blocks**
7287
7388
4. **Testing**
7489
- Verify adequate test coverage
@@ -81,6 +96,7 @@ jobs:
8196
- Check API documentation accuracy
8297
8398
Provide detailed feedback using inline comments for specific issues.
99+
Use suggestion blocks liberally to make it easy for developers to apply fixes.
84100
Use top-level comments for general observations or praise.
85101
86102
# Tools for comprehensive PR review
@@ -91,4 +107,4 @@ jobs:
91107
# - Creates a tracking comment with progress checkboxes
92108
# - Includes all PR context (comments, attachments, images)
93109
# - Updates progress as the review proceeds
94-
# - Marks as completed when done
110+
# - Marks as completed when done

0 commit comments

Comments
 (0)