1. Понятны ли имена. Если функция называется handleData, это красный флаг. 2. Есть ли тесты. Не покрытие в процентах. Есть ли тест на критический сценарий. 3. Можно ли это объяснить джуну за 2 минуты. Если нет, архитектура слишком сложная. 4. Нет ли скрытых зависимостей. Модуль не должен знать о деталях другого модуля. 5. Что происходит при ошибке. Если нет обработки, это не фича, это бомба с таймером. 6. Не нарушен ли принцип единственной ответственности. Функция на 200 строк которая делает всё сразу это проблема. 7. Есть ли смысл этого кода здесь. Иногда правильный ответ на PR это удалить код а не добавить. А что вы проверяете в code review в первую очередь?