[P2][tradein][backend] avito listings не деактивируются (is_active=true навсегда) → stale раздувает знаменатель coverage #759
Closed
opened 2026-05-30 16:06:09 +00:00 by lekss361
·
6 comments
No Branch/Tag specified
main
fix/2013-avito-cohort-null
fix/rbac-test-user2-expired
fix/expire-brusnika-trial-access
feat/tradein-domclick-house-sku-schema
chore/design-sync-tradein
fix/tradein-deploy-scraper-paths-filter
fix/tradein-no-numberless-house-insert
fix/tradein-dedup-fias-pass-no-geoguard
feat/tradein-fias-estimate-frontend
fix/tradein-dedup-by-fias
feat/tradein-fias-tier-matching
fix/tradein-v2-uiux-p1
fix/objective-lots-dedup-1964
fix/audit-followups-1953
feat/concept-house-picker-1965-stage3b
feat/concept-house-catalog-1965-stage3a
feat/concept-live-massing-1965-stage2b
feat/concept-recompute-1965-stage2a
feat/concept-section-1965-stage1
fix/best-layouts-per-object-snapshot-1956
fix/deficit-segmentation-1959
feat/tradein-hedonic-ground-floor
fix/report-map-points-1961
fix/report-microcopy-1963
fix/report-nspd-zoning-1962
fix/report-frontend-display-1953
perf/tradein-estimate-audit
perf/best-layouts-objects-first
fix/171-migration-path
perf/171-objective-crm-report-month-idx
fix/1926-app-log-stdout
fix/estimator-tiebreaker-nanalogs
fix/1917-fetch-json-network-retry
fix/1926-scrape-runs-total-seen
fix/1781-house-fk-source-identity
feat/39-osrm-per-category
feat/1746-utility-infra-map
fix/test-849-savepoint-count
fix/parking-economics-by-class
fix/zouit-engineering-area-gate
fix/zoning-resolution-decouple
feat/dcf-velocity-schedule
feat/zone-regulation-backfill
chore/mock-financial-estimate
fix/finmodel-split-disclosure-threshold
feat/finmodel-financing
feat/finmodel-sitefinder-bridge
feat/finmodel-dcf
feat/finmodel-price-calibration
feat/finmodel-cost-cascade
audit/1871-p2-estimator-dedup-split
audit/1871-p2-avito-coord
audit/1871-p3-uiux
audit/1871-p2-invest-moi
audit/1871-p2-velocity-market
audit/1871-p1-3-isreal-stubs
audit/1871-p1-2-ghost-anchor-guard
audit/1871-p1-scenarios-collapse
revert/site-finder-v2
fix/v2-remove-resource-legend
fix/v2-map-legend-position
feat/site-finder-v2-map-light-polish
rollback/site-finder-home-light
feat/site-finder-v2-cockpit
rollback/sf-old-analysis-design
feat/ptica-tabs-6-6-reports-params
fix/ptica-vri-row
feat/ptica-map-glyphs-sitefinder-dark
fix/ptica-3d-preview-product-verdict
fix/ptica-button-fontsize
fix/ptica-parity-data-mapping
chore/ptica-iso-preview-spin
fix/ptica-inter-map-product
fix/ptica-visual-parity
fix/ptica-left-rail
feat/ptica-cockpit-complete-bottom
feat/ptica-default-and-rich-map
feat/ptica-real-ksit-frontend
feat/tradein-ekb-geoportal-buildings
fix/sf-feedback-batch
feat/avito-newbuilding-sweep
feat/browser-concurrent-pool
fix/backfill-kadastr-column
fix/avito-proactive-ip-rotate
feat/activate-domclick-sweep
feat/avito-newbuilding-capture
fix/house-id-fk-realtime-link
fix/migration-162-table-if-exists
feat/riasurt-sverdl-layers-108
fix/sber-dataset-paths-902
fix/conflict-resolution-freshness-1539
feat/objective-lots-history-writer-307-obj6
feat/velocity-mv-61
fix/objective-mapping-is-reviewed-gate-307-obj2
fix/cian-detail-rich-fields
fix/avito-novostroyki-segment
feat/yandex-novostroyki-newflat
fix/ci-dedup-runner-storm-1709
fix/emiss-pk-granularity-1687
feat/listings-segment-param-1188
fix/inn-regex-word-boundary-1682
fix/excel-deficit-horizon-1590
fix/admin-scrape-validation
fix/admin-leads-interval-cast-1694
fix/dkp-corridor-doc-p10p90-1695
fix/weather-wind-d-allnone
test/residential-zone-rawprops-1681
fix/domrf-badge-regex-narrow-1686
feat/yandex-camoufox-transport
fix/score-sat-factor-1509
fix/base-pace-room-bucket-1593
fix/gate-verdict-no-blocker-1617
fix/confidence-deal-count-window-1637
fix/match-economics-1511-1512
fix/listings-partial-indexes-1398
fix/smart-suggestions-threshold-1503
fix/dkp-corridor-percentile-1520
fix/enrich-economics-window-1513
fix/leads-stats-window-1383
fix/poi-score-scale-1486
fix/geo-weight-floor-1633
fix/inn-regex-context-1640
fix/domrf-status-classify-1609
fix/emiss-period-dedup-1606
fix/cian-paginate-tail-1393
fix/normalize-zero-fill-guard-1638
fix/supply-view-devgroup-key-1650
fix/residential-zone-detect-1353
fix/score-label-threshold-1357
fix/trend-ratio-vprior-zero-1508
fix/district-median-dedup-1352
fix/supply-layers-overlap-1355
fix/pdf-expires-at-1530
fix/quartirography-area-bucket-1406
fix/cad-zones-multipolygon-1344
fix/sold-pct-market-verdict-1407
fix/week-review-finish
fix/week-review-backend-v2
feat/plan-vectorization-spike-299
fix/1668-analyze-confidence
fix/1667-metro-block
fix/1666-layout-velocity-beat
chore/remove-n1-scraper
fix/cian-kitchen-area
feat/yandex-detail-backfill
feat/avito-detail-backfill
fix/best-layouts-bucket-normalization
fix/gknspecial-incremental-commit-per-prefix
fix/okn-objects-full-registry-load
fix/parcel-ird-overlap-except-expand
fix/report-assembler-confounded-keys
fix/job-settings-json-persist-v2
fix/nspd-geo-soft-time-limit-passthrough
fix/nspd-soft-time-limit-v2
fix/pilot-request-modal-validation
fix/recommend-mix-saturation-latest-snapshot
fix/scrape-cadastre-fail-counter-v2
fix/izyatie-ocr-cad-normalize
fix/job-settings-panel-json-state-persist
fix/noise-loader-savepoint-per-row
fix/objective-sync-config-bare-except
fix/parcel-snapshot-mdash-double-escape
fix/poi-loader-leap-year-date-replace
fix/scrape-cadastre-permanent-fail-counter
fix/usermenu-analyst-role-label
feat/location-948b-r
feat/location-entity-948b
feat/insight-entity-948a
feat/analyst-role-audit-962
feat/cbr-inflation-macro-946
feat/cian-zhk-resolve-972
perf/forecast-parallel-horizons-1129
feat/newbuilding-enrich-backfill-972
feat/emiss-macro-indicators-946
feat/rosstat-emiss-macro-946
ci/frontend-vitest-job
chore/frontend-vitest-wkt-tests
feat/forecast-meta-footer
feat/forecast-future-supply-competitors
feat/forecast-docx-pptx-buttons
feat/forecast-scoring-panel
feat/forecast-product-tz-block
feat/market-map-layers-999
feat/forecast-chart-section6
feat/forecast-export-pptx-959
feat/forecast-export-docx-959
feat/forecast-export-ui-959
feat/forecast-export-telegram-959
feat/forecast-export-md-959
feat/analyze-response-contract-961
feat/competitor-velocity-spatial-fallback-968
fix/frontend-healthcheck-hostname
feat/analyze-market-coords-999
fix/site-finder-entry-crash-1001
feat/horizon-selector-996
feat/forecast-section-998
feat/forecast-endpoint-horizon-enqueue
feat/forecast-celery-task-persist
feat/forecast-orchestrator-service
feat/demand-supply-months-of-inventory
feat/wire-district-resolver
feat/district-vocab-reconciliation
fix/affordability-market-spread
fix/backtest-sig-gate
feat/wire-978-almon-rate-regime
feat/backtest-oos-almon-seasonal
ci/forgejo-pytest-gate
fix/tradein-praktika-logo-form-scroll
feat/yandex-exhaustive-price-room
feat/794-sber-time-adjust
chore/bot-harness-tuning
docs/agent-usage-throttle-qa-terminal
fix/avito-sweep-single-run-guard
chore/analyst-inbound-queue-tradein-db
feat/801-preview-full-wt
chore/bot-safety-followups
feat/tradein-avito-imv-fix
feat/tradein-repair-coef-calibration
feat/tradein-avito-houses-fix
feat/tradein-address-precision
feat/tradein-backfill-prioritize-listings
feat/tradein-geocode-deals-nominatim
feat/tradein-geocode-deals-from-houses
feat/tradein-same-building-match
feat/claude-envvars-and-stale-cron
No results found.
Labels
Clear labels
Admin UI / internal tools / pipeline tracker
Аналитика данных / matching
Authentication / authorization / multi-tenancy
Автоматизация / cron / scheduler
Something isn't working
Коммерческая воронка / пилоты / pipeline / реестр ПО
Operational housekeeping
Forgejo Actions / build pipeline
152-ФЗ / реестр ПО / legal / TOS
Data sources, parsers, ETL
Уникальные данные — sales tracker, CRM aggregation
README / runbooks / onboarding
This issue or pull request already exists
Developer experience / build speed
New feature or request
Находки полного мульти-агентного ревью (Claude Fable 5, 2026-06-09)
Feedback from Max (Senior partner)
Generative Design — concept generation engine
Новое ТЗ от Ген Геныча (июнь 2026): Site Finder v2 — ИИ-прогноз продуктовой стратегии девелопера. Актуальный приоритет.
Pending team sync / decision
Human review/decision required
Monitoring / metrics / logs / Sentry / Uptime
Kill-switch — all autonomous bots pause
Latency / throughput / DB query tuning
Critical / production-blocker
High priority
Normal priority
Низкий приоритет (follow-up, tech-debt)
Backend Python — FastAPI, Celery, scrapers
DB migrations, indexes, SQL
docker-compose, Caddy, workflows
Frontend — Next.js, React, TypeScript
Tests, smoke, Playwright
Скрапперы (Avito/Cian/Yandex/Rosreestr)
Security audit / secrets / vulnerabilities
Site Finder scoring quality & data layer
Stage 1: Generative MVP (нед 2-7)
Stage 2: Site Finder MVP (нед 8-12)
Blocked — needs human intervention
Closed, completed
Raw/fuzzy task filed by human — auto-analyst подхватывает: code-archeology + декомпозиция в actionable scope/* + status/ready (или split на под-issues)
Reviewer/QA вернул worker на доработку — worker сам чинит свой PR (fixup), НЕ требует human (в отличие от status/blocked)
PR merged, awaiting smoke test
Ready for pickup by worker bot
PR open, awaiting code review
Work in progress (worker claimed)
Технический долг
Trade-In MVP проект
User experience / responsive / conversion
Week review 1 — находки multi-agent аудита проекта (2026-06-14)
This will not be worked on
Trade-In Estimator — проект оценки вторичного жилья
Исходно-разрешительная документация — слой ограничений участка для analyze-таблицы
admin
Admin UI / internal tools / pipeline tracker
analytics
Аналитика данных / matching
auth
Authentication / authorization / multi-tenancy
automation
Автоматизация / cron / scheduler
bug
Something isn't working
business
Коммерческая воронка / пилоты / pipeline / реестр ПО
chore
Operational housekeeping
ci
Forgejo Actions / build pipeline
compliance
152-ФЗ / реестр ПО / legal / TOS
data
Data sources, parsers, ETL
data-moat
Уникальные данные — sales tracker, CRM aggregation
docs
README / runbooks / onboarding
duplicate
This issue or pull request already exists
dx
Developer experience / build speed
enhancement
New feature or request
Fable 5 ревью
Находки полного мульти-агентного ревью (Claude Fable 5, 2026-06-09)
feedback/max
Feedback from Max (Senior partner)
generative
Generative Design — concept generation engine
GG-форсайт
Новое ТЗ от Ген Геныча (июнь 2026): Site Finder v2 — ИИ-прогноз продуктовой стратегии девелопера. Актуальный приоритет.
needs-discussion
Pending team sync / decision
needs-human
Human review/decision required
observability
Monitoring / metrics / logs / Sentry / Uptime
pause-bots
Kill-switch — all autonomous bots pause
performance
Latency / throughput / DB query tuning
priority/p0
Critical / production-blocker
priority/p1
High priority
priority/p2
Normal priority
priority/p3
Низкий приоритет (follow-up, tech-debt)
scope/backend
Backend Python — FastAPI, Celery, scrapers
scope/db
DB migrations, indexes, SQL
scope/devops
docker-compose, Caddy, workflows
scope/frontend
Frontend — Next.js, React, TypeScript
scope/qa
Tests, smoke, Playwright
scrapers
Скрапперы (Avito/Cian/Yandex/Rosreestr)
security
Security audit / secrets / vulnerabilities
site-finder
Site Finder scoring quality & data layer
stage/1
Stage 1: Generative MVP (нед 2-7)
stage/2
Stage 2: Site Finder MVP (нед 8-12)
status/blocked
Blocked — needs human intervention
status/done
Closed, completed
status/needs-analysis
Raw/fuzzy task filed by human — auto-analyst подхватывает: code-archeology + декомпозиция в actionable scope/* + status/ready (или split на под-issues)
status/needs-fix
Reviewer/QA вернул worker на доработку — worker сам чинит свой PR (fixup), НЕ требует human (в отличие от status/blocked)
status/qa
PR merged, awaiting smoke test
status/ready
Ready for pickup by worker bot
status/review
PR open, awaiting code review
status/wip
Work in progress (worker claimed)
tech-debt
Технический долг
tradein
Trade-In MVP проект
ux
User experience / responsive / conversion
week ревью 1
Week review 1 — находки multi-agent аудита проекта (2026-06-14)
wontfix
This will not be worked on
вторичка
Trade-In Estimator — проект оценки вторичного жилья
ИРД
Исходно-разрешительная документация — слой ограничений участка для analyze-таблицы
No labels
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
Fable 5 ревью
feedback/max
generative
GG-форсайт
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
вторичка
ИРД
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set.
Reference: lekss361/gendesign#759
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Проблема
save_listings(tradein-mvp/backend/app/services/scrapers/base.py:214; ON CONFLICT на стр.280-281:SET last_seen_at=NOW(), is_active=true) реактивирует каждую ре-увиденную строку, но НЕТ механизма ставитьis_active=falseдля avito-листингов, которые перестали появляться в выдаче (проданы/сняты). У cian скоуп через snapshot — у avito нет.Live-факт (2026-05-31, tradein-БД):
listings WHERE source='avito'= 9765, все is_active=true; не виделись >7 дней — 5113 (52%).last_seen_atзаполнен 100%, обновляется (свежий 2026-05-30). Раздутый знаменатель искажает coverage (#726) и медиану аналогов в estimator (мёртвые лоты в выборке).Решение (DECIDED — воркер НЕ выбирает)
TTL по
last_seen_at. Вариант «run_id sweep-scoped» ОТКЛОНЁН: колонкиrun_idвlistingsНЕТ (провереноinformation_schema.columns) — инфраструктуры под него нет.last_seen_atесть и живой → TTL единственно реализуемый.Порог N = 10 дней. Обоснование: avito_city_sweep (#814) покрывает 5 анкоров EKB за ~2-3 ночи → 10 дней заведомо больше полного цикла покрытия города ⇒ нет ложных деактиваций из-за частичного sweep'а одним анкором.
Files
tradein-mvp/backend/app/services/scheduler.py— добавить scheduled taskdeactivate_stale_avito_listings(), окно после ночного avito_city_sweep (напр. 06:00 UTC). Паттерн регистрации — как у существующих задач в этом файле.CAST(... AS interval), НЕ::interval):AVITO_STALE_TTL_DAYS = 10вtradein-mvp/backend/app/core/config.py.Definition of Done (бинарно проверяемо)
deactivate_stale_avito_listingsзарегистрирован вscheduler.py, запускается после avito-sweep.SELECT count(*) FILTER (WHERE is_active) FROM listings WHERE source='avito'строго меньше total (деактивированы не-виденные >10д).WHERE source='avito';SELECT count(*) FILTER (WHERE NOT is_active) FROM listings WHERE source<>'avito'не изменился.uv run pytest(tradein backend) зелёный;uv run ruff checkна изменённых файлах clean.Не делать (out of scope)
is_active=false(история нужна для backtest #667).Refs #726, #814.
🔍 Рефайн аналитика: тело переписано в однозначный спек (TTL по last_seen_at, N=10д решено; run_id-вариант отклонён — колонки нет). Verified на live tradein-БД (9765 avito, все is_active, 52% stale >7д). Готово к работе.
bot-analyst referenced this issue2026-05-31 07:04:24 +00:00
Working on this in PR #862 (status/review). TTL-деактивация по
last_seen_at>10д, source='avito' only, is_active=false (no DELETE), scheduler task + seed migration 090. 32 tests pass, ruff clean. Live "после прогона" проверки — post-deploy (qa).⏳ QA: code-уровень verified против merged diff (PR #862, merged 07:19:37), runtime — deploy+scheduled-fire+tradein-DB gated.
✅ Code DoD (проверено по диффу
42007e9b):deactivate_stale_avito_listingsзарегистрирован: новыйapp/tasks/deactivate_stale_avito.py+trigger_deactivate_stale_avito_run+ dispatch-веткаelif source == "deactivate_stale_avito"вscheduler_loop. Окно seed-migration 090 = 06:00–07:00 UTC, после avito_city_sweep (02:00–05:00).next_run_at= завтра 06:00 UTC ⇒ НЕ стреляет на деплое (нет преждевременной деактивации по неполному sweep'у). ✓WHERE source='avito' AND is_active=true; unit-тестtest_sql_targets_only_avito_sourceэто фиксирует. ✓CAST(:ttl_days || ' days' AS interval)(bind-param, никакого:x::type— regex-guard в тестах), нет DELETE (толькоis_active=false, история для #667 цела), commit/rollback + mark_done/mark_failed. 20 unit-тестов в диффе (SQL shape, CAST-discipline, scheduler wiring, migration idempotencyON CONFLICT(source) DO NOTHING, behavioural fake-db rowcount→counter). PR: 32 passed, ruff clean. ✓⏳ Runtime DoD — НЕ оценимо из QA-окна (by design + tooling-limit):
count(*) FILTER (WHERE is_active) < total для avitoи DoD-3-livenon-avito is_active unchanged— требуют (а) деплоя миграции 090 + scheduler.py, (б) первого прогона задачи 06:00 UTC завтра, (в) доступа к tradein-БД, которого у QA-окна нет.→ оставляю
status/qa. Re-QA после первого 06:00 UTC fire (у вас есть live tradein-доступ — вы уже мерили 9765 avito / 52% stale в рефайне):Если после fire
active == total(0 деактивировано) при наличии >10д-stale строк — классифицировать как scheduler-dispatch регресс (needs-fix, assignee bot-backend), НЕ infra.Refs #726, #814.
🔎 QA update — read-only prod-БД проверка (теперь есть
docker exec tradein-postgres psql).Подтвердил вживую на tradein-БД (2026-05-31 ~08:00 UTC):
Код + schedule ✅: задача
deactivate_stale_avitoзарегистрирована, schedule id=111 живой на проде (enabled, next_run 2026-06-01 06:00 UTC) — verified в #865 smoke. SQL корректен (WHERE source='avito' AND is_active AND last_seen_at < NOW()-CAST(:ttl_days||' days' AS interval), no DELETE).⚠️ Но runtime-DoD «после прогона count<total» сейчас НЕ демонстрируем: 0 строк превышают 10-дневный TTL. Tension с постановкой issue: тело говорит «5113 (52%) не виделись >7 дней», но порог задачи = 10 дней, и эти 5113 сейчас в полосе [7,10)д ⇒ ближайший fire (06-01 06:00) деактивирует 0. Плюс avito_city_sweep запланирован 06-01 02:00 (ДО deactivate) — если он переувидит часть строк, их last_seen_at обновится и они вообще не состарятся до 10д.
→ Это не дефект (10д выбран намеренно против ложных деактиваций), но значит: (а)
count<totalне подтвердится на ближайшем прогоне; (б) для фактической чистки 5113-раздутия denominatorّа порог 10д сработает только когда строки доживут до >10д без переувидения. Оставляюstatus/qa, re-verify после 06-01 06:00 fire (доберу через psql). Если к тому моменту eligible всё ещё ~0 — возможно стоит пересмотреть, решает ли 10д-TTL исходную проблему inflated-denominator (#726), или нужен порог ниже / другой механизм для «7-10д» полосы. Это уже product-вопрос → тебе.Refs #726, #814.
QA INCONCLUSIVE → needs-fix (06.06) — код заведён, но в проде не работает:
✅ Код PASS: task
deactivate_stale_avito_listingsзарегистрирован (scheduler.py:573,1000, dispatch ok), SQL avito-scoped + psycopg-v3-safe (tasks/deactivate_stale_avito.py:33-41), TTL изAVITO_STALE_TTL_DAYS(default 10).✅ Scope guard PASS: деактивация строго avito; cian/yandex/n1 не затронуты.
❌ Прод-блокер:
scrape_schedulesid=111 (deactivate_stale_avito) —enabled=false,last_run_id=NULL,next_run_at=2026-06-01(в прошлом). Задача ни разу не отработала.scrape_runs WHERE source ILIKE '%deactiv%'= 0 строк. 1247 inactive avito (7.9%) не атрибутируются этой задаче (вероятно более ранний ручной UPDATE).Fix: либо
UPDATE scrape_schedules SET enabled=true WHERE id=111на проде, либо миграция, вставляющая строку сenabled=true(docstring говорит «enabled by default» — расходится с фактом).status/qa→status/needs-fix.Fix готов → PR #1096. Root cause: миграция
090(ON CONFLICT DO NOTHING) пропустила существующую прод-строкуenabled=false. Добавлена идемпотентная миграция100_enable_deactivate_stale_avito.sql(UPDATE enabled=true + next_run_at=NOW(), guardenabled=false) + 7 тестов (39 passed, ruff clean).needs-fix→review.