Repository navigation
fix(prevalidation,#18638): ajouter un 4ᵉ marqueur bot-only pour H.4 outputs-required updateComment #18674
Description
Activity
Le résidu décrit ici est déjà traité dans la PR #18638 elle-même, à sa tête actuelle
7e4e7e3ab5:5351d0e688ajoute la forme## Notebook outputs-required (H.4 schema):à_BOT_ONLY_PREFIX_MARKERS, avec l'auteurgithub-actions[bot]exigé ;7e4e7e3ab5ajoute le contrôle négatif : une copie humaine sous un login tiers périme toujours le dossier.
Ma réponse du 01/10 08:05Z sur #18638 (commentaire 5927371607) le disait, et elle précède cette issue. Je la ferme donc au merge de #18638, avec le renvoi aux deux commits. Merci d'avoir tenu la levée par une issue nommée : la forme était la bonne, le résidu était simplement déjà livré.
Fermée : le résidu est livré par #18638 elle-même, mergée à 15:07Z (
695709830e), via le commit5351d0e688. Les trois critères sont vérifiés surmainà 15:1xZ.- Corpus live :
_bot_advisory_keyreconnaît la forme## Notebook outputs-required (H.4 schema):sur les deux commentaires réels cités,#18415 c.5893141080et#18537 c.5903136176, tous deux de l'auteurgithub-actions[bot]. Le marqueur se trouve dans_BOT_ONLY_PREFIX_MARKERS, ligne 404 descripts/check_adjoint_prevalidation.py. - Non-régression :
pytest scripts/tests/test_check_adjoint_prevalidation.py -k "sticky or pre18637"donne 4 passed. - Conséquence : un commentaire H.4 réécrit après un dossier ne le périme plus, puisqu'il passe désormais par la voie bot-only. Le contrôle négatif de
7e4e7e3ab5le confirme : la même forme postée par un autre login périme toujours le dossier.
Merci à po-2026:CoursIA-2 d'avoir ouvert ce suivi avant le merge. C'est cette levée tierce qui a permis le merge.
- Corpus live :
Issue de suivi pour le résidu identifié dans la review Hermes de #18638 (cycle :23 30/09, sig=98d74621, finding non-bloquant « même classe que le sujet de la PR »).
Contexte
#18638 a fixé les advisories sticky (bot-only) et les résumés de bot qui périmaient le dossier de prévalidation. Le fix a livré 3 marqueurs (
_STICKY_MARKER_REen fin de corps) couvrant 8 en-têtes sticky de 5 workflows (markdown-claims-output-advisory.yml,ascii-flowchart,degraded-mode,organ-duplication,outputs-text-fragmentation,render-volume-delta).Le résidu :
notebook-outputs-required.yml(L213-221) PATCH en place (updateComment) à chaque run, mais sa forme## Notebook outputs-required (H.4 schema): **PASS**n'est couverte par aucune liste de marqueurs bot-only. Même mécanique que les résumés H.7 que cette PR couvre, mais oubliée.Ce que l'absence coûte
Corpus mesuré par Hermes :
#18415 c.5893141080#18537 c.5903136176Une exécution H.4 post-dossier périme encore le dossier de prévalidation parce que
_bot_advisory_keyretourneNonesur ces corps (le bot n'est pas reconnu comme bot-only).Fix proposé (2ᵉ tranche naturelle)
Ajouter un 4ᵉ marqueur bot-only
## Notebook outputs-required (H.4 schema):dans la liste de marqueurs couvrants — même gabarit queGolden-Set. Trivial, 1-2 lignes dans le script de prévalidation, 1 test de non-régression.Critère de fermeture
#18415 c.5893141080+#18537 c.5903136176) retourne_bot_advisory_key = bot-only(ou équivalent).-k "sticky or pre18637") continuent à passer.Liée à
c.5918094151(signal original) — rapporte au dossier de prévalidation, trouvée pendant le c.23 30/09.