Imported from Zhavoronkov24/agents_dev (
.opencode/skills/code-review/SKILL.md). Install upstream withnpx skills add Zhavoronkov24/agents_dev --skill code-review. Copyright stays with the author.
Код-ревью
Порядок
- Контекст: spec.md (AC), plan.md, tasks.md,
git log --oneline develop..HEAD(для hotfix —main..HEAD). - Дифф:
git diff develop...HEAD --stat, затем по файлам. - Прогони тесты и линтер сам — не верь на слово.
- Пройди чек-листы review.md и запиши замечания.
На что смотреть
- Спецификация: каждая AC реализована и покрыта тестом; нет функциональности сверх spec.
- Корректность: граничные случаи из spec §6, ошибки, конкуренция, транзакции, идемпотентность.
- Тесты: проверяют поведение, а не реализацию; нет отключённых/ослабленных тестов.
- Безопасность: валидация входа, инъекции, авторизация, секреты, персональные данные в логах.
- Наблюдаемость: логи с контекстом, метрики для новых сценариев.
- Сопровождаемость: понятные имена, нет дублирования и мёртвого кода, соответствие стилю проекта.
- Контракты: обратная совместимость, версии, миграции с откатом.
Замечание
R-n | blocker/major/minor/nit | файл:строка | что не так и почему | как исправить.
Вердикт
- Есть blocker/major →
Verdict: CHANGES_REQUESTED, затемflow.py back implement --reason "R-1, R-4"(bugfix:back fix). - Иначе →
Verdict: APPROVED,flow.py check→advance(→ verify, владелец qa). - Коммит:
docs(<id>): ревью — <вердикт>.
