Content
67%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.
A well-structured index-style SKILL.md that practices good progressive disclosure and stays lean, with clean prioritization of rules. Its weakness is actionability: the body defers all executable detail (code examples) to rule files, leaving only abstract one-liners in the main file, and its trigger/when sections duplicate the frontmatter description.
Suggestions
Inline one minimal incorrect/correct code pair for the highest-priority rule (architecture-avoid-boolean-props) so the core pattern is actionable without opening a rule file.
Fix the react19-no-forwardref bullet: it conflates two separate changes — forwardRef is replaced by ref-as-a-prop, while use() replaces useContext() — and the current wording implies use() is the forwardRef replacement.
Dedupe the "When to Apply" section against the frontmatter description (or replace it with a pointer to the priority table) to save tokens and remove repetition.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean ~75-line index with no padding and no explanation of concepts Claude already knows. Not 5: the "When to Apply" list restates the frontmatter description's triggers, and the priority table's category names are repeated verbatim as Quick Reference headings, so some tokens don't earn their place. | 4 / 5 |
Actionability | Each rule carries a concrete do/don't directive ("Don't add boolean props to customize behavior; use composition", "Use children for composition instead of renderX props"), but the body itself contains no code examples despite the skill's purpose being code refactoring patterns, and "use composition" without an example is abstract. Key executable detail is entirely deferred to rule files, which is incomplete guidance in the body itself, matching the 'some concrete guidance but incomplete' anchor rather than 'mostly executable'. | 3 / 5 |
Workflow Clarity | Navigation flow is unambiguous for a simple index skill: consult rules by category, "Read individual rule files for detailed explanations and code examples", then "For the complete guide with all rules expanded: AGENTS.md". Not 5 because the How to Use section shows only two example paths and never states the lookup convention for the other five rule files. | 4 / 5 |
Progressive Disclosure | Textbook overview structure: priority table, one-line-per-rule quick reference, and clearly signaled one-level-deep references ("Read individual rule files...", "For the complete guide... AGENTS.md"). Not 5: the referenced paths (rules/*.md, AGENTS.md) are not present in the provided bundle, so the reference targets cannot be confirmed to exist. | 4 / 5 |
Total | 15 / 20 Passed |