Merge pull request 'fix(tradein/estimator): резолвить target-город коридора из payload.address (C2 hotfix)' (#2490) from fix/tradein-oblast-c2fix-corridor-city into main

C2 city-scope коридора был NO-OP для не-ЕКБ: estimate_quality резолвил город из geo.full_address, но геокодер РОНЯЕТ город для не-ЕКБ («Ленина, 1» вместо «Нижний Тагил, Ленина, 1») → target_city=None → коридор либо unscoped, либо None. Verified на проде: Н.Тагил corridor=None.

Фикс: резолвить город из payload.address (город есть) с fallback на canonical/geo; коридор матчить тоже по payload.address. GET: row.address приоритетнее canonical.

Verified прод-диагностикой: corridor(address=payload.address, city=resolve) → Н.Тагил=9, ЕКБ=15 (было: Н.Тагил=None). EKB regression-гейт байт-зелёный. geocoder full_address city-strip (display «Ленина,1») → cosmetic, E.

tradein-mvp/backend/app/api/v1/trade_in.py

@@ -284,10 +284,13 @@ def get_estimate(
         else None
     )
 
-    # (oblast C2): city-scope корридора на GET-rehydrate тоже — тот же
-    # resolver, что и на POST (estimate_quality), над персистированным адресом
-    # (canonical_address приоритетнее сырого row.address, если DaData отработал).
-    target_city = _resolve_target_city(row.canonical_address or row.address)
+    # (oblast C2): city-scope корридора на GET-rehydrate. NB: row.address здесь =
+    # geo.full_address (персистится на estimate-time), который РОНЯЕТ город для
+    # не-ЕКБ («Ленина, 1») → target_city=None → corridor unscoped/None для не-ЕКБ
+    # (KNOWN LIMITATION: POST-path чинит через payload.address; GET получит паритет
+    # когда raw payload.address начнёт персиститься — follow-up). Для ЕКБ ок; reorder
+    # ниже безвреден (оба source city-stripped для не-ЕКБ, оба екб для ЕКБ).
+    target_city = _resolve_target_city(row.address) or _resolve_target_city(row.canonical_address)
     dkp_raw = _fetch_dkp_corridor(
         db, address=row.address, rooms=row.rooms, area=area_f, city=target_city
     )

tradein-mvp/backend/app/services/estimator.py

@@ -3207,13 +3207,18 @@ async def estimate_quality(
     # (oblast C2): city-scope корридора — та же precedence, что и для
     # match_house_readonly выше (dadata.canonical_address приоритетнее сырого
     # geo.full_address, если DaData отработал).
-    target_city = _resolve_target_city(
+    # payload.address (пользовательский ввод) — САМЫЙ надёжный источник города:
+    # geocoder.full_address/canonical часто роняют город для не-ЕКБ («Ленина, 1»
+    # вместо «Нижний Тагил, Ленина, 1») → без этого city-scope коридора (C2) был
+    # бы no-op для всей области (verified прод: tc=None → corridor=None). Fallback
+    # на canonical/geo для bare-адресов без города.
+    target_city = _resolve_target_city(payload.address) or _resolve_target_city(
         (dadata.canonical_address if dadata else None) or geo.full_address
     )
     dkp_raw = await asyncio.to_thread(
         _fetch_dkp_corridor,
         db,
-        address=geo.full_address,
+        address=payload.address,
         rooms=payload.rooms,
         area=payload.area_m2,
         city=target_city,