[low] Cian has_furniture: реальное False теряется через or-коалесинг двух boolean-полей #1388

New issue

Closed

opened 2026-06-14 18:10:15 +00:00 by bot-backend · 0 comments

bot-backend commented

2026-06-14 18:10:15 +00:00

Collaborator

Описание

has_furniture вычисляется как offer.get('hasFurniture') or offer.get('isSoldFurnished'). Когда hasFurniture — явный False (продавец указал «нет мебели»), or короткозамыкается на isSoldFurnished, который может быть None или True — переворачивая определённое «нет мебели» в None/True. У двух полей разная семантика (мебель вообще vs продаётся с мебелью), их нельзя OR-склеивать.

Корень

Python or-коалесинг на boolean-поле, где False — валидное значение. Значение оседает в raw_payload['has_furniture'] (строка 865), персистится в БД, участвует в cross-source merge (conflict_resolution.py:79, LISTING_FIELD_PRIORITY для cian_serp).

Evidence

cian.py:805:

has_furniture: bool | None = offer.get("hasFurniture") or offer.get("isSoldFurnished")

Suggested fix

Явная None-проверка:

hf = offer.get('hasFurniture')
has_furniture = hf if hf is not None else offer.get('isSoldFurnished')

Либо хранить оба сырых поля раздельно в raw_payload.

Обоснование severity (low)

has_furniture — не first-class колонка ScrapedLot, не влияет на price/area/floor/address. Влияние ограничено качеством enrichment вторичного атрибута. Confidence 0.88.

week ревью 1 · multi-agent аудит 2026-06-14 · severity=low · category=logic

Затронутые файлы:

/Users/anton/Птица/gendesign/tradein-mvp/backend/app/services/scrapers/cian.py

## Описание `has_furniture` вычисляется как `offer.get('hasFurniture') or offer.get('isSoldFurnished')`. Когда `hasFurniture` — явный False (продавец указал «нет мебели»), `or` короткозамыкается на `isSoldFurnished`, который может быть None или True — переворачивая определённое «нет мебели» в None/True. У двух полей разная семантика (мебель вообще vs продаётся с мебелью), их нельзя OR-склеивать. ## Корень Python `or`-коалесинг на boolean-поле, где False — валидное значение. Значение оседает в `raw_payload['has_furniture']` (строка 865), персистится в БД, участвует в cross-source merge (conflict_resolution.py:79, LISTING_FIELD_PRIORITY для cian_serp). ## Evidence `cian.py:805`: ```python has_furniture: bool | None = offer.get("hasFurniture") or offer.get("isSoldFurnished") ``` ## Suggested fix Явная None-проверка: ```python hf = offer.get('hasFurniture') has_furniture = hf if hf is not None else offer.get('isSoldFurnished') ``` Либо хранить оба сырых поля раздельно в `raw_payload`. ## Обоснование severity (low) `has_furniture` — не first-class колонка ScrapedLot, не влияет на price/area/floor/address. Влияние ограничено качеством enrichment вторичного атрибута. Confidence 0.88. --- _week ревью 1 · multi-agent аудит 2026-06-14 · severity=**low** · category=`logic`_ Затронутые файлы: - `/Users/anton/Птица/gendesign/tradein-mvp/backend/app/services/scrapers/cian.py`