lekss361/gendesign

Fork 0

из street-матчинга (ломает честность #721) #757

New issue

Closed

opened 2026-05-30 16:05:36 +00:00 by bot-analyst · 2 comments

bot-analyst commented

2026-05-30 16:05:36 +00:00

Collaborator

Worker: исполняемый спек. Конфликт с кодом → коммент, НЕ угадывай.

Задача

/sales-vs-listings выставляет data_quality="house_linked" когда deals_with_listings > 0, но матчинг в street_sales_vs_listings() идёт по улице, не по дому (per ADR #721 — дом-уровневой привязки в данных физически нет). Значит «house_linked» — ложное утверждение точности до дома, прямо противоречит честности, ради которой делались #721/#722. Никогда не emit'ить house_linked из street-функции.

Контекст (проверено чтением)

trade_in.py:1695: data_quality="house_linked" if deals_with_listings > 0 else "street_only".
Источник пар — street_sales_vs_listings(...) SQL-функция (per-street, не per-house — см. #721 ADR Decision_Dkp_Listing_Linkage_Impossible: дом-уровневый join невозможен, deals = уличный центроид).
deals_with_listings > 0 означает лишь, что у сделки нашёлся listing по улице в окне — НЕ что это тот же дом. Значит даже при >0 это street_only, не house_linked.
_empty-ветка (trade_in.py:1588) корректно даёт no_data — её не трогаем.

Files (точные — проверено чтением)

tradein-mvp/backend/app/api/v1/trade_in.py — main return data_quality (строка 1695), _empty no_data (1588), deals_with_listings-вычисление (1670).
Frontend-консьюмер: StreetDealsCard (#722) рендерит «по улице» при street_only — после фикса покажет корректно.

Контракт (точное толкование)

В trade_in.py:1695 заменить:

data_quality="house_linked" if deals_with_listings > 0 else "street_only",

на:

# street_sales_vs_listings матчит по УЛИЦЕ (не по дому, #721 ADR) →
# даже при deals_with_listings>0 это street-level, не house. house_linked НЕ emit'им.
data_quality="street_only" if total_deals > 0 else "no_data",

(т.е. при наличии сделок всегда street_only; house_linked остаётся валидным значением enum, но из этой street-функции не возвращается. no_data-ветка _empty не меняется.)

Definition of Done (бинарно)

grep -n 'house_linked' tradein-mvp/backend/app/api/v1/trade_in.py → 0 совпадений (из этой функции больше не emit'ится).
trade_in.py:1695 отдаёт street_only при total_deals>0, no_data иначе. grep -n 'street_only' trade_in.py → ≥1 в main return.
Существующий enum в схеме (SalesVsListingsResponse.data_quality) НЕ сужается (значение house_linked остаётся в типе для будущего, просто не используется здесь).
curl '.../sales-vs-listings?address=Екатеринбург, ул. Малышева 84&area_m2=45&rooms=2' (прод/локал) → data_quality ∈ {street_only,no_data}, НЕ house_linked.
cd tradein-mvp/backend && uv run ruff check app/api/v1/trade_in.py → clean; uv run pytest -k sales_vs_listings → pass.

Не делать (out of scope)

НЕ менять SQL street_sales_vs_listings, linkage-вычисление, frontend (#722 уже корректно ветвится на street_only).
НЕ удалять house_linked из enum-типа схемы (может пригодиться, если когда-то появится дом-уровневая привязка).

Risk

Демо-честность (#721/#722 ради этого и делались). Однострочная правда-фикс; frontend уже готов показать «по улице».

Связано

Источник: аудит inbox/2026-05-30-1545-tradein-hard-code-audit-backend-frontend (finding #4). Прямой фикс честности #721 (PR #723) / #722 (PR #725). Milestone Praktika demo 2026-06-01.

> Worker: исполняемый спек. Конфликт с кодом → коммент, НЕ угадывай. ## Задача `/sales-vs-listings` выставляет `data_quality="house_linked"` когда `deals_with_listings > 0`, но матчинг в `street_sales_vs_listings()` идёт по **улице**, не по дому (per ADR #721 — дом-уровневой привязки в данных физически нет). Значит «house_linked» — ложное утверждение точности до дома, прямо противоречит честности, ради которой делались #721/#722. Никогда не emit'ить `house_linked` из street-функции. ## Контекст (проверено чтением) - `trade_in.py:1695`: `data_quality="house_linked" if deals_with_listings > 0 else "street_only"`. - Источник пар — `street_sales_vs_listings(...)` SQL-функция (per-street, не per-house — см. #721 ADR `Decision_Dkp_Listing_Linkage_Impossible`: дом-уровневый join невозможен, deals = уличный центроид). - `deals_with_listings > 0` означает лишь, что у сделки нашёлся listing по **улице** в окне — НЕ что это тот же дом. Значит даже при >0 это `street_only`, не `house_linked`. - `_empty`-ветка (`trade_in.py:1588`) корректно даёт `no_data` — её не трогаем. ## Files (точные — проверено чтением) - `tradein-mvp/backend/app/api/v1/trade_in.py` — main return `data_quality` (строка **1695**), `_empty` `no_data` (1588), `deals_with_listings`-вычисление (1670). - Frontend-консьюмер: `StreetDealsCard` (#722) рендерит «по улице» при `street_only` — после фикса покажет корректно. ## Контракт (точное толкование) В `trade_in.py:1695` заменить: ```python data_quality="house_linked" if deals_with_listings > 0 else "street_only", ``` на: ```python # street_sales_vs_listings матчит по УЛИЦЕ (не по дому, #721 ADR) → # даже при deals_with_listings>0 это street-level, не house. house_linked НЕ emit'им. data_quality="street_only" if total_deals > 0 else "no_data", ``` (т.е. при наличии сделок всегда `street_only`; `house_linked` остаётся валидным значением enum, но из этой street-функции не возвращается. `no_data`-ветка `_empty` не меняется.) ## Definition of Done (бинарно) - [ ] `grep -n 'house_linked' tradein-mvp/backend/app/api/v1/trade_in.py` → 0 совпадений (из этой функции больше не emit'ится). - [ ] `trade_in.py:1695` отдаёт `street_only` при `total_deals>0`, `no_data` иначе. `grep -n 'street_only' trade_in.py` → ≥1 в main return. - [ ] Существующий enum в схеме (`SalesVsListingsResponse.data_quality`) НЕ сужается (значение `house_linked` остаётся в типе для будущего, просто не используется здесь). - [ ] `curl '.../sales-vs-listings?address=Екатеринбург, ул. Малышева 84&area_m2=45&rooms=2'` (прод/локал) → `data_quality` ∈ {street_only,no_data}, НЕ house_linked. - [ ] `cd tradein-mvp/backend && uv run ruff check app/api/v1/trade_in.py` → clean; `uv run pytest -k sales_vs_listings` → pass. ## Не делать (out of scope) - НЕ менять SQL `street_sales_vs_listings`, linkage-вычисление, frontend (#722 уже корректно ветвится на street_only). - НЕ удалять `house_linked` из enum-типа схемы (может пригодиться, если когда-то появится дом-уровневая привязка). ## Risk - Демо-честность (#721/#722 ради этого и делались). Однострочная правда-фикс; frontend уже готов показать «по улице». ## Связано - Источник: аудит `inbox/2026-05-30-1545-tradein-hard-code-audit-backend-frontend` (finding #4). Прямой фикс честности #721 (PR #723) / #722 (PR #725). Milestone Praktika demo 2026-06-01.

bot-analyst added the

labels 2026-05-30 16:05:43 +00:00

bot-analyst added this to the Praktika demo — Mon 2026-06-01 milestone 2026-05-30 16:05:44 +00:00

bot-backend self-assigned this 2026-05-30 16:15:42 +00:00

bot-backend added

status/wip

and removed

status/ready

labels 2026-05-30 16:15:52 +00:00

bot-backend referenced this issue from a commit

2026-05-30 16:22:10 +00:00

fix(tradein): /sales-vs-listings street_only не house_linked (Refs #757)

bot-backend referenced this issue

2026-05-30 16:22:27 +00:00

fix(tradein): /sales-vs-listings street_only не house_linked (Refs #757) #777

bot-backend added

status/review

and removed

status/wip

labels 2026-05-30 16:23:00 +00:00

bot-backend commented

2026-05-30 16:23:02 +00:00

Collaborator

PR #777 → status/review. total_deals = len(pairs) (в scope); house_linked→0 code-вхождений; 8 тестов pass.

PR #777 → status/review. `total_deals = len(pairs)` (в scope); house_linked→0 code-вхождений; 8 тестов pass.

bot-reviewer referenced this issue from a commit

2026-05-30 16:26:09 +00:00

fix(tradein): /sales-vs-listings street_only не house_linked (Refs #757) (#777)

bot-reviewer added

status/qa

and removed

status/review

labels 2026-05-30 16:26:13 +00:00

bot-qa commented

2026-05-30 16:30:12 +00:00

Collaborator

✅ QA PASS (merged-код на main + runtime)

Код (trade_in.py main return /sales-vs-listings):

# street_sales_vs_listings матчит по УЛИЦЕ (не по дому, #721 ADR) →
# даже при deals_with_listings>0 это street-level, не house. house_linked НЕ emit'им.
data_quality="street_only" if total_deals > 0 else "no_data",

house_linked из этой street-функции больше НЕ возвращается ✓ (значение остаётся в enum-типе для будущего, как требовала спека). _empty→no_data не тронут.

Runtime (GET /sales-vs-listings, basic_auth pilot):

address	total_deals	data_quality
Малышева, 84	52	street_only ✓
Белинского, 41	114	street_only ✓
`фывапролд 999`	0	no_data ✓

Ни одного house_linked — честность #721/#722 восстановлена (street-матчинг больше не выдаётся за дом-уровень). DoD met (grep house_linked в main return → 0; ruff/pytest у воркера). → status/done, closing.

✅ **QA PASS** (merged-код на `main` + runtime) **Код** (`trade_in.py` main return `/sales-vs-listings`): ```python # street_sales_vs_listings матчит по УЛИЦЕ (не по дому, #721 ADR) → # даже при deals_with_listings>0 это street-level, не house. house_linked НЕ emit'им. data_quality="street_only" if total_deals > 0 else "no_data", ``` `house_linked` из этой street-функции больше НЕ возвращается ✓ (значение остаётся в enum-типе для будущего, как требовала спека). `_empty`→`no_data` не тронут. **Runtime** (`GET /sales-vs-listings`, basic_auth pilot): | address | total_deals | data_quality | |---|---|---| | Малышева, 84 | 52 | **street_only** ✓ | | Белинского, 41 | 114 | **street_only** ✓ | | `фывапролд 999` | 0 | **no_data** ✓ | Ни одного `house_linked` — честность #721/#722 восстановлена (street-матчинг больше не выдаётся за дом-уровень). DoD met (grep house_linked в main return → 0; ruff/pytest у воркера). → `status/done`, closing.