🌊 Rapport d'analyse CodeWave

Tour 1 : Analyse initiale

• Risque mémoire critique : le passage aux buffers en mémoire (QrBillService, PdfService) sans limitation visible du nombre de documents fusionnables ni de leur taille maximale peut provoquer des OOM en production sous charge concurrente - impact business direct si le service devient indisponible
• Absence de tests automatisés pour download_pdf_controller.ts - seule approche manuelle mentionnée, insuffisante pour une fonctionnalité de téléchargement critique où une régression bloquerait les utilisateurs
• Mélange de préoccupations dans un seul commit : la nouvelle fonctionnalité métier (fusion PDF) et la refactorisation technique (buffers en mémoire) devraient être séparées pour faciliter rollback et revue - si la refactorisation introduit un bug, la fonctionnalité métier est aussi bloquée
• L'ajout de ky côté dashboard (package.json) sans documentation claire de son rôle par rapport à la gestion Ky backend crée une ambiguïté architecturale sur la stratégie de gestion d'erreurs
• La suppression de la gestion unlink dans QrBillService sans vérification apparente que tous les appelants sont mis à jour risque des régressions silencieuses dans les workflows existants de génération QR-bill

• Risque mémoire (OOM) si des PDF > 50MB sont traités en production, car les buffers multiples sont conservés en RAM pendant toute la fusion sans garde-fou.
• La fonction `cleanUpLocalFiles` utilise `unlink` asynchrone sans attendre la suppression réelle, ce qui peut laisser des fichiers orphelins si le processus s'arrête brutalement.
• Absence de tests unitaires automatisés pour le nouvel endpoint et la logique de fusion en mémoire, les tests manuels ne couvrant pas les cas limites (PDF corrompu, timeout Kdrive).

• Risque OOM critique sur download_pdf_controller : aucun plafond sur documentIds.length ni sur la taille totale des buffers. Recommandation : valider max(documentIds.length, 20) et ajouter un timeout sur les requêtes kdrive individuelles
• Absence de validation des documentIds dans le contrôleur - les IDs devraient être validés comme des entiers ou UUIDs avant d'être passés au service kdrive pour éviter des erreurs imprévisibles
• Aucun test automatisé dans le diff - PdfService.merge, QrBillService.generate et download_pdf_controller sont des candidats prioritaires pour des tests unitaires et d'intégration
• URL.revokeObjectURL appelé immédiatement après a.click() dans useDownloadDocuments.ts - le téléchargement peut ne pas s'initialiser à temps. Devrait utiliser setTimeout(100) pour délayer la révocation
• L'ancien pattern de TempFileUrlGetter qui retournait error.response.json() comme succès était un bug silencieux - vérifier qu'aucun consommateur ne dépendait de ce comportement pour gérer les erreurs

• Aucun fichier de test dans le commit - 0% de couverture test sur les nouvelles fonctionnalités
• Le nouveau endpoint download_pdf_controller.ts (+59 lignes) n'a aucun test unitaire ni d'intégration
• La refactorisation buffer vs fichiers temporaires modifie le comportement sans tests de régression
• La fonction cleanUpLocalFiles utilise un pattern callback asynchrone sans Promise wrapping, risquant des erreurs non interceptées et des race conditions
• La gestion centralisée des HTTPError Ky dans le handler d'exceptions n'est pas testée - impact sur toute l'application

• apps/backend/app/utils/files.ts : cleanUpLocalFiles utilise l'API callback `unlink` de 'node:fs' au lieu de 'node:fs/promises/unlink' - la fonction ne retourne pas de Promise, les suppressions sont fire-and-forget, créant des conditions de race si les chemins sont réutilisés avant suppression effective
• apps/backend/app/exceptions/handler.ts : `await error.response.json()` propage directement la réponse brute Kdrive au client - risque de fuite d'informations internes (structure API, messages d'erreur, identifiants de requête internes)
• apps/backend/app/controllers/add_qr_code_to_bills_controller.ts : incohérence architecturale - écrit sur disque via `pdf.move()` puis relit en mémoire via `readFile()`, contredisant la refactorisation en mémoire du PdfService
• apps/backend/app/controllers/add_qr_code_to_bills_controller.ts : `await` supprimé devant `pdf.move()` à la ligne 18 - l'écriture fichier n'est plus attendue, créant une condition de race avec le `readFile(pdf.tmpPath!)` subséquent qui dépend de cette écriture
• apps/backend/app/controllers/add_qr_code_to_bills_controller.ts : assertion non-null `pdf.tmpPath!` sans vérification - risque TypeError au runtime si le fichier temporaire n'est pas disponible après un move échoué

