fix(avito): post-merge polish — unify fetch_mode guard + empty-render guard (#1811) #1909

Merged

bot-backend merged 1 commit from fix/avito-serp-polish-1811 into main

2026-06-26 07:44:22 +00:00

Author	SHA1	Message	Date
Light1YT	9b9cc99c11	fix(avito): post-merge polish — unify fetch_mode guard + empty-render guard (#1811 ) All checks were successful CI / changes (pull_request) Successful in 6s Details CI / backend-tests (pull_request) Has been skipped Details CI / frontend-tests (pull_request) Has been skipped Details CI / openapi-codegen-check (pull_request) Has been skipped Details Косметические минор'ы из ревью PR #1809 (avito SERP → BrowserFetcher), функционально без изменений: - __aenter__: settings.scraper_fetch_mode != "curl_cffi" → == "browser" (унификация с scrape_pipeline.py и avito_detail_backfill.py; consistency при гипотетическом 3-м значении режима). - _fetch_serp_html_browser: явный guard `if not html: return None` — пустой рендер (таймаут sidecar'а) больше не утекает как «валидный» HTML; caller (fetch_around) трактует None как конец пагинации. - scrape_pipeline.py: logger.debug на shared-browser sweep-путях — cffi-fallback пропускается (by design), логируем для observability при firewall. - test_avito_browser_mode.py: переформулирован вводящий в заблуждение коммент про _cffi is None (direct _browser assignment bypasses __aenter__). Refs #1811	2026-06-26 12:20:13 +05:00

Author

SHA1

Message

Date

Light1YT

9b9cc99c11

fix(avito): post-merge polish — unify fetch_mode guard + empty-render guard (#1811 )

CI / changes (pull_request) Successful in 6s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

Косметические минор'ы из ревью PR #1809 (avito SERP → BrowserFetcher),
функционально без изменений:

- __aenter__: settings.scraper_fetch_mode != "curl_cffi" → == "browser"
  (унификация с scrape_pipeline.py и avito_detail_backfill.py; consistency
  при гипотетическом 3-м значении режима).
- _fetch_serp_html_browser: явный guard `if not html: return None` — пустой
  рендер (таймаут sidecar'а) больше не утекает как «валидный» HTML; caller
  (fetch_around) трактует None как конец пагинации.
- scrape_pipeline.py: logger.debug на shared-browser sweep-путях — cffi-fallback
  пропускается (by design), логируем для observability при firewall.
- test_avito_browser_mode.py: переформулирован вводящий в заблуждение коммент
  про _cffi is None (direct _browser assignment bypasses __aenter__).

Refs #1811

2026-06-26 12:20:13 +05:00