Мы собрали 13 приложений на 11 языках и отправили в каждое Merge Request. Бот отревьюил их так же, как ревьюит рабочий код. Всё открыто: заходите в MR и читайте замечания целиком.
Демо показывает, как выглядит ревью. Насколько оно полезно именно вашей команде — проверяется только на вашем коде и ваших правилах: поставьте на один репозиторий и посмотрите.
В каждом репозитории лежат договорённости команды (docs/adr) и конфиг ревью с правилами, которые из них выведены. Бот проверяет именно их — поэтому находки говорят на языке проекта, а не общими словами.
Angular
preset: angularОфлайн-корзина: очередь изменений без сети и синхронизация при её появлении
Сервис сам подписывается на холодный HTTP-поток и его же отдаёт наружу: один клик «Синхронизировать» уходит на сервер двумя запросами.
NestJS
preset: nestjsИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
У позиции заказа нет поля priceKopecks — оно называется unitPriceKopecks, поэтому цена не сохранялась бы вовсе. Бот сверил код с сущностью, а не с намерением.
React
preset: reactОфлайн-корзина: очередь изменений без сети и синхронизация при её появлении
localStorage.clear() выполняется до ответа сервера: при обрыве сети несинхронизированные изменения исчезают вместе с остальными данными приложения.
Django
preset: djangoИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
Эндпоинт удаляет позиции всех заказов и переписывает выгрузку, но не объявляет ни permission_classes, ни authentication.
Gin
preset: ginИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
INSERT не заполняет колонку source, по которой строкой выше идёт DELETE: следующий импорт удалит не то, что вставил предыдущий.
Spring Boot
preset: springИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
SELECT lastval() возвращает последовательность в пределах соединения — на пуле соединений позиция привяжется к чужому заказу.
Android · Compose
preset: androidОфлайн-корзина: очередь изменений без сети и синхронизация при её появлении
Автосинхронизация запущена прямо в теле composable: каждая перерисовка экрана заводит ещё один бесконечный цикл в GlobalScope.
Rails
preset: railsИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
Новые запросы фильтруют по source и external_id, а миграции с индексами в MR нет — таблица заказов растёт, запрос останется.
Laravel
preset: laravelИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
Цикл импорта не обёрнут в try/catch: исключение на одной строке валит весь импорт, а в логе не остаётся ни строки, ни причины.
Actix Web
preset: actixИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
Разбор строки индексирует parts[0..5] без проверки длины: короткая строка выгрузки роняет обработчик паникой.
ASP.NET Core
preset: aspnetИмпорт суточной выгрузки поставщика: полный срез заказов за день одним CSV
Прежние заказы поставщика удаляются до цикла, поэтому проверка на существующий external_id внутри цикла уже ничего не находит — дедупликация мертва.
Flutter
preset: flutterОфлайн-корзина: очередь изменений без сети и синхронизация при её появлении
Timer.periodic заведён прямо в build() у StatelessWidget: таймеры плодятся на каждой перерисовке и не отменяются никогда.
iOS · SwiftUI
preset: iosОфлайн-корзина: очередь изменений без сети и синхронизация при её появлении
Перед синхронизацией стираются все ключи UserDefaults приложения, а не только очередь корзины.
В части корпоративных сетей внешний GitLab недоступен. Чтобы demo не оказалось для вас набором нерабочих ссылок, приводим сводку из MR по C# / ASP.NET дословно — так её видит разработчик в Merge Request.
🤖 ReviewGate — AI-ревью
MR добавляет импорт суточной выгрузки поставщика, но реализация несёт серьёзные риски: неограниченный DELETE стирает позиции ВСЕХ заказов (не только поставщика), SQL-инъекция через интерполяцию external_id, деньги считаются в double, весь процесс не транзакционен и не защищён авторизацией, а обработка строк CSV делает по несколько запросов к БД на каждую запись. Требуется существенная переработка перед мержем.
Пресет: aspnet.
❌ Severity gate (critical) НЕ пройден: блокирующих замечаний 6.
Замечаний: 12 (⛔ 1 · 🔴 5 · 🟠 5 · 🔵 1); новых строчных: 8. По правилам команды (📐): 6.
Из ранее найденного — уже на ревью, не дублирую: 4.
▸ 🔬 Диагностика прогона
⚙️ Это ревью: 52,3K токенов вход (из них 7,6K из кэша) · 42K выход · ≈ 0,75 $
Строка стоимости — не наша бухгалтерия, а функция бота: он считает прогон по прайсу, который вы указали в конфиге, в вашей валюте. Свёрнутый блок «Диагностика прогона» раскрывается в MR: там видно, какие модели вызывались и что судья отбросил. Кода в блоке нет — только метаданные.
Находки привязаны к строкам диффа. Открывайте вкладку Changes — они там же, где их читает разработчик.
Один комментарий с итогом прогона и вердиктом. Во всех демо порог выставлен на critical, поэтому проверка красная, пока блокирующие замечания не закрыты.
Свёрнутый блок в сводке: какие модели вызывались, сколько токенов ушло, что и почему судья отбросил или понизил. Кода в блоке нет — только метаданные.
У части находок — готовый фикс: площадка показывает кнопку, правка ложится в ветку одним кликом. Такой фикс бот даёт только там, где уверен.
В каждом запросе разработчик спорит с одной из находок, и бот отвечает в том же треде — где-то отстаивает замечание разбором кода, где-то соглашается с нюансом, которого не учёл.
Демо ревьюил обычный ReviewGate — тот же движок и те же настройки из .reviewgate/config.yml, что ставятся у вас в контуре. Разница только в том, что эти репозитории публичные: ваш код в своём контуре и остаётся.