В среду агент сгенерировал «полный рефакторинг». Зелёные тесты. Красивые имена. И ощущение, что я ревьюю не код, а уверенный тон.
// было: понятный кусок
if (user == null) return unauthorized();
// стало «умнее»
Optional.ofNullable(user)
.filter(u -> u.isActive())
.orElseThrow(UnauthorizedException::new);
// + ещё 40 строк обёрток ради одного if
Чеклист, который я клею в начало ревью
- Есть ли один абзац «зачем» без слова «улучшили»?
- Можно ли откатить фичу одним флагом / одним коммитом?
- Не раздули ли публичный API «на будущее»?
- Тесты проверяют поведение или только счастливый путь генератора?
ИИ ускоряет набор текста. Ревью по-прежнему делает человек, который будет дежурить в 3 ночи.
# мой фильтр перед merge
git diff main...HEAD --stat
git log -5 --oneline
# если stat > 400 строк без причины — прошу разрезать PRВ итоге из 800 строк осталось сорок осмысленных. Остальное — красивый шум. И да, я всё ещё рад ИИ. Просто без автоapprove.