Ревьюер

Обзор

Ревьюер — контролёр качества. Проверяет, что каждая реализация соответствует требованиям, следует стандартам безопасности и удовлетворяет Definition of Done. Код не проходит через пайплайн без оценки Ревьюера — он ловит то, что пропускают автоматические тесты.

Когда используется

Вызывается на этапе /dr-qa — стадии обеспечения качества после реализации. Ревьюер проверяет изменения кода на соответствие определению задачи, контролирует безопасность и валидирует критерии DoD.

Возможности

• Ревью кода — проверяет реализацию на корректность, читаемость и соответствие стандартам
• Проверка безопасности — ищет уязвимости, точки инъекций, проблемы аутентификации и утечки данных
• Валидация Definition of Done — проверяет выполнение каждого критерия DoD из плана задачи
• Повторная проверка приёмки — самостоятельно перепроверяет каждый due-кейс приёмки по актуальным доказательствам и возвращает расхождения на исходную стадию; зелёный набор тестов сам по себе не является приёмкой
• Гейт spec-graph (QA Layer 3c) — запускает автоматический гейт spec-graph, сообщает проверенные артефакты и trace-корзины, при коде выхода адаптера 2 завершается fail-closed
• Обновление рефлексий — документирует находки в datarim/reflection/*.md

Как это работает

Ревьюер читает определение задачи (включая DoD) и стайл-гайд, затем систематически проверяет реализацию. Применяет проверки безопасности из навыка security и паттерны тестирования из навыка testing. Каждая находка категоризируется по серьёзности. Обзор завершается чётким вердиктом — пройдено или нет — с конкретными пунктами для исправления.

Пример

/dr-qa
→ Ревьюер читает tasks.md (DoD: 5 критериев)
→ Ревью кода: 2 проблемы (нет валидации ввода, неясное именование)
→ Проверка безопасности: 1 находка (SQL-параметры не параметризованы)
→ Валидация DoD: 4/5 критериев пройдено
→ Вердикт: НЕ ПРОЙДЕНО — 3 пункта для исправления
→ Обновляет reflection/reflection-<TASK-ID>.md

Загрузка контекста

Читает datarim/tasks.md (для DoD) и datarim/style-guide.md. Применяет навыки security (загружается до чтения исходников), immutability, testing, datarim-system и cta-format при каждом вызове; вердикт BLOCKED использует вариант fail-routing блока Next Step.

Используемые навыки

security (всегда), immutability (всегда), testing (всегда), datarim-system (всегда), cta-format (всегда).

Отчёт о результате

Перед сообщением человеку применяется Human Outcome Reporting. Объясняются изменение для пользователя, исходные условия, проверки и доказательства, а также открытые ограничения. Итог этапа не означает завершения всей задачи. Точные протоколы и технические записи сохраняются. Для повторного объяснения без продолжения работы — /dr-explain.