Repository navigation
Conversation
…d one The loader reads every rules/mov-*.mjs, and a written count has gone stale each time a rule was added (the README said "all nine" with ten rule files present). README, SKILL.md and action.yml now say "every rule" instead of a number, and the selftest README no longer claims case 11 covers all rules. package.json is left to the maintainer. Co-Authored-By: Claude Sonnet 5 <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
1 participant
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.
Four places state how many lint rules there are, and all are stale.
rules/holds ten files (mov-001to006,008,011,012,013), and two of the descriptions also leave out MOV-013.lint.mjsloads everyrules/mov-*.mjsat run time, so a hand-written count can only go stale. It has: it was synced to nine once, then MOV-013 landed. This drops the numbers rather than bumping them to ten.README.mdinstall tree: "(all nine, needed for --lint)" becomes "(every rule file, needed for --lint)".SKILL.mddescription: "9 rules, 10 checks:" becomes "every rule in rules/, e.g.", so the listed rules read as examples.action.ymllintinput: the "(9 rules: …)" list becomes "(every rule in rules/)".eval/gate-selftest/README.mdcase 11: "All 9 lint rules" becomes "Nine of the lint rules", noting MOV-013 is pinned by case 23. Case 11 imports exactly those nine.Left alone:
package.jsonline 4 says "10 lint rules (MOV-001–013)". It is true today and it is your line, so I didn't touch it, but it will drift at the next rule.🤖 Generated with Claude Code