Ревью вывода в масштабе
Вот честная проблема: когда агенты пишут сотни строк через несколько задач, вы не сможете прочесть каждую строку так, как вычитываете diff из десяти строк. Притворство, будто можете, ведёт либо к проштамповке (одобряете не глядя), либо к параличу (не доверяете ничему). Ни то ни другое не выкатывается.
Путь — ставить фильтры на нескольких слоях вместо того, чтобы читать всё:
- Тесты — это пол. Никакой вывод агента не сливается без зелёных профильных тестов и чистой проверки типов. Это не обсуждается, и это автоматически — эту часть делает машина.
- Делайте выборочное ревью рискованных мест. Прочесть всё нельзя, поэтому читайте места с большим радиусом поражения: всё, что касается аутентификации, денег, удаления данных или внешних вызовов. Остальное просмотрите на предмет тревожных признаков из главы о создании фич — файлы, которые не следовало трогать, тихо удалённое, крошечный запрос, породивший огромный diff.
- Читайте diff, а не кодовую базу.
git diff --statза секунды скажет, что изменилось и насколько. Задача, очерченная одним модулем, но показывающая изменения в шести, — сигнал присмотреться прежде, чем прочесть хоть строку. - Запустите это. Зелёный набор тестов — не работающее приложение. Нажмите на эту штуку. Финальная проверка всегда та же, что и в любой другой главе: делает ли оно у вас на глазах именно то, о чём вы просили?
Очерчивание окупается здесь дважды. Туго очерченная задача даёт diff, который вы и вправду можете отревьюить; пущенная на волю — тот, который не можете. Проблема ревью в основном решается выше по течению, в брифе.