refactor: inject TbankAccessService directly into stock screener (#151) #153

Closed
codex-bot wants to merge 1 commits from refactor/151-screener-validation-di into develop
Owner

Summary

Удалён строковый DI-алиас 'TBANK_ACCESS'; StockScreenerService инжектит TbankAccessService из tbank.api напрямую, как соседние сервисы.

Linked Issue

Closes #151

Changes

  • instruments.module.ts: убран { provide: 'TBANK_ACCESS', useExisting: TbankAccessService }.
  • stock-screener.service.ts: убран самописный тип TbankAccess и @Inject.
  • stock-screener.service.spec.ts: jest.mock для tbank.api (паттерн соседних spec).
  • stock.controller.ts: ручной allowed-Set сохранён с комментарием — без него e2e получает 200 вместо 400, потому что глобальный whitelist pipe срезает неизвестные ключи раньше param-pipe с forbidNonWhitelisted. Вывод CQ-023 аудита не подтвердился (см. комментарий в Issue).

Verification

  • npm run verify — exit 0, 41 suites / 308 tests.
  • node test/sqlite.cjs stocks.e2e-spec.ts — 48/48.

Breaking Changes

Нет.

## Summary Удалён строковый DI-алиас `'TBANK_ACCESS'`; `StockScreenerService` инжектит `TbankAccessService` из `tbank.api` напрямую, как соседние сервисы. ## Linked Issue Closes #151 ## Changes - `instruments.module.ts`: убран `{ provide: 'TBANK_ACCESS', useExisting: TbankAccessService }`. - `stock-screener.service.ts`: убран самописный тип `TbankAccess` и `@Inject`. - `stock-screener.service.spec.ts`: `jest.mock` для `tbank.api` (паттерн соседних spec). - `stock.controller.ts`: ручной `allowed`-Set **сохранён** с комментарием — без него e2e получает 200 вместо 400, потому что глобальный `whitelist` pipe срезает неизвестные ключи раньше param-pipe с `forbidNonWhitelisted`. Вывод CQ-023 аудита не подтвердился (см. комментарий в Issue). ## Verification - `npm run verify` — exit 0, 41 suites / 308 tests. - `node test/sqlite.cjs stocks.e2e-spec.ts` — 48/48. ## Breaking Changes Нет.
codex-bot added the Kind/Refactor label 2026-11-02 07:15:56 +03:00
codex-bot added 1 commit 2026-11-02 07:15:56 +03:00
refactor: inject TbankAccessService directly into stock screener (#151)
Backend verify / commits (pull_request) Failing after 1m18s
Backend verify / security (pull_request) Skipped
Backend verify / verify (pull_request) Skipped
064a4033a2
Drop the 'TBANK_ACCESS' string alias. The raw-body whitelist in searchScreener stays: the global whitelist pipe strips unknown keys before the param pipe can reject them.
codex-bot closed this pull request 2026-11-02 07:31:45 +03:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/backend#153