🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• Risque juridique résiduel critique : .filter(Boolean) exclut silencieusement les co-copropriétaires sans email sans log ni notification - un co-copropriétaire non informé pourrait contester la validité de l'AG
• Duplication de code sur 3 services (saveTheDate.ts, shareAgPv.ts, sendInvitations.js) : toute évolution future devra être répliquée 3 fois, avec risque d'oubli sur un service
• Requête GraphQL par interpolation de chaînes (in: [${coproprietaireIds}]) au lieu de variables paramétrées - vulnérable aux injections et erreurs de formatage si un ID contient un caractère spécial
• Aucun test automatisé pour une fonctionnalité à enjeu juridique - la vérification manuelle mentionnée ne garantit pas la non-régression future
• Correction UX du bouton désactivé (client.tsx finally block) non documentée - si c'est un bug de production actif, les utilisateurs impactés devraient être informés

• Vulnérabilité d'injection GraphQL: L'interpolation directe de ${coproprietaireIds} et ${ppeId} dans les requêtes gql contourne le système de variables GraphQL. Ce pattern est cohérent avec le code existant mais devrait être migré vers des variables paramétrées pour sécurité et performance (cache côté serveur).
• Aucun test unitaire: getCoCoprosEmailFor et les modifications des 3 services email n'ont pas de tests. Une régression sur l'envoi d'emails AG impacterait directement les copropriétaires.
• Duplication de logique: L'appel à getCoCoprosEmailFor est répété dans 3 services avec un pattern similaire - devrait être factorisé dans un service partagé ou un hook.
• Séparation des responsabilités: getCoCoprosEmailFor mélange logique de requête Apollo Client et transformation de données, rendant les tests unitaires difficiles sans mock du client Apollo.

• VIOLATION SRP : Import d'Apollo Client dans regieQueries.ts mélange définition et exécution de queries - fichier devrait rester pure déclaration GraphQL
• DUPLICATION : Logique d'ajout co-copros répétée dans 3 services email (saveTheDate.ts, shareAgPv.ts, sendInvitations.js) sans utilitaire commun - extraction recommandée dans un service partagé
• ZÉRO TEST : Aucun test automatisé ajouté pour une fonctionnalité critique d'envoi d'emails à de nouveaux destinataires - risque de régression élevé
• NOMMAGE AMBIGU : getCoCoprosEmailFor vs getOwnershipsForCopros - distinction sémantique floue nécessite clarification ou renommage
• GESTION D'ERREUR : Robustesse incertaine pour emails co-copros invalides/manquants - nécessite vérification du code complet

• getOwnershipsForCopros utilise l'interpolation de chaînes au lieu de variables GraphQL paramétrées - risque d'injection et d'échec silencieux sur formats invalides (tableau vide, IDs non numériques), sans aucun test unitaire pour valider ces cas
• getCoCoprosEmailFor est non testable unitairement : l'import direct d'ApolloClient (regieQueries.ts ligne 2) empêche l'injection de mocks, nécessitant un refactoring vers l'injection de dépendances
• Les 3 services d'email (saveTheDate.ts +16/-2, shareAgPv.ts +15/-4, sendInvitations.js +15/-1) concatènent des listes de destinataires sans tests vérifiant le comportement avec emails null/undefined, doublons, ou listes vides
• L'approche de test déclarée est entièrement manuelle ('vérifier la réception des e-mails') - non reproductible en CI/CD, ne couvre pas les cas limites, et ne peut pas détecter les régressions futures
• Le bug fix du bouton désactivé dans finally (client.tsx) n'a pas de test de régression - le même défaut peut réapparaître

• VULNÉRABILITÉ CRITIQUE : L'interpolation de chaînes dans la requête GraphQL (${coproprietaireIds}) expose à des attaques par injection. Les variables GraphQL paramétrées doivent être utilisées à la place.
• Violation du SRP : Le fichier regieQueries.ts importe désormais getApolloClient, mélangeant la définition de requêtes et la logique d'exécution. La séparation des préoccupations est compromise.
• Bug potentiel : Le paramètre 'archived' interpolé comme ${archived} génère la chaîne 'false' au lieu du booléen GraphQL attendu, ce qui pourrait causer un comportement inattendu.
• Duplication de code : Le pattern d'ajout des co-copropriétaires comme destinataires est répété entre saveTheDate.ts et shareAgPv.ts sans abstraction commune, augmentant la dette de maintenance.
• Absence de tests automatisés : La stratégie de test décrite est uniquement manuelle. Aucun test unitaire ou d'intégration n'accompagne cette fonctionnalité critique d'envoi d'emails.

