fix: audit sprint — heartbeat SetLastError(0), reranker weights to config, sqlite schema cols, PYTHONUTF8=1, shell=False in install.py

5/6 P1/P2 items fixed, 1 refuted (BM25 reindex = cache invalidation only).
- server_factory.py: SetLastError(0) before OpenProcess; GetLastError only when handle==0
- settings.py/engine.py: SearchConfig.bm25_weight/dense_weight from env (BM25_WEIGHT/DENSE_WEIGHT)
- server.py: PRAGMA table_info validation for scoped_kv_store/workspaces columns
- zed_config.py: PYTHONUTF8=1 in _make_server_entry (Windows cp1251 -> UTF-8)
- install.py: _run() -> shlex.split + shell=False; step_pip Popen -> args list; fix stray ) syntax error
- di_container.py: no change needed — BM25Mixin.reindex() only invalidates _bm25 cache (bm25.py:37-40)
Tests: 756 passed, 4 skipped, 0 failed

Author: MSCodeBase Agent

AGENT_DIARY.md

@@ -223,3 +223,26 @@
 **Guard:** 37 passed (test_write_tools) + 10 passed (health/architecture/index_guard); import OK; ext синхронизирован (cp); live-проверка после Reload Window.
 **verified_from_clean_state:** ✅ yes — live-подтверждено на RUN_ID e3f3aabd7186: codebase(action="index", path="status") → 4842 chunks; codebase(action="write", ...) → modification guard + impact_token.
 
+
+---
+
+## [2026-08-03 23:55] — Сессия: §1.19 Hard Triggers + аудит 29 пунктов + docs sync + commit/push
+
+**Status:** ✅ Done
+**Root Cause:** протокол требовал жёстких триггеров (§1.19), аудит audit.md был неразмечен, README/doc badges устарели (649→747), DEV_DIARY не архивирован, CHANGELOG пуст, unpushed commit.
+**Fix:** (1) §1.19 Hard Triggers в личный AGENTS.md (5 блокираторов «запрещено без»); (2) 29 вердиктов в experiments/audit.md (4✅, 3⚠️, 7❌, 15📝); (3) README/docs/{ru,zh}: badges 747, 48 tools, dates 2026-08-03; (4) onnx off-by-one parents[3]; hub write sub-action dispatch; Py3.14 get_running_loop; (5) verify_diary 3 ложных ❌; (6) DEV_DIARY → ARCHIVED header; (7) commit 8a07c23e + push origin/main.
+**Guard:** pre-commit (verify_diary 25✅/0❌, stale_detector OK); bump_version --check ✅ (3.3.11); live E2E MCP chain verified on RUN_ID e3f3aabd7186 (edit→notify→reindex→search_code 4857 chunks).
+**verified_from_clean_state:** ✅ yes — clean clone+venv+install+pytest (747 passed) + live MCP chain verified.
+
+---
+
+## [2026-08-04] — Спринт: 6 пунктов аудита (5 ✅ Fixed, 1 ❌ Refuted) + docs + commit/push
+
+**Status:** ✅ Done (5/6 FIXED, 1/6 REFUTED)
+**Root Cause:** 6 ❌ P1/P2 пунктов из experiments/audit.md требовали фикса: Heartbeat GetLastError, hardcoded reranker weights, BM25 sync reindex, SQLite schema cols, PYTHONUTF8, shell=True.
+**Fix:** (1) server_factory.py:57-68 SetLastError(0)+GetLastError только при handle==0; (2) settings.py SearchConfig.bm25/dense_weight из env + engine.py:86-89; (3) Item 8 REFUTED — BM25Mixin.reindex() = только инвалидация кэша `_bm25=None` под локом (bm25.py:37-40), блокировки нет; (4) server.py:266-289 PRAGMA table_info для {key,value}/{workspace,data}; (5) zed_config.py:270-273 env[PYTHONUTF8]="1"; (6) install.py _run()→shlex.split+shell=False, step_pip Popen→список, фикс stray `)` (syntax error).
+**Guard:** 756 passed, 4 skipped, 0 failed; AST OK на 7 файлах; shlex.split проверен на PowerShell-команде (4 args, скрипт цел); Ledger §0.1.1 — прошлая запись «Item 8 FIXED via ThreadPoolExecutor» была ложной (di_container.py не менялся) — сверять ledger с git status (§9 pitfall 1).
+**verified_from_clean_state:** ⚠️ no — pytest полный (756 passed) + AST, но verify_clean_state.sh (clone+venv) не гонялся в этой сессии.
+
+---
+

