🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• Absence totale de tests automatisés : la vérification manuelle est insuffisante pour un changement affectant 10 requêtes et la fusion de données métier critiques du tableau de bord
• Risque critique de dégradation non gracieuse avec Promise.all : si une seule requête comme getTicketEventsByUserQuery échoue, tout le tableau de bord devient inutilisable au lieu d'afficher les données partielles disponibles
• Multiplication des requêtes réseau simultanées : 10 requêtes parallèles par chargement de tableau de bord risquent de surcharger le serveur GraphQL en cas d'utilisation concurrente par plusieurs utilisateurs
• Types any persistants dans defineTicketEvents et les fonctions de formatage : aucune amélioration de la sécurité type malgré l'opportunité d'une refactorisation complète
• Aucune métrique de performance avant et après documentée : impossible de valider le retour sur investissement métier de cette optimisation sans données comparatives mesurables

• Absence totale de tests automatisés - Risque élevé pour une refactorisation touchant 12+ requêtes et le flux de données principal du dashboard. La validation repose uniquement sur des tests manuels.
• Types 'any' omniprésents - Les fonctions defineTicketEvents, defineTaskEvents, defineConvocationEvents utilisent des tableaux 'any[]', rendant le code fragile et les futures modifications risquées en l'absence de typage TypeScript.
• Promise.all sans gestion d'erreur individuelle - Une seule requête échouée fait échouer tout le dashboard (fail-fast) au lieu de dégrader gracieusement les sections concernées. C'est un choix défendu mais qui impacte la résilience.
• Pollution du diff par le formatage - Les changements automatiques de Prettier (~40% du diff) masquent les véritables changements fonctionnels et compliquent la revue de code et le débogage régressif.

• RÉGRESSION CRITIQUE - Typage TypeScript perdu dans queries.tsx (chunks [1][2][4]) : les nouvelles fonctions ({ userId, endDate }) remplacent ({ userId, regieId, endDate }: getEventsByUserId) sans annotation de type, rendant les erreurs d'arguments silencieuses à la compilation
• RISQUE DE RÉSILIENCE - Promise.all dans dashboard.store.tsx : une seule requête défaillante sur 12+ provoque l'échec total du chargement. Alternative recommandée : Promise.allSettled avec agrégation des résultats réussis pour permettre un affichage partiel
• DUPLICATION DE SCHÉMA - Les structures de filtres GraphQL (filters: { and: [...] }) et sélections de champs sont dupliquées à travers 12+ requêtes. Un changement de schéma AGS nécessitera des modifications synchronisées dans 12+ emplacements
• TYPES ANY NON RÉSOLUS - defineTicketEvents (chunk [6]) conserve tickets: any[] et t: any, empêchant la détection d'erreurs sur les accès aux propriétés (ticket?.attributes?.ticketClass, ticket?.attributes?.message)
• ABSENCE DE TESTS - Aucun test automatisé pour une refactorisation critique du chargement du tableau de bord. La vérification manuelle proposée est insuffisante pour garantir la non-régression sur 12+ requêtes parallèles

• Bug de copier-coller dans queries.tsx chunk 3 : getTasksEventsByPpeIdQuery filtre par assignedCollaborators au lieu de ppe, régression fonctionnelle non détectable sans test unitaire sur la structure des filtres GraphQL
• Aucun test automatisé pour valider l'équivalence des données entre la requête monolithique et les 12 requêtes divisées, risque de régression silencieuse sur les événements affichés
• Promise.all sans gestion d'erreur partielle dans dashboard.store.tsx, un seul échec réseau fait tomber tout le dashboard, aucun test de résilience pour ce scénario critique
• Interpolations de chaînes non échappées userId et ppeId dans les requêtes GraphQL, vulnérabilité d'injection, aucun test de sécurité
• Interfaces TypeScript supprimées sans remplacement, perte de sécurité de type sur les paramètres des 12 nouvelles fonctions exportées

• Risque critique de résilience dans dashboard.store.tsx : Promise.all provoque un échec total si UNE seule des 12+ requêtes échoue. Promise.allSettled avec filtrage des résultats valides permettrait une dégradation gracieuse, pattern essentiel pour un tableau de bord.
• Duplication structurelle dans queries.tsx : les filtres and: [{ user: { id: { eq: userId } } }] et les blocs de champs de réponse se répètent dans getConvocationEventsByUserQuery, getTenueAgEventsByUserQuery, etc. Les fragments GraphQL auraient éliminé cette duplication et centralisé l'évolution du schéma.
• Interfaces de paramètres incohérentes : getTicketEventsByUserQuery(regieId) vs getConvocationEventsByUserQuery(userId, endDate) vs getTenueAgEventsByPpeIdQuery(ppeId, endDate). Cette asymétrie ad hoc complique la maintenance et suggère un manque de conception systématique.
• Import de 13 fonctions depuis queries.tsx dans le store : signal de faible cohésion du module queries. Une organisation par domaine (userQueries, ppeQueries) améliorerait la navigabilité.
• Aucun test automatisé visible pour le comportement de Promise.all en cas d'échec partiel, la cohérence des données agrégées après concaténation, ou la validité des requêtes GraphQL générées dynamiquement.

