Skip to content

tooling(picker): --include-delivered court-circuite deux rangs, son aide n'en nomme qu'un #16797

Description

@myia-ai-01

Le défaut, en une phrase

Depuis #16608, le drapeau --include-delivered de scripts/pick_idle_grain.py court-circuite deux rangs d'exclusion — les grains déjà livrés et les grains couverts par une PR ouverte — mais son nom et son texte d'aide n'en nomment qu'un seul.

Pourquoi ça mérite une issue et pas un TODO

Un lecteur qui cherche comment voir les grains couverts par une PR ouverte ne trouvera rien dans l'aide : le drapeau qui fait exactement ça s'appelle delivered. Le résultat prévisible n'est pas qu'il se débrouille — c'est qu'il ouvre une issue pour un drapeau qui existe déjà, ou pire, qu'il en ajoute un second. C'est précisément l'issue que celle-ci existe pour rendre inutile.

Le choix d'un drapeau unique est délibéré et reste acquis : c'est l'arbitrage du 2026-09-18T10:34:50Z sur #16608 — « deux drapeaux pour une seule échappatoire est de la flexibilité dont personne n'a encore eu besoin, et ils divergeraient au premier changement de l'un des deux rangs ». Ce n'est donc pas le drapeau qu'il faut corriger, c'est ce qu'il dit de lui-même.

Provenance

Réserve 2 de la review Hermes du 2026-09-17T23:34:29Z sur #16608, levée sur le fond le 2026-09-18T10:34:50Z avec ce résidu explicitement reporté : « une seule chose est due, et elle ne tient pas le merge : le texte d'aide du drapeau doit nommer les deux cas, sinon le nom delivered ment sur la moitié de ce qu'il fait ».

Cette issue est le nommage de ce report. Il avait été écrit en phrase sans être rattaché à une issue — ce qui, au sens strict de B.0, n'est pas un report valide : un report se nomme avant le merge, sinon il n'existe que dans la mémoire de celui qui l'a écrit.

Ce qui est attendu

  • Le texte d'aide de --include-delivered nomme les deux rangs qu'il court-circuite. Formulation suffisante, proposée à l'arbitrage : « inclut aussi les grains couverts par une PR ouverte ».
  • Vérifier qu'aucune autre surface ne décrit le drapeau à moitié : --help, le docstring du module, et toute mention dans docs/ ou .claude/rules/ (git grep -n 'include-delivered').
  • Ne pas ajouter de second drapeau : l'arbitrage du 18/09 est explicite et tient.

Non-objectifs

Comment savoir que c'est mort

python scripts/pick_idle_grain.py --help mentionne les grains couverts par une PR ouverte dans la description de --include-delivered.

Grain: LIGHT/tooling — quelques lignes, aucune logique touchée.

Activity

  1. jsboige commented on Sep 19, 2026

    @jsboige
    Owner

    [CLAIMED] #16797 — lane myia-po-2023:CoursIA — 2026-09-19T04:5xZ — paths: scripts/pick_idle_grain.py

    Preflight : 0 commentaire sur l'issue, aucune PR ouverte touchant pick_idle_grain.py (#16729 = rules doc, sans rapport ; #16768 couvre #16764, autre organe). Plan conforme à l'acceptance : (1) texte d'aide de --include-delivered nommant les DEUX rangs court-circuités (livrés + couverts par PR ouverte) ; (2) git grep -n 'include-delivered' pour rattraper toute surface à moitié descriptive (--help, docstring, docs/, .claude/rules/) ; (3) PAS de second drapeau, PAS de renommage (arbitrage 18/09 acquis). Test de mort : --help mentionne les grains couverts par une PR ouverte.

  2. jsboige commented on Sep 19, 2026

    @jsboige
    Owner

    [CLAIMED] #16797 — myia-po-2023:CoursIA — 2026-09-19T17:50Z

    Texte d'aide de --include-delivered nommant les deux rangs + verif surfaces (git grep include-delivered). Pas de second drapeau (arbitrage 18/09 tient).

  3. jsboige commented on Sep 19, 2026

    @jsboige
    Owner

    [DELIVERED] #16797 — deja livre par cette lane : PR #16802 (ouverte 19/09 matin, commit 489cd88, en attente review/merge ai-01). Le [CLAIMED] ci-dessous etait redondant — etat correct : travail en file de review.

  4. added a commit that references this issue on Sep 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions