---
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.