🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• RGPD - Divulgation antérieure : Des emails sensibles ont pu être envoyés à des copropriétaires hors périmètre PPE avant cette correction. Une évaluation d'impact rétroactif et une notification potentiellement nécessaire.
• Absence de tests automatisés : Aucun test sur getMailAddressOfRecipients pour vérifier que le filtre ppeId est appliqué correctement. Risque de régression élevé si le paramètre est retiré.
• Sécurité - Injection GraphQL : Le pattern d'interpolation de chaînes (${recipients}, ${ppeId}) dans les requêtes GraphQL est une vulnérabilité d'injection. Devrait utiliser des variables GraphQL paramétrées ($ppeId, $recipients).
• Robustesse - Paramètre manquant : Si ppeId est undefined, le filtre 'eq: undefined' peut avoir un comportement imprévisible. Ajouter une validation obligatoire du paramètre.
• Traçabilité : Aucun log ni audit trail pour documenter les emails envoyés aux mauvais destinataires avant correction.

• Aucun test automatisé ajouté pour valider le filtre ppeId - risque de régression silencieuse si le schéma GraphQL évolue
• L'interpolation de chaîne brute (${recipients}, ${ppeId}) dans la requête GraphQL reste un risque d'injection - devrait utiliser des variables GraphQL proprement dites
• Le paramètre ppeId n'est pas validé avant interpolation - si undefined/null est passé, la requête échouera silencieusement ou retournera des résultats incorrects

