lekss361/gendesign

Fork 0

bug(#254): POST /custom-pois возвращает 201 но row не persisted в DB (missing commit?) #261

New issue

Closed

opened 2026-05-17 06:58:18 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-05-17 06:58:18 +00:00

Owner

Симптом

После deploy PR #257 (merge 8cc9447) — post-deploy smoke:

$ curl -X POST https://gendsgn.ru/api/v1/custom-pois \
    -H "X-Session-Id: smoke-test-001" \
    -H "Content-Type: application/json" \
    -d '{"name":"Smoke test POI","weight":2.0,"lon":60.597,"lat":56.838,"notes":"deploy verify"}'

HTTP/1.1 201 Created
X-Session-Id: smoke-test-001
{"id":1,"user_id":"smoke-test-001","parcel_cad":null,"name":"Smoke test POI",
 "category":null,"weight":2.0,"lon":60.597,"lat":56.838,"notes":"deploy verify",
 "created_at":"2026-05-17T06:51:59.688099Z","updated_at":"2026-05-17T06:51:59.688099Z"}

Через 5 минут:

SELECT * FROM user_custom_pois;
-- (0 rows)

И:

$ curl https://gendsgn.ru/api/v1/custom-pois?user_id=smoke-test-001 -H "X-Session-Id: smoke-test-001"
[]

$ curl -X PATCH https://gendsgn.ru/api/v1/custom-pois/1 -H "X-Session-Id: smoke-test-001" -d '{"weight":3.5}'
HTTP 404 {"detail":"Custom POI not found"}

Root cause hypothesis

backend/app/services/site_finder/custom_pois.py create_custom_poi — скорее всего отсутствует db.commit() после db.execute(INSERT ... RETURNING ...). SQLAlchemy session возвращает RETURNING row из in-flight transaction (отсюда id=1 в response), но при HTTP response close — transaction откатывается потому что не было explicit commit.

Аналогично для update_custom_poi, delete_custom_poi.

Fix

В backend/app/services/site_finder/custom_pois.py:

После INSERT: db.commit() → потом return result
После UPDATE: db.commit()
После DELETE: db.commit()

ИЛИ register dependency что использует with db.begin(): context. См. pattern в weight_profiles.py create_profile.

Дополнительно

backend/app/api/v1/custom_pois.py endpoints могут использовать Depends(get_db) который возвращает Session (не SessionLocal() + begin()). Если get_db имеет yield-based context который коммитит при finally — нужно проверить почему НЕ работает.

Проверить:

Реальный код create_custom_poi в service
Транзакционная модель get_db в core/db.py
Tests в test_custom_pois.py — они проходили потому что использовали db.flush() + проверка через тот же session (без real commit)

Acceptance

POST /custom-pois → row реально в DB после request close
GET возвращает созданные POI
PATCH/DELETE работают на real rows
Tests дополнить assertion с fresh session (через db.commit() + reopen)

Severity

🔴 P0 — фича из PR #257 не работает (нет persistence). Frontend PR #258 тоже не сможет работать pока этот не fixed.

Workaround

До fix — ничего нельзя сохранить через UI. Frontend useAddCustomPoi мутация будет возвращать 201, optimistic update сразу invalidate, но GET вернёт пусто → marker disappear immediately.

(QA discovery 2026-05-17 06:52 UTC, post-deploy verify PR #257)

## Симптом После deploy PR #257 (merge `8cc9447`) — post-deploy smoke: ```bash $ curl -X POST https://gendsgn.ru/api/v1/custom-pois \ -H "X-Session-Id: smoke-test-001" \ -H "Content-Type: application/json" \ -d '{"name":"Smoke test POI","weight":2.0,"lon":60.597,"lat":56.838,"notes":"deploy verify"}' HTTP/1.1 201 Created X-Session-Id: smoke-test-001 {"id":1,"user_id":"smoke-test-001","parcel_cad":null,"name":"Smoke test POI", "category":null,"weight":2.0,"lon":60.597,"lat":56.838,"notes":"deploy verify", "created_at":"2026-05-17T06:51:59.688099Z","updated_at":"2026-05-17T06:51:59.688099Z"} ``` Через 5 минут: ```sql SELECT * FROM user_custom_pois; -- (0 rows) ``` И: ```bash $ curl https://gendsgn.ru/api/v1/custom-pois?user_id=smoke-test-001 -H "X-Session-Id: smoke-test-001" [] $ curl -X PATCH https://gendsgn.ru/api/v1/custom-pois/1 -H "X-Session-Id: smoke-test-001" -d '{"weight":3.5}' HTTP 404 {"detail":"Custom POI not found"} ``` ## Root cause hypothesis `backend/app/services/site_finder/custom_pois.py` `create_custom_poi` — скорее всего отсутствует `db.commit()` после `db.execute(INSERT ... RETURNING ...)`. SQLAlchemy session возвращает RETURNING row из in-flight transaction (отсюда `id=1` в response), но при HTTP response close — transaction откатывается потому что не было explicit commit. Аналогично для `update_custom_poi`, `delete_custom_poi`. ## Fix В `backend/app/services/site_finder/custom_pois.py`: - После INSERT: `db.commit()` → потом return result - После UPDATE: `db.commit()` - После DELETE: `db.commit()` ИЛИ register dependency что использует `with db.begin():` context. См. pattern в `weight_profiles.py` create_profile. ## Дополнительно `backend/app/api/v1/custom_pois.py` endpoints могут использовать `Depends(get_db)` который возвращает `Session` (не `SessionLocal()` + `begin()`). Если `get_db` имеет yield-based context который коммитит при finally — нужно проверить почему НЕ работает. Проверить: 1. Реальный код `create_custom_poi` в service 2. Транзакционная модель `get_db` в `core/db.py` 3. Tests в `test_custom_pois.py` — они проходили потому что использовали `db.flush()` + проверка через тот же session (без real commit) ## Acceptance - [ ] `POST /custom-pois` → row реально в DB после request close - [ ] GET возвращает созданные POI - [ ] PATCH/DELETE работают на real rows - [ ] Tests дополнить assertion с **fresh** session (через `db.commit()` + reopen) ## Severity 🔴 **P0** — фича из PR #257 не работает (нет persistence). Frontend PR #258 тоже не сможет работать pока этот не fixed. ## Workaround До fix — ничего нельзя сохранить через UI. Frontend `useAddCustomPoi` мутация будет возвращать 201, optimistic update сразу invalidate, но GET вернёт пусто → marker disappear immediately. (QA discovery 2026-05-17 06:52 UTC, post-deploy verify PR #257)

lekss361 referenced this issue from a commit

2026-05-17 07:16:44 +00:00

fix(#261): add db.commit() regression tests для custom_pois service

lekss361 referenced this issue from a pull request that will close it,

2026-05-17 07:18:00 +00:00

fix(#261): custom_pois — regression tests для db.commit() в create/update/delete (P0 persistence) #262

lekss361 commented

2026-05-17 07:20:01 +00:00

Author

Owner

Re-verify 2026-05-17 07:19 UTC — НЕ воспроизводится.

POST /api/v1/custom-pois → 201 {"id":5,...}
GET /api/v1/custom-pois → [{"id":5,...}]
SELECT * FROM user_custom_pois → 1 row (id=5)

Pencil-check custom_pois.py:132 — db.commit() присутствует во всех 3 mutations (create/update/delete). См. PR #262 (regression tests).

Гипотеза первого симптома: мой first smoke POST id=1 совпал по времени с deploy PR #258 (frontend) ~06:50 UTC — возможно container restart прервал транзакцию между INSERT и commit, либо PR #257 deploy не успел применить миграцию полностью до того как PR #258 запустился. После того как оба deploys устаканились — всё работает.

Закрываю как not reproducible, оставляю PR #262 (regression tests добавлены, полезны для будущего).

**Re-verify 2026-05-17 07:19 UTC — НЕ воспроизводится.** ``` POST /api/v1/custom-pois → 201 {"id":5,...} GET /api/v1/custom-pois → [{"id":5,...}] SELECT * FROM user_custom_pois → 1 row (id=5) ``` Pencil-check `custom_pois.py:132` — `db.commit()` **присутствует** во всех 3 mutations (create/update/delete). См. PR #262 (regression tests). **Гипотеза первого симптома:** мой first smoke POST `id=1` совпал по времени с deploy PR #258 (frontend) ~06:50 UTC — возможно container restart прервал транзакцию между INSERT и commit, либо PR #257 deploy не успел применить миграцию полностью до того как PR #258 запустился. После того как оба deploys устаканились — всё работает. Закрываю как **not reproducible**, оставляю PR #262 (regression tests добавлены, полезны для будущего).

lekss361 closed this issue

2026-05-17 07:20:03 +00:00

lekss361 referenced this issue from a commit

2026-05-17 07:26:56 +00:00

Merge pull request 'fix(#261): custom_pois — regression tests для db.commit() в create/update/delete (P0 persistence)' (#262) from fix/261-custom-pois-missing-commit into main

lekss361 referenced this issue

2026-05-17 07:27:26 +00:00

fix(#261): custom_pois — regression tests для db.commit() в create/update/delete (P0 persistence) #262

lekss361 referenced this issue

2026-05-17 07:41:54 +00:00

fix(#264): apiFetch — global X-Session-Id header (custom POI scoring в UI) #266