Content
71%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 well-structured, token-efficient overview with excellent progressive disclosure — a single real reference file holds the code examples and verification detail. Its main weakness is actionability: the Check step and verification tooling are described only at a general level in the body, and the how-to specifics live entirely in the reference, so the body alone leaves minor gaps in what to concretely execute.
Suggestions
Make the Check step executable in the body: state how to detect a missing accessible name (inspect the rendered accessibility tree or run axe, and flag controls whose only announcement is the role).
Inline one minimal good/bad markup example (e.g., an icon-only <button> with vs. without aria-label) so the rule is verifiable without opening the reference.
Cut the redundancy between Quick Reference and Fix, and drop the opening sentence explaining why unlabeled buttons are bad — Claude already knows this.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean (~35 lines) with short, purposeful sections and no padding. Minor trimmable instances remain: the opening sentence explains a concept Claude already knows ("screen reader users will only hear the role (e.g., 'button') without knowing what it does"), and the Quick Reference bullet "Accessible names can be provided via text content, aria-label, or aria-labelledby" duplicates the Fix section almost verbatim. Not a 5 because of these two over-explanations. | 4 / 5 |
Actionability | Concrete guidance exists — the Fix names specific mechanisms ("inner text, aria-label, or aria-labelledby") and Check/Fix/Explain/Code Review give a usable shape — but key executable details are deferred rather than included: the Check section says to find elements that "lack a clear accessible name" without saying how to inspect the rendered result (the accessibility tree, axe, or similar are only in references/rule.md), and no markup example appears in the body. This matches "some concrete guidance but incomplete; missing key details." | 3 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections form a clear, coherent sequence for a single-purpose review skill, and the Code Review section does mention verification ("note how to verify the fix with browser accessibility tooling or assistive tech"). It is not a 5 because validation checkpoints are only loosely implied — no explicit verify step (e.g., "confirm the accessible name appears in the accessibility tree before considering the fix done") closes the loop. | 4 / 5 |
Progressive Disclosure | The body is a concise, well-sectioned overview that defers "full implementation details, code examples, and framework-specific guidance" to a single clearly signaled reference, references/rule.md, which actually exists and contains exactly that (code examples, why-it-matters, exceptions, standards, verification steps). The reference is one level deep with no further nesting, matching the anchor for a clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 16 / 20 Passed |