Content
60%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.
A well-structured, clearly sectioned governance skill with a real fail-fast validation loop and a genuine (verified) reference bundle. Its weaknesses are abstractness — no named commands, canonical paths, or code anywhere in the body — plus meta-filler sections and safety boilerplate that dilute the token budget, and several cited paths that don't resolve within the bundle.
Suggestions
Make the Workflow and Validation sections executable: name the actual repo design-system validation commands and the canonical token/alias file paths (e.g. the concrete token layer files to load), or point to where in references/ those commands live, instead of "Run the repo-specific design-system checks or nearest available validation".
Delete or relocate the meta content: the Philosophy section (skill-design commentary), generic safety boilerplate in Constraints, and the Inputs/Outputs lists that duplicate references/contract.yaml — keeping one canonical copy in the reference file.
Fix the dangling references: either move the Infrastructure/references lens-pack, expertise-map, and deferred-skill-context materials into this skill's references/ directory or drop the citations, so every path cited in Progressive Disclosure resolves from the skill bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is bullet-based and free of concept over-explanation, but contains unnecessary padding: the "Philosophy" section ("Keep the skill focused on the decision and workflow the user actually requested") is meta-commentary about skill design rather than design-system guidance, "Constraints" restates generic safety boilerplate (untrusted input, redaction, destructive commands) not specific to this skill, and Inputs/Outputs duplicate the same lists already machine-encoded in references/contract.yaml. Above anchor 2 (no heavy padding or tutorial prose), below anchor 4 (several sections could be trimmed or removed outright). | 3 / 5 |
Actionability | There is some concrete conceptual guidance — the layer taxonomy "brand, alias, mapped, component, or documentation layers", the gotcha that screenshots don't prove compatibility, and worked example prompts — but nothing executable: no validation commands, no canonical file paths, no code. "Run the repo-specific design-system checks or nearest available validation" never names the checks, and "Run the smallest command or test that exercises the changed behavior" gives no command. This matches anchor 3 (some concrete guidance but incomplete, missing key executable details) rather than anchor 2, since the layer taxonomy and examples do direct real decisions. | 3 / 5 |
Workflow Clarity | The Workflow section gives an ordered approach (scope small, load smallest canonical surface, classify the layer, prefer existing tokens, explain impact, run checks), and the Validation section supplies an explicit fail-fast feedback loop ("stop at first failed gate; do not proceed until it is fixed and rerun") reinforced by a Failure Mode blocker rule. This is not a missing-validation case, so the cap at 3 does not apply. Below anchor 5 because checkpoints are decoupled from the workflow steps (a separate section, not inline), the steps mix policies with sequence, and no concrete command is attached to any checkpoint. | 4 / 5 |
Progressive Disclosure | Scored against the actual bundle: the three local references cited (references/contract.yaml, references/evals.yaml, references/task-profile.json) all exist, are one level deep, and each is signaled with its purpose in a dedicated Progressive Disclosure section — matching anchor 4's "references mostly clear". Held below anchor 5 because three of the six cited paths (Infrastructure/references/software-literature-expert-lens-pack.md, Infrastructure/references/software-literature-skill-expertise-map.md, and Infrastructure/references/deferred-skill-context/frontend-ui-design-system/) do not exist in the bundle and are unresolvable from this skill, and Inputs/Outputs duplicate content already in contract.yaml. | 4 / 5 |
Total | 14 / 20 Passed |