Прилетел PR на 4700 строк от агента, как это вообще ревьюить

[wavman]

ситуация. коллега, нормальный мидл, присылает PR: 4700 строк, 38 файлов, внизу честное generated with claude code. сама фича строк на 300, остальное тесты на каждый чих, рефакторинг соседних модулей раз уж зашел, и переименования. я открыл дифф, поскроллил минут пять и закрыл. физически невозможно отревьюить так, чтобы потом за это отвечать.
попросил разбить. он обиделся, говорит что агент делает атомарно связанные изменения, разбивка убьет контекст. тимлид молчит, правил под агентские PR у нас нет.
как у вас? лимит на размер? запрет попутного рефакторинга? ревьюит другой агент? объем кода за год вырос раза в три, ревьюеров столько же. я серьезно не понимаю, как с этим дальше жить.

[nixos_andy]

правило старое как мир: не можешь отревьюить, не аппрувь. кем код написан, вообще не важно. PR на 4700 строк это неуважение к ревьюеру, точка.

[redis7]

ревьюит другой агент, лол. а того ревьюит третий. ревью all the way down

[fandango1305]

агент делает атомарно связанные изменения, разбивка убьет контекст

отмазка. claude code нормально режет работу на стек коммитов, если попросить, под stacked PR даже готовые промпты по интернету гуляют. твой коллега просто не хочет тратить лишние 20 минут. атомарно связанные изменения в 38 файлах это оксюморон.

[nilcetinkaya]

скажу непопулярное. мы год назад ввели лимит 400 строк на PR и выяснили, что ревью все равно фикция: человек скроллит, пишет lgtm и идет дальше. честнее признать что ревью глазами умерло и вкладывать время в тесты, контракты и канареечный деплой. инциденты на проде считаются, а не отступы.

[golanggeek]

честнее признать что ревью глазами умерло

у вас умерло, у нас живое. на прошлой неделе именно глазами поймали агентский код, который при ретрае платежа списывал деньги второй раз. тесты были зеленые, потому что агент написал тесты под свою же кривую логику. твоя канарейка поймала бы это на живых деньгах клиентов. спасибо, не надо.

[async2025]

38 файлов это еще цветочки. видели что бывает, когда агенту разрешают прогнать eslint fix по всему репозиторию? у нас git blame умер как класс, все строки ведут в один мегакоммит style: format

[icu2]

@fandango1305, подниму тему. та же беда, ревьюить некому. с понедельника вводим лимит 600 строк и запрет рефакторинга в фичевых PR, отпишусь через месяц, как пошло