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 well-structured, appropriately short overview with excellent progressive disclosure to a real references/rule.md containing the code examples. Its weaknesses are redundancy between the intro, Quick Reference, and Explain sections, and actionability gaps — no inline markup example or named verification tooling, leaving the Check/Fix guidance concrete but not fully executable on its own.
Suggestions
Merge the Explain section and the overlapping Quick Reference bullets into Check/Fix to remove the near-verbatim restatement of the intro.
Inline one small concrete example, e.g. flag `video[autoplay]:not([muted])` and the corrected `<video controls>` markup, so the fix is executable without opening the reference.
Name specific verification tooling in Check (e.g., axe DevTools, Lighthouse, or tabbing through the first few stops) instead of the vague 'browser accessibility tooling or assistive tech'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and mostly lean, but there is real redundancy: the 'Explain' section ("Explain how autoplaying media interferes with screen readers, startles users...") restates the intro paragraph almost verbatim, and the Quick Reference bullets duplicate the Check/Fix content (e.g., the 5-second rule appears twice). This matches 'mostly efficient but includes some unnecessary explanation or could be tightened'. | 3 / 5 |
Actionability | The Fix section gives genuinely concrete direction ("Remove autoplay attributes from media elements... add muted attribute and provide prominent, keyboard-accessible pause controls") and the 5-second stop rule is a concrete criterion. But the body contains no markup examples or selectors (e.g., video[autoplay]:not([muted])), and the Code Review section's "note how to verify the fix with browser accessibility tooling or assistive tech" is vague direction with no named tool or command — some concrete guidance but incomplete, with executable detail deferred to the reference file. | 3 / 5 |
Workflow Clarity | The Check → Fix → Explain → Code Review sections give a coherent detect/fix/communicate/verify sequence for a single-purpose review task, and verification is mentioned ("confirm media is muted by default and pause controls are immediately accessible via keyboard"). Not a 5 because the verification step names no explicit method or tool, leaving the checkpoint implicit rather than a concrete feedback loop. | 4 / 5 |
Progressive Disclosure | The body is a clear ~30-line overview with well-organized sections, and the single external reference is explicitly and accurately signaled: "For full implementation details, code examples, and framework-specific guidance, see references/rule.md" — the file exists and contains exactly that (HTML/TSX examples, framework guidance), one level deep with easy navigation. | 5 / 5 |
Total | 15 / 20 Passed |