Как нормально ревьюить код который написал агент, нужны рабочие приёмы

[christo]

Прошу поделиться рабочими приёмами как ревьюить код сгенерированный агентом, потому что объёмы выросли и глазами всё читать уже не успеваю. Раньше PR был на 200 строк раз в день, теперь агент за час накидывает три PR по 800 строк и всё это надо проверять. Чувствую что начинаю пропускать косяки потому что замыливается глаз. Как у вас построен процесс.

[tx3300]

главное правило, заставляй агента делать маленькие коммиты. не один PR на 800 строк а восемь по 100 с осмысленными сообщениями. ревьюить мелкие куски реально, простыню нет

[kotlinwizard]

gpt тоже умеет ревьюить, ставь второго агента на проверку первого

[vaultaddict]

ставь второго агента на проверку первого

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

[svelte88]

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

[vuemaker]

расскажу как перестроил процесс когда захлебнулся в агентских PR, помогло реально. сместил фокус ревью с как написано на что и зачем. конкретно. первое, читаю не код а diff тестов и описание PR. если агент правильно сформулировал что он делает и покрыл это тестами, реализацию проглядываю по диагонали, доверяю но проверяю границы. второе, держу чеклист типовых косяков именно агентского кода, их немного но они повторяются. лишние try-except которые глотают ошибки, выдуманные методы которых нет в либе,N+1 запросы потому что агент не видит как это бьёт по базе, захардкоженные значения вместо конфига, и обработка edge case которых в задаче не было но агент на всякий случай нагородил. третье, всегда сам проверяю места с деньгами, правами и удалением данных, тут построчно и без скидок. четвёртое, прогоняю статанализ и линтер до того как смотрю глазами, пусть тулза отсеет ерунду, я смотрю смысл. и пятое, если PR больше 400 строк я его не принимаю а прошу агента разбить. большой агентский diff это всегда красный флаг, значит он сделал больше чем надо. вот так из захлёбываюсь стало успеваю, при том что объём генерации только вырос

[peopelle]

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

[kernel2]

линтер и статанализ это база, странно что про это вообще спрашивают в 2026

[tavogo]

агент пишет уверенно и красиво даже когда несёт чушь

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