code-review-change-size

Codex skill code-review-change-size задаёт простой критерий для размера изменения перед review. Если изменение не является механическим, суммарный diff не должен превышать 800 изменённых строк. Для сложных изменений установлен более строгий ориентир: менее 500 строк. Эти числа нужны не для соревнования в минимальности, а чтобы человек мог качественно прочитать, понять и проверить одну связную порцию работы. Механические правки выделены отдельно, потому что одинаковая автоматическая замена не создаёт той же когнитивной нагрузки, что изменение логики. Навык применяется к фактическому diff, а не к приблизительной оценке задачи. Сначала нужно посмотреть размер и состав изменения, затем определить, является ли оно механическим или содержит сложную логику. Важно считать не только новые файлы: в review следует учитывать добавленные, удалённые и изменённые строки, а также то, насколько изменение объединяет разные причины. Если сложный diff приближается к порогу, это сигнал проверить, не смешаны ли в нём функциональность, рефакторинг, тесты, форматирование и миграция. Когда изменение превышает подходящий предел, инструкция не требует механически откатывать работу. Она просит объяснить, можно ли разбить diff на последовательные reviewable stages, и назвать smallest coherent stage, который имеет смысл вынести первым. Такой этап должен быть самодостаточным настолько, чтобы его можно было проверить без чтения всего будущего изменения. Разбиение не должно разрывать обязательную зависимость или оставлять промежуточное состояние, которое невозможно собрать и протестировать; порядок определяется реальными зависимостями, а не только размером файлов. Критерий выбора этапов основан на трёх источниках контекста: самом diff, зависимостях и affected call sites. Diff показывает, какие строки и поведения меняются. Зависимости показывают, какие части должны появиться раньше, чтобы следующая стадия имела корректные типы и интерфейсы. Call sites показывают фактических потребителей изменения и помогают не спрятать несовместимость в большом PR. Поэтому одна крупная, но механическая миграция может рассматриваться иначе, чем небольшое изменение, затрагивающее сложную логику и много путей вызова. Практический результат skill — короткое объяснение размера, классификации и плана разбиения, если порог превышен. Для небольшого diff достаточно подтвердить, что он остаётся в разумных пределах; для сложного — зафиксировать меньший целевой объём и минимальный связный первый этап. Инструкция не обещает автоматически посчитать качество review и не заменяет тесты, линтер или проверку архитектуры: она помогает подготовить границы изменения и сделать последующую проверку обозримой. Если разбиение невозможно из-за тесной зависимости, это нужно явно объяснить и указать, какая связность делает один этап необходимым.