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 a clean, well-structured overview with excellent progressive disclosure to references/rule.md. It is held back by repeated rationale phrasing across sections and Fix guidance that stays high-level rather than giving concrete remediation steps.
Suggestions
Remove the duplicated 'breaks user expectations/interferes with assistive technologies/motor or cognitive disabilities' phrasing from either the intro or the Explain section to tighten conciseness.
Make the Fix section more actionable by adding a concrete pointer or short snippet (e.g., remove preventDefault on wheel/touchmove, prefer CSS scroll-snap, gate effects behind prefers-reduced-motion) rather than only 'Remove JavaScript that overrides default scroll behavior'.
Add an explicit verification checkpoint in the Check section (e.g., 'Confirm scroll feels native via keyboard Page Up/Down and a screen reader; see references/rule.md#verification') to lift workflow clarity.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient but repeats the 'breaks user expectations, interferes with assistive technologies... motor impairments or cognitive disabilities' idea across the intro, a Quick Reference bullet, and the Explain section, which is unnecessary duplication that could be tightened. | 3 / 5 |
Actionability | The Check section lists specific things to verify (scroll speed, direction, hijacked events), but the Fix section gives only high-level direction ('Remove JavaScript that overrides default scroll behavior') with no concrete code or commands in the body itself. | 3 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections give a clear, logical review sequence that is well-organized for a single-purpose review skill, though explicit verification checkpoints are only implicit rather than called out. | 4 / 5 |
Progressive Disclosure | The body is a concise overview that delegates full implementation details, code examples, and framework guidance to a clearly signaled, one-level-deep references/rule.md, which exists as a real file. | 5 / 5 |
Total | 15 / 20 Passed |