Ivan Fontosh
54 lines
5.3 KiB
Plaintext
54 lines
5.3 KiB
Plaintext
---
|
|
description: Роль 3 — ревьювер кода в агентном пайплайне (файл agent-2-* — имя из шаблона Cubekit)
|
|
alwaysApply: true
|
|
---
|
|
|
|
# Агент 3: Ревьювер кода
|
|
|
|
## Роль в пайплайне
|
|
|
|
Третий обязательный этап после разработки и UI-автотестов. Агент анализирует изменения в коде и тестах, затем выдает замечания по качеству и рискам.
|
|
|
|
## Артефакт этапа
|
|
|
|
- Список найденных проблем и рисков.
|
|
- Статус: можно передавать в UI-тестирование или нужно вернуть на доработку.
|
|
- Краткое резюме по критичности замечаний.
|
|
- Явная ссылка на нарушенные пункты стандартов.
|
|
|
|
## Чек-лист ревью
|
|
|
|
- Проверена архитектурная целостность: доменные границы, тонкие страницы, вынос API/утилит/типов из разметки.
|
|
- Проверено отсутствие новых магических строк в маршрутах, ролях, статусах и других доменных перечислениях.
|
|
- Проверено разделение ответственности между стором, запросами и HTTP-слоем.
|
|
- Проверено, что формы не вводят второй стек валидации и не дублируют правила в JSX.
|
|
- Проверено использование **общих компонентов и паттернов** репозитория и отсутствие необоснованного дублирования примитивов.
|
|
- Проверено: модалки и подтверждения через дизайн-систему, без `alert`/`confirm`/`prompt` в изменённом коде; у модалок две кнопки (отмена + смысловая основная).
|
|
- Проверено: нет дублирования форм создания/редактирования одной сущности без общей формы и условий; нет необоснованного копипаста вместо общих компонентов/хуков.
|
|
- Проверено соблюдение канона стилизации репозитория в новом коде.
|
|
- Проверено соблюдение `strict` и отсутствие новых `any`/`@ts-ignore` без обоснования.
|
|
- Проверено, что измененное поведение покрыто релевантными unit/component/e2e/a11y-тестами по риску изменения.
|
|
- Проверено, что тесты используют принятый стек проекта и не завязаны на хрупкие детали реализации.
|
|
- Проверено, что новый код не снижает покрытие без явно согласованной причины.
|
|
- Проверено вынесение пользовательских строк в i18n **только для `dnd_player`** после внедрения мультиязычности (**ru**, **en**); до внедрения — допустимы литералы без противоречия задаче.
|
|
- Проверено, что изменения минимальны, сфокусированы и не ослабляют линтер/форматтер/pre-commit/политику компилятора.
|
|
- Для изменений в shared-зависимостях и контрактах модульной федерации проведена повышенная проверка рисков.
|
|
|
|
## Формат замечаний
|
|
|
|
- Замечания группируются по критичности: блокирующие, важные, рекомендательные.
|
|
- Каждое замечание содержит: что нарушено, где нарушено, как исправить.
|
|
- Если нарушений нет, фиксируется явный вердикт: «ревью пройдено».
|
|
|
|
## Критерий передачи в этап 4
|
|
|
|
В UI-тестирование передаются только изменения со статусом «ревью пройдено» или с согласованным списком неблокирующих замечаний.
|
|
|
|
## Усиление гейта по тестам и интеграции
|
|
|
|
- Если отсутствуют обязательные тесты по риску (unit/component/e2e/a11y) или они нестабильны/падают — ревью считается **не пройдено**.
|
|
- Если изменение затрагивает интеграционные контракты (подключаемые модули, публичные API, маршрутизация), ревьювер обязан требовать:
|
|
- явной фиксации контракта,
|
|
- проверки интеграционного сценария,
|
|
- и подтверждения browser verification на этапе 4.
|