diff --git a/memory/2026-06-03.md b/memory/2026-06-03.md index 64089d2..7f51845 100644 --- a/memory/2026-06-03.md +++ b/memory/2026-06-03.md @@ -228,3 +228,32 @@ test_taskdesc_includes_description, test_name_fetched_when_payload_empty, test_a - HTML коммента `

...

` — формально невалидная вложенность (Plane/TipTap переваривает, ссылки рабочие). Косметика. - **#6 висит в In Review, ждёт Славу.** Approved → должна уехать в Architecture БЕЗ мелькания In Progress (PR #12, ещё не проверено живьём). Слава решит: сам в UI или мне симулировать. - Боевая ветка теперь `feature/ET-011-popup-enduro-trails` (slug исправлен). + +--- + +## PR #14 — ссылки в комменте analyst: localhost → внешний публичный Gitea (03.06 ~20:50) + +**Merge main = `b01643f`.** Ветка `fix/gitea-public-url` из main `dce9ac8` (после PR #13). Один коммит `ca63bc2` `feat(config): external gitea_public_url for clickable doc links`. Задеплоен. + +### Проблема +Ссылки на доки в комменте analyst строились от `settings.gitea_url` = `http://localhost:3000` (env `ORCH_GITEA_URL`) — это ВНУТРЕННИЙ URL для git-операций (clone/push из docker), из браузера Славы не кликается. + +### Решение (НЕ менять gitea_url — он для git!) +- `src/config.py`: новое поле `gitea_public_url: str = ""` (env `ORCH_GITEA_PUBLIC_URL`, префикс ORCH_, fallback на gitea_url). +- `src/stage_engine.py` `_build_analyst_ready_comment` (~стр.299): `base = (getattr(settings,"gitea_public_url","") or settings.gitea_url).rstrip("/")`. Только эта строка. +- `gitea_url` НЕ тронут — git-операции по-прежнему localhost. + +### Env на проде (ставила Я, не Dev): `ORCH_GITEA_PUBLIC_URL=https://git.mva154.duckdns.org` дописан в `.env` (аккуратно, не затирая), проверен в контейнере. + +### Тесты (мой прогон): **208 passed + 9 failed** (те же baseline pre-existing). test_analyst_comment_asks_approved_with_links расширен (public задан→ссылки от него) + новый test_analyst_comment_falls_back_to_gitea_url. + +### ✅ БОЕВОЙ ПРОГОН (task 29, run 63, exit 0, 20530 out, $1.37): ссылки в комменте на `https://git.mva154.duckdns.org`, кликабельны, все 6 доков. Коммент правильный (Approved/Rejected). **#6 = In Review ДЕРЖИТСЯ.** + +### ⚠️ Состояние на конец сессии: +- **#6 висит в In Review, ждёт вердикта Славы** в Plane (ссылки рабочие — глянуть доки → Approved или Rejected+причина). +- **НЕ проверено живьём:** переход Approved → Architecture без мелькания In Progress (PR #12). Слава решит сам в UI или симулировать. +- Был сбой PGPASSWORD на UPDATE статуса при ручном переводе #6 — webhook всё равно прошёл (analyst стартанул). Если повторять ручной перевод статуса через psql — подставлять реальный `$PW`, не литерал. +- HTML коммента `

...

