intermediate
Code review
Review diff на корректность, поддерживаемость, риск, тесты и операционный эффект — а не только стиль.
Code review оценивает корректность, поддерживаемость, безопасность, тесты, observability и риск интеграции. Git даёт diff и тред обсуждения; ревьюер рассуждает о runtime-поведении и операционном эффекте.
Слои эффективного review:
- **Поведение** — решает ли задачу безопасно?
- **Тесты** — покрыты ли рискованные пути?
- **API и контракты данных** — миграции, обратная совместимость
- **Стиль** — по возможности в lint/format
На интервью: scope review, уважительный feedback, глубина по риску, ротация ownership и как автоматизация снимает рутину.
Типовые ошибки: review только стиля при пропуске логики; блокировка по субъективным предпочтениям вне tooling; rubber-stamp огромных diff под дедлайн.
Компромисс — строгость review против скорости команды; механику в CI, суждение — людям.
Чеклист:
- Сначала поведение и риск, потом придирки.
- Тесты на failure modes.
- Tooling для форматирования и lint.
- Feedback конкретный и уважительный.