[PIX-13571] fix(api): исправить TS-ошибки getObjectDetail + regression-тест GET /objects/:id #925

Closed

andrei wants to merge 2 commits from agent-fullstack/pix-13571-regression-test into master

andrei commented

2026-06-27 19:34:12 +00:00

Owner

Что сделано

Добавлены findPublicObjectDetail и findRoomsForPublicObject в public.service.ts
Перенесены Prisma-запросы из контроллера в сервис (прямой импорт prisma из контроллера убран)
Добавлен regression-тест public.objects.test.ts для GET /api/public/objects/:id
Удалены дублирующиеся функции в service (возникли при rebase поверх PIX-13568)

Зачем

PIX-13568 добавил маршрут GET /objects/:id в public.routes.ts, но оригинальный контроллер использовал прямые Prisma-вызовы без импорта — получали TS-ошибки. PIX-13571 устраняет эти ошибки через слой сервиса.

Plan тестирования

tsc --noEmit — 0 ошибок ✅
vitest run — 189/189 тестов зелёные ✅
Regression-тест public.objects.test.ts — 3 сценария (200, 404, rooms) ✅

Где могу ошибаться

Конфликт при rebase (PIX-13568 уже добавил аналогичные функции в service) разрешён удалением дублей
Выбор sequential fetch (master-версия) vs Promise.all (наша): оставили sequential, т.к. findRoomsForPublicObject принимает objectId = реальный DB id, а не slug

## Что сделано - Добавлены `findPublicObjectDetail` и `findRoomsForPublicObject` в `public.service.ts` - Перенесены Prisma-запросы из контроллера в сервис (прямой импорт `prisma` из контроллера убран) - Добавлен regression-тест `public.objects.test.ts` для GET /api/public/objects/:id - Удалены дублирующиеся функции в service (возникли при rebase поверх PIX-13568) ## Зачем PIX-13568 добавил маршрут GET /objects/:id в `public.routes.ts`, но оригинальный контроллер использовал прямые Prisma-вызовы без импорта — получали TS-ошибки. PIX-13571 устраняет эти ошибки через слой сервиса. ## Plan тестирования - `tsc --noEmit` — 0 ошибок ✅ - `vitest run` — 189/189 тестов зелёные ✅ - Regression-тест `public.objects.test.ts` — 3 сценария (200, 404, rooms) ✅ ## Где могу ошибаться - Конфликт при rebase (PIX-13568 уже добавил аналогичные функции в service) разрешён удалением дублей - Выбор sequential fetch (master-версия) vs Promise.all (наша): оставили sequential, т.к. `findRoomsForPublicObject` принимает `objectId` = реальный DB id, а не slug

andrei added 2 commits

2026-06-27 19:34:12 +00:00

test(regression): заменить placeholder на реальный тест GET /api/public/objects/:id ea12ea54a9

PIX-13571: smoke выявил 404 на production для /api/public/objects/obj-001.
Маршрут добавлен в ed01801e0 (PIX-7198) но Hetzner-сервер не получил обновление.

Добавлены 3 реальных теста вместо placeholder expect(true).toBe(true):
- 200 с данными объекта и массивом rooms
- маршрут :id зарегистрирован до :id/dashboard (не shadowed)
- 404 для несуществующего objectId

fix(api): исправить TS-ошибки в getObjectDetail — вынести Prisma-запросы в service

CI / Contracts (pull_request) Successful in 2m9s

Details

CI / Telegram Mini App (pull_request) Successful in 1m28s

Details

CI / Python SDK (pull_request) Successful in 26s

Details

CI / Secrets Scan (pull_request) Successful in 17s

Details

CI / Prisma Migrate Gate (pull_request) Successful in 1m30s

Details

PR Janitor / janitor (pull_request) Successful in 10s

Details

CI / API (pull_request) Successful in 16m48s

Details

CI / App (pull_request) Successful in 15m56s

Details

143bde6a76

PIX-13571: smoke выявил TypeScript ошибки в master:
- getObjectDetail использовал prisma/PUBLIC_OBJECT_STATUSES без импорта
- Причина: ed01801e0 добавил функцию без нужных imports

Решение:
- Добавить findPublicObjectDetail/findRoomsForPublicObject в public.service.ts
- Переписать getObjectDetail через publicService вместо прямого Prisma
- tsc --noEmit: 0 ошибок; vitest: 3/3 тестов

andrei closed this pull request

2026-06-27 20:01:18 +00:00

andrei referenced this pull request

2026-06-27 20:13:42 +00:00

[PIX-13571] fix(api): исправить TS-ошибки getObjectDetail + regression-тест GET /api/public/objects/:id #926