fix: stop failing closed on always-empty top views, close orphaned tabs (#22, #9)

[axisrow]

Что и почему

Fixes #22, fixes #9. Порядок — #22 первым, т.к. он блокировал живую проверку самого себя (и #9): пока первый же вид (top_popular) обрывал прогон, dynamics/regions не собирались никогда.

#22 — fail-closed на всегда-пустом top_popular

_is_untrustworthy_empty_export считала пустой CSV недоверенным для top_popular/top_related/dynamics разом, на основании структурной посылки: раз _select_view доказал TABLE_ROW_SELECTOR.length > 0 перед кликом «Скачать», пустой экспорт для любого табличного вида — невозможное состояние.

Issue #11 эту посылку опровергает фактом: top_popular/top_related пусты у Вордстата всегда, это подтверждено вручную (клик по ссылке экспорта, мимо кода). top_popular идёт первым в VIEW_SELECTORS — значит каждый полный прогон падал на первом же виде, и инструмент не мог закончить сбор целиком.

Решение (направление из issue: пометить вид несобранным / развести правила по видам):

• предикат теперь fail-closed только для dynamics (эмпирически подтверждено — 24 строки в трёх живых прогонах top_popular/top_related возвращают 0 строк данных при --keep-raw, хотя интерфейс показывает данные #11, PR docs: Issue #6 phase 1 live granularity research #20);
• пустые top_popular/top_related записываются как обычный ExportSummary с row_count: 0 — честно, не как «missing».

Чтобы не потерять смысл issue #16 (status: complete при нулевых данных не должен молча проезжать), CollectionManifest получил вычисляемое поле empty_views (виды с row_count == 0), и status теперь "incomplete", если непусто missing_views или empty_views. Так что манифест с двумя нулевыми топами виден как неполный, а не выдаётся за успех — закрывает и #16 для этих двух видов, отдельно ничего лишнего не трогал.

#9 — осиротевшие вкладки

collect_many создавала новую вкладку (session.new_page()) на каждый вызов CLI и никогда её не закрывала — keep_alive=True защищает браузер пользователя, но заодно не даёт session.stop() закрыть и нашу собственную вкладку.

Теперь вкладка закрывается через session.close_page(page) в том же finally, до session.stop() (CDP-хендл может быть непригоден после остановки сессии), с той же защитой try/except, что и session.stop() — сбой закрытия не выбрасывает уже собранные results/failures. page = None объявлена до try, чтобы finally не пытался закрыть вкладку, если session.start() упал раньше её создания.

Проверка — живой прогон, CDP http://127.0.0.1:9223

collector.py не покрыт юнит-тестами by design; зелёный pytest — не доказательство. Проверено вручную:

• Одна фраза, полный прогон дошёл до конца: dynamics — 24 строки, regions — 788 строк. Манифест:

  "status": "incomplete",
  "missing_views": [],
  "empty_views": ["top_popular", "top_related"]
  

  (top_popular/top_related — row_count: 0, честно видно, не complete).
• --granularity daily: dynamics_daily.parquet — 59 строк, собралось.
• Батч из двух фраз: одна вкладка на весь батч (4 скачивания, не 8), обе фразы собраны.
• Путь с ошибкой (заведомо невалидный регион, после создания вкладки): PhraseFailure дошёл до вывода (Собрано 0 из 1), CLI вернул exit code 1 без traceback, вкладка не осталась.
• Счётчик вкладок через curl -s http://127.0.0.1:9223/json/list: 13 → 13 на каждом из прогонов выше (single run, daily run, batch run). Путь с ошибкой (одиночная фраза) — тоже 13 → 13.
• Chrome и остальные вкладки/сессия пользователя не тронуты (allowed_domains не менялся, порт 9222 не трогался).
• 11 старых осиротевших вкладок (висели до начала этой сессии, все с одинаковым URL-паттерном прошлых прогонов) закрыты в конце после проверки, что среди них нет пользовательских.

Тесты

ruff check .        → All checks passed!
pytest -q           → 151 passed in ~0.67-0.95s (было 143 passed за 0.68s на main — рост числа тестов, не деградация скорости, issue #23 не задет)

tests/test_collector_view.py: параметризованный тест на все три вида заменён — _is_untrustworthy_empty_export теперь fail-closed только для DYNAMICS; добавлены отдельные тесты, подтверждающие, что пустой top_popular/top_related этим гейтом не флагуется.

Что не расширял

Issue #24 (fail-open месячной динамики) не трогал — следующая отдельная задача, как и указано в брифе.

[axisrow]

🔍 Local review (cycle 1) — round 7895f235-f497-401c-bf4f-491e955174d2

Reviewed locally (/review + Codex companion), no bots pinged.

Verdict	Reviewer	Finding	Location
—	claude	No findings — the diff is internally consistent across the narrowed empty-export gate, the page-close guard, and the new empty_views computed field.	—
SKIP	codex	The manifest completeness flag can never read complete on a live run because two Wordstat reports are always empty by design, so status conveys no per-run signal even though both reports are fully collected and written to disk with the correct empty schema — a deliberate, already-tested tradeoff from this same PR to preserve an earlier guarantee against silently masking zero-row data as success.	src/wordstat/models.py:132-140

No FIX verdicts this round. Both configured reviewers (claude, codex) answered.

[axisrow]

📋 Review summary — all cycles

Cycle	Reviewer	Finding	Verdict	Resolution
1	codex	Manifest status can never read complete while two Wordstat reports are always empty by design; this is a deliberate, already-tested tradeoff preserving an earlier zero-row-safety guarantee, not a data-loss bug.	SKIP	Left as-is — deliberate design tradeoff documented in PR body
1	claude	No findings	—	—

Totals: 0 FIX, 1 SKIP, 0 UNVERIFIED. Reviewed locally (built-in /review + Codex companion, no bots pinged). Lint (ruff check .) and full test suite (pytest -q, 151 passed) are green at the reviewed head.