Skip to content

test(a11y): add structured markup audit harness#977

Merged
XingYu-Zhong merged 1 commit into
developfrom
codex/replace-926-a11y-harness
Jul 19, 2026
Merged

test(a11y): add structured markup audit harness#977
XingYu-Zhong merged 1 commit into
developfrom
codex/replace-926-a11y-harness

Conversation

@XingYu-Zhong

Copy link
Copy Markdown
Collaborator

Summary

  • Rebuilds the useful accessibility-test idea from test(a11y): add static accessibility harness #926 on the current develop branch.
  • Adds a structured, deterministic markup audit for component tests without presenting it as a replacement for browser or assistive-technology testing.

Changes

  • Parses server-rendered HTML with parse5 instead of matching markup with regular expressions.
  • Detects duplicate IDs, every missing aria-labelledby reference, unnamed interactive controls, form-label gaps, modal-dialog gaps, and custom keyboard-semantics gaps.
  • Ignores hidden implementation subtrees and exercises a real production component (RuntimeBanner).

Tests

  • npx vitest run src/renderer/src/testing/accessibility-harness.test.ts (5 passed).
  • Full root app suite (4488 passed) plus all extension-package suites (58 passed).
  • Root typecheck, targeted ESLint, and git diff --check.

Original contribution and test direction: @luoye520ww in #926.

Part of #885. Replaces #926.

@XingYu-Zhong
XingYu-Zhong force-pushed the codex/replace-926-a11y-harness branch from 8ccf45c to aec80d2 Compare July 19, 2026 05:00
@XingYu-Zhong
XingYu-Zhong marked this pull request as ready for review July 19, 2026 05:01
@XingYu-Zhong
XingYu-Zhong merged commit f5e10a6 into develop Jul 19, 2026
9 of 15 checks passed
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.

1 participant