feat: ревью-инструмент авто-маппингов Объектив↔ДОМ.РФ (/admin/mapping-review) #2278

Merged

bot-backend merged 2 commits from feat/mapping-review-page into main

2026-07-03 07:59:14 +00:00

bot-backend commented

2026-07-03 07:38:51 +00:00

Collaborator

Что

Инструмент ручного ревью авто-маппингов objective_domrf_mapping (после autofill-волны #2177: ~170 строк is_reviewed=false, включая 14 ambiguous — им нужен глаз).

Backend

app/services/etl/mapping_review.py:
- list_mapping_review — постраничный список unreviewed с LATERAL-обогащением (последний снапшот domrf + developers, scoped по project_name = objective_complex_name — без глобальных агрегатов);
- approve_mapping — is_reviewed=true + append в note;
- reject_mapping — DELETE по PK с RETURNING: полная строка логируется (до commit) и эхом в ответе — ручной re-INSERT возможен из лога. Удаление обязано быть DELETE: строка иначе продолжает питать mv_layout_velocity §4.2.
3 endpoint'а в admin_etl.py (GET list / POST approve / POST reject), за общим Caddy basic_auth.

Frontend

/admin/mapping-review — таблица с warn-подсветкой строк, где ядро имени Объектива ≠ ядру ДОМ.РФ; confirm-модалка на reject («сделки исчезнут из §4.2»); пагинация; таб в admin layout.

Ревью

code-reviewer: ✅ APPROVE (minor: лог перенесён до commit — учтено). 10 тестов passed, ruff/tsc/eslint clean, CAST-trap grep clean.

## Что Инструмент ручного ревью авто-маппингов `objective_domrf_mapping` (после autofill-волны #2177: ~170 строк `is_reviewed=false`, включая 14 ambiguous — им нужен глаз). ### Backend - `app/services/etl/mapping_review.py`: - `list_mapping_review` — постраничный список unreviewed с LATERAL-обогащением (последний снапшот domrf + developers, scoped по `project_name = objective_complex_name` — без глобальных агрегатов); - `approve_mapping` — `is_reviewed=true` + append в note; - `reject_mapping` — DELETE по PK с RETURNING: полная строка логируется (до commit) и эхом в ответе — ручной re-INSERT возможен из лога. Удаление обязано быть DELETE: строка иначе продолжает питать `mv_layout_velocity` §4.2. - 3 endpoint'а в `admin_etl.py` (GET list / POST approve / POST reject), за общим Caddy basic_auth. ### Frontend - `/admin/mapping-review` — таблица с warn-подсветкой строк, где ядро имени Объектива ≠ ядру ДОМ.РФ; confirm-модалка на reject («сделки исчезнут из §4.2»); пагинация; таб в admin layout. ## Ревью code-reviewer: ✅ APPROVE (minor: лог перенесён до commit — учтено). 10 тестов passed, ruff/tsc/eslint clean, CAST-trap grep clean.

bot-backend added 1 commit 2026-07-03 07:38:52 +00:00

feat: mapping-review tool — list/approve/reject auto-mappings + admin page

CI Trade-In / changes (pull_request) Successful in 12s

Details

CI / changes (pull_request) Successful in 12s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Successful in 1m40s

Details

CI / openapi-codegen-check (pull_request) Failing after 2m45s

Details

CI / backend-tests (pull_request) Successful in 15m13s

Details

959fc8e7bf

- backend: mapping_review.py (list с LATERAL-обогащением domrf+developers,
  approve = is_reviewed+note, reject = DELETE по PK с RETURNING-логом)
- 3 endpoint'а в admin_etl.py (GET list / POST approve / POST reject)
- frontend: /admin/mapping-review — таблица с warn-подсветкой несовпадающих
  ядер, confirm-модалка на reject, пагинация; таб в admin layout
- 10 тестов (happy/404/null-score)

Light1YT added 1 commit 2026-07-03 07:43:36 +00:00

chore: regen api-types.ts for mapping-review endpoints

CI Trade-In / changes (pull_request) Successful in 7s

Details

CI / changes (pull_request) Successful in 6s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Successful in 1m29s

Details