fix(tradein/geocoder): не подставлять Екатеринбург молча — явный city_hint (#2576) #2580

Merged

lekss361 merged 2 commits from fix/tradein-geocoder-city-default into main

2026-07-31 15:23:35 +00:00

lekss361 commented

2026-07-31 13:44:04 +00:00

Owner

Эпик #2576, этап B. Самый опасный из тихих дефектов. Геокодер молча добавлял префикс "Екатеринбург, {адрес}", если в тексте не было маркера города — а фронт город не передаёт вовсе. Житель Нижнего Тагила, вводя «Ленина, 1», получал координаты екатеринбургской улицы и оценку по чужому рынку с обычной уверенностью. Прецедент уже был: Тагил оценивался в 187 500 ₽/м² вместо реальных ~84 000.

Решение: приоритет разрешения города — маркер в самом адресе → явный city_hint → ничего (вместо молчаливого Екатеринбурга). Запрос уходит без префикса города, а географический bias смещается с EKB_BBOX на OBLAST66_VIEWBOX (вся область). Строку «Свердловская область, …» не подставляем: текстовому поиску она ничего не даёт (bbox уже задан), зато не порождает мусор вида «Екатеринбург, Нижний Тагил, Ленина 10».

Честный признак неопределённости: GeocodeResult.city_ambiguous — «город определил провайдер, а не пользователь», проставляется независимо от того, какой тир дал результат, и пробрасывается в ответ оценки как target_city_ambiguous. Поле эфемерное, миграции не требует.

API: city_hint — опциональный query-параметр на /lookup и /suggest, плюс поле во входе оценки. Контракт не сломан.

ЕКБ не деградирует: при явном маркере города или переданном city_hint поведение прежнее.

Кэш: ключ становится "{адрес}|city={hint}" только когда hint передан, иначе прежний — обратная совместимость сохранена. NB: пока фронт не шлёт city_hint, одинаковые адреса разных городов по-прежнему неразличимы по ключу — это структурное ограничение, снимается фронтовой частью. Рекомендована точечная чистка geocode_cache: записи provider IN ('yandex','nominatim') с екатеринбургским full_address, где исходный адрес не содержал маркера города (аналог чистки 614 строк после #2504). Чистку не выполнял — отдельным шагом.

Тесты: +23 (test_geocoder_city_hint.py), на старом коде падают (ImportError: _cache_key). Полный прогон: 2849 passed, 1 pre-existing fail. ruff чисто.

Следующий шаг (отдельный PR): фронт должен передавать город — до этого city_ambiguous будет истинным почти всегда для адресов без города в тексте. Это честно, но UI пока не использует флаг.

Эпик #2576, этап B. **Самый опасный из тихих дефектов.** Геокодер молча добавлял префикс `"Екатеринбург, {адрес}"`, если в тексте не было маркера города — а фронт город не передаёт вовсе. Житель Нижнего Тагила, вводя «Ленина, 1», получал координаты екатеринбургской улицы и оценку по чужому рынку с обычной уверенностью. Прецедент уже был: Тагил оценивался в 187 500 ₽/м² вместо реальных ~84 000. **Решение:** приоритет разрешения города — маркер в самом адресе → явный `city_hint` → **ничего** (вместо молчаливого Екатеринбурга). Запрос уходит без префикса города, а географический bias смещается с `EKB_BBOX` на `OBLAST66_VIEWBOX` (вся область). Строку «Свердловская область, …» не подставляем: текстовому поиску она ничего не даёт (bbox уже задан), зато не порождает мусор вида «Екатеринбург, Нижний Тагил, Ленина 10». **Честный признак неопределённости:** `GeocodeResult.city_ambiguous` — «город определил провайдер, а не пользователь», проставляется независимо от того, какой тир дал результат, и пробрасывается в ответ оценки как `target_city_ambiguous`. Поле эфемерное, миграции не требует. **API:** `city_hint` — опциональный query-параметр на `/lookup` и `/suggest`, плюс поле во входе оценки. Контракт не сломан. **ЕКБ не деградирует:** при явном маркере города или переданном `city_hint` поведение прежнее. **Кэш:** ключ становится `"{адрес}|city={hint}"` только когда hint передан, иначе прежний — обратная совместимость сохранена. NB: пока фронт не шлёт `city_hint`, одинаковые адреса разных городов по-прежнему неразличимы по ключу — это структурное ограничение, снимается фронтовой частью. Рекомендована точечная чистка `geocode_cache`: записи `provider IN ('yandex','nominatim')` с екатеринбургским `full_address`, где исходный адрес не содержал маркера города (аналог чистки 614 строк после #2504). **Чистку не выполнял** — отдельным шагом. Тесты: +23 (`test_geocoder_city_hint.py`), на старом коде падают (`ImportError: _cache_key`). Полный прогон: 2849 passed, 1 pre-existing fail. ruff чисто. **Следующий шаг (отдельный PR):** фронт должен передавать город — до этого `city_ambiguous` будет истинным почти всегда для адресов без города в тексте. Это честно, но UI пока не использует флаг.

Rows
Columns