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

[#8] Polish, tests, docs, changelog, nettoyage legacy #76

New issue
Closed
opened 2026-04-13 15:08:39 +00:00 by maximus · 0 comments
maximus commented 2026-04-13 15:08:39 +00:00
Owner
Copy link

Objectif

Dernière issue du milestone : tests, validation manuelle, documentation, changelog, nettoyage final de useReports legacy.

Spec : spec-refonte-rapports.md
Dépend de : #71, #72, #73, #74, #75

Tâches

Tests automatisés (vitest)

  • Smoke test getHighlights — cas nominal + profil vide
  • Smoke test getCompareMonthOverMonth — avec et sans données mois précédent
  • Smoke test getCategoryZoom avec fixture cyclique (catégories A→B→A) — doit terminer grâce à la CTE bornée depth < 5
  • Tests AddKeywordDialog : validation longueur (< 2 → bloque, > 64 → bloque, whitespace → bloque, 2–64 → passe)

Validation manuelle des flows utilisateur

  • Hub /reports : panneau faits saillants + 4 cartes
  • Navigation vers chaque sous-rapport, retour au hub
  • Toggle chart/table sur chacun des 4 rapports
  • Période bookmarkable : /reports/trends?from=2025-01-01&to=2025-12-31 restaure l'état ; copier l'URL, fermer/rouvrir la fenêtre, coller → même état
  • Persistance localStorage des préférences viewMode par section après redémarrage de l'app
  • Clic droit → AddKeywordDialog → preview → apply → vérifier que le keyword apparaît dans /categories et que les transactions sont recatégorisées
  • Comportement « mot-clé déjà existant » avec dialog de remplacement

Nettoyage final du hook legacy

  • Supprimer définitivement les champs legacy de useReports laissés en backward-compat pendant #70 (maintenant que tous les rapports consomment les nouveaux hooks)
  • Si useReports n'est plus utilisé nulle part, supprimer le fichier complètement
  • Vérifier qu'aucun import de useReports ne subsiste

Documentation

  • Mettre à jour docs/architecture.md section Rapports :
    • Nouveaux hooks par domaine (useReportsPeriod, useHighlights, useTrends, useCompare, useCategoryZoom)
    • Nouveaux endpoints de reportService (getHighlights, getCompareMoM, getCompareYoY, getCategoryZoom)
    • Suppression du pivot
    • Structure des composants src/components/reports/ (plat avec préfixes)
  • Mettre à jour docs/guide-utilisateur.md avec le nouveau flow (hub + 4 rapports, édition contextuelle mot-clé via clic droit). Mettre à jour aussi les clés i18n docs.* correspondantes
  • Créer docs/adr/NNNN-refonte-rapports.md couvrant :
    • Pourquoi le pivot a été supprimé (usage réel = zoom catégorie à 90%)
    • Passage hub + 4 sous-routes bookmarkables
    • Split useReports → hooks par domaine
    • Édition contextuelle des mots-clés avec garanties de sécurité

Changelog

  • Ajouter sous ## [Unreleased] dans CHANGELOG.md :
    • Added : 4 new reports (highlights, trends, compare, category), contextual keyword editing via right-click, sparklines, donut chart, bookmarkable period via query string
    • Changed : Reports page refactored into hub + 4 sub-routes, useReports split into per-domain hooks
    • Removed : Dynamic pivot table
  • Ajouter les mêmes entrées en français dans CHANGELOG.fr.md :
    • Ajouté : 4 nouveaux rapports (faits saillants, tendances, comparables, analyse par catégorie), édition contextuelle des mots-clés via clic droit, sparklines, graphique en anneau, période bookmarkable via query string
    • Modifié : page Rapports refondue en hub + 4 sous-pages, useReports splitté en hooks par domaine
    • Supprimé : tableau croisé dynamique

Vérifications finales

  • npm run build vert
  • cargo check vert
  • npm test vert (tous les nouveaux tests passent)
  • grep -ri "DynamicReport\|pivotConfig\|reports\.pivot\|normalizeString\|_plural" src/ retourne vide
  • La CI Forgejo (check.yml) passe sur la branche finale

Critères d'acceptation

  • Tous les flows manuels listés passent sans erreur
  • CI verte
  • useReports legacy supprimé
  • Docs + ADR + changelog à jour en FR et EN
## Objectif Dernière issue du milestone : tests, validation manuelle, documentation, changelog, nettoyage final de `useReports` legacy. Spec : `spec-refonte-rapports.md` Dépend de : #71, #72, #73, #74, #75 ## Tâches ### Tests automatisés (vitest) - [ ] Smoke test `getHighlights` — cas nominal + profil vide - [ ] Smoke test `getCompareMonthOverMonth` — avec et sans données mois précédent - [ ] Smoke test `getCategoryZoom` avec **fixture cyclique** (catégories A→B→A) — doit terminer grâce à la CTE bornée `depth < 5` - [ ] Tests `AddKeywordDialog` : validation longueur (< 2 → bloque, > 64 → bloque, whitespace → bloque, 2–64 → passe) ### Validation manuelle des flows utilisateur - [ ] Hub `/reports` : panneau faits saillants + 4 cartes - [ ] Navigation vers chaque sous-rapport, retour au hub - [ ] Toggle chart/table sur chacun des 4 rapports - [ ] Période bookmarkable : `/reports/trends?from=2025-01-01&to=2025-12-31` restaure l'état ; copier l'URL, fermer/rouvrir la fenêtre, coller → même état - [ ] Persistance localStorage des préférences `viewMode` par section après redémarrage de l'app - [ ] Clic droit → AddKeywordDialog → preview → apply → vérifier que le keyword apparaît dans `/categories` et que les transactions sont recatégorisées - [ ] Comportement « mot-clé déjà existant » avec dialog de remplacement ### Nettoyage final du hook legacy - [ ] Supprimer définitivement les champs legacy de `useReports` laissés en backward-compat pendant #70 (maintenant que tous les rapports consomment les nouveaux hooks) - [ ] Si `useReports` n'est plus utilisé nulle part, supprimer le fichier complètement - [ ] Vérifier qu'aucun import de `useReports` ne subsiste ### Documentation - [ ] Mettre à jour `docs/architecture.md` section Rapports : - Nouveaux hooks par domaine (`useReportsPeriod`, `useHighlights`, `useTrends`, `useCompare`, `useCategoryZoom`) - Nouveaux endpoints de `reportService` (`getHighlights`, `getCompareMoM`, `getCompareYoY`, `getCategoryZoom`) - Suppression du pivot - Structure des composants `src/components/reports/` (plat avec préfixes) - [ ] Mettre à jour `docs/guide-utilisateur.md` avec le nouveau flow (hub + 4 rapports, édition contextuelle mot-clé via clic droit). Mettre à jour aussi les clés i18n `docs.*` correspondantes - [ ] Créer `docs/adr/NNNN-refonte-rapports.md` couvrant : - Pourquoi le pivot a été supprimé (usage réel = zoom catégorie à 90%) - Passage hub + 4 sous-routes bookmarkables - Split `useReports` → hooks par domaine - Édition contextuelle des mots-clés avec garanties de sécurité ### Changelog - [ ] Ajouter sous `## [Unreleased]` dans **`CHANGELOG.md`** : - **Added** : 4 new reports (highlights, trends, compare, category), contextual keyword editing via right-click, sparklines, donut chart, bookmarkable period via query string - **Changed** : Reports page refactored into hub + 4 sub-routes, `useReports` split into per-domain hooks - **Removed** : Dynamic pivot table - [ ] Ajouter les mêmes entrées en français dans **`CHANGELOG.fr.md`** : - **Ajouté** : 4 nouveaux rapports (faits saillants, tendances, comparables, analyse par catégorie), édition contextuelle des mots-clés via clic droit, sparklines, graphique en anneau, période bookmarkable via query string - **Modifié** : page Rapports refondue en hub + 4 sous-pages, `useReports` splitté en hooks par domaine - **Supprimé** : tableau croisé dynamique ### Vérifications finales - [ ] `npm run build` vert - [ ] `cargo check` vert - [ ] `npm test` vert (tous les nouveaux tests passent) - [ ] `grep -ri "DynamicReport\|pivotConfig\|reports\.pivot\|normalizeString\|_plural" src/` retourne vide - [ ] La CI Forgejo (`check.yml`) passe sur la branche finale ## Critères d'acceptation - Tous les flows manuels listés passent sans erreur - CI verte - `useReports` legacy supprimé - Docs + ADR + changelog à jour en FR et EN
maximus added this to the spec-refonte-rapports milestone 2026-04-13 15:08:39 +00:00
maximus added the
status:ready
type:refactor
source:human
 labels 2026-04-13 15:08:39 +00:00
maximus added
status:approved
 and removed
status:ready
 labels 2026-04-14 19:30:26 +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#76
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?