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 lean, code-first pattern catalog with clear ✅/❌ contrasts that make each rule immediately applicable, and no bundle to mismanage. Its weaknesses are inlined reference material (utility types, keywords) that pads tokens without adding instruction Claude doesn't already have, and a few incomplete example skeletons. Structure and organization are solid throughout.
Suggestions
Move the utility-types list (Pick/Omit/Partial/etc.) and the 'Keywords' section into a references/ file (e.g. references/utility-types.md) — Claude already knows these built-ins, so they cost tokens without teaching anything.
Make examples fully executable: give `parse` a real body, and show `isUser` where it is first used or note it is defined in the Type Guards section below.
Apply the REQUIRED marker consistently (or drop it entirely) so it is unambiguous whether 'Never Use any', 'Import Types', and 'Utility Types' are mandates or optional conventions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — each pattern is a tight ✅/❌ code pair with a one-line rationale — but it includes content Claude already knows, such as the ten-entry utility-types reference (Pick, Omit, Partial, Record, ReturnType, etc.) and a trailing 'Keywords' section that adds no instruction. That matches 'mostly efficient but includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | Concrete, mostly executable TypeScript snippets with explicit ALWAYS/NEVER contrast and a working type guard, e.g. `const STATUS = {...} as const; type Status = (typeof STATUS)[keyof typeof STATUS];`. It falls short of 5 because a few examples are incomplete skeletons — `function parse(input: any): any { }` has an empty body, and `isUser` is used before its section defines it. | 4 / 5 |
Workflow Clarity | This is a single-purpose pattern-reference skill, not a multi-step process, so per the simple-skill note a 5 is available if everything is unambiguous; it nearly qualifies with clearly titled sections and REQUIRED markers. It sits at 4 because rules are inconsistently enforced (only two sections are marked REQUIRED) and 'Never Use any' duplicates a rule without a section-level marker, leaving minor ambiguity about which patterns are mandates vs. suggestions. | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the ~110-line body is cleanly sectioned per pattern with no nested references, so there is nothing impossible to navigate. It misses 5 because the skill exceeds the under-50-lines simple case: the utility-types lookup list and keyword list are reference material inlined in SKILL.md that would be better placed in a references/ file. | 4 / 5 |
Total | 15 / 20 Passed |