fix: profile README says 8 rules; 9 ship, and the id range is not contiguous - #1
Open
HetCreep wants to merge 1 commit into
Open
fix: profile README says 8 rules; 9 ship, and the id range is not contiguous#1HetCreep wants to merge 1 commit into
HetCreep wants to merge 1 commit into
Conversation
Nine rules ship, not eight, and MOV-001-012 reads as twelve contiguous ids when MOV-007, MOV-009 and MOV-010 do not exist. Derived from move-test-gen@main's rules directory: nine .mjs files with RULE_IDs MOV-001 through MOV-006, MOV-008, MOV-011 and MOV-012 -- consistent with SKILL.md's "9 rules, 10 checks". Co-Authored-By: Claude Fable 5.1 <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.
profile/README.md:15currently reads:Two things are off: nine rules ship, not eight, and
MOV-001–012reads as twelve contiguous ids when MOV-007, MOV-009 and MOV-010 do not exist.Derived from the rules directory on
move-test-gen@main, not from either README:whose
RULE_IDs are MOV-001–006, MOV-008, MOV-011 and MOV-012. That matchesSKILL.md, which says "9 rules, 10 checks" — the tenth check is MOV-002's second pattern, the left shift that silently discards high bits, documented in the README's rule table.This drops the id range rather than replacing it with
MOV-001–006, 008, 011, 012: the range is the half that went stale, and a landing-page bullet carrying four id groups has to be re-synced every time a rule number lands. The ids stay documented in the README andSKILL.md, where they are already correct. One line changed, nothing else touched.🤖 Generated with Claude Code