Domty × ReviaflowChaque PR back relue après sa mise en production : ce qu'elle fait, ce que j'ai vérifié, ce que j'ai trouvé, et ce qui s'est réellement passé en base.
Quatre livraisons du 26 août. Toutes conformes, une réserve de procédure sur l'une d'elles, et un constat qui vaut pour tout le mois : ce qui a été construit n'a pas encore servi.
Sept livraisons en quatre jours, quatre décisions d'architecture actées. Le découpage est juste et la conception souvent excellente. Deux réserves, dont une divergence entre deux objets jumeaux écrits à un jour d'intervalle.
Six livraisons, dont la seule du mois entier à avoir écrit sur des données réelles. L'écriture est propre et réversible, et elle corrige exactement le doublon que je t'avais signalé le 12 août.
Ce que sont devenues les 16 recommandations du 12 août, mesuré en base et dans le dépôt. Deux faites, deux entamées, neuf ouvertes — et deux qui n'auraient jamais dû t'être adressées.
Conforme, et c'est la mieux faite du lot. J'ai vérifié ton analyse jusque dans le code de l'outil de test : elle est exacte de bout en bout, y compris sur le point le plus fin. Une réserve de robustesse, pas un trou.
Conforme. Le diagnostic est juste, la correction est complète, et les huit défauts trouvés là où l'on en cherchait deux sont le bon résultat. Reste que la preuve vaut pour le 26 août, pas pour la suite.
Conforme. C'est la meilleure idée du lot, et sur ce point précis ton outillage passe devant le mien. Une réserve, et elle est lourde : rien ne déclenche jamais ce banc.
Rien de cette PR n'est en production. Mon avis avant merge est FAVORABLE. Le raisonnement est le bon et c'est peut-être la meilleure décision technique du lot, sous réserve de trois corrections de texte, dont aucune ne touche le script. La date affichée est celle de l'ouverture de la PR, le 21 août, pas une date de mise en production.
Conforme. Le correctif est posé à la bonne couche, chiffré, verrouillé par deux cas de banc qui mordent, et la PR rectifie franchement un diagnostic erroné de la veille au lieu de le laisser vivre. C'est la PR la mieux faite du lot.
Conforme avec réserve. La PR fait exactement ce qu'elle annonce et le fait bien, mais elle livre un banc dont aucun cas ne garde la direction que son propre README désigne comme la seule grave, et que rien ne déclenche.
Non conforme en l'état. Le remplacement de la règle par un script qui lit la requête est le bon choix, et il n'y avait pas d'autre voie. Mais la PR ouvre un chemin par lequel une écriture qui valide obtient le silence total, sur un garde dont c'est la seule sortie silencieuse, et elle retire au passage le seul filet qui restait quand le script ne tourne pas. Les deux se corrigent, l'un en une ligne.
Repassée en BROUILLON par son auteur : elle n'est en vigueur nulle part et rien de ce qu'elle contient n'est en production. Mon avis avant merge est DÉFAVORABLE EN L'ÉTAT : ton propre document de décision interdit son application tant que #281 n'est pas mergée, et trois points de forme restent à reprendre. La forme, elle, est conforme à la condition 5.b. Sur le principe même de l'exception, je te réponds séparément : ça ne se tranche pas dans une fiche de review. La date affichée est celle de l'ouverture de la PR, le 18 août, pas une date de mise en production.
Conforme au contrôle mécanique, avec une réserve d'architecture majeure. L'index est correct, testé, utilisé, documenté au-delà de ce que la règle exige. Mais il normalise les noms exactement comme un lecteur dont la normalisation est mesurément cassée : il interdit une classe de doublons jamais observée en production et laisse debout celle qui a déjà coûté 8 846 € HT, aujourd'hui réarmée sur Würth.
Conforme au contrôle mécanique, avec une réserve fonctionnelle. Le ménage est propre, réversible, mesurément juste, et il a débloqué 8 rattachements automatiques dès le lendemain. Mais il ferme la cause sans libérer le stock qu'elle avait produit : 4 factures fournisseur restent hors de la comptabilité alors qu'un clic les y ferait entrer.
Conforme, et le cloisonnement tient : je l'ai attaqué en production, pas relu. Reste une question de fond à trancher pendant que le dossier est encore vide, et une description de PR qui ne décrit pas ce qui a été mergé.
Conforme. Le code livré est correct et plus strict que ce que sa description annonce, la CI était verte avant le merge, et il n'a touché aucune ligne de donnée. Deux points restent à reprendre, aucun ne renverse le verdict.
Une PR de dette, faite proprement : elle déclare une règle qui ne tenait que par convention, ferme un trou qu'elle a découvert en la déclarant, et documente ce qu'elle ne fait pas. Seule PR du lot dont les deux contrôles sont verts.
L'exécution la plus solide du lot. J'ai éprouvé le chemin de bout en bout sur une pièce réelle : il fonctionne, et les cinq refus mordent. Une garde reste ouverte d'un côté, et le mécanisme attend toujours son bouton.
Techniquement la meilleure PR du lot. Mais le mécanisme qu'elle installe garde un chemin par lequel un document peut disparaître définitivement de la liste des manquants, et c'est exactement ce que cette PR existe pour empêcher.
Le bon arbitrage : la règle descend dans la base, donc elle tient quel que soit celui qui écrit. Le correctif de signe, lui, ferme le cas mais pas la classe, et deux lignes en production le montrent.
Tu fermes une garantie que la veille tu affirmais par écrit sans qu'elle existe. Le correctif tient, je l'ai éprouvé en production. Une porte reste entrouverte, et c'est la PR suivante qui la rend exploitable.
Deux lignes, et le bon réflexe : la règle est écrite contre son lecteur, pas seulement pour lui. Elle transporte en revanche deux imprécisions dans le document qui fait autorité.
Le mécanisme est le bon, et tu l'as éprouvé dans les deux sens avant d'y croire. Mais il ne protège qu'un dossier sur les trois où tu travailles, et il laisse deux outils dehors.
La PR la plus propre du lot. Un revert motivé, qui préserve la garantie qu'il aurait pu emporter, et vérifié avant d'être écrit. Le seul défaut est un décompte, et il s'est propagé jusqu'au document de référence.
La réparation elle-même est juste et rapide. Mais le nouveau fichier de tests livré à côté ne contient aucune des cinq vérifications que la règle du projet exige, et il annonce garantir une propriété qui était fausse au moment où il passait au vert.
Le lot répare un vrai trou, et il a déjà servi. Mais deux points de méthode passent avant le code : la PR est partie en production avec les tests au rouge, et l'alerte quotidienne qui surveille la production sonne sans réponse depuis sept jours.
Le trou est bien fermé, et les protections tiennent à l'épreuve du réel. Deux réserves : une ligne de documentation qui contredit ton propre code, et une table que personne ne lit.
Le rattrapage est borné exactement à ce qui pouvait l'être, et il laisse à vide ce qui doit rester inconnu. C'est le bon arbitrage.
Une correction de données sur une pièce comptable, faite exactement comme il faut. J'ai rejoué le résultat en base : ça boucle au centime.
La donnée juste était lue sur le PDF puis jetée. Elle est enfin conservée — reste à la faire lire par les calculs qui utilisent encore l'ancienne méthode.
La plus importante des trois : sans elle, le récapitulatif porte plus de CA que la facture, en silence. Trois gardes bien placées, et deux pièges d'arrondi rattrapés avant la mise en production.
Une ligne de code, un import débloqué. Et une hypothèse fausse rattrapée avant d'être gravée dans la documentation.
Une garde héritée du budget bloquait des factures d'avoir légitimes. Levée au bon endroit, et seulement pour la facture.
Le dernier module resté sur Airtable rentre à la maison. Deux affirmations structurantes vérifiées en base : les deux sont exactes, et l'une durcit le patron plutôt que de le copier.
Celle-ci est la mienne, pas la tienne : je la consigne ici pour que le registre soit complet, et parce que la règle qu'elle ajoute te concerne aussi.
La PR du lot qui a le plus écrit en production, et elle tient : 329 factures actives, zéro doublon, les affectations bouclent au centime. Une réserve sur ce qui n'est pas vérifié dans ce que le modèle lit.
La vue est juste et bien exposée. Mais la colonne « Facturé » affiche aujourd'hui 7 631 € là où la base en porte 6,65 millions, sans rien qui le signale.
Le seul lot du lot d'août qui ait déjà servi pour de vrai. Une réserve : déplacer une affaire ou un chantier d'organisation rend ses notes définitivement immodifiables.
Décalque d'un patron déjà en place, exécuté sans écart. Rien à reprendre.
Une des migrations les mieux tenues du dépôt. Une seule réserve, et elle porte sur ce que la dérivation rend possible, pas sur ce qu'elle fait.
Le défaut est bien fermé, et la façon dont tu l'as fermé est la bonne. Une réserve sur un chemin symétrique qui n'a pas reçu la même protection.
Rien à reprendre sur le fond. Une réserve d'architecture pour la suite, et une suggestion sur l'endroit où consigner un revirement.
Un point à discuter de vive voix, ou un constat à contester sur pièce ? Prends un créneau dans mon agenda.