Content
75%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 tightly written constraint/review skill: lean checklist rules, concrete attribute-level directives, and copy-paste before/after fixes, with clear invocation modes and prioritized review order. Remaining gaps are small — some directional rules lack the specific attributes they demand, no dialog focus-trap example, and no post-fix verification step.
Suggestions
For "if a role is used, required aria attributes must be present", name the pairs (e.g., role=menu → aria-orientation/menuitem children) or add a micro-table so the rule is executable without external lookup.
Add a fourth common-fix example showing focus trapping and restoration for dialogs, since that is a critical-priority category with no code illustration.
Add a brief verification step to the review workflow (e.g., re-read the fixed snippet to confirm the name/keyboard issue is resolved and nothing else regressed) to close the feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean, checklist-style rule set with no explanations of what accessibility is or why it matters beyond one short sentence per rule — it assumes Claude's competence throughout. Minor trimmable redundancy exists: the priority table's "impact" column duplicates the priority labels already shown in each section heading ("### 1. accessible names (critical)"), keeping it just below the every-token-earns-its-place 5 anchor. | 4 / 5 |
Actionability | Rules are concrete directives with exact attribute names ("aria-describedby", "aria-invalid", "do not use tabindex greater than 0") and the "common fixes" section provides copy-paste-ready before/after HTML for three common cases. A few rules remain directional rather than executable — e.g., "if a role is used, required aria attributes must be present" never says which attributes, and there is no example for focus trapping in dialogs — so it sits below fully-executable 5. | 4 / 5 |
Workflow Clarity | The invocation modes are explicit (apply constraints conversation-wide, or per-file review with a defined report format: quote snippet, why it matters, concrete fix), and "review guidance" sequences the work (fix critical categories first, prefer native elements, use established primitives for complex widgets). This is a review skill, not a destructive/batch operation, so no validation cap applies; the gap keeping it below 5 is the absence of any post-fix verification step (e.g., re-checking the fixed snippet or confirming no regression). | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent) and the body references no external paths, so scoring is against the single-file structure. Organization is good: numbered sections, a priority index table, a quick-reference hierarchy, and a common-fixes code block. The 5 anchor's well-signaled one-level-deep reference structure does not apply and the skill exceeds the under-50-lines simple-skill exception at ~137 lines, so this lands at 4 — well-structured, with the rule catalog being the kind of content that could optionally split into a reference file. | 4 / 5 |
Total | 16 / 20 Passed |