feat(balance): service securities + save snapshot detaille (holdings) #212

New issue

Closed

opened 2026-06-04 01:36:42 +00:00 by maximus · 0 comments

maximus commented

2026-06-04 01:36:42 +00:00

Owner

Couche service du detail par titre : CRUD securities, ecriture transactionnelle ligne agregee + holdings, validation kind-aware, gain latent.

Fichiers concernes

src/services/balance.service.ts
src/shared/types/index.ts

Depends on

#210

Criteres d acceptation

findOrCreateSecurity (UPSERT sur symbol normalise), listSecurities, getSecurity, updateSecurity
saveSnapshotAtomic/upsertSnapshotLines : compte detailed -> ligne agregee (value=somme) + holdings dans le meme BEGIN/COMMIT (lastInsertId, recompute value)
value agregee = somme des holdings ARRONDIS AU CENT, comparaison exacte (pas de tolerance flottante) [decision ce soir]
validateDetailedSnapshot (detailed+holdings => qty/price NULL + value=somme ; detailed sans holdings = pre-pivot tolere) ; validateLineKindInvariants inchange pour simple
Garde service : kind detailed->simple rejete (erreur typee) si des holdings existent
getHoldingsForLatestSnapshot (titres+qty+book_cost reportes, qty 0 exclus) ; listHoldingsBySnapshotLine
computeUnrealizedGain (value-book_cost, % sur book_cost, garde-fou book_cost=0 -> N/A)
Tests unitaires de chaque fonction ; npm test vert

Review caveats

TECH (rouge, RESOLU): saveSnapshotAtomic n a aujourd hui aucune conscience des holdings — DELETE+INSERT holdings dans la txn, recompute value, test de rollback.
SEC (jaune, RESOLU): arrondi au cent (decision ce soir) au lieu d une tolerance 0.01 scalee.
ARCH (jaune): agregateurs existants getSnapshotTotalsBy* inchanges — ne pas les modifier.

Decisions prises ce soir

(ce soir) Invariant : arrondir chaque holding au cent avant de sommer, comparaison exacte.
(spec) book_cost saisi direct par position ; gain latent = value - book_cost.

Spec source

spec-plan-bilan-detail-titres.md (+ spec-decisions-bilan-detail-titres.md)

Couche service du detail par titre : CRUD securities, ecriture transactionnelle ligne agregee + holdings, validation kind-aware, gain latent. ## Fichiers concernes - src/services/balance.service.ts - src/shared/types/index.ts ## Depends on - #210 ## Criteres d acceptation - [ ] findOrCreateSecurity (UPSERT sur symbol normalise), listSecurities, getSecurity, updateSecurity - [ ] saveSnapshotAtomic/upsertSnapshotLines : compte detailed -> ligne agregee (value=somme) + holdings dans le meme BEGIN/COMMIT (lastInsertId, recompute value) - [ ] value agregee = somme des holdings ARRONDIS AU CENT, comparaison exacte (pas de tolerance flottante) [decision ce soir] - [ ] validateDetailedSnapshot (detailed+holdings => qty/price NULL + value=somme ; detailed sans holdings = pre-pivot tolere) ; validateLineKindInvariants inchange pour simple - [ ] Garde service : kind detailed->simple rejete (erreur typee) si des holdings existent - [ ] getHoldingsForLatestSnapshot (titres+qty+book_cost reportes, qty 0 exclus) ; listHoldingsBySnapshotLine - [ ] computeUnrealizedGain (value-book_cost, % sur book_cost, garde-fou book_cost=0 -> N/A) - [ ] Tests unitaires de chaque fonction ; npm test vert ## Review caveats - TECH (rouge, RESOLU): saveSnapshotAtomic n a aujourd hui aucune conscience des holdings — DELETE+INSERT holdings dans la txn, recompute value, test de rollback. - SEC (jaune, RESOLU): arrondi au cent (decision ce soir) au lieu d une tolerance 0.01 scalee. - ARCH (jaune): agregateurs existants getSnapshotTotalsBy* inchanges — ne pas les modifier. ## Decisions prises ce soir - (ce soir) Invariant : arrondir chaque holding au cent avant de sommer, comparaison exacte. - (spec) book_cost saisi direct par position ; gain latent = value - book_cost. ## Spec source spec-plan-bilan-detail-titres.md (+ spec-decisions-bilan-detail-titres.md)