Tour 2 : Préoccupations et questions

• RISQUE JURIDIQUE CRITIQUE : .filter(Boolean) exclut silencieusement les co-copropriétaires sans email des convocations, PV et invitations - aucun log ni notification. Un co-copropriétaire non informé peut contester la validité de l'AG (art. 9 décret 17/03/1967). La fonctionnalité crée un faux sentiment de conformité.
• Duplication sur 3 services (saveTheDate.ts, shareAgPv.ts, sendInvitations.js) : toute correction devra être répliquée 3 fois avec risque d'incohérence entre les types de communications.
• Zéro test automatisé pour une fonctionnalité à enjeu juridique - les cas limites (emails null/undefined, doublons, listes vides, IDs non numériques) ne sont pas couverts.
• Vulnérabilité injection GraphQL : interpolation de chaînes au lieu de variables paramétrées. Bug spécifique : ${archived} génère 'false' chaîne au lieu du booléen GraphQL false.
• Violation SRP regieQueries.ts : import d'ApolloClient mélange définition et exécution, rendant getCoCoprosEmailFor non testable unitairement.

• Dette technique tests : 4-6h pour tests unitaires getCoCoprosEmailFor (mock Apollo Client) et tests d'intégration 3 services email avec cas limites (emails null/undefined, doublons, listes vides, IDs non numériques)
• Interpolation GraphQL ${archived} génère chaîne 'false' au lieu du booléen GraphQL attendu - vérifier comportement côté serveur
• Logging des co-propriétaires filtrés sans email recommandé pour traçabilité juridique future
• Refactoring global de regieQueries.ts vers variables GraphQL paramétrées nécessaire à terme mais hors périmètre de cette PR

• VIOLATION SRP CRITIQUE : regieQueries.ts importe getApolloClient - mélange définition et exécution de queries, rendant les tests unitaires impossibles sans mock global d'Apollo
• DUPLICATION CONFIRMÉE : pattern getCoCoprosEmailFor + concaténation destinataires répété 3 fois sans abstraction - risque de divergence future
• BUG POTENTIEL ARCHIVED : interpolation ${archived} génère la chaîne 'false' au lieu du booléen GraphQL - comportement incorrect si le schéma attend Boolean
• ZÉRO TEST AUTOMATISÉ : fonctionnalité d'envoi d'emails à enjeu juridique sans aucune couverture de test - risque de régression élevé
• INTERPOLATION GRAPHQL : extension du pattern vulnérable `in: [${coproprietaireIds}]` sans migration vers variables paramétrées - dette de sécurité

• AUCUN test automatisé pour une fonctionnalité à enjeu juridique (notification AG) - risque de régression critique en production
• Architecture non-testable : getCoCoprosEmailFor importe directement Apollo Client, empêchant l'injection de mocks pour les tests unitaires
• Cas limites non couverts : emails null/undefined exclus silencieusement par .filter(Boolean), doublons potentiels entre listes de destinataires, listes vides, IDs malformés
• Duplication de logique sur 3 services email sans utilitaire partagé - triple effort de test et risque d'inconsistance
• Vulnérabilité d'injection GraphQL par interpolation de chaînes sans test de validation d'entrée

• VULNÉRABILITÉ CRITIQUE : L'interpolation de chaînes dans les requêtes GraphQL (${coproprietaireIds}, ${ppeId}, ${archived}) expose à des injections et génère des bugs de typage (${archived} → 'false' string vs boolean). Refactoring obligatoire en variables paramétrées.
• VIOLATION SRP : L'import de getApolloClient dans regieQueries.ts mélange définition de requêtes et logique d'exécution, cassant la testabilité et la séparabilité des préoccupations.
• DUPLICATION DE LOGIQUE : Le pattern d'ajout des co-copropriétaires est répété 3 fois sans abstraction commune - violation DRY qui multipliera l'effort de maintenance future.
• ÉCHEC SILENCIEUX : .filter(Boolean) supprime les co-copropriétaires sans email sans log ni notification - risque juridique pour la validité des AG si un co-copropriétaire n'est pas informé.
• ABSENCE DE TESTS : Zéro test automatisé pour une fonctionnalité à enjeu juridique. La vérification manuelle ne garantit pas la non-régression.

Tour 3 : Validation

