Aller au contenu

root / tags / tests-automatises

#tests automatisés

2 fiches

Agents de codage IA & Skills

Maybe We Shouldn't Be Reviewing All This Code

Billet de **Rachel Laycock**, CTO de **Thoughtworks**, publié le **2 septembre 2026** sur *Rachel's Ramblings*, la section de martinfowler.com qu'elle tient, environ **8 000 caractères**. Le texte est une **réponse assumée** à [[houck-dx-what-are-code-reviews-for-2026-08-05]], après un désaccord public entre les deux auteurs sur un panel de **Code Remix** hébergé par **Moderne**. Laycock accepte le diagnostic chiffré de Houck (+**106 %** de lignes par diff chez **Meta**, +**64 %** de taille médiane de PR côté DX) et la liste des fonctions non-défectologiques de la revue, puis déplace la question : *« why are we waiting until code review to do all of those things? »* (A) Le principe directeur est le raccourcissement des boucles de rétroaction appris chez Thoughtworks — si un retour a de la valeur, ne pas le supprimer, le rapprocher de la décision qu'il informe. Chaque fonction attribuée à la revue est réaffectée en amont : exploration d'alternatives avant l'implémentation, transfert de connaissance par le **pair programming**, propriété collective par le **mob programming** et les sessions de design d'équipe, alignement architectural par le design commun puis les **fitness functions**, et automatisation de tout ce qui est déterministe — *« We really shouldn't still be arguing about whitespace in 2026. »* (B) La revue par exception remplace la revue systématique : changement architectural fondamental, frontière de sécurité sensible, rayon d'impact large, zone inconnue d'un système critique, ou simple déclaration d'inconfiance de l'équipe. (1) L'automatisation de la revue par un agent qui imite le relecteur humain est écartée comme *« automating the ceremony rather than questioning why the ceremony exists »*. (2) La dette cognitive de Houck est reconnue comme réelle, mais la PR obligatoire est jugée une défense faible : *« We need engineers to understand systems, not diffs. »*

#revue de code#revue par exception#pull request

Rachel Laycock · CTO de Thoughtworks · sur *Rachel's Ramblings* (martinfowler.com).