Content
70%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 clean, well-structured overview body that delegates detail correctly to references/rule.md. It loses points on duplication between the Quick Reference, Check, and Code Review sections and on fix guidance that names the goal but not the concrete mechanism (no inline markup example, no aria-label mention).
Suggestions
Merge the Quick Reference bullets with the Check section — they restate the same accessible-name/aria-describedby verification three times — and drop the generic "Code Review" boilerplate that repeats the description.
Add a short inline correct/incorrect markup example (a trigger with aria-describedby and a role="tooltip" element with aria-label) so the Fix section shows the concrete mechanism, not just the goal.
State the fix attributes explicitly in the Fix section (e.g., name via aria-label or associate via aria-describedby) instead of the abstract "Assign an accessible name... or ensure it is correctly referenced".
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is compact, but the Quick Reference bullets ("Tooltips must have an accessible name or be referenced by `aria-describedby`") nearly duplicate the Check section ("Verify that all elements with role="tooltip" have an accessible name or are linked... via aria-describedby"), and the "Code Review" section is generic boilerplate that largely restates the description. This is mostly efficient but could be noticeably tightened, matching anchor 3 rather than the minor-trim profile of anchor 4. | 3 / 5 |
Actionability | The Check section is concrete (specific role="tooltip" and aria-describedby attributes to look for), but the Fix section — "Assign an accessible name to the tooltip or ensure it is correctly referenced" — never states how (e.g., aria-label) and the body includes no correct/incorrect markup example, deferring all specifics to the reference. Some concrete guidance but missing key executable details (anchor 3); anchor 4 would require concrete code or commands in the body with only minor gaps. | 3 / 5 |
Workflow Clarity | This is a simple, single-purpose review skill under 50 lines, and the single action is unambiguous: Check states exactly what to verify (tooltip elements have an accessible name or an aria-describedby association). The Quick Reference → Check → Fix → Explain → Code Review sequence is clear and well-organized, so the simple-skill exception applies. | 5 / 5 |
Progressive Disclosure | The body is a concise overview with a clearly signaled one-level-deep reference — "For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`" — and that file actually exists with exactly that class of content (code examples, why-it-matters, exceptions, verification steps). Content is appropriately split between overview and reference with easy navigation. | 5 / 5 |
Total | 16 / 20 Passed |