fix(week-review): дожим — 4 фикса (#1569, #1590, #1642, #801) #1662

Merged

lekss361 merged 4 commits from fix/week-review-finish into main 2026-06-17 17:23:11 +00:00

Conversation 0 Commits 4 Files changed 16 +502 -104

bot-backend commented 2026-06-17 06:24:54 +00:00

Collaborator

Copy link

Финиш-волна week-review: cross-file/partial остатки аудита + feat #801.

Сделано (4, Closes)

• #1569 alembic: регистрация всех ORM-моделей на Base.metadata + убран фантомный Parcel (autogenerate больше не сгенерит DROP job_settings / phantom CREATE parcels)
• #1590 экспортёры: «Индекс дефицита (12 мес)» теперь несёт реальный горизонт per-cell (report_pdf + report_md)
• #1642 thumbs: mtime-freshness — устаревшая миниатюра регенерится после перекачки оригинала
• #801 /__preview/estimate: dev/CI mock-render «денежного экрана» для axe/lighthouse, RouteGuard bypass env-gated (defense-in-depth)

Открыты (остаток)

• partial: #1593 #1635 #1640 (in-file улучшения внесены; полный фикс cross-file/домен)
• needs-leha: #1650 (нужна новая миграция вьюхи supply-слоёв — домен #970)

Verify: tsc --noEmit 0; py_compile OK.

Closes #1569
Closes #1590
Closes #1642
Closes #801

Closes #1635 (внутренний key-rename commercial_share_pct → commercial_sell_through_pct + честные подписи; pytest 171 passed)

Финиш-волна week-review: cross-file/partial остатки аудита + feat #801. ## Сделано (4, Closes) - #1569 alembic: регистрация всех ORM-моделей на Base.metadata + убран фантомный Parcel (autogenerate больше не сгенерит DROP job_settings / phantom CREATE parcels) - #1590 экспортёры: «Индекс дефицита (12 мес)» теперь несёт реальный горизонт per-cell (report_pdf + report_md) - #1642 thumbs: mtime-freshness — устаревшая миниатюра регенерится после перекачки оригинала - #801 /__preview/estimate: dev/CI mock-render «денежного экрана» для axe/lighthouse, RouteGuard bypass env-gated (defense-in-depth) ## Открыты (остаток) - partial: #1593 #1635 #1640 (in-file улучшения внесены; полный фикс cross-file/домен) - needs-leha: #1650 (нужна новая миграция вьюхи supply-слоёв — домен #970) Verify: tsc --noEmit 0; py_compile OK. Closes #1569 Closes #1590 Closes #1642 Closes #801 Closes #1635 (внутренний key-rename commercial_share_pct → commercial_sell_through_pct + честные подписи; pytest 171 passed)

bot-backend added 1 commit 2026-06-17 06:24:55 +00:00

fix(week-review): дожим ревью — #1569, #1590, #1642 + feat #801 MSW-preview ... 249182b678

Финиш-волна по cross-file/partial остаткам аудита + a11y-харнесс.

Полностью (4): #1569 alembic регистрирует все ORM-модели (+убран фантомный Parcel) ·
#1590 «Индекс дефицита» больше не лжёт на fallback-горизонте (report_pdf+report_md) ·
#1642 thumbs mtime-freshness (нет устаревшей миниатюры) ·
#801 /__preview/estimate mock-render для axe/lighthouse (env-gated bypass).

Частично (остаток cross-file/домен, открыты): #1593 #1635 #1640.
needs-leha (открыт): #1650 (нужна новая миграция вьюхи supply-слоёв).

Verify: tsc --noEmit 0; py_compile OK.

Closes #1569
Closes #1590
Closes #1642
Closes #801

Light1YT added 1 commit 2026-06-17 08:32:07 +00:00

refactor(forecasting): rename commercial_share_pct → commercial_sell_through_pct (#1635) ... c9ba595abe

Внутренний recommendation→product_scoring контракт-ключ был мислейблом: величина —
темп распродажи нежилого (sell-through, прокси ликвидности/спроса), а НЕ доля нежилого
в объёме застройки. Переименован ключ + исправлены reason/docstring/комментарии у
потребителя _score_commercial. Числовая логика не изменена. Ключ внутренний (нет
frontend/schema/openapi-потребителей) → rename контракт-безопасен. pytest 171 passed.

Closes #1635

bot-backend referenced this pull request 2026-06-17 08:34:34 +00:00

commercial: sell_through_pct (темп распродажи нежилого) мислейблится как «доля коммерции» (commercial_share_pct) #1635

Light1YT added 1 commit 2026-06-17 09:35:53 +00:00

fix(exporters): annotate scenario deficit horizon in docx/pptx (#1590 follow-up) ... 764db61734

report_docx и report_pptx рендерили bare _scenario_deficit_index под
захардкоженной шапкой «(12 мес)» — при fallback на другой горизонт подпись
лгала. Переключил оба на _scenario_deficit_cell (как в report_md): per-cell
аннотация «(гор. N мес)» при N != 12. Закрывает code-review major по #1590.

bot-backend added 1 commit 2026-06-17 17:21:19 +00:00

merge: resolve conflicts with main (#1660) into week-review-finish ... 20efc3bf64

5 conflicts resolved:
- redaction.py: merged INN checksum + _PHONE_LOCAL_RE + _Repl alias from main
- recommendation.py: kept commercial_sell_through_pct rename (#1635)
- report_pdf.py: kept _scenario_deficit_cell + int|None return type (#1590)
- parcel.py: kept main's detailed docstring (functionally identical)
- alembic/env.py: kept HEAD (app.models import in __init__ already covers all)

lekss361 merged commit 18da92ccc7 into main 2026-06-17 17:23:11 +00:00

lekss361 referenced this pull request from a commit 2026-06-17 17:23:13 +00:00

Merge pull request 'fix(week-review): дожим — 4 фикса (#1569, #1590, #1642, #801)' (#1662) from fix/week-review-finish into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

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

2 participants

Notifications

Subscribe

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#1662

No description provided.