- IAuriel— Structure3 constats
- IIUriel— Sécurité4 constats
- IIICassiel— Patterns IA7 constats
- IVBarachiel— Performance2 constats
- VZadkiel— Qualité5 constats
- VIRaziel— Documentation3 constats
- Constats
- 24
- Majeurs
- 4
- Mineurs
- 20
- Voix dominante
- Auriel
Auriel
StructureTrois modules se citent en boucle, deux d’entre eux n’ont jamais été nommés ensemble. C’est une dette qui se paiera la prochaine fois qu’un nouveau venu cherchera la porte d’entrée.
Le dossier « core » a vu naître onze enfants en quatre semaines. Aucun n’a son aîné pour parrain. La généalogie reste à écrire.
Une dépendance circulaire silencieuse traverse vos trois domaines : auth → user → permissions → auth. Elle ne casse rien aujourd’hui — elle interdit les évolutions de demain.
Uriel
SécuritéUne clé d’API dort dans config.dev.ts depuis quarante-huit jours. Les humains qui ont accès à votre dépôt l’ont vue. Les robots aussi.
Trois autres traces de la même négligence figurent à la suite, classées par profondeur d’oubli plutôt que par gravité — l’oubli s’aggrave avec le temps.
Le middleware d’authentification accepte un en-tête signé sans vérifier sa fraîcheur. Un jeton volé reste valide jusqu’à expiration, plus longtemps qu’il ne le devrait.
Cassiel
Patterns IASept fonctions ont la même odeur — variables génériques, commentaires qui paraphrasent le code, structure trop régulière. Elles ont été écrites en série.
Elles peuvent être lues en série, ou repensées une à une. Le rythme est à vous.
Quatre TODO restent debout, oubliés depuis deux mois. Aucun d’eux ne désigne son auteur. Le code parle, mais ne se souvient plus de qui parlait.
Barachiel
PerformanceCette boucle s’exécute mille fois quand le composant monte. Mille fois pour calculer ce qui ne change jamais.
Une mémoïsation, ou un déplacement hors du cycle de rendu, suffirait. La décision vous appartient.
Un appel à la base est répété sans cache à chaque rafraîchissement de la liste. À petite échelle, c’est sans effet ; à mille utilisateurs, c’est un goulot.
Zadkiel
QualitéQuatre-vingt-trois pour cent du fichier respecte vos conventions. Le reste est récent, et tire dans une autre direction. Le moment d’arbitrer.
Treize fragments de code sont déclarés et jamais appelés. Ils ne nuisent pas. Ils s’accumulent.
L’indentation, la nomenclature, le séparateur de modules — tout cela tient sur une seule règle absente. Un linter à publier, un standard à graver.
Raziel
DocumentationLe README parle d’une commande qui n’existe plus. Le test du jeudi quatorze a été désactivé sans note. Ce sont les petites infidélités qui font qu’on cesse d’y croire.
Sept types publics ne portent aucune description. Ce qui ne s’écrit pas finit par se réinventer, mal.
Quatre dossiers n’ont pas de README. Un nouveau venu qui ouvrirait celui-ci passerait quinze minutes à comprendre où commence le travail.