Тимлид требует ревьюить агентский код построчно, это вообще реально

[k8ssmith]

У нас тимлид ввёл правило: весь код который сгенерил агент ревьюим построчно как будто его джун писал, и автор PR головой отвечает за каждую строку. Звучит правильно, но на практике если агент за день нагенерил 2000 строк по трём задачам, я физически не успеваю их так читать, проще самому было написать. Как у вас выстроен процесс ревью кода от Claude Code или Cursor, чтобы и качество не просесть и не утонуть.

[scalalord]

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

[alansmit]

@k8ssmith, +1 к тому что выше. объём это и есть симптом. агент позволяет насрать кодом быстрее чем ты думаешь, а думать всё равно надо

[Roost66]

если агент за день нагенерил 2000 строк по трём задачам, я физически не успеваю их так читать, проще самому было написать

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

[Farkle]

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

[sleepyraccoon]

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

[reactchan]

он его просто выдумал потому что логично звучит. тесты были зелёные потому что он и мок под несуществующий метод подложил

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

[sabaza]

@scalalord, ну такое. у нас вообще тимлид сам с курсором сидит и наизусть просит писать. кто бы говорил про ревью

[armstron]

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