maximus/Simpl-Resultat
1
0
Fork 0
Code Issues 10 Pull requests 4 Projects Releases 28 Packages Wiki Activity Actions

feat(budget) : adopter le repli multi-niveaux sur la grille Budget #289

New issue
Open
opened 2026-07-14 00:41:40 +00:00 by maximus · 0 comments
maximus commented 2026-07-14 00:41:40 +00:00
Owner
Copy link

Refs: spec-decisions-collapse-multi-niveaux.md + spec-plan-collapse-multi-niveaux.md (plan v2)

Depends on

  • #288 (socle du repli multi-niveaux — cette issue en est le consommateur)

Contexte

La grille Budget (src/components/budget/BudgetTable.tsx) est hierarchique (elle lit deja depth/is_parent/parent_id, indente sur 4 niveaux) mais fut deliberement exclue du repli par #278. Avec ~150 lignes d'un bloc, cette decision est revue : la grille adopte le meme repli multi-niveaux, meme defaut (tout replie).

Compromis assume

Seules les feuilles sont editables (BudgetTable.tsx:105 filtre !r.is_parent ; les parents l.175 ne rendent que du texte). Tout replie ⇒ la page ouvre sur 11 lignes racines sans champ de saisie visible. Accepte : page compacte, etat persiste (cout paye une seule fois), « Tout deplier » a un clic et visible d'emblee.

Correction critique heritee de /review-spec

Le plan v1 croyait la grille Budget alimentee par budgetService.buildSubGroup. Faux : elle est construite par le hook useBudget.ts:134-365, un 3e builder que le plan v1 ignorait. Son tri final (l.345-365) est par niveau (depth croissant), PAS DFS. C'est ce qui a casse l'algo v1. L'algo v2 (remontee d'ancetres, cf. #288) est order-independant → aucun changement de tri requis, mais l'invariant doit etre acte par un test.

