Content
47%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 maximally lean but is a pure delegation stub: its primary directive points at this skill's own path, creating a circular reference, and every substantive instruction lives in files absent from the bundle. A reader of this skill cannot execute, follow, or navigate to any actual product-UI guidance.
Suggestions
Replace the circular redirect ('Read .claude/skills/product-ui-taste/SKILL.md and execute it exactly as written' — that is this file's own path) with an inlined concise overview of the product-UI method: budget the frame, surface checklist, and design-system mapping.
Ship the referenced files (the playbook, .agents/rules/cog.md, worker definitions) inside the bundle's references/ so the delegation chain is navigable and only one level deep, or drop the delegation and inline the essentials.
Add validation checkpoints (e.g., verify overflow, long labels, empty/error/loading states, and keyboard/a11y behavior before declaring an interface done) so the workflow has explicit feedback loops.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The five-line body wastes nothing: no explanation of concepts Claude already knows, no padding — every line is a directive ('Read ... and execute it exactly as written', 'Then follow .agents/rules/cog.md') or a concrete conditional substitution rule. Lean and efficient throughout. | 5 / 5 |
Actionability | Concrete file paths are given, but the body contains none of the actual executable guidance for the skill's stated purpose; the single directive delegates to '.claude/skills/product-ui-taste/SKILL.md', which is this very skill's own path (circular), and none of the referenced files ('.agents/rules/cog.md', '.agents/agents/<name>.md') exist in the bundle. High-level redirect with the specific steps to execute missing. Not 1 because it does instruct with explicit paths rather than merely describe. | 2 / 5 |
Workflow Clarity | A rough sequence is present (read playbook → follow cog.md, plus a conditional substitution rule), but the primary step is incoherent as written — its target is the skill's own file — and no validation checkpoints exist anywhere in the body. Not 3 because the sequence cannot actually be followed from what is written: the playbook it depends on is either absent from the bundle or the file itself. | 2 / 5 |
Progressive Disclosure | References are explicitly signaled but the body is a bare redirect with no overview content of its own, and the delegation chain extends two-plus levels (this SKILL.md → the playbook → '.claude/agents/<name>' workers → '.agents/agents/<name>.md'), with none of the referenced files present in the bundle, making navigation impossible. Not 1 because the body itself is short, organized into two clearly labeled parts, and its references are not buried; not 3 because the skill provides no overview to organize and the chain dead-ends at absent files. | 2 / 5 |
Total | 11 / 20 Passed |