Skip to content

Commit 1eea7b0

Browse files
fix(.ai): missing instructions in a11y migration skill
1 parent 5f716d6 commit 1eea7b0

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

  • .ai/skills/accessibility-migration-analysis

.ai/skills/accessibility-migration-analysis/SKILL.md

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ Then add rows for **name**, **states**, **properties**, **visual-only props**, *
102102
### Shadow DOM and cross-root ARIA Issues
103103

104104
- **Heading text must be exactly:** `### Shadow DOM and cross-root ARIA Issues` (word **Issues** capitalized).
105-
- **If** the component has **no** cross-root ARIA concerns (no reliance on **ID references** that must resolve across shadow boundaries, e.g. `aria-labelledby` / `aria-describedby` pointing at shadow-only IDs) **and** it is **not** a **form-associated** control, the **entire body** of the subsection should be a single word: **`None`**.
105+
- **If** the component has **no** cross-root ARIA concerns (no reliance on **ID references** that must resolve across shadow boundaries, e.g. `aria-labelledby` / `aria-describedby` pointing at shadow-only IDs) **and** it is **not** a **form-associated** control, the **entire body** of the subsection should be a single word: **`None`** (no extra sentences).
106106
- **Otherwise** describe the concrete issues and expectations (e.g. `ElementInternals`, `aria-*` delegation, proposed ID strategies).
107107

108108
### Live regions and announcements
@@ -111,7 +111,7 @@ When the component or its docs touch **live regions** or **frequent** status upd
111111

112112
- Call out **over-announcing** as a risk: docs should **warn** authors not to flood screen reader users.
113113
- **Never** recommend **`aria-live="assertive"`** for loading or routine progress.
114-
- Treat **`aria-live="polite"`** as **rare**: prefer **native role semantics** (e.g. **`progressbar`**) and **one** primary message for related loaders when possible.
114+
- Treat **`aria-live="polite"`** as **rare**: polite regions still **queue** speech, and **several** components or regions updating together becomes **noisy** (bursts, backlog). Prefer **native role semantics** (e.g. **`progressbar`**) and **one** primary message for related loaders when possible.
115115

116116
See `progress-circle/accessibility-migration-analysis.md` for a full example.
117117

@@ -130,6 +130,8 @@ See `progress-circle/accessibility-migration-analysis.md` for a full example.
130130

131131
### Testing
132132

133+
### Automated tests
134+
133135
- Table: **Kind of test** | **What to check** (unit, aXe/Storybook, Playwright ARIA snapshots, contrast, etc. — match what the repo actually uses for that component).
134136

135137
### Summary checklist
@@ -145,3 +147,9 @@ See `progress-circle/accessibility-migration-analysis.md` for a full example.
145147
- Follow **text-formatting** rules: sentence case for headings (proper nouns such as **ARIA**, **WCAG**, **APG** stay as usual).
146148
- Prefer plain, scannable wording; avoid duplicating the rendering doc — **link** to it instead.
147149
- **Verify** behavior and ARIA in **2nd-gen source** before stating what the component exposes.
150+
151+
## Related rules and skills
152+
153+
- **contributor-doc-update.md** — when to run `update-nav.js` after heading or structure changes.
154+
- **component-migration-analysis** skill — for `rendering-and-styling-migration-analysis.md`, not this file.
155+
- **stories-documentation.md** / **stories-format.md** — Storybook docs, separate from this contributor planning doc.

0 commit comments

Comments
 (0)