Ответ вслух Senior Код-ревью и правила команды спрашивают часто обновлено

Правила работы с ИИ для команды

Вопрос

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

Попробуйте сами

Ответьте вслух за 2 минуты, как на собесе. Потом сверьтесь с тезисами: что прозвучало, а что нет.

Показать ответ и разбор

Коротко

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

Что должно прозвучать в ответе

  • Какие инструменты разрешены и для какого кода, по политике компании

  • Автор отвечает за код полностью и сам ревьюит дифф до отправки

  • Лимит размера пулреквеста, механика и рефакторинг отдельно

  • Тесты обязательны и тоже читаются на ревью

  • Новые зависимости проверяются отдельно: существование, поддержка, лицензия

  • Файл правил для ассистентов в репозитории, правки через ревью

  • ИИ-ревьюер как необязательный первый проход, одобряет человек

Что могут спросить следом

  • Как поймёте, что правила работают?
  • Нужна ли пометка о коде, написанном с ИИ?

Разбор

Образцовый ответ: «Сначала договорюсь о рамках: какие инструменты разрешены и можно ли отправлять в них наш код, это решает политика компании и безопасность. Дальше главное правило: автор отвечает за каждую строку и сам читает свой дифф перед ревью, ревьюер не первый читатель. Введу лимит размера пулреквеста и попрошу выносить рефакторинг и форматирование отдельно. Тесты обязательны, и на ревью смотрим их тоже, потому что сгенерированные тесты часто ничего не проверяют. Новые зависимости проверяем отдельно. Общий файл правил для ассистентов кладём в репозиторий и меняем через ревью. ИИ-ревьюер подключим как первый проход, но одобряет человек. Через месяц посмотрю на время ревью, размер пулреквестов и баги, пришедшие из прода».

Ответьте сами, а не прочитайте

В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.