Content
65%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is well-structured with good progressive disclosure to a real reference file and concrete BEM-rename guidance, but it lacks a verification step for the batch class-rename operation and carries redundant content duplicated from both the reference file and the frontmatter description.
Suggestions
Add a verification step to the Fix workflow, e.g. "After renaming, search the codebase for the old class names to catch orphaned selectors/JS hooks and confirm the rendered layout is unchanged across breakpoints."
Remove or trim the opening "why it matters" paragraph, which is duplicated verbatim in references/rule.md's "Why It Matters" section.
Replace the Code Review section, which repeats the description almost word-for-word, with distinct review-level guidance (e.g., what specific naming violations to flag and how to report them).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is brief and sectioned, but the opening "Without a naming convention, class names become a guessing game..." paragraph is motivational explanation duplicated verbatim in references/rule.md's "Why It Matters", and the Code Review section restates the frontmatter description almost word-for-word. | 3 / 5 |
Actionability | Concrete, specific guidance for an instruction-only skill: "Rename classes to follow BEM methodology: Block, Block__Element, Block--Modifier pattern", kebab-case enforcement, and concrete audit questions in Check. Minor gap: no before/after rename example inline (examples live in the reference file). | 4 / 5 |
Workflow Clarity | The Check → Fix section order implies a rough sequence, but the steps are not explicitly ordered and there is no verification step after renaming (e.g., searching for orphaned old class names or confirming the rendered layout still matches). Per the rubric, a batch operation like renaming classes across a stylesheet without validation is capped at 3. | 3 / 5 |
Progressive Disclosure | The body is a short, well-sectioned overview (~45 lines) with a clearly signaled, one-level-deep reference that exists on disk ("see `references/rule.md`" for implementation details and code examples), and the 132-line rule.md appropriately holds the bulk material. | 5 / 5 |
Total | 15 / 20 Passed |