🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• AUCUN TEST AUTOMATISÉ : le regroupement par copropriétaire et l'agrégation des lots sont des logiques métier critiques nécessitant des tests pour les cas limites (copropriétaire sans événement, lots multiples, signatures manquantes)
• DUPLICATION ENTRE GETTERS : le type CoproprietaireListPresence est défini dans le fichier final uniquement, mais la logique de regroupement semble identique dans le getter initial - risque de divergence future estimé à 2h de dette
• CHAMPS NULLABLES INCOHÉRENTS : TagList a present/represented/missing en boolean | null - si les trois sont null simultanément, le statut de présence est indéfini, nécessitant une règle métier de validation
• RISQUE DE RÉGRESSION SUR TEMPLATES : le changement de structure (regroupement par copropriétaire vs par lot) modifie les variables disponibles pour les documents générés - les templates existants doivent être vérifiés
• MANQUE DE VALIDATION MÉTIER : pas de validation visible sur les heures (départ avant arrivée) ni sur les statuts de présence mutuellement exclusifs

• Décalage horaire codé en dur (+2h) au lieu d'utiliser Intl ou date-fns-tz. Justifié par le contexte métier (AG en France métropolitaine uniquement), mais dette technique de ~3h pour une internationalisation future.
• Complexité algorithmique O(n*m) du filtrage agEvents.filter() imbriqué dans map(). Acceptable pour le volume actuel, mais un pré-calcul avec Map serait plus robuste et lisible.
• Absence de tests unitaires sur les transformations critiques (filtrage événements, conversion horaire, construction CoproprietaireListPresence). La validation repose sur les PDF générés, ce qui ralentit le feedback loop.

• ag.d.ts : Duplication DRY - `TagList` et `AgList` partagent 6 champs identiques (present, represented, missing, procurationName, signatureURL, signatureKdriveID). Impact concret : risque de divergence lors d'évolution de schéma. Remédiation : extraire interface commune. Coût : 1h.
• ag.d.ts : Nom `TagList` sémantiquement incorrect - 'Tag' suggère un système de balisage, pas un suivi de présence/signature. Alternatives : `PresenceEntry`, `AttendanceRecord`, `ListPresenceEntry`.
• ag_list_presence_final_variables_getter.ts : Convention snake_case (coproprietaire_name, signature_url) incohérente avec camelCase du codebase. Si requis par le moteur de templates, ajouter un commentaire JSDoc explicatif pour les développeurs futurs.
• ag_list_presence_final_variables_getter.ts : Types anonymes `{ time: string }[]` et `{ lotNumber: string; thousandths: string; name: string }[]` devraient être nommés (`TimeEvent`, `OwnershipEntry`) pour traçabilité et réutilisation.
• Absence critique de tests : (a) logique de regroupement par copropriétaire, (b) extraction événements arrivée/départ, (c) gestion erreurs template dans client.tsx. Complexité cyclomatique ~8-10 par getter justifie des tests unitaires.

