Content
88%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 strong, expert-level skill body: executable code throughout, hard-won failure modes stated precisely, and a real multi-step reference workflow with verification. The only flaws are minor redundancy around `default="none"` and a duplicated wrapper example, plus a dead pointer to a nonexistent AGENTS.md compiled document.
Suggestions
Consolidate the `default="none"` guidance into one section (e.g. 'Use default="none" Deliberately') and reference it from the Props and shared-element sections instead of restating the semantics three times.
Drop the duplicated `DirectionalTransition` wrapper code from the body (keep the one-sentence advice to extract a reusable wrapper) since the full component already lives in references/implementation.md.
Remove or generate the 'Full Compiled Document: AGENTS.md' pointer — the file does not exist in the bundle, so following it dead-ends; alternatively ship it in the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious expert knowledge (placement rules, trigger semantics, router.back() type loss, nested-VT limitations) and explains no basics, but a few things could be trimmed: the implications of `default="none"` are spelled out in three separate places (Props, 'Use default="none" Deliberately', and the shared-element bullets), the `DirectionalTransition` wrapper example is duplicated in full between the body and references/implementation.md, and time-sensitive version specifics (React 19.3, Chromium 125+, Firefox 144+, Safari 18.2+) sit inline rather than in a deprecation/old-patterns section. These are minor trimmable instances matching the 4 anchor rather than the 5 anchor's 'every token earns its place'. | 4 / 5 |
Actionability | Every section gives copy-paste-ready JSX with real prop values and imports — the works/broken placement-rule contrast, the type-map objects with the TypeScript `default`-key requirement, the nested list-identity/shared-element composition, and the Suspense fallback/content patterns. CSS class names are concretely backed by the recipes file, and failure modes come with specific fixes ('make the name conditional (via a prop) or move the named VT out of the shared component'). This fully matches the 5 anchor's executable, common-case-covering guidance. | 5 / 5 |
Workflow Clarity | The body sequences work explicitly (the priority table with 'Implement all applicable patterns from this list, in this order') and mandates 'follow references/implementation.md step by step. Start with the audit — do not skip it', which resolves to a real 7-step reference workflow ending in a per-navigation-path verification checklist and a symptom-driven troubleshooting guide as the error-recovery loop. Clear sequence, explicit validation, and a checklist for a complex process match the 5 anchor; the 4 anchor's 'minor validation gaps' does not apply since verification is a dedicated final step. | 5 / 5 |
Progressive Disclosure | The split is well done — overview and core concepts in SKILL.md, with bulk CSS, Next.js details, patterns, and debugging each in a clearly signaled one-level-deep reference file, all five of which exist with resolving anchors — but the 'Full Compiled Document' section points to `AGENTS.md`, which is not present in the bundle, creating a dead navigation path. That is a minor organization gap consistent with the 4 anchor rather than the 5 anchor's unqualified 'easy navigation'. | 4 / 5 |
Total | 18 / 20 Passed |