[tradein] admin.py: scrape_cian_newbuilding awaits sync save_newbuilding_enrichment — TypeError at runtime #2323

New issue

Closed

opened 2026-07-03 21:27:56 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-07-03 21:27:56 +00:00

Owner

Найдено при миграции Group A (#2305, эпик #2277) — pre-existing баг, НЕ связан с scraper_kit
миграцией, не входит в scope того issue, заведён отдельно для трекинга.

Баг

app/api/v1/admin.py's scrape_cian_newbuilding debug-эндпоинт делает
await save_newbuilding_enrichment(...), но эта функция — def, не async def (подтверждено
и в legacy, и в kit-версии). Вызов упадёт с TypeError: object ... can't be used in 'await' expression
при первом реальном обращении к этому debug-эндпоинту.

Приоритет

Низкий — debug/manual-trigger эндпоинт, не боевой путь (не задета продуктовая функциональность),
но реальный краш при использовании. Фикс тривиален: убрать await либо сделать функцию async
(смотря что консистентнее с соседними save_*_enrichment).

Найдено при миграции Group A (#2305, эпик #2277) — pre-existing баг, НЕ связан с scraper_kit миграцией, не входит в scope того issue, заведён отдельно для трекинга. ## Баг `app/api/v1/admin.py`'s `scrape_cian_newbuilding` debug-эндпоинт делает `await save_newbuilding_enrichment(...)`, но эта функция — `def`, не `async def` (подтверждено и в legacy, и в kit-версии). Вызов упадёт с `TypeError: object ... can't be used in 'await' expression` при первом реальном обращении к этому debug-эндпоинту. ## Приоритет Низкий — debug/manual-trigger эндпоинт, не боевой путь (не задета продуктовая функциональность), но реальный краш при использовании. Фикс тривиален: убрать `await` либо сделать функцию async (смотря что консистентнее с соседними save_*_enrichment).

lekss361 referenced this issue

2026-07-03 21:36:28 +00:00

feat(tradein/scraper-kit): migrate admin debug endpoints + domclick ingest to kit (#2305) #2326

lekss361 added the

labels 2026-07-04 10:23:46 +00:00

lekss361 commented

2026-07-04 14:01:25 +00:00

Author

Owner

Пофикшено попутно в PR #2414 (#2397 Part D4, смерджен 2026-07-04) — тот же call-site (admin.py's scrape_cian_newbuilding эндпоинт) трогался при миграции на kit cian.newbuilding, await перед sync save_newbuilding_enrichment убран. Deep/batch-review подтвердил фикс. Closing as resolved.

Пофикшено попутно в PR #2414 (#2397 Part D4, смерджен 2026-07-04) — тот же call-site (`admin.py`'s `scrape_cian_newbuilding` эндпоинт) трогался при миграции на kit `cian.newbuilding`, `await` перед sync `save_newbuilding_enrichment` убран. Deep/batch-review подтвердил фикс. Closing as resolved.

lekss361 closed this issue

2026-07-04 14:01:26 +00:00