Content
66%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 delivers an exceptionally actionable, well-sequenced pipeline with genuine validation feedback loops (preflight, dial range checks, contrast checker exit codes, adversarial critique). It is undermined by severe padding — enforcement-theater repetition and a 130-line inline persistence script — and by progressive-disclosure failures: referenced files that are not in the bundle and scripts inlined instead of split out.
Suggestions
Move the ~130-line persistence bash block into scripts/persist-design.sh and invoke it with arguments, cutting the body by roughly a quarter.
Collapse the repeated MANDATORY/HARD-GATE/You-MUST framing into a single execution-contract note at the top; the steps themselves already encode the sequence.
Ship the referenced files (skills/blocks/design-taste.md, codex-host-adapter.md) inside the bundle or inline their essential checklists, so no instruction depends on a file the skill does not carry.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 596-line body is noticeably verbose: repeated enforcement shouting ('EXECUTION CONTRACT (MANDATORY - CANNOT SKIP)', 'You MUST', 'MANDATORY', 'Do NOT skip it' appear dozens of times), a 130-line fully-inlined bash persistence script with line-by-line failure handling, and a full emoji banner template — padded sections that assume Claude will disobey rather than leverage its competence. It is above anchor 1 (the content is operational, not explaining concepts Claude knows), but well below anchor 3's 'mostly efficient'. | 2 / 5 |
Actionability | Nearly everything is copy-paste executable: the BM25 search invocations ('python3 "$SEARCH_PY" ... --domain product'), the preflight check with READY/MISSING states, the WCAG contrast checker ('contrast-check.py \"<text-hex>:<bg-hex>\"'), the provider-availability bash loop, a complete AskUserQuestion JSON template, and a fully-formed atomic persistence script including lock acquisition and cleanup traps. Anchor 5's 'fully executable; copy-paste ready' fits; only intentional placeholders like '<user\'s product description>' remain, which is appropriate. | 5 / 5 |
Workflow Clarity | Steps 1-8 are clearly sequenced with explicit validation checkpoints and feedback loops: preflight 'Only continue to Step 4 when preflight returns READY'; dial values validated before every search call ('stop and obtain a valid value rather than clamp it'); 'Exit 1 means at least one pair fails WCAG AA — fix the palette and re-run before delivering'; and the critique step institutionalizes a revise loop ('Fix must-fix items... Show the diff'). This matches anchor 5's 'explicit validation steps; feedback loops for error recovery'. | 5 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ directories exist, yet the body repeatedly points to files outside the skill ('skills/blocks/design-taste.md', 'skills/blocks/codex-host-adapter.md', 'skill-design-lineage') that are not in the bundle, and inlines ~130 lines of persistence bash that clearly belongs in a scripts/ file. This matches anchor 2's 'content that clearly belongs in separate files is inlined' — there is real section structure, but the dangling references and monolithic bulk keep it below anchor 3. | 2 / 5 |
Total | 14 / 20 Passed |