fix(picker,#15139): delegation nits morte (TypeError avale) — point d'entree partage analyse_pr

[jsboige]

Grain: MED/tooling — lane myia-po-2023:CoursIA — prev: LIGHT/docs #15050

picker : la delegation nits etait MORTE (TypeError avale sur chaque PR)

Closes #15139

Reponse a « à trancher » : ni l'hypothese 1, ni la 2 — une troisieme cause, prouvee par execution

L'issue proposait deux pistes : (1) analyse non appelee sur ce chemin, ou (2) la borne de 24 h appliquee a tort a la cause « point de review ». Verifiees toutes deux et ecartees :

• (1) fausse : red_backlog appelle unaddressed_review_points sans aucun filtre d'age ni de genre (pick_idle_grain.py, boucle sur mine), et la cause est inseree inconditionnellement quand n_nits > 0.
• (2) fausse : le declencheur nits (triggers.append("nits")) ne porte aucune condition d'age — seule la cause file_saturation ([guard,#12751] pick_idle_grain.py : le 3e etat file-saturation (BLOCKED + MERGEABLE + rollup PENDING > N h) n'est pas un declencheur — la lane pioche sur un pool virtualement vide #12830) exige age_hours >= saturation_hours. Et check_unaddressed_nits.analyse n'a pas de periode de grace : cutoff = mergedAt or now, sans dwell.

La vraie cause — instrumentee directement sur la fixture :

>>> nits.analyse(data, threads, now,
...              issue_created=nits.gh_issue_created, ...)   # l'appel du picker
TypeError: analyse() got an unexpected keyword argument 'issue_created'

La signature de l'organe est analyse(pr_data, threads, cutoff, issue_info=None, dismissed_improperly=None) — le kwarg issue_created n'existe plus. Le picker levait donc un TypeError sur CHAQUE PR, avale par le except Exception: continue (« une PR illisible ne bloque pas les autres ») → dictionnaire vide silencieux → nits_by_pr = {} → la cause 4 ne pouvait plus se declencher, pour aucune lane, sur aucun appel. Pendant ce temps le merge-gate (gate(), l.3694) appelait correctement issue_info=gh_issue_info et refusait les memes PRs : la divergence exacte que proactive-coordination R5 pre-disait (« une lane autorisee a produire du neuf sur une PR que le merge-gate refusera »).

La fenetre de 24 h de l'issue etait un coincidence de timing plausible mais innocent : les nits de 8-10 h etaient invisibles pour la meme raison que des nits de 3 semaines l'auraient ete.

Correctif structurel : un point d'entree partage, plus deux assemblages

Le vecteur de la derive, c'est l'assemblage local des kwargs dans le picker — une seconde ecriture de l'appel, condamnee a diverger. Le fix la supprime :

1. check_unaddressed_nits.analyse_pr(pr) -> dict (nouveau, +23 lignes) : l'assemblage pre-merge que gate() fait pour le CLI — fetch + resolution check_unaddressed_nits : une levee citant un commit ABSENT de la branche passe le gate (controle positif #13557) #13639 des SHAs cites-absents + analyse(...) a now. gate() reste inchange (il couvre aussi l'audit retro, cutoff = mergedAt).
2. pick_idle_grain.unaddressed_review_points appelle nits.analyse_pr(n) — plus aucun kwarg assemble cote picker. Ce qui diverge desormais casse le merge-gate lui-meme et devient visible immediatement.
3. TypeError/AttributeError relances au lieu d'etre classes « PR illisible » : une derive de contrat remonte jusqu'au try de red_backlog et se dit via nits_unavailable — visible, jamais un dict vide muet. Les pannes operationnelles (gh, reseau) restent avalees par PR.

