Skip to content

Commit fe61bd0

Browse files
committed
Allow the the code-review Claude command to update and resolve existing PR comments
1 parent 3ed6961 commit fe61bd0

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

.claude/commands/code-review.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ To do this, follow these steps precisely:
1515
- The pull request is closed
1616
- The pull request is a draft
1717
- The pull request does not need code review (e.g. automated PR, trivial change that is obviously correct)
18-
- Claude has already commented on this PR (check `gh pr view <PR> --comments` for comments left by claude)
1918

2019
If any condition is true, stop and do not proceed.
2120

@@ -73,9 +72,12 @@ Note: Still review Claude generated PR's.
7372
- For small, self-contained fixes, include a committable suggestion block
7473
- For larger fixes (6+ lines, structural changes, or changes spanning multiple locations), describe the issue and suggested fix without a suggestion block
7574
- Never post a committable suggestion UNLESS committing the suggestion fixes the issue entirely. If follow up steps are required, do not leave a committable suggestion.
75+
- If there's already a previous inline comment for the issue, update it instead of creating a new inline comment.
7676

7777
**IMPORTANT: Only post ONE comment per unique issue. Do not post duplicate comments.**
7878

79+
10. Check for previous comments from Claude about issues that are not present anymore and resolve the conversation.
80+
7981
Use this list when evaluating issues in Steps 4 and 5 (these are false positives, do NOT flag):
8082

8183
- Pre-existing issues
@@ -106,4 +108,4 @@ No issues found. Checked for bugs and CLAUDE.md compliance.
106108
- Repo name must match the repo you're code reviewing
107109
- # sign after the file name
108110
- Line range format is L[start]-L[end]
109-
- Provide at least 1 line of context before and after, centered on the line you are commenting about (eg. if you are commenting about lines 5-6, you should link to `L4-7`)
111+
- Provide at least 1 line of context before and after, centered on the line you are commenting about (eg. if you are commenting about lines 5-6, you should link to `L4-7`)

0 commit comments

Comments
 (0)