• Aucun test automatisé pour +183 lignes de logique métier critique (génération de listes de présence légales d'AG) - score testCoverage=2/10
• Offset horaire codé en dur (2*60*60*1000) sans test - ignore changements CET/CEST, risque de calculs incorrects sur documents légaux avec impact métier
• Chaînage optionnel excessif (event.coproprietaire.data?.id === list?.coproprietaire?.data?.id) sans tests null/undefined - risque d'erreurs silencieuses produisant des documents incomplets
• Type TagList avec 7 champs nullables (128 combinaisons) sans tests de robustesse pour les cas limites
• Transformation ownerships.filter().map() sans test unitaire isolé - accès à propriete.data.attributes.lotNumber plantera si propriete.data est null

• BUG CRITIQUE - Fuseau horaire codé en dur (+2h) dans ag_list_presence_final_variables_getter.ts : incorrect en heure d'hiver (CET=UTC+1). Les heures d'arrivée/départ seront fausses de novembre à mars. Remplacer par Intl.DateTimeFormat avec timeZone: 'Europe/Paris'.
• VIOLATION DRY - TagList (ag.d.ts:124-133) duplique 6 champs de AgList (ag.d.ts:112-122). Risque de désynchronisation à chaque évolution. Fusionner via Pick ou héritage de type.
• DUPLICATION INTER-FICHIERS - CoproprietaireListPresence et la logique de mapping sont copiés entre ag_list_presence_final_variables_getter.ts et ag_list_presence_initial_variables_getter.ts au lieu d'être extraits dans un utilitaire partagé.
• COMPLEXITÉ COGNITIVE - Logique imbriquée Promise.all → async map → filter agEvents → map formatage dates sans extraction en fonctions pures (formatEventTime, getArrivedEvents, getLeftEvents), empêchant les tests unitaires isolés.
• GESTION D'ERREUR - Promise.all sans fallback : si #getSignatureUrl() échoue pour un copropriétaire, toute la génération échoue au lieu de dégrader avec signature_url: null.

Tour 2 : Préoccupations et questions

• BUG CRITIQUE - Fuseau horaire codé en dur (ag_list_presence_final_variables_getter.ts) : offset +2h (2*60*60*1000) incorrect en heure d'hiver CET=UTC+1. Les heures d'arrivée/départ seront décalées d'1h de novembre à mars sur les PV d'AG. Risque juridique concret : invalidation possible des délibérations si les heures de présence sont contestées (article 14 loi du 10 juillet 1965). Correction urgente requise via Intl.DateTimeFormat avec timeZone:'Europe/Paris' (coût estimé ~1h).
• ABSENCE DE TESTS - Zéro test automatisé sur +183 lignes de logique métier légale (regroupement par copropriétaire, extraction événements arrivée/départ, conversion horaire). La transformation ownerships.filter().map() dans ag_list_presence_initial_variables_getter.ts plantera si propriete.data est null. Coût remédiation : 3-4h pour couvrir les cas limites critiques.
• RISQUE DE RÉGRESSION TEMPLATES - Le changement de structure (propriétés regroupées par copropriétaire vs par lot) modifie les variables disponibles pour le moteur de génération (coproprietaire_name, ownerships[], procurationName). Les templates existants doivent être vérifiés et mis à jour - aucun signe que cela a été fait dans ce commit.
• DUPLICATION DRY MULTI-NIVEAUX - (1) TagList (ag.d.ts:124-133) duplique 6 champs de AgList (ag.d.ts:112-122) avec risque de désynchronisation, (2) logique CoproprietaireListPresence copiée entre ag_list_presence_final_variables_getter.ts (+93 lignes) et ag_list_presence_initial_variables_getter.ts (+57 lignes). Risque : divergence produisant des documents incohérents entre liste initiale et finale. Remédiation : extraire interface commune + utilitaire partagé (~2h).
• GESTION D'ERREUR INSUFFISANTE - Promise.all sans fallback dans les deux getters : si #getSignatureUrl() échoue pour un copropriétaire, toute la génération échoue au lieu de dégrader avec signature_url:null. Pour un document légal d'AG, un document incomplet vaut mieux que pas de document du tout.

• Bug critique timezone : +2h hardcoded incorrect en heure d'hiver (CET=UTC+1). Correction urgente avec Intl.DateTimeFormat - estimé 1h
• Duplication TagList/AgList : 6 champs communs sans interface partagée. Risque de désynchronisation validé - estimé 1h pour extraction
• Promise.all sans fallback sur #getSignatureUrl() : un échec de signature fait échouer toute la génération. Promise.allSettled serait plus robuste - estimé 30min
• Absence de tests unitaires sur logique métier critique (regroupement, filtrage événements, conversion horaire) - dette de ~4h
• Types anonymes devraient être nommés pour traçabilité (TimeEvent, OwnershipEntry) - estimé 30min

• BUG CRITIQUE (ag_list_presence_final_variables_getter.ts) : Offset UTC+2 codé en dur (2*60*60*1000) incorrect en heure d'hiver CET (UTC+1) de novembre à mars. Heures fausses sur documents légaux d'AG. Remédiation : Intl.DateTimeFormat avec timeZone Europe/Paris.
• Violation DRY (ag.d.ts:124-133 vs 112-122) : TagList duplique 6 champs de AgList (present, represented, missing, procurationName, signatureURL, signatureKdriveID). Risque divergence schéma. Remédiation : Pick.
• Duplication inter-fichiers : CoproprietaireListPresence et logique ownerships.filter().map() copiées entre initial (+57 lignes) et final (+93 lignes) getters. Dette 3h extraction.
• Zéro test pour +183 lignes de logique métier critique : regroupement copropriétaire, extraction événements, conversion horaire, gestion signatures.
• Promise.all sans fallback : échec #getSignatureUrl() fait échouer toute génération au lieu de dégrader avec signature_url null.

• AUCUN test automatisé pour +183 lignes de logique métier critique de génération de documents légaux d'AG - score testCoverage maintenu à 2/10
• BUG CONFIRMÉ de fuseau horaire (hardcoded +2h incorrect en CET/heure d'hiver) - des tests unitaires sur formatEventTime auraient détecté ce bug immédiatement
• CRASH POTENTIEL non testé : ownerships.filter().map() accède à propriete.data.attributes.lotNumber sans null check - un test avec propriete.data=null exposerait ce crash silencieux
• TagList avec 7 champs nullables (128 combinaisons) sans aucun test de robustesse pour les états invalides (tous null simultanément = statut indéfini)
• Promise.all sans fallback sur #getSignatureUrl() - si un appel échoue, toute la génération de document échoue au lieu de dégrader élégamment - aucun test d'erreur

• BUG FONCTIONNEL CRITIQUE : fuseau horaire +2h codé en dur incorrect en heure d'hiver (CET=UTC+1), produisant des heures fausses sur documents légaux d'AG de novembre à mars - justification de l'auteur insuffisante
• VIOLATION DRY STRUCTURELLE : TagList duplique 6 champs de AgList sans héritage ni Pick, créant un couplage implicite qui se désynchronisera à la prochaine évolution de schéma
• DUPLICATION INTER-FICHIERS MAJEURE : CoproprietaireListPresence et logique de transformation (filter ownerships, format dates, build presence object) copiées entre initial et final getters - extraction en utilitaire partagé urgente
• POINT DE DÉFAILLANCE UNIQUE : Promise.all sur getSignatureUrl sans fallback - une signature Kdrive indisponible fait échouer toute la génération du document au lieu de dégrader élégamment
• DETTE DE TEST CRITIQUE : +183 lignes de logique métier légale (regroupement copropriétaire, agrégation lots, conversion horaire) sans aucun test automatisé - risque de régression silencieuse élevé

Tour 3 : Validation

• BUG CRITIQUE CONFIRMÉ PAR CONSENSUS - Fuseau horaire UTC+2 codé en dur (ag_list_presence_final_variables_getter.ts, offset 2*60*60*1000) incorrect en heure d'hiver CET=UTC+1. Impact : heures d'arrivée/départ décalées d'1h de novembre à mars sur les PV d'AG. Risque juridique concret : invalidation des délibérations (article 14 loi du 10 juillet 1965). Remédiation : Intl.DateTimeFormat avec timeZone:'Europe/Paris' (~1h). Priorité : URGENTE avant la prochaine période d'AG.
• RISQUE JURIDIQUE MÉTIER - La majorité des AG annuelles se tiennent en période hivernale (novembre-mars). Le bug timezone affecte donc la période la plus critique. Un copropriétaire peut contester la validité du PV si les heures de présence sont inexactes. Valeur métier du suivi horaire annulée pendant 6 mois/an.
• CRASH POTENTIEL SUR NULL - ownerships.filter().map() dans ag_list_presence_initial_variables_getter.ts accède à propriete.data.attributes.lotNumber sans null check. Si propriete.data est null (lot supprimé ou référence cassée), la génération du document échoue silencieusement. Remédiation : optional chaining propriete.data?.attributes?.lotNumber (~30min).
• ABSENCE TOTALE DE TESTS - Zéro test automatisé sur +183 lignes de logique métier légale. 128 combinaisons d'états null sur TagList non testées. Cas limites non couverts : propriete.data null, événements sans occuredAt, coproprietaire.data null. Dette estimée : 3-4h pour couverture minimale.
• DUPLICATION DRY MULTI-NIVEAUX - (1) TagList (ag.d.ts:124-133) duplique 6 champs de AgList (ag.d.ts:112-122) sans Pick. (2) Logique CoproprietaireListPresence copiée entre initial (+57 lignes) et final (+93 lignes) getters avec variations subtiles. Risque : divergence produisant des documents incohérents entre liste initiale et finale.

• BUG CRITIQUE TIMEZONE: offset +2h hardcoded (ag_list_presence_final_variables_getter.ts:96-99) incorrect en heure d'hiver CET=UTC+1, produisant heures fausses sur PV d'AG novembre-mars - risque juridique article 14 loi 10 juillet 1965 - correction urgente via Intl.DateTimeFormat avec timeZone Europe/Paris (1h)
• DETTE TESTS MAJEURE: 0 tests pour +183 lignes de logique métier légale (regroupement copropriétaire, extraction événements arrivée/départ, conversion horaire, gestion signatures Kdrive) - 4h pour couverture minimale
• DUPLICATION TagList/AgList: 6 champs communs sans interface partagée - risque désynchronisation mais couplage par Pick prématuré car contextes métier divergents (initial vs final avec événements)
• POINT DE DÉFAILLANCE: Promise.all sur #getSignatureUrl() sans fallback - échec signature Kdrive fait échouer toute génération document au lieu de dégrader avec signature_url:null - Promise.allSettled préférable (30min)
• NULL CHECKS MANQUANTS: ownerships.filter().map() accède à propriete.data.attributes.lotNumber sans vérification - risque théorique vu garanties Strapi mais défense programmative nécessaire (30min)

• BUG CRITIQUE TIMEZONE : offset +2h hardcoded incorrect en heure d'hiver CET (UTC+1) - heures fausses sur PV d'AG de novembre à mars - risque juridique d'invalidation des délibérations
• VIOLATION DRY TYPE : TagList duplique 6 champs de AgList sans Pick ni interface commune - risque de désynchronisation schéma
• VIOLATION DRY LOGIQUE : CoproprietaireListPresence et transformations copiées entre initial et final getters - divergence produira documents incohérents
• ZÉRO TEST : +183 lignes de logique métier légale sans couverture - risque de régression silencieuse élevé
• CRASH POTENTIEL : ownership.attributes.propriete.data.attributes.lotNumber sans null check - crash si propriete.data est null

• BUG CRITIQUE TIMEZONE (ag_list_presence_final_variables_getter.ts:96-103) : `2*60*60*1000` code UTC+2 permanent, incorrect en CET (nov-mars) - heures fausses sur PV légaux, risque invalidation délibérations - test paramétré été/hiver aurait détecté immédiatement
• ZÉRO TEST AUTOMATISÉ pour +183 lignes logique métier légale : 0 fichier .spec.ts/.test.ts modifié, 6 scénarios critiques non couverts - dette ~4h confirmée par l'auteur
• CRASH POTENTIEL (ag_list_presence_initial_variables_getter.ts:97-102) : `propriete.data.attributes.lotNumber` sans null check sur 4 niveaux - TypeError si Strapi renvoie propriete.data=null
• TagList (ag.d.ts:124-133) duplique 6 champs AgList sans Pick - 7 champs nullables, 3 flags mutuellement exclusifs sans validation, état (null,null,null) non géré
• Promise.all sans fallback (les deux getters) : échec #getSignatureUrl() pour 1 copropriétaire → toute la génération échoue au lieu de dégrader avec signature_url:null

• BUG TIMEZONE CRITIQUE (final getter:92-100) : offset +2h hardcoded incorrect en heure d'hiver CET=UTC+1 - heures fausses sur PV d'AG novembre-mars - risque juridique art.14 loi 10/07/1965 - correction Intl.DateTimeFormat estimée 1h
• DUPLICATION TYPE (ag.d.ts:124-133 vs 112-122) : TagList duplique 6 champs AgList sans Pick - désynchronisation schéma inévitable - correction estimée 1h
• DUPLICATION LOGIQUE (initial +57 vs final +93 lignes) : pattern filter().map() sur ownerships copié entre getters - divergence documents initiaux/finaux - extraction utilitaire estimée 2h
• NULL SAFETY (initial getter) : propriete.data.attributes.lotNumber sans null check sur 4 niveaux - crash runtime si propriete.data=null - correction estimée 1h
• PROMISE.ALL SANS FALLBACK (les deux getters) : échec #getSignatureUrl fait échouer toute la génération au lieu de dégrader - Promise.allSettled estimé 0.5h