fix(notebook-tools,#17044): un carnet illisible n'annule plus le recensement

[jsboige]

Grain: MED/tooling -- lane myia-po-2023:CoursIA -- prev: MED/tooling #17135

See #17044

Le defaut, et ce qu'il coutait

scan_root faisait return 1 dans son except (OSError, json.JSONDecodeError) (l.169-171 sur main) :

        except (OSError, json.JSONDecodeError) as e:
            print(f"ERREUR lecture {path}: {e}", file=sys.stderr)
            return 1                      # <- annule TOUT le recensement

Un seul carnet corrompu n'abime donc pas le recensement : il l'annule. Le scan perd tous ses findings (pas seulement celui du carnet fautif) et rend rc=1. Consequence : l'organe est incablable en cliquet — n'importe quel carnet corrompu du depot rougit toute PR, pour une raison sans rapport avec elle. C'est le defaut que #17044 nomme « bloquant a corriger d'abord ».

Note de mesure (G.1). Sur un worktree origin/main frais, le carnet cite par #17044 (QuantConnect/ESGF-2026/lean-workspace/BTC-ML-Researcher/research.ipynb) est absent (non suivi) : le scan y rend rc=0, donc le defaut n'est pas observable par cette instance. Il est latent sur main et reel — il se declenche sur tout carnet illisible, et ce fichier existe dans les arbres de travail reels. Je corrige donc la cause, pas l'instance.

Le correctif

Fichier	Δ	Role
scripts/notebook_tools/check_split_reading_cells.py	+33 / -5	le carnet illisible est rapporte et saute ; le mode fichier rend son rc proprement
scripts/tests/test_check_split_reading_cells.py	+64	3 tests de regression, dans la suite canonique de l'organe

• Mode dossier : le carnet illisible est rapporte (nomme sur stderr) et saute ; le recensement continue, et se declare PARTIEL (liste + compteur en fin de stderr, suffixe sur la ligne Total). Les findings des carnets sains sont rendus comme avant.
• Mode fichier designe : le contrat documente (« 1 = fichier illisible ») etait rendu par une JSONDecodeError non rattrapee — traceback jusqu'a l'interpreteur. Meme code de retour, rendu proprement.

Deux decisions, assumees et ecrites

1. La forme du --json ne change pas. Le rapport d'illisibles passe par stderr, pas par stdout : le --json reste la liste de findings que consomment les bodies deja postes et le cliquet. Exposer le compteur en JSON aurait casse les consommateurs pour un gain de lisibilite.

2. Un carnet illisible n'est jamais ignore en silence. C'est le vrai risque de ce correctif : « ne plus avorter » peut degenerer en « avaler ». Un carnet saute se lirait comme un carnet propre — exactement ce que le cliquet doit empecher. D'ou le rapport en trois endroits (ligne par carnet, liste finale, suffixe sur Total) et la mention explicite que le total est partiel.

Ce que le correctif ne fait pas : il ne rend pas rc non-nul sur un carnet illisible. Reintroduire un rc=1 global recreerait le defaut d'origine sous une autre forme (rougir tout le depot pour un fichier sans rapport).

Preuves

• 12 passed sur l'organe corrige (python -m pytest scripts/tests/test_check_split_reading_cells.py).
• Controle positif — les 3 nouveaux tests ECHOUENT sur l'organe non corrige (origin/main), et passent sur le corrige : ce sont des tests de regression, pas des tautologies. Les 9 tests d'origine restent verts dans les deux etats.
• Le carnet corrompu est place entre deux carnets sains : la preuve que le scan continue est que celui qui le suit est quand meme recense (un test qui ne verifierait que « pas de plantage » passerait aussi sur un scan qui s'arrete tot).

Perimetre, et ordre de merge

Cette PR corrige l'organe. Le volet « cablage en cliquet bloquant » de #17044 est hors perimetre : il est deja en vol dans mon #17031 (garde TRANCHE13 du registre fast_lane_registry.py). D'ou un See #17044, pas un Closes — l'issue demande les deux, cette PR n'en livre qu'un.

Ordre de merge a declarer : cette PR et #17135 touchent la meme suite de tests (scripts/tests/test_check_split_reading_cells.py). La seconde rebasera sur la premiere ; aucune des deux ne doit etre mergée en supposant l'autre acquise.

Verifie : check_unique_check_run_names.py --check OK.

🤖 Generated with Claude Code

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 17166
head: f2ff155
complete: true
body: read
comments-reviewed: 0
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 5c02697e98e3b4b1f34def18194fd261d20594a878b7f6af83ea75f58d1410c6
diff-files: 2
diff-additions: 97
diff-deletions: 5
checks: BLOCKED
b0: clear
scope: pass
domain: pass
verdict: BLOCKED
[/ADJOINT PREFLIGHT]

[github-actions]

Path-collision (organ #13359/#13615)

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

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: 17166
head: f2ff155
complete: true
body: read
comments-reviewed: 2
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: dfece489f3ad3ce1c2ee2bfee455256e4f42f1d465e1aa8ab0aa85505766e88d
diff-files: 2
diff-additions: 97
diff-deletions: 5
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]