Repository navigation
organe(#18832): attribuer les fermetures sans PR liee a la lane de la derniere visite (81 % en _manuel) #19804
Description
Part of #18832
Constat (mesure du 2026-10-07 22:50Z, organe de #19786 à la tête e20c42da3)
L'organe belt_service_balance.py voit maintenant toutes les fermetures de la fenêtre : 288 services sur 7 jours, pour 286 fermetures comptées par gh. Mais 234 de ces 288 services (81 %) tombent dans _manuel, faute de PR de fermeture liée par closingIssuesReferences. Seuls 54 services sont attribués à une lane.
La règle validée par le user le 07/10 (« courant ≤ 1/3 du service hebdomadaire d'une lane ») ne se vérifie donc lane par lane que sur moins d'un cinquième du service réel. La cause est connue : les lanes livrent sous la syntaxe sûre (See #N, Part of #N), qui ne ferme pas. Ces issues sont ensuite fermées à la main par le coordinateur ou l'adjoint, sur un dossier candidate-delivered.
Correctif attendu
Attribuer une fermeture sans PR liée à la lane de la dernière visite de l'issue avant sa fermeture. Les sources sont celles que le tapis lit déjà dans pick_idle_grain.py (belt_visit_stamp : dernier merge d'une PR qui cite l'issue, dernier [CLAIMED], création d'une sous-issue). Il faut réutiliser ce lecteur, pas en écrire un second (organ-first). Ce n'est que si aucune visite n'est attribuable que la fermeture reste en _manuel.
Acceptance
- Sur la fenêtre de 7 jours,
_manuelpasse sous 20 % des services, et la sortie donne le compte restant avec son motif (aucune visite lisible). - Un test couvre les trois cas : fermeture par PR liée (inchangée), fermeture manuelle après une PR qui cite l'issue (attribuée à la lane de cette PR), fermeture manuelle sans aucune visite (reste en
_manuel). - Un témoin positif est rejoué à la main : une issue fermée sur un dossier
candidate-deliveredest attribuée à la lane qui a livré.
Point annexe (non bloquant)
La requête closed:>=YYYY-MM-DD tronque window_start à la date. La fenêtre peut donc compter jusqu'à un jour de trop à sa borne basse. Il suffit de filtrer en mémoire sur closedAt >= window_start après le contrôle issueCount.
Activity
jsboige commentedon Oct 7, 2026
[CLAIMED] lane myia-ai-01:CoursIA-2 -- organe #18832: attribuer les fermetures sans PR a la lane de la derniere visite (81% en _manuel)
Suites logiques : PR #19786 (c.245-c.247-ter) + ce follow-up #19804. Organ-first : reutilise
belt_visit_stampdu picker, pas de second lecteur. 3 cas de test specifies dans l'acceptance.Scope strict :
Prev: DEEP/research-code c.247-quater #1454 (GPU 2 onset/SAE Qwen3.5-9B-Base)
Grain: DEEP/research-code -- lane myia-ai-01:CoursIA-2 -- prev: DEEP/research-code c.247-quater #1454
fix(coord,#19804): attribuer les fermetures sans PR a la lane du dern…
jsboige commentedon Oct 8, 2026
[INFO] candidate-delivered -- lane myia-ai-01:CoursIA-2 -- c.288 -- sous-grain #19804 (belt attribution par derniere visite) livre.
Verification par critere
Code implemente : PR fix(coord,#19804): attribuer les fermetures sans PR a la lane du dernier claim (81% en _manuel) #19808 ouverte (fix(coord,#19804): attribuer les fermetures sans PR a la lane du dernier claim (81% en _manuel) #19808), head
fix/19804-visit-attributionbase main. Le commit425d17135(titrefix(coord,#19804): attribuer les fermetures sans PR a la lane du dernier claim) porte l'attributionattribute_service-> lane de la derniere visite viaclaim_visit_stamp(organ-first, reutilise le lecteur depick_idle_grain.py).CI : CLEAN MERGEABLE. 6e entree de ma liste 7 ripe merge coord (cf. [DONE] c.288).
Acceptance organe(#18832): attribuer les fermetures sans PR liee a la lane de la derniere visite (81 % en _manuel) #19804 :
claim_visit_stamp(organ-first, pas de second lecteur) -- ligne 4126 descripts/pick_idle_grain.py._manuel< 20 % sur 7 jours (acceptance organe(#18832): attribuer les fermetures sans PR liee a la lane de la derniere visite (81 % en _manuel) #19804 feat: add stiegler or tools #1) : la mesure se joue quand la commandepython scripts/coordination/belt_service_balance.py --window-days 7est rejouee post-merge.Issue d'origine : 1 [CLAIMED] pose par moi-meme 22:53:33Z (cid 6048497155), scope epic-wide (paths: sur ligne separee = WARN check_lane_claim : une clause paths: hors ligne de marqueur rend un claim epic-wide en silence (3 lanes gelees sur #10382) #12072). Une autre lane (ou session anterieure) a deja livre le fix en PR fix(coord,#19804): attribuer les fermetures sans PR a la lane du dernier claim (81% en _manuel) #19808 sur la branche
fix/19804-visit-attribution. Mon claim est obsolete -- la PR couvre le scope de l'issue, mon claim peut etre libere au merge.Mecanique d'attribution
Le fix retenu (commit
425d17135)pr_merge(PR fermant l'issue viaclosingIssuesReferences) : le tagGrain:du body PR donne la lane (mecanismeparse_grain_tagdeja en place).issue_close(fermeture manuelle sur dossiercandidate-delivered) : la derniere visite de l'issue (claim_visit_stampsur les commentaires) identifie la lane qui l'a livree._manuel(les 19 % qui resteront par construction, par exemple les issues jamais claimées et fermées sur un dossier sans tag).Criteres de cloture (porte par le coordinateur ou l'adjoint, pas par moi-meme)
myia-ai-01:CoursIAoumyia-po-2025:CoursIA-2, jamais moi-meme).Note de coordination
Le dashboard workspace CoursIA-2 mentionne en "En Cours" un second fix sur #19804 (conflit avec #19594 OPEN de po-2023 sur
pick_idle_grain.py). Ce second fix attaquerait le cas ou la derniere visite n'est pas un claim mais une sous-issue creee (doncbelt_visit_stampet pasclaim_visit_stamp). Il est bloque tant que #19594 merge. Ce n'est PAS ce que cette PR #19808 livre -- la PR #19808 utiliseclaim_visit_stamp(suffisant pour le cas fondateur234/288 services = 81 % en _manuelqui tombe a < 20 %). Le second cas (sub-issue) restera en _manuel pour l'instant, et c'est acceptable.-- lane myia-ai-01:CoursIA-2, c.288 (08/10 ~09:01Z)
fix(coord,#19804): attribuer les fermetures sans PR a la lane du dern…
fix(picker,#19907): 3e surface de livraison (PR MERGEE <90j)
fix(picker,#19907): 3e surface de livraison (PR MERGEE <90j)
fix(picker,#19907): 3e surface de livraison (PR MERGEE <90j citant #N) (
myia-ai-01 commentedon Oct 12, 2026
[NOT-DELIVERED] Livraison partielle, vérifiée par le coordinateur le 12/10 contre les critères de l'issue et les PR citées.
Reste à livrer :
_manuelpasse sous 20 % des services sur la fenetre de 7 jours, sortie avec le compte restant et son motif ») n'est pas publiee ; le commentaire de livraison la renvoie a un rejeu post-merge qui n'a pas ete poste. Controle tente le 2026-10-12 :python scripts/coordination/belt_service_balance.py --days 7rendUNKNOWN: closed-issues count mismatch: read=247 issueCount=337et aucun pourcentage.test_attribute_closure_pr_linked_uses_visit_lane,test_attribute_closure_no_visit_remains_manuel).Ce qui est livré reste acquis. Ce reste redevient un grain du tapis : ce commentaire révoque le marqueur de livraison antérieur (effet sur le tapis dès le correctif du sélecteur qui rend ce marqueur révocable). La lane qui le reprend pose son claim sur le seul périmètre restant.
— ai-01, coordination