Agent Skills: UI Code Auditor

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.

UncategorizedID: majesticlabs-dev/majestic-marketplace/ui-code-auditor

Install this agent skill to your local

pnpm dlx add-skill https://github.com/majesticlabs-dev/majestic-marketplace/tree/HEAD/plugins/majestic-engineer/skills/ui-code-auditor

Skill Files

Browse the full folder contents for ui-code-auditor.

Download Skill

Loading file tree…

plugins/majestic-engineer/skills/ui-code-auditor/SKILL.md

Skill Metadata

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.

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

/majestic:config 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 |