Repository navigation
Move the comma-shape and encoding sweeps out of root AGENTS.md - #910
Merged
Merged
Conversation
…r/AGENTS.md The comma sweep (879 words) joins the spelling sweep #909 moved, so the sweep sequence the Workflow paragraph names (comma shapes, then name_order, then spellings) lives in one file again, which loads for vocabulary and pipeline work alike. The text moves unchanged in substance under a subheading per lesson; three openings that leaned on the sentence before them now stand alone. Root keeps the four comma paths and a pointer. Every pointer to the sweep is retargeted: seven cases.py notes, test_parser.py, test_properties.py, decisions/A1 and H5, and the vocabulary-model RFC; plus the two decisions pointers to the spelling sweep that #909 missed (C13, S2). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The combining-mark paragraph (469 words) is the last of the four sweeps and joins the other three in nameparser/AGENTS.md, unchanged in substance under a subheading per lesson (the #652 families, widths and the initial shape, composing the whole predicate, what is deliberately uncovered, the guards). Root keeps the blind spot itself and a pointer; the Workflow paragraph names the encoding as the fourth sweep. Nothing outside AGENTS.md pointed at the paragraph. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Root's comma stub kept "every row runs under all three orders" and dropped the warning beside it; it now says test_cases.py asserts only R2's partition there, so a change must parse each comma shape AND each order itself (the vacuous-coverage trap #394 found). - nameparser/AGENTS.md: the #614 kinds-not-fields gap moves next to the "A report's field" check it motivates, and the opening no longer calls it a shape. - nameparser/AGENTS.md intro: the vocabulary-only reason fits the spelling sweep, not the encoding sweep, whose vocabulary path NFC-folds; say so. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
3 tasks done
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.
This continues #909. The comma-shape and
name_ordersweep (879 words) and the encoding sweep (469 words) move from root AGENTS.md intonameparser/AGENTS.md. All four sweeps the Workflow paragraph names now sit in one file: comma shapes, thenname_order, then spellings, then the encoding. That file loads for pipeline and vocabulary work alike. Root goes from about 21,800 to about 20,700 words; it was 26,824 before #907.Commits
cases.pynotes,test_parser.py,test_properties.py;How it was checked
Test plan
uv run pytest: 31,706 passed.uv run ruff check🤖 Generated with Claude Code