Content
85%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A concise, well-structured overview that properly offloads detail to a verified one-level-deep reference and presents a clear review workflow. The main gap is actionability: the Check and Fix sections stay at the prose-instruction level with no inline executable example, selector, or verification command.
Suggestions
Add a concrete detection method to the Check section, e.g. a snippet like `Object.entries(Array.from(document.querySelectorAll('[id]')).reduce((m,e)=>(m[e.id]=(m[e.id]||[]).concat(e),m),{})).filter(([_,v])=>v.length>1)` or a DevTools approach, instead of the generic "Search for duplicate id attributes".
Include a small inline markup example in the Fix section showing the corrected unique IDs (as rule.md does), so the guidance is copy-paste ready rather than prose-only.
Specify a concrete verification step in the body itself (e.g. "Run axe-core's duplicate-id-active rule or Lighthouse and confirm zero duplicate-ID findings") rather than deferring all verification detail to references/rule.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean (~30 lines) with short Check/Fix/Explain/Code Review sections and no over-explanation of basic HTML or accessibility concepts, so every section earns its place; it does not pad like the verbose anchor at 2. | 3 / 3 |
Actionability | Check ("Search for duplicate id attributes") and Fix ("Assign a unique id to each active element") give concrete direction but no executable code, selector, or tool command, and key details are deferred to the reference, matching the some-guidance-but-incomplete anchor at 2 rather than copy-paste-ready at 3. | 2 / 3 |
Workflow Clarity | As a simple single-purpose review skill under 50 lines it is clearly organized into Check, Fix, Explain, and Code Review with a verify-the-fix mention, meeting the simple-skills allowance for a 3 without needing a multi-step workflow. | 3 / 3 |
Progressive Disclosure | The body is an overview that signals a single one-level-deep reference ("see `references/rule.md`"), the referenced file exists and contains no nested file references, and detailed material is appropriately split out, matching the clear-overview-with-well-signaled-references anchor. | 3 / 3 |
Total | 11 / 12 Passed |