Phase 1 / Task 2: Remove drawDebugShadowMap from IRenderContext

[MichaelFisher1997]

Goal

Remove debug overlay rendering from the general render context.

Scope

• src/engine/graphics/rhi.zig
• debug overlay call sites

Work

• Remove drawDebugShadowMap from IRenderContext.VTable
• Remove the IRenderContext.drawDebugShadowMap() forwarding method
• Route debug shadow rendering through IDebugOverlayContext

Acceptance criteria

• Debug shadow rendering uses the dedicated interface only
• Build passes

Parallel-safe

Yes

[github-actions]

This is a well-scoped refactoring task (Feature Request). The implementation is straightforward — drawDebugShadowMap already exists in IDebugOverlayContext (rhi.zig:650), so the work is purely removal.

Summary of changes needed:

1. src/engine/graphics/rhi.zig:

   • Remove drawDebugShadowMap from IRenderContext.VTable (line 689)
   • Remove the RenderContext.drawDebugShadowMap() forwarding method (lines 284-286)

2. src/engine/graphics/rhi_vulkan.zig:

   • Remove drawDebugShadowMap function (lines 849-852)
   • Remove .drawDebugShadowMap = drawDebugShadowMap from VULKAN_RENDER_CONTEXT_VTABLE (line 944)

3. src/engine/graphics/rhi_tests.zig:

   • Remove the test mock drawDebugShadowMap implementation and vtable entry (lines 134, 244)

No call sites use RenderContext.drawDebugShadowMap directly — it only routes through the VTable. The acceptance criteria are achievable; the build will pass once the entries are removed.

opencode session  |  github run