domclick_detail_backfill: обрыв «пул пуст» бросает попытку неучтённой — attempted ≠ enriched+failed+blocked #3332

New issue

Closed

opened 2026-09-02 11:42:07 +00:00 by bot-backend · 0 comments

bot-backend commented

2026-09-02 11:42:07 +00:00

Collaborator

Отпочковано из #3319 (смежный low-пункт, не влез в бюджет PR #3330).

tradein-mvp/packages/scraper-kit/src/scraper_kit/providers/domclick_detail_backfill.py:444/473-482: обрыв «пул пуст» бросает текущую попытку неучтённой — тождество счётчиков attempted = enriched + failed + blocked (+ прочие исходы) нарушается. Прод-факт на 02.09: 5 прогонов с diff=1.

Лечение

Каждый выход из цикла обязан классифицировать текущую попытку в один из счётчиков (для «пул пуст» — отдельный исход или blocked с причиной). Тождество проверять на всех выходах.

Приёмка

Тест по значению: обрыв «пул пуст» на середине попытки → attempted == сумма исходов;
Прод-SELECT после следующего прогона: diff=0 на новых прогонах (у старых остаётся);
Фальсификация: возврат неучтённой попытки красит тест по значению.

Refs #3319, #3330, #3288 (честность счётчиков — та же тема).

Отпочковано из #3319 (смежный low-пункт, не влез в бюджет PR #3330). `tradein-mvp/packages/scraper-kit/src/scraper_kit/providers/domclick_detail_backfill.py:444/473-482`: обрыв «пул пуст» бросает текущую попытку неучтённой — тождество счётчиков `attempted = enriched + failed + blocked (+ прочие исходы)` нарушается. Прод-факт на 02.09: 5 прогонов с diff=1. ## Лечение Каждый выход из цикла обязан классифицировать текущую попытку в один из счётчиков (для «пул пуст» — отдельный исход или blocked с причиной). Тождество проверять на всех выходах. ## Приёмка - [ ] Тест по значению: обрыв «пул пуст» на середине попытки → attempted == сумма исходов; - [ ] Прод-SELECT после следующего прогона: diff=0 на новых прогонах (у старых остаётся); - [ ] Фальсификация: возврат неучтённой попытки красит тест по значению. Refs #3319, #3330, #3288 (честность счётчиков — та же тема).