minecraft-renderer 0.1.71 → 0.1.73

package/src/wasm-mesher/worker/mesherWasm.ts

@@ -8,6 +8,7 @@ import { worldColumnKey, World } from '../../mesher-shared/world'
 import { handleGetHeightmap, EMPTY_COLUMN_HEIGHTMAP_SENTINEL } from '../../mesher-shared/computeHeightmap'
 import { collectBlockEntityMetadata, type SignMeta, type HeadMeta, type BannerMeta } from '../../mesher-shared/blockEntityMetadata'
 import { SectionRequestTracker } from './mesherWasmRequestTracker'
+import { sectionYsForLightColumnDirty } from './mesherWasmLightDirty'
 import {
   CONVERSION_CACHE_LIMIT,
   clearConversionCache,
@@ -18,6 +19,7 @@ import {
 
 let wasm: typeof import('../runtime-build/wasm_mesher.js') | null = null
 let wasmInitialized = false
+let wasmReady = false // true ONLY after wasm.default() instantiates the module; gates light-packet parsing
 
 // Pending raw `update_light` packets that arrived before WASM finished
 // loading. Parsed and drained once `initWasm` resolves. Without this queue
@@ -33,7 +35,7 @@ const pendingUpdateLightV17: Array<{ rawPacket: Uint8Array, numSections: number
 const pendingUpdateLightV16: Array<{ rawPacket: Uint8Array }> = []
 
 function processUpdateLightV17 (rawPacket: Uint8Array, numSections: number): void {
-  if (!wasm || !(wasm as any).parseUpdateLightV17) {
+  if (!wasmReady) {
     pendingUpdateLightV17.push({ rawPacket, numSections })
     return
   }
@@ -41,11 +43,13 @@ function processUpdateLightV17 (rawPacket: Uint8Array, numSections: number): voi
     const parsed: any = (wasm as any).parseUpdateLightV17(rawPacket, numSections)
     const x = (parsed.x as number) * 16
     const z = (parsed.z as number) * 16
+    const skyLight = parsed.skyLight as Uint8Array
     updateLightV17Cache.set(rawCacheKey(x, z), {
-      skyLight: parsed.skyLight as Uint8Array,
+      skyLight,
       blockLight: parsed.blockLight as Uint8Array,
     })
     invalidateConversion(x, z)
+    dirtyColumnSectionsForLightUpdate(x, z)
   } catch (err) {
     console.warn('[WASM Mesher] parseUpdateLightV17 failed:', err)
   }
@@ -57,7 +61,7 @@ function processUpdateLightV17 (rawPacket: Uint8Array, numSections: number): voi
 // chunk cache has the entry, and crossing the streams could mismatch a
 // stale 1.17 column with 1.16 light or vice versa during version switches.
 function processUpdateLightV16 (rawPacket: Uint8Array): void {
-  if (!wasm || !(wasm as any).parseUpdateLightV17) {
+  if (!wasmReady) {
     pendingUpdateLightV16.push({ rawPacket })
     return
   }
@@ -70,6 +74,7 @@ function processUpdateLightV16 (rawPacket: Uint8Array): void {
       blockLight: parsed.blockLight as Uint8Array,
     })
     invalidateConversion(x, z)
+    dirtyColumnSectionsForLightUpdate(x, z)
   } catch (err) {
     console.warn('[WASM Mesher] parseUpdateLightV17 (v16) failed:', err)
   }
@@ -81,6 +86,7 @@ async function initWasm() {
     wasmInitialized = true
     wasm = await import('../runtime-build/wasm_mesher.js')
     await wasm.default('/wasm_mesher_bg.wasm') as any
+    wasmReady = true // instance is now usable; drained packets below will pass the guard
 
     if (pendingUpdateLightV17.length > 0) {
       console.log('[WASM Mesher] draining', pendingUpdateLightV17.length, 'pending update_light v17 packets')
@@ -94,7 +100,10 @@ async function initWasm() {
       for (const item of queue) processUpdateLightV16(item.rawPacket)
     }
   } catch (err) {
-    console.error('Failed to initialize WASM mesher:', err)
+    console.error(
+      '[WASM Mesher] Failed to initialize WASM mesher — block lighting may stay at full brightness:',
+      err,
+    )
     wasmInitialized = true // Don't try to initialize again
     // Don't throw - allow worker to continue without WASM (will fail on first use)
   }
@@ -195,6 +204,15 @@ function setSectionDirty(pos: Vec3, value = true) {
   }
 }
 
+/** Re-mesh every section in a column after `update_light` updates the light cache. */
+function dirtyColumnSectionsForLightUpdate (x: number, z: number) {
+  const worldMinY = config?.worldMinY ?? 0
+  const worldMaxY = config?.worldMaxY ?? 256
+  for (const y of sectionYsForLightColumnDirty(worldMinY, worldMaxY, SECTION_HEIGHT)) {
+    setSectionDirty(new Vec3(x, y, z))
+  }
+}
+
 const softCleanup = () => {
   world = new World(world.config.version)
   globalThis.world = world
@@ -340,7 +358,7 @@ const convertParsedV17ToWasm = (
   } else {
     blockLight = new Uint8Array(totalBlocks)
     skyLight = new Uint8Array(totalBlocks)
-    skyLight.fill(15)
+    skyLight.fill(config?.skyLight ?? 15)
   }
   const biomesArray: Uint8Array = parsed.biomes
   let blockCount = 0
@@ -406,7 +424,7 @@ const convertParsedV16ToWasm = (
   } else {
     blockLight = new Uint8Array(totalBlocks)
     skyLight = new Uint8Array(totalBlocks)
-    skyLight.fill(15)
+    skyLight.fill(config?.skyLight ?? 15)
   }
   const biomesArray: Uint8Array = parsed.biomes
   let blockCount = 0
@@ -465,7 +483,7 @@ const meshColumnFromRawV18Plus = (
       meta.occludingBlocks,
       config?.enableLighting !== false,
       config?.smoothLighting !== false,
-      config?.skyLight || 15
+      config?.skyLight ?? 15
     )
   } catch (err) {
     console.warn('[WASM Mesher] generateGeometryFromMapChunkV18Plus failed, falling back:', err)
@@ -511,7 +529,7 @@ const meshColumnFromParsedV16V17 = (
       meta.occludingBlocks,
       config?.enableLighting !== false,
       config?.smoothLighting !== false,
-      config?.skyLight || 15
+      config?.skyLight ?? 15
     )
   } catch (err) {
     console.warn('[WASM Mesher] generateGeometryFromParsedV16V17 failed, falling back:', err)
@@ -572,7 +590,7 @@ const meshMultiColumnsFromRawV18Plus = (
       meta.occludingBlocks,
       config?.enableLighting !== false,
       config?.smoothLighting !== false,
-      config?.skyLight || 15
+      config?.skyLight ?? 15
     )
   } catch (err) {
     console.warn('[WASM Mesher] generateGeometryFromMapChunkV18PlusMulti failed:', err)
@@ -672,7 +690,7 @@ const meshMultiColumnsFromParsedV16V17 = (
       meta.occludingBlocks,
       config?.enableLighting !== false,
       config?.smoothLighting !== false,
-      config?.skyLight || 15
+      config?.skyLight ?? 15
     )
   } catch (err) {
     console.warn('[WASM Mesher] generateGeometryFromParsedV16V17Multi failed:', err)
@@ -976,6 +994,8 @@ function makeEmptyColumnGeometry(sx: number, sy: number, sz: number, sectionHeig
     positions: new Float32Array(0),
     normals: new Float32Array(0),
     colors: new Float32Array(0),
+    skyLights: new Float32Array(0),
+    blockLights: new Float32Array(0),
     uvs: new Float32Array(0),
     indices: new Uint32Array(0),
     indicesCount: 0,
@@ -1033,6 +1053,7 @@ function processColumnTick() {
     let preCacheHits = 0
     let preCacheMisses = 0
     let hadError = false
+    let columnMeshPath = 'none'
     // Outer-scope timestamps so we can finalize `processTime` and
     // `postPhase` AFTER the per-section emit loop runs (the loop builds
     // typed arrays, walks block-entity metadata, and calls postMessage —
@@ -1052,6 +1073,7 @@ function processColumnTick() {
         let wasmResult: any
         let t1 = 0
         let usedFusedPath = false
+        columnMeshPath = 'none'
 
         const meta = getBlockMeta(version)
 
@@ -1067,6 +1089,7 @@ function processColumnTick() {
 
           if (rawEntry) {
             wasmResult = meshColumnFromRawV18Plus(rawEntry, x, z, worldMinY, worldMaxY, meta)
+            if (wasmResult) columnMeshPath = 'v18_fused'
           } else if (v17Entry) {
             const v17Light = updateLightV17Cache.get(rawCacheKey(x, z))
             wasmResult = meshColumnFromParsedV16V17(
@@ -1075,6 +1098,7 @@ function processColumnTick() {
               v17Light?.skyLight ?? null, v17Light?.blockLight ?? null,
               x, z, worldMinY, worldMaxY, meta
             )
+            if (wasmResult) columnMeshPath = 'v17_fused'
           } else if (v16Entry) {
             const v16Light = updateLightV16Cache.get(rawCacheKey(x, z))
             const bitMapLoHi = new Uint32Array([v16Entry.bitMap >>> 0, 0])
@@ -1084,6 +1108,7 @@ function processColumnTick() {
               v16Light?.skyLight ?? null, v16Light?.blockLight ?? null,
               x, z, worldMinY, worldMaxY, meta
             )
+            if (wasmResult) columnMeshPath = 'v16_fused'
           }
 
           if (wasmResult) {
@@ -1104,6 +1129,7 @@ function processColumnTick() {
           wasmResult = meshMultiColumnsFromRawV18Plus(chunksToUse, x, z, worldMinY, worldMaxY, meta)
                     ?? meshMultiColumnsFromParsedV16V17(chunksToUse, x, z, worldMinY, worldMaxY, meta)
           if (wasmResult) {
+            columnMeshPath = 'multi_fused'
             usedFusedPath = true
             t1 = performance.now()
             wasmPhase = t1 - t0
@@ -1181,8 +1207,9 @@ function processColumnTick() {
               invisibleBlocks, transparentBlocks, noAoBlocks, cullIdenticalBlocks, occludingBlocks,
               config?.enableLighting !== false,
               config?.smoothLighting !== false,
-              config?.skyLight || 15
+              config?.skyLight ?? 15
             )
+            columnMeshPath = 'two_step_single'
           } else {
             const tBuildStart = performance.now()
             const perChunkLen = conversions[0].blockStates.length
@@ -1214,8 +1241,9 @@ function processColumnTick() {
               invisibleBlocks, transparentBlocks, noAoBlocks, cullIdenticalBlocks, occludingBlocks,
               config?.enableLighting !== false,
               config?.smoothLighting !== false,
-              config?.skyLight || 15
+              config?.skyLight ?? 15
             )
+            columnMeshPath = 'two_step_multi'
           }
         }
 
@@ -1306,14 +1334,16 @@ function processColumnTick() {
             for (cursor.x = sx; cursor.x < sx + 16; cursor.x++) {
               const b = world.getBlock(cursor)
               if (!b) continue
-              collectBlockEntityMetadata(b, cursor.x, cursor.y, cursor.z, beTarget, beOpts)
+              collectBlockEntityMetadata(b, cursor.x, cursor.y, cursor.z, beTarget, beOpts, world)
             }
           }
         }
 
         let geometry: MesherGeometryOutput
         let transferable: any[] = []
-        const hasLegacyMesh = (exported?.geometry.indices.length ?? 0) > 0
+        const hasOpaqueMesh = (exported?.geometry.indices.length ?? 0) > 0
+        const hasBlendMesh = (exported?.blendGeometry?.indices.length ?? 0) > 0
+        const hasLegacyMesh = hasOpaqueMesh || hasBlendMesh
         const hasShaderCubes = (exported?.shaderCubes?.count ?? 0) > 0
         if (exported && (hasLegacyMesh || hasShaderCubes)) {
           const maxIndex = exported.geometry.indices.length > 0
@@ -1335,6 +1365,8 @@ function processColumnTick() {
             positions: new Float32Array(exported.geometry.positions),
             normals: new Float32Array(exported.geometry.normals),
             colors: new Float32Array(exported.geometry.colors),
+            skyLights: new Float32Array(exported.geometry.skyLights),
+            blockLights: new Float32Array(exported.geometry.blockLights),
             uvs: new Float32Array(exported.geometry.uvs),
             indices: using32Array
               ? new Uint32Array(exported.geometry.indices)
@@ -1360,13 +1392,37 @@ function processColumnTick() {
               formatVersion: 3,
             }
           }
+          if (exported.blendGeometry && hasBlendMesh) {
+            const blendMax = Math.max(...exported.blendGeometry.indices)
+            geometry.blend = {
+              positions: new Float32Array(exported.blendGeometry.positions),
+              normals: new Float32Array(exported.blendGeometry.normals),
+              colors: new Float32Array(exported.blendGeometry.colors),
+              skyLights: new Float32Array(exported.blendGeometry.skyLights),
+              blockLights: new Float32Array(exported.blendGeometry.blockLights),
+              uvs: new Float32Array(exported.blendGeometry.uvs),
+              indices: blendMax > 65535
+                ? new Uint32Array(exported.blendGeometry.indices)
+                : new Uint16Array(exported.blendGeometry.indices),
+            }
+          }
           transferable = [
             geometry.positions?.buffer,
             geometry.normals?.buffer,
             geometry.colors?.buffer,
+            geometry.skyLights?.buffer,
+            geometry.blockLights?.buffer,
             geometry.uvs?.buffer,
             //@ts-ignore
             geometry.indices?.buffer,
+            geometry.blend?.positions?.buffer,
+            geometry.blend?.normals?.buffer,
+            geometry.blend?.colors?.buffer,
+            geometry.blend?.skyLights?.buffer,
+            geometry.blend?.blockLights?.buffer,
+            geometry.blend?.uvs?.buffer,
+            //@ts-ignore
+            geometry.blend?.indices?.buffer,
             geometry.shaderCubes?.words?.buffer,
           ].filter(Boolean)
 

package/src/wasm-mesher/worker/mesherWasmLightDirty.test.ts

@@ -0,0 +1,11 @@
+//@ts-nocheck
+import { describe, expect, it } from 'vitest'
+import { sectionYsForLightColumnDirty } from './mesherWasmLightDirty'
+
+describe('sectionYsForLightColumnDirty', () => {
+  it('covers every section in a 256-high overworld column', () => {
+    expect(sectionYsForLightColumnDirty(0, 256)).toEqual(
+      Array.from({ length: 16 }, (_, i) => i * 16),
+    )
+  })
+})

package/src/wasm-mesher/worker/mesherWasmLightDirty.ts

@@ -0,0 +1,15 @@
+//@ts-nocheck
+import { SECTION_HEIGHT } from '../../mesher-shared/shared'
+
+/** Section Y values to dirty for a column after `update_light` updates the light cache. */
+export function sectionYsForLightColumnDirty (
+  worldMinY: number,
+  worldMaxY: number,
+  sectionHeight = SECTION_HEIGHT,
+): number[] {
+  const ys: number[] = []
+  for (let y = worldMinY; y < worldMaxY; y += sectionHeight) {
+    ys.push(y)
+  }
+  return ys
+}

package/src/worldView/worldView.ts

@@ -320,6 +320,17 @@ export class WorldView extends (EventEmitter as new () => TypedEmitter<WorldView
     }
   }
 
+  /**
+   * Re-fetch and re-emit every loaded chunk (e.g. after mesher workers are recreated).
+   */
+  async reloadLoadedChunks(): Promise<void> {
+    const coords = Object.keys(this.loadedChunks)
+    for (const key of coords) {
+      const [x, z] = key.split(',').map(Number)
+      await this.loadChunk({ x, z }, false, 'mesher-reconfigure')
+    }
+  }
+
   /**
    * Unload all chunks.
    */