refactor: remove dead progress_state, ProjectContext jobs -> job_manager.list_jobs + jobs_failed (open thread closed)

Author: MSCodeBase Agent

AGENT_DIARY.md

@@ -12,12 +12,23 @@
 
 ---
 
+## [2026-08-05 21:56] — Открытая нить закрыта: progress_state удалён (dead code), project_context → job_manager (единый источник прогресса)
+
+**Status:** ✅ Fixed
+**Root Cause:** `_create_progress_callback`/`_last_progress` (src/core/progress_state.py) в проде не вызывались (внутренний callback layer.py маппит прогресс в `job.progress`, не в `_last_progress`) → get_last_progress() всегда пуст → `ProjectContext._capture_jobs` вечно 0/0. `JobManager.cleanup_old_jobs()` определён, но не вызывался нигде (латентный рост `jobs`).
+**Fix:** механизм удалён целиком: progress_state.py (92 строки) + 6 реэкспортов в mcp/server.py + 11 легаси-тестов; `_capture_jobs` переключен на `job_manager.list_jobs()` (новый метод с ленивым cleanup — cleanup_old_jobs теперь реально работает); в снэпшот добавлен честный счётчик `jobs_failed`; исторические комментарии (architecture_linter, test_architecture_lifecycle) обновлены.
+**Guard:** tests/test_index_progress.py переписан (9 тестов: JobManager lifecycle/list/cleanup + _capture_jobs mapping running/completed/failed); полный pytest 796 passed / 4 skipped (0 failed); grep `progress_state|_create_progress_callback|_last_progress` — 0 ссылок в src/tests (только обновлённые комментарии).
+**Pattern:** NEW-класс «диагностический accessor без проверки прод-использования» — механизм держался 1 версию «для диагностики», никто не проверил, что он мёртв. Guard: открытая нить закрыта явно, по плану владельца.
+**verified_from_clean_state:** ⚠️ не проверено — clean-clone не запускался (нет repo URL/сети); полный pytest 796 passed запущен явно. Runtime (после install+reload): ✅ MCP PID 12076, BUILD_ID=HEAD; live-проверка — intel_trigger_reindex (инкрементальный job) → intel_get_project_context().jobs = {running:0, completed:1, failed:0} (до фикса было вечно 0/0).
+
+---
+
 ## [2026-08-05 22:50] — Следующий шаг: get_last_progress → core, фикс bump_version, фикс sys.path-загрязнения теста (FIXED, будет закоммичено)
 
 **Status:** ✅ Fixed (не закоммичено — коммит следом)
 **Root Cause:** (1) техдолг из ARCH-03-цепочки: `project_context` импортировал `get_last_progress` из mcp.server — направление core→mcp оставалось; (2) `version_manager.check_consistency` ловил ВСЕ `X.Y.Z` (версии зависимостей, старых записей) как дрифты; `scripts/bump_version.py` вставлял заголовок после первого `---`, для ru/zh он попадал в середину файла; (3) `test_architecture_lifecycle` на уровне модуля делал `sys.path.insert(0, extension_dir)` — вся pytest-сессия импортировала УСТАРЕВШУЮ копию src из установленного расширения → ModuleNotFoundError для новых core-модулей (вскрыто переносом progress_state).
 **Fix:** (1) новый `src/core/progress_state.py` (состояние+callback+cleanup), mcp.server реэкспортирует, project_context импортирует из core, исключения в architecture_linter убраны; (2) per-file версионные паттерны в check_consistency, вставка ПЕРЕД первым `## [X.Y.Z]` в обоих bump (scripts + version_manager), version_manager обновляет все три CHANGELOG; (3) sys.path/env-загрязнение перенесено в autouse-fixture с восстановлением (намерение теста «установленное расширение» сохранено).
-**Guard:** tests/test_version_manager.py (4 регрессионных: ложные дрифты, реальный дрифт, вставка заголовка, три CHANGELOG); полный pytest 799 passed / 4 skipped (0 failed); вскрытая аномалия: `_create_progress_callback` в проде не вызывается → `get_last_progress()` всегда пуст — открытая нить.
+**Guard:** tests/test_version_manager.py (4 регрессионных: ложные дрифты, реальный дрифт, вставка заголовка, три CHANGELOG); полный pytest 799 passed / 4 skipped (0 failed); вскрытая аномалия: `_create_progress_callback` в проде не вызывается → get_last_progress() всегда пуст — открытая нить.
 **Pattern:** P-002 «предположение вместо проверки» (симптом: «тесты флейкят» → реальная причина: глобальное загрязнение sys.path чужим тестом).
 **verified_from_clean_state:** ⚠️ не проверено — clean-clone скрипт не запускался (нет repo URL/сети); полный pytest 799 passed запущен явно, 2 раза подряд (стабильно).
 

KNOWN_ISSUES.md

@@ -12,6 +12,13 @@
 
 ---
 