Tour 2 : Préoccupations et questions

• RÉGRESSION DONNÉES PPE : Bug copier-coller queries.tsx chunk 3 - filtre assignedCollaborators au lieu de ppe = données incorrectes pour utilisateurs PPE
• PANNE TOTALE AU LIEU DE PARTIELLE : Promise.all dashboard.store.tsx chunk 8 = écran d'erreur complet si 1 requête échoue sur 12+
• ROI NON MESURÉ : Aucune métrique performance avant/après, problème utilisateur non documenté
• DETTE NETTE POSITIVE : +15h ajoutée vs 2h réduite, rapport valeur/coût négatif
• RISQUE SURCHARGE SERVEUR : 10+ requêtes simultanées x utilisateurs concurrents = potentiellement l'effet inverse recherché

• Bug copier-coller getTasksEventsByPpeIdQuery (chunk [3]) : assignedCollaborators au lieu de ppe - régression vs original ppe: {id: {eq: ppeId}}
• Promise.all sans erreur partielle (chunk [8]) : défendu car cohérent avec comportement original, mais Promise.allSettled préférable
• Perte typage TypeScript : interfaces supprimées sans remplacement (chunk [1])
• Absence tests automatisés : dette préexistante exacerbée par 12+ requêtes
• Types any préexistants hors périmètre ce ticket

• BUG BLOQUANT : getTasksEventsByPpeIdQuery (queries.tsx:288) filtre par assignedCollaborators au lieu de ppe - données incorrectes en vue PPE, régression par copier-coller depuis getTaskEventsByUserIdQuery
• RÉGRESSION TYPE : interfaces getEventsByUserId/getEventsByPpeId supprimées sans remplacement - 12+ fonctions exportées sans annotation de paramètres, erreurs silencieuses à la compilation
• RÉSILIENCE : Promise.all (dashboard.store.tsx) échec total si 1/12 requêtes échoue - Promise.allSettled avec filtrage fulfilled permettrait affichage partiel
• SÉCURITÉ : interpolations userId/ppeId non échappées dans requêtes GraphQL - risque d'injection sans validation UUID préalable
• DUPLICATION : sélections de champs répétées dans 12+ requêtes sans fragments GraphQL - maintenance multipliée par 12 pour chaque évolution de schéma

• BUG DE COPIER-COLLER DANS queries.tsx chunk 3 : getTasksEventsByPpeIdQuery filtre par assignedCollaborators au lieu de ppe, régression fonctionnelle prouvant l'insuffisance des tests manuels. Un test snapshot GraphQL aurait détecté cette incohérence structurelle.
• AUCUN test d'équivalence des données entre la requête monolithique et les 12 requêtes divisées. Une régression silencieuse sur les événements affichés (doublons, événements manquants, données corrompues) est indétectable sans test de régression comparatif.
• Promise.all SANS gestion d'erreur partielle dans dashboard.store.tsx. Un seul échec réseau sur 7-8 requêtes parallèles provoque l'échec total du dashboard. Scénarios non testés : timeout sur getTicketEventsByUserQuery, erreur 403 sur getDocumentSignatureEventsByUserIdQuery, réponse partielle du serveur.
• INJECTION GraphQL : les interpolations non échappées userId et ppeId dans les templates de requêtes sont vulnérables. Aucun test de sécurité avec entrées malveillantes pour vérifier l'injection de fragments GraphQL.
• SUPPRESSION des interfaces TypeScript sans remplacement. Les 12 nouvelles fonctions exportées perdent la validation compile-time des paramètres. getTaskEventsByUserIdQuery accepte n'importe quels types sans erreur.

