lekss361/gendesign

fix(tradein/v2): реальный cv + единый source-registry в overlay-мапперах (#2040, #2041) #2298

Merged

lekss361 merged 1 commit from feat/tradein-v2-overlays-wiring into main

2026-07-03 20:00:57 +00:00

lekss361 commented

2026-07-03 19:55:16 +00:00

Owner

Закрывает оставшийся разрыв в #2040 (AnalyticsView) и #2041 (SourcesView) — основной wiring на реальные данные был смёржен ранее (267585e3, #2039/#2040/#2041), этот PR подтягивает то, что появилось позже:

mapSources(): marketAds.kpi.cv теперь берёт реальный estimate.cv (backend-поле из #2043), а не FE-приближение по analogs[].price_per_m2. Приближение оставлено как fallback для оценок, закэшированных до #2043 (когда cv в ответе отсутствует).
Локальная хардкод-карта источников в mappers.ts заменена на единый @/lib/source-registry (#2211) — убирает дублирование источника правды по лейблам.

code-reviewer: ✅ APPROVE (без замечаний, блокирующих merge). Minor follow-up вне scope: saleShareUtils.ts тоже имеет локальную SOURCE_LABELS — отдельный тикет.

tsc --noEmit + npm run build — зелёные.

Закрывает оставшийся разрыв в #2040 (AnalyticsView) и #2041 (SourcesView) — основной wiring на реальные данные был смёржен ранее (267585e3, #2039/#2040/#2041), этот PR подтягивает то, что появилось позже: - `mapSources()`: `marketAds.kpi.cv` теперь берёт реальный `estimate.cv` (backend-поле из #2043), а не FE-приближение по `analogs[].price_per_m2`. Приближение оставлено как fallback для оценок, закэшированных до #2043 (когда `cv` в ответе отсутствует). - Локальная хардкод-карта источников в `mappers.ts` заменена на единый `@/lib/source-registry` (#2211) — убирает дублирование источника правды по лейблам. code-reviewer: ✅ APPROVE (без замечаний, блокирующих merge). Minor follow-up вне scope: `saleShareUtils.ts` тоже имеет локальную `SOURCE_LABELS` — отдельный тикет. `tsc --noEmit` + `npm run build` — зелёные.

lekss361 added 1 commit 2026-07-03 19:55:17 +00:00

feat(tradein/v2): wire AnalyticsView + SourcesView overlays to real API data (#2040 , #2041 )

CI Trade-In / changes (pull_request) Successful in 8s

Details

CI / changes (pull_request) Successful in 8s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Successful in 1m19s

Details

Audit of /trade-in/v2 found the FE-3/FE-4 overlay wiring (mapAnalytics/mapSources
feeding AnalyticsView/SourcesView from useEstimateHouseAnalytics,
useEstimateSellTimeSensitivity, analogs[]/actual_deals[]) already shipped in
267585e3. The remaining gap was BE-1 (#2043, merged 2026-07-02) landing real
cv/source_counts/created_at on AggregatedEstimate AFTER that wiring pass, so
v2/mappers.ts was still FE-approximating cv and duplicating a hardcoded source
label map instead of the single source-registry (#2211).

- mapSources' marketAds.kpi.cv (#2041) now prefers the real estimate.cv
  (0..1 ratio from BE-1) and only falls back to the FE coefficient-of-variation
  of analog prices for pre-#2043 cached estimates where cv is absent.
- adRows/dealRows source badges (#2041) now resolve via the shared
  src/lib/source-registry sourceLabel() instead of a local SOURCE_LABEL map,
  picking up etagi/canonical labels and staying in sync with the #2211 roster.
- AnalyticsView (#2040) KPIs/sell-time/price-history/scatterDetail were already
  wired to real data with no gaps found against the issue spec.

lekss361 merged commit 698c1be4f3 into main

2026-07-03 20:00:57 +00:00

lekss361 deleted branch feat/tradein-v2-overlays-wiring

2026-07-03 20:00:57 +00:00

lekss361 referenced this pull request from a commit

2026-07-03 20:00:58 +00:00

Merge pull request 'fix(tradein/v2): реальный cv + единый source-registry в overlay-мапперах (#2040, #2041)' (#2298) from feat/tradein-v2-overlays-wiring into main

lekss361 referenced this pull request

2026-07-03 20:29:49 +00:00

trade-in/v2 [FE-3]: overlay «Аналитика дома» (AnalyticsView) #2040

lekss361 referenced this pull request

2026-07-03 20:29:50 +00:00

trade-in/v2 [FE-4]: overlay «РЫНОК / Аналогичные объявления» (SourcesView) #2041

lekss361 referenced this pull request

2026-07-03 21:08:06 +00:00

trade-in/v2: подключить боевые ручки к новому UI (/trade-in/v2) #2036

Sign in to join this conversation.

No reviewers

No labels

Fable 5 ревью

GG-форсайт

needs-discussion

status/needs-analysis

status/needs-fix

week ревью 1

вторичка

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#2298

No description provided.