Content
77%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-sequenced, highly actionable review procedure with genuine error-recovery handling for missing inputs — a strong instruction-only skill. Its two weaknesses are the monolithic structure (three per-document-type checklists inlined where only one is ever needed) and a trimmable layer of rationale prose plus one ambiguous verdict-selection rule for mixed findings.
Suggestions
Split Phase 3A/3B/3C checklists into separate reference files (e.g., references/ux-spec-checklist.md, hud-checklist.md, pattern-library-checklist.md) loaded only for the matching document type, cutting per-invocation context by roughly half.
Trim the rationale blockquotes and the NOT ASSESSED ranking justification to bare rules — state what to report and when, without defending why the rule exists.
Add an explicit rule for the mixed case, e.g.: 'If any blocking issue exists, the verdict is NEEDS REVISION or higher regardless of unassessed dimensions; if no issues are found but dimensions are unassessed, the verdict is NOT ASSESSED.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense operational content — checklists, exact file paths, fallback chains — with no teaching of concepts Claude already knows. However, ~25–30 lines are justificatory prose (the NOT ASSESSED ranking rationale, the accessibility/pattern-library blockquotes explaining why rules exist, and meta-commentary like 'The same rule lives in /team-ui Phase 4 and is mirrored here so the two cannot drift') that could be trimmed to bare rules. Not 5 due to those trimmable passages; not 3 because the padding is minor and everything else earns its tokens. | 4 / 5 |
Actionability | For an instruction-only skill this is highly executable: exact paths (design/ux/hud.md, design/accessibility-requirements.md, project.yaml platform keys with derivation rules), a complete copy-paste output template covering all four verdict branches, and per-argument handling. The one minor gap is verdict selection under mixed findings — the NOT ASSESSED paragraph ranks severity but never states which verdict to emit when blocking issues and unassessed dimensions co-occur. Not 5 because of that ambiguity; not 3 because the rest is fully concrete. | 4 / 5 |
Workflow Clarity | Phases 1–5 are clearly sequenced with explicit error recovery for missing inputs at every step: unreadable spec → NOT ASSESSED, missing platform block → technical-preferences.md fallback, absent accessibility tier → a named NOT ASSESSED report line with a recommendation. The checklists themselves provide the checkpoint structure the top anchor requires, and the skill is read-only so no destructive-operation cap applies. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the ~330-line body inlines three full checklists (3A/3B/3C) of which only one runs per invocation — the other two (~90 lines) are dead context weight per run. This matches 'content that should be separate is inline' despite good section headers making it navigable; not 4 because nothing is split out, not 2 because headers keep it easy to navigate. Splitting each Phase 3 checklist into a one-level-deep reference file loaded only for the matching document type would materially reduce per-invocation token cost. | 3 / 5 |
Total | 16 / 20 Passed |