From 52615e4e49808b9b947c06c42ba5d2d31083a99c Mon Sep 17 00:00:00 2001 From: Anton Vakhrushev Date: Thu, 6 Aug 2026 17:55:29 +0300 Subject: [PATCH] =?UTF-8?q?review:=20=D1=82=D1=80=D0=B8=D0=B3=D0=B3=D0=B5?= =?UTF-8?q?=D1=80=D1=8B=20=D0=BF=D1=80=D0=BE=D1=84=D0=B8=D0=BB=D1=8F=20?= =?UTF-8?q?=D0=BE=D1=82=D0=B2=D1=8F=D0=B7=D0=B0=D0=BD=D1=8B=20=D0=BE=D1=82?= =?UTF-8?q?=20=D0=B7=D0=B0=D1=82=D1=80=D0=BE=D0=BD=D1=83=D1=82=D0=BE=D0=B3?= =?UTF-8?q?=D0=BE=20=D1=83=D0=B7=D0=BB=D0=B0=20-=20=D0=BF=D0=B5=D1=80?= =?UTF-8?q?=D0=B5=D1=87=D0=BD=D0=B8=20=D1=83=D0=B7=D0=BB=D0=BE=D0=B2=20?= =?UTF-8?q?=D0=BD=D0=B0=D0=B7=D0=B2=D0=B0=D0=BD=D1=8B=20=D0=BA=D0=B0=D1=80?= =?UTF-8?q?=D1=82=D0=BE=D0=B9=20=D0=BC=D0=B5=D1=81=D1=82,=20=D0=B3=D0=B4?= =?UTF-8?q?=D0=B5=20=D0=B6=D0=B8=D0=B2=D1=83=D1=82=20=D0=BF=D1=80=D0=B0?= =?UTF-8?q?=D0=B2=D0=B8=D0=BB=D0=B0;=20=D1=81=D1=82=D1=83=D0=BF=D0=B5?= =?UTF-8?q?=D0=BD=D1=8C=20=D0=BF=D0=BE=D0=B4=D0=BD=D0=B8=D0=BC=D0=B0=D0=B5?= =?UTF-8?q?=D1=82=20=20=20=D0=BD=D0=BE=D0=B2=D0=BE=D0=B5=20=D0=B8=D0=BB?= =?UTF-8?q?=D0=B8=20=D0=B8=D0=B7=D0=BC=D0=B5=D0=BD=D1=91=D0=BD=D0=BD=D0=BE?= =?UTF-8?q?=D0=B5=20=D0=BF=D1=80=D0=B0=D0=B2=D0=B8=D0=BB=D0=BE,=20=D0=B0?= =?UTF-8?q?=20=D0=BD=D0=B5=20=D0=BF=D1=80=D0=B0=D0=B2=D0=BA=D0=B0=20=D1=80?= =?UTF-8?q?=D1=8F=D0=B4=D0=BE=D0=BC=20=D1=81=20=D0=BD=D0=B8=D0=BC=20-=20?= =?UTF-8?q?=D0=B7=D0=B0=D0=B2=D0=B5=D0=B4=D1=91=D0=BD=20=D0=B7=D0=B0=D0=BA?= =?UTF-8?q?=D1=80=D1=8B=D1=82=D1=8B=D0=B9=20=D0=BF=D0=B5=D1=80=D0=B5=D1=87?= =?UTF-8?q?=D0=B5=D0=BD=D1=8C=20=D0=BE=D1=82=D1=81=D0=B5=D0=BA=D0=B0=D1=8E?= =?UTF-8?q?=D1=89=D0=B8=D1=85=20=D1=83=D1=81=D0=BB=D0=BE=D0=B2=D0=B8=D0=B9?= =?UTF-8?q?:=20=D0=B8=D1=81=D1=85=D0=BE=D0=B4,=20=D0=BD=D0=B0=D0=B7=D0=B2?= =?UTF-8?q?=D0=B0=D0=BD=D0=BD=D1=8B=D0=B9=20=D0=B4=D0=B5=D0=BB=D1=8C=D1=82?= =?UTF-8?q?=D0=B0-=D1=81=D0=BF=D0=B5=D0=BA=D0=BE=D0=B9=20=20=20=D0=BF?= =?UTF-8?q?=D0=BE=D0=B8=D0=BC=D1=91=D0=BD=D0=BD=D0=BE,=20=D0=BE=D1=82?= =?UTF-8?q?=D1=81=D1=83=D1=82=D1=81=D1=82=D0=B2=D0=B8=D0=B5=20=D0=BD=D0=BE?= =?UTF-8?q?=D0=B2=D1=8B=D1=85=20=D1=81=D1=86=D0=B5=D0=BD=D0=B0=D1=80=D0=B8?= =?UTF-8?q?=D0=B5=D0=B2,=20=D0=BF=D1=80=D0=B0=D0=B2=D0=BA=D0=B0=20=D1=81?= =?UTF-8?q?=D0=BE=D0=BE=D0=B1=D1=89=D0=B5=D0=BD=D0=B8=D0=B9=20=D0=B8=20?= =?UTF-8?q?=D1=82=D0=B5=D1=81=D1=82=D0=BE=D0=B2,=20=D1=81=D1=83=D0=B6?= =?UTF-8?q?=D0=B5=D0=BD=D0=B8=D0=B5=20=20=20=D1=81=D1=83=D1=89=D0=B5=D1=81?= =?UTF-8?q?=D1=82=D0=B2=D1=83=D1=8E=D1=89=D0=B5=D0=B9=20=D0=BD=D0=BE=D1=80?= =?UTF-8?q?=D0=BC=D0=B0=D0=BB=D0=B8=D0=B7=D0=B0=D1=86=D0=B8=D0=B8=20-=20?= =?UTF-8?q?=D0=B7=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BD=20=D0=BE=D1=80=D0=B8?= =?UTF-8?q?=D0=B5=D0=BD=D1=82=D0=B8=D1=80=20=D1=87=D0=B0=D1=81=D1=82=D0=BE?= =?UTF-8?q?=D1=82=D1=8B:=20deep=20=E2=80=94=20=D0=B8=D1=81=D0=BA=D0=BB?= =?UTF-8?q?=D1=8E=D1=87=D0=B5=D0=BD=D0=B8=D0=B5=20=D0=BD=D0=B0=20=D0=BA?= =?UTF-8?q?=D1=80=D1=83=D0=BF=D0=BD=D0=BE=D0=B9=20=D1=84=D1=83=D0=BD=D0=BA?= =?UTF-8?q?=D1=86=D0=B8=D0=BE=D0=BD=D0=B0=D0=BB=D1=8C=D0=BD=D0=BE=D1=81?= =?UTF-8?q?=D1=82=D0=B8,=20=20=20=D0=B7=D0=B0=D0=B4=D0=B0=D1=87=D0=B0-?= =?UTF-8?q?=D1=83=D0=B1=D0=BE=D1=80=D0=BA=D0=B0=20=D0=B8=D0=B4=D1=91=D1=82?= =?UTF-8?q?=20=D0=B2=20quick=20=D0=B8=D0=BB=D0=B8=20standard=20=D0=B4?= =?UTF-8?q?=D0=B0=D0=B6=D0=B5=20=D0=B2=20=D1=83=D0=B7=D0=BB=D0=B5=20=D0=B8?= =?UTF-8?q?=D0=B7=20=D0=BF=D0=B5=D1=80=D0=B5=D1=87=D0=BD=D1=8F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/review.md | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/docs/review.md b/docs/review.md index 79f18b2..6ae9741 100644 --- a/docs/review.md +++ b/docs/review.md @@ -157,6 +157,34 @@ merge-раскладка при повторном добавлении разд - «Поведение, видимое снаружи» здесь включает **тексты и карточки Telegram** — для единственного пользователя это и есть интерфейс. +**Место из списка ступень не поднимает — поднимает правило.** Перечни выше +отвечают «здесь такие правила водятся», а не «любая правка здесь идёт в `deep`». +Ступень поднимает то, что даёт работу новому проходу: заводится ключ сравнения +или меняется его состав; у разбора появляется новый вид входа или новая ветка +неоднозначности; в слияние добавляется источник или меняется победитель при +конфликте. + +**Отсекающие условия — проверяются до выбора профиля, любое сработавшее держит +ступень внизу.** Перечень закрытый, каждый пункт проверяется взглядом на дифф и +дельта-спеку: + +- **дельта-спека называет исход поимённо.** Независимая реализация окупается + выбором, которого спека не сделала. Если сценарий уже говорит, что даёт + вырожденный вход, реализация повторит спеку, и дифф покажет расхождение в + форме, а не в решении; +- **новых сценариев в дельта-спеке нет** — изменение уточняет уже описанное + поведение, а не заказывает новое; +- **правка сообщения, комментария, записи журнала, имени или теста** в узле из + перечня; +- **сужение уже существующей нормализации** без нового вида входа: вход остался + тот же, изменился исход на одном его значении. + +**Ориентир частоты.** `standard` закрывает большинство задач, `wide` — редкий +случай, `deep` — исключение на крупной функциональности, а не на уборке. Задача +типа `chore` или `bugfix`, собранная из нитей прошлого ревью, идёт в `quick` или +`standard`, даже когда трогает файл из перечня `deep`. Верхняя ступень чаще одной +задачи на спринт означает ошибку в критерии, а не спринт из сложных задач. + ### Недоступно проверке **Не проверит ни один проход** — принципиальная граница, по факту промаха не