av
42d5b73a04
- Раскладка docs/ приведена к канону 2: заведены passport/architecture/ database/security/review и research; docs/specs, drafts, backlog, review/ и BRIEF.md разобраны и удалены, беклог переехал в docs/tasks (34 задачи, 6 целей, слаги на английский). - Нарративы specs удалены как дубли openspec-спек после поимённой сверки; остаток заведён задачами (редактор маппинга ревью, крайние случаи именования), отказ от сущности title промоутнут в ADR. - Проектные копии агентов и скиллов ревью/пайплайна удалены в пользу плагинов av-dev-pm и av-dev-pipeline; в task gate добавлен шаг canon вместо er-schema.
60 lines
4.8 KiB
Markdown
60 lines
4.8 KiB
Markdown
# Агенты-ревьюверы качества (наименования, архитектура, конвенции, стиль)
|
|
|
|
- **Секция:** инфраструктура
|
|
- **Зачем:** конвейер ревью переехал в плагин `av-dev-pipeline`; осталась калибровка проходов на этом проекте и ревьювер наименований (ждёт словарь единого языка)
|
|
- **Теги:** goal:dev-process-quality
|
|
|
|
Набор проходов ревью поверх ревью-процесса из CLAUDE.md. Развивает ревью-процесс
|
|
OpenSpec в сторону воспроизводимых автопроверок, не заменяя человеческое ревью.
|
|
|
|
## Сделано (2026-07-10)
|
|
|
|
Заведены два кастомных ревьювера: оптика спек/требований и оптика кода
|
|
(архитектура, инварианты, конвенции, стиль, дублирование); оба подключены
|
|
чекпоинтами в пайплайн задачи.
|
|
|
|
## Сделано (2026-07-23) — переработка конвейера
|
|
|
|
Конвейер пересобран по типу проходов, а не по ролям: гейт → сверка со спекой в
|
|
обе стороны → generative-проходы → архитектура → враждебные постановки → триаж,
|
|
профили `quick`/`standard`/`deep`/`design`, контракт находок, границы покрытия,
|
|
храповик «находка → конвенция → правило → удаление», журнал проскочивших
|
|
дефектов и процедура калибровки. Подробности — ADR
|
|
[ADR-2026-07-23-review-pipeline-generative](../../adr/ADR-2026-07-23-review-pipeline-generative.md).
|
|
|
|
Открытый вопрос «дробить ли проход по конвенциям на узкие оптики» закрыт:
|
|
**не дробим** — декорреляция внимания без декорреляции суждения почти не
|
|
добавляет recall, но линейно удорожает триаж.
|
|
|
|
## Сделано (2026-08-04) — переезд в плагин
|
|
|
|
Проектные копии агентов (`.claude/agents/jellybit-review-*`) и скиллов
|
|
(`review-pipeline`, `task-pipeline`, `task-batch`) удалены в пользу плагина
|
|
`av-dev-pipeline`. Проектная специфика теперь приходит из документов канона —
|
|
[docs/review.md](../../review.md): типовые узлы, ложноположительные, вопросы к
|
|
проходам, триггеры профиля, недоступное проверке.
|
|
|
|
Два прохода плагин при этом **упразднил**, и это надо помнить:
|
|
|
|
- `idiom` — поимённая сверка со стайлгайдами языка не задаётся теперь ни одним
|
|
проходом; способные части переселены в `ops` и `architecture`. Класс
|
|
обратимый (портит форму кода, не данные) и признаётся в границах покрытия.
|
|
- `negative` — вопрос «что опытный человек отсюда удалил бы» вошёл в
|
|
`architecture` вторым обязательным.
|
|
|
|
## Осталось
|
|
|
|
- **Ревьювер наименований** (соответствие словарю единого языка) — отдельной
|
|
оптикой не выделен: зависит от задачи «Словарь единого языка», без глоссария
|
|
проверять не по чему. Завести после неё.
|
|
- **Калибровка проходов** по процедуре `references/calibration.md` скилла
|
|
`av-dev-pipeline:review-pipeline` — ни один проход ещё не замерен инъекцией.
|
|
До замера ничего не удаляем и промпты не правим.
|
|
- **Заполнить журнал дефектов** в [docs/review.md](../../review.md) случаями,
|
|
которые уже проскочили ревью, — они станут первыми пробами калибровки.
|
|
- **Решить судьбу упразднённых проходов:** нужен ли проекту свой `idiom` поверх
|
|
плагина, или записи в «Недоступно проверке» достаточно.
|
|
|
|
Связано: CLAUDE.md (ревью-процесс, конвенции),
|
|
[docs/conventions/](../../conventions/README.md), «Словарь единого языка».
|