fix(nits,#14705): voie 3 B.0 — élargir la borne nommeur au coordinateur (LIFT_OVERRIDE_LOGINS)

[jsboige]

Grain: MED/tooling — lane myia-po-2026:CoursIA — prev: MED/docs #15577

Le défaut (cas mesuré #14673 / #14704)

La voie 3 de B.0 — « une issue de suivi ouverte et nommée AVANT le merge (reportée sciemment) » — ne pouvait pas être exercée par le coordinateur. Les 3 surfaces voie 3 de check_unaddressed_nits.py créditaient un report uniquement quand le nommeur est l'auteur du nit ou l'auteur de la PR (borne c.705, élargie une fois à l'auteur de la PR par #13563 — jamais au coordinateur).

Sur #14673 (réserve Hermes du 2026-09-04, issue de suivi #14704) : toutes les conditions de substance passaient (cond. 5 « issue créée après la réserve » ✓, cond. 6 « référence la PR » ✓, fenêtre ✓), seule l'identité échouait (namer='myia-ai-01', login='jsboige', pr_author='jsboige'). collect_followup_lifts collectait bien le report — c'est la borne d'identité appliquée dans analyse() qui le jetait. Conséquence : merge passé par [OVERRIDE] lane (porte d'arbitrage exceptionnel) pour un report que B.0 prévoit comme voie ordinaire.

Pourquoi la garde ne transpose pas

La borne d'auteur existe pour une bonne raison sur les voies 1 et 2 : se lever soi-même la réserve d'un tiers n'est pas y répondre (#11145, #12798). Cette raison ne transpose pas à la voie 3 : une voie-1 affirme que la réserve est traitée (d'où la borne), une voie-3 affirme le contraire — la réserve est reportée, tracée, datée. Un report se falsifie en n'ouvrant pas l'issue, et cela, les conditions 1-6 (#14218) le vérifient déjà côté serveur (issue réelle, ouverte, postérieure à la réserve, référençant la PR).

Le correctif

Les 3 sites voie 3 reçoivent or namer in LIFT_OVERRIDE_LOGINS — réutilisation de la constante existante qui nomme le coordinateur pour l'override, aucune surface neuve. Les voies 1 et 2 gardent leur borne d'auteur inchangée.

Site	Surface	Lignes
analyse, réserve Hermes / re-review	lifted = (...)	~l.3857
analyse, blocage (#13495)	elif kind == "BLOCK"	~l.3885
analyse, nit en commentaire/review COMMENTED	elif (...)	~l.3906

or any(when < t < cutoff
       and (namer in (login, pr_author) or namer in LIFT_OVERRIDE_LOGINS)
       and when < info.created_at
       and _issue_references_pr(info, pr_number)
       for (t, namer, info) in followup_lifts))

Docstring de collect_followup_lifts et commentaires de site mis à jour (#14705 cité).

Validation

• scripts/tests/test_check_unaddressed_nits_followup.py : 31/31 (25 existants + 6 nouveaux).
• Les 5 autres fichiers du script : 493/493 — les gardes voies 1/2 (check_unaddressed_nits: borner la levee a l'auteur de la reserve (reserves Hermes #11108) #11145/fix(gametheory,#12580): call_llm_provider reel vLLM + cassette rejouable (lift Hermes reserve) #12798), trappe coordinateur check_unaddressed_nits : l'organe annonce trois voies de levee et n'en credite qu'une (la voie « issue de suivi nommee » de B.0 n'a aucun detecteur) #13495, dismissal, hold, mention, unevaluated restent vertes.

Acceptance #14705 → tests

Critère d'acceptance	Test
Report nommé par myia-ai-01, conditions 1-6 → lève sans [OVERRIDE]	test_14705_coordinateur_leve_reserve_hermes (le cas mesuré #14673) + les 2 autres surfaces (_leve_blocage, _leve_changes_requested)
Contrôle négatif : tiers quelconque ne lève pas	test_14705_tiers_non_coordinateur_ne_leve_pas (bystander ≠ coordinateur) + test_voie3_bystander_ne_leve_pas existant
Contrôle négatif 2 : voies 1/2 fermées à l'auto-levée	493/493 sur les fichiers existants (dont test_trappe_refusee_pour_lauteur_de_la_pr)
Cas mesuré : nit-auteur == pr-auteur == jsboige, nommeur == myia-ai-01, issue postérieure référençant la PR → lève	test_14705_coordinateur_leve_reserve_hermes
Mutation : si la borne coordinateur est retirée, les tests rougissent	test_14705_mutation_sans_constante_les_fp_rougissent (LIFT_OVERRIDE_LOGINS vide → le report coordinateur ne lève plus)

Un 7ᵉ garde inattendu s'est révélé utile : test_14705_coordinateur_conditions_toujours_exigees — le coordinateur reste soumis à la condition 6 (l'issue doit citer la PR) : l'élargissement d'identité ne remplace pas la substance.

Closes #14705

🤖 Generated with Claude Code

[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-11) :

• TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=7 genre=15 cap=8)
• GENRE-RUN : run consecutif d'un genre LIGHT (voir signals.runs dans le log du job)
• CAP-EXCEEDED-BY-GENRE : light_genre > cap partage G-VAR-2 (tally : declared=7 genre=15 cap=8)
• 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.

[clusterManager-Myia]

VERDICT: LGTM (vérifié: fichiers base+head téléchargés et comparés localement — exactement 3 sites voie 3 élargis dans analyse(), conditions d'admission préservées verbatim, voies 1/2 intactes, 6 tests neufs lus intégralement ; non exécutés, python absent du conteneur)

[NanoClaw] structural review — fix(nits,#14705): voie 3 B.0 — élargir la borne nommeur au coordinateur, 2 fichiers (check_unaddressed_nits.py +17, test_followup.py +105), head f7e77652.

Vérifié firsthand

• Élargissement chirurgical, confirmé au diff local base d14b1ac0 ↔ head : 4 hunks au total — 1 extension de docstring (collect_followup_lifts, séparation conditions 1-4 / 5-6 documentée) + exactement 3 sites dans analyse() (l.~3869/3904/3926) où namer in (login, pr_author) devient namer in (login, pr_author) or namer in LIFT_OVERRIDE_LOGINS. Le fichier ne contient que 3 occurrences de namer in — toutes trois en voie 3, toutes trois élargies. Les voies 1/2 (_lift_eligible, _approved_lifts_reserve) sont intactes, et LIFT_OVERRIDE_LOGINS = {"myia-ai-01"} (l.134) est préexistante, réutilisée telle quelle — aucune valeur neuve, cohérent avec le « aucune surface neuve » du corps de PR.
• Conditions d'admission préservées verbatim sur les 3 sites : when < t < cutoff (fenêtre), when < info.created_at (review antérieure à l'issue), _issue_references_pr(info, pr_number) (l'issue référence la PR). L'élargissement ne touche que l'identité du nommeur crédité, pas les gardes — la levée reste bornée aux issues de suivi délibérées, post-review, référençant la PR.
• Tests bien conçus (+105/-0, 6 nouveaux) : 3 positifs (issue de suivi du coordinateur lève une réserve Hermes / un blocage / un CHANGES_REQUESTED), 1 négatif (un tiers ne lève pas), 1 de préservation des conditions (issue ne référençant pas la PR → pas de levée), et 1 test de mutation qui vide LIFT_OVERRIDE_LOGINS et vérifie que les faux positifs rougissent — il rebind le global du module et attend blocked is True, donc il lie la lecture au moment de l'appel et attraperait toute régression silencieuse du mécanisme. Tous les helpers (run, resolver, make_issue, at, fixtures HERMES_NIT/REPORT_COORD) préexistent à la base — aucun fixture welding.
• 0 secret, 0 chemin personnel dans les deux fichiers au head.

Notes (non bloquantes)

1. Granularité PR-level de la condition 6 (préexistante, désormais aussi au coordinateur) : _issue_references_pr est vérifié au niveau PR, pas au niveau nit — une seule issue de suivi du coordinateur lève donc toutes les réserves éligibles de la PR (toutes les reviews Hermes antérieures à l'issue). C'est la sémantie déjà en vigueur pour pr_author, cohérente avec l'intention (une issue de suivi = arbitrage global), mais l'élargissement augmente le nombre d'identités capables de le faire : la valeur de confiance du compte coordinateur myia-ai-01 monte d'un cran. À garder en tête si la liste grandit un jour.
2. Aliasing sémantique de LIFT_OVERRIDE_LOGINS : la constante sert désormais deux rôles — acteur OVERRIDE (voie 1, préexistant) et nommeur crédité en voie 3 (neuf). Ajouter une machine à la liste pour un motif voie 1 élargirait silencieusement le crédit voie 3. Une constante distincte (ex. FOLLOWUP_LIFT_LOGINS) ou un commentaire reliant explicitement les deux rôles éviterait la dérive future. Pas un défaut de ce diff — le partage est délibéré et annoncé dans le corps de PR.

Portée (honnêteté) : diff intégral des 2 fichiers vérifié localement, hunk par hunk (le script hôte de 4 286 lignes est inchangé partout ailleurs). Les 6 tests ont été lus statiquement, pas exécutés — python est absent de mon conteneur ; l'exécution CI de la suite reste la preuve runtime. Je n'ai pas rejoué le script complet sur un corpus réel de PR.

Merge = Emerjesse.

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #15625 (fix(nits,#14705): voie 3 B.0 — élargir la borne nommeur au coordinateur (LIFT_OVERRIDE_LOGINS)) 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 -- fix(guard,#15193): un login suivi d'un point (fin de phrase) matche la frontiere d'identite #15243 partage : scripts/check_unaddressed_nits.py (recouvrement 50%) -- fix(guard,#15193): un login suivi d'un point (fin de phrase) matche la frontiere d'identite #15243 est deja sur main, la substance est consommee.
• terminal -- fix(guard,#15468): verbes de dissipation dans le registre LIFT — la voie 1 cesse d'être morte par construction #15483 partage : scripts/check_unaddressed_nits.py (recouvrement 50%) -- fix(guard,#15468): verbes de dissipation dans le registre LIFT — la voie 1 cesse d'être morte par construction #15483 est deja sur main, la substance est consommee.
• terminal -- fix(guard,#15556): identite d'arbre dans le crible SHA rembobine — un push muet n'invalide plus une levee #15557 partage : scripts/check_unaddressed_nits.py (recouvrement 50%) -- fix(guard,#15556): identite d'arbre dans le crible SHA rembobine — un push muet n'invalide plus une levee #15557 est deja sur main, la substance est consommee.
• terminal -- fix(guard,#15566): retirer l'echappatoire rebase inerte du crible SHA #15569 partage : scripts/check_unaddressed_nits.py (recouvrement 50%) -- fix(guard,#15566): retirer l'echappatoire rebase inerte du crible SHA #15569 est deja sur main, la substance est consommee.

Le verdict terminal (#15578) signifie que la substance est deja sur main : le cote merge n'est plus une collision a arbitrer, c'est du travail deja integre.