Content
78%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 reference skill: an expert-level overview with executable anti-pattern examples, explicit scope delegation to sibling skills, and a clean one-level-deep reference layout that was verified against the actual bundle. The main improvements are merging the redundant trigger sections and weaving the type-check validation into a more explicit workflow.
Suggestions
Merge the overlapping trigger lists in 'When to Apply' and 'Use When' into a single section — the bullet list and the quoted questions cover the same triggers twice (conciseness).
Add a compact positive example of the most common operations (e.g., a working `infer` extraction or mapped type) directly in the body so the frequent cases are covered without opening a reference file (actionability).
Make `npx tsc --noEmit` an explicit verification checkpoint in the guidance flow rather than only a standalone Quick Command, e.g., 'after modeling the type, verify against real call sites with tsc --noEmit' (workflow_clarity).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes expertise — the Mindset section gives non-obvious judgment calls ("wrap the checked type in a tuple (`[T] extends [U]`) to opt out of distribution") rather than explaining what unions are. It misses the top anchor because "When to Apply" and "Use When" largely duplicate each other's trigger lists, which could be merged into one section. | 4 / 5 |
Actionability | Concrete, executable guidance is present: valid BAD/GOOD TypeScript pairs for all three anti-patterns (e.g., `type PropType<T, K extends keyof T> = ...`), the copy-paste command `npx tsc --noEmit`, and specific inline techniques like `ReturnType<typeof getUser>`. It is not the top anchor because the common positive cases (actually writing a conditional with `infer`, a mapped type, a template literal type) are delegated to the reference files rather than shown in the body, leaving minor gaps in executable coverage. | 4 / 5 |
Workflow Clarity | For a reference-style skill the guidance flow is clear — applicability triggers, explicit In/Out of Scope with sibling delegation, a "When NOT to Use" section, and a validation command (`npx tsc --noEmit`). It falls short of the top anchor because there is no explicit sequence tying modeling to verification (the tsc check appears only in Quick Commands and a warning, not as a checkpoint in a workflow), and short of anchor 3 because validation is present and nothing is ambiguous. | 4 / 5 |
Progressive Disclosure | The body is a genuine overview: mechanics are summarized in Mindset/Anti-Patterns, and depth is split across a References table mapping all 8 files to their coverage (each verified to exist, one level deep, e.g., `references/infer-extraction.md` covering "return types, parameters, array elements, promise values, tuple elements, and instance/`this` types"), plus three official Handbook links. This matches the clear-overview, well-signaled, one-level-deep anchor exactly. | 5 / 5 |
Total | 17 / 20 Passed |