ИИ-ревью пул-реквестов: помощник или генератор шума? Делимся опытом

[lentyaj]

Внедрили автоматическое ревью PR через Claude в CI. Половина комментов по делу, половина — «рассмотрите возможность добавить обработку ошибок» на каждый чих. Команда начала их игнорить. Как настроить, чтобы был сигнал, а не шум?

[tavogo]

Ключ в промпте ревьювера. Запрети ему стилистику и nit-пики, оставь только баги, безопасность и явные логические дыры. У нас после этого комментов стало в 4 раза меньше, но каждый по делу.

[redislover]

Дай ему контекст проекта и гайдлайны, иначе он ревьюит абстрактный код из учебника. С CLAUDE.md где описаны наши соглашения, попаданий стало заметно больше.

[Bill2001]

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

[chuckger]

Со стороны процессов добавлю: важно чтобы джуны не воспринимали боторевью как истину. Был кейс, человек 3 раза переписал нормальный код под капризы модели.

[rsal56]

hr_tech, прям наш случай. Окей, забираю: жёсткий промпт без nit-пиков + CLAUDE.md + не блокирующий, информационный режим. Попробуем спринт, отпишусь.

[Manuelriere]

Только не вздумайте слать диффы приватного репо во внешнюю модель без галочки в договоре и zero-retention. Юристы и безопасники должны это подписать до внедрения, а не после.

[delphin]

Конкретное решение которое убрало 60% шума: написали system prompt с явным списком типов замечаний которые НЕ нужны — стилистические предпочтения, «рассмотрите добавление логирования», TODO-предложения без конкретики. Оставили только: потенциальные баги с объяснением как воспроизвести, нарушения явных инвариантов кодовой базы, проблемы безопасности. Количество комментов упало с 15-20 до 4-6 на PR, качество выросло.

[infern]

Другой работающий подход — давать ревьюеру контекст через файл в репозитории (.claude/review_context.md или через custom instructions). Там описываешь: архитектурные решения принятые намеренно, паттерны которые в этой кодовой базе ОК, список файлов которые не трогать в ревью (автогенерированный код, легаси). Без этого контекста ИИ ревьюит в вакууме и предлагает рефакторить то что специально сделано определённым образом.

[misha12]

Ещё один угол: отдели ревью от блокера. Настрой CI так чтобы ИИ-комментарии шли отдельным шагом с меткой 'suggestions' и не блокировали мерж. Разработчик видит их как advisory, а не как ошибки. Когда замечания не блокируют — меньше раздражения, команда начинает их реально читать и иногда соглашаться. Мы так сделали и adoption вырос: люди перестали воспринимать бота как препятствие.