refactor(tradein): консолидировать corridor/street-deals city-scope на _resolve_target_city (п.3) #2501

Merged

bot-backend merged 1 commit from fix/tradein-corridor-dedup-scope into main

2026-07-12 20:31:06 +00:00

bot-backend commented

2026-07-12 20:18:25 +00:00

Collaborator

п.3 — консолидация corridor/street-deals city-scope (cleanup двойного #2488↔#2489)

Твой #2488 (_resolve_target_city + LOWER(d.city)=:target_city) смёржен НА ВЕРХ моего #2489 (substring) → в _fetch_dkp_corridor был двойной city-фильтр (benign, но избыточный + латентный ё/е edge).

_fetch_dkp_corridor: убран мой substring, оставлен твой _resolve_target_city (словарь ~30 городов обл.66 вкл. ЕКБ + все sweep-города; city=None → фильтр не применяется — прежнее #2488-поведение). Убран unused :address bind.
get_street_deals: substring заменён на тот же _resolve_target_city паттерн (консистентность; #2488 не трогал street-deals).

Live-verified: Ленина 2к ЕКБ → median 118052, 1 город (байт-в-байт как substring). Regression-гейт байт-зелёный (коридор заморожен в фикстуре), 53 теста, ruff clean.

## п.3 — консолидация corridor/street-deals city-scope (cleanup двойного #2488↔#2489) Твой #2488 (`_resolve_target_city` + `LOWER(d.city)=:target_city`) смёржен НА ВЕРХ моего #2489 (substring) → в `_fetch_dkp_corridor` был **двойной city-фильтр** (benign, но избыточный + латентный ё/е edge). - `_fetch_dkp_corridor`: убран мой substring, оставлен твой `_resolve_target_city` (словарь ~30 городов обл.66 вкл. ЕКБ + все sweep-города; `city=None` → фильтр не применяется — прежнее #2488-поведение). Убран unused `:address` bind. - `get_street_deals`: substring заменён на тот же `_resolve_target_city` паттерн (консистентность; #2488 не трогал street-deals). Live-verified: Ленина 2к ЕКБ → **median 118052, 1 город** (байт-в-байт как substring). Regression-гейт байт-зелёный (коридор заморожен в фикстуре), 53 теста, ruff clean.

Rows
Columns