The line editor holds "data" as plural, and the prose gate searches for it - #73
Open
syncytium2 wants to merge 1 commit into
Open
syncytium2 wants to merge 1 commit into
syncytium2 wants to merge 1 commit into
Conversation
…es for it Tony's house rule of 2026-09-14: "data" takes plural verbs and pronouns. It reached his global CLAUDE.md and bugarach (PR #572, sapper SAP015), but bugarach cannot put it in the line editor: its murderboard copy is vendored and re-copied. So it lands here, where every consumer picks it up on the next refresh. - doc_review_process.md role 5 states the rule, its exceptions (metadata, dataset, quoted words), and that it is a house convention a consumer may drop. - murderboard_prose.sh searches for the form "the data is": a determiner, then "data", then a singular verb. The determiner keeps it off `data is None`. The selftest proves it fires on "The data is noisy" and passes plural prose and code. - DRAFTING-PROMPT.md gives the writer the rule before the review does. - Three singular uses in this repo's own docs are fixed; agents regenerated. - Plugin 0.12.0 -> 0.13.0, because the payload changed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Tony's house rule of 2026-09-14 says "data" is plural: the data are, the data show, these data. It is already in his global CLAUDE.md and in bugarach (PR #572, sapper SAP015). bugarach cannot add it to the line editor, because its murderboard copy is vendored and gets re-copied rather than edited. So the rule goes into murderboard, and every repo that vendors murderboard picks it up on its next refresh.
What changes
doc_review_process.md) states the rule and its exceptions. Metadata and dataset keep their own grammar, and a quotation keeps the speaker's grammar. Like the banned list, it is marked as a house convention that a consuming project may drop.murderboard_prose.shgains the formthe data is: a determiner, then data, then a singular verb. It reuses SAP015's approach. Requiring a determiner keeps it off code such asdata is None. Two new selftest cases check it both ways: "The data is noisy" trips it, while plural prose andif data is Nonepass.tests/drafting_prompt_test.pyconfirms that role 5 names the new form.DRAFTING-PROMPT.mdgains the rule as rule 8, so writers see it before the review does.doc_review_process.mdand one inmetrics/README.md. All three are fixed, and the agents are regenerated (roles 5 and 8).Known gap
A match split across a line break is missed, because grep reads one line at a time. Role 5's text says so and tells the reviewer to read for it.
Checked locally
These all pass: the roster, freshness, commit-before-message, prompt, prose and model-gate selftests, the no-heredoc hook test, the revendor selftest, the version gate against
origin/main, and the agents-generated, drafting-prompt, published-page and plugin-manifest tests.🤖 Generated with Claude Code