` — формально невалидная вложенность (Plane переваривает, косметика). + +### Хронология багов входа/выхода analyst (закрыто за сессию): +PR #12 (status-only verdict, баг3 эхо-самоудар) → PR #13 (баги A description, B name, C коммент+ссылки) → PR #14 (ссылки на внешний домен). Все боевые прогоны чистые, #6 In Review стабилен. diff --git a/tasks/orchestrator/DEV_TASK_APPROVED_ADVANCE.md b/tasks/orchestrator/DEV_TASK_APPROVED_ADVANCE.md new file mode 100644 index 0000000..cdb83a5 --- /dev/null +++ b/tasks/orchestrator/DEV_TASK_APPROVED_ADVANCE.md @@ -0,0 +1,63 @@ +# DEV TASK — БАГ 4: Approved из analysis НЕ двигает конвейер на architecture + +**Проект:** orchestrator | **Сервер:** slin@82.22.50.71 | **Репо:** /home/slin/repos/orchestrator | **Контейнер:** orchestrator (8500) +**Ветка:** `fix/approved-advances-stage` из свежего main (`git checkout main && git pull && git checkout -b fix/approved-advances-stage`). main = `b01643f` (PR #14). + +⚠️ **ГРАБЛЯ push (ORCH-7):** после push `git log origin/main..origin/fix/approved-advances-stage` ДОЛЖЕН показать коммит ДО отчёта «PR готов». +ℹ️ Токен Gitea: `docker exec orchestrator printenv ORCH_GITEA_TOKEN`. + +## Симптом (боевой, воспроизведён) +Слава перевёл #6 (task 29) в статус **Approved** из analysis. Лог: `Task 29: Approved status -> advance from analysis` → сразу `200 OK`. **Но:** task 29 остался `stage=analysis`, нового job на architect НЕ создалось, статус в Plane не уехал в Architecture. Конвейер встал на первой стадии. **Это блокер — ручной Approved не продвигает.** + +## Корень +`src/stage_engine.py`, `advance_stage()` (~стр.191-197): +```python +if qg_name and qg_name in QG_CHECKS: + if qg_name == "check_analysis_approved": + _handle_analysis_approved_flow(task_id, current_stage, repo, work_item_id, branch, agent, result) + return result # ← ВСЕГДА ранний return, блок Advance (стр.~228+) недостижим +``` +`_handle_analysis_approved_flow` (стр.312) в начале: +```python +if not (agent == "analyst" and work_item_id): + return # webhook Approved: agent=None → выходит, НИЧЕГО не делает +``` +**Два пути через один гейт, оба упираются в ранний `return result`:** +- **Launcher** (analyst закончил, `finished_agent="analyst"`) → ставит In Review + коммент. ✅ Правильно, НЕ ломать. +- **Webhook Approved-вердикт** (`finished_agent=None`, зовётся из `_try_advance_stage` в plane.py) → `_handle_analysis_approved_flow` делает `return` (agent≠analyst) → блок **Advance** (`update_task_stage(next_stage)` + `enqueue_job(architect)`) НИКОГДА не достигается. + +`finished_agent` УЖЕ различает пути (launcher='analyst', webhook=None) — фикс на этом и строим. + +## Фикс — развести ветку check_analysis_approved по agent +`src/stage_engine.py`, `advance_stage`, ветка `if qg_name == "check_analysis_approved":` (~стр.193): +```python +if qg_name == "check_analysis_approved": + # Launcher path (analyst just finished): set In Review + ask for Approved. + # This gate never advances on its own — a human Approved does that. + if agent == "analyst": + _handle_analysis_approved_flow(task_id, current_stage, repo, work_item_id, branch, agent, result) + return result + # Webhook Approved-verdict path (agent is None): the human approved -> the + # analysis gate is satisfied. Skip the approval-flow and fall through to the + # generic Advance block below (advance stage + enqueue next agent). + # (do NOT return here) +``` +Т.е. при `agent is None` (Approved-вердикт) — **НЕ** заходим в approved-flow и **НЕ** делаем ранний return, а проваливаемся в общий блок Advance (стр.~228): `update_task_stage(task_id, next_stage)` → `plane_notify_stage` → `enqueue_job(architect)`. + +⚠️ **Проверь блок Advance:** `next_agent = get_agent_for_stage(current_stage)` (стр.~240) — для analysis вернёт `architect` (по `STAGES["analysis"]["agent"]`? сверь: в stages.py `"analysis": {"next":"architecture","agent":"architect",...}` — `get_agent_for_stage("analysis")` должно вернуть architect). Если вернёт analyst (повторный запуск!) — это вторая бага, останови и сообщи ДО фикса. По коду ORCH-4 fix комментарий «current_stage, not next» — значит agent берётся для СТАДИИ, которую покидаем; убедись что для analysis это architect, не analyst. + +## Ограничения +- 🚫 НЕ ломай launcher-путь (`agent="analyst"` → In Review + коммент остаётся как есть). +- 🚫 НЕ трогай: остальные QG (check_ci_green/check_tests_local/check_review_approved), rollback-ветки, Rejected-логику, Needs Input, nginx/openclaw.json/.env/deploy/HMAC/очередь(кроме enqueue)/webhook URL Plane/PLANE_STATES/M-6/uniqueness-guard/status-only PR#12/PR#13/PR#14/conftest.py/gitea_public_url. +- Conventional Commits, один коммит: `fix(stage): approved verdict advances analysis->architecture instead of re-running gate`. + +## Тесты (контейнер) +`IMG=$(docker inspect orchestrator --format '{{.Config.Image}}'); docker run --rm -v /home/slin/repos/orchestrator:/code -w /code --entrypoint python3 $IMG -m pytest tests/ -q` +Новый `test_approved_verdict_advances_analysis_to_architecture`: +- вызов `advance_stage(task_id, "analysis", repo, wid, branch, finished_agent=None)` (Approved-путь) → результат: `result.advanced is True`, `result.to_stage == "architecture"`, `result.enqueued_agent == "architect"` (мокни enqueue_job/update_task_stage/QG как в соседних тестах). +- регресс-проверка: `advance_stage(..., finished_agent="analyst")` (launcher-путь) → по-прежнему In Review/коммент, НЕ advance (result.advanced НЕ True, _handle_analysis_approved_flow вызван). +Baseline: **208 passed + 9 pre-existing failed** — не ломать pre-existing. + +## Отчёт +- НЕ деплоить, НЕ мержить (мерж + боевой прогон делает ассистент). +- В отчёте: точные строки правки, подтверди что launcher-путь (agent=analyst) не сломан, что get_agent_for_stage("analysis")=architect (не analyst!), вывод нового+релевантных тестов. Если get_agent_for_stage вернул analyst — СТОП, сообщи. Один исполнитель, маленькая правка.