🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• AUCUN test automatisé pour des documents à valeur légale - Risque juridique si un PV est mal formaté. La vérification manuelle est insuffisante pour garantir la non-régression.
• Pattern de null-coalescing INCONSISTANT - ag_variables_getter.ts utilise ?? "" sans trim/fallback, tandis que ag_list_presence_*_variables_getter.ts utilise le pattern complet .trim() || ' - '. Comportements d'affichage différents selon le document généré.
• Cause racine non traitée - Pourquoi des copropriétaires ont-ils des noms null en base ? Le fallback masque un problème de qualité de données en amont. Audit de l'API Strapi recommandé.
• Risque de régression kDrive - Le nouveau service getFileOrDirectory modifie le flux de sauvegarde. Sans tests d'intégration, une régression pourrait causer la perte silencieuse de documents légaux.
• Conformité légale non vérifiée - Les documents modifiés doivent rester conformes aux exigences légales d'affichage des noms dans les assemblées générales de copropriété.

• CRITIQUE : Console.log debug dans uploadFile.js (lignes 14-19) doivent être retirés avant production - risque de fuite d'informations sensibles (directory_id, filenames)
• DETTE TECHNIQUE : Pattern null-handling dupliqué 5 fois sans fonction utilitaire partagée - extraction recommandée en formatCoproprietaireName() pour DRY
• RISQUE REGRESSION : Aucun test unitaire sur les 5 corrections null - modification future d'un fichier sans les autres = incohérence d'affichage entre listes initiales et finales
• BREAKING CHANGE : Signature createOnlyOfficeDocument modifiée (number vers objet) - vérifier tous les appelants existants
• CODE SMELL : Valeur par défaut 'uploads/tmp' dans uploadFile.js masque un problème de configuration en amont

• Duplication de la logique de sanitisation des noms dans plusieurs générateurs - devrait être une fonction utilitaire partagée (ex: sanitizeOwnerName)
• Cas limite non géré : si firstName et lastName sont tous deux null/undefined, le nom de fichier devient ' - Convocation...' commençant par un tiret et un espace
• Le fallback `|| ''` après `.trim()` est redondant et suggère une méconnaissance du comportement de trim() sur chaîne vide
• Aucun test unitaire visible pour les changements critiques de génération de documents et de gestion kDrive
• L'ajout d'une ligne vide dans kdrive.js est du bruit de commit qui devrait être séparé ou retiré

• Couverture de test effective de 0% - aucun fichier de test sur 11 fichiers modifiés, ratio test/code de 0:85 lignes ajoutées
• convocation_generator.ts ligne 402 : cas limite non testé où coproprietaireNameSafe vide génère un fileName ' - Convocation...' avec tiret orphelin en préfixe
• getFileOrDirectory.js : `throw Error(error)` écrase la stack trace Axios et perd le contexte HTTP (statusCode, config) - anti-pattern de gestion d'erreur sans test de vérification
• getFileOrDirectory.js : appel axios.get sans timeout configurable - risque de suspension indéfinie en production, aucun test de résilience
• pv_generator.ts ligne 174 : directoryPath 'ags/${ag.data.id}' sans validation - si ag.data.id est undefined, chemin devient 'ags/undefined' sans test de régression

• Console.log de debug en production dans uploadFile.js (6 lignes) et onlyOffice.js - risque de fuite de données sensibles (directoryId, fileName). Remplacer par un logger structuré type winston/pino.
• Traitement des nulls dispersé et inconsistent: `?? ''` vs `?? '-'` selon les fichiers sans utilitaire centralisé. Créer un helper formatPersonName() pour garantir la cohérence.
• Regex de sanitisation inline dans signed_pv_generator.ts - extraire dans un utilitaire sanitizeFilename() partagé pour éviter la duplication dans les autres générateurs.
• Logique de fallback kDrive dans onlyOffice.js sans try/catch: si getFileOrDirectory(kdriveId) échoue, kdriveDirectoryId reste undefined et l'upload échoue silencieusement sans feedback utilisateur.
• Variable fileName définie deux fois dans signed_pv_generator.ts (raw puis sanitized) - la version raw n'est pas utilisée, code mort qui prête à confusion.

