@woosh/meep-engine 2.164.0 → 2.165.1

package/src/generation/theme/ThemeEngine.js

@@ -12,7 +12,6 @@ import TaskGroup from "../../core/process/task/TaskGroup.js";
 import { TaskSignal } from "../../core/process/task/TaskSignal.js";
 import { actionTask } from "../../core/process/task/util/actionTask.js";
 import { countTask } from "../../core/process/task/util/countTask.js";
-import { emptyTask } from "../../core/process/task/util/emptyTask.js";
 import { futureTask } from "../../core/process/task/util/futureTask.js";
 import { TerrainLayer } from "../../engine/ecs/terrain/ecs/layers/TerrainLayer.js";
 import { TerrainFlags } from "../../engine/ecs/terrain/ecs/TerrainFlags.js";
@@ -560,12 +559,48 @@ export class ThemeEngine {
     }
 
     /**
+     * Synchronize the GPU-side terrain state after theming has mutated the CPU-side data
+     * behind the renderer's back.
+     *
+     * {@link applyTerrainThemes} adds the real terrain layers and paints splat weights, and
+     * the cell rules write heights, all straight into the backing arrays — none of which
+     * flags the corresponding GPU resources dirty. If the {@link Terrain} was linked (and
+     * thus already rendering) before theming, the shader keeps rendering the link-time state
+     * — 0 layers, an empty splat map, pre-generation heights — showing up as black or
+     * partially-textured terrain.
      *
      * @param {Terrain} terrain
      * @returns {Task}
      */
     updateTerrain(terrain) {
-        return emptyTask();
+        return futureTask(new Future((resolve, reject) => {
+
+            // The material was baked with the layer count present at link time (0 in the
+            // "generate into a live terrain" flow); recompile it with the real count and
+            // rebind the splat / diffuse / scale samplers. Idempotent — updateMaterial()
+            // only recompiles when the layer count actually changed.
+            terrain.updateMaterial();
+
+            // Splat weights were written straight into the raw weightData backing store,
+            // bypassing the SplatMapping mutators that are the only writers to flag the
+            // texture dirty; force a fresh upload of the completed weights.
+            terrain.splat.weightTexture.needsUpdate = true;
+
+            // Heights were written into samplerHeight (which shares its buffer with
+            // heightTexture) without flagging the GPU copy dirty; re-upload it.
+            terrain.updateHeightTexture();
+
+            // Re-sync the build worker's height snapshot and the tile height range so that
+            // tiles streamed in after generation reflect the generated heights. This needs a
+            // running build worker; when the terrain is not linked the worker is stopped and
+            // the next startBuildService() (on link) re-syncs, so there is nothing to await.
+            if (terrain.getFlag(TerrainFlags.Linked)) {
+                terrain.updateWorkerHeights().then(resolve, reject);
+            } else {
+                resolve();
+            }
+
+        }), 'Synchronize terrain GPU state');
     }
 
     /**