fix(tradein): guard listing_segment во всех оценочных читателях listings — novostroyki вне comps (#1186) #1192

Merged

lekss361 merged 2 commits from fix/tradein-segment-guards into main

2026-06-12 09:06:29 +00:00

Author	SHA1	Message	Date
bot-backend	a9a489bba5	refactor(tradein): убрать dead listing_segment param из _fetch_anchor_comps + негативные guard-asserts (review fixup, #1186 ) All checks were successful CI / frontend-tests (push) Has been skipped Details CI / changes (pull_request) Successful in 6s Details CI / backend-tests (pull_request) Has been skipped Details CI / frontend-tests (pull_request) Has been skipped Details CI / changes (push) Successful in 6s Details CI / backend-tests (push) Has been skipped Details - Удалён параметр listing_segment: str \| None = None из сигнатуры _fetch_anchor_comps (guard хардкожен в SQL-блоках Tier A/C, параметр не использовался) - Убраны seg_counts / target_segment в caller-блоке SAME-BUILDING ANCHOR (~L2174) - Docstring Tier C обновлён: «вторичка-канон guard (#1186): NULL = legacy вторичка» - test_segment_guard_1186.py: добавлены 4 негативных assert'а: · no bare = 'vtorichka' без IS NULL OR · no deprecated <> / != 'novostroyki' form · _fetch_anchor_comps не содержит listing_segment в сигнатуре · Tier C не использует CAST(:segment AS text) (параметрическую форму)	2026-06-12 12:01:00 +03:00
bot-backend	b70ae46885	fix(tradein): guard listing_segment во всех читателях listings — novostroyki вне comps (#1186 ) Канон-предикат: (listing_segment IS NULL OR listing_segment = 'vtorichka') NULL = legacy вторичка до миграции 011 (sweep'ы были secondary-only) — отбрасывать нельзя. Затронуты все SELECT-пути, влияющие на оценку и корректирующие коэффициенты: - estimator.py: _COMMON_WHERE (Tier S/H), Tier W inline, Tier A anchor (добавлен впервые), Tier C anchor (заменён с параметрического CAST(:segment) на канон — прежний вариант отбрасывал NULL-строки при segment='vtorichka'). - asking_to_sold_ratio.py: ask_side, ask_global (_REDERIVE_SQL); bounds (_REDERIVE_BOUNDS_SQL); ask_tiered, ask_side_all, ask_global (_REDERIVE_TIERED_SQL) — 6 мест. - sber_index.py: reconciliation-запрос (диагностический asking-median). Не затронуты: скрейперы/scrape_pipeline, search matview (catalog display, не оценка), admin stats, geocoding, cian price history, matching — обосновано в инвентаризации. Тесты: новый test_segment_guard_1186.py (статика SQL + поведение mock); обновлён test_asking_to_sold_ratio.py (subset-тест нормализует guard, добавленный в refresh но отсутствующий в 080-seed).	2026-06-12 11:52:09 +03:00

Author

SHA1

Message

Date

bot-backend

a9a489bba5

refactor(tradein): убрать dead listing_segment param из _fetch_anchor_comps + негативные guard-asserts (review fixup, #1186 )

CI / frontend-tests (push) Has been skipped

Details

CI / changes (pull_request) Successful in 6s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / changes (push) Successful in 6s

Details

CI / backend-tests (push) Has been skipped

Details

- Удалён параметр listing_segment: str | None = None из сигнатуры _fetch_anchor_comps
  (guard хардкожен в SQL-блоках Tier A/C, параметр не использовался)
- Убраны seg_counts / target_segment в caller-блоке SAME-BUILDING ANCHOR (~L2174)
- Docstring Tier C обновлён: «вторичка-канон guard (#1186): NULL = legacy вторичка»
- test_segment_guard_1186.py: добавлены 4 негативных assert'а:
  · no bare = 'vtorichka' без IS NULL OR
  · no deprecated <> / != 'novostroyki' form
  · _fetch_anchor_comps не содержит listing_segment в сигнатуре
  · Tier C не использует CAST(:segment AS text) (параметрическую форму)

2026-06-12 12:01:00 +03:00

bot-backend

b70ae46885

fix(tradein): guard listing_segment во всех читателях listings — novostroyki вне comps (#1186 )

Канон-предикат: (listing_segment IS NULL OR listing_segment = 'vtorichka')
NULL = legacy вторичка до миграции 011 (sweep'ы были secondary-only) — отбрасывать нельзя.

Затронуты все SELECT-пути, влияющие на оценку и корректирующие коэффициенты:
- estimator.py: _COMMON_WHERE (Tier S/H), Tier W inline, Tier A anchor (добавлен впервые),
  Tier C anchor (заменён с параметрического CAST(:segment) на канон — прежний вариант
  отбрасывал NULL-строки при segment='vtorichka').
- asking_to_sold_ratio.py: ask_side, ask_global (_REDERIVE_SQL); bounds (_REDERIVE_BOUNDS_SQL);
  ask_tiered, ask_side_all, ask_global (_REDERIVE_TIERED_SQL) — 6 мест.
- sber_index.py: reconciliation-запрос (диагностический asking-median).

Не затронуты: скрейперы/scrape_pipeline, search matview (catalog display, не оценка),
admin stats, geocoding, cian price history, matching — обосновано в инвентаризации.

Тесты: новый test_segment_guard_1186.py (статика SQL + поведение mock); обновлён
test_asking_to_sold_ratio.py (subset-тест нормализует guard, добавленный в refresh но
отсутствующий в 080-seed).

2026-06-12 11:52:09 +03:00