name: review-code description: >- Ревью кода на корректность: баги, регрессии, потерянные инварианты в diff или заданном scope; подтверждённые находки исправляются. Триггеры: «сделай ревью кода», «проверь diff на баги», «code review перед коммитом».
review-code
Ревью на корректность (баги), не на стиль. Scope по умолчанию — текущий
незакоммиченный diff (git diff HEAD) + незапушенные коммиты; аргументом можно
передать каталог/фичу.
Шаги
- Собери scope:
git diff HEAD(+git log origin/main..HEAD). Пустой diff → спроси scope или возьми последний коммит. - Найди кандидатов — делегируй агенту
review-auditor(или нескольким параллельно при большом scope) с углами:- line-by-line: инвертированные условия, off-by-one, null/undefined, пропущенный
await, falsy-zero, copy-paste не той переменной, проглоченные ошибки; - removed-behavior: для каждой удалённой строки — какой инвариант она держала и где он теперь;
- cross-file: вызовы изменённых функций по всему репо (новые предусловия, изменённая форма результата, web/native расхождения).
- line-by-line: инвертированные условия, off-by-one, null/undefined, пропущенный
- Верифицируй каждый кандидат чтением кода: REFUTED отбрасывай, остальное чини.
- Чини и проверяй: фикс + Jest на затронутый scope +
npm run typecheck+ ESLint по затронутым файлам. Поведенческие изменения контракта — фиксируй обновлением теста. - Отчёт: находка → severity → что сделано (или почему скип).
Правила
- Не репортить стиль и невозможные сценарии (правило CLAUDE.md).
- Файлы бэкенда не править — тикет на общем MCP task board (
area=back, черезticket-board). - Не отмечать «готово» без зелёных проверок.