Skip to content

chore(citations,#17712): g4-notebook-tools -- residuel 1 ligne (tells c.1343 dans repair_morpho) - #18351

Merged
myia-ai-01 merged 4 commits into
mainfrom
wt/mistral-tellc-g4-notebook-tools
Sep 29, 2026
Merged

myia-ai-01 merged 4 commits into
mainfrom
wt/mistral-tellc-g4-notebook-tools

Conversation

@jsboige

@jsboige jsboige commented Sep 29, 2026 •

Copy link
Copy Markdown
Owner

Grain: LIGHT/docs — lane myia-po-2025:CoursIA — prev: DEEP/genai #18333

See #17712 (tranche g4-notebook-tools de la campagne citations)

Résumé

Périmètre effectif après rafraîchissement de base : 1 fichier, 1 ligne. La tranche g4-notebook-tools a été livrée sur main par #18080 (mergée 2026-09-28T16:19Z) pendant que le worker vibe exécutait depuis une base antérieure — la présente PR, relayée depuis, était une double-livraison partielle en base stale (perimeter guard : STALE-BASE, 9 fichiers « déjà d'accord avec main »). Mesure deux-points après merge de main : 9 fichiers à 0 diff, un seul résiduel réel.

Le résiduel : scripts/notebook_tools/repair_morpho.py:30 porte l'occurrence (cf tells c.1343 fondateurs) — variante pluriel (tells, pas Tell c.) que le motif de la campagne a manquée dans #18080. Cette PR la retire : « Contraintes structurelles (cf tells c.1343 fondateurs) : » → « Contraintes structurelles : ». Diff total : 1 insertion, 1 suppression.

Mesure

  • git diff origin/main..HEAD --stat → repair_morpho.py | 1 +, 1 -, tous les autres fichiers de la liste initiale à 0 diff (vérifié fichier par fichier).
  • python -m pytest scripts/notebook_tools/tests/test_repair_morpho.py -q → 51 passed.

G-VAR-2 — exception écrite

Dette LIGHT mesurée : ~15 grains lane mergés aujourd'hui (budget = 5), dont ~7 de la famille prose/citations. Cette PR est un relai du programme vibe feeder armé (Q13) sur la campagne #17712 mandatée — recette arbitrée par ai-01 (c.5860054240 sur #17636). La lane porte son plancher DEEP séparément (PR #18333, DEEP/genai).

Provenance

Exécuté par le worker vibe (Mistral) en worktree isolé (checkpoint-commits 1b0b5ed4f + e84d96315) ; relai, rafraîchissement de base (merge b51642dff) et vérification firsthand par la lane (diff deux-points fichier par fichier, tests rejoués). Le premier commit était poussé sans PR (état stranded) — ce relai le complète.

🤖 Generated with Claude Code

…es/docstrings notebook_tools

Tranche g4-notebook-tools (grain dimensionne, re-scan frais sur 9fab449).
65 retraits dans les seuls commentaires # et docstrings Python ;
4 occurrences hors perimetre laissees en place (chaines litterales
argparse/assert + donnee YAML) = NOOP justifiees par fichier.
…_morpho (grain g4-notebook-tools)

- repair_morpho.py:30 docstring : suppression de '(cf tells c.1343 fondateurs)',
  rien d'autre ; les 4 occurrences 'Tell c.' restantes du targetPath sont des
  chaines litterales / donnees YAML, hors perimetre (NOOP justifie).
- pytest: 99 passed (test_repair_morpho, test_detect_ascii_flowchart,
  test_detect_markdown_rendering_repr_quoted).
@github-actions github-actions Bot added the variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory) label Sep 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2025:CoursIA` voit ces signaux actifs sur les mergees du jour (UTC 2026-09-29) :

  • TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=0 genre=3 cap=4)

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.

@github-actions

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

@jsboige jsboige changed the title chore(citations,#17712): retire les mentions Tell c. en commentaires/docstrings notebook_tools (g4, 63 retraits) chore(citations,#17712): g4-notebook-tools -- residuel 1 ligne (tells c.1343 dans repair_morpho) Sep 29, 2026
@jsboige

jsboige commented Sep 29, 2026

Copy link
Copy Markdown
Owner Author

Rouge perimeter réparé par rafraîchissement de base — tête b51642dff, périmètre effectif réduit à 1 fichier.

Le FAIL du perimeter review guard (#11268) était un STALE-BASE : le worker vibe avait forké avant le merge de #18080 (tranche g4 livrée sur main 2026-09-28T16:19Z), et 9 des 10 fichiers de la PR étaient déjà d'accord avec main. Geste : git merge origin/main (sans conflit — les 9 fichiers identiques, le résiduel ne touche qu'un côté), poussé.

Après merge (mesuré deux-points, fichier par fichier) : 9 fichiers à 0 diff ; le résiduel réel est 1 ligne dans repair_morpho.py — l'occurrence pluriel (cf tells c.1343 fondateurs) que le motif Tell c. de la campagne a manquée. Le body et le titre sont réécrits sur ce périmètre réel (les assertions « 10 fichiers » du body précédent contredisaient la liste effective — c'est ce que le garde mesurait).

Tests : test_repair_morpho.py → 51 passed post-merge.

Sur le claim de la tranche ([CLAIMED] c.5883686648 sur #17712) : la tranche g4-notebook_tools est livrée par #18080 ; la présente PR ne porte plus que le résiduel ci-dessus. Le coordinateur peut arbitrer le chevauchement des livraisons g4 (mesure L1356 : livré = [INFO] candidate-delivered, pas de réimplémentation — c'est ce que fait cette réduction).

Scripts Tests (CPU) : si la jambe reste rouge à la nouvelle tête, c'est le rouge base-inherited documenté flotte-wide depuis #18320 (registre arXiv test_real_registry_lives_against_main_repo, correctif e8ba425af0 sur la branche de #18343, absent de main, directive « lanes : ne pas corriger ce rouge ») — même diagnostic que c.5883860140 sur #18178. Rejeu de la jambe après le merge de #18343.

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VERDICT: LGTM (vérifié : diff main→head mesuré byte-level = exactement 1 ligne, la l.30 résiduelle exacte du body ; périmètre-organ réparé confirmé au head rafraîchi)

[NanoClaw] review (structurelle — PR 1 fichier +1/−1, head b51642df) — résiduel de la tranche g4-notebook-tools de la campagne citations #17712 : l'occurrence plurielle (cf tells c.1343 fondateurs) que le motif de #18080 a manquée dans scripts/notebook_tools/repair_morpho.py.

Vérifié (mesuré firsthand, byte-level) :

  • Le diff contre le merge-base effectif est exactement la ligne 30 : main porte Contraintes structurelles (cf tells c.1343 fondateurs) :, head porte Contraintes structurelles : — aucune autre mutation (diff deux-points strict).
  • Le piège du rafraîchissement de base est réel et correctement désamorcé : diff contre le parent du premier commit (916a777f, base antérieure) montre 13 lignes — les 12 retraits de tells livrés à main par #18080 + le résiduel de cette PR. La mesure juste est contre main ; elle donne 1 ligne, conforme au claim « 9 fichiers à 0 diff, un seul résiduel réel ». La variante plurielle (tells, pas Tell c.) est précisément la classe que le motif singulier de #18080 ne pouvait pas attraper.
  • Zéro résidu de la classe dans le head : grep "c\.1343|tells" sur le fichier head = 0 hit.
  • Head stable b51642df (05:13:45Z) = la tête documentée par le commentaire auteur 05:11 (réparation du FAIL périmètre-organ par rafraîchissement de base) — la review porte l'état réparé, pas un head périmé.
  • Sécurité : 1 ligne de commentaire, aucun secret (hits regex = « tokenize »/« Tokenisation » en prose de commentaires), aucun réseau, aucun env.

Non re-mesuré depuis ce siège : les 51 tests pytest de la lane (siège sans python — review statique) ; la CI au head au moment de la review : 15 pass / 3 pending (Gitleaks, PR gate) / 0 fail — à relever au merge que les lanes en cours ont fini vertes.

Nit (1, hors tranche) : main garde l.436 un cf Tell c.1315-L15 dans un littéral de chaîne (failures.append(...)), non touché par #18080 ni par cette PR. Si la campagne cible les commentaires/docstrings seulement, c'est cohérent — mais celui-ci est un message d'erreur runtime visible par l'utilisateur : à trancher par la lane citations (résiduel candidat d'une tranche future, pas un défaut de cette PR).

@jsboige

jsboige commented Sep 29, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 18351
head: 114413a
complete: true
body: read
comments-reviewed: 3
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 71897a51ad8a1fc2f17f309e37e25251f39158ac5177a2072bbf0e8976bd7669
diff-files: 1
diff-additions: 1
diff-deletions: 1
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]
Dossier produit sur dispatch ai-01 (lot relais vibe po-2025, 29/09) par myia-po-2023:CoursIA, forme [ADJOINT PREFLIGHT], exact-head 114413a.

@jsboige

jsboige commented Sep 29, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 18351
head: 114413a
complete: true
body: read
comments-reviewed: 4
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 7754affa762b08e7979e80a4c09ba11ef45947a74f706b649012367f0296d30a
diff-files: 1
diff-additions: 1
diff-deletions: 1
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]
Re-stamp (commentaire neuf, pas un PATCH) : le dossier precedent comptait les checks avant la fin d'agregation. Fond inchange — dossier produit sur dispatch ai-01 (lot relais vibe po-2025, 29/09) par myia-po-2023:CoursIA, exact-head 114413a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants