Content
60%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 organized as a prioritized rule index with good executable examples for the most critical patterns, but it is undermined by references to rule files and a compiled document that are not present in the bundle, plus mild padding in the Ecosystem Coverage section. The progressive-disclosure claim ('read individual rule files') cannot be acted on as shipped.
Suggestions
Include the referenced rules/ directory in the bundle (or remove the 'How to Use' file references) so the 'read individual rule files' instruction is actionable — currently no rule files exist.
Delete the 'Ecosystem Coverage' section; the package descriptions ('zustand - State management', 'leva - Debug GUI') restate knowledge Claude already has, and the packages are already listed in the description and rule categories.
Fix the 'Full Compiled Document' path ('../R3F_BEST_PRACTICES.md' points outside the skill) and correct the '12 categories' claim, which mismatches the 11 rows in the priority table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The rule index and code examples are tight, but there is unnecessary padding: the 'Ecosystem Coverage' section explains what zustand/leva/postprocessing are (concepts Claude already knows), and the intro line repeats the category count. Not score 2 because the bulk of the body is a lean one-line-per-rule index; not score 4 because the Ecosystem Coverage section and category-table/heading redundancy could be trimmed. | 3 / 5 |
Actionability | The 'Critical Patterns' section gives complete, executable JSX examples (useFrame ref mutation, zustand selectors, useGLTF with preload, Suspense, Perf, visibility toggling) with BAD/GOOD contrast. Not score 5 because the remaining ~70 rules are one-line labels with no code or specifics inline, and the actual detail lives in files that are referenced but not present in the bundle. | 4 / 5 |
Workflow Clarity | As a reference skill its usage is unambiguous: a priority table routes the reader to prefixed rule IDs, and 'How to Use' shows example rule-file paths. Not score 5 because the 'How to Use' section only lists three example paths without stating how to locate a rule for a given task, and the compiled-document path ('../R3F_BEST_PRACTICES.md') introduces ambiguity about where the full content actually lives. | 4 / 5 |
Progressive Disclosure | The body references 'rules/perf-never-set-state-in-useframe.md' and a compiled '../R3F_BEST_PRACTICES.md', but the bundle contains no rules/ directory (nor references/, scripts/, or assets/) — the referenced detail files do not exist, so the progressive-disclosure structure is broken in practice, and the full document path points outside the skill. Not score 3 because the section structure itself is good but the referenced files are simply absent, which is worse than 'not clearly signaled'. | 2 / 5 |
Total | 13 / 20 Passed |