Content
71%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-organized body with a clean Check/Fix/Explain structure and exemplary progressive disclosure to a single real reference file. Its weakest point is actionability: the body tells Claude what to look for but not how (no detection command, selector, or tool invocation is given inline), and the Quick Reference section is mildly redundant with the sections that follow.
Suggestions
Add one concrete detection method to the Check section, e.g. `document.querySelectorAll('button, a, input, select, textarea [id]')` or a grep/lint command, so the check is executable rather than descriptive.
Replace the implicit verification note with an explicit checkpoint, e.g. "Verify: re-run an axe DevTools audit and confirm no 'duplicate-id' violations remain before finishing."
Merge or trim the Quick Reference bullets, which restate the Check/Fix/Explain sections, to remove redundancy.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short, well-sectioned, and assumes Claude's competence — e.g. "Duplicate IDs on active elements can cause browsers and assistive technologies to skip items, misdirect focus, or fail to trigger the correct action" adds rule-specific consequences, not general HTML knowledge. It is not 5 because the three "Quick Reference" bullets ("Ensure all focusable elements... have unique id values", "Avoid duplicate IDs...", "Prevent screen reader confusion...") largely restate the Check/Fix/Explain sections that follow, which is a minor but trimmable redundancy. | 4 / 5 |
Actionability | The Check section gives one concrete instruction — "Search for duplicate id attributes on focusable or active HTML elements" — and Fix gives a clear remedy ("Assign a unique id to each active element"), but no executable method is provided in the body: no search command, selector (e.g. document.querySelectorAll('[id]')), lint/axe invocation, or worked example. This matches the anchor 'some concrete guidance but incomplete; missing key details', and exceeds score 2 because the guidance is specific about what to look for and how to fix it. | 3 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections give a clear, unambiguous sequence for a simple, non-destructive skill, and the Code Review section includes verification guidance ("note how to verify the fix with browser accessibility tooling or assistive tech"). It is not 5 because verification is mentioned only as something to "note" rather than an explicit checkpoint (e.g. re-run an axe/lint audit and confirm no duplicate-id violations remain), so a validation step is implicit rather than built in. | 4 / 5 |
Progressive Disclosure | The body is a concise overview under 50 lines with a single clearly signaled, one-level-deep pointer — "For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`" — and that file exists and contains exactly the promised implementation details, code examples, and framework guidance. This matches the anchor for a clear overview with well-signaled one-level-deep references and easy navigation. | 5 / 5 |
Total | 16 / 20 Passed |