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

docs/work-items/ORCH-077/12-review.md

@@ -0,0 +1,82 @@
+---
+verdict: APPROVED
+work_item: ORCH-077
+stage: review
+author_agent: reviewer
+status: approved
+created_at: 2026-06-09
+model_used: claude-opus-4-8
+type: review
+work_item_id: ORCH-077
+version: 1
+---
+
+# Review ORCH-077 — ORCH-52d: канон Anthropic для 6 системных промптов + эмиссия схемы 52c
+
+## Summary
+
+Чистое **docs/prompts-only** изменение (замыкает эпик 52). Тело 6 промптов
+`.openclaw/agents/*.md` переписано в едином каноне Anthropic; каждый аддитивно эмитит
+6-польную frontmatter-схему 52c, не трогая machine-verdict ключи. Проверено построчно по
+4 осям (соответствие ТЗ, ADR, качество кода, документация) и по всем 9 критериям приёмки.
+**Нет P0/P1 findings → `APPROVED`.**
+
+Изолированный коммит задачи (`e155b01`) затрагивает ровно: 6 промптов, `CHANGELOG.md`,
+`CLAUDE.md`, `tests/test_agent_prompts_canon.py`, `tests/manual/ab_prompt_compare.md`.
+`src/**` (включая `config.py`, `launcher.py`, `frontmatter.py`, `stages.py`, `qg/checks.py`,
+`stage_engine.py`) — **не тронут**. (Прочие `src/`-изменения в трёхточечном diff против `main`
+принадлежат уже влитому ORCH-076 — к этому PR не относятся.)
+
+### Сверка критериев приёмки
+- **AC-1** (5 XML-секций `<context>`/`<task>`/`<deliverables>`/`<constraints>`/`<output_format>`
+  во всех 6): ✅ — подтверждено чтением всех 6 файлов + `test_agent_prompts_canon.py`.
+- **AC-2** (6 полей схемы в `<output_format>`, роле-специфичные `stage`/`author_agent`,
+  `model_used: claude-opus-4-8`): ✅ — analyst/architect/developer/reviewer/tester/deployer,
+  значения совпадают с картой TRZ §FR-2.
+- **AC-3** (ссылка на `docs/_templates/` + эталон ORCH-073/088 + ❌→✅ позитивные альтернативы):
+  ✅ во всех 6.
+- **AC-4** (анти-регресс инвентарь §FR-6): ✅ — verdict `APPROVED|REQUEST_CHANGES`+«src изменён,
+  доки нет→REQUEST_CHANGES» (reviewer); `result: PASS|FAIL`+pytest+smoke `/health`/`/status`/`/queue`
+  (tester); canonical `docker exec orchestrator-staging … staging_check.py`+B6-обоснование+ORCH-061
+  waiver+`pr_already_merged`+«не рестартить 8500» (deployer); 4 deliverable+Write-tool (analyst);
+  ADR-формат+сквозной ADR+эскалация (architect); TDD+«не мержить свой PR»+`--no-verify`/`--force-push`+
+  «не рестартить прод» (developer).
+- **AC-5** (код/гейты нетронуты): ✅ — `src/**` не изменён; `frontmatter_validation_strict = False`
+  (src/config.py:565).
+- **AC-6** (A/B «не хуже»): ✅ метод зафиксирован (`tests/manual/ab_prompt_compare.md`, in-vivo);
+  фактический результат фиксирует тестер в `13-test-report.md` — downstream-ответственность стадии
+  testing, не блокер review.
+- **AC-7** (документация): ✅ — ADR-001 (19KB) + сквозной adr-0021 + раздел «Слой промптов 52d» в
+  `docs/architecture/README.md` + `CLAUDE.md` + запись `## [Unreleased]` в `CHANGELOG.md`.
+- **AC-8** (frontmatter промптов валиден, без `model:`): ✅ — `test_agent_frontmatter_no_model.py`
+  зелёный.
+- **AC-9** (полный регресс): ✅ — `pytest tests/ -q` → **1244 passed**; новые структурные тесты
+  (44 passed) проходят.
+
+## Findings
+
+### P0 — Blocker
+- нет.
+
+### P1 — Must fix
+- нет.
+
+### P2 — Should fix
+- нет (блокирующих). Замечание (информационное, не требует правки): фактический результат A/B
+  (AC-6) ещё не записан — это корректно делегировано стадии `testing` (`13-test-report.md`), метод
+  уже зафиксирован в `tests/manual/ab_prompt_compare.md`. На вердикт review не влияет.
+
+## Документация
+
+**Полностью обновлена в рамках ветки** (требование CLAUDE.md «документация = golden source»
+выполнено):
+- `CLAUDE.md` — раздел про эпик 52 / ORCH-077 (канон + эмиссия схемы).
+- `docs/architecture/README.md` — раздел «#### Слой промптов: канон Anthropic + эмиссия схемы 52c
+  (ORCH-077, 52d)» точно описывает реализацию (5 секций, аддитивная схема, loading-model,
+  анти-регресс).
+- `docs/work-items/ORCH-077/06-adr/ADR-001-anthropic-prompt-canon.md` — per-work-item ADR.
+- `docs/architecture/adr/adr-0021-prompt-canon-anthropic.md` — сквозной ADR.
+- `CHANGELOG.md` — развёрнутая запись под `## [Unreleased]`.
+
+Поскольку `src/**` не изменён, обязательное правило «src/ изменён, а документация нет →
+REQUEST_CHANGES» неприменимо; при этом документация всё равно обновлена сверх минимума. Замечаний нет.