ℹ️ Где-то в глубине внезапно лежит SQL, хотя ты пришел посмотреть фронтовую правку.
ℹ️ Code review иногда правда похоже на археологию: важно не просто найти артефакт, а понять, почему он там оказался.
Два сценария
❌ Нерабочий сценарий:
Сразу писать «переделай». Так мы лечим симптом и не понимаем причину.
✔️ Рабочий сценарий:
Отделить фикс от соседних изменений; проверить, какой сценарий реально чинится; найти, что может сломаться рядом; спросить, почему выбран именно этот путь; зафиксировать, что оставляем как техдолг, а что тормозим до мержа.
ℹ️ Главный сигнал: если PR чинит одну проблему, но объяснить его можно только через три соседних модуля, это уже не просто ревью. Это раскоп контекста.
И тут задача ревьюера - не стать строгим хранителем музея, а помочь команде не закопать следующий баг еще глубже.
🖥 Исходный код — перейти на сайт!