Simpl-Resultat/STATE.md

STATE — Simpl'Résultat

Derniere MAJ : 2026-07-12 (#259 recadrée après /analyze : son corps décrivait un flux inexistant du Bilan, le vrai sujet est le mapping manuel des catégories custom en migration de taxonomie → réécrite status:ready, hors epic #260 ; l'epic #260 n'a donc plus de reliquat. Précédemment 2026-07-11 : M2 « rapports-parite » #277-#279 shippée sur main a982f9e — /pr-review ×3 [APPROVE #285/#286, REQUEST_CHANGES #287 → point I7 « filtrage Cartes » confirmé + fix 9ee5ad3], merge local pile de 3 + union CHANGELOG, tip cumulé validé 811 vitest + 98 Rust, réconciliation Forgejo complète. 30 commits non taggés depuis v0.12.0 — release en attente)

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-12 : #259 recadrée — l'issue décrivait un flux qui n'existe pas (rectifie les entrées des 2026-07-05 / 07-11 qui la classaient « mapping manuel compte sans similaire auto » et « reliquat de l'epic #260 » : les deux sont faux). Le corps d'origine, rédigé à chaud le 2026-07-05 pendant le run v0.12.0, avait transposé par analogie le signalement de Max vers le module Bilan sans ouvrir le fichier : il décrivait une auto-association des comptes standards aux comptes existants du profil dans StarterAccountsModal, avec « case désactivée quand aucun similaire n'est auto-identifié ». Aucune notion de mapping n'existe dans ce flux — la case est un simple « créer ce compte : oui/non », désactivée quand une collision EST détectée (StarterAccountsModal.tsx:160), soit la polarité inverse ; le commentaire l.8 cité (« the matching checkbox ») désignait « la case correspondante », pas « la case de matching ». Le vrai sujet (confirmé par Max) est la migration des catégories : computeMigrationPlan range les catégories en 2 seaux et un seul est éditable — plan.rows (seed) passe par le moteur d'appariement + type-ahead par ligne (#246/#252), tandis que plan.preserved (catégories custom) est poussé avec v1TargetId: null sans même être soumis au moteur d'appariement, rendu en <li> texte brut (StepSimulate.tsx:202-216, aucun picker) et déversé d'office par le writer sous le fourre-tout « Catégories personnalisées (migration) ». Sémantique tranchée avec Max : fusion (choisir une feuille standard réassigne tx/budgets/mots-clés/fournisseurs et fait disparaître la custom ; ne rien choisir = comportement actuel, et ne doit pas bloquer le bouton « Suivant »). Re-parentage écarté. Travail = câblage sur 3 couches (UI StepSimulate → MappingRow réutilisable tel quel ; reducer RESOLVE_ROW qui ne voit que plan.rows ; 4 retouches du writer) — la machinerie de fusion est déjà générique (buildMappingFromRows filtre les cibles nulles, étapes 3-7 bouclent sur la Map). Complexité Medium. Piège à couvrir : fusionner une custom parente ayant des enfants custom (liste preserved plate → l'enfant non résolu retombe au fourre-tout, pas d'orphelin, mais aucun test ne le garantit). Leçon process : une issue rédigée par analogie en fin de run, sans lecture du code, peut inverser la prémisse et se tromper de module — le /analyze l'a rattrapée 6 jours plus tard. (ref #259)
• 2026-07-11 : M2 « rapports-parite » (#277-#279) shippée — dernière grosse tranche de l'epic #260 « rapports uniformes ». Run /autopilot (3 workers en worktree → PRs #285/#286/#287), reviewée via 3 /pr-review adversariales parallèles (read-only, git show sans checkout) : APPROVE #285 (grille budget income-first, #278) + APPROVE #286 (BVA réel-vs-budget income-first, #277) + REQUEST_CHANGES #287 (dashboard Cartes, #279). Blocage #287 = le point I7 prédit au plan confirmé réel : getCartesSnapshot propageait accountIds aux sous-rapports top-movers/budget mais pas aux séries fetchMonthlyFlows (KPIs/sparklines/overlay 12 mois) ni fetchSeasonality → sur le dashboard (1re page à exposer le filtre par-dessus ces séries) les KPI montraient des totaux non filtrés à côté de barres/tendance filtrées. Fix (9ee5ad3) : threader accountIds? dans les 2 fetchers (source_id IN (...) paramétré via inPlaceholders, index $3/$4) + câblage getCartesSnapshot ; le CHANGELOG #279 promettait déjà le filtrage → code aligné sur la promesse, texte inchangé ; +2 assertions cartes. Merge local de la pile (3 merges --no-ff, conflit additif CHANGELOG ×2 résolu par union #277/#278/#279 — i18n non conflictuel car #277/#278 réutilisent des clés existantes, seul #279 touche les locales ; STATE non conflictuel car branches ne l'ont pas commité) → tip cumulé validé (tsc + vite build + 811 vitest + cargo check + 98 Rust) → push main a982f9e. Réconciliation Forgejo : 3 issues auto-fermées via Resolves #N, 3 PRs fermées manuellement (merges locaux non détectés merged), milestone overnight-2026-07-08-rapports-parite 3/3 fermée, branches supprimées, worktrees leftover nettoyés (gotcha recursion). Aucune migration DB (v1→v16), non taggé ([Unreleased]). Wrinkle process : le fork du skill /pr-review avait posté un APPROVE prématuré sur #287 (I7 gradé non bloquant) → superséé + commentaire stale supprimé par la passe adversariale indépendante (trace code + « 1re page à exposer le filtre » + CHANGELOG à honorer) ; valeur de la double-vérif indépendante. Suivi non bloquant : getDashboardSummary = code mort à retirer. Reste de #260 : #259 (mapping manuel compte sans similaire auto). (ref #260, #277-#279, PRs #285-#287)
• 2026-07-08 : Suite #260 — M1 « filtres-fondation » mergée (main fe9ae01), M2 prête. Planifiée via /plan-overnight+/review-spec (filtre multi-comptes accountIds[]/IN(...) tranché après analyse, révise le « sourceId singulier » initial). M1 livrée via /autopilot (5 workers séquentiels en worktree → PRs #280-#284 en pile linéaire) + /pr-review APPROVE ×5 + merge local fast-forward : hook useReportsPeriod additif (+accountIds URL sources + type ReportFilters, #272), 7 services en accountIds[] via helper paramétré sqlFilters.inPlaceholders (#273), <FilterPanel> slot temporel + multi-select « sources d'import » (libellé distinct du Bilan, #274), adoption Tendances (#275) + Compare/Budget (#276 — incl. CompareBudgetView ; câblage budget via getActualTotalsForYear, le body citait à tort getBudgetVsActualData). Aucune migration DB. Build + 791 vitest verts (728 + 63). NB : le « 4676 / 277 fichiers » vu en pré-merge était une pollution de worktrees — vitest récursait dans les 5 worktrees leftover sous .claude/worktrees/ (791 × ~6) ; nettoyés depuis, compteur réel = 791. Gotcha : nettoyer les worktrees (ou exclure .claude/worktrees/) avant de valider un tip. Réconciliation Forgejo complète (5 issues auto-fermées via Resolves #N, 5 PRs fermées, milestone fermée, branches supprimées ; worktrees nettoyés). M2 overnight-2026-07-08-rapports-parite (#277-#279) prête → /autopilot (BVA + grille budget income-first + dashboard Cartes). Notes review pour M2 : séries getCartesSnapshot (fetchMonthlyFlows/fetchSeasonality) non filtrées (I7/#279) ; grille budget câblée via getActualTotalsForYear. (ref #260, #272-#276, PRs #280-#284)
• 2026-07-07 : Tendances hiérarchiques livrées (1re slice de l'epic #260 « rapports uniformes »). Préparée via /plan-overnight (13 décisions drainées ; /review-spec 3 experts → 3 critiques résolus par l'approche 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 aurait faussé le Résultat net sur homonymes une fois le LIMIT 50 retiré), puis exécutée le même jour via /autopilot (session interactive, ciblée par nom de milestone hors fenêtre horaire — cf. mémoire feedback-plan-execute-temporal-decoupling). 4 workers en worktree → issues #262-#265, /pr-review APPROVE ×4, merge local de la pile (#266 standalone + chaîne #267→#268→#269 ; CHANGELOG auto-résolu par section, vérifié) ; tip cumulé validé build + 728 vitest → push main fe87313, réconciliation Forgejo (4 PRs fermées, 4 issues auto-fermées via Resolves #N, milestone overnight-2026-07-08-tendances-hierarchiques fermée, branches supprimées). Contenu : tendance par catégorie → tableau income-statement hiérarchique + collapse replié-par-défaut (défaut byCategory+table = fix « paie invisible »), getCategoryOverTime en arbre id-keyed sidecar via buildLeafDrivenTree<T> générique extrait de buildCompareTree (behavior-preserving), Résultat avant transferts interleavé. Aucune migration DB, non taggé ([Unreleased]). 2 polish non bloquants : perf dashboard (arbre construit non lu, #264), DRY typeOf dupliqué (#265). Reste de #260 (BVA-au-standard, budget, dashboard, filtres partagés) différé. (ref #260, #262-#265, PRs #266-#269)
• 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)

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)