[SDL3 GPU Migration] Task 15: Cleanup & Validation

[bryfur]

Phase 5: Cleanup - Final Cleanup & Validation

Agent: Cleanup Agent
Dependencies: #18 (Task 14: Integration & Testing)
Estimated Time: 2-3 hours

📄 Full details available in: docs/sdl3_gpu_migration_plan.md - See "Task 15: Final Cleanup & Validation" section

---

Objectives

1. Remove all legacy OpenGL code
2. Delete unused files
3. Update documentation
4. Final code review

Files to Delete

• Any remaining .glsl files (if not already removed)
• client/shader.hpp (old embedded shaders)
• Any GL-specific utilities

Files to Verify Clean

Run: grep -r "glGen\|glBind\|glDraw\|glEnable\|glDisable\|glUniform\|glVertex\|glBuffer\|glTexture\|GL_" client/

Expected result: No matches

Documentation Updates

• Update README.md with new build requirements
• Document shader compilation process
• Update any architecture docs
• Add SDL3 GPU migration notes

CMakeLists.txt Final State

Verify these are REMOVED:

• find_package(OpenGL)
• find_package(GLEW)
• OpenGL/GLEW in target_link_libraries

Verify these are PRESENT:

• SDL3 GPU features enabled
• SDL_shadercross linked
• Shader compilation rules

Code Quality

• Remove any TODO/FIXME related to GL migration
• Remove any commented-out GL code
• Consistent naming conventions
• Proper error handling
• Resource cleanup in destructors

Final Verification Commands

# Check for any remaining GL references
grep -rn "GL_" client/ --include="*.cpp" --include="*.hpp"
grep -rn "glew" client/ --include="*.cpp" --include="*.hpp"

# Verify build
rm -rf build && mkdir build && cd build && cmake .. && make

# Run game
./mmo_client

Acceptance Criteria

• Zero GL references in codebase
• Build succeeds cleanly
• Game runs on all target platforms
• Documentation is current
• Code review approved
• CI passes - Project builds successfully on Ubuntu (GitHub Actions)

---

Migration Complete! 🎉

Blocked by: #18