+## 2026-08-05 — progress_state удалён (dead code), project_context → job_manager — открытая нить закрыта (FIXED)
+
+**Symptom:** `_create_progress_callback` в проде не вызывался → `get_last_progress()` всегда пуст → `intel_get_project_context().jobs` вечно `{running: 0, completed: 0}` (открытая нить из записи 2026-08-05 «get_last_progress → core»). `cleanup_old_jobs()` не вызывался нигде.
+**Root Cause:** два механизма прогресса: прод-путь (job_manager, layer.py `_index_progress_callback`) и легаси-путь (`_last_progress`, только тесты). Легаси-путь — dead code с 3.3.12.
+**Fix:** удалён `src/core/progress_state.py` (92 строки) + 6 реэкспортов в mcp/server.py + 11 легаси-тестов; `_capture_jobs` переключен на `job_manager.list_jobs()` (новый метод: снимок + ленивый cleanup); снэпшот: добавлен `jobs_failed`.
+**Status:** ✅ Fixed | **Guard:** tests/test_index_progress.py переписан (9 тестов); полный pytest 796 passed / 4 skipped (0 failed); grep-развёртка 0 ссылок в коде.
+
 ## 2026-08-05 — get_last_progress → core (техдолг ARCH-03 закрыт) + bump_version фиксы + sys.path-загрязнение теста (FIXED)
 
 **Symptom:** (1) core→mcp импорт оставался: `project_context` брал `get_last_progress` из mcp.server; (2) bump_version: ложные дрифты (версии зависимостей), кривая вставка заголовка в ru/zh CHANGELOG; (3) при полном прогоне pytest падали 13 тестов с `ModuleNotFoundError: src.core.progress_state` только при запуске вместе с test_architecture_lifecycle.
@@ -2650,3 +2657,149 @@ Three fixes from the same review:
 **Root Cause:** 6 ❌ P1/P2 пунктов из experiments/audit.md требовали фикса: Heartbeat GetLastError, hardcoded reranker weights, BM25 sync reindex, SQLite sch...
 - **Статус:** автоматически синхронизировано
 
