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

[rabbit8]

@delphin, список запретов работает, но у меня ещё жёстче зашло одно правило: 'не пиши коммент про баг, если не можешь привести конкретный вход, который его ломает'. Спекулятивное 'рассмотрите обработку ошибок' само отмирает, потому что модель не может придумать падающий кейс — а значит молчит. А реальные баги остаются, потому что для них ввод-ломалка как раз находится. Это убирает шум без ручного перечисления всех его сортов.

[tsav]

Большая часть 'добавьте проверку на null' рождается из того, что ревьюеру скармливают голый дифф. Он не видит, что эта проверка стоит двумя функциями выше, вне диффа, и честно предлагает её снова. Дайте ему целиком изменённый файл, а лучше доступ читать соседние — и приличный кусок ложных срабатываний просто исчезает, ещё до всякой подкрутки промпта.

[bruce01]

@lentyaj, пока вы тюните промпт на глаз, вы стреляете вслепую. Повесьте на комменты бота реакцию палец вверх/вниз или считайте долю resolved-as-useful, и правьте промпт по этим цифрам, а не по ощущениям. И сделайте дедуп: если паттерн один раз пометили как 'у нас так принято' — бот не должен поднимать его в каждом следующем PR, иначе вся усталость возвращается и команда снова уходит в игнор.