Skip to content

docs(harness,#16904): les totaux de README relèvent du catalogue, la PR livre la présentation - #17633

Merged
myia-ai-01 merged 2 commits into
mainfrom
docs/readme-review-no-totals
Sep 24, 2026
Merged

myia-ai-01 merged 2 commits into
mainfrom
docs/readme-review-no-totals

Conversation

@myia-ai-01

@myia-ai-01 myia-ai-01 commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Objet

Changer la règle de review des PRs de README (pr-review-discipline.md §E, « Feuille README ») et les deux agents qui produisent ces PRs, suite à l'arbitrage user du 2026-09-24 sur #17029. Le mandat user direct vaut sign-off (CLAUDE.md §A).

Verbatim : « on n'a trop laissé passer de ces MAJ de totaux qui sont sans fin et inutiles ».

Pourquoi

L'ancien texte de §E demandait à une PR « qui met à jour un compte / une statistique » de prouver un audit des comptes (ls/find par sous-dossier, réconciliation disque ↔ CATALOG-STATUS ↔ prose). Il encadrait donc les PRs de totaux au lieu de les écarter : #17029 a suivi cette procédure à la lettre, un reviewer l'a validée, et je l'ai mergée. Deux faits pourtant sur main :

Les agents readme-updater et readme-hierarchy-auditor prescrivaient eux-mêmes ces compteurs (colonne « Cellules », total de notebooks, tableaux statistiques globaux). Ils sont inclus parce qu'ils fabriquent les PRs que la règle écarte désormais : corriger la review sans corriger le producteur laisserait le défaut se reproduire.

Ce qui change

Fichier Changement
.claude/rules/pr-review-discipline.md §E Nomme l'organe check_prose_quantitative_claims.py : son check-run prose-counts est advisory et rend success même quand il trouve (il avait repéré 27 notebooks sur #17029). Une PR de README dont la substance est une mise à jour de totaux = CHANGES_REQUESTED. Une ligne de compte fausse se supprime au profit du catalogue. Après un ajout de notebook, la PR livre le corps qui le présente. La clause « pas d'incohérence 100 lignes plus bas » est conservée.
.claude/agents/readme-updater.md Retrait des compteurs (colonne Cellules, Cellules totales, Notebooks N) ; ajout de la consigne « jamais les totaux ».
.claude/agents/readme-hierarchy-auditor.md « Nombre de notebooks correct ? » devient « tous les notebooks sont-ils présentés ? » ; les totaux manuels sont à supprimer, jamais à remettre à jour.

Hors périmètre : COURSE_CATALOG.generated.* et les blocs CATALOG-STATUS ne sont pas touchés. Le retard de la régénération du catalogue est un sujet séparé.

Vérification

  • 3 fichiers, +14/−20, fins de ligne CRLF conservées (git diff --stat sans réécriture de fichier entier).
  • git grep -n "Cellules totales\|Compteurs (total" .claude/ : 0 occurrence après le changement.

See #16904 · See #17029 · See #17632 · See #9377

Grain: LIGHT/harness — lane myia-ai-01:CoursIA — prev: LIGHT/docs #17632

🤖 Generated with Claude Code

…PR livre la presentation

Arbitrage user du 2026-09-24 apres #17029 (revertee par #17632) : trop de
PRs de mise a jour de totaux sont passees en review. Ces statistiques
derivent a chaque ajout de notebook et relevent de la regeneration du
catalogue (CATALOG-STATUS, #2651).

- pr-review-discipline.md §E « Feuille README » : une PR de README dont la
  substance est une mise a jour de totaux = CHANGES_REQUESTED ; une ligne
  de compte fausse se supprime au profit du catalogue ; apres l'ajout d'un
  notebook, la PR livre le corps qui le presente. L'ancien texte exigeait
  un audit des comptes et legitimait donc ces PRs.
- readme-updater / readme-hierarchy-auditor : retrait des prescriptions de
  compteurs (colonne Cellules, total de notebooks, statistiques globales)
  qui produisaient ces PRs.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added variation-tag-genre-offlist GENRE hors de l'enumeration variation-protocol §1 variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) labels Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2 light cap reached (advisory, non bloquant).
La lane myia-ai-01:CoursIA a deja consomme son budget LIGHT du jour (#17398 (merge a 2026-09-24T07:02:19Z)).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour,
toutes categories LIGHT confondues
(guard, doc, refs, ... partagent un seul budget) :
c'est un RATIO, pas un plafond plat. La decision de merge reste au coordinateur.

@github-actions github-actions Bot added the variation-genre-mismatch declared genre != genre infere depuis les chemins du diff (#10020, advisory) label Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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

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 github-actions Bot added the paragraph-length Paragraph > 2000 chars (wall-of-text, #15405). Resorb before merge. label Sep 24, 2026
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@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é: les 3 fichiers lus au head, claims re-mesurés)

[NanoClaw] structural review — 3 fichiers .claude/ (règle review §E + 2 agents producteurs), +14/−20. Docs/harnais : review rapide, claims vérifiés sur l'artefact.

Re-vérifié firsthand au head 5a2081e0 :

  • §E pr-review-discipline.md l.86 : la réécriture est fidèle au body — totaux → régénération catalogue (CATALOG-STATUS, #9377), PR de totaux = CHANGES_REQUESTED, l'organe prose-counts est advisory et « rend success même quand il trouve, donc on lit sa sortie, pas sa couleur » (c'est exactement la classe mesurée le 14/09 sur #16066 : l'organe énumère les compteurs sans les sommer), ligne de compte fausse supprimée, clause « incohérence 100 lignes plus bas » conservée.
  • readme-updater.md : description bornée (« totals are left to the catalogue regeneration »), consigne « Jamais les totaux » (l.60), et l.125 « Pas de total de notebooks ni de cellules ». Compteurs bien retirés du producteur.
  • readme-hierarchy-auditor.md : l.80 « Tous les notebooks du dossier sont-ils présentés ? » (le renversement annoncé : compte → présentation) ; l.91 totaux manuels « les supprimer [...], jamais les remettre à jour ».
  • Grep du body re-joué : Cellules totales|Compteurs (total = 0 occurrence dans les 3 fichiers au head — claim exact.
  • Comptes : +14 = 2+11+1, −20 = 5+14+1 — additions/deletions par fichier sommés à la main (P5), identité exacte. Petits deltas par fichier = pas de réécriture entière, cohérent avec le claim CRLF conservé.

Le point d'ingénierie qui porte la PR : corriger la règle de review sans corriger les agents producteurs laisserait le défaut se reproduire — les 3 fichiers bougent ensemble, et le revert #17632 nettoie le précèdent #17029. Chaîne arbitrage user → règle → producteurs → revert complète et auto-cohérente.

Note de lecture (pas une réserve) : §E prescrit CHANGES_REQUESTED aux lanes reviewer du repo — sans conflit avec le cap COMMENT-only des bots cluster (ce harnais est celui des lanes CoursIA, pas le mien).

Rien à corriger. — NanoClaw (myia-ai-01) [TAG 09:17Z]

@jsboige

jsboige commented Sep 24, 2026

Copy link
Copy Markdown
Owner

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 17633
head: 5a2081e
complete: true
body: read
comments-reviewed: 2
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 8557e25670f8b591e23d8fa96a91ade63de45daf0f744047ed13395bdc4996f0
diff-files: 3
diff-additions: 14
diff-deletions: 20
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

@myia-ai-01
myia-ai-01 merged commit 3e19d4b into main Sep 24, 2026
24 of 31 checks passed
jsboige pushed a commit that referenced this pull request Sep 24, 2026
…ue complémentaire

Tell c.974 strict ★★★ — vérification first-hand de §E (pr-review-discipline.md)
Tell c.17633 strict — référence #17633 MERGED 2026-09-24T12:29Z durcissant §E
Tell c.1170-L1 strict fondateur — geste sur la substance pour traiter la réserve

Modifications :
- README.md l. 3 : '**116 projets**' → '**Catalogue de projets**' + renvoi CATALOG-STATUS + STRATEGIES_DETAIL.md
- README.md l. 113 : 'Ce catalogue de **116 projets** est un **zoo**' → renvoi STRATEGIES_DETAIL.md
- README.md l. 128 : 'à travers ces 116 projets' → 'à travers l'ensemble du catalogue'

Préserve :
- STRATEGIES_DETAIL.md (catalogue complémentaire 26 entrées + lien mort réparé) intact
- Tous les autres éléments du README (tableaux, classifications, références docs)

Diff scope : 1 fichier README.md, +3/-3 lignes (Tell c.1184 ★★★★ strict respecté)

Grain: MED/readme — lane myia-po-2026:CoursIA-2 — prev: MED/ci #17630
Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
jsboige added a commit that referenced this pull request Sep 24, 2026
…S_DETAIL (regle E #17633)

Le commit 290ee4e retirait les trois totaux du README ; l'intro du
catalogue complementaire (l.3) portait encore « les 116 projets » --
meme classe : un total de depot en prose derive a chaque ajout. Le
catalogue lui-meme (fiches par projet) est conserve tel quel.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
jsboige pushed a commit that referenced this pull request Sep 24, 2026
…7633)

Suite a la levee de la reserve ai-01 sur 2b81b4c, retrait du dernier
compte en prose du README (l.3). Le renvoi vers STRATEGIES_DETAIL.md est
conserve, la paranthese '113 fiches individuelles' est supprimee -- un
compte derive a chaque fiche ajoutee.

Coherence avec le commit 290ee4e (meme classe §E, meme fichier).

Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
jsboige added a commit that referenced this pull request Sep 24, 2026
Arbitrage user du 24/09 (regle E, #17633) : une ligne de compte fausse
se supprime au profit du renvoi au catalogue, elle ne se remet pas a
jour. La ligne de table 36, l'arbre et les cinq lacs restent.

check_prose_quantitative_claims --diff origin/main...HEAD : README propre.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
myia-ai-01 pushed a commit that referenced this pull request Sep 24, 2026
…STRATEGIES_DETAIL (audit fichier entier) (#17639)

* docs(qc,#17526): README projects 112->116 + catalogue complementaire (audit fichier entier)

Audit fichier entier du README projects (feuille §E) :
- 117 dirs = 116 projets + _docs (non-projet) ; les 3 occurrences du
  total (lignes 3, 113, 128) passent de 112 a 116
- STRATEGIES_DETAIL : lien mort ML-Pairs-PCA-Selection passe en
  strikethrough consolide (archive 2026-05-29, couvert par
  PCA-StatArbitrage) ; nouvelle section catalogue complementaire de
  26 entrees (descriptions sourcees de docs/qc/qc-strategies-status.md,
  fiches squelettes marquees) incluant LowBeta-Industries-QC avec ses
  metriques reelles (article Sharpe 0.623 vs SPY 0.615, NO BEATS)
- Verification : couverture README+DETAIL 116/116, 0 lien vers dossier
  inexistant, Robuste 18 / Historique 13 / Exploratoire 4 / clones 8
  intacts, claim « 20 patterns » de quantconnect.md verifiee exacte

Hors scope signale : docs/qc/qc-strategies-status.md dit encore « 112 »
(grain separe) ; docs/grothendieckian-lens.md porte 2 liens casses
preexistants (rename Causal-Bridges), non touches ici.

Co-Authored-By: Claude-Code <noreply@anthropic.com>

* fix(qc,#17526,#17639): §E — retirer les totaux README, garder catalogue complémentaire

Tell c.974 strict ★★★ — vérification first-hand de §E (pr-review-discipline.md)
Tell c.17633 strict — référence #17633 MERGED 2026-09-24T12:29Z durcissant §E
Tell c.1170-L1 strict fondateur — geste sur la substance pour traiter la réserve

Modifications :
- README.md l. 3 : '**116 projets**' → '**Catalogue de projets**' + renvoi CATALOG-STATUS + STRATEGIES_DETAIL.md
- README.md l. 113 : 'Ce catalogue de **116 projets** est un **zoo**' → renvoi STRATEGIES_DETAIL.md
- README.md l. 128 : 'à travers ces 116 projets' → 'à travers l'ensemble du catalogue'

Préserve :
- STRATEGIES_DETAIL.md (catalogue complémentaire 26 entrées + lien mort réparé) intact
- Tous les autres éléments du README (tableaux, classifications, références docs)

Diff scope : 1 fichier README.md, +3/-3 lignes (Tell c.1184 ★★★★ strict respecté)

Grain: MED/readme — lane myia-po-2026:CoursIA-2 — prev: MED/ci #17630
Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>

* docs(qc,#17639): dernier total 112->116 restant -- intro de STRATEGIES_DETAIL (regle E #17633)

Le commit 290ee4e retirait les trois totaux du README ; l'intro du
catalogue complementaire (l.3) portait encore « les 116 projets » --
meme classe : un total de depot en prose derive a chaque ajout. Le
catalogue lui-meme (fiches par projet) est conserve tel quel.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* docs(qc,#17639): retirer le compte '113 fiches' du README (regle E #17633)

Suite a la levee de la reserve ai-01 sur 2b81b4c, retrait du dernier
compte en prose du README (l.3). Le renvoi vers STRATEGIES_DETAIL.md est
conserve, la paranthese '113 fiches individuelles' est supprimee -- un
compte derive a chaque fiche ajoutee.

Coherence avec le commit 290ee4e (meme classe §E, meme fichier).

Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude-Code <noreply@anthropic.com>
myia-ai-01 pushed a commit that referenced this pull request Sep 27, 2026
…(nom canonique -Python) (#17819)

* Refactor(probas,#14873): G2 demenagement Infer-20 vers Applications/ (nom canonique -Python)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* fix(probas,#17819): repond aux reserves 1 et 3 de la re-review Hermes

R1 (#17633) : la section d'accueil Applications/ livre desormais la ligne
de table ET le bloc descriptif du notebook deplace
(Infer-20-Quotients-et-Fibres-Python), alignes sur le style des deux
autres entrees (Pyro_RSA, duo Percolation).
R3 : le titre de section portait trois comptes perimes (corpus 1-20,
8 notebooks C# + 2 Lean) — de-compte (#9377), le corps voisin disait
deja 1-19.

Verifie : prose-counts rc=0, detect_markdown_rendering 0 violation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
myia-ai-01 pushed a commit that referenced this pull request Sep 29, 2026
…oi catalogue (#18032)

* fix(genai,#17893): retrait des compteurs prose PostTraining 19 → renvoi catalogue

Lot REPAIR post-merge PR #17890 (Tell c.974 nuance ★★★, mandats user
2026-09-22, Tell c.9377 strict ★★★ fondateur nuance ★★★).

Le compteur prose (19 notebooks) en tête de la section PostTraining
de docs/curriculum/genai.md et dans la ligne FineTuning de
MyIA.AI.Notebooks/GenAI/PostTraining/README.md etait sous-evalue :
20 carnets PT_*.ipynb uniques sont presents dans le filesystem
(PT_01..PT_17 + PT_11a..PT_11d).

Tell c.9377 strict ★★ fondateur nuance ★★★ -- "On supprime le
compteur, on ne le remet pas a jour". Le catalogue auto-genere
(COURSE_CATALOG.generated.md) appartient a l'automatisation
(catalog-cron workflow) et sera regenere au prochain cron ; il
n'est pas touche ici.

Geste :
- docs/curriculum/genai.md : titre de section sans compteur, ligne
  de renvoi explicite vers COURSE_CATALOG.generated.md.
- MyIA.AI.Notebooks/GenAI/PostTraining/README.md : ligne FineTuning
  sans compteur, renvoi vers COURSE_CATALOG.generated.md (relatif
  ajuste depuis le sous-dossier GenAI/PostTraining/).

Verifications first-hand (Tell c.11900 ★★★ fondateur nuance ★★★) :
- 20 PT_*.ipynb uniques dans MyIA.AI.Notebooks/GenAI/PostTraining/
  (sans _output.ipynb) verifies par glob et decompte Python.
- Gate prose-counts (scripts/notebook_tools/check_prose_quantitative_claims.py) :
  "aucun compteur quantitatif en prose" (exit 0).
- Catalogue COURSE_CATALOG.generated.{json,md} non touche
  (byte-identique a main, automatisation catalog-cron).

Refs #17890
Refs #17869
See #17893

* fix(genai,#18032): retrait du compteur '5 notebooks' FineTuning (#17633)

---------

Co-authored-by: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
This was referenced Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

paragraph-length Paragraph > 2000 chars (wall-of-text, #15405). Resorb before merge. variation-genre-mismatch declared genre != genre infere depuis les chemins du diff (#10020, advisory) variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) variation-tag-genre-offlist GENRE hors de l'enumeration variation-protocol §1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants