Skip to content

Commit e8d87ff

Browse files
committed
minor tweaks to pr-review command
1 parent e863cf4 commit e8d87ff

1 file changed

Lines changed: 4 additions & 9 deletions

File tree

.claude/commands/pr-review.md

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -125,19 +125,14 @@ Skip any section in the output that has nothing notable.
125125
- <correctness, security, or convention violations. Each: path:line, what's wrong, why it breaks, suggested fix. Prefix VERY IMPORTANT: / MUST: when warranted.>
126126
127127
## Suggestions
128-
- <non-blocking improvements, grouped by area if many>
128+
- <non-blocking improvements>
129129
130130
## Nits
131131
- <style, naming, tiny cleanups — prefix `nit:`>
132+
```
132133

133-
## Tests
134-
- <coverage gaps, redundant tests, missing regression tests>
135-
136-
## Migrations
137-
- <only if alembic/versions/ touched: rev id, ordering, indexes, nullability, downgrade reversibility, backfill correctness>
134+
Each item gets exactly one bullet — no item appears in more than one section. Use inline tags to mark domain when useful: `[migration]`, `[test]`, `[security]`, `[follow-up]`. Severity drives the section; the tag adds the domain colour.
138135

139-
## Follow-ups (not for this PR)
140-
- <cross-cutting refactors, consistency sweeps, scope-creep items the author should track separately>
141-
```
136+
Use `[follow-up]` for cross-cutting work the author should track separately rather than fix in this PR; place it in `Suggestions`.
142137

143138
Drop empty sections. Don't pad. Do not modify files during the review — read-only.

0 commit comments

Comments
 (0)