• Sécurité : ${ppeId} interpolé directement dans la requête GraphQL sans échappement - risque d'injection si la valeur n'est pas validée en amont
• Typage : Absence d'annotations TypeScript sur les paramètres recipients et ppeId de getMailAddressOfRecipients
• Tests : Aucun test de régression pour un correctif de bug de logique métier (destinataires d'emails)
• Indentation : Mélange tabulations/espaces dans le bloc GraphQL de coproprietaireQueries.ts
• Robustesse : Aucune validation que ppeId est défini avant construction de la requête - risque de requête GraphQL invalide si undefined

• Zéro test automatisé fourni avec cette correction de bug (testCoverage: 2/10) - risque critique de régression sur le filtrage multi-PPE
• Paramètre ppeId non typé dans getMailAddressOfRecipients(recipients, ppeId) - aucune erreur de compilation si undefined/null est passé, entraînant 'eq: undefined' dans la requête GraphQL
• Interpolation brute ${ppeId} dans le template gql`` sans validation ni échappement - risque d'injection et comportement indéfini avec des valeurs inattendues
• Aucun test de edge cases: ppeId=undefined génère 'eq: undefined' dans la requête, ppeId=0 ou inexistant retourne 0 résultats silencieusement sans notification d'erreur
• Approche de test déclarée purement manuelle ('tester le partage de documents sur plusieurs PPE') - non reproductible en CI/CD, aucun assurance de non-régression

• INTERPOLATION GRAPHQL NON SÉCURISÉE : ${ppeId} interpolé directement dans le template sans validation - si ppeId est undefined, la requête génère 'eq: undefined' causant un comportement indéterminé plutôt qu'une erreur explicite. Heure estimée de correction : 1h pour migrer vers des variables GraphQL typées
• ABSENCE DE TESTS (2/10) : Aucun test unitaire ou d'intégration pour un correctif touchant à la confidentialité des destinataires d'emails. Risque de régression élevé. Heure estimée pour ajouter des tests : 2h
• BREAKING CHANGE IMPLICITE : La signature getMailAddressOfRecipients(recipients, ppeId) rend ppeId obligatoire. Si d'autres appelants existent dans la codebase, l'absence du second argument causera un filtre undefined. Heure estimée de vérification : 0.5h
• COMPLEXITÉ COGNITIVE CROISSANTE : L'imbrication filters -> id -> in + ppes -> id -> eq atteint 3 niveaux de profondeur dans un template string, rendant la requête difficile à maintenir et déboguer sans coloration syntaxique GraphQL

Tour 2 : Préoccupations et questions

• RGPD - Violation rétroactive : copropriétaires multi-PPE recevaient des emails sensibles hors périmètre. Évaluation d'impact et notification potentiellement requises.
• Zéro test automatisé sur un correctif de confidentialité - régression silencieuse possible si le filtre ppes est retiré.
• Injection GraphQL : ${recipients} et ${ppeId} interpolés sans échappement - contradictoire avec l'objectif RGPD du correctif.
• ppeId non typé ni validé - si undefined, la requête génère 'eq: undefined' au lieu d'une erreur explicite.
• Breaking change implicite : signature modifiée sans vérification des autres appelants potentiels.

• L'interpolation GraphQL est une dette préexistante - pas introduite par ce PR mais mérite refactorisation future
• Tests automatisés à ajouter en suivi - la priorité immédiate était de corriger la fuite de confidentialité
• Validation défensive de ppeId recommandée en amélioration continue malgré le flux de données garanti

• INTERPOLATION GRAPHQL DANGEREUSE : ${ppeId} interpolé ligne 169 dans gql`` - si undefined, génère 'eq: undefined' littéral dans la requête au lieu d'une erreur explicite. Solution : utiliser des variables GraphQL paramétrées ($ppeId)
• TYPAGE TYPESCRIPT ABSENT : Signature getMailAddressOfRecipients(recipients, ppeId) sans types lignes 163 - devrait être (recipients: number[], ppeId: number) pour détecter les appels incomplets à la compilation
• ZÉRO TEST DE RÉGRESSION : Aucun test ajouté pour valider le filtre ppes.id.eq - un correctif de confidentialité sans test automatisé a un risque élevé de régression silencieuse
• VALIDATION MANQUANTE : Aucune vérification que ppeId est défini avant interpolation ligne 169 - devrait avoir if (!ppeId) throw new Error('ppeId requis')
• INDENTATION INCOHÉRENTE : Lignes 166-170 utilisent 2 espaces vs tabulations dans le reste du fichier

• ZÉRO test automatisé pour un correctif de bug RGPD - 5 scénarios critiques manquants : (1) ppeId valide dans requête GraphQL, (2) ppeId=undefined lance erreur, (3) intégration sendTicketMail filtre correctement, (4) régression appelants existants, (5) injection GraphQL
• Interpolation brute ${ppeId} dans gql`` - vulnérabilité d'injection GraphQL démontrable : ppeId='1} } coproprietaires { data { attributes { email' contourne le filtre. Seule la refactorisation vers variables GraphQL ($ppeId) élimine ce risque
• Paramètre ppeId non typé : getMailAddressOfRecipients(recipients, ppeId) au lieu de getMailAddressOfRecipients(recipients: number[], ppeId: number) - TypeScript ne peut pas prévenir les appels incorrects
• Edge case critique documenté : ppeId=undefined génère la chaîne littérale 'eq: undefined' dans la requête GraphQL, retournant 0 résultats silencieusement au lieu d'une erreur explicite
• Breaking change implicite non documenté : tout appelant existant passant 1 seul argument à getMailAddressOfRecipients causera un filtre undefined sans erreur de compilation ni test de régression

• INTERPOLATION GRAPHQL NON SÉCURISÉE : ${ppeId} étend le pattern d'injection existant sans migration vers des variables GraphQL paramétrées. Dette estimée : 1.5h pour refactoriser vers $ppeId avec variables proprement typées.
• ABSENCE DE VALIDATION DU PARAMÈTRE ppeId : Si undefined/null est passé, la requête génère 'eq: undefined' au lieu d'une erreur explicite. Comportement silencieusement incorrect plutôt que fail-fast.
• ABSENCE DE TYPAGE TYPESCRIPT : getMailAddressOfRecipients(recipients, ppeId) ne spécifie aucun type, annulant l'avantage de TypeScript et permettant des erreurs à l'exécution qui seraient détectables à la compilation.
• BREAKING CHANGE IMPLICITE : La signature modifiée rend ppeId obligatoire sans vérification que tous les appelants fournissent ce paramètre. Risque de régression si d'autres points d'appel existent.
• ZÉRO TEST AUTOMATISÉ : Un correctif touchant à la confidentialité des destinataires d'emails sans aucune couverture de test est un risque de régression critique.

Tour 3 : Validation

• RGPD - Violation de confidentialité pré-existente : copropriétaires multi-PPE ont reçu des emails sensibles hors périmètre. Évaluation d'impact rétroactive requise : nombre de personnes affectées, durée, nature des données exposées (tickets, informations financières).
• Défaillance silencieuse ppeId=undefined : la requête GraphQL génère 'eq: undefined' littéral → 0 résultats → emails envoyés sans destinataire → aucune erreur levée. Impact métier : notifications perdues sans trace, copropriétaires ne reçoivent pas leurs tickets.
• Zéro test automatisé sur correctif RGPD : le filtre ppes peut être retiré lors d'une refactorisation sans qu'aucun test ne l'empêche. Coût test unitaire ~30min vs coût régression RGPD (notification CNIL, amendes 4% CA, réputation).
• Breaking change non audité : getMailAddressOfRecipients(recipients) → getMailAddressOfRecipients(recipients, ppeId). Tout appelant existant avec 1 seul argument → ppeId=undefined → défaillance silencieuse. Absence de typage TypeScript empêche détection à la compilation.
• Raccourci injustifié : ajouter (recipients: number[], ppeId: number) et if (!ppeId) throw new Error('ppeId requis') coûte ~15min et élimine 2 risques majeurs (défaillance silencieuse + breaking change non détecté). L'argument 'urgence RGPD' ne justifie pas d'omettre une validation qui protège précisément l'objectif RGPD du correctif.

• Interpolation GraphQL ${ppeId} étend la dette préexistante - si undefined, génère 'eq: undefined' littéral au lieu d'une erreur explicite
• Zéro test automatisé sur un correctif de confidentialité - 5 scénarios critiques manquants (ppeId valide, ppeId=undefined, intégration sendTicketMail, appelants existants, injection)
• Typage TypeScript absent sur getMailAddressOfRecipients(recipients, ppeId) - devrait être (recipients: number[], ppeId: number)
• Breaking change implicite : signature modifiée sans vérification des autres appelants potentiels
• Indentation incohérente lignes 166-170 (2 espaces vs tabulations)

• INJECTION GRAPHQL : ${ppeId} ligne 169 coproprietaireQueries.ts interpolé sans échappement - contournable avec ppeId='1} } coproprietaires{data{attributes{email'
• ZÉRO TEST : 5 scénarios critiques manquants pour correctif RGPD (ppeId valide, ppeId=undefined, intégration sendTicketMail, régression appelants, injection)
• ppeId NON VALIDÉ : undefined génère 'eq: undefined' littéral dans requête → 0 résultats silencieux au lieu d'erreur explicite
• TYPAGE ABSENT : getMailAddressOfRecipients(recipients, ppeId) ligne 163 sans types → breaking change silencieux possible
• BREAKING CHANGE : signature modifiée sans vérification des autres appelants potentiels

• ZÉRO test automatisé pour un correctif RGPD/confidentialité - 5 scénarios critiques manquants documentés ci-dessus. L'engagement de tests 'en suivi' n'est pas vérifiable et ne justifie pas de merger sans couverture sur un correctif de confidentialité
• Interpolation brute ${ppeId} (ligne 169) étend le pattern d'injection GraphQL - vecteur démontrable : ppeId='1} } coproprietaires { data { attributes { email' contourne le filtre RGPD. Refactorisation vers variables GraphQL ($ppeId) estimée 1.5h par l'architecte
• Typage TypeScript absent (ligne 163) : getMailAddressOfRecipients(recipients, ppeId) au lieu de (recipients: number[], ppeId: number) - les appels incomplets ne sont pas détectés à la compilation
• Edge case ppeId=undefined génère 'eq: undefined' littéral dans la requête GraphQL (ligne 169), retournant 0 résultats silencieusement au lieu d'une erreur explicite - validation défensive manquante sur un correctif RGPD
• Breaking change implicite sur la signature getMailAddressOfRecipients : tout appelant existant passant 1 seul argument aura ppeId=undefined sans erreur de compilation ni test de régression

• INJECTION GRAPHQL (coproprietaireQueries.ts:169) : ${ppeId} interpolé sans échappement dans gql``. La surface d'attaque passe de ${recipients} à ${recipients}+${ppeId}. Refactorisation vers variables paramétrées estimée à 1.5h.
• COMPORTEMENT SILENCIEUX (coproprietaireQueries.ts:169) : ppeId=undefined génère 'eq: undefined' littéral dans la requête GraphQL, retournant 0 résultats silencieusement. Violation du principe fail-fast.
• TYPAGE ABSENT (coproprietaireQueries.ts:163) : getMailAddressOfRecipients(recipients, ppeId) sans types permet des appels incomplets sans erreur de compilation.
• ZÉRO TEST DE RÉGRESSION : 5 scénarios critiques manquants pour un correctif RGPD : ppeId valide, ppeId undefined, intégration sendTicketMail, appelants existants, injection GraphQL.
• BREAKING CHANGE IMPLICITE (ticket.ts:37) : Signature modifiée sans audit des appelants existants. Un appel avec 1 seul argument compile sans erreur mais produit un comportement incorrect.