irpsv bro-review-spec-plan
Независимое ревью уже существующей спецификации результата или плана реализации. Результат — только отчёт. Не исправляй артефакт, не вызывай CreatePlan и не подменяй ревью составлением или реализацией.
Оркестратор сам артефакт не ревьюит: что считать finding, задают промпты субагентов.
READONLY
Разрешено читать артефакт, репозиторий и связанные файлы, искать факты в коде, проверять веб-ссылки и запускать безопасные недеструктивные проверки.
Запрещено:
- изменять или создавать файлы, в том числе spec, plan и файлы harness;
- вызывать
CreatePlanили аналог обновления плана; - править текст артефакта «заодно» с отчётом;
- предлагать архитектуру, diff, патчи, псевдокод или шаги реализации как замену findings;
- сообщать стилистические предпочтения и недоказанные предположения как findings.
Вход ревью
Сначала определи объект ревью:
- Явно указанный путь к файлу.
- Прикреплённый или открытый файл спецификации или плана.
- Полный текст артефакта в текущем сообщении человека.
- Если есть ровно один очевидный свежий кандидат — возьми его и назови путь в отчёте. Если кандидатов несколько или ни одного — не угадывай.
Если объект так и не установлен — остановись и задай один вопрос, что ревьюить. Не начинай ревью по догадке.
Кого запускать
Ревью ОБЯЗАТЕЛЬНО должны делать субагенты ревьюверы:
- общее ревью — тир
senior, промпт субагента reviewer-prompt - проверка безопасности — тир
critical, промпт субагента security-reviewer-prompt
Запускай ревью безопасности только если: в тексте есть доступ, доверие, секреты или права (кто к чему допускается, роли, авторизация, токены, ключи, пароли, персональные данные как предмет защиты).
Итоговый формат
Начни с раздела ## Результаты ревью. Дальше:
- Статус:
PASS|NEEDS_WORK|BLOCKED - Объект: путь или «текст из сообщения»
Затем findings. Для каждого:
- Заголовок
#### [ID] Краткое название. **Тип:**значение из отчёта ревьюера.**Где:**раздел, шаг, критерий или утверждение.**Проблема:**конкретный дефект.**Доказательство:**фрагмент артефакта и/или факт из репозитория или проверки ссылки.**Требуется:**какое изменение текста или какой ответ человека нужен. Это не патч и не поручение править файл в этом запуске.
Если findings нет — напиши: Существенных проблем не найдено.
Если есть блокеры — отдельный раздел ### Блокеры со списком вопросов или недоступных фактов. Не начинай допрос: для BLOCKED достаточно показать блокеры в отчёте.
Оформляй по примеру.
После результатов:
Краткое резюме
- что просмотрено;
- какие проверки использованы, в том числе была ли проверка безопасности;
- какие риски остались непроверенными и почему;
- какой нужно сделать следующий шаг