Главная / Шаблоны агентов / Ревью изменений в коде
Ревью изменений в коде
Просматривает изменения в отдельной ветке, запускает тесты и выписывает замечания. Принимает изменения человек.
Для чего этот шаблон
Агент читает набор изменений, запускает тесты и выписывает замечания: ошибки, пропущенные случаи, нарушения правил проекта. Он работает в отдельной ветке и ничего не вносит в основную.
Когда не подходит. Для кода с секретами, которые нельзя передавать выбранному сервису. Для критичных изменений безопасности: ревью человека обязательно.
Инструкция для агента
Скопируйте в настройки агента или в начало диалога. Слова в фигурных скобках замените своими значениями.
Цель: просмотреть изменения {ИЗМЕНЕНИЯ}, запустить тесты и выписать замечания.
Правила:
1. Работай только в ветке {ВЕТКА}. Основную ветку не трогай, не вноси изменения в чужие ветки.
2. Запусти тесты командой {КОМАНДА ТЕСТОВ} и приведи результат. Если тесты не проходят, опиши, какие и почему.
3. Замечание оформляй так: файл и строка, что не так, чем это опасно, как исправить в одну фразу.
4. Различай уровни: «блокирует» (ошибка, уязвимость), «важно» (риск), «мелочь» (стиль).
5. Не изменяй код сам, если об этом не просили. Не добавляй зависимости и не меняй настройки.
6. Секреты, ключи и пароли в коде или изменениях не копируй в ответ.
Результат: результат тестов; список замечаний по уровням; короткий вывод: можно ли принимать, что стоит поправить до принятия.
Что подключить и что подать на вход
Права выдавайте по одному, начиная с чтения.
Инструменты и права
- Чтение репозитория
- Запуск тестов в изолированной среде
- Комментарии к изменениям (без права слияния)
Входные данные
- Набор изменений
- Команда запуска тестов
- Правила проекта (стиль, требования)
Результат и остановки
Результат должен быть проверяемым. Остановка передаёт вопрос человеку.
Что получите
- Результат тестов
- Список замечаний по уровням
- Вывод о готовности
Когда агент останавливается
- Тесты не запускаются
- Изменения касаются секретов, авторизации, платежей
- Найдена возможная уязвимость
- Изменений слишком много для одного ревью
Проверка перед запуском
Прогоните агента на этих случаях и на своих реальных примерах. Не расширяйте права, пока доля ошибок не станет приемлемой.
- Корректное небольшое изменение
- Изменение с ошибкой в логике
- Изменение, ломающее тесты
- Изменение без тестов
- Изменение с секретом в коде
- Изменение зависимостей
- Очень большое изменение
- Изменение только документации
Метрики за первый месяц
- Доля замечаний, признанных верными
- Число ошибок, найденных после слияния
- Время на ревью до и после
Типичные ошибки
- Давать права на слияние
- Запускать тесты вне изолированной среды
- Считать ревью агента достаточным для изменений безопасности
Скачать шаблон
Файл в формате Markdown: обычный текст, открывается в любом редакторе. Шаблон можно свободно использовать и менять.