Можно ли доверять агенту мержить свой код без ревью

[sepiatone]

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

[togashi]

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

[depechie]

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

[Neskuchay]

он напишет тест под свой баг и тот будет радостно проходить

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

[coder_anton]

Мы держим золотую середину и она норм работает. Агентский PR проходит ровно тот же flow что человеческий: CI, линтер, обязательный аппрув живого ревьюера. Никаких поблажек что мол это же ИИ написал, наверное правильно. На практике агентский код ревьюится даже строже, потому что у людей включается режим а ну ка что ты тут навертел. Что реально ловим на ревью чаще всего:
- лишние абстракции на ровном месте, интерфейс ради одного класса
- проглоченные исключения, catch и тишина
- N+1 запросы там где он не видел всю картину с ORM
- мёртвый код который он добавил на всякий случай
Тесты зелёные у нас необходимое но не достаточное. Доверять автомержу без человека я бы не стал даже в 2026, не потому что модель тупая, а потому что она не несёт ответственности. Отвечать перед заказчиком за упавший прод будешь ты, а не агент.

[mjp1982]

+1 к посту выше. ответственность ключевое. модель не уволят и премии не лишат

[rbacker]

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

[valru]

@mjp1982, агент за день генерит 2000 строк, как это вычитать вдумчиво
а может проблема в том что не надо генерить 2000 строк в день? если задача требует 2000 строк нового кода ежедневно то у вас либо дикий скоуп либо агент льёт воду и плодит то что не нужно. меньше генерь, больше думай что просишь

[android_roman]

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