Content
77%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 well-structured, actionable body for a simple checklist rule: clear check sequence, concrete fix guidance, and a clean pointer to a real one-level-deep reference. The main cost is redundancy — the same BCP 47/empty-lang facts are restated across Quick Reference, Check, and Explain, which could be tightened without losing information.
Suggestions
Deduplicate across sections: state BCP 47 validity and the empty-lang rule once (Quick Reference or Check) instead of four times across Quick Reference, Check, Fix, and Explain.
Drop or compress the BCP 47 subtag anatomy in Explain (primary subtag required, region subtags optional) — Claude already knows this; keep only the WCAG citations and the IANA registry pointer.
Move the Fix section's long enumeration of language codes (`en`, `fr`, `es`, `de`, `ja`, `zh-Hans`, `zh-Hant`, `ar`) into references/rule.md, keeping two or three examples inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is compact (~35 lines), but there is noticeable redundancy across sections: BCP 47 validity is stated in Quick Reference, Check, Fix, and Explain; empty `lang=''` is flagged in both Quick Reference and Check; and the Explain section re-describes BCP 47 subtag structure (primary vs. region subtags) that Claude already knows. Not a 2 because nothing is padded filler and each section has a distinct purpose (Check/Fix/Explain). | 3 / 5 |
Actionability | The Check section gives a concrete numbered verification ("(1) a `lang` attribute is present; (2) its value is non-empty; (3) its value is a valid BCP 47 language code") plus an explicit flag list ("missing `lang`, empty `lang=''`, language codes using full names ... mismatches"), and Fix provides copy-paste-ready snippets like `<html lang='fr'>`. Not a 5 because the body itself has no executable code example (correct/incorrect HTML lives in references/rule.md) and no command for validating against the IANA registry. | 4 / 5 |
Workflow Clarity | This is a simple, single-purpose skill and the action is unambiguous: a numbered inspect-and-verify sequence in Check, explicit flag conditions, a Fix section with exact remediation per case, and a validation directive ("Validate language codes against the IANA Language Subtag Registry"). The simple-skill exception applies; no destructive or batch operation requires a feedback loop. | 5 / 5 |
Progressive Disclosure | The SKILL.md body is a well-organized overview under 50 lines with clear sections (Quick Reference / Check / Fix / Explain / Code Review), and the one bundle file `references/rule.md` exists and is clearly signaled at the end ("For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`"). References are one level deep with no nested chaining. | 5 / 5 |
Total | 17 / 20 Passed |