diff --git a/docs/work-items/ORCH-092/12-review.md b/docs/work-items/ORCH-092/12-review.md new file mode 100644 index 0000000..bee166c --- /dev/null +++ b/docs/work-items/ORCH-092/12-review.md @@ -0,0 +1,94 @@ +--- +verdict: APPROVED +work_item: ORCH-092 +stage: review +author_agent: reviewer +status: approved +created_at: 2026-06-09 +model_used: claude-opus-4-8 +type: review +work_item_id: ORCH-092 +version: 1 +--- + +# Review ORCH-092 + +## Summary + +Промпт-аудит 6 агентов (эпилог эпика ORCH-52), **docs/prompts-only**. PR проверен по 4 осям: +соответствие ТЗ (`02-trz.md` FR-1…FR-11), соответствие ADR (`06-adr/ADR-001`), качество кода, +качество документации. Все критерии приёмки `03-acceptance-criteria.md` (AC-1…AC-10) выполнены; +машинные verdict-ключи и канон 52d/52c/52e — байт-в-байт; `src/**`/`STAGE_TRANSITIONS`/`QG_CHECKS`/ +схема БД не тронуты. Полный регресс `pytest tests/ -q` — **1278 passed**; целевые +`test_agent_prompts_canon.py` + `test_agent_frontmatter_no_model.py` — **75 passed**. + +**Вердикт: APPROVED.** P0/P1/P2-findings нет. + +## Findings + +### P0 — Blocker +- _нет_ + +### P1 — Must fix +- _нет_ + +### P2 — Should fix +- _нет_ + +## Проверка по осям + +### Ось 1 — Соответствие ТЗ / Acceptance +- **AC-1 / FR-1 (расхардкод `created_at`):** PASS. Во всех 6 промптах копируемый блок несёт + `created_at: ` + врезка «не копируй буквально, подставь `date +%F`». Литерала + `created_at: 2026-…` в fenced-блоках нет (закреплено `test_orch092_created_at_is_placeholder_not_literal`). +- **AC-2 / FR-2 (расхардкод `model_used`):** PASS. `model_used: ` в примерах; + литерал `claude-opus-4-8` остаётся лишь справкой в таблице полей (вне блока). +- **AC-3 / FR-3 (сверка гейтов):** PASS. Все `check_*` из 6 промптов присутствуют в `QG_CHECKS` + (интеграционный `test_orch092_gate_names_match_qg_registry`); `check_tests_passed` подтверждён + валидным, не «исправлен вслепую». +- **AC-4 / FR-4 (PR>1500 → эскалация):** PASS. «разбивай на меньшие PR» удалено, переформулировано + в декомпозицию на уровне задач; маркер «свой PR» сохранён. +- **AC-5 / FR-5 (``):** PASS. developer/reviewer/tester несут `` ПОСЛЕ + `` (порядок 5 обязательных секций цел); маршруты ролеспецифичны + (`back-to:analysis` / `back-to:dev` / `REQUEST_CHANGES`). +- **AC-6 / FR-6+FR-10 (deployer рамка + язык):** PASS. Критичные self-hosting-запреты подняты в + видную рамку в начале `` («NEVER restart the prod 8500»); язык — EN по решению ADR D2. +- **AC-7 / FR-7 (обогащение tester):** PASS. worktree-путь ветки задачи, smoke-проверка блока + `serial_gate` в `/queue`, требование покрытия каждого TC из `04-test-plan.yaml`. +- **AC-8 / FR-8 (мёртвая строка reviewer):** PASS. «не апрувь PR от того же экземпляра Developer» + удалена; живые инварианты (`REQUEST_CHANGES`, «НЕ обновлена», ось трассировки, ось обзорных доков + ORCH-079) сохранены. +- **AC-9 / FR-11 (анти-регресс):** PASS. verdict-ключи `verdict:`/`result:`/`staging_status:`/ + `deploy_status:`/`security_status:` (+ значения) байт-в-байт; 5 секций + 6 полей во всех 6 + промптах; `git diff --stat -- src/` пуст. +- **AC-10 (документация и тесты):** PASS — см. ось 4. + +### Ось 2 — Соответствие ADR +- **ADR-001 D1 (убран ручной rebase):** реализовано точно. Шаг `git fetch origin && git rebase + origin/main` удалён из алгоритма developer, заменён нормативной заметкой; маркер запрета + `--force-push` сохранён. Код-обоснование ADR **верифицировано**: `premerge_rebase_always=True` + (config.py:432, дефолт), `auto_rebase_onto_main` + `--force-with-lease` (merge_gate.py:113/151), + `_materialize_deferred_branch` (launcher.py:421). Логика верна: developer'у force-push запрещён, + а пост-push rebase его требует → ручной шаг был конфликтующим/дублирующим. +- **ADR-001 D2 (deployer EN):** реализовано — `deployer.md` оставлен на английском с явной + «Language note» как задокументированное исключение; рамка запретов на EN. +- **Глобальные ADR:** нарушений нет. Трассировка (`TRACEABILITY.md`): удалённая rebase-строка не + несла маркера `ORCH-NNN`; зафиксированные инварианты не сломаны. + +### Ось 3 — Качество кода +- `src/` не изменён. Новые TC в `test_agent_prompts_canon.py` содержательны (парсер fenced-блоков, + anti-literal regex, ролеспецифичные маршруты, интеграция с реальным реестром `QG_CHECKS`), не + тривиальны. Все зелёные. + +### Ось 4 — Документация (ОБЯЗАТЕЛЬНАЯ ПРОВЕРКА) +- `src/**` НЕ изменён → правило «src изменён, документация не обновлена = P0» не триггерится. +- Документация обновлена сверх требуемого: **CHANGELOG.md** (запись ORCH-092 под `[Unreleased]`), + **CLAUDE.md** (абзац ORCH-092 в «Стек/Агенты»), **docs/architecture/README.md** (§«Слой промптов» + — пункт ORCH-092), **ADR** `06-adr/ADR-001-developer-rebase-and-deployer-language.md` (решения + P1-2/P2-2). +- **Обзорные доки (ORCH-079):** PR не закрывает ни одного пункта `README.md` «Известные + ограничения» → обновление витрины не требуется. + +## Документация +Обновлена полностью и корректно: CHANGELOG.md, CLAUDE.md, docs/architecture/README.md, ADR-001. +Дополнительных обновлений не требуется.