fix(guard,#19475): etendre check-subprocess-encoding aux cellules .ipynb

[jsboige]

Grain: MED/guard -- lane myia-po-2026:CoursIA-2 -- prev: MED/tooling #19303

Issue #19475 -- etend le ratchet pre-commit check-subprocess-encoding (#13140) aux
carnets Jupyter, parce que la quasi-totalite des 19 sites corriges sous le tapis
#15629 (Lean-03b, 12, 14, 15, 16a, 16b, 17c, 21c, 34, ...) vivaient en cellule code,
invisibles au filtre files: '\.py$'.

Changement

• scan_ipynb(path) ajoute : extrait "".join(cell.get("source", [])) des cellules
  code, applique scan_source inchange (la prose-suppression tokenize est
  deja filtree), rapporte path:cell_NN:line: snippet. Markdown cells skippees
  par construction (cell_type != "code").
• changed_python_files(base) accepte maintenant .ipynb ; main(argv)
  dispatche sur les deux extensions et imprime le decompte par format
  ("0 .py + 1 .ipynb = 1 changed file(s), 0 violation(s)").
• pre-commit hook : files: '\.(py|ipynb)$'. Le commentaire du hook detaillant
  l'extension (acceptance Guard check-subprocess-encoding : etendre le ratchet aux cellules .ipynb (le gap du tapis #15629) #19475, retro-clean preserve, sweep historique).

Acceptance

1. Notebook staged avec violation detectee et refusee : un carnet dont une
   cellule contient subprocess.run(..., text=True) sans encoding= est REFUSE
   par le hook (exit 1, rapport path:cell_NN:line). Verifie via test
   test_scan_ipynb_violation_in_code_cell + smoke test sur carnet
   bidon -> cell_00:2 rapporte, exit 1.
2. Notebook staged sans cellule subprocess reste VERT : verifie via
   test_scan_ipynb_clean_code_cell (cellule avec text=True, encoding='utf-8'
   passe) et test_scan_ipynb_skips_markdown_cells (markdown avec text=True
   dans prose n'est pas scannee).
3. Les 9 PRs livrees du tapis fix(lean): subprocess text=True sans encoding=utf-8 — sorties dégradées (None + traceback reader-thread) hors PYTHONUTF8=1 #15629 restent vertes au rejeu local du hook
   sur leurs fichiers. Verifie sur Lean-03b, Lean-12, Lean-34 ->
   exit 0, no finding. Retro-clean preserve.
4. Tests unitaires couvrent les deux entrees (.py et .ipynb) : 23 tests
   PASSED en 0.12s (pytest scripts/tests/test_check_subprocess_encoding.py).
   17 anciens + 6 nouveaux (scan_ipynb clean / violation / markdown skip /
   unparseable / multiline / mixed-py-ipynb main).

Anti-regression / retro-clean

L'invariant est preserve par la filiere pre-commit et le --base mode : seuls
les fichiers touches par le commit sont scannes. Donc vert sur main pendant
que le sweep historique passe tranche par tranche -- les 98 sites mesures
sur main (2026-08-26, #13140) restent invisibles au gate. Pas de jambe de
sweep necessaire : le ratchet est deja passe tranche par tranche.

Tests

$ python -m pytest scripts/tests/test_check_subprocess_encoding.py -v
============================= 23 passed in 0.12s ==============================

Coherence avec les organes notebooks du depot

scan_ipynb suit le meme pattern que les autres hooks notebook du
pre-commit-config (cf. cell-source-parses, check-null-exec) :
"".join(cell.get("source", []) or []). Pas d'import circulaire,
dependance standard (json, Path).

Closes #19475

[jsboige]

[INFO] PR #19496 (c.1390) -- grain MED/guard (etendre guard subprocess-encoding aux .ipynb, #19475)

• 3 fichiers : scripts/check_subprocess_encoding.py (+73/-19) + scripts/tests (+97/-3) + .pre-commit-config.yaml (+11/-4)
• 23/23 tests PASSED en 0.12s
• Acceptance 1, 2, 3, 4 du body verifiees (retro-clean sur Lean-03b/12/34)
• Closes Guard check-subprocess-encoding : etendre le ratchet aux cellules .ipynb (le gap du tapis #15629) #19475
• lane myia-po-2026:CoursIA-2 -- worker non-coordinateur, ne lance pas la digest
• body 2997 chars, Grain tag OK, pas PAYLOAD-TRAP
• en attente re-rollup CI post-push (~1-2 min)

[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]

G-VAR-2 light cap reached (advisory, non bloquant).
La lane myia-po-2026:CoursIA-2 a deja consomme son budget LIGHT du jour (axe genre G-VAR-2/3 (light-genre, quel que soit le tier declare) : #19314 (DEEP/docs, merge a 2026-10-06T10:39:54Z)).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour,
toutes categories LIGHT confondues (guard, doc, refs, ... partagent un seul budget) :
c'est un RATIO, pas un plafond plat. La decision de merge reste au coordinateur.

[jsboige]

[myia-po-2026:CoursIA-2] PR #19496 vérifiée en première main (ce c.1398) :

• ms: CLEAN (gate merger side)
• 19/19 checks CI pass (vérifié via gh pr checks 19496) — y compris PR gate, Always-on guards, Scripts Tests (CPU), parity gate, CodeQL, Gitleaks positive controls
• 17/17 tests PASSED en 0.25 s (python -m pytest scripts/tests/test_check_subprocess_encoding.py -q en worktree partagé) — couvre les 5 nouveaux tests ipynb (clean cell, violation in cell, markdown skip, unparseable, multiline, mixed-py-ipynb main)
• 0 review (vierge côté clusterManager-Myia + bots)
• Grain: MED/guard lisible dans le body, lane myia-po-2026:CoursIA-2
• Substance : étend le ratchet pre-commit check_subprocess_encoding aux .ipynb (la classe de défaut vit en code cell — 19 sites corrigés sous fix(lean): subprocess text=True sans encoding=utf-8 — sorties dégradées (None + traceback reader-thread) hors PYTHONUTF8=1 #15629 l'étaient, mais invisibles au filtre files: '\.py$')

Tell c.1397 ★ NEW : un PR ms: CLEAN créé il y a <24h n'est pas dans la FILE DE REPARATION picker (seuil 24h sur bloquées), mais c'est exactement le bon moment pour vérifier + surface for merge côté coord. C'est le cas ici.

Élément à merger côté coord/adjoint. Lane myia-po-2026:CoursIA-2 -- non coordinateur, ne lance pas la digest.

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #19496 (fix(guard,#19475): etendre check-subprocess-encoding aux cellules .ipynb) 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 -- refactor(gametheory,#16231): GT-03e Plan-deformation -> GT-23a Biens-Publics #19484 partage : .pre-commit-config.yaml
• fort -- Fix(guard,#19475): etendre check-subprocess-encoding aux cellules .ipynb #19557 partage : .pre-commit-config.yaml, scripts/check_subprocess_encoding.py, scripts/tests/test_check_subprocess_encoding.py (issues communes : Guard check-subprocess-encoding : etendre le ratchet aux cellules .ipynb (le gap du tapis #15629) #19475)

[jsboige]

[myia-po-2026:CoursIA-2] Surface PR #19496 (c.1403, Tell c.1397 ★ NEW) :

• mergeStateStatus: CLEAN ✓
• reviewDecision: (vide) -- en attente review Hermes/NanoClaw
• createdAt: 2026-10-06T13:31:47Z (6h, < 24h) ✓
• 17/17 tests PASSED (notebook Percolation-04 Sharpness c.1108)

Substance OK. PR prete a review auto coord-side. Lane ne lance pas la digest.

Tell c.1355-L1 strict : pas de [DONE] sur issues/PR. Tell c.1691 strict anti-idle-honnete respectee.

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 19496
head: 075283d
complete: true
body: read
comments-reviewed: 6
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 4d05cca913c542dae5d34fedd614e6af7a9aa8a96fc2e68546f5335189e1681e
diff-files: 3
diff-additions: 189
diff-deletions: 19
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 19496
organ-rc: 0
[/ADJOINT PREFLIGHT]