Как разгрузить код-ревью и сохранить понимание системы

В обзоре small-change я упоминал, что хочу разобраться, как разные команды перестраивают ревью под агентскую разработку. И вот я закончил ресёрч на эту тему.

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

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

⭐️ Интересные идеи ➡️ Часть ревью можно сделать до написания кода. Инженеры Honeycomb совместно прорабатывают планы и промпты, автор разбора отмечает лучшую общую осведомлённость. Но строгий процесс оказался тяжёлым, его используют выборочно.

➡️ Обязательный approval уже убирают для части изменений. У команды Claude Code число человеческих approvals зависит от риска. Но отказ от обязательного approval работает в определённых условиях: доверенные авторы, несколько слоёв проверки, CI и процесс выпуска.

➡️ Список тест-кейсов не позволяет проверить качество самих тестов. Входные данные могут не создавать нужную ситуацию, мок — скрывать поведение, assertion — не проверять обещанное. Я изначально думал, что можно не читать тесты, но исследование меня переубедило.

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

Приятного чтения!

➖➖➖➖➖➖➖➖➖➖➖ 📝 @ulshinblog


В этом посте были ссылки, но мы их удалили по правилам Сетки