diff --git a/memory/2026-06-03.md b/memory/2026-06-03.md index 7f51845..ed88fc5 100644 --- a/memory/2026-06-03.md +++ b/memory/2026-06-03.md @@ -257,3 +257,17 @@ test_taskdesc_includes_description, test_name_fetched_when_payload_empty, test_a ### Хронология багов входа/выхода analyst (закрыто за сессию): PR #12 (status-only verdict, баг3 эхо-самоудар) → PR #13 (баги A description, B name, C коммент+ссылки) → PR #14 (ссылки на внешний домен). Все боевые прогоны чистые, #6 In Review стабилен. + +--- + +## PR #15 — БАГ 4: Approved из analysis не двигал конвейер (03.06 ~20:32) +**Merge main = `b6aa107`.** Ветка `fix/approved-advances-stage`, коммит `0b8013c` `fix(stage): approved verdict advances analysis->architecture instead of re-running gate`. Задеплоен. +- **Симптом (боевой, Слава нажал Approved на #6):** лог `advance from analysis` → 200 OK, но task 29 застрял stage=analysis, architect НЕ запущен, в Architecture не уехало. Блокер. +- **Корень:** stage_engine.py advance_stage ветка `check_analysis_approved` ВСЕГДА `_handle_analysis_approved_flow(...) + return result` → блок Advance недостижим. А _handle_analysis_approved_flow при agent=None (webhook Approved) делал `return` (т.к. `if not (agent=='analyst' ...)`). Два пути через один тупик. +- **Фикс:** развести по `finished_agent`: agent=='analyst' (launcher) → approved-flow+return (In Review+коммент, как было); agent is None (Approved-вердикт) → НЕ заходить в flow, провалиться в общий Advance (update_task_stage(architecture)+enqueue architect). +- Сама проверила ДО фикса: `get_agent_for_stage('analysis')`=architect, `get_next_stage('analysis')`=architecture → фикс не зациклит analyst. +- Тесты сама: 210 passed + 9 baseline. Новый test_approved_verdict_advances_analysis_to_architecture зелёный. +- ✅ **БОЕВОЙ:** догнала Approved (task 29 застрял) → послала Approved-вердикт → `analysis -> architecture (auto-advance after None)`, enqueued architect job 11, run 64 pid 47. Plane #6 = Architecture (3020bbb7). **Конвейер впервые прошёл стадию end-to-end через ручной Approved.** +- architect запущен, проектирует. Следить: создаст docs/work-items/ET-011/ архитектуру → Development. + +### Хронология (вся сессия): PR#12 (status-only, баг3 эхо) → PR#13 (баги A desc/B name/C коммент+ссылки) → PR#14 (ссылки внешний домен) → PR#15 (баг4 Approved-advance). Все боевые чистые.