fix(tradein/scraper-kit): проброс config= в avito build_warmed_session (#2330) #2354

Merged

lekss361 merged 2 commits from fix/tradein-avito-warmed-session-config into main

2026-07-04 00:04:47 +00:00

lekss361 commented

2026-07-04 00:01:53 +00:00

Owner

Summary

Group F2 (эпик #2277, scraper_kit migration). Фикс известного бага #2330: build_warmed_session() в scraper_kit/providers/avito/detail.py вообще не принимала параметр config — в отличие от fetch_detail() в том же файле — и потому не могла получить прокси ни при каких условиях.

build_warmed_session(config: ScraperConfig | None = None), проброшен в _build_detail_session(config) тем же способом, каким это делает fetch_detail.
Regression-тест test_avito_detail_kit_warmed_session_config.py (3 теста) — доказывает разницу через реальный (не замоканный) curl_cffi.AsyncSession.proxies, не тривиальный "не падает".
Прод-поведение НЕ меняется этим PR — единственный текущий caller (app/tasks/avito_detail_backfill.py) всё ещё импортирует build_warmed_session из legacy-модуля (намеренно, issue #2310). Это advance-prep для будущей миграции call-site.
Докстринги (в фиксе + в смежном test_scraper_kit_group_c_backfill_kit_parity.py) поправлены после code-review — убрана неточная формулировка "PROD DEFAULT", уточнено что прод-путь сегодня всё ещё на legacy.

Review

code-reviewer — ✅ APPROVE (2 minor docstring-неточности, обе исправлены в этом же PR).

Test plan

3 новых regression-теста + relevant test files: 14/14 passed
Regression доказан вручную (stash фикса → 2/3 падают с ожидаемой TypeError, restore → зелёные)
ruff check/ruff format — clean

Closes #2330

## Summary Group F2 (эпик #2277, scraper_kit migration). Фикс известного бага #2330: `build_warmed_session()` в `scraper_kit/providers/avito/detail.py` вообще не принимала параметр `config` — в отличие от `fetch_detail()` в том же файле — и потому не могла получить прокси ни при каких условиях. - `build_warmed_session(config: ScraperConfig | None = None)`, проброшен в `_build_detail_session(config)` тем же способом, каким это делает `fetch_detail`. - Regression-тест `test_avito_detail_kit_warmed_session_config.py` (3 теста) — доказывает разницу через реальный (не замоканный) `curl_cffi.AsyncSession.proxies`, не тривиальный "не падает". - **Прод-поведение НЕ меняется этим PR** — единственный текущий caller (`app/tasks/avito_detail_backfill.py`) всё ещё импортирует `build_warmed_session` из legacy-модуля (намеренно, issue #2310). Это advance-prep для будущей миграции call-site. - Докстринги (в фиксе + в смежном `test_scraper_kit_group_c_backfill_kit_parity.py`) поправлены после code-review — убрана неточная формулировка "PROD DEFAULT", уточнено что прод-путь сегодня всё ещё на legacy. ## Review `code-reviewer` — ✅ APPROVE (2 minor docstring-неточности, обе исправлены в этом же PR). ## Test plan - [x] 3 новых regression-теста + relevant test files: 14/14 passed - [x] Regression доказан вручную (stash фикса → 2/3 падают с ожидаемой TypeError, restore → зелёные) - [x] `ruff check`/`ruff format` — clean Closes #2330

lekss361 added 2 commits 2026-07-04 00:01:54 +00:00

fix(tradein/scraper-kit): проброс config= в avito build_warmed_session (#2330 ) fea054d24f

fix(tradein/scraper-kit): точность докстрингов про прод-путь build_warmed_session

CI Trade-In / changes (pull_request) Successful in 10s

Details

CI / changes (pull_request) Successful in 9s

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

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

CI Trade-In / backend-tests (pull_request) Successful in 1m40s

Details

ecec9ff6d7

code-reviewer отметил: формулировка "PROD DEFAULT" в докстрингах могла ввести в
заблуждение — прод-путь сегодня всё ещё легаси (avito_detail_backfill.py
импортирует build_warmed_session из legacy-модуля), этот фикс — advance-prep,
не изменение живого прод-поведения.