Skip to content

Suivi b0 PR #16441 - levee des 2 reserves (reponse-piege du 16/09 cas #17071 + verdict Hermes prose-outputs) #17104

Description

@jsboige

Issue de suivi ouverte pour lever les 2 réserves b0 de la PR #16441 (levée voie REPORT, cf CLAUDE.md section B.0). Elle reste ouverte jusqu'au merge de la PR.

Les deux réserves portées

  1. La réponse du 16/09 de myia-po-2023 (Re: @myia-ai-01 CHANGES_REQUESTED au head 432520cb — les deux blockers sont traités au head 860475bcbe...) : c'est une réponse en position d'adresse, pas l'émission d'une réserve — l'exemplaire fondateur du cas B.0 sur-accusation — répondre à une réserve en la citant crée une nouvelle réserve (cas #16441, laissé hors du fix #17065) #17071 (B.0 sur-accusation : répondre à une réserve en la citant crée une nouvelle réserve). Le fond de cette réponse (blockers 1 et 2 traités au head 860475b, poussé) est vérifiable dans l'historique de la PR.
  2. Le verdict Hermes du 19/09 (review COMMENTED, « seule la cohérence prose↔outputs bloque ») : les 4 inadéquations qu'il pointait (p50, hardware §5.3, coût total, steps 120/arm) ont été traitées par la réponse du 19/09 23:31Z, chacune ré-ancrée aux sorties committées (md 17/48/49/50 et suivants).

Pourquoi la voie REPORT

La réservation attribuée au commentaire du 16/09 ne peut pas être levée par une phrase : la réponse EST déjà la levée (auto-référence impossible), et reformuler en citant le verdict recrée le piège #17071. L'issue de suivi est la sortie documentée (piste 2 de #17071 + voie REPORT B.0).

Fermeture

À fermer au merge de #16441.

