Compare commits

..

14 Commits

Author SHA1 Message Date
babc475ec3 analyst(ET): auto-commit from analyst run_id=797
All checks were successful
CI / test (push) Successful in 1m15s
2026-06-17 20:59:30 +03:00
f737e6730e analyst(ET): auto-commit from analyst run_id=796
All checks were successful
CI / test (push) Successful in 1m15s
2026-06-17 20:35:32 +03:00
d8ab31d200 analyst(ET): auto-commit from analyst run_id=795
All checks were successful
CI / test (push) Successful in 1m17s
2026-06-17 19:23:14 +03:00
66428edb69 docs: init ORCH-020 business request
All checks were successful
CI / test (push) Successful in 1m15s
2026-06-17 19:16:29 +03:00
e5dc4eb655 Merge pull request 'docs(operations): FAQ по статусу STOP для пользователя (ORCH-108)' (#148) from feature/ORCH-108-19c40858 into main
Some checks failed
CI / test (push) Has been cancelled
2026-06-17 18:23:09 +03:00
deploy-finalizer
618321addb deploy(ORCH-036): finalize SUCCESS for ORCH-108
All checks were successful
CI / test (push) Successful in 1m18s
CI / test (pull_request) Successful in 1m15s
2026-06-17 18:23:07 +03:00
staging-runner
1d552aa6aa staging(ORCH-115): staging gate SUCCESS for ORCH-108
All checks were successful
CI / test (push) Successful in 1m14s
CI / test (pull_request) Successful in 1m16s
2026-06-17 18:02:03 +03:00
test-runner
fedd5fab73 test(ORCH-116): test gate PASS for ORCH-108
Some checks failed
CI / test (push) Has been cancelled
CI / test (pull_request) Successful in 1m23s
2026-06-17 18:00:27 +03:00
21dd5a7eb0 reviewer(ET): auto-commit from reviewer run_id=794
All checks were successful
CI / test (push) Successful in 1m20s
CI / test (pull_request) Successful in 1m17s
2026-06-17 17:58:42 +03:00
dbd8df6eb2 docs(operations): add user FAQ for STOP cancellation status (ORCH-108)
All checks were successful
CI / test (push) Successful in 1m15s
CI / test (pull_request) Successful in 1m16s
Create docs/operations/FAQ_STOP.md — a user-facing "вопрос → ответ" FAQ for
Plane board users explaining the STOP status: what it does, how to cancel a
task, step-by-step consequences (agent stops → jobs cancelled → working branch
and worktree removed → task → cancelled → Telegram+Plane; docs artifacts
preserved, main/prod untouched), deferred cancellation in the critical
merge/deploy window, explicit "STOP does NOT revert merged/deployed code"
(revert is a separate task), restart only via "To Analyse" from scratch, no-op
causes, where to observe the result, and STOP/Approved/Confirm Deploy disambig.

docs-only: src/**, STAGE_TRANSITIONS, QG_CHECKS, check_*, machine-verdict keys
and the DB schema are byte-for-byte untouched. STOP behaviour source of truth
remains ORCH-090 (adr-0026); the FAQ documents and links to it (link-first:
machine details markers/lease/tombstone given by reference, not duplicated).

Add two-way cross-links: docs/overview/business.md (Сценарий 6) and
docs/overview/tech-pipeline.md (Отмена: STOP → cancelled) → FAQ; FAQ → overview
+ ADR ORCH-090. Structure guarded by deterministic anti-drift test
tests/test_faq_stop_doc.py (offline, no network/LLM/subprocess; mirrors
tests/test_lite_setup_doc.py): existence + 8 section anchors + fact bricks +
cross-links + claim-level negative scan (sentence-level, not bare substrings,
so it never false-fails on correctly negating phrases — TR-3, with a
non-evergreen self-check). Full pytest tests/ green (2227 passed).

Refs: ORCH-108

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-17 17:50:58 +03:00
e97111dc74 architect(ET): auto-commit from architect run_id=792
All checks were successful
CI / test (push) Successful in 1m15s
2026-06-17 17:42:59 +03:00
d11daf11f7 analyst(ET): auto-commit from analyst run_id=791
All checks were successful
CI / test (push) Successful in 1m15s
2026-06-17 17:26:06 +03:00
56825e79fe docs: init ORCH-108 business request
All checks were successful
CI / test (push) Successful in 1m20s
2026-06-17 17:17:50 +03:00
37cbaf5b86 Merge pull request 'fix(webhooks): source-backed 00-business-request.md instead of hardcoded TBD (ORCH-119)' (#147) from feature/ORCH-119-bug-00-business-request-md-is- into main
Some checks failed
CI / test (push) Has been cancelled
2026-06-17 17:17:48 +03:00
24 changed files with 2303 additions and 16 deletions

View File

@@ -1,4 +1,4 @@
Work item: ORCH-126
Work item: ORCH-108
Repo: orchestrator
Branch: feature/ORCH-126-bug-queued-job-can-keep-stale-
Branch: feature/ORCH-108-19c40858
Stage: development

View File

@@ -3,6 +3,26 @@
Формат: [Keep a Changelog](https://keepachangelog.com/). Записи — на смысловой PR/задачу.
## [Unreleased]
- **FAQ по статусу STOP для пользователя доски** (ORCH-108, `docs`): создан пользовательский
FAQ `docs/operations/FAQ_STOP.md` в формате «вопрос → ответ» — что делает STOP, как отменить
задачу, что происходит пошагово (агент останавливается → job'ы снимаются → рабочая ветка и
worktree удаляются → задача → `cancelled` → Telegram+Plane; **docs-артефакты сохраняются**,
`main`/прод не трогаются), отложенная отмена в критичном окне (merge/deploy), явное «STOP **не
откатывает** влитый в `main`/прод код» (revert — отдельная задача), перезапуск только через
«To Analyse» с нуля, причины no-op («ничего не произошло»), где увидеть результат, и разведение
STOP/Approved/Confirm Deploy. **docs-only:** `src/**` / `STAGE_TRANSITIONS` / `QG_CHECKS` /
`check_*` / machine-verdict / схема БД — байт-в-байт не тронуты; поведение STOP — источник истины
ORCH-090 (`adr-0026`), FAQ его лишь документирует и ссылается, не форкая (link-first: машинные
детали маркеры/lease/тумбстон — только ссылками). Добавлены двусторонние перекрёстные ссылки:
витрина `docs/overview/business.md` (Сценарий 6) и обзор `docs/overview/tech-pipeline.md`
(«Отмена: STOP → `cancelled`») → FAQ; FAQ → обзор + ADR ORCH-090. Структуру FAQ закрывает
детерминированный анти-дрейф тест `tests/test_faq_stop_doc.py` (offline, без сети/LLM/subprocess;
образец `tests/test_lite_setup_doc.py`): существование + 8 секций-якорей + факты-«кирпичи» +
кросс-ссылки + **негативный скан запрещённых утверждений на уровне предложений, а не голых
подстрок** (не фолзит на корректно отрицающих фразах — TR-3, проверено non-evergreen-самочеком).
**Норматив сопровождения:** правишь поведение STOP (`src/cancel.py` / `cancel_task` / маршрут
`stop`) → обнови `docs/operations/FAQ_STOP.md` в том же PR. ADR:
`docs/work-items/ORCH-108/06-adr/ADR-001-faq-stop-placement-and-anti-drift.md`.
- **Source-backed `00-business-request.md` вместо хардкода `TBD`** (ORCH-119, `fix`, Bug-трек): раздел «Description» файла `00-business-request.md` теперь несёт **фактический текст запроса** из Plane-issue (`description`/`description_stripped`) вместо литерала `TBD` — терялся source-backed контекст запроса. Фикс работает на **обоих** путях создания: прямой (путь A, `serial_gate` не применим — `start_pipeline` передаёт `description` в `_create_initial_docs`) и **отложенный срез ветки** (путь B, ORCH-088, доминирует на self-hosting `orchestrator`). Для пути B `description` **персистится durable** при создании задачи (аддитивная колонка `tasks.description` через `_ensure_column`, зеркало `tasks.title`, записывается **внутри того же атомарного INSERT** `create_task_atomic` — race-safe относительно анти-dup-claim ORCH-053) и читается из строки `tasks` в `launcher._spawn``_materialize_deferred_branch` на момент claim (без сетевого вызова в горячем пути, NFR-4). **Fail-safe (FR-4):** пустое/whitespace/`None`/нечитаемое описание → явный безопасный маркер `_(описание отсутствует в источнике)_` через чистый рендер-хелпер `_render_business_request` (never-raise; создание задачи не падает). **Идемпотентность:** Gitea 422 (файл существует) → no-op, ранее записанное тело не перезаписывается. **Инвариант (AC-5):** `STAGE_TRANSITIONS`/`QG_CHECKS`/`check_*`/machine-verdict-ключи — байт-в-байт; единственное изменение схемы — аддитивная `tasks.description` (базовый `CREATE TABLE tasks` не тронут); анти-stale-base инвариант ORCH-088 цел (момент/условие среза не меняются — только источник данных дополняется). Обратимость — revert PR (колонка остаётся инертной). Покрытие — `tests/test_orch119_business_request.py` (TC-01 обязательный регресс red→green; TC-02…TC-07). Дополнительно в том же PR починена **тест-гермеичность** `tests/test_orch123_staging_runner_exec.py::test_r2_held_deploy_staging_not_rolled_back`: тест переиспользовал собственный (теперь смерженный в `main`) work-item id `ORCH-123`, и при дефолтном `repos_dir=/repos` staging-гейт через origin/main-fallback (`check_staging_status``_staging_log_from_main`) находил **реальный** `staging_status: SUCCESS`-лог ORCH-123 в `origin/main`, делая намеренно-красный гейт зелёным (флак проявлялся только в полном прогоне сьюта — singleton `repos_dir` берётся из первого импортирующего config файла, побеждая import-time `ORCH_REPOS_DIR=/tmp` этого модуля); autouse-фикстура `fresh_db` теперь пинит `repos_dir` в изолированный пустой tmp-каталог (зеркало уже пиннимого `worktrees_dir`), сохраняя проверяемость инварианта ORCH-123 R-2 (infra-held `deploy-staging` удерживается, не откатывается в `development`) независимо от порядка тестов. ADR: `docs/work-items/ORCH-119/06-adr/ADR-001-source-backed-business-request-doc.md`.
- **Открытые вопросы аналитика → Needs Input (приоритет, неблокирование serial-gate, resume)** (ORCH-120, `fix`, трек Bug→escalate full-cycle): активирован и достроен ранее **мёртвый** путь «аналитик задаёт блокирующие вопросы → `01-questions.md` → Needs Input». Четыре согласованных изменения, аддитивно, под kill-switch, скоуп self-hosting, never-raise; `STAGE_TRANSITIONS` / реестр `QG_CHECKS` / семантика и имена `check_*` / machine-verdict-ключи / схема БД — **байт-в-байт не тронуты** (поток — pre-gate-ветка движка, **не** Quality Gate; `01-questions.md`**сигнальный** артефакт, **не** machine-verdict). (1) **Контракт + канон.** `.openclaw/agents/analyst.md` документирует канал «блокирующие вопросы → `01-questions.md`, НЕ фабриковать deliverables» + поведение на resume; новый скелет `docs/_templates/01-questions.md`; строка манифеста + примечание о префиксе `01-` в `docs/_standards/PIPELINE_DOCS.md`. (2) **Приоритет «вопросы активны» > «файлы готовы»** в `_handle_analysis_approved_flow` (DQ-3): чистая логика решения вынесена в leaf `src/analyst_questions.py` (`questions_gate_applies`/`autopause_applies`/`questions_active`), side-effects — в `stage_engine` (`_decide_analysis_outcome`/`_emit_analysis_needs_input`/`_emit_analysis_in_review`/`_emit_analysis_empty`); блокирующие вопросы достигают Needs Input даже при сфабрикованном полном пакете. (3) **Авто-park (DQ-1)** при Needs Input через ось «пауза» ORCH-124 (`db.set_task_paused`) → задача исключается из «активного» предиката serial-gate (ORCH-088), FIFO репо не клинит, пока ждём человека; **resume + unpark** в `handle_status_start` (analysis-ветка, `db.clear_task_paused`). (4) **Гигиена устаревания (DQ-2)** — детерминированный offline freshness-supersede по `mtime` (вопросы активны, пока пакет неполон ИЛИ `01-questions.md` не старше всех 4 deliverables) → полный свежий пакет supersedeит старый файл без зависимости от LLM (нет бесконечной петли Needs Input). Флаги (`config.py`, безопасные дефолты): `analyst_questions_gate_enabled` (kill-switch) / `analyst_questions_gate_repos` (CSV; **пусто → self-hosting only**) / `analyst_needs_input_autopause_enabled` (независимый тумблер авто-park/unpark; `False` → operator-park `POST /serial-gate/pause`). off/out-of-scope → байт-в-байт как до ORCH-120 (enduro не затронут); ORCH-066 (Needs Input только у аналитика) не расширяется. Покрытие — `tests/test_orch120_analyst_needs_input.py` (TC-01 обязательный регресс: красный до фикса, зелёный после), `tests/test_orch120_serial_gate_needs_input.py`, `tests/test_orch120_resume_unpark.py`, `tests/test_orch120_questions_artifact_canon.py` + assert в `tests/test_agent_prompts_canon.py`. Витрина системы `docs/overview/` обновлена в том же PR (ось ORCH-011): абзац пауз `tech-pipeline.md` и пункт `GET /queue` в `tech-observability.md` теперь называют **два** источника паузы (оператор + авто-park движком на Needs Input), `tech-agents.md` — when-applicable сигнальный канал `01-questions.md` у `analyst` (`tests/test_system_docs.py` зелёный). ADR: `docs/work-items/ORCH-120/06-adr/ADR-001-analyst-open-questions-needs-input.md`, сквозной `docs/architecture/adr/adr-0053-analyst-open-questions-needs-input-flow.md`.
- **Гигиена run-ownership строки `jobs` — инвариант «queued ⇒ run_id/pid/started_at IS NULL»** (ORCH-126, `fix`, трек Bug): багфикс контрол-плейна (инцидент ORCH-124/125) — при `ORCH_SERIAL_GATE_ENABLED=false` queued analyst-job'ы зависали навсегда (job 2286: `status=queued + run_id=759/760 + pid=35/42 + started_at=NULL` — физически невозможное состояние). **Причина:** ни один путь возврата job в `queued` (restart `requeue_running_jobs` / retry `mark_job('queued')` / transient `mark_job_transient` / reap `reap_running_job('queued')`) **не сбрасывал run-ownership** (`run_id`/`pid`); после рестарта контейнера pid мог быть **переиспользован** ОС`pid_alive(stale)=True` → job-reaper (ORCH-065) Tier-1 «видел живой» фантомный `running` и при `max_concurrency=1` клинил клейм **всей** общей очереди всех проектов. **Инвариант (adr-0052):** `status='queued' ⇒ run_id IS NULL AND pid IS NULL AND started_at IS NULL` — queued-job никогда не несёт run-ownership (история run'а — в `agent_runs`, не в `jobs.run_id`). Фикс на **существующих колонках**: `STAGE_TRANSITIONS` / реестр `QG_CHECKS` / `check_*` / machine-verdict-ключи / **схема БД** — байт-в-байт не тронуты; для здоровых job'ов и enduro поведение байт-в-байт; миграция не требуется. ADR: `docs/work-items/ORCH-126/06-adr/ADR-001-queued-job-run-ownership-hygiene.md`, сквозной `docs/architecture/adr/adr-0052-queued-job-run-ownership-invariant.md`.

View File

@@ -0,0 +1,87 @@
# FAQ: отмена задачи через статус STOP
Эта страница — для пользователя доски Plane. Она объясняет простыми словами, что делает статус
**STOP**, как им безопасно остановить задачу и чего от него ждать. Читать код для этого не нужно.
Технические детали механизма — в
[инженерном обзоре конвейера](../overview/tech-pipeline.md#отмена-stop--cancelled) и в
[ADR ORCH-090](../work-items/ORCH-090/06-adr/ADR-001-stop-cancel-task.md) (источник истины
поведения). Эта страница их **не дублирует**, а пересказывает «для человека» и ссылается на них.
## Что делает статус STOP?
STOP — это «кнопка отмены» задачи. Перевод задачи в статус STOP останавливает работу агента,
снимает задачу с очереди, прибирает рабочие материалы (ветку и worktree) и помечает задачу
отменённой (`cancelled`). Нажимать его безопасно даже посреди конвейера.
## Как отменить задачу?
Переведите issue в статус **STOP** на доске Plane — так же, как меняете любой другой статус.
Предусловие: на доске должен быть заведён статус **STOP** (группа `cancelled`). Если его нет, STOP
не сработает (см. раздел «Я нажал STOP, но ничего не произошло — почему?»).
## Что происходит, когда я нажимаю STOP?
По шагам:
1. Активный агент **останавливается** (мягкая остановка процесса).
2. Все **job'ы** этой задачи в очереди снимаются и больше не перезапускаются.
3. Рабочая **ветка** задачи и её **worktree** удаляются. Ветка `main` и прод-контейнер при этом
никогда не трогаются.
4. Задача переходит в терминальное состояние **`cancelled`**.
5. Приходит уведомление в **Telegram** («🛑 … задача ОТМЕНЕНА (STOP)») и **комментарий в Plane**.
При этом **документы задачи** (бизнес-запрос, анализ, ТЗ, ADR и т.д.) **сохраняются** — удаляются
только рабочая ветка и worktree, не история документов.
## Что если задача в этот момент сливается или деплоится?
Если STOP пришёл во время **необратимого шага** (слияние в `main` или выкладка в прод), отмена
**аккуратно откладывается** до честного завершения этого шага. Вы увидите уведомление вида
«⏸️ … отмена ОТЛОЖЕНА». Ветка `main` и прод при этом не трогаются; как только шаг честно
завершится, отмена применяется автоматически.
Иными словами: STOP **не прерывает** уже начатый необратимый шаг на полпути — он дожидается его
честного завершения, а затем отменяет задачу.
## Откатит ли STOP уже выложенный код?
**Нет.** STOP сбрасывает **незавершённый прогресс** задачи (рабочую ветку, worktree, очередь), но
**не откатывает** код, который уже влит в `main` или выложен в прод. Откат уже выложенного —
это отдельная задача (revert), и STOP её **не делает**.
## Как перезапустить отменённую задачу?
Отменённую задачу **нельзя «продолжить с середины»**. Чтобы начать заново, переведите её в статус
**«To Analyse»** — задача будет создана **с нуля**: новая ветка от актуального `main`, новый анализ
и полный проход конвейера.
## Я нажал STOP, но ничего не произошло — почему?
Вероятные причины:
- На доске **нет статуса STOP** — переход не распознаётся (безопасный no-op). Заведите статус STOP
(группа `cancelled`) и попробуйте снова.
- Задача **уже завершена или уже отменена** — повторный STOP ничего не меняет (это нормально,
идемпотентный no-op, задача не ломается).
- Отмена **отключена для репозитория** настройкой оператора (`stop_status_enabled` /
`stop_status_repos`) — обратитесь к оператору.
## Где увидеть, что задача отменена?
- **Карточка задачи в Telegram** покажет «🛑 … ОТМЕНЕНА (STOP)».
- В **Plane** появится комментарий об отмене.
- Оператор может увидеть отмену на служебной странице состояния `GET /queue` — в блоке `stop`.
## STOP, Approved и Confirm Deploy — в чём разница?
Это разные управляющие статусы, их легко перепутать:
- **STOP** — *отменить* задачу и сбросить её незавершённый прогресс.
- **Approved** — *одобрить* артефакт анализа (двигает задачу дальше по конвейеру); деплой он **не**
запускает.
- **Confirm Deploy** — *подтвердить* выкладку в прод.
Подробнее об управляющих статусах и их семантике — в
[инженерном обзоре конвейера](../overview/tech-pipeline.md). Эта страница описывает только STOP.

View File

@@ -97,6 +97,7 @@
Передумали — переводите задачу в статус «STOP»: работа агента останавливается, ветка и
рабочие материалы прибираются, задача помечается отменённой. Если задача в этот момент в
необратимой фазе выкладки — отмена аккуратно откладывается до её честного завершения.
Подробнее — [FAQ по статусу STOP](../operations/FAQ_STOP.md).
---

View File

@@ -126,6 +126,8 @@ freeze, ни объявленные зависимости. Свежесть б
(идёт слияние/выкладка) — отмена откладывается и применяется после честного завершения шага.
STOP никогда не трогает `main` и прод-контейнер.
Пользовательская инструкция «как этим пользоваться» — [FAQ по статусу STOP](../operations/FAQ_STOP.md).
## Статусная модель Plane: индикация ≠ управление
Статусы в Plane — слой **индикации**: они показывают человеку осмысленную картину хода задачи,

View File

@@ -0,0 +1,7 @@
# Business Request: Оценка задачи: стоимость, время и сложность (адаптивный выбор моделей)
Work Item ID: ORCH-020
## Description
Оценка задачи: стоимость, время и сложность (для адаптивного выбора моделей)Цель: добавить оркестратору функцию ОЦЕНКИ задачи — прогноз стоимости и времени реализации. Шаг 2: оценка СЛОЖНОСТИ задачи для адаптивного выбора моделей агентов.📊 Шаг 1 — Оценка стоимости и времениПеред запуском (или на этапе аналитики) оркестратор прогнозирует: сколько будет стоить задача (токены × тариф = $) и сколько займёт времени.Данные уже есть: учёт токенов и cost_usd по прошлым задачам (наблюдаемость из PR #20). ET-014 ~35 мин — есть фактура для калибровки.База оценки — история похожих задач (по типу/стадиям/стеку): средние токены/время/стоимость.Где показывать: коммент в Plane / Telegram-уведомление при заведении задачи — Слава видит прогноз до старта.Post-factum: сравнение прогноз vs факт → уточнение модели оценки (петля, связь с ORCH-8 саморазвитие).🧠 Шаг 2 — Оценка сложности → адаптивный выбор моделейКлассификация сложности задачи (trivial / small / medium / complex) на этапе триажа/аналитики.На основе сложности — адаптивно выбирать модель агента: простая задача → дешёвая/быстрая модель; сложная → сильная модель. Прямая связь с ORCH-13 (мультипровайдерность).Оптимизация: не жечь дорогую модель на тривиальной правке (экономия), но не недокормить сложную задачу слабой моделью (качество).Сложность также влияет на выбор трека (связь с ORCH-19 багфикс: trivial → hotfix, complex → полный цикл).🔧 Что проработатьСигналы для оценки сложности: текст постановки, тип (фича/баг), затронутые файлы/стек, исторические аналоги.Кто оценивает: отдельный шаг-оценщик, под-функция аналитика, или эвристика на входе.Модель оценки: эвристика по истории / отдельный LLM-вызов-оценщик / гибрид.Точность vs стоимость самой оценки (оценка не должна стоить дороже экономии).Маппинг сложность → модель (конфигурируемый, связь с ORCH-13 и манифестом проекта ORCH-9).🔗 СвязкиORCH-13 (мультипровайдерность) — оценка сложности кормит адаптивный выбор модели. Тесная пара.ORCH-19 (багфикс-трек) — сложность определяет глубину пайплайна.ORCH-8 (саморазвитие) — петля прогноз vs факт уточняет модель оценки.PR #20 (наблюдаемость, токены/cost_usd) — фактура для калибровки.❓ Открытые вопросы СлавеГде показывать прогноз стоимости/времени — Telegram при заведении, Plane-коммент, оба?Оценка обязательна для каждой задачи или по запросу/для крупных?Адаптивный выбор модели — автоматический по сложности, или с твоим подтверждением для дорогих?Шкала сложности — фиксированная (trivial/small/medium/complex) или числовая (story points)?Создано 2026-06-04. Статус: Backlog, на проработку. Источник: голосовая постановка Славы + раскладка Стрим.

View File

@@ -0,0 +1,238 @@
---
work_item: ORCH-020
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 01 — BRD (бизнес-требования): ORCH-020 — Оценка задачи (прогноз стоимости/времени/story points), запускаемая статусом «Оценка»
Work Item: **ORCH-020** · Repo: **orchestrator** · Стадия: analysis
> **Revision после REJECT (Plane, 2026-06-17).** Заказчик отклонил предыдущий пакет: «**что я не
> увидел в БРД — как запускать оценку?** Я хотел бы переводить задачу в статус "Оценка", после чего
> запускался бы механизм оценки, и после завершения оценки задача бы меняла статус на backlog. На
> оценку я буду отправлять задачи **массово через Plane**. Также я могу **переоценивать задачи много
> раз**.» Этот раунд **переписывает модель триггера**: оценка теперь — **операторское действие,
> запускаемое выделенным Plane-статусом «Оценка»** (а не «автоматически для каждой задачи на
> `start_pipeline`», как в отклонённой версии). Прочие требования (что прогнозируем, куда пишем,
> леджер прогноз↔факт, leaf-инварианты) сохранены и согласованы с новым триггером. Полный пакет
> `01``04` supersedeит прежний по mtime.
## 1. Бизнес-контекст и проблема
Заказчик планирует работу по бэклогу вручную и хочет **до отправки задачи в работу** видеть прогноз:
сколько задача будет стоить (токены × тариф = $), сколько займёт времени и насколько она сложна
(размер в story points). Сейчас этих данных до старта нет: оркестратор собирает фактуру
(`input_tokens`/`output_tokens`/`cache_*`/`cost_usd`/`model`/`effort`, тайминги
`agent_runs.started_at/finished_at`, `tasks.created_at/updated_at`) **только постфактум** через
`src/usage.py` (`task_usage_summary`, `agent_cost_totals`, `record_usage`). Контур **прогноза до
старта** отсутствует.
**Корень REJECT — отсутствовал способ ЗАПУСКА оценки.** Заказчик мыслит оценку как **операторский
жест в Plane**, а не как невидимый авто-шаг: он сам решает, какие задачи бэклога оценить, **массово**
переводит их в выделенный статус, получает прогнозы и продолжает планирование. Отклонённая версия
прятала триггер в `start_pipeline` («оценка обязательна для каждой задачи автоматически») и явно
называла точку триггера «реализационной деталью» — это и есть то, что заказчик «не увидел» и
отверг.
Цитаты заказчика (Plane, 2026-06-17):
- REJECT: «как запускать оценку? Я хотел бы **переводить задачу в статус "Оценка"**, после чего
запускался бы механизм оценки, и после завершения оценки задача бы **меняла статус на backlog**. На
оценку я буду отправлять задачи **массово через Plane**. Также я могу **переоценивать задачи много
раз**.»
- Раунд Needs Input: «В Plane есть поле оценка, туда и нужно записывать оценку. По факту завершения
задачи вписать в смежное поле… для оценки есть два поля.»; «Только Шаг 1, без выбора модели»;
«Модели не выбираем и не меняем. Это вне скоупа».
Установленные факты по коду (на которые опирается решение, не изобретать):
- **Прецедент «статус-триггер уже есть в платформе.** Plane-статусы — слой B (индикация, ORCH-066) и
НЕ управляют машиной стадий; но платформа уже имеет **операторские action-статусы**, запускающие
side-механизмы: **STOP** (ORCH-090, отмена задачи) и **Confirm Deploy** (ORCH-059, прод-деплой).
Оба разбираются в `webhooks/plane.py::handle_issue_updated` через
`proj_states.get("<key>")` и оба **намеренно отсутствуют** в `plane_sync._DEFAULT_STATES`
(fail-closed: доска без статуса → `None` → ветка не активируется). Статус «Оценка» — **третий
представитель этого же семейства**.
- **Маппинг имени статуса → логический ключ** — `plane_sync._PLANE_NAME_TO_KEY` (`"STOP"→"stop"`,
`"Confirm Deploy"→"confirm_deploy"`); `get_project_states` резолвит UUID статуса per-project из
Plane API.
- **Массовость — «бесплатно».** Plane multi-select по N задачам в статус «Оценка» порождает N
отдельных `issue.updated`-вебхуков (по одному на issue); каждый обрабатывается независимо. Отдельный
«batch-UX» в оркестраторе не требуется — массовость обеспечивает сам Plane.
- **Фактура для калибровки уже накоплена** (`agent_runs`, агрегаты `task_usage_summary` /
`agent_cost_totals`, тайминги). Это сырьё для «истории похожих задач».
- **Plane-поля существуют.** На issue присутствуют поля `estimate_point` (ОЦЕНКА) и `point` (ФАКТ);
estimate-система на проекте (`project.estimate`) на момент анализа **не настроена** — инфра-
предусловие (NFR-7).
- **Выбор модели/эффорта статичен по роли** (`resolve_agent_model`/`resolve_agent_effort`,
ORCH-41/74; дефолт `claude-opus-4-8`) и в этой задаче **не трогается** (Шаг 2 вне объёма).
- **leaf-паттерн платформы** (`serial_gate`/`coverage_gate`/`labels`/`lessons`/`cancel`): never-raise,
kill-switch `*_enabled`, `*_repos` CSV (пусто → self-hosting only), read-only блок в `GET /queue`.
## 2. Объём (scope)
### В объёме (Шаг 1 — Оценка, запускаемая статусом)
- **Триггер «Оценка» (ядро правки).** Перевод issue в выделенный Plane-статус **«Оценка»** запускает
механизм оценки этой задачи. Оператор делает это **вручную и массово** (multi-select в Plane).
- **Жизненный цикл статуса:** `Backlog → (оператор) «Оценка» → [оркестратор: оценка] → (оркестратор)
Backlog`. По завершении оценки оркестратор **сам возвращает** issue в статус **`Backlog`**.
- **Пере-оценка много раз.** Повторный перевод в «Оценка» переоценивает задачу заново (идемпотентно:
перезапись `estimate_point` и строки леджера). Применимо при изменении скоупа.
- **Прогноз четырёх величин:** стоимость ($), время, токены и **сложность в story points** из
фиксированной шкалы `{1, 2, 3, 5, 8}`.
- **Шкала story points (фиксированная, ответ Q-3 = вариант A):** `1` — мелкая docs/label/config;
`2` — небольшой фикс; `3` — средняя; `5` — сложная (код + тесты); `8` — эпик / разбивать.
- **Запись прогноза в Plane-поле `estimate_point`** (это ОЦЕНКА).
- **Запись факта в Plane-поле `point`** по завершении задачи (фактическая реализованная сложность в
story points из фактических токенов/времени/стоимости по той же шкале) — для калибровки.
- **Отображение прогноза на двух поверхностях** (ответ Q-5 = оба): Plane-коммент + пункт **«Оценка»**
в общей Telegram-карточке задачи (`src/notifications.py`) — **время, токены, стоимость**.
- **Локальный леджер прогноз↔факт** (фундамент петли калибровки, связь с ORCH-8): хранение прогноза,
факта и дельты, **ключ — `work_item_id`** (issue может ещё не иметь pipeline-задачи на момент
оценки — она на бэклоге).
### Вне объёма
- **Шаг 2 — адаптивный выбор моделей агентов** (ответы Q-1/Q-2: «Только Шаг 1, без выбора модели»;
«Модели не выбираем и не меняем. Это вне скоупа»). Горячий путь `resolve_agent_model`/
`resolve_agent_effort`/`_spawn` **не модифицируется**.
> **ACTION (поручение заказчика, Plane 16:34):** «заведи отдельную задачу в Plane для адаптивного
> выбора модели и укажи зависимость на мультипровайдеров (ORCH-13)». Создание Plane-issue — действие
> уровня заказчика/PM и **вне write-объёма аналитика** (Write ограничен `docs/work-items/<id>/*`).
> Фиксирую как обязательный follow-up: новый work item «Адаптивный выбор модели агента по сложности»
> с зависимостью на **ORCH-13**; оценщик сложности из ORCH-020 — его вход. Оператору: подтвердить
> создание или создать вручную.
- **Автопереключение трека по сложности** (связка с ORCH-19) — позже; здесь сложность лишь
вычисляется и публикуется как сигнал.
- **Авто-ретроспективщик / RICE-приоритизатор** (E2/E3 ORCH-8) — вне объёма; леджер — фундамент.
- **Автоматическая оценка КАЖДОЙ задачи на `start_pipeline`** — **исключена явно** (модель
отклонённой версии). Оценка — операторский on-demand жест через статус «Оценка».
- **Изменение тарифной/биллинговой модели** — используется существующий `cost_usd` из `usage.py`.
- **Новый «batch-UX»/массовый эндпоинт как ОСНОВНОЙ путь** — не нужен (массовость даёт Plane
multi-select → N вебхуков). Программный `POST /estimate*` допустим лишь как **опциональное**
удобство/диагностика, не как основной триггер (см. TRZ §4).
## 3. Заинтересованные стороны
- **Заказчик / владелец продукта (Слава)** — инициатор оценки (переводит задачи в «Оценка»),
потребитель прогноза для планирования бэклога; принимает результат.
- **Оркестратор (self-hosting)** — носитель функции; общий прод обслуживает и `enduro-trails`.
- **Будущая петля саморазвития (ORCH-8)** — потребитель леджера прогноз↔факт для калибровки.
- **ORCH-13 (мультипровайдерность)** — будущий потребитель сигнала сложности (через follow-up Шаг 2).
## 4. Бизнес-требования (BR)
### Триггер и жизненный цикл (ядро ревизии)
- **BR-T1 — Запуск оценки статусом «Оценка».** Перевод issue в выделенный Plane-статус **«Оценка»**
запускает оценку именно этой задачи. Это **единственный обязательный** способ запуска (массовый и
ручной), реализуемый по образцу операторских action-статусов STOP (ORCH-090) / Confirm Deploy
(ORCH-059).
- **BR-T2 — Авто-возврат в Backlog.** По завершении оценки (успех или best-effort-пропуск)
оркестратор **сам** переводит issue обратно в статус **`Backlog`**. Заказчик видит задачу
вернувшейся в бэклог с заполненным `estimate_point`.
- **BR-T3 — Массовость через Plane.** Массовый перевод N задач в «Оценка» (multi-select Plane)
оценивает все N; каждый issue обрабатывается независимо (N вебхуков). Отдельный массовый UX в
оркестраторе не требуется.
- **BR-T4 — Пере-оценка много раз (идемпотентно).** Повторный перевод задачи в «Оценка»
переоценивает её заново; прогноз и строка леджера **перезаписываются** (не дублируются). Число
пере-оценок не ограничено.
- **BR-T5 — Fail-closed статус.** На доске без статуса «Оценка» (enduro / частичная конфигурация /
Plane недоступен) триггер **не активируется** (ключ резолвится в `None`) — нулевая регрессия;
это инфра-предусловие (NFR-7), а не ошибка.
- **BR-T6 — Не нарушать машину стадий и in-flight работу.** Статус «Оценка» запускает **side-
механизм**, а не переход стадии. Если у issue есть **активная** pipeline-задача (queued/running
job), триггер — **no-op + лог** (не выдёргивать выполняемую работу в Backlog, не трогать
`STAGE_TRANSITIONS`). Авто-возврат в Backlog **не** создаёт цикла: статус `Backlog` ни одной веткой
`handle_issue_updated` не обрабатывается (no-op-эхо).
### Содержание оценки (сохранено, согласовано с триггером)
- **BR-1 — Прогноз.** Для задачи оркестратор производит прогноз четырёх величин: **стоимость ($)**,
**время**, **токены** и **сложность в story points** из фиксированной шкалы `{1,2,3,5,8}`.
- **BR-2 — База оценки — история.** Прогноз строится на истории похожих **завершённых** задач (по
типу/стадиям/стеку): средние токены/время/стоимость из уже накопленной фактуры (`agent_runs`,
`task_usage_summary`, `agent_cost_totals`, тайминги). При отсутствии истории — разумный bootstrap-
дефолт (не блокирует).
- **BR-3 — Шкала story points фиксированная** с точной семантикой `1/2/3/5/8` (см. §2). Значение `8` —
«эпик: разбивать».
- **BR-4 — On-demand, не блокирующая.** Оценка производится **по запросу** (перевод в «Оценка»), а не
для каждой задачи автоматически; строго best-effort — сбой/выключение оценки **никогда** не тормозит
конвейер и не меняет маршрут.
- **BR-5 — Доступность до старта работы.** Поскольку оператор оценивает задачи на **бэклоге** (до
`To Analyse`/`start_pipeline`), прогноз доступен **до** перевода задачи в работу — он и нужен для
планирования отправки задач.
- **BR-7 — Запись прогноза в Plane.** Прогноз сложности (story points) записывается в поле issue
**`estimate_point`** (= ОЦЕНКА).
- **BR-8 — Запись факта в Plane.** По завершении задачи (переход в `done`) фактическая реализованная
сложность (story points из фактических токенов/времени/стоимости по той же шкале) записывается в
смежное поле **`point`** — для калибровки; прогноз `estimate_point` при этом **не перезаписывается**.
- **BR-9 — Отображение на двух поверхностях.** Прогноз публикуется: (a) **Plane-комментом**;
(b) пунктом **«Оценка»** в общей Telegram-карточке задачи — **время, токены, стоимость**.
- **BR-10 — Леджер прогноз↔факт (калибровка).** Прогноз и факт сохраняются локально вместе с дельтой
(ключ `work_item_id`); фундамент петли уточнения модели оценки (связь с ORCH-8). Достаточно
фиксировать обе величины и дельту (авто-уточнение модели — позже).
## 5. Нефункциональные требования (NFR)
- **NFR-1 — Оценка ≠ Quality Gate / ≠ переход стадии.** Модуль — наблюдатель/продюсер.
`STAGE_TRANSITIONS` / `QG_CHECKS` / `check_*` / machine-verdict-ключи (`verdict:`/`result:`/
`deploy_status:`/`staging_status:`/`security_status:`/`coverage_status:`) / схемы **существующих**
таблиц — **байт-в-байт не тронуты**. Статус «Оценка» не добавляет ребра в машину стадий; он
запускает side-механизм и сам возвращает issue в Backlog.
- **NFR-2 — leaf-паттерн.** never-raise (любой сбой → warning + безопасный дефолт), kill-switch
`*_enabled`, скоуп `*_repos` (CSV; **пусто → self-hosting only**), read-only блок в `GET /queue`.
- **NFR-3 — self-hosting safety.** Модуль не рестартит/не роняет прод-контейнер, не трогает `main`/
force-push, **не вмешивается в горячий путь запуска агентов** (`resolve_agent_model`/
`resolve_agent_effort`/`_spawn` не модифицируются). Выключенный флаг / неприменимый репо → нулевая
регрессия для `enduro-trails` и `orchestrator`.
- **NFR-4 — Стоимость оценки ≪ её ценности.** Сама оценка должна быть дешёвой и быстрой относительно
выгоды планирования. Выбор механизма (эвристика по истории / отдельный LLM-вызов / гибрид) и баланс
«точность vs стоимость» — **архитектурное** решение (`06-adr`); в TRZ — лишь требование-ограничение.
- **NFR-5 — Толерантность к массовости.** Массовый перевод (десятки задач разом → десятки вебхуков
почти одновременно) **не должен** перегружать прод/конвейер: оценка best-effort, изолирована от
control-path; механизм сглаживания нагрузки (дешёвая эвристика / очередь / троттлинг) — деталь
`06-adr`. Требование: bulk не роняет и не тормозит обслуживание других проектов.
- **NFR-6 — Запись в Plane через существующие примитивы.** `estimate_point`/`point`/коммент/возврат в
Backlog пишутся через `plane_sync` и подчиняются sandbox write-guard (ORCH-117): в боевом рантайме
(`uvicorn`) — штатная запись, из тест/worktree-процесса — заблокирована. **Новых секретов/токенов не
вводится.**
- **NFR-7 — Fail-safe и инфра-предусловия Plane.** (a) Статус **«Оценка»** должен существовать на
доске проекта (его отсутствие = fail-closed no-op, BR-T5). (b) estimate-система Plane со значениями
`1/2/3/5/8` (Fibonacci) для `estimate_point` должна быть настроена; при её отсутствии запись
`estimate_point`/`point` **best-effort пропускается** (+ лог) и **не роняет** конвейер. Детали и
точные группы статуса — `07-infra-requirements.md` (архитектор).
- **NFR-8 — Обратная совместимость данных.** Хранение прогноз↔факт — **аддитивная** новая таблица
(`CREATE TABLE IF NOT EXISTS`); существующие таблицы/колонки не изменяются.
## 6. Допущения и ограничения
- Оценка на бэклоге работает по **issue** (описание/тип/лейблы из Plane API + история похожих), а не
по локальной pipeline-задаче: на момент оценки `tasks`-строки может **не быть** → леджер и запись
ключуются по `work_item_id`, `task_id` — нуллабелен до старта пайплайна.
- Статус «Оценка» — транзиентный (issue в нём лишь на время оценки, затем Backlog); его Plane-группа
(`backlog`/`unstarted`) косметична — деталь онбординга/инфры (ORCH-009 расширяется на 23-й статус).
- Фактура `usage.py`/`agent_runs` достаточна для расчёта факта при завершении; «фактические story
points» выводятся из факта по той же шкале `{1,2,3,5,8}`.
- Без ORCH-13 «выбор модели» бессмыслен (один дефолт) — Шаг 2 корректно вынесен в follow-up.
- Точная Plane-семантика `estimate_point` (FK на estimate-point estimate-системы) vs `point`
(целочисленный) — деталь реализации/инфры (архитектор + NFR-7).
## 7. Критерии успеха
Заказчик **массово переводит** задачи бэклога в статус **«Оценка»**; по каждой оркестратор
производит прогноз (стоимость/время/токены/story points), пишет его в `estimate_point`, публикует в
Plane-комменте и пункте «Оценка» Telegram-карточки, сохраняет в леджер прогноз↔факт и **возвращает
issue в Backlog**; пере-оценка повтором перевода идемпотентна; по завершении задачи факт пишется в
`point`. Всё это — без единого изменения control-path/гейтов, без касания горячего пути запуска
агентов, без выдёргивания in-flight работы; на доске без статуса «Оценка» / при выключенном флаге —
нулевая регрессия. Детальные PASS/FAIL — `03-acceptance-criteria.md`.
## 8. Риски
- **Статус «Оценка» дёргает in-flight задачу** → снимается BR-T6 (no-op при активном job) + авто-
возврат только в Backlog, никогда не трогая стадии.
- **Цикл вебхуков** (возврат в Backlog → новый webhook) → снимается тем, что `Backlog` не
обрабатывается ни одной веткой `handle_issue_updated` (no-op-эхо) — анти-loop по построению.
- **Перегрузка от массового перевода** → снимается NFR-5 (best-effort, дешёвый механизм/сглаживание —
`06-adr`).
- **Запись в боевой Plane** (`estimate_point`/`point`/коммент/состояние) на общей доске → снимается
write-guard (ORCH-117) + best-effort/fail-safe (NFR-6/NFR-7).
- **Неточность прогноза на холодном старте** (мало истории) → bootstrap-дефолт + петля калибровки
(BR-10).
- **Расползание в Шаг 2** (control-path) → жёсткий out-of-scope + NFR-3.
Детальный разбор — `10-tech-risks.md` (архитектор).

View File

@@ -0,0 +1,132 @@
---
work_item: ORCH-020
stage: analysis
author_agent: analyst
status: needs-input
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 01 — Открытые вопросы (Open Questions): ORCH-020 — Оценка задачи: стоимость, время и сложность (адаптивный выбор моделей)
Work Item: **ORCH-020** · Repo: **orchestrator** · Стадия: analysis
> **Сигнальный** when-applicable артефакт (ORCH-120, adr-0053). Пишется аналитиком ТОЛЬКО при
> **блокирующей** неоднозначности, когда выпустить корректные 4 deliverables (`01-brd`/`02-trz`/
> `03-acceptance-criteria`/`04-test-plan`) нельзя без ответа заказчика. **Не** machine-verdict;
> гейтом не парсится — сигнал движку (`_handle_analysis_approved_flow`) увести задачу в **Needs
> Input**. После ответов в Plane аналитик перезапускается (resume), читает свежие комментарии и
> выпускает полный пакет.
## 1. Контекст
Бизнес-запрос (`00-business-request.md`) ставит **две связанные, но разные по риску** функции и
**сам перечисляет 4 нерешённых вопроса для Славы** («❓ Открытые вопросы Славе»). Это решения
уровня владельца продукта, а не аналитические дефолты: они определяют **объём**, **модель данных**
и — главное — затрагивается ли **прод-control-path выбора модели агента** на self-hosting инстансе,
который из ОДНОГО процесса с ОБЩЕЙ БД обслуживает и `enduro-trails`.
**Что установлено по фактическому коду (карта src/, на которую опираются вопросы):**
- **Фактура для оценки уже есть, но только post-run.** `src/usage.py` парсит токены/`cost_usd`/
модель из вывода Claude CLI ПОСЛЕ финиша агента; `agent_runs` несёт `input_tokens`/`output_tokens`/
`cache_*`/`cost_usd`/`model`/`effort`; есть агрегаты `task_usage_summary`/`agent_cost_totals` и
тайминги (`agent_runs.started_at/finished_at`, `tasks.created_at/updated_at/brd_review_*`).
**Прогноз ДО старта** как контур сейчас отсутствует — это новый продюсер, не правка гейта.
- **Выбор модели/эффорта статичен по роли.** `resolve_agent_model`/`resolve_agent_effort`
(`src/agents/launcher.py`, ORCH-41/74) резолвят модель из config/project-override и применяют её в
`_spawn` (`--model`/`--effort`, launch-стамп ORCH-109). **Хука «варьировать модель по сложности
задачи нет».** Любой адаптивный per-task override — это вмешательство в горячий путь запуска
агентов на общем проде.
- **ORCH-13 (мультипровайдерность) ещё не реализован.** Дефолт один — `claude-opus-4-8`; реально
различимы лишь **тиры эффорта** (`low/medium/high/xhigh/max`). Без ORCH-13 «выбор модели»
фактически вырождается в «выбор эффорта».
- **`tasks.track` (ORCH-019) уже существует** (`'full'|'bug'`) — связка «сложность → трек»
опирается на готовый механизм, отдельной модели данных под трек не требует.
- **`lessons` (ORCH-098)** — журнал ОТКЛОНЕНИЙ, НЕ леджер «прогноз vs факт»: готовой калибровочной
основы под петлю прогноз↔факт (связь с ORCH-8) пока нет.
- **Leaf-паттерн** (`serial_gate`/`coverage_gate`/`labels`/`lessons`): never-raise, kill-switch
`*_enabled`, `*_repos` CSV (**пусто → self-hosting only**), read-only блок в `GET /queue`. Любой
новый модуль оценки обязан ему следовать — это ограничение, а не предмет вопроса.
Без ответов ниже корректные BR/TRZ/AC/тест-план выпустить нельзя: для пунктов Q-2/Q-3 существуют
**взаимоисключающие** варианты спецификации (разные AC, разная модель данных, разный контракт), а
Q-1 определяет, какие из «Шаг 1 / Шаг 2» вообще входят в этот work item.
## 2. Блокирующие вопросы
> Q-1…Q-4 — жёсткие блокеры. Q-5 — вопрос самого Славы с безопасным дефолтом (включён, чтобы
> закрыть все его вопросы за один раунд Needs Input и не плодить второй цикл).
- **Q-1 — Объём и очерёдность: ORCH-020 = «Шаг 1 + Шаг 2» сразу, или «Шаг 1 сейчас» + «Шаг 2 после
ORCH-13»?**
- Вариант A *(рекомендуемый)*: **Шаг 1 (прогноз стоимости/времени) сейчас**; Шаг 2 (адаптивный
выбор модели) — отдельным work item ПОСЛЕ ORCH-13, т.к. без мультипровайдерности «выбор модели»
сводится к выбору эффорта, а сам по себе оценщик сложности можно поставлять как сигнал.
- Вариант B: **оба шага в одном work item** — тогда оценщик сложности обязан выдавать решение,
готовое кормить адаптивный выбор уже сейчас (на тирах эффорта, до ORCH-13).
- Вариант C: **только Шаг 2** (классификация сложности + маппинг), прогноз стоимости/времени — позже.
- Почему блокирует: определяет, выпускаю ли я deliverables на Шаг 2 вообще и какие BR/AC в них
попадут. Объём — зона аналитика, угадывать его нельзя.
- **Q-2 — Адаптивный выбор модели: автоматический по сложности, или с твоим подтверждением для
«дорогих»/нетривиальных?** *(вопрос Славы №3; самый критичный — self-hosting safety)*
- Вариант A *(рекомендуемый)*: **advisory-only** — оценщик лишь предлагает класс/модель (коммент +
карточка), фактический `resolve_agent_model` НЕ трогается; смена модели — ручной/конфиг-шаг.
Прод-control-path неизменен, риск для `enduro-trails` нулевой.
- Вариант B: **авто-override на self-hosting только** — per-task выбор модели/эффорта применяется
автоматически, под kill-switch, скоуп `*_repos` пустой = только `orchestrator`, никогда не влияет
на чужой репозиторий и на уже запущенные задачи.
- Вариант C: **авто для дешёвых тиров, подтверждение для дорогих** (порог по прогнозу $/сложности).
- Почему блокирует: A и B/C — это **разные спецификации** (advisory ⇒ AC про коммент/карточку и
отсутствие правок горячего пути; auto ⇒ AC про безопасный gated override, скоуп, инвариант «не
трогает enduro и in-flight»). Невозможно написать корректные FR/AC, не зная, ввязываемся ли мы в
прод-путь запуска агентов на общем проде.
- **Q-3 — Шкала сложности: фиксированные категории (`trivial/small/medium/complex`) или числовая
(story points)?** *(вопрос Славы №4)*
- Вариант A *(рекомендуемый)*: **фиксированные категории** (как в постановке) — простая модель
данных (`tasks.complexity TEXT`, аддитивно по паттерну `tasks.track`), прозрачный маппинг
«класс → эффорт/модель/трек».
- Вариант B: **числовая** (story points / 1N) — гибче для калибровки, но требует решить пороги
отображения и маппинг диапазон→модель.
- Вариант C: **гибрид** (число внутри + ярлык-категория для людей).
- Почему блокирует: задаёт контракт выхода оценщика, тип новой колонки и форму маппинга
«сложность → модель/трек». Это часть требований (модель данных), не реализационная деталь.
- **Q-4 — Оценка обязательна для КАЖДОЙ задачи, или по запросу / только для крупных?** *(вопрос Славы №2)*
- Вариант A *(рекомендуемый)*: **для каждой задачи автоматически** на входе/в аналитике
(`start_pipeline`), но строго never-raise/best-effort — сбой оценки никогда не тормозит конвейер.
- Вариант B: **по запросу** (эндпоинт/лейбл Plane) и/или только при превышении порога размера.
- Вариант C: **только self-hosting `orchestrator`** на первом этапе (обкатка), enduro — позже.
- Почему блокирует: определяет точку интеграции (триггер в `start_pipeline` для всех проектов
общего прода vs опциональный путь) и формулировку NFR про область раската и обратимость.
- **Q-5 — Где показывать прогноз стоимости/времени: Telegram при заведении, Plane-коммент, оба?**
*(вопрос Славы №1; мягкий — есть безопасный дефолт)*
- Вариант A *(рекомендуемый дефолт)*: **оба** — Plane-коммент (`plane_sync.add_comment`) +
live-карточка/уведомление в Telegram (`notifications.send_telegram`); это безопасный суперсет
поверх уже существующих поверхностей.
- Вариант B: только Telegram. Вариант C: только Plane-коммент.
- Почему (мягко) блокирует: влияет на объём (одна поверхность vs две) и на AC отображения; если
промолчишь — зафиксирую дефолт A.
## 3. Что разблокирует анализ
- **Ответы на Q-1…Q-4** (и подтверждение/override Q-5) комментарием в Plane к ORCH-020.
- Минимально для старта достаточно: **Q-1** (объём), **Q-2** (advisory vs auto — определяет, трогаем
ли control-path) и **Q-3** (шкала). Q-4/Q-5 имеют безопасные дефолты (A), которые я приму при
молчании.
- На **resume**: прочту свежие комментарии-ответы и (а) если все блокеры сняты — выпущу **полный**
валидный пакет из 4 файлов (он автоматически supersedeит этот файл по mtime, повторного Needs
Input не будет); (б) если часть вопросов осталась — **перепишу** этот файл, оставив только
актуальные блокеры (снова Needs Input). Устаревшие вопросы вперемешку с новыми оставлять не буду.
> Подразумеваемые инварианты, которые я зафиксирую в пакете при ЛЮБЫХ ответах (не вопросы — границы):
> новый функционал следует leaf-паттерну (never-raise, kill-switch, `*_repos` пусто → self-hosting
> only, блок в `GET /queue`); `STAGE_TRANSITIONS`/`QG_CHECKS`/`check_*`/machine-verdict-ключи/схемы
> существующих таблиц — не трогаются (оценка — наблюдатель/продюсер, **не** Quality Gate); прод-
> контейнер не рестартится; `main`/force-push не затрагиваются. Выбор механизма оценки (эвристика по
> истории / LLM-оценщик / гибрид) и «точность vs стоимость самой оценки» — **архитектурное** решение
> (06-adr), его я не предрешаю; в TRZ зафиксирую лишь требование-ограничение «стоимость оценки ≪
> ожидаемой экономии».

View File

@@ -0,0 +1,166 @@
---
work_item: ORCH-020
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 02 — ТЗ (TRZ): ORCH-020 — Оценка задачи, запускаемая Plane-статусом «Оценка»
Work Item: **ORCH-020** · Repo: **orchestrator** · Стадия: analysis
> ТЗ описывает **конкретные изменения к реализации**, выведенные из BRD и фактического кода.
> Архитектурное обоснование/решения (выбор механизма оценки эвристика vs LLM vs гибрид, точные
> сигнатуры врезок, индексы, формулы маппинга, сглаживание массовой нагрузки, Plane-группа статуса
> «Оценка») — задача архитектора (`06-adr`).
## 1. Сводка изменения
Вводится **новый операторский Plane-статус «Оценка»** — триггер механизма оценки (по образцу
action-статусов **STOP**/ORCH-090 и **Confirm Deploy**/ORCH-059). Перевод issue в «Оценка»
(в т.ч. **массово** через Plane multi-select) запускает **новый leaf-модуль оценки**
(`src/estimator.py`, never-raise), который прогнозирует **стоимость / время / токены / сложность
(story points `{1,2,3,5,8}`)** на основе истории завершённых задач (агрегаты `src/usage.py`).
Прогноз: (a) пишется в Plane-поле `estimate_point`, (b) публикуется Plane-комментом, (c) добавляется
пунктом «Оценка» (время/токены/стоимость) в общую Telegram-карточку, (d) сохраняется в **новой
аддитивной таблице** `task_estimates` (леджер прогноз↔факт, ключ `work_item_id`). По завершении
оценки оркестратор **возвращает issue в статус `Backlog`**. По завершении самой задачи (переход в
`done`) **факт** пишется в Plane-поле `point`. Пере-оценка — повтор перевода в «Оценка»
(идемпотентно).
**Инвариант (NFR-1/NFR-3):** оценка — наблюдатель/продюсер, **не** Quality Gate и **не** переход
стадии. `STAGE_TRANSITIONS` / `QG_CHECKS` / `check_*` / machine-verdict-ключи / схемы существующих
таблиц — байт-в-байт; горячий путь `resolve_agent_model`/`resolve_agent_effort`/`_spawn` — не
трогается. Статус «Оценка» не добавляет ребра в машину стадий.
## 2. Задействованные модули / пути
| Путь | Действие |
|------|----------|
| `src/plane_sync.py` | **изменить** — (1) `_PLANE_NAME_TO_KEY += {"Оценка": "estimate"}`; ключ `estimate` **НЕ добавлять** в `_DEFAULT_STATES` (fail-closed, как `stop`/`confirm_deploy`); (2) новые write-хелперы `set_issue_estimate_point(work_item, value)`, `set_issue_point(work_item, value)`, `set_issue_backlog(work_item)` (все через guard `_guard_allows_write`, ORCH-117); (3) read-хелпер текущих полей `estimate_point`/`point`. fail-safe при отсутствии estimate-конфига |
| `src/webhooks/plane.py` | **изменить** — в `handle_issue_updated` добавить **fail-closed ветку** `estimate_state = proj_states.get("estimate")``handle_estimate(data, project_id)` (распознаётся как **отдельный** жест, не алиасит stop/to_analyse/confirm_deploy/approved/rejected). Новый `handle_estimate`: резолв issue (pipeline-задачи может не быть), guard `estimator.applies(repo)`, guard «нет активного job» (BR-T6), запуск оценки, затем `set_issue_backlog` |
| `src/estimator.py` | **создать** — leaf: `estimate(work_item_id, issue|description, repo)` → прогноз `{tokens,seconds,cost_usd,story_points}`; маппинг величин → story-point bucket `{1,2,3,5,8}` (чистая функция); расчёт факта из `usage.py`; `applies(repo)`, `should_estimate(task|None)` (анти-disruption), `snapshot()`; never-raise |
| `src/db.py` | **изменить** — аддитивная таблица `task_estimates` (`CREATE TABLE IF NOT EXISTS` в `init_db()`) + хелперы `record_estimate`/`set_actual`/`get_estimate`/`estimates_snapshot`; существующие таблицы/колонки не трогать |
| `src/usage.py` | **переиспользовать** (read-only) — `task_usage_summary`/`agent_cost_totals`/тайминги для **факта**; при необходимости тонкий read-only агрегат «история похожих задач» |
| `src/notifications.py` | **изменить** — пункт «Оценка» (время · токены · стоимость) в рендере общей карточки; never-raise, пустой прогноз → пункт опускается |
| `src/main.py` | **изменить** — (опц.) `POST /estimate?work_item=<id>` / `POST /estimate/backlog` как программное удобство; **read-only блок `estimator` в `GET /queue`** |
| `src/config.py` | **изменить** — флаги (см. §7) |
| `tests/test_orch020_estimator.py` | **создать** — покрытие (см. `04-test-plan.yaml`) |
## 3. Функциональные требования
### FR-T1 — Статус «Оценка» как триггер (BR-T1, BR-T5)
`_PLANE_NAME_TO_KEY["Оценка"] = "estimate"`; ключ `estimate` **отсутствует** в `_DEFAULT_STATES`.
В `handle_issue_updated` — отдельная ветка: `estimate_state = proj_states.get("estimate")`;
`if estimate_state and new_state == estimate_state: await handle_estimate(...)`. Доска без статуса →
`estimate_state is None` → ветка инертна (fail-closed, зеркало `stop`/`confirm_deploy`). Ветка не
должна аннулировать/перехватывать STOP/`to_analyse`/`confirm_deploy`/approved/rejected (UUID
«Оценка» отличен от всех; порядок ветки выбирает архитектор, инвариант — взаимоисключение жестов).
### FR-T2 — Обработчик `handle_estimate` (BR-T1, BR-T6)
`handle_estimate(data, project_id)`: резолвит `plane_id`/`work_item_id`; `repo` определяется по
проекту. Guard-цепочка (все — no-op-with-log при невыполнении, never-raise):
1. `estimator.applies(repo)` — kill-switch + скоуп (False → no-op);
2. **анти-disruption (BR-T6):** если у issue есть pipeline-задача с **активным** job
(`has_active_job_for_task`) → no-op + лог (не выдёргивать in-flight работу). Issue без задачи
(бэклог) или с терминальной/idle-задачей → оценка допустима.
Далее: `estimator.estimate(...)` → запись прогноза (FR-T3) → **`set_issue_backlog(work_item)`**
(BR-T2). Контракт never-raise: любая ошибка логируется, вебхук-флоу не падает.
### FR-T3 — Прогноз задачи (BR-1, BR-2, BR-3)
`estimator.estimate(work_item_id, description|issue, repo)` возвращает `{forecast_tokens,
forecast_seconds, forecast_cost_usd, story_points}`, `story_points ∈ {1,2,3,5,8}`. База — история
похожих **завершённых** задач (средние токены/время/стоимость из `usage.py`-агрегатов); пустая
история → bootstrap-дефолт. Маппинг величин → bucket — чистая функция (пороги — `06-adr`).
never-raise: сбой → безопасный дефолт + warning.
### FR-T4 — Семантика story points (BR-3)
Шкала фиксированная: `1` docs/label/config · `2` небольшой фикс · `3` средняя · `5` сложная
(код+тесты) · `8` эпик/разбивать. Значения вне набора не выдаются.
### FR-T5 — Авто-возврат в Backlog + анти-loop (BR-T2, BR-T6)
После оценки `handle_estimate` зовёт `set_issue_backlog(work_item)` → issue возвращается в `Backlog`.
Это **не** создаёт цикла: `Backlog`-UUID не совпадает ни с одной триггер-веткой `handle_issue_updated`
(`stop`/`to_analyse`/`confirm_deploy`/`approved`/`rejected`/`estimate`) → входящий webhook «state →
Backlog» = no-op-эхо. Возврат best-effort: сбой записи статуса не роняет флоу (прогноз уже записан).
### FR-T6 — Массовость и пере-оценка (BR-T3, BR-T4)
Массовый перевод N задач в «Оценка» = N независимых `issue.updated`-вебхуков → N вызовов
`handle_estimate` (никакого спец-batch-кода). Пере-оценка = повторный перевод: `estimate`
идемпотентно **перезаписывает** прогноз в `task_estimates` (UPSERT по `work_item_id`) и
`estimate_point`; дублей строк нет.
### FR-T7 — Запись прогноза и факта в Plane (BR-7, BR-8, NFR-6, NFR-7)
- Прогноз story points → `set_issue_estimate_point` → поле issue `estimate_point`.
- По завершении задачи (переход в `done`, врезка в существующий done-путь): из `usage.py` считается
факт (токены/время/стоимость) → маппится в story-point bucket → `set_issue_point` → поле `point`;
`estimate_point` не перезаписывается.
- Все записи через `plane_sync` под guard ORCH-117; отсутствие estimate-конфига/поля → best-effort
пропуск + лог (не падать).
### FR-T8 — Отображение (BR-9)
- **Plane-коммент** с прогнозом (стоимость/время/токены/story points) — `plane_sync.add_comment`.
- **Telegram-карточка** — пункт **«Оценка»**: время · токены · стоимость (`notifications`).
Обе поверхности — best-effort, не блокируют конвейер.
### FR-T9 — Леджер прогноз↔факт (BR-10)
`task_estimates` хранит прогноз (на момент оценки) и факт (на момент `done`) + дельту, ключ
`work_item_id` (т.к. на момент оценки `task_id` может быть `NULL` — issue на бэклоге). Фундамент
калибровки (ORCH-8); авто-уточнение модели в объём не входит.
### FR-T10 — leaf-инварианты (NFR-2, NFR-3)
`applies(repo)` = `estimator_enabled` ∧ скоуп `estimator_repos` (пусто → self-hosting only),
проверяется локально и ПЕРВЫМ (без сети). Выключено → весь модуль инертен (нулевая регрессия:
статус «Оценка» не обрабатывается, ничего не пишется). read-only блок `estimator` в `GET /queue`
(флаг/скоуп/счётчики прогнозов/записей/возвратов-в-Backlog).
## 4. Изменения API
| Метод/путь | Назначение |
|------------|-----------|
| **Plane-статус «Оценка»** (не HTTP-эндпоинт) | **Основной триггер**: перевод issue в статус → `handle_estimate`. Массовость — multi-select Plane. |
| `POST /estimate?work_item=<id>` *(опц.)* | Программно произвести/обновить прогноз одной задачи (то же ядро, что статус-триггер) — удобство/диагностика, не основной путь |
| `POST /estimate/backlog` *(опц.)* | Программно оценить backlog-задачи проекта — удобство; основной массовый путь — статус «Оценка» |
| `GET /estimate?work_item=<id>` *(опц.)* | Прочитать текущий прогноз vs факт из `task_estimates` |
| `GET /queue` | **+ read-only блок `estimator`**; existing-поля не меняются |
Существующие эндпоинты/контракты не изменяются. Webhook-контракт `issue.updated` не меняется —
добавляется лишь распознавание ещё одного целевого статуса.
## 5. Изменения схемы БД
**Новая аддитивная таблица** `task_estimates` (`CREATE TABLE IF NOT EXISTS`, без правки существующих):
`work_item_id` (ключ/UPSERT-цель) · `task_id` (нуллабелен до старта пайплайна) · `repo` · прогноз
(`forecast_tokens`, `forecast_seconds`, `forecast_cost_usd`, `forecast_story_points`) · факт
(`actual_tokens`, `actual_seconds`, `actual_cost_usd`, `actual_story_points`) · дельта (`delta_*`
или вычисляемая) · `source` (`status`/`manual`/`api`) · `estimate_count` (число пере-оценок,
опц.) · `created_at` · `updated_at`. Точные типы/индексы/уникальность (UNIQUE по `work_item_id`
для идемпотентного UPSERT) — `06-adr`. Существующие таблицы (`tasks`/`agent_runs`/`jobs`/…) — **не
изменяются** (NFR-8).
## 6. Требования к новым/изменённым QG checks
**Нет.** Оценка — наблюдатель/продюсер, не Quality Gate; статус «Оценка» — операторский side-триггер,
не ребро `STAGE_TRANSITIONS`. `QG_CHECKS` / `check_*` / machine-verdict-ключи / `STAGE_TRANSITIONS`
**не трогаются**. Новых номерных артефактов pipeline (`NN-*.md`) и новых вердикт-парсеров нет (оценка
публикуется в Plane/Telegram/`task_estimates`, не во frontmatter-гейтах).
## 7. Совместимость / регресс
- **Флаги** (`config.py`, дефолты безопасные): `estimator_enabled` (kill-switch, env
`ORCH_ESTIMATOR_ENABLED`), `estimator_repos` (CSV, env `ORCH_ESTIMATOR_REPOS`; **пусто →
self-hosting only**). Доп. тюнинг (bootstrap-дефолты, пороги bucket, целевой возврат-статус,
сглаживание массовой нагрузки) — конфиг-ключи на усмотрение `06-adr`.
- **Откат** = `ORCH_ESTIMATOR_ENABLED=false` → модуль инертен: статус «Оценка» не обрабатывается
(`applies`=False до сети), ни записи в Plane, ни строки карточки, ни обращений к таблице; конвейер
байт-в-байт до ORCH-020. Доп. откат «на уровне доски» — не создавать статус «Оценка» (fail-closed,
BR-T5).
- **Область раската:** по умолчанию self-hosting `orchestrator`; `enduro-trails` не затронут (скоуп
`estimator_repos` пуст + на его доске статуса «Оценка» нет → fail-closed).
- **never-raise / fail-safe:** все публичные функции и врезки изолированы (`try/except` → warning +
безопасный дефолт). Сбой оценки/записи в Plane/возврата статуса/рендера карточки — не роняет
конвейер (NFR-2/6/7).
- **Анти-disruption / анти-loop:** активный job → no-op (BR-T6); возврат в Backlog — no-op-эхо
(FR-T5). Машина стадий и in-flight задачи не затрагиваются.
- **Горячий путь не тронут:** `resolve_agent_model`/`resolve_agent_effort`/`_spawn` — без изменений
(NFR-3).
- **Инфра-предусловия (NFR-7):** (a) статус **«Оценка»** на доске проекта (онбординг ORCH-009 →
23-й статус; группа — `06-adr`/`07-infra-requirements.md`); (b) estimate-система Plane
(`1/2/3/5/8`) для `estimate_point`; их отсутствие → fail-closed/best-effort пропуск, не падение.

