Skip to content

Commit eeaf833

Browse files
fix(pull-request): enforce reply before resolve + cleanup outdated threads
Perfect mode now includes a final cleanup step that replies to all outdated threads without a reply. Section 6c reinforced: never resolve a thread silently, always reply with SHA as audit trail.
1 parent 492ed89 commit eeaf833

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.claude/skills/pull-request/SKILL.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,11 +221,17 @@ REPEAT:
221221
3. Run diff audit → audit `git diff master...HEAD` for security, logic bugs,
222222
data integrity, API design, performance issues
223223
4. If audit has findings → fix all, commit, push, GOTO 1
224-
5. If 0 CodeRabbit comments AND 0 audit findings → STOP ✓
224+
5. If 0 CodeRabbit comments AND 0 audit findings → run cleanup (step 6), then STOP ✓
225+
6. Cleanup outdated threads → list all threads with isOutdated==true that have NO reply
226+
explaining the fix. For each, reply with the SHA and a brief
227+
explanation of how/why it was addressed, then resolve if not
228+
already resolved.
225229
```
226230

227231
**Safety limit:** 5 outer iterations max — report to user if still not converged.
228232

233+
**Important:** In section 6c, you MUST reply to every thread before resolving it. Never resolve a thread silently — the reply serves as audit trail.
234+
229235
## 8. Conflict resolution
230236

231237
```bash

0 commit comments

Comments
 (0)