stats["skipped"] был объявлен в контракте, возвращался и печатался в лог — и
никогда не увеличивался. Все исходы сваливались в failed: и WAF-блок, и битый
разбор, и «UPDATE затронул 0 строк». По такому счётчику нельзя отличить временную
помеху от настоящей регрессии разбора, а сам он всегда показывал ноль.
Законный источник пропуска в коде БЫЛ: obj_id берутся из БД, но снимок мог
смениться между выборкой и UPDATE'ом — тогда строки (obj_id, snapshot_date) уже
нет. Ветка с логом «not in DB?» существовала и возвращала False, попадая в failed.
Третье состояние сделано через None, а не новым Literal, намеренно: прежние
True/False сохраняют смысл, поэтому существующие вызывающие и тесты не
переписываются. Проверено прогоном — 531 passed, включая
test_domrf_catalog_object_browsersession_throttle (fake возвращает True) и тесты
предохранителя из #2971, ни один не тронут. Это и был довод против Literal:
менять чужой тест ради своей правки — плохая цена за красоту сигнатуры.
Против origin/main:
skipped=0, failed=3 вместо skipped=2, failed=1 → падает
сумма счётчиков сходится с processed — контроль, зелёный с обеих сторон
True/False сохраняют смысл — контроль, зелёный с обеих сторон
Первый контроль ловит «починку», при которой пропуск считался бы дважды или
терялся; второй фиксирует ровно то свойство, ради которого выбран None.
Прогоны: tests/services/scrapers + tests/workers — 531 passed rc=0.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
