Content
78%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 lean, well-structured body for a simple single-rule skill: an unambiguous check, a concrete fix, and a properly signaled one-level reference to references/rule.md that carries the code examples and verification detail. The weaknesses are minor — a vague "Explain" section, templated repetition of the rule title in the Code Review paragraph, and verification being implied rather than stated as an explicit step.
Suggestions
Replace the vague "Explain" directive with one or two concrete facts to convey (e.g. the whole page is removed from the accessibility tree, making the site unusable for screen reader users) or fold it into the intro and delete the section.
Make verification an explicit checkpoint in the Check/Fix flow, e.g. "Verify: inspect the accessibility tree or run axe to confirm body is exposed while a modal is open", instead of the generic "note how to verify the fix" phrasing.
Rewrite the Code Review paragraph so it does not splice the rule title into the sentence ("interactive states that affect Do not use aria-hidden on the document body" → "interactive states that affect body-level aria-hidden").
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~30-line body is efficient: Quick Reference bullets, one-sentence Check/Fix sections, and a clearly signaled pointer to references/rule.md, with no padding or explanation of concepts Claude already knows. It is not a 5 because the "Explain" section ("Explain the catastrophic impact...") and the twice-repeated rule title inside the "Code Review" paragraph ("interactive states that affect Do not use aria-hidden on the document body") are trimmable filler. | 4 / 5 |
Actionability | As an instruction-only skill the guidance is concrete: the Check names the exact element, attribute, and lifecycle scope ("the <body> element has aria-hidden=\"true\" applied at any point in the lifecycle"), and the Fix names the exact remedy and the correct alternative (background containers when modals are open). It is not a 5 because no inline verification command or code snippet is given — e.g. a selector to run or an axe/DevTools check — leaving the reader to construct the check themselves, and the "Explain" step is vague direction. | 4 / 5 |
Workflow Clarity | The single task is unambiguous and well sequenced (Quick Reference → Check → Fix → Explain → Code Review), and the "Code Review" section references verifying the fix with browser accessibility tooling, satisfying the simple-skill case. It is not a 5 because verification is only gestured at ("note how to verify the fix with browser accessibility tooling") rather than given as an explicit checkpoint, and the overlapping Check/Code Review sections blur the sequence slightly. | 4 / 5 |
Progressive Disclosure | The body is under 50 lines with well-organized sections and a single clearly signaled, one-level-deep reference ("For full implementation details, code examples, and framework-specific guidance, see references/rule.md") that exists in the bundle. Content is appropriately split — overview inline, examples and framework guidance in the reference file — so navigation is easy and nothing is inlined that belongs elsewhere. | 5 / 5 |
Total | 17 / 20 Passed |