Files

T

ревью: скилл review-pipeline — стадии, профили, контракт находок, храповик

Конвейер собран по типу проходов, а не по ролям: гейт (детерминированный,
блокирующий) → сверка с дельта-спеками в обе стороны → generative-проходы →
архитектура → враждебные постановки → обязательный триаж. Профили quick /
standard / deep / design с правилом выбора по факту изменения.

Контракт находок: заголовок через последствие, обязательное поле «Последствие»,
critical без оракула или построенного пути не существует, потолок 7 пунктов и
разметка «инлайн | развилка» — отчёт читает оркестратор и молча реализует
прочитанное, поэтому потолок защищает код от незаказанных правок.

Храповик находка → конвенция → правило → удаление из прозы и промптов; журнал
проскочивших дефектов и калибровка инъекцией с вердиктами keep/retune/drop.
Секция границ покрытия обязательна: отчёт без неё потребляет ощущение
проверенности, ничего не гарантируя.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

2026-07-23 18:17:52 +03:00

6.3 KiB

Raw Blame History

Контракт находок

Единый формат для всех проходов конвейера ревью. Проход, нарушивший контракт, считается сломанным — триаж вправе выбросить его вывод целиком.

Форма находки

### <краткая формулировка ПОСЛЕДСТВИЯ, не симптома>
- Файл: internal/layout/link.go:120-134
- Severity: critical | major | minor | nit
- Confidence: high | medium | low
- Оракул: <падающий тест / команда с выводом / положение гайда / нет>
- Последствие: <что произойдёт и при каких условиях>
- Предложение: <конкретное изменение>
- Найдено проходом: <имя агента>

Правила

Заголовок через последствие. Не «нет проверки владельца», а «пользователь может прочитать чужой заказ по id». Не «путь не санитизируется», а «архив с ../ в имени файла разложит хардлинк вне paths.movies». Симптом в заголовке — это заявка на то, что читатель сам достроит последствие; он не достроит, он просто починит симптом.
critical без оракула или построенного пути не существует. Оракул — это падающий тест, вывод выполненной команды или поимённое положение гайда. Не «вероятно, здесь гонка», а CGO_ENABLED=1 go test -race с выводом детектора.
confidence: low — это «так обычно пишут». Такие находки допустимы, но не поднимаются выше minor. Частотность конструкции в публичном Go — не аргумент.
Находка без поля «Последствие» не выводится вовсе. Пустое «Последствие: ухудшает читаемость» равносильно отсутствию поля.
nit допустим только при нарушении записанной конвенции — со ссылкой на файл и раздел docs/conventions/* либо на правило .golangci.yml. Если правило механизируемо, но не механизировано — это не находка ревью, это Promote candidate (см. promote.md).
Расхождение — не дефект, пока не названо последствие. Особенно для jellybit-review-reimpl: «я бы сделал иначе» без последствия не выводится.

Шкала severity

Severity	Что это	Пример
`critical`	нарушение инварианта безопасности данных, потеря/порча данных, утечка секрета, построенный путь к отказу	хардлинк за пределы `paths.movies`, пароль qBittorrent в поле лога
`major`	сломанное требование дельта-спеки, необрабатываемый отказ штатного сценария, флаки-тест, поведение вне спеки, меняющее исход	ретрай, которого нет в спеке, маскирует ошибку записи в БД
`minor`	отступление от конвенции с реальной ценой, отсутствующая наблюдаемость, дублирование, которое разойдётся	стадия воркера не пишет `state transition`, разбор по логам невозможен
`nit`	нарушение записанной конвенции без последствий за пределами чтения	`msg` с интерполяцией вместо константы

Блок границ покрытия

Каждый проход завершает вывод этим блоком. Он не сокращается и не заменяется фразой «всё проверено».

## Coverage of this pass
- проверено: <что реально прочитано/запущено, с путями и командами>
- не проверялось и почему: <бюджет, недоступный инструмент, вне входа>
- принципиально недоступно этому проходу: <из charter'а агента>

Финальный отчёт триажа

Секции строго в этом порядке, потолок — 7 пунктов в первых двух:

Блокирует мердж (≤3, каждая с оракулом);
Стоит исправить сейчас (≤4);
Гипотезы без доказательства — что понижено и почему;
Promote candidates — кандидаты в конвенцию или правило линтера;
Границы покрытия — сводная, обязательная.

Каждая находка в секциях 1–2 несёт дополнительное поле:

- Действие: инлайн | развилка

инлайн — оркестратор чинит сам, не спрашивая и не логируя. развилка — цена исправления сопоставима с переработкой, либо выбор меняет scope, либо решение трогает инвариант: идёт человеку через AskUserQuestion вопросом с вариантами.

Потребитель отчёта — оркестратор, который реализует прочитанное. Поэтому потолок в 7 пунктов — не забота о внимании читателя, а защита кодовой базы от правок, которых никто не заказывал.

6.3 KiB Raw Blame History Unescape Escape