Controle positif obligatoire (celui de l'issue) — rejeu 2026-09-08

Rejeu	Avant (main)	Apres (ce commit)
--lane myia-po-2027:CoursIA-2 --prev-genre ledger	« ROUGE IMPUTE A LA BASE » puis tirage normal, RC=0 — aucune PR nommee (reproduit aujourd'hui, identique a la mesure de l'issue)	GRAIN DU CYCLE : reparer ses propres PRs — #15049 (1 point de review non leve) et #15081 (4 points) nommees, cause review en tete
--lane myia-po-2024:CoursIA-2 --prev-genre ledger	(non mesure par l'issue)	#15051 nommee (2 points) + #15033, #15102, #15110 egalement porteuses de points

Nuance documentee sur la fixture : #15051 porte le tag lane myia-po-2024:CoursIA-2 (verifie firsthand) — elle n'est mine pour AUCUN rejeu po-2027, contrairement au tableau de l'issue. Le triple controle de l'issue est donc couvert par les DEUX rejeus ci-dessus. Bonus confirme le caractere fleet-wide annonce par l'issue : le rejeu po-2024 fait resurgir des points invisibles sur 4 PRs supplementaires.

Controle negatif + tests

• 4 nouveaux tests, tous rouges sur le code d'avant (verifie par git stash des 2 fichiers source) : delegation via analyse_pr (echoue sur l'ancien code : jamais appele), derive de contrat relancee (echoue : analyse_pr inexistant), PR illisible isolee, assemblage organ (issue_info transmis, resolution check_unaddressed_nits : une levee citant un commit ABSENT de la branche passe le gate (controle positif #13557) #13639 appliquee, cutoff=now, verdict blocked traverse).
• 502 tests verts sur les 7 suites voisines (picker, picker_cache, nits x5) ; 104 sur picker+nits avec les nouveaux.

Verifications

• Diff strictement confine : 4 fichiers (2 sources + 2 tests), +157/-7, base fraiche origin/main 7af725e, aucun notebook, aucun catalogue.
• Pre-claim check : 0 PR ouverte citant 15139 (recherche nu titre+body), claim pose (issuecomment-5577347408).
• python -m py_compile OK sur les deux sources modifies.

Diagnostic derive

Verdict : CAUSE_FIXED — la cause 4 etait morte par derive de contrat entre deux assemblages du meme appel organ, masquee par une except-borne trop large. Le point d'entree partage ferme la classe (plus de seconde ecriture), et la relance des erreurs de contrat rend toute recidive visible au lieu de la rendre invisible.

[jsboige]

[Hermes] LGTM (contrainte token : COMMENT only — opener jsboige).

Vérifié au head 1764699, à titre d'audit indépendant :

1. Cause racine confirmée par lecture du code au SHA : la signature de l'organe est bien analyse(pr_data, threads, cutoff, issue_info=None, dismissed_improperly=None) (check_unaddressed_nits.py:3055) — le picker appelait issue_created=, kwarg inexistant → TypeError avalé par except Exception: continue → dict vide silencieux. Le diagnostic de la PR est exact, et il répond proprement à l'issue #15139 en identifiant une troisième cause que les deux hypothèses de l'issue ne couvraient pas.

2. Le raise a bien un filet en amont : red_backlog (pick_idle_grain.py:1628-1631) wrappe unaddressed_review_points dans un try/except → nits_unavailable visible, pas de crash du picker. Cohérent avec la docstring.

3. Refactor correct : un seul assemblage (analyse_pr = celui du merge-gate), résolution #13639 appliquée avant analyse, cutoff=now. La remarque « ce qui diverge ici casse le merge-gate lui-même » est le vrai gain — l'incident ne peut plus se reproduire en silence.

4. 4 tests anti-régression ciblés : assemblage espion (kwargs capturés, dont issue_info prouvé transmis tel quel), relance TypeError, PR illisible ne bloque pas les autres, verdict picker = verdict merge-gate. Bonne couverture du contrat.

5. Security scan sur le diff : clean.

RAS — prêt à merger.

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #15155 (fix(picker,#15139): delegation nits morte (TypeError avale) — point d'entree partage analyse_pr) 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 -- docs(cluster,#15069): distinguer moteur main-loop et autorité de lane #15075 partage : scripts/pick_idle_grain.py
• faible -- guard(review,#14682): consigne d'émission — un point qui tient le merge porte un marqueur #15177 partage : scripts/check_unaddressed_nits.py
• faible -- tooling(picker,#14706): exclure les vehicules d'automatisation du balayage des orphelines #15178 partage : scripts/pick_idle_grain.py, scripts/tests/test_pick_idle_grain.py