Content
63%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 well-structured with a clean Check/Fix/Explain/Code Review flow and exemplary reference hygiene (one real, clearly signaled reference file). It is held back by verbatim duplication of the accessibility-impact rationale and by check guidance that says what to verify but not how to detect or verify it concretely.
Suggestions
Deduplicate the accessibility-impact rationale: the 'Explain' section repeats the opening paragraph nearly verbatim and 'Quick Reference' restates it again — keep one canonical statement.
Add concrete detection guidance to 'Check', e.g. search rendered JS for wheel/touchmove/scroll listeners that call preventDefault(), and check CSS scroll-behavior/overscroll-behavior overrides.
Replace the vague 'browser accessibility tooling' in 'Code Review' with named verification steps (e.g. keyboard-only scroll pass, prefers-reduced-motion emulation in devtools, a screen-reader smoke test).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The "Explain" section ("Explain how scrolljacking disrupts expected navigation patterns, interferes with assistive technologies, and creates unpredictable experiences for users with motor impairments or cognitive disabilities") repeats the opening paragraph nearly verbatim, and "Quick Reference" restates "Scrolljacking breaks assistive technologies and user expectations" already covered in the intro. Mostly efficient overall, but this duplication could be trimmed — below anchor 4's minor-trimming level, well above anchor 2's padded verbosity. | 3 / 5 |
Actionability | Some concrete direction exists — "Check that scroll speed is not modified, scroll direction is not inverted, and scroll events are not hijacked" and "respect prefers-reduced-motion" — but the how is missing: no detection technique (e.g. searching for wheel/touchmove listeners calling preventDefault, or overscroll-behavior/scroll-behavior CSS), no commands, and no verification steps; the executable detail is entirely deferred to references/rule.md. This matches anchor 3's 'concrete guidance but incomplete, missing key details' rather than anchor 4's mostly-executable guidance. | 3 / 5 |
Workflow Clarity | A clear Check → Fix → Explain → Code Review sequence with a verification checkpoint present ("note how to verify the fix with browser accessibility tooling or assistive tech"). Not 5 because the verification step is named but not concretely specified and "verify it feels native" is subjective; not 3 because the sequence is coherent and a fix-verification checkpoint is explicitly included. | 4 / 5 |
Progressive Disclosure | A short (~30 line) body with well-organized sections and a single, clearly signaled one-level-deep reference — "For full implementation details, code examples, and framework-specific guidance, see `references/rule.md`" — and that file exists and indeed contains the implementation detail (bad-pattern code examples, definitions table). Content split between overview and reference is appropriate for a simple skill. | 5 / 5 |
Total | 15 / 20 Passed |