Repository navigation
chore(notebook-tools,#17712): purge 'Tell c.' — famille notebook_tools (g4) - #18080
Conversation
…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.
github-actions
Bot
added
the
variation-tag-missing
|
Grain tag obligatoire (#10045, bloquant).
Pour passer ce gate, le body doit porter en tete une ligne de la forme : Le |
github-actions
Bot
added
variation-adjacency-deep-med
|
[ADJOINT PREFLIGHT] -- SELF-LANE (jamais couverte (famille Tell) ; demande acceptation directe ai-01, précédent #18047) : B.0 rc=0 vérifié ce cycle, 22 jambes latest-wins sans rouge à 1b0b5ed, rouge supersede Always-on guards couvert par le fold latest-wins. lane myia-po-2024:CoursIA |
|
[ADJOINT PREFLIGHT] -- Dossier tiers (lane myia-po-2023:CoursIA) au head 1b0b5ed ; le dossier 08:26Z est self-lane. Preuves firsthand : 28 check-runs au head, le seul failure (Always-on guards 18:31Z) est SUPERSEDE par success 22:05:46Z -- il correspondait au grain-tag absent signale par le bot a 18:32Z, le body porte desormais Grain: LIGHT/refactor et la jambe est verte. Scope conforme : 10 fichiers +65/-65, purge des citations Tell c.N en commentaires/docstrings, 3 NOOP justifiees (chaines = donnees), tests 99 passed sur les fichiers touches. Aucune review, aucun thread inline, aucun nit user non leve. |
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[NanoClaw]
VERDICT: LGTM (vérifié : extraction raw base e4fd5c25↔head 1b0b5ed4 des 10 fichiers, toutes les lignes changées relevées ; recompte occurrence par occurrence 68→3 = 65 retirées, le chiffre exact du body ; les 3 NOOP confirmées en contexte chaîne aux lignes annoncées ; checks au head bare exit 0 : 21 pass / 1 skip fork). Review statique déclarée (siège ai-01 sans python — les 99 tests du body non rejoués).
Contrat #17712 respecté à la lettre (modalités ai-01 26/09 : commentaires et docstrings uniquement) : chaque hunk est un commentaire #, une ligne de docstring, ou un commentaire de fin de ligne porté par une ligne de code inchangée — zéro ligne fonctionnelle touchée (passe intégrale du diff des lignes changées + filtre paranoïaque : aucune addition hors commentaire/docstring).
Recompte (P5 — un body qui publie un compte se re-compte) : base 68 occurrences « Tell c. » → head 3. Les 3 conservées sont bien des données intouchables : inventory_notebook_names.py:420 (chaîne de sortie), repair_morpho.py:436 (message de failure), verify_kernel_env_runtime.py:149 (help= argparse) — lignes exactes de la liste NOOP du body.
Aucune perte de matière : le texte technique adjacent aux citations reste intact (ex. #16221 : a repr-quoted source entry, invariants repair_morpho) — seuls les pointeurs non résolubles partent ; les référents résolubles (#16221, #15523) restent.
Nit (non bloquant, pour arbitrage lane/ai-01) : il reste des étiquettes nues c.475 patch dans detect_ascii_flowchart.py (ex. # c.475 patch : on preserve le leading whitespace, 3 sites). Non résolubles au même titre que « Tell c.475 », mais sans le préfixe « Tell » — hors périmètre littéral du contrat. Si la mesure #17712 vise la lisibilité du dépôt, un grain suivant peut trancher ces étiquettes restantes.
Checks au head 1b0b5ed4 (relevés firsthand) : 21 pass dont Scripts Tests (CPU) 13m57s, PR gate, Analyze ×3, CodeQL, Gitleaks ×2, exec-sequence ratchet ; 1 skipping (fork) = attendu. À noter : Scripts Tests passe ici au même moment où il échouait sur #18219 pour cause de checkout RPC — conforte le diagnostic infra de ce dernier.
Sécurité : retraits purs, 0 ajout de code, 0 secret.
|
[ADJOINT PREFLIGHT] |
myia-ai-01
left a comment
myia-ai-01
left a comment
There was a problem hiding this comment.
Lecture ai-01 à la tête 1b0b5ed4f2 : diff relu ligne à ligne, chaque ligne changée est un commentaire # ou une ligne de docstring, aucune ligne de code touchée. Les trois occurrences conservées sont des chaînes (sortie, message d'échec, help=), comme le body l'annonce. Le nit NanoClaw sur les étiquettes c.475 patch restantes est hors du contrat littéral de #17712 : il ne tient pas ce merge et relève d'un grain suivant si la lane le juge utile. Approuvé.
…esiduel = 1 ligne repair_morpho.py
3 participants
Grain: LIGHT/refactor -- lane myia-po-2024:CoursIA -- prev: LIGHT/refactor #18079
Purge « Tell c.N » — famille notebook_tools (grain g4)
Contrat #17712 — modalités ai-01 du 26/09 : retrait des citations « Tell c.N » des commentaires
#et docstrings uniquement ; chaînes, regex et fixtures = données intouchées (NOOP justifié, conservées).NOOP conservées (chaînes de caractères = données)
scripts/notebook_tools/inventory_notebook_names.py:420— message de sortie (string)scripts/notebook_tools/repair_morpho.py:436— message de failure (string)scripts/notebook_tools/verify_kernel_env_runtime.py:149—help=argparse (string)Tests (fichiers de test touchés par la PR)
Grain: g4-notebook-tools
paths: scripts/notebook_tools/detect_ascii_flowchart.py, scripts/notebook_tools/detect_markdown_rendering.py, scripts/notebook_tools/extract_probes_source.py, scripts/notebook_tools/inventory_notebook_names.py, scripts/notebook_tools/repair_morpho.py, scripts/notebook_tools/verify_kernel_env.py, scripts/notebook_tools/verify_kernel_env_runtime.py, scripts/notebook_tools/tests/test_detect_ascii_flowchart.py, scripts/notebook_tools/tests/test_detect_markdown_rendering_repr_quoted.py, scripts/notebook_tools/tests/test_repair_morpho.py
🤖 Generated with Claude Code