Большая сила маленьких PR-ов в эру AI

Значимый прорыв формирует отрасль ИИ: mipo256 4 минуты назад Большая сила маленьких PR-ов в эру AI Простой 6 мин 10 Блог компании Spring АйО Программирование * Java * Мнение Всем привет, меня зовут Михаил Поливаха, я являюсь техническим лидером проекта Axelix. В Axelix со временем добавляется всё больше и больше сторонних контрибьютеров, которых мы очень приветствуем. И всё больше и больше я сталкиваюсь с некоторым непониманием некоторых строгих гайдлайнов, которые установлены на проекте.

Именно сегодня, речь пойдёт про размер PR-а. У нас есть собственный GitHub Action, который блокирует PR в том случае, если суммарное количество измененных строк кода превышает 500. В этой статье я хочу рассказать про то, почему такое правило важно, и почему оно вам тоже нужно.

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

Откуда ноги растутВ современном мире AI Agent-ов количество кода растёт довольно быстро. Растёт скорость изменений. И я считаю, что это крайне важно, чтобы скорость этих изменений не влияла негативно на качество продукта и кодовой базы.

Какое-то влияние будет, но хотелось бы его ограничить. Для этого, мы настраиваем так называемый процесс Quality Gate. В него входят тесты, linter-ы, статический анализ кода и т.

И очень важным компонентом Quality Gate является ревью CODEOWNER-а. Мы в Axelix хоть и имеем Code Rabbit, и наши PR-ы смотрит усиленный скилами и чем только можно AI Agent в облаке, финальное решение принимает всегда человек. Был в своё время довольно известный research (“Needle in the Repo”), в котором говорилось, что:The main limitation exposed by NITR is not raw implementation ability alone, but difficulty maintaining design discipline during repository-level changeТо есть, речь о том, что AI Agent-ы, как правило, могут сделать хорошее, качественное локальное изменение, но часто они не до конца учитывают архитектуру проекта целиком.

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

И если дать AI Agent-у разгуляться, то система работать-то будет. Но если потом посмотреть на неё немного сверху, то всё это будет выглядеть как какая-то каша. Я думаю, многие с этим на практике сталкивались (Вы не одни такие, research говорит о том же).

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

В общем, задача - сделать так, чтобы ревью человека было максимально качественным. Настолько, насколько возможно. Размер PR-аИ вот тут мы подходим к основной теме.

Событие, по словам экспертов, усилит конкуренцию в сфере ИИ.