Content
65%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 overview that delegates implementation detail appropriately to a real one-level reference, with concrete actionable Quick Reference bullets. Weaker on conciseness (rhetorical opener and restated sections) and workflow clarity (no explicit validation checkpoints).
Suggestions
Cut the rhetorical opening sentence about modals being "invisible traps" — Claude already understands the impact of inaccessible modals.
De-duplicate the Check/Fix/Explain/Code Review sections, which restate the same ARIA/focus/keyboard points in different framings; merge or trim to reduce repetition.
Add an explicit validation checkpoint in the workflow (e.g. a numbered "Verify" step pointing to the manual/automated checks in references/rule.md) so the fix → verify loop is unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, but the opening rhetorical sentence ("modal dialogs are invisible traps for keyboard users—they can't navigate, can't escape...") explains a concept Claude already knows, and the Check/Fix/Explain/Code Review sections restate the same ARIA/focus/keyboard points with noticeable redundancy. | 3 / 5 |
Actionability | The Quick Reference bullets give concrete, actionable attributes (role='dialog', aria-modal='true', aria-labelledby, Escape close, focus return), and the Code Review section specifies what to flag; minor gap is the absence of any inline code (delegated to references/rule.md). | 4 / 5 |
Workflow Clarity | A rough Check → Fix → Explain → Code Review sequence is present, but there are no numbered steps or explicit validation checkpoints; verification of the fix is only implicitly referenced rather than stated as a checkpoint. | 3 / 5 |
Progressive Disclosure | The body is a well-organized overview (Quick Reference, Check, Fix, Explain, Code Review) with a clearly signaled one-level-deep reference — "see references/rule.md" — and that bundle file exists, keeping detail out of the overview. | 5 / 5 |
Total | 15 / 20 Passed |