feat(balance): audit quick wins — terminology, optional symbol, movable snapshot date #201

Merged
maximus merged 2 commits from fix/balance-quick-wins into main 2026-05-31 21:05:15 +00:00

2 commits

Author SHA1 Message Date
le king fu
9d95d2e189 feat(balance): audit quick wins — optional symbol, movable snapshot date, clearer terminology
All checks were successful
PR Check / rust (pull_request) Successful in 22m53s
PR Check / frontend (pull_request) Successful in 2m25s
Step 0 of the balance audit (docs/audit-bilan-2026-05.md):
- #198 terminology: "category" -> "type" across the balance UI + user guide
  (avoids collision with transaction categories); relabel Cash/Funds-ETF;
  gloss "snapshot" at first use. i18n FR/EN in lockstep.
- #199 make the ticker symbol optional for priced accounts (only needed for
  the price-fetch button; manual quantity x price never used it).
- #200 allow moving an existing snapshot's date: atomic date move + line
  rewrite in one transaction; collision on the target date rolls back with a
  typed snapshot_date_exists error.

No schema change. Build (tsc + vite) and balance.service tests green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-05-31 16:38:19 -04:00
le king fu
528e7374f5 docs: add balance sheet audit report (2026-05)
Two-expert critique (CPA/financial planner + fintech product designer) of
the Balance page, prioritized by severity x effort. Root finding: the flat
model conflates tax vehicle and asset class, blocking the aggregated ->
detailed progression. Defines a 3-step additive trajectory (quick wins ->
vehicle axis -> per-security holdings) and recommends superseding ADR 0012.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-05-31 16:38:19 -04:00