From 21550ecff283a3fd742eed6ed0407d58eaa1e3e9 Mon Sep 17 00:00:00 2001 From: Erik Date: Sat, 9 May 2026 22:25:26 +0200 Subject: [PATCH] fix(A.5 T2): document Kind placeholder in HandleJob Code review on commit 90a2027 flagged that HandleJob silently ignores load.Kind. Add a TODO(A.5 T11/T16) comment at the case arm so the unused field reads as a planned stub, not a bug. No semantic change. Co-Authored-By: Claude Opus 4.7 (1M context) --- src/AcDream.App/Streaming/LandblockStreamer.cs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/AcDream.App/Streaming/LandblockStreamer.cs b/src/AcDream.App/Streaming/LandblockStreamer.cs index a325fb6..b79946a 100644 --- a/src/AcDream.App/Streaming/LandblockStreamer.cs +++ b/src/AcDream.App/Streaming/LandblockStreamer.cs @@ -157,6 +157,11 @@ public sealed class LandblockStreamer : IDisposable switch (job) { case LandblockStreamJob.Load load: + // TODO(A.5 T11/T16): route by load.Kind. LoadFar will skip + // LandBlockInfo + scenery generation; PromoteToNear will skip + // mesh build (terrain already on GPU). Today every Kind takes + // the full-load path via _loadLandblock, which matches today's + // single-tier semantics. try { var lb = _loadLandblock(load.LandblockId);