Phase 3 / Task 3: Extract GPU acceleration coordinator

[MichaelFisher1997]

Goal

Split GPU block-buffer and GPU meshing coordination out of WorldStreamer.

Scope

• src/world/world_streamer.zig
• src/world/gpu_block_buffer.zig
• src/world/gpu_mesher.zig

Work

• Move gpu_block_buffer and gpu_mesher ownership into a dedicated helper/coordinator
• Keep CPU fallback behavior intact
• Preserve safe-mode and GPU fallback logic

Acceptance criteria

• GPU acceleration code is separated from core queue coordination
• CPU fallback still functions when GPU meshing is unavailable

Parallel-safe

Yes

[github-actions]

This is a Feature Request / Refactoring Task - extracting GPU acceleration coordination from WorldStreamer.

Analysis

Current State:

• WorldStreamer directly owns gpu_block_buffer: ?*GpuBlockBuffer and gpu_mesher: ?*GpuMesher (lines 156-157)
• GPU logic is woven into three methods:
  • processUploads() (lines 368-412): handles GPU buffer allocation/upload and GPU meshing queue
  • processMeshJob() (lines 564-567): early-exits to GPU if available
  • processUnloads() (lines 453-455): frees GPU buffer slots

Proposed Architecture:
A new GpuAccelerationCoordinator (or similar name) would own the GPU resources and expose a clean interface:

• uploadChunkBlocks() - handle block buffer upload
• queueGpuMesh() / processGpuMeshes() - coordinate GPU meshing
• CPU fallback path preserved when GPU unavailable

Scope Assessment:

• Bounded scope: 3 files, single responsibility extraction
• No API changes to ChunkStorage, JobQueue, or other subsystems
• CPU fallback logic stays in WorldStreamer, triggered when gpu_mesher == null
• Parallel-safe: coordinator would remain !Sync but usable from main thread only

Implementation Suggestion:
Create src/world/gpu_acceleration_coordinator.zig that encapsulates:

1. Slot allocation & block uploads via GpuBlockBuffer
2. Mesh dispatch & result finalization via GpuMesher
3. Expose beginFrame() / endFrame() or per-chunk methods with clear fallback semantics

Check for Duplicates: No existing coordinator found in codebase. This aligns with the "Batch 5" comment on line 155 referencing Issue #389.

opencode session  |  github run