Skip to content

feat: update a11y skill and references#89

Merged
AnnaPS merged 15 commits into
mainfrom
feat/update_a11y_skill
May 29, 2026
Merged

feat: update a11y skill and references#89
AnnaPS merged 15 commits into
mainfrom
feat/update_a11y_skill

Conversation

@AnnaPS
Copy link
Copy Markdown
Contributor

@AnnaPS AnnaPS commented May 5, 2026

Description

Update the accessibility reference and skills files to match WCAG 2.2.

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Code refactor (refactor)
  • Documentation (docs)
  • CI change (ci)
  • Chore (chore)

@AnnaPS AnnaPS requested a review from a team as a code owner May 5, 2026 11:45
@AnnaPS AnnaPS self-assigned this May 5, 2026
Copy link
Copy Markdown
Contributor

@jorgecoca jorgecoca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Note for the future: when cupertino/material split out of the core libraries, we will need to push a new update

Comment thread skills/accessibility/SKILL.md Outdated
@AnnaPS AnnaPS requested a review from RuiMiguel May 10, 2026 17:13
Comment thread skills/accessibility/references/platform-matrix.md Outdated
@ryzizub
Copy link
Copy Markdown
Contributor

ryzizub commented May 25, 2026

@AnnaPS can I ask you to update the PR with comments above ? ❤️ so @andyhorn can ship skill on wingspan

AnnaPS and others added 7 commits May 25, 2026 18:54
… files

Implement feedback to split platform-specific checks into separate files (iOS, Android, Web, macOS, Windows, Linux) so Claude loads only the platforms being audited, saving tokens on multi-platform audits.

Changes:
- Create references/platforms/{ios,android,web,macos,windows,linux}.md with per-platform WCAG 2.2 checks, testing tools, and Flutter-specific gotchas
- Delete references/platform-matrix.md (consolidated into per-platform files)
- Update SKILL.md to reference platform files in Phase 2 and Additional Resources
- Move cross-platform severity table from platform-matrix.md into audit-templates.md
- Update widget-mapping.md links from WCAG 2.1 to WCAG 2.2

All platforms now cite WCAG 2.2 (not 2.1). Per-platform content includes latest Flutter quirks (iOS liveRegion bug, Android headingLevel regression in 3.38.0, web semantics opt-in, etc.).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
…ency

Implement XML structural tags in SKILL.md to improve Claude's parsing efficiency:
- Wrap core standards in <rule> tags with wcag IDs for easy reference
- Add <phase> tags around workflow phases 1-4 with metadata (id, name)
- Tag <action>, <outcome>, <question>, <constraint>, <finding-format> within phases
- Add <category> tags for the 7 audit check categories with ids
- Wrap anti-patterns in <antipattern> tags with category and wcag attributes
- Add <instructions>, <context> tags for structural clarity

This XML structure helps Claude understand the organization without affecting markdown readability.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
The CLAUDE.md project guidelines specify only these frontmatter fields:
- name (required)
- description (required) — describes when the skill should be triggered
- allowed-tools (optional)
- argument-hint (optional)

Remove the non-standard `when_to_use` and `effort` fields. Consolidate the "when to use" context into the `description` field per project conventions.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@AnnaPS AnnaPS requested a review from ryzizub May 25, 2026 17:30
Copy link
Copy Markdown
Contributor

@ryzizub ryzizub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM just two small comments in frontmatter

Comment thread skills/accessibility/SKILL.md
Comment thread skills/accessibility/SKILL.md
Comment thread skills/accessibility/SKILL.md Outdated
Comment thread skills/accessibility/references/platforms/android.md
Comment thread skills/accessibility/references/platforms/ios.md
Comment thread skills/accessibility/references/platforms/linux.md
Comment thread skills/accessibility/references/platforms/macos.md
Comment thread skills/accessibility/references/platforms/web.md
Comment thread skills/accessibility/references/platforms/windows.md
Copy link
Copy Markdown
Contributor

@RuiMiguel RuiMiguel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Copy Markdown
Contributor

@ryzizub ryzizub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AnnaPS AnnaPS merged commit ef057c1 into main May 29, 2026
5 checks passed
@AnnaPS AnnaPS deleted the feat/update_a11y_skill branch May 29, 2026 09:54
@vgvbot vgvbot mentioned this pull request May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants