feat(insights): manual-entry Insight entity CRUD + §19 audit (#948 part A) #1164

Merged

bot-backend merged 1 commit from feat/insight-entity-948a into main

2026-06-08 07:42:16 +00:00

bot-backend commented

2026-06-08 07:42:14 +00:00

Collaborator

Что

#948 Part A — сущность «Инсайт» (ручная, ACL-гейт, §7.13/§8.9). Аналитик пишет free-form intel про локацию/участок с пометкой «непублично». Часть #944. (Part B — «Локация» как first-class entity §8.2 — отдельный follow-up, это больший district-рефактор.)

Разблокировано #962 (циклическая зависимость EPIC18↔EPIC4 разорвана — roles теперь есть).

Содержимое

data/sql/145_insight.sql — таблица insight (idempotent; geom GENERATED из lon/lat; индексы). Аддитивно расширяет CHECK audit_log.action += insight_write (тот же ck_audit_log_action, to_regclass-guarded, superset — не ломает #962-аудит, не правит applied 144).
api/v1/insights.py + services/insights.py (raw-SQL, зеркало custom_pois) + schemas/insight.py (Pydantic v2). CRUD: POST/GET(list+filters+pagination)/GET{id}/PUT/DELETE.
audit_middleware.classify_path (#962, моё): method-aware → insight_write для POST/PUT/DELETE (GET не аудируется), обратно совместимо.
ACL: created_by из X-Authenticated-User (не из тела — спуфинг автора невозможен). Backend rbac_guard hard-блокирует только /api/v1/admin/*; pilot отсекается frontend RouteGuard+Caddy (как parcels/custom_pois). is_confidential — пометка без per-record backend-гейта (analyst видит, per #962).

Качество

code-reviewer: ✅ APPROVE — оба гейта: cross-migration CHECK ALTER корректен (правильное имя constraint, idempotent, guarded, superset → #962-аудит цел) + zero SQL-injection (всё bind, CAST not ::, pagination 1-200). Author-integrity airtight. Docstrings исправлены на точную ACL-формулировку. 86 тестов (28 insight + 5 audit), ruff, без новых deps. parcels.py не тронут (NON-colliding).

Deploy/verify

145 применяется до нового кода (стандартный _schema_migrations order). #962+#948A backend-деплой пока в очереди раннера — проверю обе на проде когда backend пересоберётся.

Refs #948 (Part A; Part B Location — follow-up).

## Что #948 **Part A — сущность «Инсайт»** (ручная, ACL-гейт, §7.13/§8.9). Аналитик пишет free-form intel про локацию/участок с пометкой «непублично». Часть #944. (Part B — «Локация» как first-class entity §8.2 — отдельный follow-up, это больший district-рефактор.) Разблокировано #962 (циклическая зависимость EPIC18↔EPIC4 разорвана — roles теперь есть). ## Содержимое - `data/sql/145_insight.sql` — таблица `insight` (idempotent; geom GENERATED из lon/lat; индексы). Аддитивно расширяет CHECK `audit_log.action += insight_write` (тот же `ck_audit_log_action`, to_regclass-guarded, superset — **не ломает #962-аудит, не правит applied 144**). - `api/v1/insights.py` + `services/insights.py` (raw-SQL, зеркало custom_pois) + `schemas/insight.py` (Pydantic v2). CRUD: POST/GET(list+filters+pagination)/GET{id}/PUT/DELETE. - `audit_middleware.classify_path` (#962, моё): method-aware → `insight_write` для POST/PUT/DELETE (GET не аудируется), обратно совместимо. - **ACL:** `created_by` из `X-Authenticated-User` (не из тела — спуфинг автора невозможен). Backend rbac_guard hard-блокирует только /api/v1/admin/*; pilot отсекается frontend RouteGuard+Caddy (как parcels/custom_pois). `is_confidential` — пометка без per-record backend-гейта (analyst видит, per #962). ## Качество - `code-reviewer`: ✅ APPROVE — оба гейта: cross-migration CHECK ALTER корректен (правильное имя constraint, idempotent, guarded, superset → #962-аудит цел) + zero SQL-injection (всё bind, CAST not ::, pagination 1-200). Author-integrity airtight. Docstrings исправлены на точную ACL-формулировку. **86 тестов** (28 insight + 5 audit), ruff, без новых deps. `parcels.py` не тронут (NON-colliding). ## Deploy/verify 145 применяется до нового кода (стандартный `_schema_migrations` order). #962+#948A backend-деплой пока в очереди раннера — проверю обе на проде когда backend пересоберётся. Refs #948 (Part A; Part B Location — follow-up).

Rows
Columns