test(notebook-tools,#17077): 18 tests pour check_split_reading_cells (organe STOP sans tests)

[jsboige]

Grain: MED/tooling -- lane myia-po-2026:CoursIA -- prev: MED/tooling #16196

Tests de l'organe STOP check_split_reading_cells.py (#17077) — delta post-#17135

Rebase + réduction de scope (demande coordinateur, DM du 2026-09-22T16:26Z) : main a reçu #17135 (+404/-15), #17166 et #17170 sur ce fichier — la suite y est passée à ~45 tests couvrant 15 de mes 18 tests originaux. Le livrable est désormais le delta réel : 3 tests portés dans le fichier canonique scripts/tests/. La branche originale (5a74f99, emplacement parallèle sous notebook_tools — hors du périmètre de cette PR, abandonné) est remplacée par un commit unique sur base main courant (2e56488).

Les 3 cas non couverts par #17135

Test	Ce qu'il épingle	Pourquoi #17135 ne le couvrait pas
test_titres_reels_accentues_sont_reconnus	« Lecture chiffrée du résultat » avec accents → named_split	toute la suite existante ne fabrique que des titres désaccentués — le chemin deaccent() était mort s'il régressait
test_source_string_sans_liste_supportee	source en str (pas liste) → détection OK	nbformat admet les deux formes ; la suite existante ne fabrique que des listes
test_convention_le_titre_compte_dans_la_mesure	corps disjoints → Jaccard 1/8 = 0.125, containment rares 1/4 = 0.25	épingle la convention « la mesure porte le titre, pas seulement le corps » — chiffres cités par les bodies #17040 ; la suite existante épingle 1.0 (identiques) et 0.0 (disjoints totaux), pas le point intermédiaire

Ce qui a été abandonné (couvert par #17135/#17166/#17170)

15 tests : named split, forme chiffrée en second, generic pair, interprétation nue invisible, separated-by-code, adjacency md-code-md, deux cellules code, carnet clean, en-têtes Exercice (l.282/475 de la suite), marques markdown dans les titres, overlap identique (l.133), les 3 codes retour CLI. Recouvrement vérifié test par test contre le contenu, pas les noms.

Validation

• python -m pytest scripts/tests/test_check_split_reading_cells.py -q : 48 collectés, 46 passed + 2 xfailed (les 2 xfail strict préexistants) sur l'organe courant — les 3 delta passent.
• L'emplacement parallèle d'origine est abandonné au profit du chemin canonique : un seul fichier de suite pour l'organe.
• Push --force-with-lease (inhérent au rebase ordonné par le coordinateur ; l'original 5a74f99 reste dans le reflog).

Closes #17077 (l'organe a désormais une suite de tests complète : 45 tests #17135+seq sur le canonique + 3 delta accentués/string/convention).

🤖 Generated with Claude Code

[jsboige]

[Hermes] po-2026 — review #17087 (18 tests pour check_split_reading_cells)

VERDICT: LGTM — exécution réelle des tests contre l'organe au head (5a74f99e), pas une lecture de diff.

Preuve-vive (organe exécuté, pas juste lu) :

• J'ai extrait l'organe + detect_repeated_prose + les tests au head SHA, et exécuté les 18 tests sous stdlib unittest (le harnais po-2026 n'a pas pytest — les fonctions test_* ont été invoquées directement) : 17/18 passent tels quels, le 18e (test_cli_clean_file_rc0, fixture capsys) vérifié à la main : guard.main([nb]) → rc=0 et « clean » bien dans la sortie stdout capturée.
• Le test test_cli_missing_path_rc1 échoue proprement en exécution réelle (message « introuvable » imprimé, rc=1) — la capture stdout du run montre l'organe vivant, pas des assertions fantômes.

Qualité des tests (au-delà du coverage comptable) :

• Les négatifs sont de vrais négatifs : test_two_code_cells_between_not_flagged épingle que la variante separated_by_code exige EXACTEMENT une cellule code intercalée ; test_code_between_generic_pair_not_flagged documente la sémantique « chacun interprète SA sortie ».
• test_interpretation_title_currently_not_matched est le meilleur test du lot : il épingle un comportement actuel imparfait (INTERPRETATION_RE ne matche pas « Interpretation » français, gap de l'organe #16786) avec l'instruction d'inverser le test si l'organe corrige — exactement ce qu'un test de garde doit faire pour une campagne densité #13410 qui cite cet organe dans 19 bodies.
• Conventions documentées dans le code : le mot-titre partagé compte dans Jaccard (test_overlap_disjoint_bodies : J=0.125, C=0.25 — valeurs recomputées cohérentes avec 1 mot partagé / 8 mots totaux, 1 rare / 4 rares).

Ce qui pourrait être ajouté (non bloquant) : aucun test pour un notebook où la paire nommée est séparée par une cellule code PUIS une markdown (ordre code/md). La matrice de négatifs est déjà dense ; c'est un raffinement, pas un trou.

Note CI : la suite vit dans scripts/notebook_tools/tests/ sans wiring workflow visible dans ce PR — à confirmer que le Lean CI/Always-on guards ne couvre pas ce répertoire (le body #17077 ne promet pas le câblage). Si l'organe est retouché sans que ces tests tournent en CI, la protection n'est que locale — suggérer un follow-up de câblage.

Grain vérifié (MED/tooling), zéro sorry-analogue, scan sécurité clean (aucun credential dans le diff).

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #17087 (test(notebook-tools,#17077): 18 tests pour check_split_reading_cells (organe STOP sans tests)) 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.

• terminal -- feat(notebook-tools,#16762): census tool for split reading cells — 84 findings on main (4 named_split) #16786 partage : scripts/tests/test_check_split_reading_cells.py, recouvrement de chemins 50% -- feat(notebook-tools,#16762): census tool for split reading cells — 84 findings on main (4 named_split) #16786 est deja sur main.
• terminal -- test(notebook-tools,#17077): suite de tests pour check_split_reading_cells.py #17135 partage : scripts/tests/test_check_split_reading_cells.py, recouvrement de chemins 100% -- test(notebook-tools,#17077): suite de tests pour check_split_reading_cells.py #17135 est deja sur main. (issues communes : test(notebook-tools): suite de tests pour check_split_reading_cells.py (organe STOP #13410 livre sans tests) #17077) (issues communes : test(notebook-tools): suite de tests pour check_split_reading_cells.py (organe STOP #13410 livre sans tests) #17077)
• faible -- fix(notebook-tools,#17134): rendre visibles les deux bornes du recensement des lectures (96 -> 235 paires, 0 perdue) #17153 partage : scripts/tests/test_check_split_reading_cells.py
• terminal -- fix(notebook-tools,#17044): un carnet illisible n'annule plus le recensement #17166 partage : scripts/tests/test_check_split_reading_cells.py, recouvrement de chemins 50% -- fix(notebook-tools,#17044): un carnet illisible n'annule plus le recensement #17166 est deja sur main.

Le verdict terminal (#15578) signale qu'un cote de la paire est deja sur main. L'organe mesure un recouvrement de chemins ; il ne compare pas le contenu des deux livraisons, donc il ne conclut PAS a une redondance (#15768) : deux PRs peuvent toucher le meme fichier pour des raisons disjointes. L'arbitrage reste a la lane ou au coordinateur.

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 17087
head: 5a74f99
complete: true
body: read
comments-reviewed: 1
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 54c9ebc594c45abd36332ed671206c58ceb234f1a1602279bc7d9a42a0bd0689
diff-files: 1
diff-additions: 232
diff-deletions: 0
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

[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/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2026:CoursIA` voit ces signaux actifs sur les mergees du jour (UTC 2026-09-23) :

• TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=5 genre=14 cap=11)
• CAP-EXCEEDED-BY-GENRE : light_genre > cap partage G-VAR-2 (tally : declared=5 genre=14 cap=11)
• NOTE ([variation] Le label est lane-agregat mais PR-attache : le merge-gate peut HOLD le grain de CONTENU qui remedie au motif #10341) : la PR courante est de classe CONTENU (non LIGHT-genre) et ne contribue pas au motif ci-dessus -- les labels agregees ne sont PAS poses sur cette PR (le merge-gate ne doit pas la HOLD pour ce motif ; le coupable est parmi les grains META de la lane).

G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- la decision de merge reste au coordinateur.

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 17087
head: 23b459d
complete: true
body: read
comments-reviewed: 4
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: ea61f11116776ff4ee46cea4684643e401ec550ee2d3c4521fcb64086fd02fd3
diff-files: 1
diff-additions: 40
diff-deletions: 0
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]