Скил receiving-code-review — проверять перед внедрением замечаний ревью
Как скил receiving-code-review в Datarim заменяет показное согласие шестишаговым паттерном — прочитать, понять, проверить по кодовой базе, оценить, ответить, реализовать.
AI-ассистент, который отвечает «Абсолютно верно! Реализую прямо сейчас» на каждое замечание ревьювера, не выполняет код-ревью — он занимается социальным перформансом. Скил receiving-code-review заменяет этот паттерн шестишаговой последовательностью, построенной вокруг одного принципа: сначала проверь, потом реализуй.
Шестишаговый паттерн
Последовательность реакции фиксирована: прочитать всю обратную связь без немедленной реакции, сформулировать техническое требование своими словами (или попросить уточнения, если оно неясно), проверить по реальной кодовой базе, оценить, обоснована ли правка технически для данного проекта, ответить с технически выверенным комментарием или аргументированным возражением, затем реализовать по одному пункту с тестированием каждого. Если хотя бы один пункт отзыва неясен — последовательность останавливается целиком на втором шаге: частичная реализация при неполном понимании даёт неверный результат.
Когда возражать
Скил перечисляет конкретные условия для возражения: когда правка ломает существующую функциональность, когда ревьювер не имеет полного контекста кодовой базы, когда изменение добавляет фичу, которую ничто сейчас не вызывает (YAGNI), когда правка технически некорректна для используемого стека, или когда она противоречит ранее принятым архитектурным решениям. Механизм возражения — технические аргументы: проверка, сломает ли правка существующие тесты, ссылка на причину, по которой текущая реализация именно такая, или конкретный уточняющий вопрос. Скил рассматривает предложения внешнего ревью как элементы для оценки, а не приказы для исполнения.
Никакого показного согласия
Ответы вида «Отличное замечание!» или «Спасибо, что поймали это!» перечислены как запрещённые — не потому что благодарность груба, а потому что она подменяет технические сигналы социальными. Когда отзыв верен — правильный ответ это краткое описание того, что изменилось, или сама правка. Когда неверен — конкретный технический контраргумент. В обоих случаях нужна точность, а не теплота.
Ответы на inline-комментарии GitHub
При ответе на inline-комментарии ревью в GitHub скил указывает: ответы идут в тред комментария через эндпоинт gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies, а не как комментарии к PR верхнего уровня. Это держит обсуждение под строкой кода, к которой оно относится, а не рассыпает его по всему PR. Подробнее — в справочнике receiving-code-review или в посте что такое Datarim.