KNOWN_ISSUES.md

@@ -2373,3 +2373,72 @@ Three fixes from the same review:
 **Fix:** все записи < 2026-08-01 перенесены в docs/archive/AGENT_DIARY_2026_07.md (заголовок ARCHIVE — см. A...
 - **Статус:** автоматически синхронизировано
 
+
+## 2026-08-03 23:55 — §1.19 Hard Triggers внедрены в личный AGENTS.md (RESOLVED)
+
+**Symptom:** протокол (§1.15, §1.16, §1.18, §3.5, §0.1.1) сформулирован как «обязан» — агент выполняет шаги, но не думает по ним (нет Phase Zero, Red Team, обобщения, немедленной meta-check, Verification Ledger).
+**Root Cause:** «Обязан» — карта, которую можно не читать. Нужны «рельсы» («запрещено без»).
+**Fix:** §1.19 ЖЁСТКИЕ ТРИГГЕРЫ: 5 блокираторов (Phase Zero, Red Team, grep-обобщение, немедленный META-CHECK, Verification Ledger ≥3 подзадачи). Формат «запрещено без» + Red Team формат `[🔓 RED TEAM] Атака N → Защита/Нет защиты`.
+**Status:** ✅ внедрено в C:\Users\misha\AppData\Roaming\Zed\AGENTS.md (1375→1423 строки). Red Team 5/5 защищено на самой правке.
+
+## 2026-08-03 23:55 — Аудит audit.md: 29 вердикты размечены (PARTIAL — 7 неисправлено, 15 рекомендаций)
+
+**Symptom:** audit.md (2053 строк) содержал 24 пункта + solo-dev 5 без статусов.
+**Root Cause:** аудит накапливался без обновления статусов.
+**Fix:** скрипт .local/patch_audit_status.py вписал 29 вердиктов с File:Line: 4✅ (DI, resolve root, RRF, ack_impact), 3⚠️ (asyncio.Lock, progress cleanup, _distance inconsistency), 7❌ (Heartbeat GetLastError, SearchResultReranker weights, BM25 sync reindex, SQLite schema cols, PYTHONUTF8, shell=True, CodeParser leak), 15📝 рекомендаций (cancellation, OTel, Prometheus, ConfigReloader, chaos, property-based, AgentFriendlyError, ResourceGuard, uninstall, DevModeReloader и др.).
+**Status:** ⚠️ частично — баги 1,2,3,6,8,10,11,12,13,16,17 остаются открытыми; рекомендации 15-29 — новые фичи, не баги.
+
+## 2026-08-03 23:55 — Документация синхронизирована (RESOLVED)
+
+**Symptom:** README/docs badges 649/667 tests (реально 747), tool counts 42/48 (реально 48), dates 2026-07-21, CHANGELOG пуст в корне.
+**Fix:** README.md + docs/{ru,zh}/README.md: badges 747 passed, 48 tools, dates 2026-08-03; ru/zh heading anchors fixed; docs/en/CHANGELOG.md уже актуален (3.3.11, 48 tools).
+**Status:** ✅ docs/en/ru/zh/README.md + bump_version --check ✅ (3.3.11).
+
+## 2026-08-03 23:55 — Commit+push выполнен (RESOLVED)
+
+
+## 2026-08-03 — Аудит audit.md: открытые пункты (29 вердиктов, 6✅, 6❌, 6⚠️, 11📝)
+
+### ✅ ИСПРАВЛЕНО (спринт 2026-08-04, 5/6 FIXED + 1 REFUTED)
+
+| # | Пункт | Файл | Статус | Доказательство |
+|---|-------|------|--------|----------------|
+| 2 | HeartbeatService: нет SetLastError(0) перед OpenProcess, fail-open | src/mcp/server_factory.py:57-68 | ✅ Fixed | SetLastError(0) перед OpenProcess; GetLastError читается только при handle==0; fail-open сохранён. Red Team 5/5. |
+| 6 | SearchResultReranker: hardcoded веса bm25_weight=0.3, dense_weight=0.7 | src/config/settings.py + src/core/search/engine.py:86-89 | ✅ Fixed | SearchConfig.bm25_weight/dense_weight из env (BM25_WEIGHT/DENSE_WEIGHT, дефолты 0.3/0.7); engine читает get_config().search.* |
+| 8 | BM25 reindex callback: синхронный reindex в DebounceBatch | src/core/di_container.py:296-300 | ❌ Refuted | `BM25Mixin.reindex()` (src/core/search/bm25.py:37-40) = только `_bm25 = None` под `_bm25_lock` (O(1), инвалидация кэша). Полный rebuild — лениво при следующем поиске. Блокировки потока НЕТ. Предыдущая запись ledger «FIXED via ThreadPoolExecutor» — ложь, правки не было. |
+| 11 | SQLite schema validation: только таблицы, нет колонок | src/mcp/server.py:266-289 | ✅ Fixed | PRAGMA table_info(scoped_kv_store) → {key, value}; PRAGMA table_info(workspaces) → {workspace, data} |
+| 12 | Encoding: нет PYTHONUTF8=1 | src/utils/zed_config.py:270-273 | ✅ Fixed | env[\"PYTHONUTF8\"] = \"1\" в _make_server_entry (Windows cp1251 → UTF-8) |
+| 13 | install.py: shell=True в subprocess | install.py:254-264, 549-556 | ✅ Fixed | _run() → shlex.split + shell=False; step_pip Popen → список аргументов + shell=False; фикс stray `)` (синтаксическая ошибка) |
+
+### ⚠️ ЧАСТИЧНО (P2 — требуют доработки)
+
+| # | Пункт | Файл | Статус |
+|---|-------|------|--------|
+| 3 | asyncio.Lock создаётся вне event loop (cross-loop risk) | src/core/search/engine.py:91 | 🟡 Partial |
+| 4 | Progress tracking: эвристический cleanup (len > 10, 1ч) | src/mcp/server.py:202-229 | 🟡 Partial |
+| 10 | LanceDB _distance: engine OK, multi_project_searcher использует raw | src/core/multi_project_searcher.py:161-169 | 🟡 Partial |
+| 17 | PropertyGraph: lock есть, _recover_from_wal отсутствует | src/core/graph.py:742-795 | 🟡 Partial |
+| 19 | Rate limiting: только provider-level, нет MCP-level | src/core/rate_limiter.py | 🟡 Partial |
+| 26 | Agent-friendly errors: error_boundary есть, AgentFriendlyError нет | src/mcp/tools/write_tools.py:135 | 🟡 Partial |
+
+### 📝 РЕКОМЕНДАЦИИ (P3/P4 — tech debt / новые фичи)
+
+| # | Пункт | Файл | Статус |
+|---|-------|------|--------|
+| 15 | Cancellation handling: MCP запросы не отменяются | — | 📝 Tech Debt |
+| 16 | Tree-sitter: CodeParser leak (нет close/__del__) | src/core/indexing/parser.py:21 | 📝 Tech Debt |
+| 18 | MCP Progress notifications: не используются notifications/progress | — | 📝 Tech Debt |
+| 20 | OpenTelemetry: нет distributed tracing | — | 📝 Tech Debt |
+| 21 | Prometheus: нет metrics integration | — | 📝 Tech Debt |
+| 22 | Config hot-reload: ConfigReloader отсутствует | — | 📝 Tech Debt |
+| 23 | Chaos-тесты: kill process during indexing | — | 📝 Tech Debt |
+| 24 | Property-based тесты для scoring (RRF, cosine) | — | 📝 Tech Debt |
+| 27 | ResourceGuard: защита от OOM | — | 📝 Tech Debt |
+| 28 | One-command uninstall в install.py | install.py | 📝 Tech Debt |
+| 29 | Hot-reload кода: DevModeReloader отсутствует | — | 📝 Tech Debt |
+
+
+**Symptom:** 19+ изменённых файлов, unpushed commit 5ce0eaa3 (origin/main = ab44b00d).
+**Fix:** commit 8a07c23e (34 files, +2362/-522), pre-commit OK (verify_diary 25✅, stale_detector OK), push origin/main.
+**Status:** ✅ origin/main up to date (HEAD = 8a07c23e).
+

install.py

@@ -252,11 +252,19 @@ def _is_within(path: Path, root: Path) -> bool:
 
 
 def _run(cmd: str, timeout: int = 120, capture: bool = True) -> Optional[subprocess.CompletedProcess]:
-    """subprocess.run с таймаутом и безопасным возвратом None вместо исключения."""
+    """subprocess.run с таймаутом и безопасным возвратом None вместо исключения.
+
+    Использует shell=False со списком аргументов для безопасности.
+    Команда разбивается через shlex.split() для корректной обработки
+    путей с пробелами и спецсимволами.
+    """
+    import shlex
+
     try:
+        args = shlex.split(cmd)
         if capture:
-            return subprocess.run(cmd, shell=True, timeout=timeout, capture_output=True, text=True)
-        return subprocess.run(cmd, shell=True, timeout=timeout)
+            return subprocess.run(args, shell=False, timeout=timeout, capture_output=True, text=True)
+        return subprocess.run(args, shell=False, timeout=timeout)
     except subprocess.TimeoutExpired:
         logger.debug("command timed out after %ss: %s", timeout, cmd[:80])
         return None
@@ -539,8 +547,8 @@ def step_pip(lines, lang):
     _run(f'"{PYTHON_EXE}" -m pip install --upgrade pip', timeout=60)
 
     proc = subprocess.Popen(
-        f'"{PYTHON_EXE}" -m pip install -r "{req}"',
-        shell=True,
+        [PYTHON_EXE, "-m", "pip", "install", "-r", str(req)],
+        shell=False,
         cwd=str(ZED_EXT_DIR),
         stdout=subprocess.PIPE,
         stderr=subprocess.STDOUT,

src/config/settings.py

@@ -138,6 +138,14 @@ class SearchConfig:
     default_search_limit: int = int(os.getenv("DEFAULT_SEARCH_LIMIT", "6"))
     max_search_results: int = int(os.getenv("MAX_SEARCH_RESULTS", "20"))
 
+    # BM25 + Dense weights (SearchResultReranker legacy)
+    bm25_weight: float = field(
+        default_factory=lambda: float(os.getenv("BM25_WEIGHT", "0.3"))
+    )
+    dense_weight: float = field(
+        default_factory=lambda: float(os.getenv("DENSE_WEIGHT", "0.7"))
+    )
+
     # Query expansion
     query_synonyms_enabled: bool = (
         os.getenv("QUERY_SYNONYMS_ENABLED", "true").lower() == "true"

src/core/search/engine.py

@@ -83,7 +83,10 @@ def __init__(self, indexer, embedder):
         self._fts5_lock = threading.Lock()
         self._bm25_df: Any = None
         self._tokenizer_re = re.compile(r"\W+")
-        self._reranker = SearchResultReranker(bm25_weight=0.3, dense_weight=0.7)
+        self._reranker = SearchResultReranker(
+            bm25_weight=get_config().search.bm25_weight,
+            dense_weight=get_config().search.dense_weight
+        )
         self._multi_reranker: Optional[MultiProviderReranker] = None
         self._multi_reranker_initialized: bool = False
         self._multi_reranker_lock = asyncio.Lock()

src/mcp/server.py

@@ -264,7 +264,8 @@ def _cleanup_old_progress():
 
 
 def _check_sqlite_schema_health(conn) -> Optional[str]:
-    """Проверяет, что таблицы scoped_kv_store и workspaces существуют.
+    """Проверяет, что таблицы scoped_kv_store и workspaces существуют
+    и содержат ключевые колонки.
 
     Принимает уже открытое соединение — не вызывает _get_sqlite_connection()
     рекурсивно. Вызывается один раз при старте.
@@ -273,6 +274,8 @@ def _check_sqlite_schema_health(conn) -> Optional[str]:
         return "Zed SQLite DB недоступна — workspace-резолвинг будет degraded"
     try:
         cur = conn.cursor()
+
+        # Проверяем таблицы
         cur.execute(
             "SELECT name FROM sqlite_master WHERE type='table' AND name='scoped_kv_store'"
         )
@@ -283,6 +286,23 @@ def _check_sqlite_schema_health(conn) -> Optional[str]:
         )
         if cur.fetchone() is None:
             return "workspaces не найдена! workspace-резолвинг будет degraded"
+
+        # Проверяем ключевые колонки scoped_kv_store
+        cur.execute("PRAGMA table_info(scoped_kv_store)")
+        kv_columns = {row[1] for row in cur.fetchall()}
+        required_kv = {"key", "value"}
+        missing_kv = required_kv - kv_columns
+        if missing_kv:
+            return f"scoped_kv_store: отсутствуют колонки {missing_kv} — схема устарела"
+
+        # Проверяем ключевые колонки workspaces
+        cur.execute("PRAGMA table_info(workspaces)")
+        ws_columns = {row[1] for row in cur.fetchall()}
+        required_ws = {"workspace", "data"}
+        missing_ws = required_ws - ws_columns
+        if missing_ws:
+            return f"workspaces: отсутствуют колонки {missing_ws} — схема устарела"
+
         return None
     except Exception as e:
         return f"Ошибка проверки схемы SQLite: {e}"

src/mcp/server_factory.py

@@ -55,16 +55,22 @@ def is_parent_alive(self) -> bool:
         try:
             if sys.platform == "win32":
                 import ctypes
+                # Сбрасываем last error перед вызовом OpenProcess
+                # (OpenProcess при успехе не сбрасывает GetLastError)
+                ctypes.windll.kernel32.SetLastError(0)
                 handle = ctypes.windll.kernel32.OpenProcess(0x0400, False, pid)
                 if handle:
                     ctypes.windll.kernel32.CloseHandle(handle)
                     return True
-                return ctypes.windll.kernel32.GetLastError() != 87
+                # Проверяем ошибку ТОЛЬКО при handle == 0
+                err = ctypes.windll.kernel32.GetLastError()
+                return err != 87  # ERROR_INVALID_PARAMETER
             os.kill(pid, 0)
             return True
         except (ProcessLookupError, PermissionError, OSError):
             return False
         except Exception:
+            # fail-open: не убиваем процесс из-за ошибки проверки
             return True
 
     async def _monitor(self) -> None:

src/utils/zed_config.py

@@ -256,7 +256,7 @@ def _make_server_entry(existing: dict | None, executable: str, args: list, ext_d
     """Build the merged server entry, preserving user customizations in `env`.
 
     Authoritative keys (ours) are always set: enabled, command, args,
-    PYTHONPATH, PROJECT_PATH. Optional keys (EMBEDDING_*) keep the user's
+    PYTHONPATH, PROJECT_PATH, PYTHONUTF8. Optional keys (EMBEDDING_*) keep the user's
     value if present. Any other user-added keys are preserved.
     """
     entry = dict(existing) if existing else {}
@@ -268,6 +268,9 @@ def _make_server_entry(existing: dict | None, executable: str, args: list, ext_d
     # Authoritative (ours):
     env["PYTHONPATH"] = str(ext_dir)
     env["PROJECT_PATH"] = "$ZED_WORKTREE_ROOT"
+    # Windows: без PYTHONUTF8=1 Python читает исходники в cp1251,
+    # ломая индексацию русских комментариев и имён (аудит Item 12/5).
+    env["PYTHONUTF8"] = "1"
     # Optional: keep user value if present, else default.
     env.setdefault("EMBEDDING_PROVIDER", "e5_onnx")
     env.setdefault("EMBEDDING_DIMENSION", "768")