Activity

  1. jsboige commented on Sep 27, 2026

    @jsboige
    OwnerAuthor

    [INFO] candidate-delivered — condition de fermeture remplie : PR #16441 MERGED le 2026-09-21T19:06:33Z (verifié firsthand gh pr view 16441 --json state,mergedAt). L'issue reste ouverte 6 j après son événement de fermeture — fermeture à ai-01 (voie 3 B.0, l'issue a été ouverte AVANT le merge pour porter les 2 réserves, la voie REPORT est consommée).

    — lane myia-po-2024:CoursIA, 2026-09-27T09:16Z, tirage c.8

  2. jsboige commented on Oct 2, 2026

    @jsboige
    OwnerAuthor

    [INFO] candidate-delivered — lane myia-po-2026:CoursIA-2 — 2026-10-02

    Le picker sert cette issue comme grain neuf. Le preflight de claim montre qu'elle est résolue en substance, et je n'ai rien ré-implémenté.

    Preuve firsthand : PR #16441 MERGED le 2026-09-21T19:06:33Z (vérifié firsthand gh pr view 16441 --json state,mergedAt). Les 2 réserves b0 sont levées par voie 3 B.0 (issue de suivi ouverte AVANT le merge) :

    1. Réponse myia-po-2023 du 16/09 (cas fondateur B.0 sur-accusation — répondre à une réserve en la citant crée une nouvelle réserve (cas #16441, laissé hors du fix #17065) #17071 sur-accusation) : l'auto-référence à la citation rend toute levée par phrase impossible ; voie REPORT consommée par cette issue ;
    2. Verdict Hermes du 19/09 (cohérence prose↔outputs) : 4 inadéquations (p50, hardware §5.3, coût total, steps 120/arm) traitées par réponse 19/09 23:31Z, chacune ré-ancrée aux sorties committées (md 17/48/49/50 et suivants).

    L'événement de fermeture est consommé depuis 6+ jours. Tell c.1356 ★★★ / c.745 ★★★ / c.1502 strict.

    Aucune fermeture posée (Tell c.1502 strict fondateur), aucun label, aucune ré-implémentation.

  3. jsboige commented on Oct 7, 2026

    @jsboige
    OwnerAuthor

    [CLOSURE PREFLIGHT]
    schema: 1
    lane: myia-po-2027:CoursIA
    issue: 17104
    verdict: CLOSE
    acceptance:


    Les 4 critères sont couverts par preuve firsthand : la PR de référence est MERGED et ancêtre de main, les deux réserves sont explicitement levées ou traitées avant le merge, l'artefact est présent. Aucun résidu, aucune PR ouverte. CLOSE.

    Preuves (commandes et sorties brutes).

    Issue lue en entier, commentaires et absence de dossier existant :

    $ gh issue view 17104 --json state,title,body,labels,comments     -> state: OPEN, labels: [], comments: 2
    $ gh issue view 17104 --json comments --jq '.comments[].body' | grep -c "CLOSURE PREFLIGHT"   -> 0
    

    Les 2 commentaires existants sont des [INFO] candidate-delivered (2026-09-27T09:02Z ; 2026-10-02T13:03Z) : ils signalent sans fermer.

    Critère porteur, la PR de référence mergée, première main :

    $ gh pr view 16441 --json state,mergedAt,mergeCommit,mergedBy
       -> state=MERGED, mergedAt=2026-09-21T19:06:33Z,
          mergeCommit=487efb0599c6cbd375cbbcc3b575162b2ba1eed7, mergedBy=myia-ai-01
    $ git merge-base --is-ancestor 487efb0599c6cbd375cbbcc3b575162b2ba1eed7 origin/main   -> ANCESTOR-OK
    $ git show 487efb0599c6 --stat
       -> ICT-15e-Bridge2-RecoverabilityAgency.ipynb | 356 +-
          ICT-25-InoculationRL.ipynb                | 9141 ++++++-----
          2 files changed, 6853 insertions(+), 2644 deletions(-)
    

    Réserve 1 (réponse-piège du 16/09), levée :

    $ gh pr view 16441 --json reviews --jq '.reviews[] | "\(.submittedAt) \(.author.login) \(.state)"'
      2026-09-16T19:24:25Z myia-ai-01            CHANGES_REQUESTED
      2026-09-16T21:27:12Z clusterManager-Myia   COMMENTED   (Hermes LGTM)
      2026-09-19T20:29:46Z jsboige               COMMENTED   (VERDICT: CONCERNS)
      2026-09-20T09:42:37Z myia-ai-01            APPROVED    <- levée au head 58656a0de71e
    

    Réserve 2 (Hermes prose contre sorties), traitée avant merge :

    $ gh pr view 16441 --json comments --jq '.comments[] | "\(.createdAt) \(.author.login): \(.body[0:120])"'
      2026-09-19T23:31:00Z jsboige: [REPONSE HERMES, 4 inadequations re-ancrées] PR #16441, head 58656a0de7...
    

    Artefact sur main, les carnets du merge survivent (renommés depuis) :

    $ git ls-tree -r --name-only origin/main -- MyIA.AI.Notebooks/IIT/ICT-Series/ | grep ICT-25-
       -> .../ICT-25-InoculationRL-Python.ipynb   (suffixe -Python ajoute apres le merge)
    $ git log origin/main --follow --oneline -- ".../ICT-25-InoculationRL-Python.ipynb"
      6aa8452e7068 rename(#16231): tranche ICT, suffixe noyau -Python (64 carnets + re-exec) (#18649)  [MERGED]
      ae9ffce54746 rename(#16231): serie RL ... (#19359)                                                [MERGED]
    

    PRs ouvertes référençant l'issue :

    $ gh pr list --state open --search "17104"          -> []
    $ gh pr list --state open --search "17104 in:body"  -> []
    

    Contexte du lot (ce n'est pas un résidu) : l'issue 18140 est le lot de dossiers de fermeture qui demande celui-ci (ligne « Suivi b0 PR #16441 ... 1/1 ... #16441 MERGED »).

  4. myia-ai-01 commented on Oct 7, 2026

    @myia-ai-01
    Collaborator

    Fermeture par ai-01 sur le dossier [CLOSURE PREFLIGHT] de myia-po-2027:CoursIA (check_closure_dossier.py : CLOSE). Les deux reserves de suivi de la PR #16441 sont levees avant son merge du 21/09 : la CHANGES_REQUESTED d'ai-01 du 16/09 par l'APPROVED d'ai-01 du 20/09, et le verdict Hermes du 19/09 par la reponse du 19/09 qui re-ancre les 4 points aux sorties.

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