refactor: remove unreachable catalog producer and sync orchestration (WealthLedger/backend#62) #98

2026-10-15T07:37:27+03:00

codex-bot commented

2026-10-15 07:37:27 +03:00

Closes #62

Что удалено

TbankSyncService.request и dispatch (orchestration экрана): контроллер использует только status; зависимости membership, portfolio, metadata, discovery, sources, candles убраны из сервиса.
CatalogDiscoveryService.request, refresh, drain и producer-состояние (очередь, observers, completions, logger). read больше не запускает producer; параметр observe у read удалён (вызовы из search/detail/byTicker передают только observe-ветки).
Orphan DTO: TbankSyncAcceptedDto, HourlySyncSectionDto.
Producer-only тесты в tbank-catalog-discovery.e2e-spec.ts (coalescing, late publication, forced retry, recovery, refresh-stability). Оставшиеся тесты переписаны на прямой seed через prisma, без producer.
Неактуальные записи в eslint-suppressions.json (7 строк).

Не тронуто (по границам issue)

Status/sourceStatus, public sync=observe, shared repository.read/publishSource/readSources/work, instrument schema/FK, hourly/candle/ledger owners.

Проверки

npm run verify: 40 suites, 268 tests, lint, typecheck app/test, coverage, nest build - зелёные.
npm run test:e2e: 20/20 SQLite suites, 22 tests в сумме по выводу - зелёные.
npm run test:openapi-contract: 3/3.

Ограничения

repository.reserve/publish/finish (catalog) и TbankSyncRepository.reserve/progress остаются в репозиториях: им нет вызывающих в production, но их удаление относится к follow-up, а не к этому PR, чтобы не расширять shared-owner scope.

Closes #62 ## Что удалено - `TbankSyncService.request` и `dispatch` (orchestration экрана): контроллер использует только `status`; зависимости `membership, portfolio, metadata, discovery, sources, candles` убраны из сервиса. - `CatalogDiscoveryService.request`, `refresh`, `drain` и producer-состояние (очередь, observers, completions, `logger`). `read` больше не запускает producer; параметр `observe` у `read` удалён (вызовы из `search/detail/byTicker` передают только observe-ветки). - Orphan DTO: `TbankSyncAcceptedDto`, `HourlySyncSectionDto`. - Producer-only тесты в `tbank-catalog-discovery.e2e-spec.ts` (coalescing, late publication, forced retry, recovery, refresh-stability). Оставшиеся тесты переписаны на прямой seed через prisma, без producer. - Неактуальные записи в `eslint-suppressions.json` (7 строк). ## Не тронуто (по границам issue) Status/sourceStatus, public sync=observe, shared `repository.read/publishSource/readSources/work`, instrument schema/FK, hourly/candle/ledger owners. ## Проверки - `npm run verify`: 40 suites, 268 tests, lint, typecheck app/test, coverage, nest build - зелёные. - `npm run test:e2e`: 20/20 SQLite suites, 22 tests в сумме по выводу - зелёные. - `npm run test:openapi-contract`: 3/3. ## Ограничения - `repository.reserve/publish/finish` (catalog) и `TbankSyncRepository.reserve/progress` остаются в репозиториях: им нет вызывающих в production, но их удаление относится к follow-up, а не к этому PR, чтобы не расширять shared-owner scope.

codex-bot referenced this pull request

2026-10-15 07:46:21 +03:00

test: replace late portfolio sleep with started and done barriers (WealthLedger/backend#63) #99

ksv741 force-pushed refactor/62-remove-dead-sync-orchestration from 60f4ca6b80 to 344e2291b6

2026-10-15 08:12:37 +03:00

Compare

codex-bot changed title from ~~refactor: remove unreachable catalog producer and sync screen orchestration (WealthLedger/backend#62)~~ to refactor: remove unreachable catalog producer and sync orchestration (WealthLedger/backend#62)

2026-10-15 08:17:46 +03:00

codex-bot closed this pull request

2026-10-15 08:32:34 +03:00

codex-bot reopened this pull request

2026-10-15 08:32:39 +03:00

ksv741 added 1 commit 2026-10-15 10:32:53 +03:00

refactor: remove unreachable catalog producer and sync orchestration (WealthLedger/backend#62 )

Backend verify / commits (pull_request) Successful in 31s

Details

Backend verify / security (pull_request) Successful in 11s

Details

Backend verify / verify (pull_request) Successful in 4m53s