Content
50%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 well-structured as an overview with excellent progressive disclosure to a single real reference file, but it reads as a template skeleton: Check/Fix/Explain sections state what to do without any concrete method, and the Quick Reference bullets duplicate the intro. Moving one or two concrete anchors (a valid/invalid role-attribute pair, a checker command) inline and cutting the redundant bullets would materially raise actionability and conciseness.
Suggestions
Inline one concrete good/bad example in the Check or Fix section (e.g., aria-checked is valid on role="checkbox" but invalid on role="heading") so the body is actionable without opening the reference.
Delete or merge the 'Quick Reference' bullets that restate the intro paragraph, particularly 'Using unsupported attributes can confuse assistive technologies' and 'Ensures the accessibility tree remains valid and meaningful'.
Add an explicit verification checkpoint in the body's workflow (e.g., 'Confirm the fix with the browser accessibility tree or an axe/Lighthouse run') instead of only 'note how to verify the fix'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and doesn't explain basics Claude already knows, but the 'Quick Reference' bullets duplicate the intro ('Using unsupported attributes can confuse assistive technologies' restates 'Invalid ARIA attributes are ignored by screen readers or may cause them to misinterpret') and 'Ensures the accessibility tree remains valid and meaningful' is filler. Not 4 because real redundancy exists that could be trimmed; not 2 because there is no padded multi-paragraph over-explanation. | 3 / 5 |
Actionability | The body states goals rather than executable steps: 'Verify that all ARIA attributes used on elements are valid for their assigned WAI-ARIA roles' and 'Remove or correct ARIA attributes that are not supported' give no method — no role/attribute pairing examples, no checker commands, no spec lookup guidance; the concrete material (code example, axe/Lighthouse verification) lives only in references/rule.md. Not 3 because even the body's core Check/Fix sections lack any concrete detail; not 1 because the sections do name the exact rule target and the reference pointer is explicit. | 2 / 5 |
Workflow Clarity | Check → Fix → Explain → Code Review provides a recognizable sequence, but validation checkpoints are only implicit ('note how to verify the fix with browser accessibility tooling') and the actual verification steps are delegated to the reference. Not 4 because no explicit verify-then-proceed checkpoint exists in the body; not 2 because the section ordering does define a coherent rough sequence for the review task. | 3 / 5 |
Progressive Disclosure | The body is a lean overview with well-organized sections and exactly one clearly signaled, one-level-deep reference ('For full implementation details, code examples, and framework-specific guidance, see references/rule.md'), which exists and appropriately holds the code example, exceptions, standards, and verification detail. Not 4 because there are no organization gaps: the split is clean and navigation is trivial. | 5 / 5 |
Total | 13 / 20 Passed |