fix(tradein/estimator): дедуп кросс-source дублей в anchor-пути (#2265 follow-up) #2311

Merged

lekss361 merged 1 commit from fix/tradein-anchor-dedup-gap into main

2026-07-03 20:35:38 +00:00

lekss361 commented

2026-07-03 20:32:54 +00:00

Owner

Summary

Живой QA-smoke на проде (2026-07-03) после мерджа PR #2273 воспроизвёл тот же класс бага дублей в analogs — gap оказался в покрытии дедупа, не в алгоритме.

_dedup_cross_source() (введена PR #2273) вызывалась только на радиусном пути (_price_from_inputs() → listings). Anchor-тиры (Tier A same_building / Tier C micro_radius, _fetch_anchor_comps() — отдельный raw SQL) никогда не дедуплицировались, а anchor_comps_used фидит и показываемые analogs[], и n_analogs, как только сработал анкорный тир — именно этот путь и сработал на реальном тестовом адресе («Рассветная 5», физлот ул. Сыромолотова 11 показан дважды через cian+domklik).

Фикс: anchor_comps = _dedup_cross_source(anchor_comps) перед _compute_same_building_anchor и перед anchor_comps_used — оба потребителя читают один и тот же постдедуп список, n_analogs и показанные analogs остаются синхронными без отдельной логики.

Test plan

4 новых теста (test_estimator_anchor_dedup_gap_2265.py): точный live-репро физлот на anchor-пути (2→1), честность n_analogs, дедуп на пуле >2, anti-false-merge контроль
Регрессия подтверждена: без фикса 3/4 новых теста падают (воспроизводят баг), с фиксом — все зелёные
Targeted suite (dedup/anchor/estimator/comps): 651 passed, 4 skipped
Full backend suite: 3130 passed, 6 skipped, 1 failed (test_search_api.py::test_search_cache_hit — pre-existing, не связан, подтверждено прогоном на чистом forgejo/main)
Существующие 9 тестов PR #2273 (радиусный путь) не тронуты, все зелёные
post-merge: прод-smoke — переоценка кейса «Рассветная 5», n_analogs без дублей на anchor-тире

Refs #2265, #2262

## Summary Живой QA-smoke на проде (2026-07-03) после мерджа PR #2273 воспроизвёл тот же класс бага дублей в analogs — gap оказался в покрытии дедупа, не в алгоритме. `_dedup_cross_source()` (введена PR #2273) вызывалась только на радиусном пути (`_price_from_inputs()` → `listings`). Anchor-тиры (Tier A same_building / Tier C micro_radius, `_fetch_anchor_comps()` — отдельный raw SQL) никогда не дедуплицировались, а `anchor_comps_used` фидит и показываемые `analogs[]`, и `n_analogs`, как только сработал анкорный тир — именно этот путь и сработал на реальном тестовом адресе («Рассветная 5», физлот ул. Сыромолотова 11 показан дважды через cian+domklik). Фикс: `anchor_comps = _dedup_cross_source(anchor_comps)` перед `_compute_same_building_anchor` и перед `anchor_comps_used` — оба потребителя читают один и тот же постдедуп список, n_analogs и показанные analogs остаются синхронными без отдельной логики. ## Test plan - [x] 4 новых теста (`test_estimator_anchor_dedup_gap_2265.py`): точный live-репро физлот на anchor-пути (2→1), честность n_analogs, дедуп на пуле >2, anti-false-merge контроль - [x] Регрессия подтверждена: без фикса 3/4 новых теста падают (воспроизводят баг), с фиксом — все зелёные - [x] Targeted suite (dedup/anchor/estimator/comps): 651 passed, 4 skipped - [x] Full backend suite: 3130 passed, 6 skipped, 1 failed (test_search_api.py::test_search_cache_hit — pre-existing, не связан, подтверждено прогоном на чистом forgejo/main) - [x] Существующие 9 тестов PR #2273 (радиусный путь) не тронуты, все зелёные - [ ] post-merge: прод-smoke — переоценка кейса «Рассветная 5», n_analogs без дублей на anchor-тире Refs #2265, #2262

Rows
Columns