вычитка обслуживания: план называет глубину, уставы знают прогон без метки
Проходы берут вход, потолки и состав половин из метки, а на прогоне обслуживания метки нет — оба взяли бы их наугад и молча по-разному. План сценария теперь называет глубину прямо: basics — сверка с потолком 2, code — вход small, потолки 3 и 2 и включённая третья половина. Третья половина code включена не для полноты: без неё security и architecture не смотрит вообще никто. Границы покрытия переписаны честно — requirements не смотрел никто, две другие темы сверены только против записанных инвариантов и только если проход по коду шёл. Уставы review-code и review-basics знают прогон без метки, их описания тоже; триаж знает, что план сценария встаёт на место плана разметки. В скилле задач уточнено: тип не выбирает метку, но предлагает сценарий, а меняется он командой edit --type, а не исполнителем по ходу. Плюс язык: заголовок, два оборота и right-size на русском.
This commit is contained in:
@@ -654,20 +654,26 @@ change**: у работы, не меняющей поведения, дельт
|
||||
Разметчик без своего корпуса вернул бы величину, выведенную из ничего, — и это
|
||||
хуже отсутствующей метки, потому что выглядит измеренным.
|
||||
|
||||
**План приходит вызовом и фиксирован сценарием**, а не выводится здесь:
|
||||
**План приходит вызовом и фиксирован сценарием**, а не выводится здесь. **Он же
|
||||
называет глубину и вход каждого прохода** — их обычный источник метка, и без неё
|
||||
проходы взяли бы их наугад:
|
||||
|
||||
| Тема | Кто закрывает | Когда |
|
||||
|---|---|---|
|
||||
| `autotests` | `review-autotests` | всегда |
|
||||
| `operations` | `review-basics`, глубина «сверка» | всегда |
|
||||
| `conventions` + технический разбор | `review-code` | дифф трогает код, а не только оснастку |
|
||||
| Тема | Кто закрывает | Глубина и вход | Когда |
|
||||
|---|---|---|---|
|
||||
| `autotests` | `review-autotests` | как обычно | всегда |
|
||||
| `operations` | `review-basics` | сверка, потолок 2 | всегда |
|
||||
| `conventions` + технический разбор | `review-code` | вход `small` (индекс конвенций), потолки 3 и 2, третья половина включена — потолок 1 | дифф трогает код |
|
||||
|
||||
Триаж обязателен и здесь — он единственный сток и единственный, кто сверяет план
|
||||
с исходом. Темы `requirements` и `security` в плане отсутствуют: у первой нет
|
||||
предмета, у второй нет проходчика на этом сценарии, — и обе обязаны быть названы
|
||||
в границах покрытия. Дом плана — сценарий, а не этот скилл:
|
||||
`av-dev-code:resolve`, `references/maintain.md`, раздел «Ревью — план фиксирован
|
||||
сценарием».
|
||||
с исходом; на его вход подаётся этот план вместо плана разметки. Тема
|
||||
`requirements` в плане отсутствует за отсутствием предмета; `security` и
|
||||
`architecture` закрыты только сверкой с записанными инвариантами внутри `code` —
|
||||
третья половина этого прохода включается здесь по той же причине, что и при
|
||||
метке `small`. Все три обязаны быть названы в границах покрытия, а **сигнал о
|
||||
заниженной метке на таком прогоне не работает**: поднимать нечего.
|
||||
|
||||
Дом плана — сценарий, а не этот скилл: `av-dev-code:resolve`,
|
||||
`references/maintain.md`, раздел «Ревью — план фиксирован сценарием».
|
||||
|
||||
**Правило гейта на таком прогоне работает жёстче обычного.** Правка, которая
|
||||
трогает сам гейт, проверяется гейтом же — инструмент проверяет себя, — поэтому
|
||||
@@ -865,6 +871,10 @@ Recall темы `conventions` равен длине конвенций прое
|
||||
стартует. Получает сырые выводы всех проходов, `git diff`, режим и **план
|
||||
разметки задачи**; возвращает финальный отчёт.
|
||||
|
||||
**На прогоне без change его место занимает план сценария** — см. «Прогон без
|
||||
change»: сверять исход с планом триаж обязан и там, а другого плана в том прогоне
|
||||
не существует.
|
||||
|
||||
**План на входе у триажа — не формальность, а сверка.** Он единственный, кто
|
||||
видит и то, что размечено, и то, что пришло: «тем размечено шесть, отчёты
|
||||
покрывают пять» — находка о самом прогоне, и заметить её больше некому. Раньше он
|
||||
|
||||
Reference in New Issue
Block a user