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

[#2] Refonte useReports : hooks par domaine + periode en query string #70

New issue
Closed
opened 2026-04-13 14:59:28 +00:00 by maximus · 0 comments
maximus commented 2026-04-13 14:59:28 +00:00
Owner
Copy link

Objectif

Splitter le hook monolithique useReports en hooks par domaine et déplacer la période vers la query string pour qu'elle soit bookmarkable et partagée entre les sous-routes. Les 4 rapports existants continuent de fonctionner pendant la transition grâce à un fallback legacy.

Spec : spec-refonte-rapports.md
Dépend de #69

Contexte

useReports aujourd'hui porte tout l'état des rapports (tab, period, monthlyTrends, categorySpending, categoryOverTime, budget*, etc.) dans un seul reducer. Avec les 4 sous-routes séparées, chaque page monte son propre hook — un god-object qui refetche tout et porte des champs hors-section devient incohérent. Cf. findings architecture du review.

Tâches

Nouveau hook partagé : useReportsPeriod

  • Créer src/hooks/useReportsPeriod.ts
  • Lit/écrit la période via useSearchParams de react-router : ?from=YYYY-MM-DD&to=YYYY-MM-DD&period=custom|monthly|yearly|ytd|all
  • Par défaut (aucun paramètre) : année civile en cours (du 1er janvier au 31 décembre de l'année courante)
  • API : { from, to, period, setPeriod, setCustomDates }

Hooks par domaine (stubs avec fetch basique)

  • Créer src/hooks/useHighlights.ts — useReducer + fetch via reportService.getHighlights (stub vide pour l'instant, sera rempli en #71)
  • Créer src/hooks/useTrends.ts — stub
  • Créer src/hooks/useCompare.ts — stub
  • Créer src/hooks/useCategoryZoom.ts — stub
  • Chacun consomme useReportsPeriod pour obtenir la période courante

Transition legacy

  • Conserver temporairement src/hooks/useReports.ts avec ses champs existants (monthlyTrends, categorySpending, categoryOverTime, budgetVsActual) mais le recâbler sur useReportsPeriod au lieu de l'état local interne
  • Ajouter un commentaire // @deprecated — sera supprimé en Issue #76 une fois tous les rapports migrés
  • Adapter src/pages/ReportsPage.tsx (version temporaire avec tabs) pour consommer useReportsPeriod + les champs legacy de useReports
  • ViewModeToggle : s'assurer que la prop storageKey permet la persistance par section (reports-viewmode-highlights, -trends, -compare, -category)

Vérifications

  • npm run build vert
  • cargo check vert
  • Les 4 rapports legacy tournent toujours sur /reports sans régression
  • L'URL reflète la période courante et est bookmarkable (copier-coller d'une URL avec ?from=...&to=... restaure l'état)

Critères d'acceptation

  • Chaque nouveau hook est autonome et consomme useReportsPeriod
  • La période est bookmarkable via l'URL
  • Aucun des 4 rapports legacy n'est cassé
  • Un comment @deprecated marque l'ancien useReports
## Objectif Splitter le hook monolithique `useReports` en **hooks par domaine** et déplacer la période vers la **query string** pour qu'elle soit bookmarkable et partagée entre les sous-routes. Les 4 rapports existants continuent de fonctionner pendant la transition grâce à un fallback legacy. Spec : `spec-refonte-rapports.md` Dépend de #69 ## Contexte `useReports` aujourd'hui porte tout l'état des rapports (`tab`, `period`, `monthlyTrends`, `categorySpending`, `categoryOverTime`, `budget*`, etc.) dans un seul reducer. Avec les 4 sous-routes séparées, chaque page monte son propre hook — un god-object qui refetche tout et porte des champs hors-section devient incohérent. Cf. findings architecture du review. ## Tâches ### Nouveau hook partagé : `useReportsPeriod` - [ ] Créer `src/hooks/useReportsPeriod.ts` - [ ] Lit/écrit la période via `useSearchParams` de react-router : `?from=YYYY-MM-DD&to=YYYY-MM-DD&period=custom|monthly|yearly|ytd|all` - [ ] Par défaut (aucun paramètre) : année civile en cours (du 1er janvier au 31 décembre de l'année courante) - [ ] API : `{ from, to, period, setPeriod, setCustomDates }` ### Hooks par domaine (stubs avec fetch basique) - [ ] Créer `src/hooks/useHighlights.ts` — `useReducer` + fetch via `reportService.getHighlights` (stub vide pour l'instant, sera rempli en #71) - [ ] Créer `src/hooks/useTrends.ts` — stub - [ ] Créer `src/hooks/useCompare.ts` — stub - [ ] Créer `src/hooks/useCategoryZoom.ts` — stub - [ ] Chacun consomme `useReportsPeriod` pour obtenir la période courante ### Transition legacy - [ ] Conserver temporairement `src/hooks/useReports.ts` avec ses champs existants (`monthlyTrends`, `categorySpending`, `categoryOverTime`, `budgetVsActual`) mais le recâbler sur `useReportsPeriod` au lieu de l'état local interne - [ ] Ajouter un commentaire `// @deprecated — sera supprimé en Issue #76 une fois tous les rapports migrés` - [ ] Adapter `src/pages/ReportsPage.tsx` (version temporaire avec tabs) pour consommer `useReportsPeriod` + les champs legacy de `useReports` - [ ] `ViewModeToggle` : s'assurer que la prop `storageKey` permet la persistance par section (`reports-viewmode-highlights`, `-trends`, `-compare`, `-category`) ### Vérifications - [ ] `npm run build` vert - [ ] `cargo check` vert - [ ] Les 4 rapports legacy tournent toujours sur `/reports` sans régression - [ ] L'URL reflète la période courante et est bookmarkable (copier-coller d'une URL avec `?from=...&to=...` restaure l'état) ## Critères d'acceptation - Chaque nouveau hook est autonome et consomme `useReportsPeriod` - La période est bookmarkable via l'URL - Aucun des 4 rapports legacy n'est cassé - Un comment `@deprecated` marque l'ancien `useReports`
maximus added this to the spec-refonte-rapports milestone 2026-04-13 14:59:28 +00:00
maximus added the
status:ready
type:refactor
source:human
 labels 2026-04-13 14:59:28 +00:00
maximus added
status:review
 and removed
status:ready
 labels 2026-04-14 18:38:14 +00:00
maximus added
status:approved
 and removed
status:review
 labels 2026-04-14 18:38:19 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
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.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   
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-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
source:analyste
source:defenseur
source:human
source:medic
status:approved
status:blocked
status:in-progress
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
spec-refonte-rapports
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#70
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?