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

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

@@ -0,0 +1,81 @@
+---
+verdict: APPROVED
+work_item: ORCH-078
+stage: review
+author_agent: reviewer
+status: approved
+created_at: 2026-06-09
+model_used: claude-opus-4-8
+type: review
+work_item_id: ORCH-078
+version: 1
+---
+
+# Review ORCH-078 — ORCH-52e: стандарт трассировки `ORCH-NNN` + правило чтения ADR
+
+## Summary
+
+Docs + prompts-only задача (слой 4 эпика ORCH-52). Проверено по 4 осям. Изменение **точно
+соответствует ТЗ и ADR-001 (D1–D7)**: введён нормативный стандарт `docs/_standards/TRACEABILITY.md`,
+точечно (аддитивно, без перезаписи 52d-канона) дополнены 3 промпта (developer/architect/reviewer),
+обновлены `CLAUDE.md` / `docs/architecture/README.md` / `CHANGELOG.md`, создан сквозной `adr-0022` +
+per-work-item `ADR-001`, расширен анти-регресс-тест.
+
+Ключевые факты подтверждены прогоном:
+- `git diff --name-only origin/main` — изменены **только** `docs/**`, `.openclaw/agents/*.md`,
+  `CLAUDE.md`, `CHANGELOG.md`, `tests/test_agent_prompts_canon.py`. **`src/**` не тронут** (AC-7 ✓).
+- `pytest tests/ -q` → **1253 passed**; `test_agent_prompts_canon.py` (53 — вкл. новые 9 TC) и
+  `test_agent_frontmatter_no_model.py` зелёные. Machine-verdict ключи сохранены байт-в-байт.
+- **Самопроверяемость стандарта подтверждена против реального кода:** `src/serial_gate.py:241,269`
+  действительно несёт `t2.id < jobs.task_id` + маркер `ORCH-088`; `src/merge_gate.py` несёт
+  ORCH-043/065/071/073 (и др.); `grep -rhoE 'ORCH-[0-9]+' src/ | sort -u | wc -l` → ровно **51**.
+  Цитируемые ADR (`ORCH-088/06-adr/ADR-001-serial-gate.md`) существуют. Трассировка рабочая.
+
+Поскольку `src/` не менялся, ось «правка чужого маркированного кода без сверки с ADR» неприменима
+(нет маркированного кода в дифе) — нарушений трассировки нет по построению.
+
+**Вердикт: APPROVED.** P0/P1 findings отсутствуют. Один P2 (несинхронный индекс ADR) — не блокер.
+
+## Findings
+
+### P0 — Blocker
+- Нет.
+
+### P1 — Must fix
+- Нет.
+
+### P2 — Should fix
+- [ ] **Индекс сквозных ADR не обновлён под adr-0022.** Создан новый cross-cutting
+  `docs/architecture/adr/adr-0022-traceability-marker-standard.md`, но таблица-реестр в
+  `docs/architecture/adr/README.md` (перечисляет adr-0001…adr-0021) **не получила строку adr-0022**.
+  Это противоречит конвенции `CLAUDE.md` («Новые ADR добавляет архитектор») и устоявшемуся паттерну
+  цепочки эпика 52 (adr-0019/0020/0021 — у каждого строка + footnote). Сам ADR-001 ORCH-078 (D6)
+  заявляет adr-0022 как «точку входа для будущих агентов» — отсутствие строки в индексе подрывает
+  именно тот discoverability-эффект, ради которого задача и делается. Дополнительно footnote индекса
+  «текущий максимум — `0020`» устарел (уже 0022). **Не блокер:** adr-0022 остаётся достижим из
+  `docs/architecture/README.md` (раздел «слой 4»), `TRACEABILITY.md` и work-item ADR.
+  **Рекомендация:** добавить строку `| adr-0022 | Стандарт маркеров-трассировки ORCH-NNN + правило
+  чтения ADR | proposed | 2026-06-09 | ORCH-078 |` и поправить footnote максимума.
+
+### P3 — Nice to have
+- Нет.
+
+## Документация
+
+**Проверка обязательна — выполнена явно.** `src/**` НЕ изменён → P0-правило «изменён src без
+обновления документации» **не триггерится**. Профильная документация задачи обновлена полно:
+
+| Документ | Статус |
+|----------|--------|
+| `docs/_standards/TRACEABILITY.md` | ✓ создан; 7 смысловых блоков (FR-1), реальный проверяемый пример, fallback, анти-археология, каноничное правило чтения |
+| `.openclaw/agents/developer.md` | ✓ правило чтения чужого маркера + fallback `git show origin/main:…` в формате «❌ X → ✅ Y» (AC-2, AC-4) |
+| `.openclaw/agents/architect.md` | ✓ правило чтения + анти-археология «3+ → сводный сквозной ADR» (AC-2, AC-5) |
+| `.openclaw/agents/reviewer.md` | ✓ усиление оси «Соответствие ADR» под-пунктом, finding ≥P1 (AC-3) |
+| `CLAUDE.md` | ✓ правило #9 + ссылка на `TRACEABILITY.md` (FR-7) |
+| `docs/architecture/README.md` | ✓ раздел «слой 4 (трассировка)» со ссылкой (FR-7) |
+| `CHANGELOG.md` | ✓ запись под `## [Unreleased]` (`docs`) |
+| `docs/work-items/ORCH-078/06-adr/ADR-001-*.md` | ✓ детальное решение D1–D7 |
+| `docs/architecture/adr/adr-0022-*.md` | ✓ файл создан, НО не внесён в индекс `adr/README.md` (см. P2) |
+
+Анти-дубль (AC-6) соблюдён: промпты **ссылаются** на единый текст правила в `TRACEABILITY.md`, не
+копируют; 52d-канон (5 XML-секций, 52c-эмиссия) сохранён — врезки строго аддитивны.