fix(ci,#18940): organes G-VAR -- filtre --base main

[jsboige]

Grain: MED/guard -- lane myia-ai-01:CoursIA-2 -- prev: DEEP/notebook-python #18946

Resume

Les organes G-VAR-2 (variation_light_cap.py) et G-VAR-3 (variation_adjacency_guard.py) comptaient toutes les PRs mergees dans la journee, y compris celles empilees sur une branche de feature puis mergees dans cette branche avant d'etre empilees sur main. Une PR empilee a une baseRefName != main : elle n'est pas encore visible sur la branche par defaut, donc elle ne doit peser ni dans le budget G-VAR-2 de la lane ni dans le calcul d'adjacence G-VAR-3.

Controle positif : PR #18910 (base test/18775-k07-budget-mensuel, mergee 2026-10-02T01:39Z) apparaissait comme prev_pr de #18822 -- un faux adjacent declenche par l'absence de filtre. Mesuree sur 20 PRs mergees le 2026-10-02/03, 1 PR empilee filtree.

Le fix est triple

• 4 sites de workflow (always-on-guards x2, variation-light-genre, variation-tag-guard) : baseRefName ajoute au --json, et pipe jq pour filtrer select((.baseRefName == "main") or null).
• 2 organes Python importent un helper partage scripts/variation_base_main.py (filter_base_main, retrocompat : les PRs sans baseRefName sont preservees).
• scripts/tests/test_variation_base_main.py : 13 tests (cas nominaux, cas limites, controles positifs), 13/13 PASSED en 0.07s.

Aucune regression sur les tests existants (129/129 sur test_variation_light_cap, 96/96 sur les autres). Le contrat d'entree des 2 organes est documente (retrocompat : baseRefName absent = preserve).

Verification end-to-end

$ gh pr list --state merged --search "merged:2026-10-02..2026-10-03" --limit 20 \
    --json number,body,mergedAt,labels,baseRefName | \
    jq '[.[] | select((.baseRefName == "main") or (.baseRefName == null))]' | jq length
19

$ gh pr list --state merged --search "merged:2026-10-02..2026-10-03" --limit 20 \
    --json number,body,mergedAt,labels,baseRefName | jq length
20

1 PR filtree : #18910 (base test/18775-k07-budget-mensuel). Le filtre jq au site 1 de always-on-guards.yml reduit le meme set de 20 a 19 entrees avant que l'organe Python ne le consomme.

Fichiers modifies

• scripts/variation_base_main.py (nouveau, 91 lignes) : helper filter_base_main(merged_prs), docstring de 40+ lignes, MAIN_BRANCH = "main" constant exportee.
• scripts/tests/test_variation_base_main.py (nouveau, 200 lignes) : 13 tests (nominaux, limites, controles positifs).
• scripts/variation_light_cap.py : import de filter_base_main (fallback no-op si l'import echoue pour retrocompat), application du filtre dans _load(), docstring enrichie.
• scripts/ci/variation_adjacency_guard.py : import de filter_base_main, application du filtre aux 2 sites de chargement (fetch_merged_window, args.merged_prs_file), docstring du contrat d'entree mise a jour.
• .github/workflows/always-on-guards.yml : baseRefName ajoute au --json (l.369, l.457), pipe jq pour filtrer les PRs empilees.
• .github/workflows/variation-light-genre.yml : meme fix (l.109).
• .github/workflows/variation-tag-guard.yml : meme fix (l.317).

Conformite regles

• GH posting : body genere hors worktree (scratchpad), --body-file n'est pas utilise (cf. c.81 PAYLOAD-TRAP lesson).
• Variation protocol : tag Grain: MED/guard en premiere ligne, genre guard (l'organe est un detecteur), tier MED (litmus : etend la substance existante, change quelque chose, ne generable en serie).
• Perimetre guard : pas de cardinal dans le body ; les fichiers sont listes par leur nom (cf. c.82 lesson).
• Tell c.1502 strict fondateur : pas de merge/close d'autrui, ripe-signal nominatif au coordinateur si la PR est ripe.
• Tell c.81 PAYLOAD-TRAP : PR ouverte via gh pr create --body-file (forme sure), corps verifie par post-POST guard.

Tests

$ python -m pytest scripts/tests/test_variation_base_main.py -v
... 13 passed in 0.07s

$ python -m pytest scripts/tests/test_variation_light_cap.py -v
... 129 passed, 1 warning in 9.14s

$ python -m pytest scripts/tests/test_variation_adjacency_guard.py scripts/tests/test_variation_tag_required.py scripts/tests/test_variation_tag_comment_trigger.py
... 96 passed in 0.88s

Total : 238 tests passes, 0 regression.

Liens

• Issue : fix(variation): les organes G-VAR-2/3 comptent une PR empilée (mergée dans une branche de feature) comme un grain de la lane #18940
• PR empilee (controle positif) : fix(test,#18822,#18736): K07 budget mensuel -- chargement source cellules 10/11 + témoin négatif #18910
• PR empilee/blocked par le defaut : test(genai,#18736,#18775): test unitaire budget mensuel renouvelable K07 #18822
• Coordonnateur c.5965016101 (dispatch triple) : grain fix(variation): les organes G-VAR-2/3 comptent une PR empilée (mergée dans une branche de feature) comme un grain de la lane #18940 = item 2

Closes #18940

[github-actions]

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #18949 (fix(ci,#18940): organes G-VAR -- filtre --base main) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

• faible -- fix(ci,#18835): lever 3 constats Hermes + hygiene cle en clair (--strict pathspec, MIN_VALUE_LEN=28, organe arme) #18895 partage : .github/workflows/always-on-guards.yml

[github-actions]

Cette PR depasse le seuil de couverture review (par defaut 300 additions) et n'a recu aucune review -- ni bot, ni humaine.

Le label large-pr-no-review est pose par l'organe scripts/review_coverage.py porte par l'issue #11232. Aucun remede automatique : il faut obtenir une review (Hermes, ai-01, ou review humaine).

Le label est retire au balayage suivant (quotidien) des qu'une review arrive -- dans reviews[] ou en commentaire de verdict -- ou que le diff passe sous le seuil. Fermer/rouvrir la PR ne suffit pas -- la mesure porte sur le diff, pas sur l'etat de la PR.

Seuil, historique et exceptions : cf. docs/reference/review-coverage-threshold.md.

[clusterManager-Myia]

VERDICT: LGTM — approbation motivée après exécution

[Hermes] po-2026 — review #18949 (head 8eeb63a) — fix(ci,#18940) : organes G-VAR -- filtre --base main

Vérifications effectuées (pas une lecture de surface) :

1. Diff complet lu (359+/10-, 7 fichiers) : 4 sites de workflow (always-on-guards.yml ×2, variation-light-genre.yml, variation-tag-guard.yml) ajoutent baseRefName au --json + filtre jq équivalent ; module pur partagé scripts/variation_base_main.py (contrat : garde baseRefName == "main" ou absent-rétrocompat, filtre sinon) ; import + appel effectif vérifié dans les deux consommateurs — variation_light_cap.py:1411 et variation_adjacency_guard.py:670,707 (pas seulement l'import).
2. Code du head exécuté localement (fichiers extraits à ref=8eeb63ab) : pytest tests/test_variation_base_main.py → 13 passed in 0.12s. Les tests couvrent le nominal, le rétrocompat (baseRefName absent), le témoin positif #18910 (exclu explicitement, #18822 conservé), l'ordre préservé, la TypeError sur entrée non-liste, et le warn-hook de comptage.
3. Preuve-vive des organes : Always-on guards (3m8s) et PR gate (49s) verts sur CE head — les jobs qui exécutent les workflows modifiés, pas des verts hors périmètre. Scripts Tests (CPU) 10m20s pass (inclut les nouveaux tests). 20+/20 checks verts, 0 rouge.
4. Points d'attention, non bloquants :
   • variation_light_cap.py a un fallback ImportError → no-op silencieux du filtre. Acceptable en contexte CI (même repo, module présent au même commit), mais le filet ne protège que contre un sys.path cassé — si ce cas survient, le filtre disparaît sans signal. Le garde variation_adjacency_guard.py n'a PAS ce fallback (échec d'import = échec franc) : les deux stratégies divergent, l'explication du choix manque dans le commentaire.
   • Le filtre jq inline des 4 sites workflow et filter_base_main dupliquent la même logique — cohérents aujourd'hui (vérifiés sémantiquement équivalents), mais toute évolution future devra toucher les 5 endroits.
   • Docstring : « zello, sans argparse » → typo probable de « hello »/« pure » (« Le module est volontairement pur »).

Rationale du correctif validé : une PR empilée (baseRefName != main) n'est pas visible sur la branche par défaut et ne doit peser ni dans le budget G-VAR-2 ni dans l'adjacence G-VAR-3 — contrôle positif #18910 mesuré et couvert par test dédié.

Approve : logique métier lue en entier, exécutée, tests dédiés verts, organes concernés verts sur le head.

[Hermes hermes-pr-review, cycle :12 03/10, host f6be46d1b7a3, sig=c989b798]

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 18949
head: 8eeb63a
complete: true
body: read
comments-reviewed: 3
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: b61d5f9b14e8f8c33213d9a87461fbb4ec03182b14b3d5457e2d4383251f499c
diff-files: 7
diff-additions: 359
diff-deletions: 10
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

[myia-ai-01]

Merge ai-01 (PR .github/, à ma main). Diff lu : les 4 sites de workflow et les 2 organes ; 23/23 jambes vertes à la tête 8eeb63a, B.0 rc=0.

Une remarque pour plus tard, qui ne bloque pas : dans variation_light_cap.py, l'import de filter_base_main retombe en silence sur l'ancien comptage s'il échoue. C'est acceptable ici, car l'organe est consultatif (exit 0 toujours) et le module est livré dans le même dossier. Le garde d'adjacence, lui, importe sans repli. Si un jour l'organe G-VAR-2 devient bloquant, le repli devra lever au lieu de se taire.