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

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

@@ -0,0 +1,108 @@
+---
+verdict: APPROVED
+work_item: ORCH-019
+stage: review
+author_agent: reviewer
+status: approved
+created_at: 2026-06-10
+model_used: claude-opus-4-8
+type: review
+work_item_id: ORCH-019
+version: 1
+---
+
+# Review ORCH-019 — Багфикс-трек (упрощённый/дешёвый маршрут для багов)
+
+## Summary
+
+Реализация соответствует ТЗ (`02-trz.md`), ADR-001 и всем 9 критериям приёмки
+(`03-acceptance-criteria.md`). Корневой инвариант NFR-1 («срезается только аналитика/архитектура;
+ни один Quality Gate не тронут») соблюдён **структурно**: `src/stages.py` и `src/qg/checks.py` —
+**пустой diff**; маршрутизация багфикса реализована чистым routing-override в `advance_stage`, как
+и предписывал ADR (D3). Полный регресс `tests/` зелёный (**1551 passed**), 46 целевых тестов
+ORCH-019 (6 suites) — PASS. Документация обновлена исчерпывающе во всех требуемых поверхностях.
+Findings уровня P0/P1 — нет. → **APPROVED**.
+
+## Проверка по осям
+
+### 1. Соответствие ТЗ / Acceptance Criteria
+- **AC-1 (классификация по метке `Bug`)** ✓ — `bug_fast_track.is_bug_task` делегирует в
+  `labels.has_label` (источник истины — Plane API, не payload); `applies(repo)` (локальный)
+  проверяется ПЕРВЫМ в `start_pipeline` → при выключенном флаге нулевой сетевой оверхед.
+- **AC-2 (пропуск `architecture`)** ✓ — override на ребре выхода из `analysis`
+  (`next_stage → development`, `next_agent → developer`); `STAGE_TRANSITIONS`/`get_next_stage`/
+  `get_agent_for_stage` остались чистыми (1:1). Анти-регресс структуры — TC-06.
+- **AC-3 (все QG сохранены — корневой инвариант)** ✓ — `git diff` по `src/stages.py`/`src/qg/`
+  пуст; вердикт-ключи и порядок под-гейтов не тронуты (TC-07). Подтверждено независимой проверкой
+  diff, не только тестом.
+- **AC-4 (обязательный регресс-тест)** ✓ — ось добавлена в `.openclaw/agents/reviewer.md`
+  («фикс без теста-фиксатора → finding ≥P1»); `04-test-plan.yaml` несёт требование. (Сам ORCH-019 —
+  feature, не bugfix, поэтому правило к нему не применяется; покрытие — 46 содержательных тестов.)
+- **AC-5 (эскалация)** ✓ — `POST /bug-fast-track/escalate` (`db.set_task_track 'bug'→'full'`,
+  Telegram+Plane-коммент, never-raise) + self-escalate мини-аналитика (`analyst.md`).
+- **AC-6 (fail-safe / нулевая регрессия)** ✓ — `bug_fast_track_enabled` kill-switch; все публичные
+  функции leaf'а never-raise → False (full cycle); `get_task_track` деградирует в `'full'`.
+  Дефолт `True` согласован со всеми sibling-флагами (serial_gate/auto_label/coverage/stop/… все
+  `= True` при пустом scope = self-hosting only).
+- **AC-7 (наблюдаемость)** ✓ — read-only блок `bug_fast_track` в `GET /queue` (`snapshot()`,
+  never-raise) + отметка `🐞` в Telegram-карточке (never-raise) + лог-строки на решение.
+- **AC-8 (аддитивность / self-hosting)** ✓ — `_ensure_column(tasks, track, "TEXT DEFAULT 'full'")`
+  идемпотентна (TC-15); прод-контейнер/`main` не трогаются; полный `tests/` зелёный.
+- **AC-9 (композиция)** ✓ — serial-gate/auto-label/coverage/merge — тест композиции зелёный
+  (TC-14); override применяется ДО всех deploy-edge под-гейтов.
+
+### 2. Соответствие ADR
+Реализация точно следует ADR-001 (D1–D8): leaf `src/bug_fast_track.py`, колонка `tasks.track`,
+override в `advance_stage`, эскалация-эндпоинт, область self-hosting-only. Сквозной ADR
+`adr-0032-bug-fast-track.md` присутствует.
+**Трассировка:** `advance_stage` несёт маркеры ORCH-088/089/027/059/094; врезка ORCH-019 добавляет
+ветвление ТОЛЬКО на ребре выхода из `analysis` (до deploy-edge под-гейтов) — зафиксированные
+инварианты (порядок под-гейтов, merge-lease, terminal-sync) не нарушены. Сверено по diff. Расширение
+`mark_brd_review_ended` на `analysis → development` (ORCH-087 метрика) гейтов не касается.
+
+### 3. Качество кода
+- Leaf чист (импортирует только `config`, лениво `labels`/`db`/`qg.checks`), never-raise контракт
+  соблюдён везде, публичные функции снабжены docstrings. ✓
+- Next-agent override (`next_stage == "development"`) безопасен: единственный путь к
+  `analysis → development` — сам багфикс-override (штатно `get_next_stage("analysis") == "architecture"`). ✓
+- `get_task_by_work_item_id`/`add_comment`/`set_task_track`/`get_task_track` существуют и
+  совместимы по сигнатурам. ✓
+
+### 4. Документация — обязательная проверка
+`src/` изменён → документация ДОЛЖНА быть обновлена. **Обновлено в том же PR:**
+- `docs/architecture/README.md` — раздел «Багфикс-трек (ORCH-019)» + блок `bug_fast_track` в `GET /queue`;
+- `README.md` — таблица env (`ORCH_BUG_FAST_TRACK_*`) + обзорный раздел;
+- `.env.example` — три новых переменных;
+- `docs/architecture/adr/adr-0032-bug-fast-track.md` (сквозной) + `06-adr/ADR-001`;
+- `docs/architecture/internals.md`, `CLAUDE.md`, `CHANGELOG.md` (`feat:`);
+- `07-infra-requirements.md` / `08-data-requirements.md` / `10-tech-risks.md`;
+- `.openclaw/agents/analyst.md` (lite-пакет + self-escalate) и `reviewer.md` (ось регресс-теста).
+
+Все поверхности из §8 ТЗ покрыты. **Обзорная витрина README** — добавлен раздел, ничего из «Известных
+ограничений» не оставлено открытым в нарушение ORCH-079.
+
+## Findings
+
+### P0 — Blocker
+- (нет)
+
+### P1 — Must fix
+- (нет)
+
+### P2 — Should fix
+- (нет)
+
+### P3 — Nice to have
+- [ ] `snapshot.est_saved_architecture_runs == total_bug_tasks` считает ВСЕ багфикс-задачи, включая
+  `cancelled` (которые могли не дойти до пропуска `architecture`). Косметическая неточность метрики
+  экономии; на гейты/маршрут не влияет. Можно сузить до `stage NOT IN ('cancelled')` при случае.
+
+## Документация
+Обновлена полностью и согласованно во всех требуемых поверхностях (architecture/README, README env +
+обзор, оба ADR, internals, CLAUDE.md, CHANGELOG, .env.example, промпты analyst/reviewer,
+infra/data/risks). Расхождений код↔документация не обнаружено. Требований к доработке документации
+нет.
+
+## Вердикт
+Нет findings уровня P0/P1; документация обновлена; корневой инвариант подтверждён независимой
+проверкой diff и зелёным полным регрессом (1551 passed). → **APPROVED**.