Validator: recognize .md doc ids, suppress ignored amendment rows by default, and add canonical-code metadata to many charters#63
Merged
CAM-Initiative merged 5 commits intoMay 17, 2026
Conversation
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.
Motivation
.mdextensions and common punctuation/spacing variants to reduce false negatives.Annex [x]) as manual-review cases rather than blocking or silently matching.Description
validate_markdown_section_refs.pyto use aDOC_ID_WITH_EXT_REthat accepts optional.md, broadenDOC_BEFORE_SECTION_RE/CROSS_DOC_AFTER_RE/PHRASE_DOC_SECTION_REto handle punctuation/spacing, and ensure returned doc ids use the named capturedoc_id.manual_reviewwith a labelAnnex [x]when an explicit doc id plus an Annex placeholder is present.filter_display_findingsandmainto suppressIGNORED_STATUSESby default, add a--show-ignoredCLI flag to restore display, and print aSUPPRESSED_IGNORED_ROWSsummary when rows are hidden..github/scripts/tests/test_validate_markdown_section_refs.pyto reflect the suppressed-ignored behaviour, and add tests covering CAM doc id variants and annex/placeholder cases.Governance/*Markdown files, and increment versions/pinned_sha/updated timestamps inconstitution.index.jsonto reflect the content changes.Testing
.github/scripts/tests/test_validate_markdown_section_refs.pywithpytestand confirmed the test suite for the validator changes passes.pass_cross_document,manual_reviewvariants, and that ignored amendment-register rows are suppressed by default; all assertions passed.--show-ignoredrestores suppressed rows and the summary lineSUPPRESSED_IGNORED_ROWSis emitted when suppression occurs during test runs.Codex Task