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 single-purpose skill with a textbook progressive-disclosure structure: short overview body plus a clearly signaled, verified one-level reference file. Its weaknesses are redundancy — the Quick Reference and Code Review sections largely restate other sections and the description — and verification guidance that stays generic.
Suggestions
Delete the 'Quick Reference' section (its bullets restate the intro and Check/Fix sections) and cut the 'Code Review' section down to the one non-duplicated sentence about verifying fixes with browser accessibility tooling.
Name the concrete verification step, e.g. 'Verify with Tab navigation from page load or a screen reader (NVDA/VoiceOver) that focus stays at the page top' instead of the generic 'browser accessibility tooling'.
Add one inline example of the check, such as the grep/search pattern for autofocus (e.g. `grep -rn 'autofocus' --include='*.html'`), so the Check section is copy-paste executable without opening the reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short but noticeably redundant: the 'Quick Reference' bullets restate the intro ('Autofocus disrupts screen reader users who lose page context') and the Check/Fix sections, and the 'Code Review' section largely repeats the frontmatter description ('Review the rendered markup and interactive states that affect Avoid autofocus on form fields'). This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than anchor 4's 'minor instances', since whole sections are duplicative; it is far from anchor 2's heavy padding. | 3 / 5 |
Actionability | Concrete, executable instruction-only guidance: 'Search for autofocus attributes on form elements', 'Remove autofocus attributes from form elements', 'use JavaScript to set focus after user interaction rather than automatically on page load'. Per the code-vs-instruction note, absent code is not penalized since the guidance is actionable; minor gaps remain (no grep pattern, no named browser a11y tooling) keeping it below 5 but above anchor 3's 'incomplete, missing key details'. | 4 / 5 |
Workflow Clarity | A clear implied sequence — Check (find autofocus, verify focus doesn't jump on load) → Fix (remove attribute / JS focus after interaction) → Explain → Code Review (flag elements, 'note how to verify the fix with browser accessibility tooling') — with the verification step present but generic. Not 5 because verification names no specific tool or procedure; not 3 because the sequence and its verify step are explicit rather than missing. | 4 / 5 |
Progressive Disclosure | The body is a concise overview and cleanly delegates detail with a well-signaled, one-level-deep reference — 'For full implementation details, code examples, and framework-specific guidance, see references/rule.md' — and references/rule.md exists as a real 131-line file with code examples and framework guidance. This matches the anchor for a clear overview with appropriately split content and easy navigation. | 5 / 5 |
Total | 16 / 20 Passed |