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.
The body is a lean, well-sectioned overview with a coherent Check/Fix/Explain/Code Review flow and an appropriately signaled one-level-deep reference that exists and carries the code examples and verification detail. The main gaps are the absence of any inline violating-markup example, mild redundancy between the intro and the Explain section, and validation being alluded to rather than an explicit step.
Suggestions
Inline one compact ✅/❌ markup pair (or link to the reference's Code Example section from the Check section) so the violating pattern is visible without opening references.
Merge the intro paragraph and the Explain section, which both state that role='text' flattens descendant semantics, to remove the duplication.
Add an explicit Verify step in the body, e.g., "After fixing, re-check the container with keyboard navigation or an axe/Lighthouse run", instead of only mentioning verification inside Code Review.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~35-line body is lean with no library tutorials or padding, but there is redundancy: the intro paragraph ("forces screen readers to treat everything inside as a single string of text") and the Explain section ("role='text' overrides the semantics of descendant elements") state the same thing, and the "black holes" Quick Reference bullet repeats it again. This fits anchor 4 (efficient, minor instances that could be trimmed) rather than anchor 5 (every token earns its place). | 4 / 5 |
Actionability | The Check section is specific ("Identify elements with role='text' and verify they do not contain any focusable elements like buttons, links, or inputs") and the Fix section gives two concrete remedies (remove role='text' or move interactive elements outside). It falls short of anchor 5 because the body contains no inline example of the violating markup (delegated to references) and "browser accessibility tooling" names no specific tool, leaving minor gaps per anchor 4. | 4 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections form a clear, well-ordered sequence for a simple single-rule skill, with "note how to verify the fix with browser accessibility tooling or assistive tech" mentioning validation. It does not reach anchor 5 because the body has no explicit validation step or feedback loop of its own — verification is only alluded to and lives in the reference file — fitting anchor 4 (clear sequence, minor validation gaps). | 4 / 5 |
Progressive Disclosure | The body is a concise, well-sectioned overview that correctly delegates detail via a clearly signaled one-level-deep pointer ("For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`"), and the referenced file exists and contains the code examples and verification detail. The split is appropriate and navigation is easy, matching anchor 5; the pointer's "framework-specific guidance" phrase slightly overpromises, but not enough to drop to anchor 4's 'minor organization gaps'. | 5 / 5 |
Total | 17 / 20 Passed |