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.
The body is efficient and highly actionable with a clearly sequenced two-phase workflow and concrete validation commands, but its single progressive-disclosure reference is broken: TEMPLATES.md does not exist, leaving the prompt-template guidance unresolvable. Fixing or inlining that reference plus adding error-recovery steps for failed checks would lift the weakest dimensions.
Suggestions
Create the missing TEMPLATES.md (containing the actual page-agent and foundation prompt templates) or inline the essential template skeleton into SKILL.md and delete the dead link.
Make the foundation checkpoint executable: replace 'Layout renders, UI components compile' with concrete commands (e.g., a build/typecheck command) so agents can verify rather than assume.
Add an error-recovery loop for the per-page check: state what to do when 'grep -rF "style={{" src/pages/' returns hits or imports fail to resolve (fix by replacing inline styles with token-based classes, then re-run until 0 hits).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean: rules delivered via tables, an executable grep check, and no explanation of concepts Claude already knows (no CSS-variable tutorials, no library comparisons). Not 5 because the 22-line tokens.css example mostly demonstrates a naming convention Claude could infer from a 5-line sample, and the 'Convoy Integration' fenced block restates its five items in prose; not 3 because there is no padded or over-explained section. | 4 / 5 |
Actionability | Concrete, usable guidance throughout: exact paths ('src/styles/tokens.css', 'src/components/Layout.tsx'), an executable check ('grep -rF "style={{" src/pages/'), naming rules (PascalCase components, camelCase props), and a copy-paste-ready tokens.css with the var-only usage pattern 'bg-[var(--color-primary)]'. Not 5 because the foundation checkpoint's 'Layout renders, UI components compile' gives no command or procedure to actually verify those, and the convoy section defers its substance to a file that isn't there. | 4 / 5 |
Workflow Clarity | The two-phase sequence is explicit — 'Phase 1 (sequential): foundation-setup creates tokens, Layout, UI library, style guide brief. Phase 2 (parallel): every page task imports from Phase 1' — with two numbered validation checkpoints including a zero-hit grep gate. Not 5 because there is no error-recovery feedback loop telling an agent what to do when the per-page grep returns hits or imports fail to resolve; not 3 because checkpoints are explicit and concrete, not merely implied. | 4 / 5 |
Progressive Disclosure | Sections are well-organized, but the body's only external reference — 'Prompt templates: see [TEMPLATES.md](./TEMPLATES.md)' — points to a file that does not exist anywhere in the skill directory, and no references/, scripts/, or assets/ bundle is present. Navigation to the prompt templates (the operational core of the Convoy Integration section) is a dead end, matching the anchor where referenced content is effectively unreachable. Not 3 because a clearly signaled-but-missing reference is worse than one that is merely 'not clearly signaled'; not 1 because the in-body content is genuinely sectioned and self-navigable. | 2 / 5 |
Total | 14 / 20 Passed |