ui-code-auditor

name	ui-code-auditor
description	Review source code for UI quality, accessibility violations, and performance anti-patterns. Static code analysis (NOT screenshot-based). Provides file:line references with severity, WCAG refs, and fix suggestions.
allowed-tools	Read Grep Glob Bash

UI Code Auditor

Static code analysis for UI quality and accessibility. Complements screenshot-based tools (visual-validator, ui-ux-designer).

Input

path: string           # File or directory to audit (default: src/ or app/)
tech_stack: string[]   # From config, e.g., ["react", "rails"]
focus: string          # Optional: "accessibility", "performance", "all"

Review Process

1. Detect Tech Stack

config_read("tech_stack", "generic")

2. Load Rules

Load rules from resources folder:

Read(resources/generic-rules.md)  # Always

If tech_stack contains "react":
  Read(resources/react-rules.md)

If tech_stack contains "rails":
  Read(resources/rails-rules.md)

3. Find UI Files

# React/JS projects
find . -name "*.tsx" -o -name "*.jsx" | head -100

# Rails projects
find . -name "*.erb" -o -name "*.html.erb" | head -100

# CSS/Tailwind
find . -name "*.css" -name "*.scss" | head -50

4. Audit Each File

For each UI file:

1. Read file content
2. Apply patterns from loaded rules
3. Record findings with:
   • File path and line number
   • Severity (Critical, Serious, Moderate, Minor)
   • Issue description
   • WCAG reference (if applicable)
   • Fix suggestion

5. Generate Report

Output Format

## UI Code Audit Summary

**Verdict: PASS | FAIL**

- **Files Reviewed**: [count]
- **Critical Issues**: [count] → FAIL if > 0
- **Serious Issues**: [count]
- **Moderate Issues**: [count]
- **Minor Issues**: [count]

### Critical Findings

| File:Line | Issue | WCAG | Fix |
|-----------|-------|------|-----|
| `src/Button.tsx:45` | DIV with onClick, not keyboard accessible | 2.1.1 | Use `<button>` element |
| `src/Card.tsx:23` | Image missing alt text | 1.1.1 | Add `alt="description"` |

### Serious Findings

| File:Line | Issue | WCAG | Fix |
|-----------|-------|------|-----|
| `src/Modal.tsx:89` | Focus not trapped in modal | 2.4.3 | Implement focus trap |

### Moderate Findings

| File:Line | Issue | WCAG | Fix |
|-----------|-------|------|-----|
| `src/List.tsx:12` | Touch target 24px (< 44px) | 2.5.5 | Use `min-w-11 min-h-11` |

### Recommendations

- Consider using Radix UI for accessible dropdown implementation
- 3 animations found without `prefers-reduced-motion` support

Severity Definitions

Severity	Definition	Verdict Impact
Critical	Blocks accessibility (can't use with keyboard/screen reader)	FAIL
Serious	Significantly degrades UX for assistive tech users	Warning
Moderate	Best practice violation, minor UX impact	Warning
Minor	Polish opportunity, no functional impact	Info

WCAG Quick Reference

Code	Name	Common Violations
1.1.1	Non-text Content	Missing alt text
1.3.1	Info and Relationships	Missing form labels
2.1.1	Keyboard	Click-only handlers
2.1.2	No Keyboard Trap	Modal without escape
2.4.3	Focus Order	tabindex > 0
2.4.7	Focus Visible	outline: none
2.5.5	Target Size	Touch target < 44px
4.1.2	Name, Role, Value	Missing ARIA

Quality Gate Integration

When invoked from quality-gate:

• Return Verdict: PASS or Verdict: FAIL
• FAIL if any Critical issues found
• Include summary counts for aggregation

Edge Cases

Scenario	Handling
No UI files found	Return PASS with "No UI files to audit"
tech_stack not configured	Use generic rules only
Stack rules fail to load	Log warning, continue with generic
File cannot be parsed	Skip file, note in report
Multiple tech_stacks	Load rules for each, merge findings, deduplicate