[tradein] scraper_kit Group F4d: миграция domclick/* на shared base (_base.py) #2364

New issue

Closed

opened 2026-07-04 00:56:43 +00:00 by lekss361 · 0 comments

lekss361 commented

2026-07-04 00:56:43 +00:00

Owner

Родительский: #2352 (F4). Зависит от #2358 (F4-Foundation, смерджен) — теперь можно мигрировать.

Scope

scraper_kit/providers/domclick/{serp,detail}.py — заменить локальную конструкцию BrowserFetcher(...) на build_browser_fetcher() из providers/_base.py.

⚠️ Caveat (из code-review #2358, обязательно проверить явно)

domclick/serp.py сейчас конструирует BrowserFetcher только с source+endpoint (implicitly proxy_provider=None, use_pool=False через class defaults) — тот же caveat что и F4a (avito). При переключении на build_browser_fetcher(config, source) — use_pool начнёт читать config.use_proxy_pool_browser вместо hardcoded False. Реальное изменение поведения ЕСЛИ этот флаг когда-либо True в окружении domclick. Проверить явно ДО миграции.

DoD

Все конструкции BrowserFetcher в domclick/{serp,detail}.py используют shared base
Явно проверен и задокументирован use_pool-caveat выше
Существующий parity/regression test suite для domclick остаётся зелёным
Дублирование снижено (замерить строк до/после)

Не пересекается по файлам с F4a/F4b/F4c — можно параллельно.

Refs #2352, #2358, #2309

Родительский: #2352 (F4). Зависит от #2358 (F4-Foundation, смерджен) — теперь можно мигрировать. ## Scope `scraper_kit/providers/domclick/{serp,detail}.py` — заменить локальную конструкцию `BrowserFetcher(...)` на `build_browser_fetcher()` из `providers/_base.py`. ## ⚠️ Caveat (из code-review #2358, обязательно проверить явно) `domclick/serp.py` сейчас конструирует `BrowserFetcher` только с `source`+`endpoint` (implicitly `proxy_provider=None, use_pool=False` через class defaults) — тот же caveat что и F4a (avito). При переключении на `build_browser_fetcher(config, source)` — `use_pool` начнёт читать `config.use_proxy_pool_browser` вместо hardcoded `False`. Реальное изменение поведения ЕСЛИ этот флаг когда-либо `True` в окружении domclick. **Проверить явно** ДО миграции. ## DoD - [ ] Все конструкции `BrowserFetcher` в domclick/{serp,detail}.py используют shared base - [ ] Явно проверен и задокументирован use_pool-caveat выше - [ ] Существующий parity/regression test suite для domclick остаётся зелёным - [ ] Дублирование снижено (замерить строк до/после) Не пересекается по файлам с F4a/F4b/F4c — можно параллельно. Refs #2352, #2358, #2309