Repeat quality audit + сравнение с baseline

[axisrow]

Child of #879.

Воспроизводимый замер прогресса через scripts/code_health.py: установить radon/vulture, прогнать, сравнить с baseline #879 (движение F→E→D, рост MI, число C-файлов, снижение общего LOC). Повторять после каждого рефактор-PR.

Baseline #879: 344 файла, ср. сложность A (3.49), 94% A по MI; 8 функций F (худшие — _collect_channel F(100), chat_stream F(91), CollectionQueue._run_single_worker F(54)); 10 файлов C.

Детальный план выполнения — в комментарии ниже.

[axisrow]

План выполнения

Файл: scripts/code_health.py. Зависит от всех предыдущих; повторять после каждого рефактор-PR.

Baseline (#879)

344 файла, ср. сложность A (3.49), 94% файлов A по MI; 8 функций ранга F (худшие — Collector._collect_channel F(100), ClaudeSdkBackend.chat_stream F(91), CollectionQueue._run_single_worker F(54)); 10 файлов ранга C.

Шаги

1. Установить инструменты: pip install radon vulture (сейчас отсутствуют — code_health.py падает на проверке зависимостей).
2. Прогнать python scripts/code_health.py, сравнить с baseline: движение F→E→D, рост MI, число C-файлов, снижение общего LOC.
3. Vulture: находка legacy_path уйдёт сама после Breaking: убрать legacy compat-поверхности (my-telegram alias + enc:v1 сессии) #881 (route удалён) — whitelist не нужен; убедиться, что high-confidence находок не осталось.
4. Зафиксировать новый baseline комментарием к refactor: снизить риск в самых опасных зонах кода после quality-аудита #879; опционально включить --fail-on F в CI-шаг.

Acceptance

• свежий замер приложен к refactor: снизить риск в самых опасных зонах кода после quality-аудита #879.
• топ-F-функции сдвинулись из ранга F.
• общий объём кода меньше baseline; vulture без high-confidence находок.

[axisrow]

Свежий замер (main после мёржа #881, #882, #884) vs baseline #879

pip install radon vulture + python scripts/code_health.py:

Метрика	Baseline (#879)	Сейчас
Файлов src	344	344 (75 457 строк)
Ср. цикломатика	A (3.49)	A (3.49)
Распределение CC	8×F	A:2873 B:378 C:164 D:26 E:6 F:8
MI-ранги	94% A, 10×C	94% A, 10×C
vulture (≥80%)	2 находки	0 ✅

Что сдвинулось

• vulture: 2 → 0. Обе baseline-находки устранены: progress_offset (удалён в Collector: safety-тесты + удалить мёртвый код (progress_offset) #884), legacy_path (удалён вместе с legacy-роутом в Breaking: убрать legacy compat-поверхности (my-telegram alias + enc:v1 сессии) #881). Whitelist в code_health.py не понадобился.
• Удалены legacy-поверхности (Breaking: убрать legacy compat-поверхности (my-telegram alias + enc:v1 сессии) #881: ~800 строк, my-telegram + enc:v1).

Что ещё НЕ сдвинулось (ожидаемо)

Топ-F-функции на месте, т.к. снижающие их рефакторы отложены/не вмёржены:

• Collector._collect_channel F(100) → Collector: схлопнуть _collect_channel F(100) in-place #885 (deferred, handoff-план готов)
• ClaudeSdkBackend.chat_stream F(91) → Agent #887.1: unify SSE payload + collapse error/stderr in chat_stream #899 (PR refactor(agent): unify SSE frame construction in chat_stream (#899) #907, ещё не вмёржен) + Agent #887.2: extract event-dispatch in chat_stream #900 (deferred)
• CollectionQueue._run_single_worker F(54), _run_telegram_live_checks F(44) и др.

CI-гейт --fail-on F

Пока не включать: на main всё ещё 8 функций ранга F → гейт сразу покраснит CI. Включить после того, как #885/#899/#900 уберут F-функции.

Issue оставляю открытым как повторяющийся замер — следующий прогон после мёржа #899 и реализации #885.

[axisrow]

Замер после #893/#894/#895 (collector) + #899/#900 (agent)

Метрика	Baseline #879	Сейчас
Функций ранга F	8	7
Collector._collect_channel	F(100)	F(73)
ClaudeSdkBackend.chat_stream	F(91)	F(58)
vulture (≥80%)	2	0

Оставшиеся F-функции:

• _collect_channel F(73), chat_stream F(58) — снижены, но всё ещё F (полный выход требует доп. извлечений вне плана Collector: схлопнуть _collect_channel F(100) in-place #885/Slim agent chat_stream F(91) in-place #887).
• CollectionQueue._run_single_worker F(54), _build_collector_health_context F(50), handle_settings_page F(45), GraphBuilder.build F(44), match_message_filter F(42) — не входили в декомпозицию refactor: снизить риск в самых опасных зонах кода после quality-аудита #879; их снижение — новый scope.

--fail-on F по-прежнему не включать (7 F-функций). Issue остаётся открытым как повторяющийся замер.

[axisrow]

Замер после dedup round 2 (#807 follow-up: PR #915/#916/#917/#919 + CI-гард #918)

Прогон на интеграции всех 4 дедуп-PR поверх main. code_health.py (radon+vulture) + jscpd.

jscpd (основной эффект round 2)

Метрика	main (до round 2)	После round 2
Клонов	36	32
Дублированных строк	462 (0.62%)	400 (0.53%)

Убраны настоящие клоны логики: Collector._resolve_stats_entity_or_deactivate (P1), duck-typed геттер аккаунтов (_registry._get_accounts), photo client-acquisition (_acquire_client_and_resolve), settings provider-write guard. Нетто −23 строки исходника.

code_health snapshot (регрессий ранга нет)

Метрика	Прошлый замер (#889)	Сейчас
Функций ранга F	7	7
Ср. цикломатика	—	A (3.48)
Распределение CC	—	A:2882 B:379 C:165 D:27 E:8 F:7
MI-ранги	94% A, 10×C	94% A, 10×C
vulture (≥80%)	0	0

Что НЕ сдвинулось (ожидаемо)

Дедуп убирает дублирование, а не цикломатику — топ-F на месте:

• Collector._collect_channel F(73), ClaudeSdkBackend.chat_stream F(58) — снижение требует доп. извлечений вне scope refactor: устранить дублирование кода (10 кластеров, ~677 строк) #807.
• CollectionQueue._run_single_worker F(54), _build_collector_health_context F(50), handle_settings_page F(45), GraphBuilder.build F(44), match_message_filter F(42) — не входили в декомпозицию refactor: снизить риск в самых опасных зонах кода после quality-аудита #879, это отдельный scope.

CI-гард

Добавлен advisory non-blocking jscpd-шаг (PR #918, --threshold 1, continue-on-error). --fail-on F по-прежнему не включать — на main 7 функций ранга F.

Замер на интеграции PR #915/#916/#917/#919; радон/vulture/jscpd, 2026-06-19.

[axisrow]

Замер после стабилизации тестов #883 (PR #920 + #921), 2026-06-20

Прогон на main (HEAD 5b23caa). code_health.py (radon+vulture) + jscpd. С прошлого замера (dedup round 2) смержены только #920 (герметичная проверка search-счётчика) и #921 (timing-тесты EmbeddedWorker на asyncio.Event) — оба test-only, src/ не затронут. Замер это подтверждает: дельта по всем src-метрикам нулевая, регрессий ранга нет.

Метрика	Прошлый замер (dedup round 2)	Сейчас	Δ
Файлов src / строк	344 / ~75.5k	344 / 75 472	—
Ср. цикломатика	A (3.48)	A (3.48)	—
Распределение CC	A:2882 B:379 C:165 D:27 E:8 F:7	A:2882 B:379 C:165 D:27 E:8 F:7	—
MI-ранги	94% A, 10×C	A:325 B:9 C:10 (94% A, 10×C)	—
vulture (≥80%)	0	0	—
jscpd клонов / строк	32 / 400 (0.53%)	32 / 400 (0.53%)	—

Топ-F (без изменений)

Collector._collect_channel F(73), ClaudeSdkBackend.chat_stream F(58), CollectionQueue._run_single_worker F(54), _build_collector_health_context F(50), handle_settings_page F(45), GraphBuilder.build F(44), match_message_filter F(42).

Эти 7 функций — отдельный scope вне декомпозиции #879 (кроме первых двух, уже частично снижённых). Снижение цикломатики не входило в стабилизацию #883.

Статус зонтичного #879

• Stabilize coverage/timing-sensitive tests #883 закрыт (completed) — timing-флак убран, оба lane зелёные, в т.ч. под coverage.
• Repeat quality audit + сравнение с baseline #889 (этот замер) остаётся открытым как повторяющийся.

CI-гейт

--fail-on F по-прежнему не включать (на main 7 функций ранга F). Следующий разумный триггер для снижения F — отдельный refactor-issue на _run_single_worker / _build_collector_health_context / handle_settings_page (не входили в #879).

Инструменты: radon/vulture/jscpd@5.0.10, 2026-06-20, HEAD 5b23caa.

[axisrow]

Замер после раунда #922 (снижение ранга F), 2026-06-20

Прогон на main (HEAD def2598) после мёржа 5 PR #924–#928. code_health.py (radon cc/mi + vulture) + jscpd.

Главное: F: 7 → 2 ✅

Метрика	До раунда (#889 пред.)	После #922	Δ
Функций ранга F	7	2	−5
Ср. цикломатика	A (3.48)	A (3.46)	↓
Распределение CC	A:2882 B:379 C:165 D:27 E:8 F:7	A:2900 B:392 C:173 D:27 E:8 F:2
MI-ранги	94% A, 10×C	94% A, 10×C	—
vulture (≥80%)	0	0	—
jscpd клонов / строк	32 / 400 (0.53%)	32 / 400 (0.53%)	—
Файлов / строк	344 / 75 472	344 / 75 724	+252*

* рост LOC ожидаем — извлечённые хелперы/докстринги + новый tests/test_pipeline_filters.py (40 safety-кейсов).

Снижены 5 функций (по одному PR на каждую)

Функция	Было	Стало	PR
CollectionQueue._run_single_worker	F(54)	C(17)	#927
_build_collector_health_context	F(50)	C(17)	#924
handle_settings_page	F(45)	A(3)	#925
GraphBuilder.build	F(44)	A(3)	#926
match_message_filter	F(42)	A(3)	#928

Все рефакторы — без изменения внешнего поведения (CLI/Web/Agent/фильтры), подтверждено существующими тестами + новым safety-suite для match_message_filter. Прошли cloud-ревью (@claude / @codex); по #927 находка Claude про «потерю traceback в logger.exception» эмпирически опровергнута (CPython _PyErr_GetTopmostException идёт по цепочке previous_item).

Остаются F:2 — обе вынесены в #923 (deferred)

• Collector._collect_channel F(73) — src/telegram/collector.py
• ClaudeSdkBackend.chat_stream F(58) — src/agent/manager.py

Это самые критичные пути (сбор/стриминг), берутся отдельно и аккуратно. CI-гейт --fail-on F (blocking) включается в #923 после того, как эти две дойдут до ≤ E, т.е. при F:0.

Инструменты: radon/vulture/jscpd@5.0.10, 2026-06-20, HEAD def2598.

[chatgpt-codex-connector]

Summary

• Extracted Claude CLI stderr parsing and SSE status/warning emission into _build_claude_stderr_handler, keeping the callback behavior centralized outside the main streaming method. src/agent/manager.pyL575-L645
• Updated ClaudeSdkBackend.chat_stream to build and pass the extracted stderr handler into ClaudeAgentOptions, reducing inline complexity in the critical streaming path. src/agent/manager.pyL721-L769
• Committed the change on the current branch: 482f310 (refactor(agent): extract Claude stderr handler).
• Created a follow-up PR with title: refactor(agent): extract Claude stderr handler.

Testing

• ✅ ruff check src/ tests/ conftest.py
• ✅ python -m py_compile src/agent/manager.py
• ✅ git diff --check
• ⚠️ pytest tests/ -q -m "not aiosqlite_serial" -n auto (environment limitation: pytest-xdist is unavailable, so -n / --dist=loadfile are unrecognized)
• ⚠️ pytest tests/ -q -m "not aiosqlite_serial" -o addopts='' (environment limitation: repo conftest.py registers pytest_xdist_auto_num_workers, but pytest-xdist is unavailable)
• ⚠️ python scripts/code_health.py --top 5 (environment limitation: radon and vulture are missing)
• ⚠️ python -m pip install radon vulture (environment limitation: package index tunnel returned 403 Forbidden)

View task →

[axisrow]

Замер после #923 (последние 2 горячие F-функции) — F: 0 🎯

Прогон на main (HEAD e6ca79c) после мёржа #929/#930/#931. code_health.py (radon cc/mi + vulture).

Достигнут F:0

Метрика	После #922 (#889 пред.)	После #923	Δ
Функций ранга F	2	0	−2
Ср. цикломатика	A (3.46)	A (3.46)	—
Распределение CC	A:2900 B:392 C:173 D:27 E:8 F:2	A:2902 B:393 C:175 D:32 E:8 F:0
MI-ранги	94% A, 10×C	94% A, 10×C	—
vulture (≥80%)	0	0	—

Снижены 2 горячие функции

Функция	Было	Стало	PR
Collector._collect_channel	F(73)	D(29)	#929
ClaudeSdkBackend.chat_stream	F(58)	D(27)	#930

Обе — в критических путях (сбор/стриминг). Рефакторинг через result/сентинел-протокол (_ResolveOutcome) и isinstance-диспетчеры; поведение сохранено дословно (включая flood-operation метку RESOLVE_USERNAME_OPERATION). Cloud-ревью @claude/@codex пройдено: Codex чисто на обоих; Claude approved #929 с полной таблицей behavioral-эквивалентности, clean #930. Добавлены точечные safety-тесты на хитрые resolve-ветки.

CI-гейт включён (blocking)

#931 добавил blocking-шаг python scripts/code_health.py --fail-on F в .github/workflows/ci.yml + radon/vulture в [dev]. Теперь любая функция, дорастающая до ранга F (CC ≥ 41), краснит CI. Сам шаг прошёл зелёным на F:0.

Итог программы снижения сложности

Совокупно по #922 + #923: 7 функций ранга F → 0. Цепочка quality-рефакторинга #879 → #922 → #923 завершена; дальнейшие регрессии ранга F блокируются автоматически. Этот замер-issue (#889) остаётся открытым как повторяющийся аудит.

Инструменты: radon/vulture, 2026-06-20, HEAD e6ca79c.