+## 2026-08-05 21:56 — Открытая нить закрыта: progress_state удалён (dead code), project_context → job_manager (единый источник прогресса)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed
+**Root Cause:** `_create_progress_callback`/`_last_progress` (src/core/progress_state.py) в проде не вызывались (внутренний callback layer.py маппит прогресс в `job.progress`, не в...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 22:50 — Следующий шаг: get_last_progress → core, фикс bump_version, фикс sys.path-загрязнения теста (FIXED, будет закоммичено)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (не закоммичено — коммит следом)
+**Root Cause:** (1) техдолг из ARCH-03-цепочки: `project_context` импортировал `get_last_progress` из mcp.server — направление core→mcp оставалось;...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 22:10 — experiments/audit.md: 16 пунктов верифицировано, 12 исправлено (FIXED, не запушено)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (не закоммичено — по команде владельца)
+**Root Cause:** audit.md накопил 4 наложенных аудита; свежий (ARCH/BL/WIN/ZED/SEC/TEST) содержал подтверждаемые проблемы: version drift (pyp...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 21:15 — Триаж KNOWN_ISSUES#2026-08-04-21:00 (Zed crash-loop) — цифры верифицированы замером
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** 🟡 Partial — loop остановлен (последний краш 08-04 21:27), риск сохраняется; дедлайн владельца 08-11
+**Root Cause:** подтверждён: Zed commit 8.54GB при commit-лимите 18.5GB (свободно 1.14GB...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 — D1: schema-слой из Neuro-Symbolic спайка → CypherExecutor (архитектурное закрытие P-004, FIXED, не запушено)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии; push — по команде владельца)
+**Root Cause:** P-004 «разрыв валидации между слоями»: неизвестные label/rel (галлюцинация LLM: `MATCH (f:SERVICE)`) принимались...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 20:10 — C1-C4 Cypher-стек: 4 бага KNOWN_ISSUES#2026-08-05 (FIXED, не запушено)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии; push — по команде владельца)
+**Root Cause:** C1 — label/edge сравнивались точно (=/IN) в cypher_sql.py, лексер принимает любой регистр LABEL → тихий пустой р...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 — A2 (внешний аудит): sandbox threat model — ADR-0001 ✅ Accepted (Вариант A)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Done (решение по умолчанию, §1.10 — владелец не выбрал B/C; переопределение возможно)
+**Root Cause:** внешний аудит: blacklist-модель sandbox принципиально обходима (чистый Python без ОС...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 01:50 — Tech debt: subprocess text=True без encoding ×7 закрыт + пин ruff (DONE)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Done (коммит в этой сессии)
+**Root Cause:** text=True без encoding в 7 местах декодирует вывод через locale (cp1251/cp1252 на Windows) — UnicodeDecodeError при не-ASCII выводе (тот же кл...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-05 00:05 — CI красный: ruff I001 (10 импорт-блоков, НЕ coverage) (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит a7a7a9e7, запушен)
+**Root Cause:** CI-прогоны b121ab19/6dc8d2ae упали на lint-шаге `ruff check src/ tests/` — 10 ошибок I001 (неотсортированные импорты) в 8 файлах: src/cor...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:59 — CI: кэш pip + coverage 41% (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** Next Action #9-#10: CircuitBreaker «dead» — ❌ REFUTED (подключён к embedder напрямую, di_container:337-345); coverage отсутствовал.
+**Fix:** ...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:59 — Триаж bare-except: 4 рискованных silent-блока залогированы (PARTIAL)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** 🟡 partial (коммит в этой сессии)
+**Root Cause:** scan нашёл 106 silent-блоков (except → pass); большинство намеренные (CancelledError/таймауты/best-effort).
+**Fix:** логирование в 4 местах...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:59 — Баг-клоуза: layer.py порт LM + резолв 7 VERIFY (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** из 3 аудитов остались VERIFY-пункты; единственный реальный баг — layer.py:504 хардкод порта LM Studio 1234 (рядом код уже читал порты из conf...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:58 — Hotfix: pickle P1 закрыт restricted unpickler'ом (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** index_guard.py:367 обычный pickle.load на legacy symbol_index.pkl — RCE-вектор (OWASP десериализация).
+**Fix:** `_LegacyPickleLoader(pickle.U...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:50 — Глубокий аудит (2-й проход): верификация 26 пунктов (TRIAGE)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** 🟡 триаж завершён, фиксы запланированы (коммит в этой сессии)
+**Root Cause:** второй внешний аудит (async, subprocess, BLE001, coverage, порты). Проверено по коду: create_task fire-and-forg...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:30 — Триаж внешнего ревью: 165 находок, тесты зелёные (TRIAGE)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** 🟡 триаж завершён, фиксы запланированы (коммит в этой сессии)
+**Root Cause:** внешний инструмент нашёл 165 проблем; критические проверены по коду: SQL_INJECTION (graph.py x4) — ❌ ложные (па...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 23:00 — CI clean-state: No module named pytest (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** Linux-ветка verify_clean_state.sh ставила `pip install -e ".[dev]" --no-deps` — dev-зависимости (pytest и др.) не входят в requirements-lock....
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 22:25 — scripts/monitor.py: UnboundLocalError avg_log (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** переменная `avg_log` присваивалась только в ветке фаз эмбеддинга (PHASE_EMBED/WRITING/IVF), а читалась в блоке «Тренд» при любой фазе — при ф...
+- **Статус:** автоматически синхронизировано
+
+
+## 2026-08-04 22:40 — test_job_history: изоляция от переиспользования tmp_path (FIXED)
+
+- **Источник:** AGENT_DIARY.md
+- **Описание:** **Status:** ✅ Fixed (коммит в этой сессии)
+**Root Cause:** JobHistoryStore пишет во внешний `<data_root>/projects/<hash>/metrics/job_history.json`, а pytest переиспользует temp-пути между запусками (с...
+- **Статус:** автоматически синхронизировано
+

scripts/architecture_linter.py

@@ -34,9 +34,10 @@
     "mcp.tools",
 }
 
-# Core больше не импортирует MCP (resolve_project_root — закрыт в 3.3.11,
-# get_last_progress — закрыт в 3.3.12). Оставшиеся исключения — tools.base
-# из intelligence.layer (документированный мост для _grep_fallback).
+# Core больше не импортирует MCP (resolve_project_root — закрыт в 3.3.11;
+# progress_state/get_last_progress — удалён, единый источник прогресса —
+# job_manager в src/core/intelligence/jobs.py). Оставшиеся исключения —
+# tools.base из intelligence.layer (документированный мост для _grep_fallback).
 _ALLOWED_CORE_MCP_IMPORTS: dict[str, list[str]] = {
     "src.core.intelligence_layer": ["src.mcp.tools.base"],
     "src.core.intelligence.layer": ["src.mcp.tools.base"],

src/core/intelligence/jobs.py

@@ -11,7 +11,7 @@
 import time
 import uuid
 from dataclasses import dataclass
-from typing import Any, Dict, Optional
+from typing import Any, Dict, List, Optional
 
 __all__ = [
     "BackgroundJob",
@@ -73,6 +73,17 @@ def get_job(self, job_id: str) -> Optional[BackgroundJob]:
         """Возвращает задачу по ID."""
         return self.jobs.get(job_id)
 
+    def list_jobs(self) -> List[BackgroundJob]:
+        """Возвращает снимок всех задач (после ленивой очистки завершённых).
+
+        Единственная точка перечисления job_manager'а — используется
+        ProjectContext._capture_jobs для агрегированной статистики.
+        Ленивый cleanup на чтении держит self.jobs ограниченным:
+        cleanup_old_jobs() иначе не вызывается нигде (латентный рост).
+        """
+        self.cleanup_old_jobs()
+        return list(self.jobs.values())
+
     def cleanup_old_jobs(self, max_age_seconds: int = 3600):
         """Удаляет старые завершённые задачи (защита от memory leak)."""
         now = time.time()