Tour 2 : Préoccupations et questions

• RISQUE RGPD CRITIQUE : Console.log dans uploadFile.js expose noms de copropriétaires et IDs kDrive en production - données personnelles visibles dans les logs serveur
• Correctif FRAGILE : 5 patterns de null-handling différents (? '', ? '-', trim||fallback, || '', ?? '') sans utilitaire partagé - la prochaine maintenance cassera la cohérence entre convocation, listes de présence et PV
• Cause racine NON TRAITÉE : Des copropriétaires avec noms null en base de données indiquent un problème de qualité de données en amont (API Strapi, imports, synchronisation) - le fallback masque ce problème
• RISQUE JURIDIQUE : Documents légaux (PV d'AG, convocations) modifiés sans aucun test automatisé - une régression future pourrait invalider des documents à valeur légale sans détection
• Erreur silencieuse kDrive : Le fallback dans onlyOffice.js sans try/catch signifie qu'un échec getFileOrDirectory laisse kdriveDirectoryId undefined et l'upload échoue sans feedback utilisateur

• Console.log de debug dans uploadFile.js doivent être retirés avant production - fuite potentielle de données sensibles
• Dette technique de tests : ~5h pour couvrir les 5 corrections null et le flux kDrive
• Extraction utilitaire sanitizeFilename() recommandée si d'autres générateurs ajoutent la même logique
• Le fallback ' - ' pour noms vides mérite un guard supplémentaire pour éviter les tirets orphelins en préfixe de filename

• BUG CRITIQUE : fileNameSanitized calculé mais jamais utilisé dans signed_pv_generator.ts - fileName non sanitizé passé à uploadFile, échec d'upload pour caractères spéciaux dans noms de PPE
• Code mort : variable fileNameSanitized définie mais inutilisée, preuve d'un bug d'implémentation incomplète
• Console.log de debug en production dans uploadFile.js - risque de fuite de données sensibles (directory_id, filenames)
• Inconsistance null-handling : ?? '' dans ag_variables_getter.ts vs .trim() || ' - ' dans les listes de présence - affichages différents pour même donnée dans documents légaux
• Absence totale de tests (0%) sur 11 fichiers modifiés incluant des documents à valeur légale de copropriété

• Couverture de test de 0% confirmée - 0 fichier de test sur 11 fichiers modifiés, ratio test/code de 0:85 lignes
• Aucun test unitaire pour les 5 corrections null-coalescing - risque de régression d'affichage entre documents initiaux et finaux
• Regex de sanitisation filename sans test de cas limites (accents, Unicode, chaîne vide après sanitisation générant un tiret orphelin)
• Anti-pattern `throw Error(error)` dans getFileOrDirectory.js écrase stack trace Axios - aucun test pour vérifier le contexte d'erreur préservé
• Breaking change createOnlyOfficeDocument (number vers objet) sans test de régression pour les appelants existants

• BUG CRITIQUE signed_pv_generator.ts : fileNameSanitized calculé mais fileName (non sanitisé) passé à uploadFile - noms de PPE avec caractères / : * causeront des erreurs kDrive ou chemins corrompus sur documents légaux
• Console.log de debug en production dans uploadFile.js et onlyOffice.js - fuite de directoryId, fileName, chemins de fichiers légaux dans les logs serveur
• Inconsistance null-handling : ag_variables_getter.ts utilise ?? '' mais ag_list_presence_*_variables_getter.ts utilise .trim() || ' - ' pour le même domaine métier - violation DRY et cohérence d'affichage
• Breaking change createOnlyOfficeDocument : signature passe de (number) à ({kdriveId, kdriveParentId}) sans @deprecated ni surcharge rétrocompatible
• getFileOrDirectory.js : throw Error(error) écrase stack trace Axios et perd contexte HTTP (statusCode, config, url) - rend le diagnostic kDrive impossible en production

Tour 3 : Validation

• BUG CRITIQUE métier : fileNameSanitized calculé mais fileName non sanitisé passé à uploadFile dans signed_pv_generator.ts - les PV signés avec caractères spéciaux dans les noms de PPE échoueront silencieusement à l'upload kDrive, invalidant des documents à valeur légale
• Risque RGPD : console.log en production dans uploadFile.js expose noms de copropriétaires, directoryId et chemins de fichiers légaux - exposition de données personnelles dans les logs serveur
• Cause racine non traitée : les noms null en base de données indiquent un problème de qualité de données en amont que le fallback masque au lieu de résoudre
• Inconsistance d'affichage juridique : ?? '' dans ag_variables_getter.ts vs .trim() || ' - ' dans les listes de présence - le même copropriétaire apparaît différemment selon le document
• Erreur silencieuse kDrive : le fallback sans try/catch dans onlyOffice.js signifie qu'un échec getFileOrDirectory laisse kdriveDirectoryId undefined et l'upload échoue sans feedback utilisateur

• BUG CRITIQUE : signed_pv_generator.ts - fileNameSanitized calculé mais fileName non sanitisé passé à uploadFile, échec upload kDrive pour PPE avec caractères spéciaux (/ : *) dans noms
• Console.log RGPD : uploadFile.js:14-20 et onlyOffice.js:111-113 exposent directoryId, fileName, filepath en production - données personnelles copropriétaires dans logs serveur
• throw Error(error) dans getFileOrDirectory.js:20 écrase stack trace Axios - diagnostic production impossible, remplacer par throw error ou wrapper avec cause
• Null-handling inconsistency : ?? '' vs .trim() || ' - ' sans utilitaire partagé - maintenance fragile entre convocation_generator.ts et ag_list_presence_*_variables_getter.ts
• 0% couverture test sur 11 fichiers modifiés incluant documents légaux (convocations, PV d'AG, listes de présence)

• BUG CRITIQUE : fileNameSanitized calculé mais JAMAIS UTILISÉ - fileName non sanitisé passé à uploadFile dans signed_pv_generator.ts
• Console.log RGPD : 6 logs exposant directoryId/fileName/filepath en production (uploadFile.js lignes 14-19)
• Null-handling incohérent : ?? '' vs .trim()||' - ' vs || '' pour même domaine légal
• Breaking change createOnlyOfficeDocument : (number) → ({kdriveId, kdriveParentId}) sans @deprecated
• throw Error(error) écrase stack trace Axios dans getFileOrDirectory.js

• BUG CRITIQUE: fileNameSanitized calculé mais jamais utilisé dans signed_pv_generator.ts - un test unitaire simple aurait détecté ce bug immédiatement
• 0% couverture tests: 0 fichier de test sur 11 fichiers modifiés, 0 ligne de test sur +85 lignes ajoutées
• 5 patterns null-handling inconsistants sans tests paramétrés pour vérifier cohérence d'affichage entre documents légaux
• Breaking change createOnlyOfficeDocument (number→objet) sans test de régression pour appelants existants
• Console.log RGPD en production exposant directoryId, fileName, chemins fichiers légaux - aucun test de logging

• BUG CRITIQUE : fileNameSanitized calculé mais fileName non sanitisé passé à uploadFile dans signed_pv_generator.ts - échec d'upload pour caractères spéciaux dans noms de PPE
• Violation DRY : 5 patterns null-handling différents pour même domaine métier crée des incohérences d'affichage dans documents légaux
• Breaking change createOnlyOfficeDocument sans @deprecated ni surcharge rétrocompatible - risque pour appelants existants
• throw Error(error) dans getFileOrDirectory.js écrase stack trace Axios et perd contexte HTTP pour diagnostic production
• Console.log de debug en production dans uploadFile.js et onlyOffice.js - exposition de données personnelles (RGPD)