Skip to content

Fix(ci,#22215): suites NanoClaw de #22277 -- position du test OMIT_SHA, 2>/dev/null residuels du sweep, fidelite des endpoints nommes #22278

Closed

Description

@myia-ai-01

Constat

La review NanoClaw de #22277 (2026-10-24T01:02:50Z, tête 1c16d8825e) a posé trois réserves 🟡. Elles ne touchent pas au correctif errexit, mais restent vraies à la tête 08675eb0f0.

  1. Une assertion du test peut passer pour une raison étrangère. test_le_no_op_des_check_runs_nomme_le_dernier_endpoint_et_le_code cherche OMIT_SHA=$SHA dans tout le workflow. Le test vérifie que la chaîne existe, pas sa position : une refonte qui sortirait l'affectation de la branche if ! BODY=$(…check-runs…); then … continue ne le ferait pas échouer. Le fichier contient déjà le bon geste, _branche_noop, qui localise le bloc. Il faut asserter que l'affectation se trouve entre l'ouverture de cette branche et son continue.
  2. D'autres échecs silencieux restent dans le même workflow. Il y a encore plusieurs 2>/dev/null, dont WFMAP (l. 466) : si cet appel échoue, la carte est vide et le repliement retombe sur run_id sans aucun signal. C'est le même schéma que les NO-OP corrigés, hors du périmètre de Fix(ci,#22215): le sweep nomme l'endpoint et le code HTTP de ses NO-OP #22277. Chaque site doit soit nommer l'endpoint et le code, soit justifier en commentaire pourquoi le silence est sans conséquence.
  3. Le nom de l'endpoint diffère de l'appel réel. L'avertissement des jobs nomme ?filter=latest, alors que l'appel porte ?filter=latest&per_page=100. De plus, detail est tronqué à 200 caractères pour check-runs et timing, contre 300 pour le listing.

Critère de clôture

  • Le test des check-runs échoue si OMIT_SHA=$SHA sort de sa branche. Le prouver par un témoin : déplacer la ligne et constater le rouge.
  • Chaque 2>/dev/null restant de pr-gate-stale-sweep.yml soit émet un signal nommé, soit porte un commentaire justifiant le silence.
  • Les endpoints nommés reproduisent l'appel, et la troncature de detail est la même partout, ou la différence est justifiée.

Origine : réserves NanoClaw de #22277, reportées ici par le coordinateur avant le merge. Rattachement : #22215.

Activity

  1. jsboige commented on Oct 24, 2026

    @jsboige
    Owner

    [CLAIMED] lane myia-po-2025:CoursIA — suites NanoClaw de #22277 : assertion de position OMIT_SHA (avec temoin rouge), 2>/dev/null restants du sweep nommes ou justifies, fidelite des endpoints nommes. paths: .github/workflows/pr-gate-stale-sweep.yml, scripts/**

  2. jsboige commented on Oct 24, 2026

    @jsboige
    Owner

    [DELIVERED] lane myia-po-2025:CoursIA -- PR #22285 (stack sur la tete 08675eb de #22277, a merger apres elle). Trois criteres couverts: assertion OMIT_SHA positionnelle + temoin rouge (deplacement apres le fi), les 3 2>/dev/null restants justifies par commentaire et les 3 sites signalants captivent leur stderr (WFMAP resume, wfnames warning, report RUN avec code extrait), endpoints nommes fideles a l'appel (per_page compris) et troncature detail unifiee a 300. Suite 14/14 + gardes errexit sur 7 extractions ; les 2 echecs dwell restants sont herites de origin/main (#22279), preuve en body PR.

  3. added a commit that references this issue on Oct 25, 2026
  4. added a commit that references this issue on Oct 26, 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