[tradein][tech-debt] Mock-bands legacy code removal — PR-8 из 8PR roadmap #579

New issue

Closed

opened 2026-05-25 06:02:36 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-05-25 06:02:36 +00:00

Owner

Проблема

В estimator есть legacy mock-bands (когда не было реальных listings). Сейчас listings есть → убрать заглушки.

Что сделать

grep -r "mock_band\|MOCK_BAND" backend/
Удалить, обновить тесты.

Refs

decisions/Decision_TradeIn_DataQuality_8PR_Roadmap PR-8

## Проблема В estimator есть legacy mock-bands (когда не было реальных listings). Сейчас listings есть → убрать заглушки. ## Что сделать 1. `grep -r "mock_band\|MOCK_BAND" backend/` 2. Удалить, обновить тесты. ## Refs - `decisions/Decision_TradeIn_DataQuality_8PR_Roadmap` PR-8

lekss361 added the

tech-debt

tradein

labels 2026-05-25 06:02:36 +00:00

Light1YT commented

2026-05-29 08:30:17 +00:00

Owner

Проверено 2026-05-29: mock-bands dead-код уже удалён — ни одного определения _mock_estimate/_estimate_legacy_mock/_PRICE_BANDS/_EKB_STREETS/_gen_analogs/_floor_factor/_repair_factor в backend/app/ нет, в api/v1/trade_in.py следов нет (вычищено в Audit Cleanup May24, роадмап вырос до 10 PR). Остался безвредный docstring-комментарий в estimator.py. Закрываю как выполненный.

Проверено 2026-05-29: mock-bands dead-код уже удалён — ни одного определения `_mock_estimate`/`_estimate_legacy_mock`/`_PRICE_BANDS`/`_EKB_STREETS`/`_gen_analogs`/`_floor_factor`/`_repair_factor` в `backend/app/` нет, в `api/v1/trade_in.py` следов нет (вычищено в Audit Cleanup May24, роадмап вырос до 10 PR). Остался безвредный docstring-комментарий в estimator.py. Закрываю как выполненный.