Content
67%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, actionable overview with a clean one-level-deep split into a real references/rule.md file. Its main weakness is redundancy: the role-mapping and why-it-matters content each appear twice, which could be consolidated for better token efficiency.
Suggestions
Keep the container-to-child role mapping in one place (Quick Reference or Check) and remove the duplicate, letting references/rule.md's table carry the full detail.
Merge the intro paragraph and the Explain section, which both cover screen-reader navigation consequences.
Inline one or two concrete verification commands (e.g., axe DevTools or the browser accessibility pane) alongside the Code Review section instead of deferring all verification detail to the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is compact overall, but the container-to-child role mapping is stated twice ('tablist → tab, list/listbox → listitem/option...' in Quick Reference and again in full in Check), the intro paragraph and the Explain section both cover screen-reader consequences, and the Code Review section is generic boilerplate. Mostly efficient, but the duplication could be tightened into a single mapping. | 3 / 5 |
Actionability | Concrete and specific throughout: Check enumerates exact role-to-required-child requirements ('tablist must own tab elements; list must own listitem...') and Fix gives three executable remediation steps ('add role=tab to each tab button', 'add role=none/presentation to those wrappers', 'prefer <ul>/<li> instead of role=list/listitem'). For an instruction-only review skill this is largely executable; the only gaps are verification tooling specifics deferred to the reference. | 4 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections form a clear, unambiguous sequence with per-role verification criteria, and the task is single-purpose and non-destructive so the validation cap does not apply. Checkpoints are mostly present ('note how to verify the fix with browser accessibility tooling'), though the concrete verification steps (axe, accessibility tree) live only in the reference file. | 4 / 5 |
Progressive Disclosure | The body is a proper overview and full implementation details (code examples, standards, exceptions, verification) are split into a single, existing, one-level-deep reference, clearly signaled ('For full implementation details, code examples, and framework-specific guidance, see references/rule.md'). Minor gaps: the pointer appears only at the bottom, and the role-mapping table is duplicated between SKILL.md and rule.md. | 4 / 5 |
Total | 15 / 20 Passed |