Content
90%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 dense, highly actionable review checklist: every bullet names a concrete detection pattern tied to a numbered DESIGN.md rule, with explicit severity mapping, scope exclusions, and report format. Its only real weakness is that the workflow (read DESIGN.md → scope the diff → check categories → report) is implied by section order rather than stated as an explicit sequence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is telegraphic, checklist-style bullets with zero padding and no explanation of concepts Claude already knows — every line encodes repo-specific detection heuristics ("`border` + `box-shadow` on the same lifted surface", "durations > 300ms on UI controls", "compare document scrollWidth to measured clientWidth"). This clearly matches the lean/efficient anchor; not 4, as there are no over-explanations to trim. | 5 / 5 |
Actionability | For an instruction-only skill the guidance is fully executable: exact paths to grep (`apps/app/src/components/ui/`), exact code patterns (`text-gray-400`, `#000`, `outline: none`, `CardDescription` under `CardTitle`), concrete thresholds (320px viewport, 305px usable, >300ms), an explicit severity mapping (medium/low, never high), and a defined report format. It is not 4 — there are no minor gaps in the concrete guidance. | 5 / 5 |
Workflow Clarity | The sequence is discernible and stated where it matters ("Read `DESIGN.md` at the repository root first", then scope changed lines, check the category checklists, then report per the format section) with inline checkpoints ("Grep ... before reporting", scope exclusions, zero-findings-is-valid). It is not 5 because the workflow is never laid out as an explicit ordered sequence with validation checkpoints, leaving the overall flow implicit. | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the single file is well-sectioned (What to check / Do NOT report / Report format) with its one external reference (DESIGN.md) clearly signaled as the source of truth. It is not 5 because the ~123-line rule catalog is entirely inlined with no split of any kind, and not 3 because the structure and navigation are genuinely good. | 4 / 5 |
Total | 18 / 20 Passed |