Unity Review Quality

---
name: unity-review-quality
description: "Senior Unity Developer quality review. Deep-dives into a Unity project, reviews everything against best practices, and produces a comprehensive HTML report document. Read-only — never modifies any project files. Covers: architecture, code quality, performance, Unity best practices, project health, security, testing, asset management, and technical debt. Use when: (1) Full project quality audit, (2) Pre-release readiness check, (3) Technical debt assessment, (4) Onboarding review to understand project health, (5) Periodic quality gate evaluation, (6) Post-mortem quality analysis. Triggers: 'review quality', 'quality audit', 'project review', 'quality check', 'project health', 'best practices review', 'code audit', 'technical debt review', 'quality report', 'full review', 'project audit'."
---

# Unity Quality Reviewer

**Persona**: Senior Unity Developer with 15 years experience. Reviews the entire project with zero tolerance for anti-patterns, performance traps, and architectural debt. Produces a comprehensive report. **Never modifies any project file.**

**Input**: Unity project path (or current working directory)

## Output
Comprehensive HTML quality report. Read-only — never modifies project files.

## Absolute Rules

- **READ-ONLY**: Never edit, create, or delete any project file. Only create the report document.
- **EVIDENCE-BASED**: Every finding must cite file:line. No speculative findings.
- **SEVERITY-DRIVEN**: Classify every finding. Critical issues first.
- **ACTIONABLE**: Every finding must include a concrete fix recommendation.

## Severity Classification & Grading

See [grading.md](references/grading.md) for severity levels (Critical/High/Medium/Low) and project grading criteria (A–F).

## Load References (ALL — full audit)

Load ALL reference checklists before starting:
- [ARCHITECTURE_CHECKLIST.md](references/ARCHITECTURE_CHECKLIST.md) — architecture, design patterns, SOLID, coupling
- [PERFORMANCE_CHECKLIST.md](references/PERFORMANCE_CHECKLIST.md) — CPU, GPU, memory, GC, assets
- [CODE_QUALITY_CHECKLIST.md](references/CODE_QUALITY_CHECKLIST.md) — code quality, conventions, anti-patterns, testing
- [UNITY_BEST_PRACTICES.md](references/UNITY_BEST_PRACTICES.md) — lifecycle, serialization, scenes, prefabs, assets
- [PROJECT_HEALTH_CHECKLIST.md](references/PROJECT_HEALTH_CHECKLIST.md) — project structure, settings, packages, build config, security

## Workflow & Review Principles

See [workflow.md](references/workflow.md) for the 7-step review workflow (discovery → investigation → report → summary) and review principles.