Repository navigation
fix(hierarchy,#16633): canonical git-aware walk for scan_md_hierarchy (H1-DEEP emitter) - #16722
Merged
Merged
Conversation
… notebook_walk
The local p.rglob('*.ipynb') counted untracked run artifacts
(*_executed.ipynb) present in lane worktrees: the +1 H1-DEEP drift of
#16633 targets an untracked Lean-14c Golly artifact — an EMITTER defect,
not a .gitignore defect (c.16633-1). The canonical walker (#8650) is
tracked_only via git ls-files + SKIP_DIRS + papermill artifact filter;
out-of-repo invocations (tmp fixtures, tarballs) degrade to the previous
disk scan with the documented stderr warning, so fixture-based tests are
unchanged. Committed baseline audited: 4 keys, zero artifact keys — no
drift-noise possible from the walk change. Two integration tests: tmp git
repo (tracked source + untracked _executed twin -> only source yielded)
and out-of-repo degradation (both yielded, old behavior pinned).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Grain: LIGHT/guards -- lane myia-po-2026:CoursIA -- prev: LIGHT/deaccent #16659
Defect (emetteur, pas .gitignore)
Le finding
+1 H1-DEEPde #16633 porte surLean-14c-Conway-Game-of-Life-Golly_executed.ipynb— un artefact de run untracked présent dans un worktree de lane.scan_md_hierarchy.pymarchait via son proprep.rglob('*.ipynb'): il comptait les fichiers ignorés/intrackables, exactement la dérive que le walker canonique (#8650) a été créé pour éliminer. Verbatim c.16633-1 : « un défaut de l'émetteur, pas un défaut du .gitignore ».Fix
Le walk local des répertoires est routé vers
notebook_walk.iter_notebooks(même pattern d'import quedetect_papermill_path_leak.py:322) :tracked_onlyviagit ls-files(drop des fichiers non trackés, règle*_executed.ipynby compris),SKIP_DIRScanonique, filtre artefacts papermill. Hors repo git (fixtures tmp, tarball) : dégradation à l'ancien scan disque avec le warning stderr documenté (#8650) — comportement pré-fixe préservé hors repo, pas changé silencieusement.Mesures
md_hierarchy_baseline.json= 4 clés, 0 clé artefact → aucun drift-noise possible depuis le changement de walk (le gate drift ne lit que les clés des notebooks changés par la PR — un artefact untracked n'y figure jamais)._executed.ipynbnon commitée → seule la source est rendue (fondateur reproduit et éteint) ; (2) hors repo git — les deux rendus (dégradation pinnée).Tranche de #16633 (artefact vs livrable)
ARTEFACT — copie
_executedd'un run (convention #16657). La règle nommée.gitignorearrive via #16657 (OPEN, fichiers distincts : cette PR ne touche que l'organe + ses tests). Additif et indépendant, per c.16633-1 : les deux gestes ferment l'issue ensemble.See #16633 (fermeture complète quand #16657 est mergé — règle nommée + émetteur conscient)
🤖 Generated with Claude Code