View File

@@ -0,0 +1,221 @@
---
work_item: ORCH-020
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 03 — Критерии приёмки (Acceptance Criteria): ORCH-020 — Оценка задачи, запускаемая статусом «Оценка»
Work Item: **ORCH-020** · Repo: **orchestrator** · Стадия: analysis
Формат: каждый критерий имеет **PASS** (что должно быть истинно для приёмки) и **FAIL** (что
считается провалом). Reviewer/тестер проверяет их буквально по файлам репозитория и поведению.
---
## AC-T1 — Запуск оценки статусом «Оценка» (ядро ревизии)
**Условие:** перевод issue в Plane-статус «Оценка» запускает оценку этой задачи.
- **PASS:** `_PLANE_NAME_TO_KEY` содержит `"Оценка" → "estimate"`; `handle_issue_updated` имеет
отдельную ветку `proj_states.get("estimate")``handle_estimate(...)`; при переводе issue в
«Оценка» вызывается оценка (прогноз вычислен и записан).
- **FAIL:** триггера-статуса нет; оценка по-прежнему авто-запускается на каждой задаче в
`start_pipeline`; ветка «Оценка» аннулирует/перехватывает STOP/`to_analyse`/`confirm_deploy`/
approved/rejected.
---
## AC-T2 — Авто-возврат в Backlog
**Условие:** по завершении оценки issue возвращается в статус `Backlog`.
- **PASS:** после записи прогноза `handle_estimate` вызывает `set_issue_backlog(work_item)` и issue
оказывается в `Backlog`; возврат best-effort (сбой записи статуса не роняет флоу, прогноз уже
записан).
- **FAIL:** issue остаётся в «Оценка»/ином статусе; возврат отсутствует; сбой возврата роняет вебхук.
---
## AC-T3 — Массовость через Plane
**Условие:** массовый перевод задач в «Оценка» оценивает их все.
- **PASS:** N задач, переведённых в «Оценка» (multi-select Plane → N `issue.updated`-вебхуков),
дают N независимых вызовов `handle_estimate`; каждая получает прогноз; спец-batch-кода для этого не
требуется.
- **FAIL:** часть задач не оценивается; обработка зависит от несуществующего «batch-режима»; один
webhook гасит остальные.
---
## AC-T4 — Пере-оценка много раз (идемпотентно)
**Условие:** повторный перевод в «Оценка» переоценивает задачу без дублей.
- **PASS:** повтор обновляет прогноз в `task_estimates` (UPSERT по `work_item_id`) и `estimate_point`;
строка одна, не дублируется; число пере-оценок не ограничено.
- **FAIL:** повтор создаёт дубль строки в `task_estimates`; повтор игнорируется/падает.
---
## AC-T5 — Fail-closed статус «Оценка»
**Условие:** на доске без статуса «Оценка» триггер не активируется.
- **PASS:** `estimate` отсутствует в `_DEFAULT_STATES`; на проекте без статуса
`proj_states.get("estimate") is None` → ветка инертна (нет KeyError, нет оценки); enduro-trails не
затронут.
- **FAIL:** `estimate` добавлен в `_DEFAULT_STATES`; отсутствие статуса даёт KeyError/ошибку; чужой
репо триггерится.
---
## AC-T6 — Анти-disruption in-flight + анти-loop
**Условие:** статус «Оценка» — side-механизм, не трогает выполняемую работу и не зацикливается.
- **PASS:** issue с активным (queued/running) job → `handle_estimate` = no-op + лог (in-flight работа
не выдёргивается в Backlog, стадии не трогаются); возврат в `Backlog` — no-op-эхо (`Backlog`-UUID не
совпадает ни с одной триггер-веткой → входящий webhook ничего не запускает).
- **FAIL:** активную задачу выдёргивает в Backlog/прерывает; возврат в Backlog порождает повторный
запуск оценки (цикл); меняется `STAGE_TRANSITIONS`.
---
## AC-1 — Прогноз четырёх величин
**Условие:** `estimator.estimate(...)` возвращает прогноз стоимости, времени, токенов и сложности.
- **PASS:** структура с `forecast_cost_usd`, `forecast_seconds`, `forecast_tokens` и `story_points`,
`story_points ∈ {1,2,3,5,8}`; пустая история → bootstrap-дефолт (не исключение).
- **FAIL:** отсутствует любая из четырёх величин; `story_points` вне `{1,2,3,5,8}`; функция бросает
исключение при отсутствии истории.
---
## AC-2 — Фиксированная семантика story points
**Условие:** маппинг величин → story-point bucket соответствует шкале заказчика.
- **PASS:** значения и смысл строго `1` (docs/label/config) · `2` (небольшой фикс) · `3` (средняя) ·
`5` (сложная код+тесты) · `8` (эпик/разбивать); чистая функция маппинга покрыта unit-тестом.
- **FAIL:** иные значения/градации (`4`, `7`, свободное число) или произвольная числовая шкала.
---
## AC-3 — Запись прогноза в Plane `estimate_point`
**Условие:** прогноз story points записывается в поле issue `estimate_point`.
- **PASS:** при оценке вызывается `set_issue_estimate_point` (через `plane_sync`/guard ORCH-117); при
настроенной estimate-системе значение оказывается в `estimate_point`.
- **FAIL:** прогноз пишется в `point` (перепутаны поля), не пишется, либо запись обходит guard.
---
## AC-4 — Запись факта в Plane `point` по завершении
**Условие:** по завершении задачи (переход в `done`) факт пишется в смежное поле `point`.
- **PASS:** на `done` факт вычисляется из `usage.py` (токены/время/стоимость), маппится в story-point
bucket и пишется в `point`; `estimate_point` не перезаписывается.
- **FAIL:** факт пишется в `estimate_point`, не пишется, либо затирает прогноз.
---
## AC-5 — Пункт «Оценка» в Telegram-карточке
**Условие:** общая карточка задачи показывает прогноз.
- **PASS:** в карточке присутствует пункт **«Оценка»** с **временем, токенами и стоимостью**; пустой
прогноз → пункт опускается (never-raise); инвариант «одна карточка на задачу» не нарушен.
- **FAIL:** пункт отсутствует; его рендер роняет/ломает карточку; нарушен инвариант одной карточки.
---
## AC-6 — Plane-коммент с прогнозом
**Условие:** прогноз публикуется комментом в Plane.
- **PASS:** постится коммент со стоимостью/временем/токенами/story points (best-effort, через
`add_comment`).
- **FAIL:** коммент не постится при включённом флаге на применимом репо без причины в логе.
---
## AC-7 — Программные эндпоинты (опциональны, не основной триггер)
**Условие:** программный путь, если реализован, использует то же ядро.
- **PASS:** `POST /estimate?work_item=<id>` / `POST /estimate/backlog` (если есть) дают тот же
результат, что статус-триггер (UPSERT в `task_estimates` + `estimate_point` + коммент + карточка),
идемпотентны; их отсутствие не нарушает приёмку (основной путь — статус «Оценка»).
- **FAIL:** эндпоинт расходится с поведением статус-триггера; преподносится как ЕДИНСТВЕННЫЙ способ
запуска (триггер-статуса нет).
---
## AC-8 — On-demand + доступность до старта, best-effort
**Условие:** оценка запускается по требованию (статус), доступна до старта работы и никогда не
блокирует конвейер.
- **PASS:** оценка идёт по переводу в «Оценка» на бэклоге (до `To Analyse`/`start_pipeline`); при
сбое оценки конвейер не затрагивается (best-effort, лог-warning); НЕ авто-обязательна на каждой
задаче.
- **FAIL:** оценка — блокирующий шаг (сбой тормозит/меняет маршрут); оценка авто-навязана каждой
задаче на `start_pipeline`.
---
## AC-9 — leaf-инварианты (kill-switch / скоуп / GET /queue)
**Условие:** модуль следует leaf-паттерну.
- **PASS:** `estimator_enabled=false` → модуль полностью инертен (статус «Оценка» не обрабатывается,
нет записей в Plane/карточку/таблицу); `estimator_repos` пуст → активен только на self-hosting
`orchestrator`; есть read-only блок `estimator` в `GET /queue`; все публичные функции never-raise.
- **FAIL:** при выключенном флаге что-то пишется/меняется; enduro-trails затронут при пустом скоупе;
нет блока в `GET /queue`; функция бросает наружу.
---
## AC-10 — Control-path и гейты не тронуты (NFR-1/NFR-3)
**Условие:** оценка ничего не меняет в машине стадий и горячем пути.
- **PASS:** `git diff` не затрагивает `STAGE_TRANSITIONS`, `QG_CHECKS`, `check_*`, machine-verdict-
ключи и схемы существующих таблиц; `resolve_agent_model`/`resolve_agent_effort`/`_spawn` — без
изменений; статус «Оценка» не добавлен как ребро стадий; зелёный анти-регресс существующих тестов.
- **FAIL:** любое из перечисленного изменено; маршрут задачи зависит от результата оценки.
---
## AC-11 — Шаг 2 (выбор модели) вне объёма
**Условие:** адаптивный выбор модели не реализуется.
- **PASS:** нет кода, выбирающего/меняющего модель/эффорт по сложности; в `01-brd.md` зафиксирован
out-of-scope + follow-up на отдельный work item с зависимостью на ORCH-13.
- **FAIL:** добавлена логика per-task override модели/эффорта; follow-up не зафиксирован.
---
## AC-12 — Леджер прогноз↔факт + fail-safe записи
**Условие:** прогноз и факт сохраняются; запись в Plane fail-safe.
- **PASS:** `task_estimates` (новая аддитивная таблица, ключ `work_item_id`, `task_id` нуллабелен)
хранит прогноз, факт и дельту; при ненастроенной estimate-системе Plane запись `estimate_point`/
`point` best-effort пропускается с логом, конвейер не падает.
- **FAIL:** существующая схема БД изменена; отсутствие estimate-конфига роняет оценку/конвейер.
---
## Сводная матрица AC ↔ FR/BR
| AC | Покрывает |
|----|-----------|
| AC-T1 | BR-T1, BR-T5 / FR-T1 |
| AC-T2 | BR-T2 / FR-T5 |
| AC-T3 | BR-T3 / FR-T6 |
| AC-T4 | BR-T4 / FR-T6 |
| AC-T5 | BR-T5 / FR-T1 |
| AC-T6 | BR-T6 / FR-T2, FR-T5 |
| AC-1 | BR-1, BR-2 / FR-T3 |
| AC-2 | BR-3 / FR-T4 |
| AC-3 | BR-7 / FR-T7 |
| AC-4 | BR-8 / FR-T7 |
| AC-5 | BR-9 / FR-T8 |
| AC-6 | BR-9 / FR-T8 |
| AC-7 | §2 «Вне объёма» / FR-T6, TRZ §4 |
| AC-8 | BR-4, BR-5 / FR-T2 |
| AC-9 | NFR-2 / FR-T10 |
| AC-10 | NFR-1, NFR-3 |
| AC-11 | §2 «Вне объёма» (Q-1/Q-2) |
| AC-12 | BR-10, NFR-7, NFR-8 / FR-T7, FR-T9 |

View File

@@ -0,0 +1,149 @@
work_item: ORCH-020
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
title: "Оценка задачи, запускаемая Plane-статусом «Оценка»: триггер/возврат в Backlog/массовость/пере-оценка + прогноз {токены,время,стоимость,story points}, запись в Plane, карточка, леджер прогноз↔факт, leaf-инварианты"
framework: pytest
scope: >
Покрывается: распознавание статуса «Оценка» как триггера (handle_estimate),
fail-closed при отсутствии статуса, авто-возврат issue в Backlog + анти-loop,
анти-disruption in-flight (no-op при активном job), массовость (N вебхуков -> N оценок),
идемпотентная пере-оценка (UPSERT по work_item_id), расчёт прогноза из истории (usage-агрегаты),
маппинг величин -> story-point bucket {1,2,3,5,8} (чистая функция), never-raise/bootstrap при
пустой истории, запись прогноза в estimate_point и факта в point (через guard ORCH-117, fail-safe
при отсутствии estimate-конфига), пункт "Оценка" в Telegram-карточке, read-only блок estimator в
GET /queue, аддитивная таблица task_estimates (ключ work_item_id, task_id нуллабелен),
kill-switch + скоуп (пусто -> self-hosting only).
Вне покрытия: адаптивный выбор модели (Шаг 2, вне объёма), авто-уточнение модели оценки (ORCH-8),
автопереключение трека по сложности (ORCH-19).
notes: >
Тесты используют изолированную временную SQLite-БД (фикстура init_db во временном файле) и
замоканные plane_sync/notifications/usage/get_project_states — без сети, без боевого Plane/Telegram,
без LLM. Триггер тестируется на уровне handle_issue_updated/handle_estimate с подставленными
proj_states (UUID статуса "Оценка"). Запись в Plane проверяется на уровне вызова write-хелперов под
guard (ORCH-117 autouse-floor conftest держит opt-in OFF — сетевая запись физически невозможна из
теста). Control-path анти-регресс: STAGE_TRANSITIONS/QG_CHECKS/check_*/machine-verdict/схемы
существующих таблиц не меняются; полный регресс tests/ остаётся зелёным.
tests:
- id: TC-01
type: integration
description: "Триггер: new_state == proj_states['estimate'] -> handle_estimate вызывается; estimate-статус добавлен в _PLANE_NAME_TO_KEY как 'Оценка'->'estimate' (AC-T1)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-02
type: integration
description: "Fail-closed: 'estimate' отсутствует в _DEFAULT_STATES; на проекте без статуса proj_states.get('estimate') is None -> ветка инертна, handle_estimate не зовётся, нет KeyError (AC-T5)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-03
type: integration
description: "handle_estimate на backlog-issue (нет pipeline-задачи): прогноз вычислен, записан, затем set_issue_backlog -> issue возвращён в Backlog (AC-T1, AC-T2)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-04
type: integration
description: "Анти-disruption: issue с активным job (has_active_job_for_task=True) -> handle_estimate no-op + лог, оценка не запускается, статус не меняется (AC-T6)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-05
type: integration
description: "Анти-loop: возврат в Backlog не алиасит триггер-ветки (Backlog-UUID != estimate/stop/to_analyse/confirm_deploy/approved/rejected) -> входящий 'state->Backlog' webhook = no-op-эхо (AC-T6)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-06
type: integration
description: "Массовость: N issue.updated со state='Оценка' -> N независимых вызовов handle_estimate, каждый даёт прогноз; один webhook не гасит остальные (AC-T3)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-07
type: integration
description: "Идемпотентная пере-оценка: повторный перевод в 'Оценка' -> UPSERT по work_item_id обновляет одну строку task_estimates и estimate_point, не дублирует (AC-T4)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-08
type: unit
description: "estimate() возвращает {forecast_tokens,forecast_seconds,forecast_cost_usd,story_points}, story_points в {1,2,3,5,8} (AC-1)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-09
type: unit
description: "Маппинг величин -> story-point bucket: точная семантика 1/2/3/5/8 на граничных входах (AC-2)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-10
type: unit
description: "Пустая история -> bootstrap-дефолт, не исключение; estimate() never-raise при битых данных (AC-1, AC-9)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-11
type: unit
description: "Расчёт факта на done из usage-агрегатов (токены/время/стоимость) маппится в story-point bucket (AC-4)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-12
type: integration
description: "Прогноз пишется в estimate_point через set_issue_estimate_point; факт — в point через set_issue_point; поля не перепутаны, прогноз не затирается (AC-3, AC-4)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-13
type: integration
description: "Telegram-карточка содержит пункт 'Оценка' (время/токены/стоимость); пустой прогноз -> пункт опускается, карточка не падает (AC-5)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-14
type: integration
description: "Plane-коммент с прогнозом постится через add_comment (best-effort) (AC-6)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-15
type: unit
description: "kill-switch estimator_enabled=false -> модуль инертен (handle_estimate no-op, нет записей в Plane/карточку/таблицу); applies() локален и first (AC-9)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-16
type: unit
description: "Скоуп estimator_repos пуст -> активен только self-hosting orchestrator; enduro-trails -> no-op (AC-9)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-17
type: integration
description: "GET /queue содержит read-only блок estimator (флаг/скоуп/счётчики прогнозов/записей/возвратов); existing-поля не меняются (AC-9)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-18
type: unit
description: "Аддитивная таблица task_estimates: CREATE TABLE IF NOT EXISTS идемпотентна; record_estimate/set_actual/get_estimate хранят прогноз+факт+дельту с ключом work_item_id (task_id нуллабелен); существующие таблицы не изменены (AC-12)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-19
type: integration
description: "fail-safe записи в Plane: estimate-система не настроена -> set_issue_estimate_point/point best-effort пропуск + лог, без падения; авто-возврат в Backlog всё равно отрабатывает (AC-12, AC-T2, NFR-7)"
module: tests/test_orch020_estimator.py
expected: PASS
- id: TC-20
type: unit
description: "Анти-регресс control-path: STAGE_TRANSITIONS/QG_CHECKS/check_*/machine-verdict-ключи, resolve_agent_model/resolve_agent_effort не изменены; статус 'Оценка' не добавлен как ребро стадий (AC-10, AC-11)"
module: tests/test_orch020_estimator.py
expected: PASS