• BUG COPIER-COLLIER CRITIQUE (queries.tsx chunk 3) : getTasksEventsByPpeIdQuery filtre par assignedCollaborators au lieu de ppe - régression fonctionnelle par rapport au code original. Preuve : comparer chunk 3 nouveau avec chunk 2 original
• PERTE SÉCURITÉ TYPE (queries.tsx chunks 1,2,4) : interfaces getEventsByUserId et getEventsByPpeId supprimées sans remplacement. Les 12 fonctions exportées sans annotation permettent erreurs d'arguments silencieuses
• PROMISE.ALL SANS DÉGRADATION GRACIEUSE (dashboard.store.tsx chunk 8) : probabilité défaillance x12 vs monolithique. Promise.allSettled + filtrage résultats valides est le pattern requis pour un dashboard
• DUPLICATION STRUCTURELLE DRY (queries.tsx) : filtres et sélections de champs répétés dans 12+ requêtes. Fragments GraphQL auraient centralisé évolution du schéma
• TYPES ANY PERSISTANTS (dashboard.store.tsx chunk 6) : defineTicketEvents(tickets:any[], t:any) - accès non vérifiables comme ticket?.attributes?.ticketClass

Tour 3 : Validation

• RÉGRESSION BLOQUANTE VUE PPE : getTasksEventsByPpeIdQuery filtre par assignedCollaborators au lieu de ppe - utilisateurs PPE voient des données incorrectes. Bug copier-coller confirmé par auteur.
• FIABILITÉ DÉGRADÉE : Promise.all sans fallback = écran blanc complet si 1 requête sur 12 échoue. Promise.allSettled requis pour dashboard.
• ROI NON MESURÉ : Aucune métrique de performance avant/après. Bénéfice métier principal (rapidité) reste théorique.
• DETTE NETTE +18h : 20h ajoutée (typage, résilience, fragments, tests, sécurité, bug fix) vs 2h réduite.
• RISQUE SURCHARGE SERVEUR : 10+ requêtes simultanées x utilisateurs concurrents pourrait saturer Strapi.

• BUG RÉGRESSION queries.tsx:288 - getTasksEventsByPpeIdQuery filtre assignedCollaborators au lieu de ppe - vue PPE affiche données incorrectes - correction urgente 1h
• PERTE TYPES TypeScript - interfaces supprimées sans remplacement - 12 fonctions sans validation compile-time - restauration 3h
• PROMISE.ALL sans dégradation gracieuse - échec total si 1/12 requêtes échoue - Promise.allSettled préférable - ajout 2h
• DUPLICATION DRY - sélections champs répétées 12+ fois sans fragments GraphQL - maintenance x12 - centralisation 4h
• SÉCURITÉ GraphQL - interpolations non échappées - risque minimal (JWT auth) mais validation UUID recommandée

• BUG BLOQUANT : queries.tsx getTasksEventsByPpeIdQuery filtre assignedCollaborators au lieu de ppe — données PPE incorrectes en production, correction requise avant fusion
• RÉSILIENCE : dashboard.store.tsx Promise.all sans dégradation — risque panne totale x4 vs original, Promise.allSettled + filtrage fulfilled requis
• TYPE SAFETY : queries.tsx 12 fonctions exportées sans annotation paramètres — erreurs silencieuses compile-time, rétablir interfaces requis
• TESTS ZÉRO : bug copier-coller non détecté prouve insuffisance tests manuels — tests régression minimum requis avant fusion
• DRY : queries.tsx 12+ requêtes sans fragments GraphQL — maintenance x12 par évolution schéma, fragments requis

• BUG CONFIRMÉ queries.tsx chunk 3 : getTasksEventsByPpeIdQuery filtre assignedCollaborators au lieu de ppe - preuve absence tests
• ZÉRO test régression comparative entre monolithique et 12 requêtes divisées
• AUCUN test résilience Promise.all : 1 échec/12 = panne totale dashboard
• AUCUN test sécurité interpolations GraphQL non échappées (userId, ppeId)
• Interfaces TypeScript supprimées = 12 fonctions sans validation compile-time

• BUG CRITIQUE queries.tsx chunk 3 : getTasksEventsByPpeIdQuery filtre assignedCollaborators au lieu de ppe - régression fonctionnelle vue PPE prouvée par comparaison avant/après
• RÉSILIENCE dashboard.store.tsx chunk 8 : Promise.all avec 12 points de défaillance - probabilité échec total 5.8% vs 0.5% original - Promise.allSettled + filtrage fulfilled requis
• RÉGRESSION TYPE queries.tsx chunks 1,2,4 : interfaces supprimées - 12 fonctions exportées sans annotation permettent erreurs arguments silencieuses à la compilation
• VIOLATION DRY queries.tsx : sélections champs répétées 12+ fois sans fragments GraphQL - coût maintenance multiplié par 12 pour chaque évolution schéma
• CONNASCENCE POSITION dashboard.store.tsx : index tableau Promise.all couplés à déstructuration - réordonnancement accidentel = bug silencieux