Расширить portfolioSummary в списке счетов: деньги, структура и нереализованная доходность #138

New Issue

2026-10-26T20:59:55+03:00

codex-bot commented

2026-10-26 20:59:55 +03:00

Проблема

Карточка счёта на /accounts (WealthLedger/frontend) показывает только стоимость и число акций/облигаций. Для информативной карточки нужны деньги на счёте, структура активов и нереализованная доходность — эти данные уже есть в сохранённом snapshot портфеля, но не отдаются в GET /api/v1/brokerage-accounts.

Ожидаемое поведение

BrokerageAccountDto.portfolioSummary дополняется полями, вычисляемыми из того же сохранённого snapshot без новых обращений к T-Bank и без дополнительных запросов к БД на счёт:

cashBalances: MoneyDto[] — копия cashBalances snapshot;
structure: PortfolioStructureDto — копия structure snapshot;
unrealizedReturnByCurrency[] — по каждой валюте: absolute (сумма positions[].absoluteReturn), costBasis (сумма averagePurchasePrice × quantity по тем же позициям), percent (decimal-строка или null при нулевой базе), partial (true, если в валюте есть позиции с absoluteReturn = null);
fundPositionCount, totalPositionCount (ненулевые позиции).

Acceptance Criteria

Новые поля обязательны (required) в схеме, существующие поля и семантика не меняются; portfolioSummary = null сохраняется при отсутствии snapshot.
Денежные значения — decimal-строки без потери точности (без Number); валюты не смешиваются и не конвертируются.
Нет новых upstream-вызовов и N+1; данные из того же PortfolioService.saved.
Swagger/OpenAPI обновлён, openapi:check / contract-тест проходят; unit/e2e покрывают: пустой портфель, позиции с null-доходностью (partial), нулевую базу, несколько валют.

Связано: WealthLedger/frontend (карточка счёта).

## Проблема Карточка счёта на /accounts (WealthLedger/frontend) показывает только стоимость и число акций/облигаций. Для информативной карточки нужны деньги на счёте, структура активов и нереализованная доходность — эти данные уже есть в сохранённом snapshot портфеля, но не отдаются в GET /api/v1/brokerage-accounts. ## Ожидаемое поведение `BrokerageAccountDto.portfolioSummary` дополняется полями, вычисляемыми из того же сохранённого snapshot без новых обращений к T-Bank и без дополнительных запросов к БД на счёт: - `cashBalances: MoneyDto[]` — копия `cashBalances` snapshot; - `structure: PortfolioStructureDto` — копия `structure` snapshot; - `unrealizedReturnByCurrency[]` — по каждой валюте: `absolute` (сумма `positions[].absoluteReturn`), `costBasis` (сумма `averagePurchasePrice × quantity` по тем же позициям), `percent` (decimal-строка или null при нулевой базе), `partial` (true, если в валюте есть позиции с `absoluteReturn = null`); - `fundPositionCount`, `totalPositionCount` (ненулевые позиции). ## Acceptance Criteria - Новые поля обязательны (required) в схеме, существующие поля и семантика не меняются; `portfolioSummary = null` сохраняется при отсутствии snapshot. - Денежные значения — decimal-строки без потери точности (без Number); валюты не смешиваются и не конвертируются. - Нет новых upstream-вызовов и N+1; данные из того же `PortfolioService.saved`. - Swagger/OpenAPI обновлён, `openapi:check` / contract-тест проходят; unit/e2e покрывают: пустой портфель, позиции с null-доходностью (partial), нулевую базу, несколько валют. Связано: WealthLedger/frontend (карточка счёта).

codex-bot referenced this issue from WealthLedger/frontend

2026-10-26 21:00:39 +03:00

Информативная карточка брокерского счёта: доходность, деньги, структура #114

ksv741 referenced this issue from a commit

2026-10-27 05:05:54 +03:00

feat: extend account list portfolio summary (#138)

ksv741 referenced this issue from a commit

2026-10-27 05:05:54 +03:00

fix: mark unpriced positions as partial return (#138)

ksv741 referenced this issue from a commit

2026-10-27 05:06:07 +03:00

chore: sync brokerage accounts contract (WealthLedger/backend#138)