Code review с помощью ИИ
На прошлом уроке мы научились писать описание PR, которое обобщает несколько коммитов. Теперь, когда PR открыт и CI отработал, наступает шаг, который проще всего пропустить: прочитать сам diff, ровно так, как вы читали бы PR живого разработчика. Зелёный CI означает, что код не сломал существующий те
Проверять diff, который написал агент, — это как осматривать посылку, доставленную к вашей двери, прежде чем расписаться за неё: недостаточно того, что коробка запечатана и снаружи выглядит нормально, — нужно открыть её и убедиться, что внутри действительно то, что вы заказывали, а не что-то лишнее.
- diff Pull Request
- Полное отображение каждой добавленной и удалённой строки в PR, в формате unified diff; это то, что читают строку за строкой, чтобы проверить, делает ли изменение ровно то, что просили.
- запрос изменений (request changes)
- Вердикт ревью, который блокирует слияние до тех пор, пока автор не исправит конкретные пункты; отличается от произвольного комментария — он реально предотвращает слияние, пока пункт не будет устранён.
- зелёный CI (и что он не означает)
- Подтверждение того, что код прошёл lint, проверку типов и существующие тесты; оно не подтверждает, что diff ограничен тем, что просил тикет, и не проверяет граничные значения, которые не покрыты ни одним тестом.
- несвязанное изменение, включённое в diff
- Правка, которая появляется в том же PR, но не связана с запрошенным исправлением; даже если она тихо проходит CI, её следует вынести отдельно или отрецензировать отдельно.
