Коллега заливает PR от агента по 4к строк и считает что ревью это моя проблема

[radiomaker]

Накипело. Человек в команде открыл для себя Claude Code, и теперь PRы приезжают по 3-4 тысячи строк. Внутри смесь: нормальная фича, рядом переименованные переменные в файлах, которые никто трогать не просил, перегенеренные тесты на 800 строк, где половина проверяет что 2+2=4, и пара тонких изменений поведения, о которых в описании ни слова.

На ревью одного такого у меня уходит 3-4 часа. Говорю ему: дроби и вычищай. Ответ: агент так сгенерил, работает же, ты просто придираешься, ревьюй тоже агентом если долго.

Тимлид молчит, ему нравится скорость. Как у вас выстроен процесс, чтобы агентский код не превращал ревью в наказание?

[koneko2]

ревьюй тоже агентом, в чем он не прав)) клин клином

[kickmybox]

агент так сгенерил, работает же, ты просто придираешься

вот за эту фразу надо бить по рукам. кто открыл PR, тот и автор, точка. не агент сгенерил, а ОН прислал тебе 4к строк, которые сам не читал. у нас в регламенте так и записано: автор отвечает за каждую строку независимо от того, кто ее набрал, и на ревью обязан ответить на вопрос по любой строке. два раза не ответил, PR закрывается без разговоров. дисциплинирует мгновенно

[jownby]

@kickmybox, механика, которая у нас взлетела: жесткий лимит на дифф (400 строк, больше = автоматом draft и дробление), запрет несвязанных изменений в одном PR, и для агентских задач план до кода: агент сначала пишет план, план апрувится, потом код. скорость просела процентов на 20, зато ревью снова занимает полчаса, а не полдня. но без поддержки лида это не взлетит, у тебя похоже главная проблема в нем

[rsal56]

а может, проблема отчасти в тебе. 3-4 часа на 4к строк значит читаешь линейно каждую строку. перегенеренные тесты и переименования отсеиваются за 10 минут полуавтоматом, остается ядро строк на 600-800. время ревью должно зависеть от сложности изменений, а не от объема диффа

[tastytim]

перегенеренные тесты и переименования отсеиваются за 10 минут полуавтоматом

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

[jwil1440]

тимлид молчит = тимлид одобряет. иди говори с ним напрямую, иначе тут хоть обсоветуйся