reviewer(ET): auto-commit from reviewer run_id=480

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: <YYYY-MM-DD>` + врезка «не копируй буквально, подставь `date +%F`». Литерала
+  `created_at: 2026-…` в fenced-блоках нет (закреплено `test_orch092_created_at_is_placeholder_not_literal`).
+- **AC-2 / FR-2 (расхардкод `model_used`):** PASS. `model_used: <resolve ORCH-41>` в примерах;
+  литерал `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 (`<escalation>`):** PASS. developer/reviewer/tester несут `<escalation>` ПОСЛЕ
+  `</success_criteria>` (порядок 5 обязательных секций цел); маршруты ролеспецифичны
+  (`back-to:analysis` / `back-to:dev` / `REQUEST_CHANGES`).
+- **AC-6 / FR-6+FR-10 (deployer рамка + язык):** PASS. Критичные self-hosting-запреты подняты в
+  видную рамку в начале `<context>` («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.
+Дополнительных обновлений не требуется.