Content
71%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 a well-organized, lean index of 70 concrete, API-level performance rules with a sensible priority taxonomy and consistent prefix-based file naming. Its main defect is that it delegates all detail to rules/*.md files and AGENTS.md that are not present in the skill bundle, leaving the progressive-disclosure layer dangling, and the 'How to Use' section stops short of an explicit lookup sequence.
Suggestions
Ship the referenced bundle files: create the rules/ directory with the per-rule .md files and the AGENTS.md compiled document (or remove the references) — currently 'rules/async-parallel.md', 'rules/bundle-barrel-imports.md', and 'AGENTS.md' are dangling paths with no files in the skill directory.
Replace the two example paths in 'How to Use' with an explicit lookup sequence, e.g.: 1) identify the task type against 'When to Apply', 2) pick the highest-priority matching category from the priority table, 3) open rules/<rule-prefix>.md for the incorrect/correct code examples, 4) verify the fix preserves behavior.
Trim the duplicated priority/impact information — the Priority/Impact columns in the table are repeated verbatim in the section headers ('(CRITICAL)', '(MEDIUM-HIGH)', etc.); keep it in one place to save tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is essentially a functional index: 70 rule names each with a one-line actionable summary (e.g., 'Use Promise.all() for independent operations', 'Use next/dynamic for heavy components'), with no padding or explanation of concepts Claude already knows. It matches anchor 4 ('Efficient; minor instances... that could be trimmed'): the priority/impact information is duplicated between the 8-row priority table and the section headers ('### 1. Eliminating Waterfalls (CRITICAL)'), and 'When to Apply' partially restates the frontmatter description. Not a 5 because of that duplication; not a 3 because nearly every line is a usable lookup entry, not verbosity. | 4 / 5 |
Actionability | Each rule entry gives a concrete, API-level directive (e.g., 'Use React.cache() for per-request deduplication', 'Use LRU cache for cross-request caching', 'Use content-visibility for long lists', 'Use toSorted() for immutability'), and 'How to Use' gives explicit file paths to open for full code examples. This matches anchor 4 ('Mostly executable guidance; concrete code or commands with minor gaps'). Not a 5 because the body contains no code examples or commands itself — all executable detail is delegated to rule files; not a 3 because the one-liners are specific, actionable heuristics rather than vague direction. | 4 / 5 |
Workflow Clarity | The usage flow is clear and unambiguous for a reference skill: 'When to Apply' scopes the task, the priority table tells the model which category matters most, rule prefixes make file lookup deterministic, and 'Read individual rule files for detailed explanations and code examples' states the action. This matches anchor 4 ('Clear sequence with most checkpoints present; minor validation gaps'). Not a 5 because 'How to Use' only lists two example paths rather than an explicit step sequence (identify task -> pick category by priority -> open rules/<prefix>.md), and there is no guidance on what to do when multiple rules conflict; not a 3 because this is a non-destructive lookup skill where the single action (consult index, open rule file) is unambiguous. | 4 / 5 |
Progressive Disclosure | The design intent is right — a concise overview with one-level-deep, clearly signaled references ('Read individual rule files... rules/async-parallel.md', 'For the complete guide with all rules expanded: AGENTS.md'). However, scored against the actual bundle structure, none of the referenced files exist: there is no rules/ directory, no AGENTS.md, and no references/, scripts/, or assets/ directories at all. Every piece of detailed content is delegated to files that are not shipped, so navigation dead-ends. This matches anchor 3 ('Some structure but could be better organized; references present but not clearly signaled') in effect: the structure is present but the reference layer is broken. Not a 4/5 because well-signaled references only count when the referenced files are actually present in the bundle; not a 2 because the SKILL.md itself is well organized with clear sections and a consistent prefix-based naming convention. | 3 / 5 |
Total | 15 / 20 Passed |