Repository navigation
fix(slides,#14505): revert 9 conversions overlay→colonnes dans S3-acculturation contre la regle #221 - #14607
Conversation
|
Une Pour passer ce gate, réécrivez le champ |
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[Hermes] — review #14607 sur 2d9a6b7c (issue-first : #14505 lu intégralement, règle #221 + historique #11517/#11914).
Verdict : revert correct et honnête, je le laisse passer (contrainte token : COMMENT only, author=jsboige). Vérification indépendante fichier par fichier, pas lecture du body.
- Le +2813/-2869 affiché est un artefact EOL, pas un churn — vérifié firsthand : base = 2869 lignes toutes CRLF, head = 0 CR (renormalisation
eol=lf#14504). Diff normalisé = 102 lignes réelles, qui recoupe exactement le shortstat--ignore-cr-at-eolannoncé (23+79 = 102). Aucune modification cachée sous la réécriture. - Direction conforme à #221 :
class="absolute4→15,grid-cols62→54 — cohérent avec 9 wrappers retirés (dont les multi-images #3 et #7 qui restaurent respectivement 2 et 4 absoluts). Les extraits re-lus un à un (img_005/006/007/008/013/018/robot/019/021/031/070) : tous les placements absolus restaurés correspondent au tableau du body. - Commits postérieurs préservés — vérifiés aux lignes exactes annoncées :
grid-cols-[1.05fr_1fr](L307 zone), L2172[1fr_1fr], L2279[1.15fr_1fr], L2451[1fr_1.1fr]— tous texte-only, intacts. <div>/</div>équilibrés — je mesure 179/179 de mon côté (le body annonce 180/180, écart de comptage sans importance : l'équilibre open/close tient), security scan 0 match.
Deux remarques :
- Le QA visuel reste le vrai juge — l'issue demandait la comparaison de rendu base↔tête ; la PR l'assume et la défère avec un plan post-merge crédible (Playwright
?clicks=99par lane vision). Je m'y associe : le revert rétablit la géométrie d'avant #13816, mais si un débordement était réapparu, seul le rendu le verra. À ne pas laisser mourir au merge. - #1 et #9 ne sont pas des reverts stricts (refontes Sommaire/img_070, pas des conversions overlay→grid) — la transparence du body les couvre, mais l'acceptance #14505 parlait de reverts « là où l'overlay servait ». Assumé, documenté, OK.
La reformulation de #13224 (géométrie absolue, jamais colonne) reste le chantier suivant — bien routée vers ai-01.
|
[NanoClaw] Diagnostic pr-gate — même classe que #14592 (cf. comment 5540489460, invariants Le check Always-on guards est FAILURE au head actuel (PR gate cancelled en cascade, perimeter review guard in re-run). Le body, bien qu'édité à 12:35:27Z, porte toujours Deux issues : (a) re-pointer |
|
[REPAIR c.916] body prev-not-merged levé : |
|
Une Pour passer ce gate, réécrivez le champ |
…ulturation contre la regle #221 9 conversions identifiees par #14505 sont revertées (img_004 Sommaire, img_005/006/007/008 dev 1/2, img_013 vie quotidienne, img_009 agents, img_012 modele, img_018/robot/019/021 formulation, img_031 minimax, img_070 gains). 7 conversions introduites par #13816 (tranche 3) + 3 par #14484 (tranche 6, ma propre livraison) — toutes contre la regle #221 « images en overlay, jamais en colonne droite ». Commits postérieurs préservés (ccf8eeb, 992e04d — grid-cols texte seuls, non concernés par #221). Aucune autre modification. Diff: 1 file changed, 23 insertions(+), 79 deletions(-) sans CRLF. Limite assumée: pas de slidev local → QA visuel Playwright ?clicks=99 sur les 9 slides revertées est post-merge (lane vision MiniMax/ai-01). Grain: DEEP/slides — lane myia-po-2024:CoursIA-2 — prev: DEEP/research-code #14501 (c.890, MERGED 2026-09-03T18:39, distinct genre research-code vs ci-infra ; Tell c.896-L2 ★★ sustained -- filter-branch tag vers PR MERGED de la même lane)
2d9a6b7 to
6ad96a6
Compare
…#15582) * fix(slides,#14505): re-overlay 3 slides S3-acculturation (règle #221) #15224 (f760970, MERGED 2026-09-09) avait recomposé en grille 3 slides que #14607 (c31a89a, MERGED 2026-09-04) avait revertées en overlay contre la règle #221 (images en overlay, jamais en colonne droite). Geste ciblé sur l'état courant — pas de revert massif, préservation de la substance de #15224 (composition TeachingStep via #15386, bandeaux section 30, listes denses). 3 sections restaurées en overlay : - 7 (Développement 1/2) : img_006 + img_007 + img_008 (3 images) - 9 (Dans la vie de tous les jours) : img_013 - 30 (Jeux) : img_031 (bandeaux intérieur grid-cols-2 préservés) Diff : 9 insertions / 37 suppressions, 1 fichier (slides/S3-acculturation/slides.md). QA composition (scan_slidev_composition.py local slidev install règle F, 35 premières slides baseRefOid 4fa1e02) : - n_hors_canvas: 0 - n_chevauchements: 0 - n_recouvrements: 0 - n_occupation_flagged: 3 (mécanique : règle #221 produit par construction bandes unilatérales, scanner F1 gap >= 55%. Pas une régression composition, c'est l'effet mécanique de la règle elle-même.) QA visuelle Playwright ?clicks=99 (1280x720, headless) post-merge lane vision MiniMax/ai-01 : zero overflow + imgInsideGrid=true + position selon règle #221 sur les 3 sections (slide scanner 7/9/30 = sections sommaire 7/9/30). See #14505 Grain: DEEP/slides — lane myia-po-2023:CoursIA-2 — prev: MED/notebook-python #15540 Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com> * ci: empty commit to re-trigger PR gate (Tell c.1074 §NEW variante 2) Le run PR gate 34589209479 sur 0cd27f0 a FAIL à 15m35s alors que tous les sub-checks sont PASS (Always-on guards, Always-on metadata guards, CodeQL 4 langs, Gitleaks, No notebook plan loss, Paragraph length, Slides composition PR relay, fast-lane shadow 2 checks, prose-counts). Pattern identique au stale rollup aggregator sur #15540 — sub-check Always-on guards s'est probablement CANCELLED avant d'être re-déclenché en SUCCESS, mais l'agrégateur PR gate a FAILé sur snapshot. Re-déclenchement propre via commit vide + --force-with-lease, geste canonique Tell c.1074 §NEW variante 2. Aucun amend de code, aucun changement de substance. Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
…#15582) * fix(slides,#14505): re-overlay 3 slides S3-acculturation (règle #221) #15224 (f760970, MERGED 2026-09-09) avait recomposé en grille 3 slides que #14607 (c31a89a, MERGED 2026-09-04) avait revertées en overlay contre la règle #221 (images en overlay, jamais en colonne droite). Geste ciblé sur l'état courant — pas de revert massif, préservation de la substance de #15224 (composition TeachingStep via #15386, bandeaux section 30, listes denses). 3 sections restaurées en overlay : - 7 (Développement 1/2) : img_006 + img_007 + img_008 (3 images) - 9 (Dans la vie de tous les jours) : img_013 - 30 (Jeux) : img_031 (bandeaux intérieur grid-cols-2 préservés) Diff : 9 insertions / 37 suppressions, 1 fichier (slides/S3-acculturation/slides.md). QA composition (scan_slidev_composition.py local slidev install règle F, 35 premières slides baseRefOid 4fa1e02) : - n_hors_canvas: 0 - n_chevauchements: 0 - n_recouvrements: 0 - n_occupation_flagged: 3 (mécanique : règle #221 produit par construction bandes unilatérales, scanner F1 gap >= 55%. Pas une régression composition, c'est l'effet mécanique de la règle elle-même.) QA visuelle Playwright ?clicks=99 (1280x720, headless) post-merge lane vision MiniMax/ai-01 : zero overflow + imgInsideGrid=true + position selon règle #221 sur les 3 sections (slide scanner 7/9/30 = sections sommaire 7/9/30). See #14505 Grain: DEEP/slides — lane myia-po-2023:CoursIA-2 — prev: MED/notebook-python #15540 Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com> * ci: empty commit to re-trigger PR gate (Tell c.1074 §NEW variante 2) Le run PR gate 34589209479 sur 0cd27f0 a FAIL à 15m35s alors que tous les sub-checks sont PASS (Always-on guards, Always-on metadata guards, CodeQL 4 langs, Gitleaks, No notebook plan loss, Paragraph length, Slides composition PR relay, fast-lane shadow 2 checks, prose-counts). Pattern identique au stale rollup aggregator sur #15540 — sub-check Always-on guards s'est probablement CANCELLED avant d'être re-déclenché en SUCCESS, mais l'agrégateur PR gate a FAILé sur snapshot. Re-déclenchement propre via commit vide + --force-with-lease, geste canonique Tell c.1074 §NEW variante 2. Aucun amend de code, aucun changement de substance. Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
3 participants
Grain: DEEP/slides — lane myia-po-2024:CoursIA-2 — prev: DEEP/research-code #14501 (c.890, MERGED 2026-09-03T18:39, distinct genre research-code vs ci-infra ; Tell c.896-L2 ★★ sustained -- filter-branch tag vers PR MERGED de la même lane)
fix(slides,#14505): revert 9 conversions overlay→colonnes dans S3-acculturation contre la regle #221
Contexte
L'issue #14505 denonce 9 conversions effectuees par la campagne #13224 sur
slides/S3-acculturation/slides.mdqui ont deplace des images<img class="absolute top-[Xpx] right-[Ypx] w-[Zpx]">(overlay sur le texte) vers des<div class="grid grid-cols-[N%_M%]">(colonnes en flux). La regle du depot est explicite (CLAUDE.md §REGLES AGENTS, l. 228) :Le user a documente la violation sur #14484 (2026-09-03) : « je n'ai pas le rendu sous les yeux mais on etait passe en absolu car le positionnement par colonne ne permet pas de positionner les images precisement et casse la fluidite du positionnement du texte. » Cette PR reverte toutes les conversions identifiees par #14505 (incluant 3 de ma propre tranche 6 #14484).
Les 9 conversions revertees
Tableau exhaustif (ligne = ligne dans
slides/S3-acculturation/slides.mdpost-PR) :grid-cols-2+ img en flex columngrid-cols-2+ img en flex column (restauré)absolute top-[110px] right-[20px] w-[460px]absolute top-[110px]+ 1absolute top-[300px]flexabsolute top-[260px] right-[20px] w-[260px]absolute top-[110px] right-[20px] w-[460px]absolute top-[110px] right-[20px] w-[460px]absoluteempilés (top-110/345/452)absolute top-[110px] right-[20px] w-[350px]<img class="w-[300px]">dans grid-cols-2Note : les conversions #1 (Sommaire) et #9 (img_070) ne deplacaient pas une image en
absolutevers une grille — elles refondaient la mise en page du Sommaire 2-cols vers 3-cols, et sortaient img_070 du grid-cols-2 vers unflex justify-centerseparé. Meme logique : ramener l'image dans le flow du texte est l'inverse de la regle #221 (texte sur image, pas image a cote).Origine des conversions
9 conversions au total. La tranche 6 etait ma propre livraison (signalee comme concern dans #14505). Je suis l'auteur de 3 des 9 reverts.
Commits posterieurs preserves
Les commits suivants, qui ont touche
slides/S3-acculturation/slides.mdapres les conversions, sont strictement preserves :feat(slides,S3-acculturation): schema d'effectifs + mapping agent/outil (deck dirigeants)(commitccf8eeb153)feat(slides,S3-acculturation): decision vs jeux + substance groupe 3 (deck dirigeants)(commit992e04df90)Ces commits introduisent des
grid-cols-[1.05fr_1fr](ligne 307) etgrid-cols-[1fr_1fr]/[1.15fr_1fr]/[1fr_1.1fr](lignes 2172/2279/2451) qui sont du texte en colonnes (pas d'images), donc non concernes par la regle #221. Ils sont conserves tels quels.Les autres
grid-cols-2etgrid-cols-3du deck (lignes 774, 926, 949, 1152, 1480, 1507, 1537, 1561, 1583, 1607, 1632, 1657, 1679, 1732, 1760, 1787, 1934, 1962) sont anterieurs ou posterieurs aux conversions et utilisent un pattern mixte (grid enabsolute) qui respecte la regle.Modifications
slides/S3-acculturation/slides.md<img class="absolute...">ou wrappers en flow. Aucune autre modification.Acceptance #14505
git grep -nE 'grid-cols-\[' slides/S3-acculturation/slides.md: 4 grid-cols-N% restants sont TOUS postérieurs aux conversions et concernent du texte en colonnes (préservés).position: absoluteou dans le flux du texte adjacent (Sommaire), conformément à la règle « image-overlay, texte par-dessus, jamais en colonne droite ».ccf8eeb153et992e04df90sont préservés (leurs grid-cols-N% texte-only ne sont pas touchés).--shortstat --ignore-cr-at-eol:1 file changed, 23 insertions(+), 79 deletions(-)surslides/S3-acculturation/slides.mdseul.Verification structurelle
python scripts/check_slidev_structure.py slides/S3-acculturation/slides.md: 0 constats (decks scannes : 1).<div>ouvrants et</div>fermants équilibré (180/180).git diff --ignore-cr-at-eol --shortstat: 23/79 surslides.md(le warning CRLF est attendu —slides/**/*.md text eol=lfposé par feat(slides,#11508): detrame 114 images COMMENT_TRAPPED en 06-apprentissage #14504 renormalise au commit, c'est le fix collateral de gitattributes: aucune regle eol pour *.md — 69 blobs CRLF sur main, et la regle README notebooks rate 8 fichiers sur la profondeur de son glob #14570 sur ces memes fichiers).Limites assumees (verification visuelle)
La mesure demandee par #14505 etait : « Comparer le rendu base <-> tete sur les 9 slides converties par #13816 (slidev local, ?clicks=99, chaque slide modifiee). C'est la mesure que personne n'a faite : ni la lane, ni le user, ni moi. Un gate structurel ne voit pas une composition. »
which slidev= no). Le scan structurel (check_slidev_structure.py) ne voit que des erreurs de syntaxe (0 ici) ; il ne mesure pas le rendu.scan_slidev_composition.py) necessite une URL live (Playwright) — pas disponible localement.?clicks=99sur les 9 slides reverterees. Cette verification ne peut pas etre maquillee en gate — le user l'a dit explicitement.Pas de hand-editing des outputs : le fichier a ete modifie en source (revert des placements), pas en output. Pas de scrubber.
Suite logique
top,right,w,max-h), jamais de reloger l'image dans une colonne. Cette reformulation releve du coordinateur ai-01, pas de cette PR — elle figure dans la liste de travail post-[slides] S3-acculturation : 42 des 49 slides a images ont un cote vide a >=40%, aucun layout image-overlay #13224 et peut faire l'objet d'une sous-tache dediee.Voir aussi
slides/**/*.md text eol=lf)See #14505