+7 499 350-21-34 Telegram MAX WhatsApp info@agenty-ai.ru
Агенты ИИ 3D-карта обсудить проект
кейс 07 · направление 03 · ИИ для разработки

ИИ-ассистент в команде разработки: ревью за минуты

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

Клиент под NDA: имя не разглашаем, потому что связаны договором о неразглашении, для наших контрактов это обычное условие. Отрасль, задача, ход работ и цифры настоящие.

14 разработчиков в команде
минуты первый проход ревью
2 недели релизный цикл вместо месяца
Иллюстрация к кейсу «ИИ-ассистент внутри команды разработки»
01 / задача

Что было до внедрения

Очередь на ревью росла быстрее, чем разбиралась. Сеньоры между своими задачами и чужим кодом выбирали свои, и пул-реквест мог провисеть три дня. К моменту ревью автор уже забывал контекст и переключался обратно с трудом.

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

  • пул-реквесты ждали ревью до трёх дней
  • тесты и документация по остаточному принципу
  • код не должен покидать контур
02 / решение

Как внедряли

шаг 1

Правила из репозитория

Сняли правила с живого кода: стайлгайд, договорённости, типовые замечания из истории ревью. Никаких общих рекомендаций из интернета.

шаг 2

Первый проход ревью

Машина проходит по пул-реквесту первой: стиль, очевидные ошибки, дырки в тестах. Живой ревьюер получает код причёсанным и смотрит архитектуру.

шаг 3

Тесты и документация

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

шаг 4

Свой контур

Модель развёрнута в периметре компании. Наружу не уходит ни строки, и это проверяла служба безопасности, а не мы на словах.

03 / как это выглядит

Контур решения

Схема: пул-реквест, первый проход машиной, живое ревью по существу. Код заказчика под NDA.
Схема: пул-реквест, первый проход машиной, живое ревью по существу. Код заказчика под NDA.
Егор Лаптев, технический директор

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

Егор Лаптев, технический директор
04 / результат

Цифры после запуска

минутыпервый проход ревью вместо дней
-40 %времени сеньоров на ревью
2 неделирелизный цикл вместо месяца
0строк кода наружу

Первый проход ревью занимает минуты. У сеньоров высвободилось 40 % времени, релизный цикл ужался с месяца до двух недель, и код так и не покинул контур: обещание оказалось проверяемым.

Услуга по теме: направление 03 «ИИ для разработки» · все кейсы студии

05 / под капотом

Что внутри решения

Ассистент читает изменения в пул-реквесте и пишет замечания туда же, где их читают люди. Никакого отдельного интерфейса.

Где живётСерверы компании, код не покидает периметр
Связано сСистема контроля версий: комментарии прямо в пул-реквесте
Что проверяетЛогику, безопасность и совместимость; форматирование оставлено линтеру
Знает проВнутренние библиотеки компании и правила, где чем пользоваться
Что видит командаДоля принятых замечаний: по ней видно, когда ассистент начинает шуметь
06 / кому подходит

Команды разработки от десяти человек

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

Отдельный случай, ради которого к нам чаще всего и приходят: код нельзя выпускать наружу. Тогда выбор моделей сужается, качество будет ниже облачного, а сроки длиннее. Это честный размен, и решать его надо до начала проекта, а не после.

  • От 10 разработчиков и 10 пул-реквестов в неделю
  • Есть договорённости о стиле кода, хотя бы неписаные
  • Понятно, что делать с замечаниями: кто их закрывает и когда
  • Отдельная ветка проекта, если код не может покидать контур
07 / методика

Как считали эти цифры

В разработке любые цифры про скорость спорны: задачи разные, недели неравные. Ниже что мы считали и чего это не доказывает.

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

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

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

Что мерилиОт создания реквеста до первого содержательного комментария
ПериодКвартал до внедрения против квартала после
Источник данныхЖурнал системы контроля версий, все реквесты подряд
Что значит «-40 % времени сеньоров»Доля реквестов, где сеньор был первым ревьюером
ОговоркаРелизный цикл менялся и по другим причинам, не только из-за ассистента
08 / переделали

Что пришлось переделать

Разработчики самая требовательная аудитория для ИИ-инструментов, и пилот с ними это всегда переговоры. Три вещи, которые мы перестроили по их замечаниям.

Только логика, безопасность и совместимость

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

Знание внутренних библиотек

Компания годами писала свои обёртки, и ассистент предлагал заменить их публичными аналогами. Добавили в контур описание внутренних библиотек и правила, где чем пользоваться. Это входит в диагностику: собираем такой список до запуска.

Модель под требования безопасности

Код не мог покидать периметр, а модель, которая нравилась по качеству, требовала внешнего API. Взяли ту, что работает на своих серверах, и честно обсудили разницу в качестве с заказчиком. Правило: выбор модели начинается с ограничений, а не с рейтингов.

09 / сроки

Из чего сложились шесть недель

Внутри контура всё дольше: нельзя просто взять облачный сервис и подключить.

  1. Недели 1-2Разбор процесса ревью, замер базовых цифр, выбор модели, которая работает на своих серверах.
  2. Недели 3-4Развёртывание в контуре компании, интеграция с системой контроля версий, права доступа.
  3. Неделя 5Настройка правил: что проверяем, что игнорируем, описание внутренних библиотек.
  4. Неделя 6Работа на живых реквестах, сбор обратной связи от команды, отсев лишних замечаний.

Шестая неделя оказалась самой важной. Без неё ассистент остался бы генератором шума, который команда отключила бы через месяц.

10 / кто делал

Команда кейса

Егор Лаптев, технический директор
Егор Лаптев Технический директор в кейсе: архитектура и контур
Тимур Хайруллин, ml-инженер, языковые модели
Тимур Хайруллин ML-инженер, языковые модели в кейсе: модели и дообучение
Анна Дорохова, devops / mlops
Анна Дорохова DevOps / MLOps в кейсе: развёртывание и CI

Похожая задача?

Покажите репозиторий и процесс ревью, скажем, что машина заберёт уже в первый месяц.

Папка с документами, из которой поднимается столбчатая диаграмма: выгрузка превращается в расчёт

Полезнее всего выгрузка обращений или заявок за месяц. Подойдут также регламент, прайс, пример договора или акта.
PDF, DOC, DOCX, XLS, XLSX, CSV, TXT, ZIP, RAR, 7Z, PNG, JPG, JPEG. До 20 МБ на файл, не больше 5 файлов, суммарно до 50 МБ.

Файлы уходят вместе с заявкой и дальше отдела продаж не идут. NDA подпишем до передачи данных, если нужно.