$npx -y skills add gtrabanco/agentic-workflow --skill review-a11yInternal accessibility review pass of the agentic-workflow review pack — composed in-turn by review-change and product-audit; not a menu entry. Checks the changed user-facing surface for accessibility: semantics, keyboard, focus, contrast, and ARIA correctness — applies only to u
| 1 | # Review Accessibility (internal) |
| 2 | |
| 3 | Composed by `review-change` / `product-audit` within their conversation — on any |
| 4 | agent, follow this file inline as the routed step. **Findings only; never edits, |
| 5 | never refactors.** |
| 6 | |
| 7 | ## Scope |
| 8 | |
| 9 | The diff or path/glob the caller passes; default the current change vs the |
| 10 | default branch. State the scope at the top of the returned table. |
| 11 | |
| 12 | ## Checklist (evaluate EVERY item — none is optional; n/a must be stated) |
| 13 | |
| 14 | ✓ Semantic elements over generic ones (button not clickable div; headings in order; landmarks present) |
| 15 | ✓ Every input has an associated label; every image an alt (empty alt only when decorative) |
| 16 | ✓ Full keyboard operability of the changed surface (tab order, Enter/Space activation, Escape to dismiss) |
| 17 | ✓ Focus management on dynamic changes (modals trap + restore focus; route changes move focus) |
| 18 | ✓ Contrast of changed text/UI meets the project's declared level (default WCAG AA) — cite computed values where checkable |
| 19 | ✓ ARIA used correctly and only when semantics can't do it (wrong ARIA is a major finding) |
| 20 | ✓ No information conveyed by color alone |
| 21 | ✓ Motion/animation respects reduced-motion preferences (n/a if no motion added) |
| 22 | ✓ Error messages programmatically associated with their fields |
| 23 | |
| 24 | ## Return exactly |
| 25 | |
| 26 | ``` |
| 27 | REVIEW ACCESSIBILITY — scope: <scope> |
| 28 | |
| 29 | | # | Finding | Sev | Evidence | Suggested fix | |
| 30 | |---|---------|-----|----------|---------------| |
| 31 | | 1 | <what> | critical|major|minor | <file:line> | <smallest action> | |
| 32 | |
| 33 | Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>) |
| 34 | Summary: <1-2 sentences> |
| 35 | Decision: PASS | FAIL |
| 36 | ``` |
| 37 | |
| 38 | FAIL if any critical or major finding is open; PASS otherwise. Minor findings |
| 39 | never block — they route to the caller's triage step. |
| 40 | |
| 41 | ## Done when |
| 42 | |
| 43 | - Every checklist item was evaluated with evidence (file:line or command output) |
| 44 | or explicitly marked n/a with the reason. |
| 45 | - The fixed-format block above is returned — nothing more, nothing less — and |
| 46 | no code was changed. |