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

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

@@ -0,0 +1,85 @@
+---
+verdict: APPROVED
+work_item: ORCH-108
+stage: review
+author_agent: reviewer
+status: approved
+created_at: 2026-06-17
+model_used: claude-opus-4-8
+type: review
+work_item_id: ORCH-108
+version: 1
+---
+
+# Review ORCH-108 — FAQ: как использовать STOP для отмены задачи
+
+## Summary
+
+Docs-only задача: создаёт пользовательский FAQ `docs/operations/FAQ_STOP.md` (формат «вопрос →
+ответ»), двусторонние перекрёстные ссылки витрина/обзор ⇄ FAQ ⇄ ADR ORCH-090 и детерминированный
+анти-дрейф тест `tests/test_faq_stop_doc.py`. Поведение STOP — источник истины ORCH-090
+(`adr-0026`) — **не меняется**.
+
+Проверены все 4 оси. **Соответствие ТЗ/AC (1–11)** — полное. **Соответствие ADR** — все решения
+D1–D5 реализованы. **Качество** — тест содержателен, детерминирован, с non-evergreen-самочеком.
+**Документация (приоритетная ось)** — CHANGELOG обновлён, витрина `docs/overview/` обновлена, ADR
+заведён, `src/**` не тронут → нет необновлённой документации. **P0/P1 findings отсутствуют.**
+
+Верификация по ключевым осям:
+- **AC-9 (фактическая корректность) — самая важная для docs-FAQ.** Все 9 утверждений FAQ сверены с
+  кодом (`src/stage_engine.py::cancel_task`, `src/cancel.py`, `src/webhooks/plane.py`,
+  `src/gitea.py`, `src/db.py`) — **каждое CONFIRMED**, противоречий коду нет: graceful SIGTERM-стоп
+  (`launcher.stop_process`); job'ы → терминальный `cancelled`, не реквью'ятся (`claim_next_job`
+  берёт только `queued`); удаление worktree + рабочей ветки с guard `_PROTECTED_BRANCHES={main,
+  master}` (никогда `main`/force-push); docs-артефакты сохраняются; отложенная отмена в критичном
+  окне (`cancel.in_critical_window` → только `queued`-job'ы, running-актор не трогается, finalizer
+  применяет позже); STOP не делает revert влитого; релонч гейтится строго стадией `analysis`
+  (дыра релонча ORCH-090 D6 закрыта); fail-closed no-op (нет `stop` в `_DEFAULT_STATES`) +
+  идемпотентный no-op для терминальной задачи + kill-switch `stop_status_enabled`/`stop_status_repos`.
+- **AC-8 / TR-4 (риск висячего якоря).** Внутренняя ссылка FAQ `…tech-pipeline.md#отмена-stop--cancelled`
+  корректно резолвится в заголовок `## Отмена: STOP → \`cancelled\`` (slug с двойным дефисом от
+  удалённого `→` — совпадает байт-в-байт). Цель ADR-ссылки
+  `docs/work-items/ORCH-090/06-adr/ADR-001-stop-cancel-task.md` существует. Обратные ссылки из
+  `business.md` (Сценарий 6) и `tech-pipeline.md` (Отмена: STOP → `cancelled`) присутствуют.
+- **AC-10 / AC-11.** `git diff origin/main...HEAD`: только `docs/**`, `CHANGELOG.md`,
+  `tests/test_faq_stop_doc.py` (+ scratch `.task-dev.md`); `src/**` / `STAGE_TRANSITIONS` /
+  `QG_CHECKS` / `check_*` / схема БД — не тронуты. `tests/test_faq_stop_doc.py` — 12 passed;
+  `tests/test_system_docs.py` (витрина) — 29 passed.
+
+## Findings
+
+### P0 — Blocker
+- _нет_
+
+### P1 — Must fix
+- _нет_
+
+### P2 — Should fix
+- _нет_
+
+### P3 — Nice-to-have
+- [ ] `.task-dev.md` (scratch-файл dev-трекинга в корне) попал в коммит, обновлён с `ORCH-126` на
+  `ORCH-108`. Это существующий трекируемый файл, к deliverables не относится, рантайм/конвейер не
+  затрагивает — инконсеквентно, фиксации не требует. Отмечено только для полноты.
+
+## Документация
+
+Приоритетная ось пройдена. Это **docs-задача**, `src/**` не изменён → требование «изменил `src/` →
+обнови доку в том же PR» не активируется; при этом всё, что задача обязана обновить, обновлено:
+- **`CHANGELOG.md`** — запись ORCH-108 присутствует (раздел `[Unreleased]`), с инвариантом docs-only
+  и нормативом сопровождения. ✓
+- **Витрина системы `docs/overview/` (ORCH-011)** — `business.md` (Сценарий 6) и `tech-pipeline.md`
+  (Отмена: STOP → `cancelled`) дополнены ссылкой на FAQ; `tests/test_system_docs.py` зелёный
+  (инвариант курируемого каталога витрины не нарушен — FAQ положен в `docs/operations/`, не в
+  `docs/overview/`, см. ADR D1). ✓
+- **ADR** — `docs/work-items/ORCH-108/06-adr/ADR-001-faq-stop-placement-and-anti-drift.md` заведён;
+  сквозной global-ADR обоснованно N/A (локальное docs-only решение, нет нового QG/стадии/компонента/
+  таблицы — критерий `PIPELINE_DOCS.md` §4). ✓
+- **README «Известные ограничения» (ORCH-079)** — ORCH-108 не закрывает ни один из открытых пунктов
+  (Telegram 48h / intra-repo deps / пакетный автоном); STOP уже документирован в README §«Отмена
+  задачи: статус STOP». Обновление README не требуется. ✓
+- **link-first (ADR D4)** — машинные детали (`тумбстон`/`merge-lease`/`_ensure_column`) в FAQ не
+  дублируются, даются ссылками; проверено тестом (`test_faq_links_back_to_overview_and_adr`). ✓
+
+Документация = golden source: обновлена корректно и согласованно. Нет необновлённой документации →
+блокирующего finding'а по этой оси нет.