lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

feat(parcels): persist analysis runs + real last_analysis_date (#994) #1028

Merged
bot-backend merged 1 commit from feat/994-persist-analysis-runs into main 2026-06-03 12:36:31 +00:00
Conversation 0 Commits 1 Files changed 4 +310 -7
bot-backend commented 2026-06-03 12:36:26 +00:00
Collaborator
Copy link

#961 EPIC 17 (961-C3): персист прогонов анализа + реальный last_analysis_date (ТЗ §22).

  • Persist: каждый успешный POST /analyze → строка в analysis_runs (м.127). Best-effort: SAVEPOINT (begin_nested) + explicit db.commit() + try/except. db.commit() критичен — get_db() без commit-on-success, RELEASE savepoint сам не персистит → без коммита строка откатывалась бы на teardown (silent no-op, поймано code-review). Провал persist НЕ ломает ответ/не травит сессию. result через jsonable_encoder→json.dumps (Pydantic-модели/даты); confidence/status нормализованы под CHECK'и 127.
  • last_analysis_date: by-bbox заполняется из v_analysis_runs_latest одним batch-запросом (НЕ N+1), заменяет заглушку #307. Чтение тоже best-effort — view-missing deploy-окно/drift НЕ должен ронять карту (fallback None).
  • Additive only — форма ответа analyze не меняется.
  • Тесты: by-bbox-сьюит патчит latest_run_dates (mock-db отдаёт те же rows на любой запрос) + новый тест на ISO-дату из latest-рана. Analyze mock-сьюит не затронут (graceful side_effect overflow + best-effort persist поглощают лишний INSERT). Incidental: ruff-format поправил одну pre-existing f-string (строка ~2391).

⚠️ Cross-tx write-тест (строка переживает commit) — post-deploy QA: локально нет TEST_DATABASE_URL + WeasyPrint native libs (macOS) блокируют сбор pytest. CI-Linux прогонит сьюит.

Closes #994. Refs #961.

**#961 EPIC 17 (961-C3): персист прогонов анализа + реальный last_analysis_date (ТЗ §22).** - **Persist**: каждый успешный POST /analyze → строка в `analysis_runs` (м.127). Best-effort: SAVEPOINT (begin_nested) + explicit `db.commit()` + try/except. `db.commit()` критичен — get_db() без commit-on-success, RELEASE savepoint сам не персистит → без коммита строка откатывалась бы на teardown (silent no-op, **поймано code-review**). Провал persist НЕ ломает ответ/не травит сессию. result через jsonable_encoder→json.dumps (Pydantic-модели/даты); confidence/status нормализованы под CHECK'и 127. - **last_analysis_date**: by-bbox заполняется из `v_analysis_runs_latest` одним batch-запросом (НЕ N+1), заменяет заглушку #307. Чтение тоже best-effort — view-missing deploy-окно/drift НЕ должен ронять карту (fallback None). - **Additive only** — форма ответа analyze не меняется. - **Тесты**: by-bbox-сьюит патчит latest_run_dates (mock-db отдаёт те же rows на любой запрос) + новый тест на ISO-дату из latest-рана. Analyze mock-сьюит не затронут (graceful side_effect overflow + best-effort persist поглощают лишний INSERT). Incidental: ruff-format поправил одну pre-existing f-string (строка ~2391). ⚠️ Cross-tx write-тест (строка переживает commit) — post-deploy QA: локально нет TEST_DATABASE_URL + WeasyPrint native libs (macOS) блокируют сбор pytest. CI-Linux прогонит сьюит. Closes #994. Refs #961.
bot-backend added 1 commit 2026-06-03 12:36:27 +00:00
feat(parcels): persist analysis runs + real last_analysis_date (#994)
All checks were successful
Deploy / changes (push) Successful in 5s
Details
Deploy / build-frontend (push) Has been skipped
Details
Deploy / build-backend (push) Successful in 1m35s
Details
Deploy / build-worker (push) Successful in 2m31s
Details
Deploy / deploy (push) Successful in 1m42s
Details
72eae3bc9b 
Persist each successful POST /analyze into analysis_runs (migration 127):
best-effort SAVEPOINT-wrapped INSERT in a thin analysis_runs repository, then
explicit db.commit() (get_db has no commit-on-success; SAVEPOINT RELEASE alone
does not persist — without the commit the row rolls back on request teardown,
a silent no-op caught in code review). A persist failure never breaks the
response or poisons the session. Result serialized via jsonable_encoder→
json.dumps (handles embedded Pydantic models/dates); confidence/status
normalized to satisfy the 127 CHECK constraints (unknown→NULL/'complete').

Populate by-bbox last_analysis_date from v_analysis_runs_latest via a single
batch query (no N+1), replacing the #307 placeholder. The read is best-effort
wrapped too — a view-missing deploy window or future drift must not 500 the
map (falls back to last_analysis_date=None).

Additive only — analyze response shape unchanged. Tests: by-bbox suite patches
latest_run_dates (mock-db returns same rows for any query) + new test asserts
ISO last_analysis_date from a latest run. Analyze mock-suite unaffected
(graceful side_effect overflow + best-effort persist absorb the extra INSERT).
Incidental: ruff-format fixed one pre-existing f-string spacing (line ~2391)
the format hook flags on touch.

Closes #994. Refs #961.
bot-backend merged commit 72eae3bc9b into main 2026-06-03 12:36:31 +00:00
bot-backend deleted branch feat/994-persist-analysis-runs 2026-06-03 12:36:31 +00:00
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
1 participant
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#1028
No description provided.
Delete branch "feat/994-persist-analysis-runs"

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?