fix(tradein/pii): удалить телефоны продавцов (NULL) + PII клиента (DROP) (#2217, #1969) #2303

Merged

lekss361 merged 1 commit from fix/tradein-pii-purge-phones-client into main

2026-07-03 20:21:47 +00:00

lekss361 commented

2026-07-03 20:14:21 +00:00

Owner

Закрывает #2217 (телефоны продавцов) + #1969 (PII клиента) — решения user зафиксированы в комментах issue.

#2217 — listings.phones → NULL (не DROP, решение user буквально):

Миграция 166: UPDATE listings SET phones = NULL WHERE phones IS NOT NULL (17342 строки).
Write-path остановлен в ОБОИХ зеркалах (cian.py + scraper-kit) — golden-parity тесты подтверждают синхронность.
Убрана мёртвая запись phones из LISTING_FIELD_PRIORITY (zero production callers).
management_companies.phones (другая сущность, УК/ТСЖ) не тронута.
Подтверждено: phones НЕ участвует в живом дедупе/matching.

#1969 — client_name/client_phone → DROP COLUMN (решение user буквально):

Миграция 167: ALTER TABLE trade_in_estimates DROP COLUMN IF EXISTS client_name, client_phone — нет VIEW/индексов, чисто.
Убраны из schemas/estimator.py (оба INSERT-пути)/frontend форм и типов.
client_email физически не существовало (только в Sentry-scrub allowlist) — трогать нечего.

code-reviewer: ✅ APPROVE (20 мин, учитывая необратимость данных — grep подтвердил отсутствие пропущенных референсов, golden-parity зелёные). Локально: uv run pytest → 3119 passed / 6 skipped (1 деселект известного флейка test_search_cache_hit); npm run type-check/lint — чисто.

Закрывает #2217 (телефоны продавцов) + #1969 (PII клиента) — решения user зафиксированы в комментах issue. **#2217 — listings.phones → NULL (не DROP, решение user буквально):** - Миграция 166: `UPDATE listings SET phones = NULL WHERE phones IS NOT NULL` (17342 строки). - Write-path остановлен в ОБОИХ зеркалах (cian.py + scraper-kit) — golden-parity тесты подтверждают синхронность. - Убрана мёртвая запись `phones` из `LISTING_FIELD_PRIORITY` (zero production callers). - `management_companies.phones` (другая сущность, УК/ТСЖ) не тронута. - Подтверждено: phones НЕ участвует в живом дедупе/matching. **#1969 — client_name/client_phone → DROP COLUMN (решение user буквально):** - Миграция 167: `ALTER TABLE trade_in_estimates DROP COLUMN IF EXISTS client_name, client_phone` — нет VIEW/индексов, чисто. - Убраны из schemas/estimator.py (оба INSERT-пути)/frontend форм и типов. - `client_email` физически не существовало (только в Sentry-scrub allowlist) — трогать нечего. code-reviewer: ✅ APPROVE (20 мин, учитывая необратимость данных — grep подтвердил отсутствие пропущенных референсов, golden-parity зелёные). Локально: `uv run pytest` → 3119 passed / 6 skipped (1 деселект известного флейка `test_search_cache_hit`); `npm run type-check`/`lint` — чисто.

Rows
Columns