Travail a faire

  • src/hooks/useBudget.ts — lire et documenter l'invariant d'ordre (tri par niveau, l.345-365). Avec la remontee d'ancetres, aucun changement de tri — mais l'acter par un test (la v1 s'y etait trompee). Ajouter src/hooks/useBudget.ts aux fichiers concernes.
  • BudgetTable.tsx — brancher useCollapsibleGroups ; declarer BUDGET_COLLAPSE_ACCESSORS + BUDGET_EXPANDED_KEY inline au module scope (comme ComparePeriodTable:64 et BudgetVsActualTable:35). Ne PAS creer budgetTableModel.ts : overTimeTableModel.ts existe pour de la vraie logique (groupOverTimeSections), pas pour 3 lambdas.
  • Chevron + aria-expanded sur toute ligne parente dans renderRow (l.168-205) — parents = texte pur aujourd'hui.
  • Bouton « Tout deplier / Tout replier » (reutiliser reports.collapse.expandAll/.collapseAll).
  • Remplacer le sous-total de section fait-main (l.301-311) par sumLeavesForType de budgetTableResults.ts:42 — deja teste, prouve le critere « sous-totaux sur lignes brutes ».
  • Corriger la collision localStorage : BudgetTable.tsx:21 et BudgetVsActualTable.tsx:30 declarent tous deux STORAGE_KEY = "subtotals-position" → prefixer celle du budget en budget-subtotals-position (comme ComparePeriodTable:60 prefixe deja la sienne). Effet de bord : la preference de position revient a son defaut une fois.
  • CHANGELOG.md + CHANGELOG.fr.md.

Fichiers concernes

  • src/hooks/useBudget.ts (lu + teste)
  • src/components/budget/BudgetTable.tsx
  • src/components/budget/budgetTableResults.ts (lu — sumLeavesForType:42)
  • src/i18n/locales/{fr,en}.json, CHANGELOG.md, CHANGELOG.fr.md

Tests

  • budgetTableResults : sumLeavesForType couvre deja les sous-totaux de section — le brancher prouve l'invariant.
  • Regression : replier un parent n'altere aucune valeur saisie ; totaux de section + lignes de resultat (#278) identiques replie/deplie.
  • Regression : ordre des lignes de useBudget est par niveau (documente l'invariant que la v1 a suppose a tort).
  • Regression : deplacer les sous-totaux dans la grille Budget n'affecte plus le rapport reel-vs-budget.

Review caveats

  • 🔴 3e builder (useBudget) absent du plan v1 → ajoute ici. C'est la source de l'ordre par niveau.
  • 🟡 budgetTableModel.ts : abandonne (accesseurs inline comme les 2 autres tables).
  • 🟡 Sous-total fait-main (l.301-311) duplique sumLeavesForType → remplacer.

Decisions prises en planification

  • Grille Budget : defaut replie comme les rapports (uniformite), malgre l'absence de champs visibles au 1er rendu. Tranche avec Max.
  • Persistance user_preferences (heritee de #288).

Criteres d'acceptation

  • Toute categorie parente de la grille repliable a n'importe quel niveau
  • La grille s'ouvre entierement repliee ; « Tout deplier » visible d'emblee
  • Aucune valeur de budget alteree par un repli ; totaux de section + lignes de resultat inchanges
  • Sous-totaux de section via sumLeavesForType (plus de calcul fait-main)
  • Position des sous-totaux de la grille independante de celle du rapport reel-vs-budget

Spec source

spec-plan-collapse-multi-niveaux.md (v2)

Refs: `spec-decisions-collapse-multi-niveaux.md` + `spec-plan-collapse-multi-niveaux.md` (plan **v2**) ## Depends on - #288 (socle du repli multi-niveaux — cette issue en est le consommateur) ## Contexte La grille Budget (`src/components/budget/BudgetTable.tsx`) est **hierarchique** (elle lit deja `depth`/`is_parent`/`parent_id`, indente sur 4 niveaux) mais fut **deliberement exclue du repli** par #278. Avec ~150 lignes d'un bloc, cette decision est revue : la grille adopte le meme repli multi-niveaux, meme defaut (tout replie). ## Compromis assume **Seules les feuilles sont editables** (`BudgetTable.tsx:105` filtre `!r.is_parent` ; les parents l.175 ne rendent que du texte). Tout replie ⇒ la page ouvre sur 11 lignes racines **sans champ de saisie visible**. Accepte : page compacte, etat **persiste** (cout paye une seule fois), « Tout deplier » a un clic et visible d'emblee. ## Correction critique heritee de /review-spec > Le plan v1 croyait la grille Budget alimentee par `budgetService.buildSubGroup`. **Faux** : elle est construite par le hook **`useBudget.ts:134-365`**, un 3e builder que le plan v1 ignorait. Son tri final (l.345-365) est par **niveau** (`depth` croissant), PAS DFS. C'est ce qui a casse l'algo v1. L'algo v2 (remontee d'ancetres, cf. #288) est **order-independant** → aucun changement de tri requis, mais l'invariant doit etre **acte par un test**. ## Travail a faire - [ ] `src/hooks/useBudget.ts` — **lire et documenter l'invariant d'ordre** (tri par niveau, l.345-365). Avec la remontee d'ancetres, **aucun changement de tri** — mais l'acter par un test (la v1 s'y etait trompee). Ajouter `src/hooks/useBudget.ts` aux fichiers concernes. - [ ] `BudgetTable.tsx` — brancher `useCollapsibleGroups` ; declarer `BUDGET_COLLAPSE_ACCESSORS` + `BUDGET_EXPANDED_KEY` **inline au module scope** (comme ComparePeriodTable:64 et BudgetVsActualTable:35). **Ne PAS creer `budgetTableModel.ts`** : overTimeTableModel.ts existe pour de la vraie logique (`groupOverTimeSections`), pas pour 3 lambdas. - [ ] Chevron + `aria-expanded` sur **toute** ligne parente dans `renderRow` (l.168-205) — parents = texte pur aujourd'hui. - [ ] Bouton « Tout deplier / Tout replier » (reutiliser `reports.collapse.expandAll`/`.collapseAll`). - [ ] **Remplacer le sous-total de section fait-main** (l.301-311) par `sumLeavesForType` de `budgetTableResults.ts:42` — deja teste, *prouve* le critere « sous-totaux sur lignes brutes ». - [ ] **Corriger la collision localStorage** : `BudgetTable.tsx:21` et `BudgetVsActualTable.tsx:30` declarent tous deux `STORAGE_KEY = "subtotals-position"` → prefixer celle du budget en `budget-subtotals-position` (comme ComparePeriodTable:60 prefixe deja la sienne). Effet de bord : la preference de position revient a son defaut une fois. - [ ] CHANGELOG.md + CHANGELOG.fr.md. ## Fichiers concernes - `src/hooks/useBudget.ts` (lu + teste) - `src/components/budget/BudgetTable.tsx` - `src/components/budget/budgetTableResults.ts` (lu — `sumLeavesForType:42`) - `src/i18n/locales/{fr,en}.json`, `CHANGELOG.md`, `CHANGELOG.fr.md` ## Tests - `budgetTableResults` : `sumLeavesForType` couvre deja les sous-totaux de section — le brancher *prouve* l'invariant. - Regression : replier un parent n'altere aucune valeur saisie ; totaux de section + lignes de resultat (#278) identiques replie/deplie. - Regression : ordre des lignes de `useBudget` est par niveau (documente l'invariant que la v1 a suppose a tort). - Regression : deplacer les sous-totaux dans la grille Budget **n'affecte plus** le rapport reel-vs-budget. ## Review caveats - 🔴 **3e builder (`useBudget`)** absent du plan v1 → ajoute ici. C'est la source de l'ordre par niveau. - 🟡 **`budgetTableModel.ts`** : abandonne (accesseurs inline comme les 2 autres tables). - 🟡 **Sous-total fait-main** (l.301-311) duplique `sumLeavesForType` → remplacer. ## Decisions prises en planification - Grille Budget : defaut **replie** comme les rapports (uniformite), malgre l'absence de champs visibles au 1er rendu. Tranche avec Max. - Persistance `user_preferences` (heritee de #288). ## Criteres d'acceptation - [ ] Toute categorie parente de la grille repliable a n'importe quel niveau - [ ] La grille s'ouvre entierement repliee ; « Tout deplier » visible d'emblee - [ ] Aucune valeur de budget alteree par un repli ; totaux de section + lignes de resultat inchanges - [ ] Sous-totaux de section via `sumLeavesForType` (plus de calcul fait-main) - [ ] Position des sous-totaux de la grille **independante** de celle du rapport reel-vs-budget ## Spec source spec-plan-collapse-multi-niveaux.md (v2)
maximus added this to the planned-2026-07-15-collapse-multi-niveaux milestone 2026-07-14 00:41:40 +00:00
maximus added the
status:ready
type:feature
source:human
 labels 2026-07-14 00:41:40 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
issue-291-collapse-docs
issue-290-categories-unify
issue-289-budget-collapse
issue-288-collapse-socle
fix-database-locked-serialization
issue-228-v16-guard-convertible-scope
issue-187-balance-cleanup-post-184-185
issue-188-webkit-date-picker-7-inputs
issue-180-postcss-audit-fix
issue-190-restructure-settings-pages
issue-161-smoke-scaffold
issue-171-ci-drop-push-trigger
issue-169-asset-type-balance-categories
issue-67-feedback-widget
issue-97-cartes-page
issue-96-compare-refactor
issue-51-compte-maximus-oauth
issue-46-license-commands-entitlements
issue-48-gate-auto-updates
issue-47-license-ui-card
fix/simpl-resultat-54-argon2id-pin
fix/simpl-resultat-59-bump-vite
issue-43-update-picomatch
fix/simpl-resultat-41-category-time-report-filter
fix/simpl-resultat-31-dashboard-month-dropdown
fix/simpl-resultat-29-budget-visual-adjustments
fix/simpl-resultat-23-dashboard-category-ordering
fix/simpl-resultat-16-prev-year-budget
fix/simpl-resultat-19-tooltip-transparency
fix/simpl-resultat-17-remove-piechart-title
fix/simpl-resultat-16-budget-previous-year
v0.13.0
v0.12.0
v0.11.0
v0.10.1
v0.10.0
v0.9.1
v0.9.0
v0.8.4
v0.8.3
v0.8.2
v0.8.1
v0.8.0
v0.7.4
v0.7.3
v0.7.2
v0.7.1
v0.7.0
v0.6.7
v0.6.6
v0.6.5
v0.6.4
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.2
v0.5.0
v0.4.2
v0.4.0
v0.3.11
v0.3.10
v0.3.9
v0.3.8
v0.3.7
v0.3.6
v0.3.5
v0.3.4
v0.3.3
v0.3.2
v0.3.1
v0.3.0
v0.2.12
v0.2.11
v0.2.10
v0.2.9
v0.2.8
v0.2.7
v0.2.6
v0.2.5
v0.2.4
v0.2.3
v0.2.2
v0.2.1
v0.2.0
v0.1.2
v0.1.1
v0.1.0
Labels
Clear labels   
autopilot:pending-human

PR produced by /autopilot — awaits human review/merge

  
source:analyste

Créée par le Super-Analyste

  
source:defenseur

Créée par l Escouade Défenseur

  
source:human

Créée par un humain

  
source:medic

Créée par le Medic (auto-détection)

  
status:approved

PR approuvée, prête à merger

  
status:blocked

Bloquée (question, dépendance, clarification)

  
status:in-progress

Un agent (Medic/Dev) travaille dessus

  
status:needs-clarification

Issue blocked by ambiguity — Max needs to clarify before re-running autopilot

  
status:needs-fix

Reviewer demande des corrections

  
status:ready

Analysée et documentée, prête à prendre

  
status:review

PR créée, en attente du Reviewer

  
status:triage

Nouvelle issue, pas encore analysée

  
type:bug

Correction de bug → Medic

  
type:feature

Nouvelle fonctionnalité → Dev

  
type:infra

Config, CI/CD, déploiement → Dev

  
type:refactor

Refactoring / amélioration technique → Dev

  
type:schema

Modèle de données / migrations → Dev

  
type:security

Issue de sécurité → Medic ou Dev

No labels
autopilot:pending-human
source:analyste
source:defenseur
source:human
source:medic
status:approved
status:blocked
status:in-progress
status:needs-clarification
status:needs-fix
status:ready
status:review
status:triage
type:bug
type:feature
type:infra
type:refactor
type:schema
type:security
Milestone
Clear milestone
No items
No milestone
planned-2026-07-15-collapse-multi-niveaux
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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#289
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?