Tour 2 : Préoccupations et questions

• AUCUN test dans 23 fichiers modifiés pour un refactoring critique du workflow QR Bill - risque de régression sur les factures suisses
• Suppression de generateAndMerge() : les appelants existants (add_qr_code_to_bills_controller.ts) doivent être migrés vers le nouveau pattern ArrayBuffer
• Renommage ResponseOk → Response : type exporté publiquement, risque de casser les consommateurs TypeScript
• DocumentShareForm.tsx : nouvelle fonctionnalité UI sans tests E2E visibles pour valider l'expérience utilisateur
• Ajout de 'ky' : nouvelle dépendance HTTP (alternative à fetch/axios) ajoutant ~13KB au bundle et nécessitant une courbe d'apprentissage

• Absence tests unitaires PdfService.merge(ArrayBuffer[]), QrBillService.generate(), cleanUpLocalFiles() - risque régression silencieuse sur futures modifications
• TempFileUrlGetter sans try-catch : si API KDrive retourne erreur non-JSON, le handler global pourrait crasher de manière inattendue en production
• cleanUpLocalFiles utilise callbacks Node.js au lieu de promesses - incohérent avec style async/await du codebase
• useDownloadDocuments.ts manque documentation sur gestion erreurs côté client

• CRITIQUE - TempFileUrlGetter.ts : suppression du try/catch transforme les erreurs API Ky en exceptions non interceptées. L'ancien code retournait error.response.json(), le nouveau laisse l'exception se propager. Le type de retour Promise est maintenant inexact car les erreurs ne sont plus gérées dans cette fonction.
• CRITIQUE - useDownloadDocuments.ts lignes 18-22 : fuite mémoire potentielle. URL.createObjectURL() alloue une ressource navigateur qui doit être libérée. Si a.click() lance une erreur ou si le code entre createObjectURL et revokeObjectURL échoue, la ressource n'est jamais libérée. Correction : encapsuler dans try/finally.
• MAJEUR - Aucun test unitaire ou d'intégration pour les 3 nouvelles fonctionnalités (contrôleur, hook, service refactoré). Dette de test : ~3-4h.
• MAJEUR - QrBillService.ts : suppression de generateAndMerge() sans visibilité sur les appelants. Si cette méthode était appelée ailleurs, c'est une rupture silencieuse.
• MINEUR - useDownloadDocuments.ts ligne 20 : nom de fichier 'documents.pdf' codé en dur. Devrait être paramétrable ou inclure un identifiant.

• Aucun test automatisé ajouté pour les nouvelles fonctionnalités (contrôleur PDF, utilitaire files, routes)
• Suppression de la gestion d'erreurs dans TempFileUrlGetter sans tests de régression pour documenter le changement de comportement
• cleanUpLocalFiles utilise l'API callback fs.unlink au lieu de fs/promises, réduisant la testabilité
• Les erreurs dans cleanUpLocalFiles sont seulement loguées, pas propagées - comportement qui devrait être vérifié par des tests
• Absence de tests d'intégration pour les nouvelles routes API

• BUG CRITIQUE - AddQrCodeToBillsController : pdf.move() sans await suivi de readFile(pdf.tmpPath!). Condition de race : readFile s'exécute avant la fin du déplacement asynchrone. Impact : erreurs ENOENT intermittentes en production, non reproductibles, difficiles à diagnostiquer.
• PLACEHOLDER PRODUCTION - AddQrCodeToBillsController : filename='unittests.pdf' codé en dur dans Content-Disposition au lieu de pdf.clientName. Impact : tous les téléchargements portent le même nom non-descriptif.
• FIRE-AND-FORGET - utils/files.ts cleanUpLocalFiles : unlink() callback-based sans retour Promise. Impact : fichiers temporaires orphelins si le processus se termine avant le nettoyage asynchrone, fuite de disque progressive.
• SÉCURITÉ - exceptions/handler.ts : relayage aveugle de error.response.status + body JSON d'erreurs HTTP externes (kDrive, Strapi). Impact : exposition potentielle de détails internes, URLs, messages d'erreur serveur.
• DRY - AddQrCodeToBillsController lignes 23-26 : conversion Buffer→ArrayBuffer inline (buffer.slice avec byteOffset/byteLength). Impact : duplication inévitable si d'autres contrôleurs nécessitent cette conversion.