[Bug] Кэш брокерского счёта дважды входит в totals/structure; неверный знак % у коротких позиций #140

New Issue

2026-10-28T07:53:06+03:00

codex-bot commented

2026-10-28 07:53:06 +03:00

Problem

totalsByCurrency (tbank-portfolio.mapper.ts) суммирует cash из getPositions().money и все позиции портфеля, включая CURRENCY (RUB000UTSTOM, USD800UTSTOM), которые являются тем же кэшем. portfolioStructure кладёт CURRENCY в OTHER, а кэш — в CASH. Завышенные totals сохраняются в TbankPortfolioValuePoint и складываются в сводке «все счета».
- dev.db, счёт 2197225275: фонд 120.24 + cash 1.25 → сохранено 122.74 вместо 121.49.
- brokerage-accounts.service.ts уже исключает CURRENCY ("must not duplicate the separately returned cash balances"), totals/structure — нет.
unrealizedReturnByCurrency.percent делит на cost basis avgPrice × quantity, который у короткой позиции отрицателен: qty −2, avg 100, current 110 → absolute −20, cost −200, percent +10 (убыток показан как прибыль). Лонг и шорт одной валюты взаимно гасят cost basis.

Acceptance Criteria

totalsByCurrency = сумма currentValue не-CURRENCY позиций + Cash Balance по валюте.
structure не содержит CURRENCY позиций в OTHER; кэш учитывается только в CASH.
percent = null, если в валюте есть позиция с quantity ≤ 0 (как в instrument-position.ts).
Regression-тесты: currency position + cash; короткая позиция.

Out of scope

Уже сохранённые точки TbankPortfolioValuePoint (решение отдельно).
Blocked cash, НКД, FX-конвертация.
Уточнение формулировки specs/portfolio-view («сумма позиций и Cash Balance») — follow-up в specs; связано с WealthLedger/frontend#59.

## Problem 1. `totalsByCurrency` (`tbank-portfolio.mapper.ts`) суммирует cash из `getPositions().money` **и** все позиции портфеля, включая `CURRENCY` (`RUB000UTSTOM`, `USD800UTSTOM`), которые являются тем же кэшем. `portfolioStructure` кладёт `CURRENCY` в `OTHER`, а кэш — в `CASH`. Завышенные totals сохраняются в `TbankPortfolioValuePoint` и складываются в сводке «все счета». - dev.db, счёт 2197225275: фонд 120.24 + cash 1.25 → сохранено 122.74 вместо 121.49. - `brokerage-accounts.service.ts` уже исключает `CURRENCY` ("must not duplicate the separately returned cash balances"), totals/structure — нет. 2. `unrealizedReturnByCurrency.percent` делит на cost basis `avgPrice × quantity`, который у короткой позиции отрицателен: qty −2, avg 100, current 110 → absolute −20, cost −200, percent `+10` (убыток показан как прибыль). Лонг и шорт одной валюты взаимно гасят cost basis. ## Acceptance Criteria - [ ] `totalsByCurrency` = сумма currentValue не-`CURRENCY` позиций + Cash Balance по валюте. - [ ] `structure` не содержит `CURRENCY` позиций в `OTHER`; кэш учитывается только в `CASH`. - [ ] `percent` = `null`, если в валюте есть позиция с quantity ≤ 0 (как в `instrument-position.ts`). - [ ] Regression-тесты: currency position + cash; короткая позиция. ## Out of scope - Уже сохранённые точки `TbankPortfolioValuePoint` (решение отдельно). - Blocked cash, НКД, FX-конвертация. - Уточнение формулировки `specs/portfolio-view` («сумма позиций и Cash Balance») — follow-up в specs; связано с WealthLedger/frontend#59.

ksv741 referenced this issue from a commit

2026-10-28 08:00:23 +03:00

fix: count brokerage cash once and skip short-position return percent (WealthLedger/backend#140)

ksv741 referenced this issue from a commit

2026-10-28 08:00:23 +03:00

fix: reset portfolio value points inflated by double-counted cash (WealthLedger/backend#140)

codex-bot referenced a pull request that will close this issue

2026-10-28 08:00:31 +03:00

fix: count brokerage cash once in portfolio totals (WealthLedger/backend#140) #141

codex-bot referenced this issue from WealthLedger/specs

2026-10-28 08:08:30 +03:00

[Spec] portfolio-view: CURRENCY-позиции дублируют Cash Balance в totals/structure #60