• RISQUE JURIDIQUE CRITIQUE : .filter(Boolean) dans shareAgPv.ts (ligne 68) exclut silencieusement les co-copropriétaires sans email des convocations, PV et invitations. Aucun log ni notification. Un co-copropriétaire non informé peut contester la validité de l'AG (art. 9 décret 1967). La fonctionnalité crée un faux sentiment de conformité légale. Action requise : ajouter un log obligatoire pour chaque destinataire exclu et/ou bloquer l'envoi si des destinataires sont manquants.
• Duplication sur 3 services (saveTheDate.ts, shareAgPv.ts, sendInvitations.js) du pattern getCoCoprosEmailFor + concaténation destinataires + .filter(Boolean). Toute correction devra être répliquée 3 fois avec risque d'inconsistance. Action requise : extraire un utilitaire commun (buildRecipients) pour éliminer la duplication.
• Zéro test automatisé pour une fonctionnalité à enjeu juridique. Cas limites non couverts : emails null/undefined, doublons entre coproprietaireEmails et coCoproprietaireEmails, listes vides, IDs non numériques passés à l'interpolation GraphQL. Action requise : tests unitaires getCoCoprosEmailFor avec mocks Apollo + tests d'intégration 3 services.
• Vulnérabilité injection GraphQL dans regieQueries.ts : interpolation de chaînes (${coproprietaireIds}, ${ppeId}, ${archived}) au lieu de variables paramétrées. Bug spécifique : ${archived} génère la chaîne 'false' au lieu du booléen GraphQL false. Action requise : migrer vers des variables GraphQL paramétrées.
• Violation SRP regieQueries.ts : import de getApolloClient mélange définition de queries et exécution, rendant getCoCoprosEmailFor non testable unitairement. Action requise : séparer la définition des queries de leur exécution.

• Bug ${archived} générant 'false' string au lieu de boolean GraphQL - correction rapide nécessaire
• Logging recommandé pour les co-copropriétaires filtrés sans email - traçabilité juridique
• Refactoring global de regieQueries.ts vers variables paramétrées nécessaire à terme
• Tests unitaires et d'intégration à ajouter dans un sprint dédié

• BLOQUANT - Bug ${archived} : interpolation JS génère string 'false' au lieu de boolean GraphQL - vérifier schéma serveur et corriger (0.5h)
• BLOQUANT - .filter(Boolean) silencieux : exclut destinataires sans email sans log - ajouter logging pour traçabilité juridique AG (1h)
• Violation SRP : getApolloClient importé dans regieQueries.ts mélange définition et exécution - extraire vers service dédié (2h dette)
• Duplication 3x : pattern recipients fusionné sans abstraction - créer utilitaire mergeRecipients (2h dette)
• Zéro test : aucune couverture pour fonctionnalité à enjeu juridique - risque régression critique (4-6h dette)

• ZÉRO TEST AUTOMATISÉ pour fonctionnalité à enjeu juridique (notification AG aux co-copropriétaires) - risque de régression critique en production
• Architecture non-testable : getCoCoprosEmailFor importe directement Apollo Client, empêchant l'injection de mocks pour tests unitaires - violation SRP dans regieQueries.ts
• Échec silencieux .filter(Boolean) exclut les co-copropriétaires sans email sans log ni notification - risque juridique pour validité AG (art. 9 décret 17/03/1967)
• Duplication de logique sur 3 services email (saveTheDate, shareAgPv, sendInvitations) sans utilitaire partagé - triple effort de test et risque d'inconsistance
• Bug de typage GraphQL : interpolation ${archived} génère chaîne 'false' au lieu du booléen GraphQL - aucun test pour valider le comportement

• VULNÉRABILITÉ INJECTION GRAPHQL : Interpolation de chaînes (${coproprietaireIds}, ${ppeId}, ${archived}) au lieu de variables paramétrées. Le bug ${archived} générant 'false' string est confirmé par l'auteur. Extension d'un pattern vulnérable existant au lieu de le corriger.
• VIOLATION SRP CONFIRMÉE : Import de getApolloClient dans regieQueries.ts mélange définition et exécution. L'auteur reconnaît le problème mais le diffère hors périmètre - la dette persistera en production.
• DUPLICATION DRY SUR 3 SERVICES : Pattern co-copropriétaires répété dans saveTheDate.ts, shareAgPv.ts, sendInvitations.js sans abstraction commune. L'auteur et les reviewers convergent sur ce diagnostic.
• RISQUE JURIDIQUE - ÉCHEC SILENCIEUX : .filter(Boolean) exclut les destinataires sans email sans traçabilité. Pour des convocations d'AG, un co-copropriétaire non informé peut contester la validité de l'assemblée. L'auteur reconnaît le besoin de logging.
• ABSENCE CRITIQUE DE TESTS : Zéro test automatisé pour une fonctionnalité à enjeu juridique. L'auteur estime 4-6h pour les tests nécessaires, confirmant l'ampleur de la dette.