Skip to content

Commit 062393c

Browse files
drnclaude
andcommitted
Improve rereview and polish skills from session handoffs.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c29158b commit 062393c

File tree

3 files changed

+18
-0
lines changed

3 files changed

+18
-0
lines changed

agents/skills/polish/SKILL.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ CURRENT FILE CONTENTS:
177177
{full contents of files being modified}
178178
179179
RULES -- follow these strictly:
180+
0. Before splitting files, check `git log --oneline -5 {file}` for recent activity on the base branch. If the file was modified in the last 3 commits on the base branch, warn the user that a rebase will likely conflict and suggest completing the split in a fast follow-up PR to minimize conflict surface.
180181
1. One refactoring at a time. Do not combine unrelated changes.
181182
2. Preserve all existing behavior. Refactoring changes structure, not behavior.
182183
3. Follow existing code style and conventions in the project.

agents/skills/polish/references/ANALYZERS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,7 @@ You are the GO IDIOM SPECIALIST. Check every item:
230230
- [ ] Context not propagated through call chain
231231
- [ ] Missing defer for resource cleanup
232232
- [ ] Goroutine leaks (goroutines without shutdown mechanism)
233+
- [ ] Bubble Tea View() methods with multiple switch/if blocks dispatching on the same state variable — consolidate into a single switch
233234
234235
For each finding, report file, line, current code, idiomatic alternative, severity, and concrete benefit.
235236

agents/skills/rereview/SKILL.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,22 @@ Use `subagent_type="general-purpose"` and `model: "sonnet"` for all reviewers.
9797

9898
Each Task call should use the reviewer briefing template below, with only the reviewer name changed (ALPHA, BRAVO, CHARLIE).
9999

100+
### Large Diff Handling
101+
102+
If the diff exceeds 30KB, save it to a temp file instead of passing it inline:
103+
104+
```bash
105+
git diff {base}...HEAD > /tmp/review-diff-{branch}.txt
106+
```
107+
108+
Then replace the `FULL DIFF` section in the briefing with:
109+
110+
```
111+
FULL DIFF: Read the diff from /tmp/review-diff-{branch}.txt
112+
```
113+
114+
This prevents context window overflow in reviewer agents.
115+
100116
### Reviewer Briefing Template
101117

102118
```

0 commit comments

Comments
 (0)