Контрольные точки ревью: чек-лист перед мержем

Контрольная точка ревью — это короткий список того, что должно быть истинным, прежде чем изменение попадёт в эксплуатацию. Пройдитесь по нему сами или попросите AI подтвердить каждый пункт:

• Я прочитал дифф и понимаю, что изменилось и почему
• Изменение ограничено тем, о чём я действительно просил
• Тесты проходят (и у нового поведения есть тест)
• Проверка типов и линтер чисты
• Не осталось секретов, отладочных логов или закомментированного кода
• Никакие проверки безопасности не были тихо удалены
• Я сам попробовал изменение, хотя бы счастливый путь

Если не можете отметить пункт — вы не закончили.

Чек-лист намеренно короткий, потому что пропущенная контрольная точка — вовсе не контрольная точка. Соблазн, особенно когда вы движетесь быстро и изменение «выглядит нормально», — пропустить его не глядя. Сопротивляйтесь этому во всём, что видит пользователь. Полезный трюк — заставить AI заполнить его за вас: вставьте список и попросите: «Пройди по этой контрольной точке ревью для изменения, которое ты только что сделал. По каждому пункту скажи PASS или FAIL с одной строкой доказательства». Он не может честно написать «тесты проходят», не запустив их, а требование цитировать доказательство построчно превращает расплывчатое «выглядит хорошо» в нечто, что можно проверить.