feat(site-finder): §1 POI-точки карты показывают максимум информации #2110

Merged

bot-backend merged 1 commit from feat/poi-popup-max-info into main

2026-06-30 12:42:14 +00:00

bot-backend commented

2026-06-30 12:26:54 +00:00

Collaborator

Что

POI-маркеры карты §1 («Объект») раньше показывали в попапе только категорию · название · расстояние · дату правки OSM. При этом таблица osm_poi_ekb.tags (jsonb) уже хранит адрес, оператора/бренд, часы работы, сайт и телефон — но эти данные не доходили до фронтенда.

Теперь каждая точка показывает максимум доступной информации (по прямому запросу: «все точки должны показывать максимум информации»).

Backend (`app/api/v1/parcels.py`)

_clean_tag / _poi_detail_from_tags — вытаскивают человекочитаемые поля из tags:
- адрес = addr:street + addr:housenumber
- оператор = operator → fallback brand
- часы = opening_hours
- сайт = contact:website → website → url
- телефон = contact:phone → phone
- мусор отбрасывается (yes/no/fixme/?/пустое/не-строки); всегда фиксированный набор из 5 ключей с None при отсутствии.
В SQL POI-выборки добавлен столбец tags; результат спредится в score_breakdown.
Контракт API не меняется: score_breakdown: dict[str, Any] (opaque map) — app.openapi() не трогается, codegen не нужен. Проверено reviewer'ом.

Frontend

ParcelAnalysisPoi: добавлены опциональные address/operator/opening_hours/website/phone.
SiteMap попап POI рендерит эти поля:
- сайт — через safeHttpUrl (allowlist http(s)://, иначе показываем как текст, не ссылку — XSS-защита по frontend.md);
- телефон — tel: с санитайзингом ([^\d+] вырезается).

Тесты

tests/api/v1/test_poi_detail_from_tags.py — полный набор тегов, приоритет contact:*, brand-fallback, фиксированный keyset, отбрасывание мусора (5/5 проходят).

Проверка

Бэкенд-юнит-тесты ✅, ruff/ruff-format ✅
Frontend tsc --noEmit ✅, eslint ✅, prettier ✅
Подтверждено по prod-БД: POI в радиусе 1 км от тестового участка 66:41:0702017:131 реально имеют эти теги (аптеки «Вита Экспресс»/«Живика»/«Классика» с часами и адресами «улица Вилонова, 14А»).
Pre-push review: ✅ APPROVE (без блокеров; два минорных cleanup'а сложены в этот же коммит).

После merge+deploy — скриншот-верификация попапа на проде.

## Что POI-маркеры карты §1 («Объект») раньше показывали в попапе только **категорию · название · расстояние · дату правки OSM**. При этом таблица `osm_poi_ekb.tags` (jsonb) уже хранит **адрес, оператора/бренд, часы работы, сайт и телефон** — но эти данные не доходили до фронтенда. Теперь каждая точка показывает максимум доступной информации (по прямому запросу: «все точки должны показывать максимум информации»). ## Backend (`app/api/v1/parcels.py`) - `_clean_tag` / `_poi_detail_from_tags` — вытаскивают человекочитаемые поля из `tags`: - адрес = `addr:street` + `addr:housenumber` - оператор = `operator` → fallback `brand` - часы = `opening_hours` - сайт = `contact:website` → `website` → `url` - телефон = `contact:phone` → `phone` - мусор отбрасывается (`yes`/`no`/`fixme`/`?`/пустое/не-строки); всегда фиксированный набор из 5 ключей с `None` при отсутствии. - В SQL POI-выборки добавлен столбец `tags`; результат спредится в `score_breakdown`. - **Контракт API не меняется**: `score_breakdown: dict[str, Any]` (opaque map) — `app.openapi()` не трогается, codegen не нужен. Проверено reviewer'ом. ## Frontend - `ParcelAnalysisPoi`: добавлены опциональные `address/operator/opening_hours/website/phone`. - `SiteMap` попап POI рендерит эти поля: - сайт — через `safeHttpUrl` (allowlist `http(s)://`, иначе показываем как текст, не ссылку — XSS-защита по `frontend.md`); - телефон — `tel:` с санитайзингом (`[^\d+]` вырезается). ## Тесты `tests/api/v1/test_poi_detail_from_tags.py` — полный набор тегов, приоритет `contact:*`, `brand`-fallback, фиксированный keyset, отбрасывание мусора (5/5 проходят). ## Проверка - Бэкенд-юнит-тесты ✅, ruff/ruff-format ✅ - Frontend `tsc --noEmit` ✅, eslint ✅, prettier ✅ - Подтверждено по prod-БД: POI в радиусе 1 км от тестового участка 66:41:0702017:131 реально имеют эти теги (аптеки «Вита Экспресс»/«Живика»/«Классика» с часами и адресами «улица Вилонова, 14А»). - Pre-push review: ✅ APPROVE (без блокеров; два минорных cleanup'а сложены в этот же коммит). После merge+deploy — скриншот-верификация попапа на проде.

bot-backend added 1 commit 2026-06-30 12:26:55 +00:00

feat(site-finder): §1 POI-точки карты показывают максимум информации

CI / changes (pull_request) Successful in 9s

Details

CI / frontend-tests (pull_request) Successful in 1m8s

Details

CI / openapi-codegen-check (pull_request) Successful in 2m6s

Details

CI / backend-tests (pull_request) Successful in 14m29s

Details

a1be20c79b

POI-маркеры §1 раньше показывали только категорию, название, расстояние и
дату правки OSM. При этом osm_poi_ekb.tags (jsonb) уже хранит адрес, оператора/
бренд, часы работы, сайт и телефон — но они не доходили до попапа.

Backend (parcels.py):
- _clean_tag / _poi_detail_from_tags: вытаскивают человекочитаемые поля из tags
  (addr:street+housenumber → адрес, operator|brand, opening_hours,
  contact:website|website|url, contact:phone|phone), отбрасывая мусор (yes/no/
  fixme/пустое), фиксированный набор из 5 ключей с None при отсутствии;
- в SQL POI-выборки добавлен столбец tags; результат спредится в score_breakdown.
  Контракт не меняется: score_breakdown — dict[str, Any] (OpenAPI не трогаем).

Frontend:
- ParcelAnalysisPoi: +address/operator/opening_hours/website/phone (опциональны);
- SiteMap попап POI рендерит эти поля; safeHttpUrl(http/https-allowlist) для
  ссылки на сайт (иначе текст), tel: с санитайзингом для телефона.

Тест: tests/api/v1/test_poi_detail_from_tags.py (полный набор/приоритеты/мусор).