auto-sync: 2026-06-03 23:40:01
This commit is contained in:
@@ -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). Все боевые чистые.
|
||||
|
||||
Reference in New Issue
Block a user