Simpl-Resultat/STATE.md

STATE — Simpl'Résultat

Derniere MAJ : 2026-07-07 (#254 collapse/expand des sous-catégories mergé — PR #261, repli par défaut ; #258 « db-locked » fermé comme déjà corrigé en v0.10.1 ; epic #260 « rapports uniformes » : 1re slice cadrée via /plan-overnight → milestone overnight-2026-07-08-tendances-hierarchiques prête, 4 issues)

Position actuelle

v0.9.1 shippée (2026-05-10). Milestones spec-refonte-rapports, spec-refonte-seed-categories-ipc, spec-price-fetching, overnight-2026-04-26-bilan et overnight-2026-04-27-prices complétées et fermées. Maximus-api 0.3.0 LIVE en prod (api.lacompagniemaximus.com) — pubkey Ed25519 alignée, smoke test Phase B (#161) validé. Backlog actif concentré sur spec-monetisation (7/12) : activation en ligne (#53), pipeline Stripe desktop (#50, #52) et maximus-api Stripe webhooks (#135, #136).

Audit critique de la page Bilan livré (docs/audit-bilan-2026-05.md, revue CPA + UX). Étape 0 — quick wins terminologie « catégorie »→« type », symbole optionnel, date de snapshot déplaçable — mergée (#201, issues #198/#199/#200 fermées). Chantier structurel = Étapes 1-2 de l'audit (séparation véhicule fiscal × classe d'actif via vehicle_type, puis détail par titre balance_securities+holdings+book_cost, bascule agrégé→détaillé, réouverture ADR 0012). Étape 1 livrée (sprint 2026-06-01 : PRs #206-#209 mergées, milestone overnight-2026-06-01-bilan-axe-vehicule fermée 4/4) — vehicle_type nullable = enveloppe fiscale portée par le compte, catégorie = pure classe d'actif (5 classes), migrations additives v12/v13 (v1-v11 intactes), renommage via custom_label (fix bug I), axe graphique classe/enveloppe + rendements repliables persistés ; ADR 0014 Accepted, ADR 0012 Rejected. CHANGELOG sous [Unreleased] (pas encore taggé). Étape 2 livrée (merge manuel 2026-06-09 : pile de 9 PRs stackées #219-#227 mergées bottom-up, milestone overnight-2026-06-05-bilan-detail-titres à 9/10, issues #210-#218 fermées) — détail par titre via balance_securities + balance_snapshot_holdings, balance_accounts.kind ('simple'|'detailed') + detailed_since, migrations additives v14/v15/v16 (v1-v13 intactes), conversion des comptes cotés existants en détaillés 1-position (v16), service securities transactionnel, reducer holdings + dispatch account.kind, UI multi-titres (SecurityPicker), assistant détailler-un-compte (date pivot), drill-down par titre + gain latent, tests intégration/régression ; ADR 0015 Accepted. #228 mergé (PR #229, fix-forward : garde d'abort v16 scopée aux comptes convertibles via JOIN balance_categories + c.asset_type IS NOT NULL dans les 3 copies — Migration v16 / V16_SQL / V16_CORRUPT — + test régression ; CI vert car #229 ciblait main) → milestone overnight-2026-06-05-bilan-detail-titres complète 10/10 et fermée. v0.10.0 shippée (2026-06-29 : Étapes 1+2 du bilan, migrations v12→v16) puis hotfix v0.10.1 (2026-06-30, PR #230, déployé) — corrige un « database is locked » introduit en 0.10.0 (apparaissait après l'abandon d'un snapshot en cours) : tout l'accès DB est désormais sérialisé via withTransaction dans db.ts (tauri-plugin-sql = pool sqlx multi-connexions sans primitive de transaction JS → BEGIN/COMMIT en db.execute séparés pouvaient strander une transaction d'écriture = verrou zombie), appliqué aux 5 sites transactionnels ; + console de log live-update + API logInfo/logWarn/logError. 20 tables / 24 index, 16 migrations (v1→v16).

Milestone deps-security-2026-07 livrée (2026-07-01, 4/4 fermée) — 4 vulns de dépendances Defenseur remédiées en 2 PRs (#239 react-router-dom 7.18.1 ; #240 vite 6.4.3 + vitest 4.1.9) ; npm audit 5→0 — le dernier @babel/core low remédié le 2026-07-04 (override scopé @vitejs/plugin-react→@babel/core@^7.29.7, issue #241 / PR #242 mergée). Nouveau milestone spec-ci-build-optimization (2/4) ouvert (baseline #231 : cause #2 = reserveCache timeout du runner ; #234 connectivité cache VPS, #232 split workflows).

v0.11.0 shippée (2026-07-05, tag v0.11.0 → CI release Windows/Linux + JSON updater) — milestone overnight-2026-07-05-bilan-rapports-ux complétée 5/5 et fermée via un run /autopilot (5 workers séquentiels, PRs #248-#252 toutes /pr-review APPROVE, puis merge local de la pile + réconciliation Forgejo). Contenu : rapport comparable réel-vs-réel hiérarchique avec sous-totaux (#247) ; netting des transferts ciblé (catégories type transfer → SUM signé ~0, autres types inchangés au byte) dans le compare (#243) ; landing Bilan en tuiles HubReportNavCard + guard empty-state découplé + gestion comptes accessible partout (#244) ; import CSV de titres dans un snapshot détaillé (front pur, autoDetectHoldingColumns, prix flexible) (#245) ; cible de migration catégories éditable sur chaque ligne + type-ahead CategoryCombobox feuilles-seulement (#246). Aucune migration DB (v1→v16 inchangées), 20 tables / 24 index, 16 migrations. Point de suivi non bloquant : nets de transfert signés traversent du code UI compare pensé pour des magnitudes positives (mord seulement les transferts déséquilibrés à une jambe).

v0.12.0 shippée (2026-07-05, tag v0.12.0 → CI release #311) — recadrage de #243 (montants de transfert inclus dans le comparable) par Max en « analyse de résultat » : deux rapports comparables refondus en états de résultat (income statement). Le compare réel-vs-réel (#253) et le tableau de tendance par catégorie (#256) affichent le revenu, un Résultat avant transferts (revenus − dépenses) et un Résultat net (après transferts), sections ordonnées revenu→dépense→transfert ; les rapports comparables ouvrent par défaut sur le mois complet précédent (#253). Modules purs extraits et testés (compareResults.ts, overTimeResults.ts) ; 2 blocages /pr-review corrigés (fuite du revenu dans les top-movers Cartes ; garde StrictMode sur le sync du mois de référence). Livré en 2 issues/PRs stackées #255→#257 (merge local + réconciliation Forgejo, branches supprimées), 701 vitest verts, aucune migration DB (v1→v16 inchangées), 20 tables / 24 index. Backlog ouvert au passage : #258 (db-locked proposeStarterAccounts) → fermé le 2026-07-07 (déjà corrigé v0.10.1, pas de nesting) ; #254 (collapse/expand) → livré le 2026-07-07 (PR #261) ; #259 (mapping manuel d'un compte standard sans similaire auto) et #260 (epic « rapports uniformes » — tous les rapports sur le modèle income-statement) ouverts.

Decisions recentes

• 2026-07-07 : Milestone overnight-2026-07-08-tendances-hierarchiques préparée via /plan-overnight (1re slice de l'epic #260 « rapports uniformes ») — 4 issues status:ready : #262 (défaut Trends → byCategory+table, insurance paie, 2 one-liners), #263 (extraire un constructeur d'arbre feuille-driven générique depuis buildCompareTree, behavior-preserving, tests compare = garde), #264 (getCategoryOverTime → arbre tendances sidecar id-keyed + computeOverTimeResults), #265 (CategoryOverTimeTable hiérarchique + collapse replié-par-défaut + Résultat interleavé). /review-spec (3 experts) verdict 🟡 : 3 critiques résolus par le sidecar — garder le pivot par mois pour chart + dashboard (DashboardPage:142 = consommateur partagé oublié), ajouter un arbre clé-par-id pour le tableau (pivot clé-par-nom = Résultat net faux sur homonymes une fois le LIMIT 50 retiré). Ordre autopilot #262∥#263→#264→#265. BVA-au-standard/budget/dashboard/filtres partagés = reste de #260 différé. Décisions drainées : 13. Spec scratch spec-plan-tendances-hierarchiques.md. (ref #260, milestone overnight-2026-07-08-tendances-hierarchiques)

• 2026-07-07 : #254 (collapse/expand des sous-catégories) livré + #258 (db-locked) fermé. #254 : chevron par catégorie parente + « tout replier/déplier » sur les 2 rapports comparables hiérarchiques (ComparePeriodTable réel-vs-réel, BudgetVsActualTable réel-vs-budget) ; repli par défaut (contrainte remontée de #260), état persisté par rapport en localStorage. Repli purement visuel — sous-totaux/totaux/résultats calculés sur les lignes brutes, jamais les visibles (tracé en /pr-review). Module collapsibleRows.ts (pur) + hook useCollapsibleGroups.ts + 12 tests → 713 vitest, aucune migration DB. CategoryOverTimeTable exclu (liste plate top-N, rien à replier) → relève de l'epic #260. Livré via sous-agent en worktree isolé + /pr-review APPROVE + merge API (single PR, pas de conflit i18n/CHANGELOG) → PR #261, #254 auto-fermée. #258 fermé comme déjà corrigé en v0.10.1 (8cb7e53, présent v0.11.0/v0.12.0) : la piste « BEGIN imbriqué » du body était une mauvaise lecture — withTransaction (db.ts) n'émet aucun BEGIN, proposeStarterAccountsInTransaction:720 est le seul (même pattern que saveSnapshotAtomic/upsertSnapshotLines qui ne deadlockent jamais) ; smell latent noté (garde inTransaction = booléen global, pas un compteur → requête simple concurrente pendant une transaction bypasse le verrou). Prochaine étape : /plan-overnight de l'epic #260. (ref #254/#258, PR #261)

• 2026-07-05 : v0.12.0 shippée (tag v0.12.0 → CI release #311). Compare réel-vs-réel (#253) + tableau de tendance par catégorie (#256) refondus en analyse de résultat : revenu affiché, Résultat avant transferts (revenus − dépenses) + Résultat net (après transferts), sections revenu→dépense→transfert ; rapports comparables par défaut au mois complet précédent (#253). Origine : #243 (montants de transfert inclus dans le comparable) recadré par Max en « voir si je suis over/under ». Livré en 2 issues/PRs stackées #255→#257 (merge local + réconciliation Forgejo, PRs fermées, branches supprimées). Modules purs compareResults.ts/overTimeResults.ts testés ; 2 blocages /pr-review corrigés (fuite du revenu dans les top-movers Cartes via COMPARE_DELTA_SQL élargi income+transfer → filtre expense-only + régression ; garde StrictMode value-change sur le sync mois-réf, syncReferenceOnPeriodChange pur). 701 vitest verts, aucune migration DB. Ouvert au passage : #258 (db-locked proposeStarterAccounts, probable BEGIN imbriqué dans withTransaction), #259 (mapping manuel compte sans similaire), #254 (collapse/expand). Suivi différé : aligner le placement « Résultat avant transferts » de CategoryOverTimeTable (bas) sur le compare (interleavé). (ref #253/#256, PRs #255/#257)

• 2026-07-05 : Run /autopilot sur overnight-2026-07-05-bilan-rapports-ux (préparée la veille via /analyze #243-#246 + split #247) → 5 issues livrées, PRs #248-#252 toutes /pr-review APPROVE. Merge local de la pile (l'API merge échoue sur les conflits i18n/CHANGELOG garantis — 4 fichiers partagés fr.json/en.json/CHANGELOG.md/CHANGELOG.fr.md, voir mémoire reference-shared-conflict-files) ; réconciliation Forgejo (PRs fermées manuellement, issues auto-fermées via Resolves #N, milestone fermée) ; release v0.11.0 taggée. Validation locale du tip cumulé : build + 679 vitest + cargo verts (check.yml ne tourne ni sur push main ni sur les bases non-main type #249). Polish #252 post-review : picker migration feuilles-seulement + helper comboboxCategoriesForTarget testé. (ref #243-#247, PRs #248-#252)

• 2026-07-04 : Override @babel/core mergé (issue #241, PR #242 — review /pr-review APPROVE + CI verte rust/frontend) — clôt le reliquat @babel/core low du sprint deps-security-2026-07. GHSA-4x5r-pxfx-6jf8 (arbitrary file read via sourceMappingURL, CVSS 3.2, dev/build-only via @vitejs/plugin-react) : npm audit fix inopérant (version pinnée par le parent → empile une copie sans résoudre), fix = overrides scopé {"@vitejs/plugin-react":{"@babel/core":"^7.29.7"}} → patch 7.x non-breaking (7.29.0→7.29.7), arbre 109→109 (pas d'explosion), npm audit 1→0. Build + 635 vitest verts. Détecté via /analyse-vulnerabilite — rapport VPS --fresh inaccessible (auth navigateur Tailscale SSH en attente, jamais reçu le lien) → npm audit live utilisé ; rapport local périmé (2026-05-06) portait 1 faux positif secrets (« Generic Token Assignment » sur balance.service.ts, contenu disparu depuis la réécriture Bilan). Voir mémoire reference-analyse-vulnerabilite-inflation-transitive (ref #241, PR #242)

• 2026-07-01 : Sprint deps-security-2026-07 livré (4/4, milestone fermée) via /sprint — 4 vulns Defenseur (2026-06-30) remédiées en 2 PRs séquentielles (fichier partagé package.json → pas de worktrees parallèles). PR A #239 : react-router-dom 7.13→7.18.1 ferme #235 (racine, 7 advisories dont turbo-stream RCE) + #238 (0 CVE propre, héritage transitif — 1 bump du parent suffit, npm audit viaParents). PR B #240 : vite 6.4.2→6.4.3 (#236) + vitest 4.0.18→4.1.9 (#237), restés en 6.x/4.x (majors 8.x/latest écartés), 635 vitest verts (bump vitest 4.0→4.1 sans régression). npm audit 5→1 ; reste @babel/core low volontairement hors scope (npm audit fix ajoute 77 paquets sans résoudre, pinné par parent → override scopé dédié à faire). Surface réelle faible (app Tauri desktop, pas de serveur SSR/RSC). Voir mémoire reference-analyse-vulnerabilite-inflation-transitive (ref #235-#238, PRs #239/#240)

• 2026-06-30 : hotfix v0.10.1 shippé + déployé (PR #230) — corrige le « database is locked » 0.10.0 (après abandon d'un snapshot en cours). Cause racine : tauri-plugin-sql charge SQLite via un pool sqlx multi-connexions (max 10) sans primitive de transaction JS → db.execute("BEGIN")/"COMMIT" en appels séparés pouvaient frapper des connexions différentes → transaction d'écriture zombie. Fix : sérialisation FIFO de tout l'accès DB dans db.ts ; withTransaction() tient le verrou sur tout le BEGIN..COMMIT ; appliqué aux 5 sites (saveSnapshotAtomic, upsertSnapshotLines, proposeStarterAccounts, applyKeywordWithReassignment, applyMigration) par extraction de helper en place ; db.test.ts (régression). + console de log : getLogs retourne un snapshot immuable (live-update) + API logInfo/logWarn/logError. 635 vitest, CI vert. Voir mémoire reference-tauri-sql-transactions (ref #230)

• 2026-06-29 : #228 mergé (PR #229) — garde d'abort migration v16 scopée aux comptes convertibles (JOIN balance_categories + c.asset_type IS NOT NULL dans Migration v16 + V16_SQL + V16_CORRUPT). Un compte simple portant un symbole résiduel (recat priced→simple) avec lignes qty-NULL ne fait plus aborter v16 → l'app redémarre. Test régression migration_v16_leaves_simple_account_with_residual_symbol_intact. CI vert (#229 → main, contrairement à la pile #219-#227). Milestone overnight-2026-06-05-bilan-detail-titres complète 10/10, fermée. Reste : tag release 0.10.0 (ref #228)

• 2026-06-09 : Etape 2 bilan « detail par titre » livree — pile de 9 PRs stackees #219-#227 mergee bottom-up (merge manuel via API Forgejo, pas /fix-issue ; retarget auto base->main entre chaque), milestone overnight-2026-06-05-bilan-detail-titres 9/10, issues #210-#218 fermees. balance_securities + balance_snapshot_holdings, account.kind ('simple'|'detailed') + detailed_since, migrations additives v14/v15/v16 (v1-v13 intactes), conversion comptes cotes -> detailles 1-position, service securities transactionnel, reducer holdings + dispatch account.kind, UI SecurityPicker + wizard date-pivot, drill-down + gain latent, ADR 0015 Accepted. Validation locale pre-merge (build + 627 vitest + cargo check + 97 Rust) car check.yml ne couvre pas les PRs vers bases non-main. Reste #228 (fix-forward garde v16) avant tag release (ref #210-#218, #219-#227)

• 2026-06-01 : Etape 1 bilan « axe vehicule » livree via /sprint (4 issues, PRs #206-#209 mergees, milestone fermee). vehicle_type nullable = enveloppe fiscale sur le compte ; categorie = pure classe d'actif (5) ; migrations additives v12/v13 (v1-v11 intactes, checksum SHA-384) ; reclass ex-CELI/REER -> Autres ; custom_label (fix bug I renommage) ; toggle graphique classe/enveloppe + rendements repliables persistes. ADR 0014 Accepted, ADR 0012 Rejected. Etape 2 (detail par titre) reportee (ref #202-#205, #206-#209)## Blockers actifs

• #135 / #136 — maximus-api Stripe webhooks license auto-generate (BLOCKED par maximus-api Phase 2)

• #53 — online activation + machine limit enforcement (status:needs-fix)

• #50 / #52 — Stripe integration desktop + purchase page (status:ready, design en attente)