Un avertissement sans issue est une plainte : dire quoi faire, pas seulement quoi - #54
Merged
Merged
Conversation
…ulement quoi Le message posé hier à la place de l'arbre expliquait exactement la plateforme — async-profiler, aucun binaire Windows, le reste est mesuré — et laissait le lecteur sans rien à faire. C'est ce qu'il a répondu le lendemain matin : « ce message n'est pas concret, il ne dit pas où est le problème ni comment le résoudre ». Il a raison, et c'est la règle que ce projet applique partout ailleurs, jusque dans le conseil d'exclusion de Footprint. La branche Windows nomme donc WSL, dit que le jar et les options ne changent pas, et donne le seul piège de cette route : garder --out côté Linux plutôt que sous /mnt/c, parce que le rapport écrit des milliers de petits fichiers et que cette traversée est lente. Les quatre autres causes avaient déjà leur geste ; ViewContractTest tient maintenant les cinq gestes et non plus seulement les cinq diagnostics. La ligne de console au lancement portait la même plainte — « timing measurements unavailable on this platform » — et reçoit le même traitement, avec le nom du système plutôt que « cette plateforme ». Vérifié au rendu : 0 différence sur les 8 610 chaînes des dix-huit états, la bande Windows étant lue sur un contexte forcé. 296 tests, 72 vérifications de recette. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J956wjynbd7fkZx4HpjHzP
Beennnn
marked this pull request as ready for review
September 7, 2026 08:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Le message posé hier à la place de l'arbre expliquait exactement la plateforme — async-profiler, aucun binaire Windows, le reste est mesuré — et laissait le lecteur sans rien à faire. C'est ce qu'il a répondu le lendemain matin : « ce message n'est pas concret, il ne dit pas où est le problème ni comment le résoudre concrètement ».
Il a raison, et c'est la règle que ce projet applique partout ailleurs, jusque dans le conseil d'exclusion de
Footprint: un avertissement sans issue est une plainte.Avant / après
Avant
Après
Le piège
/mnt/cest nommé parce qu'il est la conséquence directe du modèle de coût déjà mesuré dans ce dépôt : le prix est le nombre de fichiers multiplié par la latence d'unopen, et une traversée 9p est précisément une latence paropen.Les cinq causes portent maintenant leur geste
/mnt/c--level coverage--levelSOURCE_DIRS--classeset--sourcesViewContractTesttient désormais les cinq gestes et non plus seulement les cinq diagnostics : une cause qui perdrait son issue casse la construction.La console avait la même plainte
devient
Le nom du système remplace « cette plateforme » : sur la machine où on lit ça, il n'y a rien d'autre pour vérifier.
Vérification
🤖 Generated with Claude Code
https://claude.ai/code/session_01J956wjynbd7fkZx4HpjHzP
Generated by Claude Code