View File

@@ -0,0 +1,7 @@
# Business Request: FAQ: как использовать STOP для отмены задачи
Work Item ID: ORCH-108
## Description
_(описание отсутствует в источнике)_

View File

@@ -0,0 +1,181 @@
---
work_item: ORCH-108
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 01 — BRD (бизнес-требования): ORCH-108 — FAQ: как использовать STOP для отмены задачи
Work Item: **ORCH-108** · Repo: **orchestrator** (self-hosting) · Стадия: analysis
Тип: документация (пользовательский FAQ). Объём — **только аналитик** (docs-only, без правки `src/**`).
---
## 1. Бизнес-контекст и проблема
Механизм отмены задачи через выделенный Plane-статус **STOP** реализован (ORCH-090,
`docs/architecture/adr/adr-0026-stop-cancel-task.md`): оператор переводит задачу в статус STOP, и
оркестратор останавливает агента, снимает job'ы с очереди, прибирает ветку/worktree и переводит
задачу в терминальное состояние `cancelled`. **Но пользовательской документации «как этим
пользоваться» нет.** Упоминания STOP разрознены и адресованы разным читателям:
- `docs/overview/business.md` — «Сценарий 6: остановить задачу» (витрина, 1 абзац);
- `docs/overview/tech-pipeline.md` — «Отмена: STOP → `cancelled`» (инженерный обзор);
- ADR ORCH-090 — глубокое архитектурное обоснование (не для конечного пользователя).
Пользователь доски Plane (тот, кто заводит/ведёт задачи) не имеет **единой пошаговой инструкции**:
что именно делает STOP, что происходит с веткой/статусом/уведомлениями, что будет, если нажать STOP
во время деплоя, откатывается ли уже влитый в `main` код, и как перезапустить отменённую задачу.
Из-за этого вероятны ошибочные ожидания (например: «STOP откатит мой код из прода» — **неверно**) и
лишние обращения к оператору.
**Боль/риск, который закрываем:** отсутствие самодостаточного FAQ → неверная ментальная модель STOP
→ ошибочные действия на проде (self-hosting: один инстанс обслуживает все проекты) и нагрузка на
оператора.
**Установленные факты (сверено по коду, не изобретать):**
- Маршрут STOP: `src/webhooks/plane.py::handle_issue_updated` распознаёт логический ключ `stop`
(fail-closed: на доске без статуса STOP ветка не активируется) → `handle_stop`
`src/stage_engine.py::cancel_task`.
- `cancel_task` (сверено, `src/stage_engine.py:2443`):
1. **Идемпотентно** — отсутствующая или уже терминальная (`done`/`cancelled`) задача → no-op.
2. **Критичное окно** (`src/cancel.py::in_critical_window`) — задача в необратимой фазе
merge/deploy → **отложенная отмена** (`cancel_requested_at`, снимаются только `queued`-job'ы,
алерт «⏸️ … отложена»; finalizer применяет отмену после честного завершения шага). STOP
**никогда** не трогает `main`, не делает force-push, не рестартит прод-контейнер.
3. **Полный сброс** (вне критичного окна) — SIGTERM агента (graceful-каскад), все job'ы →
терминальный `cancelled`, очистка deploy-state + освобождение merge-lease, снятие worktree,
удаление **рабочей** Gitea-ветки (**не** `main`, без force-push), тумбстон натуральных ключей +
`stage='cancelled'`. **Docs-артефакты сохраняются.**
4. **Наблюдаемость** — Telegram «🛑 … задача ОТМЕНЕНА (STOP)» + Plane-комментарий + обновление
карточки.
- **Перезапуск с нуля** — только «To Analyse» (тумбстон ключей → `get_task_by_plane_id` вернёт
`None` → создаётся свежая задача от актуального `origin/main`). Релонч середины пайплайна закрыт:
«To Analyse» на существующей не-`analysis` задаче → no-op + подсказка «STOP → To Analyse».
- **Простой на `deploy` в ожидании `Confirm Deploy`** (lease держится, но актор не бежит) — **не**
критичен → немедленный полный сброс (ORCH-090 review P1).
- Конфиг: `stop_status_enabled` (kill-switch), `stop_status_repos` (CSV; пусто → все репо). При
выключенном флаге / отсутствии статуса STOP — fail-safe no-op.
- Наблюдаемость для оператора: read-only блок `stop` в `GET /queue` (`src/cancel.py::snapshot`):
`enabled`/`repos`/счётчик `cancelled`/`deferred_pending`/последние отмены.
> **Уточнение к формулировке бизнес-запроса.** В описании сказано: «орк запускает cancel_request,
> откат, затем cancelled». Здесь «откат» = **сброс прогресса задачи** (снятие job'ов, удаление
> рабочей ветки/worktree, возврат задачи в терминал `cancelled`), а **НЕ** git-revert уже влитого в
> `main` кода. `cancel_request` — это путь **отложенной** отмены в критичном окне
> (`cancel_requested_at`), он срабатывает **не всегда**, а только если STOP пришёл во время
> необратимого шага. FAQ обязан развести эти понятия явно (см. BR-4, BR-5).
---
## 2. Объём (scope)
### В объёме
- Создать **пользовательский FAQ** о статусе STOP — единый, самодостаточный, пошаговый документ для
пользователя доски Plane.
- FAQ покрывает: назначение STOP; как отменить задачу; что происходит пошагово (агент, job'ы,
ветка/worktree, статус, уведомления); поведение в критичном окне merge/deploy (отложенная отмена);
явный ответ «STOP не откатывает влитый в `main` код»; как перезапустить отменённую задачу
(«To Analyse»); идемпотентность повторного STOP; что делать, если STOP «не сработал»
(инфра-предусловие — статус STOP на доске, kill-switch); где увидеть результат (Telegram /
Plane-комментарий / `GET /queue`).
- Перекрёстные ссылки между новым FAQ и существующими упоминаниями STOP (витрина / инженерный
обзор), без дублирования источника истины.
### Вне объёма
- Любые изменения кода `src/**`, поведения STOP, `STAGE_TRANSITIONS`/`QG_CHECKS`/`check_*`/схемы БД.
Это **docs-only** задача; функциональность STOP уже реализована (ORCH-090) и не меняется.
- Изменение архитектуры/механики отмены, добавление новых статусов/эндпоинтов.
- Перевод FAQ на другие языки, видео/скриншоты-гайды.
- Документирование смежных гейтов (Confirm Deploy / Approved) сверх ссылки-разграничения «STOP ≠
Confirm Deploy».
---
## 3. Заинтересованные стороны
- **Заказчик:** владелец продукта (нужен понятный пользовательский FAQ по STOP).
- **Затрагивает:** пользователей доски Plane (заводят/ведут/отменяют задачи), оператора
(меньше обращений), будущих внешних операторов Lite/Bundled-тиража.
- **Принимает результат:** reviewer (стадия `review`) — проверяет наличие, полноту и фактическую
корректность FAQ против кода.
---
## 4. Бизнес-требования (BR)
- **BR-1 — Единый пользовательский FAQ.** Существует один самодостаточный документ-FAQ о статусе
STOP, написанный для пользователя доски Plane (не для разработчика), в формате «вопрос → ответ».
- **BR-2 — Пошаговая инструкция отмены.** FAQ объясняет, как отменить задачу (перевести issue в
статус STOP на доске) и что для этого нужно (статус STOP должен существовать на доске).
- **BR-3 — Что происходит при STOP.** FAQ описывает наблюдаемые пользователем последствия: агент
останавливается, job'ы снимаются, рабочая ветка/worktree удаляются, задача переходит в
`cancelled`, приходит уведомление в Telegram и комментарий в Plane; **docs-артефакты задачи
сохраняются**.
- **BR-4 — Отложенная отмена в критичном окне.** FAQ объясняет: если STOP нажат во время
необратимого шага (слияние/выкладка), отмена **откладывается** до честного завершения шага;
`main`/прод при этом не трогаются.
- **BR-5 — STOP ≠ откат прод-кода.** FAQ содержит **явный** ответ: STOP сбрасывает незавершённый
прогресс задачи, но **не откатывает** код, уже влитый в `main`/прод (revert — отдельная задача).
- **BR-6 — Перезапуск отменённой задачи.** FAQ объясняет: отменённую задачу нельзя «продолжить с
середины»; перезапуск — только «To Analyse», который создаёт задачу **с нуля** (новая ветка от
актуального `main`).
- **BR-7 — Идемпотентность и «не сработало».** FAQ объясняет: повторный STOP по уже отменённой/
завершённой задаче безопасен (no-op); если STOP «ничего не сделал» — вероятные причины (статус
STOP не заведён на доске / задача уже терминальна / отмена отключена для репо).
- **BR-8 — Где увидеть результат.** FAQ указывает источники подтверждения отмены: карточка
Telegram, комментарий в Plane, read-only блок `stop` в `GET /queue`.
- **BR-9 — Согласованность с витриной.** FAQ не противоречит существующим упоминаниям STOP в
`docs/overview/business.md` и `docs/overview/tech-pipeline.md`; ссылки связывают их без
дублирования источника истины.
---
## 5. Нефункциональные требования (NFR)
- **NFR-1 — Docs-only, нулевой рантайм-риск.** Никаких изменений `src/**`, конвейера, гейтов, схемы
БД. Self-hosting-безопасно: задача не деплоит/не рестартит прод/не трогает `main`.
- **NFR-2 — Фактическая точность.** Каждое утверждение FAQ verifiable против кода (`src/cancel.py`,
`src/stage_engine.py::cancel_task`, `src/webhooks/plane.py`, `src/config.py`). Запрещены неверные
обещания (например «STOP откатит прод»).
- **NFR-3 — Язык и аудитория.** Русский, тон — пользовательский (без требования читать код/ADR);
термины пайплайна поясняются простыми словами.
- **NFR-4 — Сопровождаемость / анти-дрейф.** Структуру FAQ закрывает детерминированный структурный
тест (без сети/LLM/subprocess), по образцу `tests/test_lite_setup_doc.py`, чтобы будущие правки не
«отклеивали» FAQ от фактов.
- **NFR-5 — Без форка источника истины.** FAQ ссылается на канон (ADR ORCH-090, инженерный обзор), а
не копирует его дословно; машинные детали — ссылками.
---
## 6. Допущения и ограничения
- **Допущение A1 (размещение).** FAQ размещается как новый документ `docs/operations/FAQ_STOP.md`
(раздел эксплуатации/операторских runbook'ов — там же `ONBOARDING.md`, `PHANTOM_MERGE_RUNBOOK.md`).
Это **разумный дефолт** исходя из аудитории «оператор/пользователь доски»; точное имя/раздел
reviewer/architect может скорректировать, но это не блокирует анализ (не сигнальный вопрос).
- **Допущение A2 (язык).** Русский — основной язык пользовательской документации проекта
(соответствует `docs/overview/*`).
- **Ограничение C1.** Поведение STOP фиксировано ORCH-090; FAQ его **документирует**, а не меняет.
Если по ходу обнаружится расхождение «доки vs код» — это дефект, заводится отдельно (правило
агентов №4: не комментировать ТЗ задним числом, возвращать в анализ).
- **Ограничение C2.** Никаких блокирующих неоднозначностей не выявлено → файл `01-questions.md`
**не создаётся** (ORCH-120): сделанных допущений (A1/A2) достаточно для корректного пакета.
---
## 7. Критерии успеха
Документ-FAQ создан, покрывает все темы BR-1…BR-9, фактически согласован с кодом, перекрёстно связан
с витриной, и закрыт структурным анти-дрейф тестом. Полный регресс `tests/` остаётся зелёным.
Детальные PASS/FAIL — в `03-acceptance-criteria.md`.
## 8. Риски
- **R1 — Дрейф «доки ↔ код».** Будущая правка STOP сделает FAQ неверным. Митигейшн — структурный
тест (NFR-4) + правило «правишь STOP → обнови FAQ в том же PR».
- **R2 — Ошибочное размещение/дубль.** FAQ продублирует витрину вместо ссылки. Митигейшн — BR-9 +
AC на перекрёстные ссылки.
- Детали/полный перечень — `10-tech-risks.md` (заполняет архитектор; для docs-only задачи риски
минимальны).

View File

@@ -0,0 +1,189 @@
---
work_item: ORCH-108
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 02 — ТЗ (TRZ): ORCH-108 — FAQ: как использовать STOP для отмены задачи
Work Item: **ORCH-108** · Repo: **orchestrator** (self-hosting) · Стадия: analysis
> ТЗ описывает **конкретные изменения к реализации**, выведенные из BRD и фактического кода.
> Это **docs-only** задача: код `src/**` и поведение STOP не меняются. Источник истины поведения —
> ORCH-090 (`adr-0026`); здесь — требования к **документации** этого поведения.
> Архитектурное обоснование (если потребуется) — задача архитектора (`06-adr`).
## 1. Сводка изменения
Создать пользовательский **FAQ по статусу STOP** — новый Markdown-документ
`docs/operations/FAQ_STOP.md` в формате «вопрос → ответ», для пользователя доски Plane. Добавить
перекрёстные ссылки из существующих упоминаний STOP (витрина / инженерный обзор) на FAQ. Закрыть
структуру FAQ детерминированным анти-дрейф тестом. **Никаких изменений `src/**`, конвейера, гейтов,
схемы БД, API.** Полный черновик содержания FAQ — в Приложении A (готов к переносу разработчиком;
объём «только аналитик» → существенное наполнение сделано на стадии анализа).
## 2. Задействованные модули / пути
| Путь | Действие |
|------|----------|
| `docs/operations/FAQ_STOP.md` | **создать** — пользовательский FAQ по STOP (основной deliverable; содержание — Приложение A) |
| `docs/overview/business.md` | изменить — добавить ссылку «Подробнее: FAQ по STOP» в «Сценарий 6: остановить задачу» |
| `docs/overview/tech-pipeline.md` | изменить — добавить ссылку на FAQ в раздел «Отмена: STOP → `cancelled`» |
| `CHANGELOG.md` | изменить — запись `docs: ORCH-108 FAQ по статусу STOP` |
| `tests/test_faq_stop_doc.py` | **создать** — структурный анти-дрейф тест FAQ (образец `tests/test_lite_setup_doc.py`) |
**Описываемые (read-only) модули — FAQ их излагает, НЕ меняет** (для верификации фактов reviewer'ом):
- `src/webhooks/plane.py` — `handle_issue_updated` (распознавание ключа `stop`, fail-closed),
`handle_stop` (делегирование в `cancel_task`), `handle_status_start` (гейт релонча: «To Analyse»
перезапускает только с нуля, не середину пайплайна).
- `src/stage_engine.py::cancel_task` — оркестрация отмены (идемпотентность / критичное окно /
полный сброс / наблюдаемость).
- `src/cancel.py` — `applies` (kill-switch + repo-scope), `in_critical_window` (классификация
необратимого окна), `snapshot` (блок `stop` в `GET /queue`).
- `src/config.py` — `stop_status_enabled` (env `ORCH_STOP_STATUS_ENABLED`), `stop_status_repos`
(env `ORCH_STOP_STATUS_REPOS`, CSV; пусто → все репо).
- `src/main.py` — read-only блок `stop` в `GET /queue`.
## 3. Функциональные требования
### FR-1 — Документ FAQ существует и адресован пользователю (BR-1)
Создать `docs/operations/FAQ_STOP.md`: H1-заголовок про STOP, вводный абзац «для кого/зачем», далее
секции «вопрос → ответ». Тон — пользовательский (без требования читать код). Язык — русский.
### FR-2 — Обязательные секции FAQ (BR-2…BR-8)
FAQ содержит как минимум следующие тематические секции (заголовки — стабильные якоря для теста
NFR-4 / TC-02), каждая отвечает на свой вопрос:
1. **«Что делает статус STOP?»** — назначение: отмена + сброс прогресса задачи.
2. **«Как отменить задачу?»** — перевести issue в статус **STOP** на доске Plane; предусловие —
статус STOP заведён на доске.
3. **«Что происходит, когда я нажимаю STOP?»** — пошагово: агент останавливается → job'ы снимаются
→ рабочая ветка и worktree удаляются → задача переходит в `cancelled` → приходит уведомление в
Telegram и комментарий в Plane. **Docs-артефакты задачи сохраняются.**
4. **«Что если задача в этот момент сливается или деплоится?»** — отложенная отмена: отмена
откладывается до честного завершения необратимого шага; `main`/прод не трогаются.
5. **«Откатит ли STOP уже выложенный код?»** — **Нет.** STOP сбрасывает незавершённый прогресс
задачи, но не делает git-revert уже влитого в `main`/прод кода (это отдельная задача).
6. **«Как перезапустить отменённую задачу?»** — только через «To Analyse»; задача создаётся **с
нуля** (новая ветка от актуального `main`); «продолжить с середины» нельзя.
7. **«Я нажал STOP, но ничего не произошло — почему?»** — вероятные причины: статус STOP не заведён
на доске (fail-closed no-op); задача уже завершена/отменена (идемпотентный no-op); отмена
отключена для репозитория (`stop_status_enabled`/`stop_status_repos`).
8. **«Где увидеть, что задача отменена?»** — карточка Telegram («🛑 … ОТМЕНЕНА (STOP)»), комментарий
в Plane, read-only блок `stop` в `GET /queue`.
### FR-3 — Разграничение STOP ↔ другие управляющие статусы (BR-9)
FAQ кратко разграничивает STOP и человеческие гейты `Approved`/`Confirm Deploy` (STOP — отмена, не
одобрение/деплой), ссылкой на инженерный обзор, без переписывания их семантики.
### FR-4 — Перекрёстные ссылки без дублирования (BR-9, NFR-5)
- В `docs/overview/business.md` («Сценарий 6») и `docs/overview/tech-pipeline.md` («Отмена: STOP →
`cancelled`») добавить ссылку на `FAQ_STOP.md`.
- В FAQ — обратные ссылки на инженерный обзор и ADR ORCH-090 как на источник истины поведения.
- **Не дублировать** машинные детали (маркеры/lease/тумбстон) — давать ссылками.
### FR-5 — Фактическая корректность (NFR-2)
Каждое утверждение FAQ соответствует коду на момент написания (см. §2 read-only модули). Запрещены
утверждения, противоречащие коду — в частности: «STOP откатывает прод», «STOP трогает `main`/делает
force-push», «отменённую задачу можно продолжить с середины», «STOP мгновенно убивает идущий
деплой».
### FR-6 — Анти-дрейф тест (NFR-4)
Создать `tests/test_faq_stop_doc.py` (детерминированный, без сети/LLM/subprocess; только парсинг
файла): FAQ существует; присутствуют все обязательные секции-якоря (FR-2); присутствуют ключевые
факты-«кирпичи» (STOP, `cancelled`, «To Analyse», «main … не», отложенная/deferred); присутствуют
перекрёстные ссылки (FR-4); отсутствуют запрещённые неверные утверждения (FR-5, негативный скан).
## 4. Изменения API
Нет. (FAQ лишь упоминает существующий read-only `GET /queue` блок `stop`.)
## 5. Изменения схемы БД
Нет.
## 6. Требования к новым/изменённым QG checks
Нет. `STAGE_TRANSITIONS` / `QG_CHECKS` / `check_*` / machine-verdict — не затрагиваются.
Замечание по coverage-гейту (ORCH-027): docs-only изменение не добавляет строк `src/` → базовая
линия покрытия не меняется; новый `tests/test_faq_stop_doc.py` не покрывает `src/` (структурный
тест документа) и на метрику не влияет.
## 7. Совместимость / регресс
- **Обратная совместимость — полная.** Только добавление/правка docs + новый структурный тест.
Поведение рантайма байт-в-байт прежнее; kill-switch не требуется (нет исполняемого кода).
- **Self-hosting-безопасно.** Не деплоит/не рестартит прод/не трогает `main`; реальный прод-деплой
этой задачи безопасен (docs).
- **Регресс.** Полный `tests/` остаётся зелёным; новый тест читает только файл FAQ.
- **Сопровождение (норматив).** Правишь поведение STOP (`src/cancel.py`/`cancel_task`/маршрут
`stop`) → обнови `docs/operations/FAQ_STOP.md` в том же PR (правило агентов №2 / №6: reviewer
требует обновлённую доку).
---
## Приложение A — Черновик содержания FAQ (готов к переносу в `docs/operations/FAQ_STOP.md`)
> Нормативный ориентир содержания (объём «только аналитик»). Разработчик переносит как тело
> документа; точные формулировки можно полировать, фактическую часть менять нельзя без возврата в
> анализ (правило №4).
```markdown
# FAQ: отмена задачи через статус STOP
Эта страница — для пользователя доски Plane. Она объясняет, что делает статус **STOP**, как им
безопасно остановить задачу и чего от него ждать. Технические детали механизма — в
[инженерном обзоре](../overview/tech-pipeline.md#отмена-stop--cancelled) и
[ADR ORCH-090](../work-items/ORCH-090/06-adr/ADR-001-stop-cancel-task.md).
## Что делает статус STOP?
STOP — это «кнопка отмены» задачи. Перевод задачи в статус STOP останавливает работу агента, снимает
задачу с очереди, прибирает рабочие материалы (ветку и worktree) и помечает задачу отменённой
(`cancelled`). Безопасно нажимать даже посреди конвейера.
## Как отменить задачу?
Переведите issue в статус **STOP** на доске Plane — так же, как меняете любой другой статус.
Предусловие: на доске должен быть заведён статус **STOP** (группа `cancelled`). Если его нет, STOP
не сработает (см. «ничего не произошло»).
## Что происходит, когда я нажимаю STOP?
По шагам:
1. Активный агент останавливается (мягкая остановка процесса).
2. Все задачи в очереди по этой задаче снимаются и больше не перезапускаются.
3. Рабочая ветка задачи и её worktree удаляются. **Ветка `main` и прод никогда не трогаются.**
4. Задача переходит в терминальное состояние `cancelled`.
5. Приходит уведомление в Telegram («🛑 … задача ОТМЕНЕНА (STOP)») и комментарий в Plane.
**Документы задачи (анализ, ТЗ и т.д.) сохраняются** — удаляются только рабочая ветка и worktree.
## Что если задача в этот момент сливается или деплоится?
Если STOP пришёл во время необратимого шага (слияние в `main` или выкладка), отмена **аккуратно
откладывается** до честного завершения этого шага. Вы увидите уведомление «⏸️ … отмена отложена».
`main` и прод при этом не трогаются; после завершения шага отмена применяется автоматически.
## Откатит ли STOP уже выложенный код?
**Нет.** STOP сбрасывает **незавершённый прогресс** задачи (ветку/worktree/очередь), но **не
откатывает** код, который уже влит в `main` или выложен в прод. Откат выложенного — это отдельная
задача (revert), STOP её не делает.
## Как перезапустить отменённую задачу?
Отменённую задачу нельзя «продолжить с середины». Чтобы начать заново, переведите её в статус
**«To Analyse»** — задача будет создана **с нуля** (новая ветка от актуального `main`, новый анализ).
## Я нажал STOP, но ничего не произошло — почему?
Вероятные причины:
- На доске **нет статуса STOP** — переход не распознаётся (безопасный no-op). Заведите статус STOP
(группа `cancelled`).
- Задача **уже завершена или уже отменена** — повторный STOP ничего не меняет (это нормально).
- Отмена **отключена для репозитория** настройкой (`stop_status_enabled` / `stop_status_repos`) —
обратитесь к оператору.
## Где увидеть, что задача отменена?
- Карточка задачи в **Telegram** покажет «🛑 … ОТМЕНЕНА (STOP)».
- В **Plane** появится комментарий об отмене.
- Оператор может увидеть отмену в служебной странице состояния `GET /queue` (блок `stop`).
## STOP, Approved и Confirm Deploy — в чём разница?
- **STOP** — отменить задачу.
- **Approved** — одобрить артефакт анализа (двигает задачу дальше), деплой не запускает.
- **Confirm Deploy** — подтвердить прод-выкладку.
Подробнее об управляющих статусах — в [инженерном обзоре](../overview/tech-pipeline.md).
```

View File

@@ -0,0 +1,141 @@
---
work_item: ORCH-108
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 03 — Критерии приёмки (Acceptance Criteria): ORCH-108 — FAQ: как использовать STOP
Work Item: **ORCH-108** · Repo: **orchestrator** · Стадия: analysis
Формат: каждый критерий имеет **PASS** (что должно быть истинно для приёмки) и **FAIL** (что
считается провалом). Reviewer проверяет их буквально по файлам репозитория.
---
## AC-1 — FAQ-документ существует и адресован пользователю
**Условие:** создан `docs/operations/FAQ_STOP.md` в формате «вопрос → ответ» для пользователя Plane.
- **PASS:** файл существует; есть H1 про STOP и вводный абзац «для кого/зачем»; тон
пользовательский (не требует чтения кода); язык русский.
- **FAIL:** файла нет; либо это разработческий/архитектурный текст, а не пользовательский FAQ; либо
нет формата «вопрос → ответ».
---
## AC-2 — Покрыты все обязательные темы
**Условие:** FAQ содержит секции, отвечающие на 8 обязательных вопросов TRZ §FR-2.
- **PASS:** присутствуют все темы — (1) что делает STOP; (2) как отменить; (3) что происходит
пошагово; (4) отложенная отмена в критичном окне; (5) STOP не откатывает прод-код; (6) перезапуск
через «To Analyse» с нуля; (7) «ничего не произошло — почему»; (8) где увидеть результат.
- **FAIL:** отсутствует хотя бы одна из тем (1)(8).
---
## AC-3 — Пошаговые последствия STOP описаны верно
**Условие:** тема (3) перечисляет наблюдаемые последствия согласно `cancel_task`.
- **PASS:** перечислены — остановка агента; снятие job'ов; удаление рабочей ветки и worktree; явное
«`main`/прод не трогаются»; переход в `cancelled`; уведомление Telegram + комментарий Plane; явное
«docs-артефакты сохраняются».
- **FAIL:** пропущен или искажён любой из этих пунктов (например утверждается, что удаляются docs,
или что трогается `main`).
---
## AC-4 — Отложенная отмена в критичном окне
**Условие:** тема (4) корректно описывает поведение при STOP во время merge/deploy.
- **PASS:** сказано, что отмена **откладывается** до честного завершения необратимого шага; что
`main`/прод не трогаются; что после завершения шага отмена применяется.
- **FAIL:** утверждается мгновенное прерывание деплоя/слияния, либо что STOP убивает идущий
необратимый шаг, либо тема отсутствует.
---
## AC-5 — STOP ≠ откат прод-кода (явный ответ)
**Условие:** тема (5) явно разводит «сброс прогресса» и «revert выложенного».
- **PASS:** есть явное «Нет»: STOP **не откатывает** код, уже влитый в `main`/прод; revert — отдельная
задача.
- **FAIL:** FAQ обещает/намекает, что STOP откатит прод-код, либо тема отсутствует.
---
## AC-6 — Перезапуск отменённой задачи
**Условие:** тема (6) описывает перезапуск.
- **PASS:** сказано, что перезапуск — только «To Analyse»; задача создаётся **с нуля** (новая ветка
от актуального `main`); «продолжить с середины» нельзя.
- **FAIL:** утверждается возможность продолжить отменённую задачу с середины, либо неверный
механизм перезапуска, либо тема отсутствует.
---
## AC-7 — «Не сработало» и идемпотентность
**Условие:** тема (7) перечисляет причины no-op.
- **PASS:** перечислены — статус STOP не заведён на доске (fail-closed); задача уже терминальна
(идемпотентный no-op); отмена отключена для репо (`stop_status_enabled`/`stop_status_repos`).
- **FAIL:** причины не описаны или описаны неверно (например, утверждается, что повторный STOP
ломает задачу).
---
## AC-8 — Перекрёстные ссылки без дублирования
**Условие:** FAQ связан с витриной/обзором двусторонними ссылками (TRZ §FR-4).
- **PASS:** `docs/overview/business.md` («Сценарий 6») и `docs/overview/tech-pipeline.md` («Отмена:
STOP → `cancelled`») содержат ссылку на `FAQ_STOP.md`; FAQ ссылается на инженерный обзор и ADR
ORCH-090 как на источник истины; машинные детали не дублируются, а даются ссылками.
- **FAIL:** ссылок нет (FAQ-«сирота»); либо FAQ дословно копирует ADR/обзор вместо ссылки.
---
## AC-9 — Фактическая корректность (нет ложных утверждений)
**Условие:** утверждения FAQ соответствуют коду (`src/cancel.py`, `src/stage_engine.py::cancel_task`,
`src/webhooks/plane.py`, `src/config.py`); запрещённых неверных утверждений нет.
- **PASS:** в FAQ отсутствуют утверждения «STOP трогает `main`/делает force-push», «STOP откатывает
прод», «отменённую задачу можно продолжить с середины», «STOP мгновенно убивает идущий деплой».
- **FAIL:** присутствует хотя бы одно противоречащее коду утверждение.
---
## AC-10 — Docs-only, нулевой рантайм-регресс
**Условие:** изменения ограничены документацией + структурным тестом.
- **PASS:** `git diff` не затрагивает `src/**`, `STAGE_TRANSITIONS`/`QG_CHECKS`/`check_*`/схему БД;
изменены только `docs/**`, `CHANGELOG.md`, `tests/test_faq_stop_doc.py`; полный `tests/` зелёный.
- **FAIL:** затронут `src/**` или поведение гейтов/конвейера; либо регресс `tests/` красный.
---
## AC-11 — Анти-дрейф тест присутствует и зелёный
**Условие:** структурную целостность FAQ закрывает детерминированный тест (TRZ §FR-6).
- **PASS:** `tests/test_faq_stop_doc.py` существует; проверяет наличие файла, обязательных
секций-якорей, ключевых фактов-«кирпичей», перекрёстных ссылок и отсутствие запрещённых
утверждений; не делает сети/LLM/subprocess; проходит зелёным.
- **FAIL:** теста нет; либо он не детерминирован (сеть/LLM/subprocess); либо красный.
---
## Сводная матрица AC ↔ FR/BR
| AC | Покрывает |
|----|-----------|
| AC-1 | BR-1 / FR-1 |
| AC-2 | BR-2…BR-8 / FR-2 |
| AC-3 | BR-3 / FR-2(3) |
| AC-4 | BR-4 / FR-2(4) |
| AC-5 | BR-5 / FR-2(5) |
| AC-6 | BR-6 / FR-2(6) |
| AC-7 | BR-7 / FR-2(7) |
| AC-8 | BR-9 / FR-3, FR-4 |
| AC-9 | NFR-2 / FR-5 |
| AC-10 | NFR-1 / FR (docs-only), §7 |
| AC-11 | NFR-4 / FR-6 |

View File

@@ -0,0 +1,74 @@
work_item: ORCH-108
stage: analysis
author_agent: analyst
status: ready-for-review
created_at: 2026-06-17
model_used: claude-opus-4-8
title: "Анти-дрейф структурного FAQ по статусу STOP (docs-only)"
framework: pytest
scope: >
Покрывается СТРУКТУРНАЯ целостность и фактическая непротиворечивость нового
пользовательского документа docs/operations/FAQ_STOP.md (детерминированно: только
парсинг файлов, без сети/LLM/subprocess; образец tests/test_lite_setup_doc.py).
Вне покрытия: поведение STOP в рантайме — оно реализовано и протестировано в
ORCH-090 (tests/ по cancel_task/cancel.py), эта задача его НЕ меняет (docs-only).
notes: >
Docs-only задача: src/** не меняется, поэтому юнит/интеграционных тестов кода нет —
только структурные тесты документа. Полный регресс tests/ должен оставаться зелёным
(новый тест читает лишь файлы docs/, на src/-покрытие/coverage-baseline не влияет).
Все тесты — type: unit (без сети/LLM/subprocess), модуль tests/test_faq_stop_doc.py.
tests:
- id: TC-01
type: unit
description: "FAQ существует: docs/operations/FAQ_STOP.md присутствует, непустой, есть H1 про STOP и вводный абзац для пользователя (AC-1)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-02
type: unit
description: "Обязательные секции-якоря присутствуют: все 8 тем FR-2 (что делает STOP / как отменить / пошагово / отложенная отмена / не откатывает прод / перезапуск To Analyse / 'ничего не произошло' / где увидеть) (AC-2)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-03
type: unit
description: "Пошаговые последствия и сохранность: упомянуты остановка агента, снятие job'ов, удаление рабочей ветки/worktree, переход в cancelled, уведомление Telegram+Plane, явное 'docs сохраняются' (AC-3)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-04
type: unit
description: "Критичное окно: присутствует факт отложенной отмены (deferred / 'отложена') и явное 'main/прод не трогаются' (AC-4)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-05
type: unit
description: "STOP ≠ откат прод-кода: присутствует явный отрицательный ответ ('не откатывает' влитый в main/прод код) (AC-5)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-06
type: unit
description: "Перезапуск: упомянуто 'To Analyse' и создание задачи 'с нуля', отсутствует обещание 'продолжить с середины' (AC-6)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-07
type: unit
description: "Негативный скан фактов: в FAQ НЕТ запрещённых утверждений — 'откатит прод', 'трогает main/force-push', 'продолжить отменённую с середины', 'мгновенно убивает деплой' (AC-9)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-08
type: unit
description: "Перекрёстные ссылки: business.md (Сценарий 6) и tech-pipeline.md (Отмена: STOP → cancelled) содержат ссылку на FAQ_STOP.md; FAQ ссылается на инженерный обзор/ADR ORCH-090 (AC-8)."
module: tests/test_faq_stop_doc.py
expected: PASS
- id: TC-09
type: unit
description: "Docs-only регресс-инвариант: полный прогон tests/ зелёный; новый тест не импортирует src/ рантайм и не делает сети/subprocess (AC-10, AC-11)."
module: tests/test_faq_stop_doc.py
expected: PASS

View File

@@ -0,0 +1,173 @@
---
work_item: ORCH-108
stage: architecture
author_agent: architect
status: proposed
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# ADR-001: Размещение пользовательского FAQ по STOP и контур анти-дрейфа
Work Item: **ORCH-108** — FAQ: как использовать статус STOP для отмены задачи
Стадия: **architecture**
Сквозная регистрация: **N/A — локальное решение задачи** (docs-only; новых QG/стадий/
компонентов/таблиц нет, маркеры `ORCH-NNN` в `src/**` не вводятся → сквозной
`docs/architecture/adr/adr-NNNN-*` не требуется; критерий — `docs/_standards/PIPELINE_DOCS.md` §4).
## Статус
Proposed
## Контекст
Механизм отмены задачи через Plane-статус **STOP** реализован в ORCH-090
(`docs/architecture/adr/adr-0026-stop-cancel-task.md`, `src/cancel.py`,
`src/stage_engine.py::cancel_task`, `src/webhooks/plane.py`). Пользовательской
инструкции «как этим пользоваться» нет — упоминания STOP разрознены и адресованы разным
читателям (витрина `docs/overview/business.md` «Сценарий 6», инженерный обзор
`docs/overview/tech-pipeline.md` «Отмена: STOP → `cancelled`», глубокий ADR ORCH-090). Это
порождает неверную ментальную модель («STOP откатит мой код из прода» — **неверно**) и нагрузку
на оператора (self-hosting: один инстанс на все проекты).
Аналитик (BRD/TRZ/AC, `ready-for-review`) полностью описал требуемый артефакт и приложил готовый
черновик содержания (TRZ Приложение A). Это **docs-only** задача: `src/**`, `STAGE_TRANSITIONS`,
`QG_CHECKS`, `check_*`, схема БД — не меняются; поведение STOP фиксировано ORCH-090 и FAQ его лишь
**документирует**. Архитектурных решений по существу два: (1) куда положить FAQ в дереве доков и
(2) как структурно защитить его от дрейфа «доки ↔ код». Остальное — исполнение на стадии
development.
Факты, сверенные на ветке задачи (read-only):
- Цели перекрёстных ссылок **существуют**: `docs/overview/business.md` §«Сценарий 6: остановить
задачу» (стр. 96), `docs/overview/tech-pipeline.md` §«Отмена: STOP → `cancelled`» (стр. 122),
`docs/work-items/ORCH-090/06-adr/ADR-001-stop-cancel-task.md`. Ссылки FR-4 не «висячие».
- Семантика разделов доков (ORCH-011, `adr-0039`): `overview/` — витрина «что за система»,
`architecture/` — инженерный справочник, `deployment/` — «как развернуть у себя»,
`operations/` — «как эксплуатировать наш прод» (runbook'и: `ONBOARDING.md`,
`PHANTOM_MERGE_RUNBOOK.md`, `STAGING.md`, …).
- `docs/overview/`**курируемый плоский каталог из 10 файлов**, чьё содержимое прибито
структурным тестом `tests/test_system_docs.py` (витрина — не свалка произвольных доков).
- Прецедент анти-дрейф теста документа — `tests/test_lite_setup_doc.py` (детерминированный,
offline; позитивные якоря-секции + «кирпичи» + кросс-ссылки + негативный скан запрещённых
литералов по `FORBIDDEN`).
## Решение
### Сводка
Размещаем FAQ как **`docs/operations/FAQ_STOP.md`** — пользовательский документ «вопрос → ответ»,
прилинкованный из витрины/обзора и закрытый детерминированным структурным тестом
`tests/test_faq_stop_doc.py`. Утверждаем разумный дефолт аналитика (A1) как архитектурное решение,
с явной фиксацией ключевого нюанса теста — **негативный скан проверяет запрещённые
утверждения, а не голые подстроки** (иначе он ложно срабатывал бы на предложениях, которые эти же
термины корректно **отрицают**).
### D1 — Размещение: `docs/operations/FAQ_STOP.md` (BR-1, A1)
FAQ ложится в `docs/operations/` рядом с операторскими runbook'ами.
Обоснование выбора между тремя кандидатами (аудитория FAQ «пользователь доски + оператор»
неоднородна, поэтому секция не очевидна):
- **`docs/overview/` — отвергнуто.** Это курируемая витрина фиксированного состава (10 файлов),
защищённая `tests/test_system_docs.py`; добавление отдельного FAQ нарушит инвариант каталога
витрины и саму семантику «обзор, а не справочник процедур».
- **Новый раздел `docs/faq/` — отвергнуто.** Заведение top-level раздела ради одного документа —
scope-creep; нет канона/индекса/норматива сопровождения для нового раздела.
- **`docs/operations/FAQ_STOP.md` — выбрано.** Это де-факто дом человеко-ориентированных процедур
и «что делать, если…» (тробл-шутинг STOP в FR-2 п.7 ссылается на `stop_status_enabled`/
`stop_status_repos`, а «где увидеть результат» в п.8 — на read-only блок `stop` в `GET /queue`;
обе темы — операторская территория). Пользователь доски и оператор на self-hosting сильно
пересекаются; именно к operations-доке оператор отсылает пользователя.
Документированная остаточная издержка: лёгкое несоответствие «аудитория-пользователь ↔
секция-operations». Принимается осознанно (см. «Последствия»); пере-размещение в будущий
`docs/faq/` остаётся дешёвым (один файл + правка двух ссылок + одного теста).
### D2 — Граница объёма: docs-only, без рантайм-поверхности (NFR-1, AC-10)
Подтверждаю и фиксирую как архитектурный инвариант:
- `src/**`, `STAGE_TRANSITIONS`, `QG_CHECKS`, `check_*`, machine-verdict ключи, схема БД — **не
трогаются**; kill-switch не требуется (нет исполняемого кода).
- **`07-infra-requirements.md` — N/A** (топология/контейнеры/сеть не меняются).
- **`08-data-requirements.md` — N/A** (таблиц/колонок/индексов не добавляется).
- `docs/architecture/README.md` / `internals.md`**не обновляются**: задача не затрагивает
стадии/QG/компоненты (новый operations-FAQ описывает уже задокументированную фичу ORCH-090, не
вводя архитектурных сущностей). Внесение FAQ в архитектурный справочник было бы дублированием
источника истины (нарушение NFR-5).
- Coverage-гейт (ORCH-027): docs-only не добавляет строк `src/` → базовая линия покрытия не
меняется; `tests/test_faq_stop_doc.py` — структурный тест документа, `src/` не покрывает и на
метрику не влияет.
### D3 — Контур анти-дрейфа: `tests/test_faq_stop_doc.py`, негативный скан на уровне утверждений (NFR-4, FR-6, AC-11)
Структурный тест по образцу `tests/test_lite_setup_doc.py` — детерминированный, **без сети/LLM/
subprocess**, только парсинг файла. Обязательный состав проверок:
1. **Существование** `docs/operations/FAQ_STOP.md`.
2. **Позитивные якоря** — все 8 обязательных секций-вопросов TRZ §FR-2 присутствуют (заголовки —
стабильные якоря; тест матчит по нормализованному заголовку, не по точной пунктуации).
3. **«Кирпичи»-факты** — присутствуют ключевые токены (`STOP`, `cancelled`, «To Analyse»,
«отлож…»/`deferred`, упоминание `GET /queue`/блока `stop`).
4. **Кросс-ссылки** (FR-4) — ссылка на `tech-pipeline.md` и на ADR ORCH-090 присутствует.
5. **Негативный скан (КЛЮЧЕВОЙ нюанс).** Запрещённые **утверждения** FR-5 («STOP откатывает
прод», «STOP трогает `main`/force-push», «продолжить с середины», «STOP мгновенно убивает
деплой») детектируются как **утверждения целиком**, а **НЕ** как голые подстроки. Причина:
корректный FAQ закономерно содержит слова `main`, «откатыва…», «force-push», «деплой» внутри
**отрицающих** предложений («STOP **не откатывает**`main`»). Наивный substring-скан по этим
словам ложно завалит именно те фразы, которые требование AC-9 предписывает иметь. Реализация:
матчить нормативно-запрещённые фразы (например, утверждение отката прод-кода **без**
отрицания рядом), либо проверять, что запрещённый токен встречается только в соседстве с
отрицанием. Конкретную форму выбирает разработчик; инвариант — **тест не должен фолзить на
фактически верном FAQ** и **обязан краснеть на реально ложном утверждении**.
Контракт теста — никогда не делать сеть/LLM/subprocess (как и эталон), чтобы оставаться частью
обычного зелёного `tests/` без инфра-зависимостей.
### D4 — Целостность ссылок и link-first (FR-4, NFR-5, AC-8)
Перекрёстные ссылки добавляются **в обе стороны** (витрина/обзор → FAQ; FAQ → обзор + ADR
ORCH-090). Источник истины поведения остаётся за ADR ORCH-090 и инженерным обзором — FAQ их
**не форкает** (машинные детали: маркеры/lease/тумбстон — только ссылками). Цели ссылок
проверены существующими (см. Контекст). Якорь-слаг на секцию обзора
(`tech-pipeline.md` «Отмена: STOP → `cancelled`») разработчик обязан сверить с фактической
генерацией якоря при переносе (риск TR-4).
### D5 — Норматив сопровождения (traceability)
Фиксируется правило: **правишь поведение STOP** (`src/cancel.py` / `cancel_task` / маршрут `stop`
в `src/webhooks/plane.py`) → **обнови `docs/operations/FAQ_STOP.md` в том же PR** (правило агентов
№2/№6; reviewer-ось «документация»). Машинный маркер `ORCH-108` в `src/**` НЕ вводится (docs-only),
поэтому анти-археологии маркеров (`docs/_standards/TRACEABILITY.md`) этот PR не порождает; связь
«код STOP ↔ FAQ» держится нормативом сопровождения + структурным тестом D3.
## Альтернативы
- **FAQ в `docs/overview/`** — отвергнуто: курируемая витрина фиксированного состава под
`tests/test_system_docs.py`; FAQ ≠ обзорный слайд (см. D1).
- **Новый раздел `docs/faq/`** — отвергнуто: scope-creep ради одного файла (см. D1).
- **Без анти-дрейф теста, полагаясь на reviewer** — отвергнуто: NFR-4 требует структурной
защиты от дрейфа «доки ↔ код»; ручная проверка не воспроизводима.
- **Негативный скан по голым подстрокам** — отвергнуто: ложные срабатывания на корректно
отрицающих предложениях (см. D3) — это сделало бы тест либо красным на верном FAQ, либо
вынудило бы выкинуть из FAQ обязательные явные отрицания.
- **Сквозной (global) ADR** — отвергнуто: решение не кросс-каттинговое (нет нового QG/стадии/
компонента/таблицы; не меняет канон доков как такой).
## Последствия
- **+** Единый самодостаточный источник для пользователя доски → меньше неверных ожиданий и
обращений к оператору (self-hosting-выгода).
- **+** Структурный тест (D3) делает дрейф «доки ↔ код» воспроизводимо ловимым; норматив D5
закрывает процессный пробел.
- **+** Нулевой рантайм-риск: docs-only, прод-деплой этой задачи безопасен.
- **** Лёгкое несоответствие «пользовательская аудитория ↔ секция operations» (D1). Митигейшн:
явный вводный абзац «для кого» в FAQ + дешёвое будущее пере-размещение.
- **** Риск чрезмерно строгого негативного скана (D3). Митигейшн: матч на уровне утверждений +
явный инвариант «не фолзить на верном FAQ» (TR-3).
- **Откат:** удалить `docs/operations/FAQ_STOP.md` и `tests/test_faq_stop_doc.py`, снять
добавленные ссылки из `business.md`/`tech-pipeline.md` и запись из `CHANGELOG.md`. Рантайм не
затрагивается.
## Ссылки
- BRD: `docs/work-items/ORCH-108/01-brd.md`
- TRZ: `docs/work-items/ORCH-108/02-trz.md` (+ Приложение A — черновик содержания FAQ)
- Acceptance: `docs/work-items/ORCH-108/03-acceptance-criteria.md`
- Tech-risks: `docs/work-items/ORCH-108/10-tech-risks.md`
- Источник истины поведения STOP: `docs/work-items/ORCH-090/06-adr/ADR-001-stop-cancel-task.md`,
`docs/architecture/adr/adr-0026-stop-cancel-task.md`
- Сверено по коду: `src/cancel.py`, `src/stage_engine.py::cancel_task`,
`src/webhooks/plane.py` (`handle_issue_updated`/`handle_stop`/`handle_status_start`),
`src/config.py` (`stop_status_enabled`/`stop_status_repos`), `src/main.py` (блок `stop` в
`GET /queue`)
- Эталон анти-дрейф теста: `tests/test_lite_setup_doc.py`
- Семантика разделов доков: `docs/architecture/adr/adr-0039-system-overview-docs-canon.md`

View File

@@ -0,0 +1,39 @@
---
work_item: ORCH-108
stage: architecture
author_agent: architect
status: proposed
created_at: 2026-06-17
model_used: claude-opus-4-8
---
# 10 — Технические риски: ORCH-108 — FAQ по статусу STOP
Work Item: **ORCH-108** · Repo: **orchestrator** (self-hosting) · Стадия: architecture
> Информационный документ (гейтом не парсится). Перечисляет риски реализации **docs-only**
> задачи и их митигейшн. Класс рисков — минимальный: рантайм/конвейер не затрагиваются.
## Реестр рисков
| ID | Риск | Вер. | Влия. | Митигейшн |
|----|------|------|-------|-----------|
| TR-1 | **Дрейф «доки ↔ код».** Будущая правка поведения STOP (`src/cancel.py`/`cancel_task`/маршрут `stop`) сделает FAQ неверным. | Сред. | Сред. | Структурный анти-дрейф тест `tests/test_faq_stop_doc.py` (ADR D3) + норматив сопровождения «правишь STOP → обнови FAQ в том же PR» (ADR D5) + reviewer-ось «документация». |
| TR-2 | **FAQ-«сирота» / дубль источника истины.** FAQ не связан с витриной или дословно копирует ADR/обзор вместо ссылки. | Низ. | Низ. | Link-first (ADR D4): двусторонние ссылки (AC-8), машинные детали — только ссылками; тест проверяет наличие кросс-ссылок. |
| TR-3 | **Ложно-строгий негативный скан.** Тест ищет запрещённые слова (`main`, «откатыва…», `force-push`) как голые подстроки → краснеет на корректно **отрицающих** предложениях FAQ (которые AC-9 предписывает иметь). | Сред. | Сред. | Негативный скан — на уровне **утверждений**, а не подстрок (ADR D3); инвариант «тест не фолзит на верном FAQ, но краснеет на реально ложном». Зеркало эталона `tests/test_lite_setup_doc.py`. |
| TR-4 | **Битый якорь кросс-ссылки.** Ссылка `tech-pipeline.md#отмена-stop--cancelled` не совпадёт с фактически генерируемым slug заголовка «Отмена: STOP → `cancelled`». | Низ. | Низ. | Разработчик сверяет slug при переносе (ADR D4); цели секций подтверждены существующими (business.md §«Сценарий 6», tech-pipeline.md §«Отмена», ADR ORCH-090). |
| TR-5 | **Фактическая неточность FAQ.** Утверждение расходится с кодом (напр. «STOP откатит прод», «убивает деплой мгновенно»). | Низ. | Выс. | NFR-2/FR-5/AC-9: каждое утверждение verifiable против read-only модулей (ADR §Ссылки); reviewer сверяет с кодом; негативный скан (TR-3) ловит запрещённый класс. Содержание выверено аналитиком (TRZ Приложение A). |
| TR-6 | **Ошибочное размещение раздела.** Аудитория FAQ — «пользователь доски», секция — `operations/` («наш прод»). | Низ. | Низ. | Осознанный компромисс (ADR D1): альтернативы (`overview/` под тестом витрины, новый `docs/faq/`) хуже; вводный абзац «для кого»; будущее пере-размещение дёшево (1 файл + 2 ссылки + 1 тест). |
## Сводный вывод
Доминирующий класс — **дрейф документации** (TR-1) и **хрупкость анти-дрейф теста** (TR-3); оба
структурно снижены решением D3 (claim-level негативный скан + детерминированный offline-тест) и
нормативом сопровождения D5. Рантайм-рисков **нет**: задача docs-only, не трогает `src/**`/
`STAGE_TRANSITIONS`/`QG_CHECKS`/схему БД, не деплоит/не рестартит прод/не трогает `main`
self-hosting-безопасна, прод-деплой безвреден.
**Эскалация не требуется.** Не `arch:major-change` (нет новой стадии/компонента/смены БД), возврат
в анализ не нужен (BRD/TRZ/AC полны и согласованы с кодом; блокирующих неоднозначностей нет —
`01-questions.md` аналитиком осознанно не создан). Остаточный риск для прод-конвейера —
**пренебрежимо мал**.

View File

@@ -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 (111)** — полное. **Соответствие ADR** — все решения
D1D5 реализованы. **Качество** — тест содержателен, детерминирован, с 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'а по этой оси нет.

View File

@@ -0,0 +1,40 @@
---
result: PASS
work_item: ORCH-108
stage: testing
author_agent: test-runner
status: success
created_at: 2026-06-17
model_used: n/a
exit_code: 0
smoke: ok
---
# Test Gate Log (deterministic runner, ORCH-116)
pytest exit-code `0` -> `result: PASS` (smoke: ok).
Вердикт зафиксирован детерминированным test-раннером (ORCH-116), не LLM. PASS/FAIL = exit-код `pytest` + read-only smoke (`/health`, `/status`, `/queue` + блок `serial_gate`).
pytest stdout (tail):
```
.................................................................... [ 64%]
........................................................................ [ 67%]
........................................................................ [ 71%]
........................................................................ [ 74%]
........................................................................ [ 77%]
........................................................................ [ 80%]
........................................................................ [ 84%]
........................................................................ [ 87%]
........................................................................ [ 90%]
........................................................................ [ 93%]
........................................................................ [ 96%]
................................................................... [100%]
=============================== warnings summary ===============================
src/config.py:8
/repos/_wt/orchestrator/feature_ORCH-108-19c40858/src/config.py:8: PydanticDeprecatedSince20: Support for class-based `config` is deprecated, use ConfigDict instead. Deprecated in Pydantic V2.0 to be removed in V3.0. See Pydantic V2 Migration Guide at https://errors.pydantic.dev/2.13/migration/
class Settings(BaseSettings):
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
2227 passed, 1 warning in 99.72s (0:01:39)
```

View File

@@ -0,0 +1,12 @@
---
deploy_status: SUCCESS
work_item: ORCH-108
hook_exit_code: 0
deployed_by: deploy-finalizer
---
# Deploy log — ORCH-036 executable self-deploy
Прод-деплой завершён хост-хуком с exit-code `0` -> `deploy_status: SUCCESS`.
Вердикт зафиксирован детерминированным finalizer'ом (Фаза C), не LLM.

View File

@@ -0,0 +1,46 @@
---
staging_status: SUCCESS
work_item: ORCH-108
stage: deploy-staging
author_agent: staging-runner
status: success
created_at: 2026-06-17
model_used: n/a
exit_code: 0
base_url: http://localhost:8501
---
# Staging Gate Log (deterministic runner, ORCH-115)
Staging suite exit-code `0` -> `staging_status: SUCCESS`.
Вердикт зафиксирован детерминированным staging-раннером (ORCH-115), не LLM. infra-tolerance (ORCH-061) уже учтена внутри `staging_check.py` — раннер её не пересуживает.
INFRA-WAIVED lines (ORCH-061, copied for observability):
- INFRA-WAIVED: C9a Branch appears in orchestrator-sandbox, C9b Analyst job enqueued in staging queue (known sandbox-infra; real checks green)
Staging suite stdout (tail):
```
(waiting for analyst job in queue)
· waiting... (waiting for analyst job in queue)
· waiting... (waiting for analyst job in queue)
· waiting... (waiting for analyst job in queue)
· waiting... (waiting for analyst job in queue)
· waiting... (waiting for analyst job in queue)
✗ FAIL C9b Analyst job enqueued in staging queue
[CLEANUP]
· CLEANUP: no branch to delete
✓ PASS CLEANUP: deleted Plane issue a38f627e-4ba4-47c3-a19f-3bb939a79a37 (HTTP 204)
· CLEANUP DB: no task row found for plane_id=a38f627e-4ba4-47c3-a19f-3bb939a79a37
· CLEANUP DB dedup: no such table: events_dedup
============================================================
 RESULT: 8/10 checks PASS
REAL failed : none
SANDBOX_INFRA failed: ['C9a Branch appears in orchestrator-sandbox', 'C9b Analyst job enqueued in staging queue']
============================================================
· tolerance: staging_infra_tolerance_enabled=True
INFRA-WAIVED: C9a Branch appears in orchestrator-sandbox, C9b Analyst job enqueued in staging queue (known sandbox-infra; real checks green)
VERDICT: SUCCESS (exit 0) — SUCCESS (infra-waived): ['C9a Branch appears in orchestrator-sandbox', 'C9b Analyst job enqueued in staging queue'] are known sandbox-infra checks; all real checks green
```

View File

@@ -1,14 +0,0 @@
---
post_deploy_status: HEALTHY
action_taken: NONE
work_item: ORCH-119
window_s: 900
checks_total: 30
checks_failed: 0
---
# Post-deploy log — ORCH-021 post-deploy monitor
Наблюдение прода завершено: `post_deploy_status: HEALTHY`, `action_taken: NONE`.
Окно наблюдения: 900s; опросов всего: 30, из них с провалом: 0.

291
tests/test_faq_stop_doc.py Normal file
View File

@@ -0,0 +1,291 @@
"""ORCH-108 (FR-6 / AC-1…AC-11): анти-дрейф контур пользовательского FAQ по STOP.
Структурные проверки golden source `docs/operations/FAQ_STOP.md` (ADR-001 D3,
образец — `tests/test_lite_setup_doc.py`): док существует и адресован пользователю
(TC-01); присутствуют все 8 обязательных секций-якорей FR-2 (TC-02); несёт ключевые
факты-«кирпичи» о последствиях STOP (TC-03), об отложенной отмене (TC-04), о том что
STOP не откатывает прод-код (TC-05) и о перезапуске через «To Analyse» с нуля (TC-06);
негативный скан запрещённых **утверждений** на уровне предложений, а НЕ голых подстрок
(TC-07, ключевой нюанс D3 — иначе тест краснел бы на корректно отрицающих фразах);
двусторонние перекрёстные ссылки витрина/обзор ⇄ FAQ ⇄ ADR ORCH-090 (TC-08);
docs-only / детерминированность самого теста — без сети/LLM/subprocess (TC-09).
Детерминировано: только парсинг файлов (read_text), без сети/LLM/subprocess.
Поведение STOP в рантайме реализовано и протестировано в ORCH-090 — эта задача его
НЕ меняет (docs-only), здесь проверяется лишь структурная целостность документации.
"""
import re
from pathlib import Path
REPO_ROOT = Path(__file__).resolve().parents[1]
FAQ = REPO_ROOT / "docs/operations/FAQ_STOP.md"
BUSINESS = REPO_ROOT / "docs/overview/business.md"
TECH_PIPELINE = REPO_ROOT / "docs/overview/tech-pipeline.md"
# Обязательные секции FR-2: распознаваемая (нормализованная, регистронезависимая)
# подстрока заголовка `## ` — стабильный якорь, толерантный к полировке пунктуации.
SECTION_ANCHORS: tuple[str, ...] = (
"что делает статус stop", # (1) назначение
"как отменить задачу", # (2) как отменить
"что происходит, когда я нажимаю stop", # (3) пошагово
"сливается или деплоится", # (4) отложенная отмена
"откатит ли stop уже выложенный код", # (5) не откатывает прод
"как перезапустить отменённую задачу", # (6) перезапуск с нуля
"ничего не произошло", # (7) no-op причины
"где увидеть, что задача отменена", # (8) где увидеть результат
)
# ---------------------------------------------------------------------------
# helpers
# ---------------------------------------------------------------------------
def _faq_text() -> str:
assert FAQ.is_file(), "docs/operations/FAQ_STOP.md отсутствует (AC-1)"
text = FAQ.read_text(encoding="utf-8")
assert text.strip(), "FAQ_STOP.md пуст (AC-1)"
return text
def _headers() -> list[str]:
"""Нормализованные (lowercase) тексты markdown-заголовков `#`/`##`/…."""
return [
re.sub(r"^#+\s*", "", line).strip().lower()
for line in _faq_text().splitlines()
if line.lstrip().startswith("#")
]
def _body_sentences() -> list[str]:
"""Предложения тела FAQ БЕЗ заголовков и без code-fence-блоков.
Заголовки этого FAQ — вопросы («Откатит ли STOP…?», «Что если…?») и по
построению не несут утверждений; негативный скан утверждений (TC-07)
исключает их, чтобы не краснеть на легитимных вопросах-якорях (D3/TR-3).
"""
lines: list[str] = []
in_fence = False
for raw in _faq_text().splitlines():
stripped = raw.strip()
if stripped.startswith("```"):
in_fence = not in_fence
continue
if in_fence or stripped.startswith("#"):
continue
lines.append(raw)
body = " ".join(lines)
return [s.strip().lower() for s in re.split(r"[.!?\n]+", body) if s.strip()]
_NEGATION = re.compile(r"\b(?:не|нет|никогда|нельзя|без)\b", re.IGNORECASE)
# Опасные claim-триггеры (D3): ЕСЛИ предложение тела матчит триггер, оно ОБЯЗАНО
# нести отрицание рядом, иначе это запрещённое FR-5 утверждение. Скан на уровне
# предложений-утверждений, НЕ голых подстрок (заголовки-вопросы исключены).
_CLAIM_TRIGGERS: tuple[tuple[str, re.Pattern[str]], ...] = (
# «STOP откатывает прод/влитый код» — должно встречаться только с отрицанием.
("откат прод-кода", re.compile(r"откат\w*", re.IGNORECASE)),
# «можно продолжить отменённую с середины».
("продолжить с середины", re.compile(r"продолж\w*[^.]{0,40}середин", re.IGNORECASE)),
# «STOP трогает main / делает force-push».
("трогает main / force-push", re.compile(r"трога\w*|force[\s-]?push", re.IGNORECASE)),
# «STOP мгновенно убивает/прерывает идущий деплой/слияние».
(
"мгновенно убивает деплой",
re.compile(r"(?:мгновенн|немедленн)\w*[^.]{0,40}(?:убива|прерыва|деплой|слиян|выклад)", re.IGNORECASE),
),
)
def _forbidden_claim_violations(sentences: list[str]) -> list[str]:
"""Предложения, которые делают запрещённое утверждение без отрицания рядом."""
violations: list[str] = []
for name, trigger in _CLAIM_TRIGGERS:
for sent in sentences:
if trigger.search(sent) and not _NEGATION.search(sent):
violations.append(f"[{name}] {sent[:120]}")
return violations
# ---------------------------------------------------------------------------
# TC-01: FAQ существует, непустой, адресован пользователю (AC-1).
# ---------------------------------------------------------------------------
def test_faq_exists_user_oriented_with_h1():
text = _faq_text()
headers = _headers()
assert headers, "в FAQ_STOP.md нет ни одного заголовка"
assert "stop" in headers[0], f"H1 FAQ не про STOP: {headers[0]!r} (AC-1)"
# Вводный абзац «для кого/зачем» (тон пользовательский, без требования читать код).
intro = text.split("\n##", 1)[0].lower()
assert "для пользователя доски plane" in intro, (
"нет вводного абзаца «для кого» (AC-1) — FAQ должен адресоваться пользователю"
)
assert "не нужно" in intro and "код" in intro, (
"вводный абзац не фиксирует пользовательский тон «читать код не нужно» (AC-1)"
)
# ---------------------------------------------------------------------------
# TC-02: присутствуют все 8 обязательных секций-якорей FR-2 (AC-2).
# ---------------------------------------------------------------------------
def test_all_mandatory_sections_present():
headers_blob = " || ".join(_headers())
missing = [a for a in SECTION_ANCHORS if a not in headers_blob]
assert not missing, f"в FAQ_STOP.md отсутствуют обязательные секции FR-2 (AC-2): {missing}"
# ---------------------------------------------------------------------------
# TC-03: пошаговые последствия и сохранность docs (AC-3).
# ---------------------------------------------------------------------------
def test_step_consequences_and_docs_preserved():
low = _faq_text().lower()
bricks = {
"остановка агента": "останав",
"снятие job'ов": "job",
"удаление рабочей ветки": "ветк",
"удаление worktree": "worktree",
"переход в cancelled": "cancelled",
"уведомление telegram": "telegram",
"комментарий plane": "plane",
"docs сохраняются": "сохран",
"main/прод не трогаются": "main",
}
missing = [name for name, token in bricks.items() if token not in low]
assert not missing, f"раздел «что происходит» не несёт фактов (AC-3): {missing}"
# ---------------------------------------------------------------------------
# TC-04: критичное окно — отложенная отмена + main/прод не трогаются (AC-4).
# ---------------------------------------------------------------------------
def test_deferred_cancel_in_critical_window():
low = _faq_text().lower()
assert "отлож" in low, "нет факта отложенной отмены в критичном окне (AC-4)"
# «main/прод не трогаются» — явное отрицание рядом с критичным окном.
assert "не трогают" in low or "не трогаются" in low, (
"нет явного «main/прод не трогаются» (AC-4)"
)
# ---------------------------------------------------------------------------
# TC-05: STOP ≠ откат прод-кода — явный отрицательный ответ (AC-5).
# ---------------------------------------------------------------------------
def test_stop_does_not_revert_prod():
low = _faq_text().lower()
assert "не откатывает" in low, "нет явного «не откатывает» прод/влитый код (AC-5)"
assert "revert" in low or "отдельная задача" in low, (
"не сказано, что revert выложенного — отдельная задача (AC-5)"
)
# ---------------------------------------------------------------------------
# TC-06: перезапуск — «To Analyse» с нуля, без «продолжить с середины» (AC-6).
# ---------------------------------------------------------------------------
def test_restart_via_to_analyse_from_scratch():
text = _faq_text()
low = text.lower()
assert "to analyse" in low, "перезапуск не через «To Analyse» (AC-6)"
assert "с нуля" in low, "не сказано, что задача создаётся «с нуля» (AC-6)"
# «продолжить с середины» допустимо только в отрицающем предложении.
for sent in _body_sentences():
if "продолж" in sent and "середин" in sent:
assert _NEGATION.search(sent), (
f"обещание продолжить отменённую задачу с середины (AC-6): {sent[:120]!r}"
)
# ---------------------------------------------------------------------------
# TC-07a: «ничего не произошло» и идемпотентность — причины no-op (AC-7).
# ---------------------------------------------------------------------------
def test_noop_reasons_present():
low = _faq_text().lower()
assert "stop_status_enabled" in low and "stop_status_repos" in low, (
"не перечислены настройки отключения отмены (AC-7)"
)
assert "уже" in low and ("заверш" in low or "отмен" in low), (
"не описан идемпотентный no-op для уже терминальной задачи (AC-7)"
)
# ---------------------------------------------------------------------------
# TC-07b: негативный скан запрещённых утверждений (claim-level, не подстроки).
# Инвариант D3: НЕ фолзит на верном FAQ; обязан краснеть на реально ложном (AC-9).
# ---------------------------------------------------------------------------
def test_no_forbidden_claims_in_real_faq():
violations = _forbidden_claim_violations(_body_sentences())
assert not violations, (
"FAQ_STOP.md содержит запрещённые FR-5 утверждения без отрицания (AC-9):\n"
+ "\n".join(violations)
)
def test_forbidden_claim_scanner_is_not_evergreen():
"""Подсаженные ложные утверждения ОБЯЗАНЫ ловиться (паттерн ORCH-101/102:
сканер не может молча стать вечнозелёным). Зеркало
test_secret_heuristic_is_not_evergreen из test_lite_setup_doc.py."""
bad = [
"stop откатывает уже выложенный в прод код автоматически",
"отменённую задачу можно продолжить с середины конвейера",
"stop делает force-push в main для зачистки",
"stop мгновенно убивает идущий деплой на полпути",
]
caught = _forbidden_claim_violations(bad)
assert len(caught) >= 4, f"сканер не ловит подсаженные ложные утверждения: {caught}"
# И обратно: корректные ОТРИЦАЮЩИЕ предложения НЕ должны ловиться (анти-TR-3).
good = [
"stop не откатывает код, уже влитый в main или выложенный в прод",
"отменённую задачу нельзя продолжить с середины",
"ветка main и прод никогда не трогаются",
]
assert _forbidden_claim_violations(good) == [], (
"сканер ложно краснеет на корректно отрицающих предложениях (TR-3)"
)
# ---------------------------------------------------------------------------
# TC-08: двусторонние перекрёстные ссылки витрина/обзор ⇄ FAQ ⇄ ADR (AC-8).
# ---------------------------------------------------------------------------
def test_overview_docs_link_to_faq():
assert "FAQ_STOP.md" in BUSINESS.read_text(encoding="utf-8"), (
"docs/overview/business.md (Сценарий 6) не ссылается на FAQ_STOP.md (AC-8)"
)
assert "FAQ_STOP.md" in TECH_PIPELINE.read_text(encoding="utf-8"), (
"docs/overview/tech-pipeline.md (Отмена: STOP → cancelled) не ссылается на FAQ_STOP.md (AC-8)"
)
def test_faq_links_back_to_overview_and_adr():
text = _faq_text()
assert "tech-pipeline.md" in text, "FAQ не ссылается на инженерный обзор (AC-8)"
assert "ORCH-090" in text and "ADR-001-stop-cancel-task.md" in text, (
"FAQ не ссылается на ADR ORCH-090 как источник истины (AC-8)"
)
# link-first: машинные детали не дублируются (даются ссылкой), а не копируются.
low = text.lower()
for machine_detail in ("тумбстон", "merge-lease", "_ensure_column"):
assert machine_detail not in low, (
f"FAQ дублирует машинную деталь {machine_detail!r} вместо ссылки (AC-8/D4)"
)
# ---------------------------------------------------------------------------
# TC-09: детерминированность самого теста — docs-only, без сети/LLM/subprocess.
# ---------------------------------------------------------------------------
def test_this_module_is_deterministic_offline():
"""Самочек контракта D3: тест-модуль не импортирует сеть/subprocess/LLM и не
тянет рантайм src/ — читает только файлы docs/ (AC-10, AC-11)."""
# Скан реальных import-стейтментов по началу строки (а не подстрок — иначе
# тест ловил бы собственные строковые литералы из этого же списка).
banned = ("subprocess", "socket", "requests", "httpx", "urllib")
offenders: list[str] = []
for line in Path(__file__).read_text(encoding="utf-8").splitlines():
stripped = line.strip()
if not (stripped.startswith("import ") or stripped.startswith("from ")):
continue
module = stripped.split()[1].split(".", 1)[0]
if module in banned or module == "src":
offenders.append(stripped)
assert not offenders, (
f"анти-дрейф тест FAQ нарушает контракт детерминированности (сеть/subprocess/"
f"рантайм src): {offenders}"
)