Когда pull request выглядит нормальным, но ревью на нём всё равно зависает

В сфере искусственного интеллекта произошло заметное событие. alyadusov 16 минут назад Когда pull request выглядит нормальным, но ревью на нём всё равно зависает Уровень сложности Простой Время на прочтение 8 мин Охват и читатели 576 Программирование * Информационная безопасность * Искусственный интеллект Аналитика Из песочницы Поводом для этого проекта был не абстрактный интерес к AI и не желание сделать ещё один инструмент для ревью. На одном из рабочих проектов довольно быстро стало видно, что на pull request уже нельзя смотреть по старой модели. Команда начала двигаться в сторону AI-first разработки.

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

Технические детали

В этот момент review начинает ломаться не на чтении синтаксиса. Оно зависает на другом. Не на вопросе “что делает эта строчка”, а на вопросах вроде: это вообще корректное изменение для этой области кодовой базы?

мы точно хотим так менять auth-логику? это правда просто чистка конфига? почему один PR одновременно трогает workflow, права доступа и доменную логику?

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

Отраслевые последствия

И вот здесь обычное review очень быстро упирается в предел. Если PR локальный, а автор хорошо понимает зону, review в основном сводится к проверке качества решения. Если PR приходит из AI-first потока, от соседней команды или от человека с поверхностным знанием домена, review превращается в другую задачу: нужно не просто прочитать diff, а восстановить, что именно это изменение делает с системой и можно ли его пропускать дальше.

В какой-то момент перестал устраивать привычный вопрос “что тут не так? Намного полезнее оказался другой: что команда должна сделать с этим PR прямо сейчас? Так и начал собираться PRShield как отдельный рабочий MVP.

Хотелось проверить, можно ли под эту задачу сделать не очередного review-бота, а отдельный слой принятия решения на этапе мержa. В самом грубом виде ответов у него всего три: SAFE REVIEW_REQUIRED BLOCK Не “найти всё плохое”. Не “оставить побольше замечаний”.

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