#145 - Bilan #7 — Documentation + ADRs - maximus/Simpl-Resultat

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: maximus/Simpl-Resultat#145

maximus commented

2026-04-25 16:09:25 +00:00

Owner

Refs: spec-decisions-bilan.md + spec-plan-bilan.md (v2 + overnight-2026-04-26)

Depends on #142, #144 (peut démarrer dès #144 mergée pour la majorité des docs ; sections price-fetching reportées tant que #143 BLOCKED)

Tâches

docs/architecture.md

Section BDD : ajouter les 5 nouvelles tables + 7 indexes + CHECK + FK RESTRICT + CHECK(currency = CAD)
Section Services : ajouter balance.service.ts (1 service unique avec 4 sections : CRUD, snapshots, transfers, prices+returns)
Section Hooks : ajouter useBalanceAccounts, useSnapshotEditor, useBalanceOverview (3 hooks scoped par page)
Section Commandes Tauri : 2 nouvelles uniquement (compute_account_return, fetch_price) — mais fetch_price reportée tant que #143 BLOCKED
Section Routing : ajouter les 3 routes /balance, /balance/snapshot, /balance/accounts

ADRs (3 fichiers, numérotation explicite)

Créer docs/adr/0008-modified-dietz-pour-rendement.md (justifie le choix vs TWR/IRR ; référence src-tauri/src/commands/return_calculator.rs)
Créer docs/adr/0009-proxy-price-fetching-via-maximus-api.md :
- Architecture proxy avec privacy considerations (header stripping, no log correlation, UA fixe simpl-resultat)
- Choix providers (Yahoo + CoinGecko) avec abstraction adapter
- Stratégie auth (Bearer activation_token, jamais en query)
- Rate limiting (client + serveur)
- Premium gating (UI + server-side)
- Note : ADR rédigée maintenant (architecture spécifiée), implémentation #143 BLOCKED
Créer docs/adr/0010-fk-restrict-balance-transfers.md :
- Justifie ON DELETE RESTRICT sur balance_account_transfers.transaction_id
- Reproductibilité des rendements Modified Dietz passés
- Trade-off : friction utilisateur (force unlink) vs intégrité historique

Guide utilisateur

Mettre à jour docs/guide-utilisateur.md : nouvelle section Bilan
- Saisie snapshot
- Ciblage transferts
- Lecture des rendements multi-horizons (3M / 1A / depuis création + non-ajusté)
- Note : section price-fetching premium ajoutée plus tard quand #143 livrée
Synchroniser i18n docs.* keys (FR + EN)

Décisions prises ce soir

ADRs numérotées 0008/0009/0010 (suite de la séquence existante 0001-0007).
Sections price-fetching d'architecture.md et guide-utilisateur.md reportées : ADR 0009 écrite maintenant, le reste quand #143 livrée.

Critères dacceptation

architecture.md reflète létat post-#144 du codebase (BDD, services, hooks, commandes, routing — sans price-fetching)
3 nouveaux ADRs publiés dans docs/adr/ (0008, 0009, 0010)
guide-utilisateur.md inclut une section Bilan navigable (sans price-fetching)
i18n docs.* synchronisé en FR et EN

Refs: spec-decisions-bilan.md + spec-plan-bilan.md (v2 + overnight-2026-04-26) Depends on #142, #144 (peut démarrer dès #144 mergée pour la majorité des docs ; sections price-fetching reportées tant que #143 BLOCKED) ## Tâches ### docs/architecture.md - [ ] Section BDD : ajouter les 5 nouvelles tables + 7 indexes + CHECK + FK RESTRICT + CHECK(currency = CAD) - [ ] Section Services : ajouter balance.service.ts (1 service unique avec 4 sections : CRUD, snapshots, transfers, prices+returns) - [ ] Section Hooks : ajouter useBalanceAccounts, useSnapshotEditor, useBalanceOverview (3 hooks scoped par page) - [ ] Section Commandes Tauri : 2 nouvelles uniquement (compute_account_return, fetch_price) — mais fetch_price reportée tant que #143 BLOCKED - [ ] Section Routing : ajouter les 3 routes /balance, /balance/snapshot, /balance/accounts ### ADRs (3 fichiers, numérotation explicite) - [ ] Créer docs/adr/0008-modified-dietz-pour-rendement.md (justifie le choix vs TWR/IRR ; référence src-tauri/src/commands/return_calculator.rs) - [ ] Créer docs/adr/0009-proxy-price-fetching-via-maximus-api.md : - Architecture proxy avec privacy considerations (header stripping, no log correlation, UA fixe simpl-resultat) - Choix providers (Yahoo + CoinGecko) avec abstraction adapter - Stratégie auth (Bearer activation_token, jamais en query) - Rate limiting (client + serveur) - Premium gating (UI + server-side) - Note : ADR rédigée maintenant (architecture spécifiée), implémentation #143 BLOCKED - [ ] Créer docs/adr/0010-fk-restrict-balance-transfers.md : - Justifie ON DELETE RESTRICT sur balance_account_transfers.transaction_id - Reproductibilité des rendements Modified Dietz passés - Trade-off : friction utilisateur (force unlink) vs intégrité historique ### Guide utilisateur - [ ] Mettre à jour docs/guide-utilisateur.md : nouvelle section Bilan - Saisie snapshot - Ciblage transferts - Lecture des rendements multi-horizons (3M / 1A / depuis création + non-ajusté) - Note : section price-fetching premium ajoutée plus tard quand #143 livrée - [ ] Synchroniser i18n docs.* keys (FR + EN) ## Décisions prises ce soir - ADRs numérotées 0008/0009/0010 (suite de la séquence existante 0001-0007). - Sections price-fetching d'architecture.md et guide-utilisateur.md reportées : ADR 0009 écrite maintenant, le reste quand #143 livrée. ## Critères dacceptation - architecture.md reflète létat post-#144 du codebase (BDD, services, hooks, commandes, routing — sans price-fetching) - 3 nouveaux ADRs publiés dans docs/adr/ (0008, 0009, 0010) - guide-utilisateur.md inclut une section Bilan navigable (sans price-fetching) - i18n docs.* synchronisé en FR et EN

maximus added this to the spec-price-fetching milestone 2026-04-25 16:09:25 +00:00

maximus added the

labels 2026-04-25 16:09:25 +00:00

maximus modified the milestone from spec-price-fetching to overnight-2026-04-26-bilan

2026-04-25 18:19:26 +00:00

maximus referenced this issue

2026-04-25 20:42:31 +00:00

feat(balance): Modified Dietz returns + transfer linking (#142) #151

maximus added

and removed

labels 2026-04-25 20:56:28 +00:00

maximus referenced this issue from a commit