minerender 2.1.0-alpha.34 → 2.1.0-alpha.36

package/dist/node/index.mjs

@@ -1504,24 +1504,20 @@ var AssetLoadError = class extends Error {
 };
 
 // src/util/BatchedExecutor.ts
+import { JobQueue as JobQueue2 } from "jobqu";
 var BatchedExecutor = class {
   constructor(interval = 1, batch = 30) {
+    if (!Number.isFinite(interval) || interval < 0) throw new RangeError("Batch interval must be finite and nonnegative");
+    if (!Number.isInteger(batch) || batch < 1) throw new RangeError("Batch size must be a positive integer");
     this.interval = interval;
     this.batch = batch;
-    this.queue = [];
-    this.task = setInterval(() => this.run());
+    this.queue = new JobQueue2(async (task) => task(), { interval, maxPerRun: batch, maxActive: batch });
   }
   submit(task) {
-    this.queue.push(task);
+    return this.queue.add(() => task());
   }
-  run() {
-    for (let i = 0; i < this.batch; i++) {
-      this.runNext();
-    }
-  }
-  runNext() {
-    let next = this.queue.shift();
-    if (next) next();
+  stop() {
+    this.queue.end();
   }
 };
 
@@ -7243,15 +7239,12 @@ var Skins = class {
 };
 
 // src/world/Chunk.ts
-import { Box3 as Box32, Object3D as Object3D6, Vector3 as Vector311 } from "three";
+import { Vector3 as Vector311 } from "three";
 
 // src/world/MineRenderWorld.ts
 import { Vector3 as Vector310 } from "three";
 var MineRenderWorld = class {
   constructor(scene) {
-    this._size = 4;
-    //TODO: expandable
-    this._blockSize = this._size * 16;
     this._chunks = /* @__PURE__ */ new Map();
     this.scene = scene;
   }
@@ -7262,12 +7255,8 @@ var MineRenderWorld = class {
     if (isTripleArray(posOrX)) {
       return this.getBlockAt(new Vector310(posOrX[0], posOrX[1], posOrX[2]));
     }
-    const chunk = this.getChunkAt(posOrX);
-    if (typeof chunk === "undefined") {
-      return void 0;
-    }
     this.validatePosBounds(posOrX);
-    return chunk.getBlockAt(posOrX);
+    return this.getChunkAt(posOrX)?.getBlockAt(posOrX);
   }
   async setBlockAt(posOrX, yOrBlock, z, block) {
     if (typeof posOrX == "number") {
@@ -7277,32 +7266,48 @@ var MineRenderWorld = class {
       return this.setBlockAt(new Vector310(posOrX[0], posOrX[1], posOrX[2]), yOrBlock);
     }
     this.validatePosBounds(posOrX);
-    const chunk = this.getOrCreateChunkAt(posOrX);
-    return chunk.setBlockAt(posOrX, yOrBlock);
-  }
-  async placeMultiBlock(multiblock, useBatches = true, executor = new BatchedExecutor()) {
-    const keys = new Set(multiblock.blocks.map((block) => AssetKey.parse("blockstates", block.type)));
-    await BlockStates.getAll(keys);
-    const place = async (block) => {
-      if (useBatches && typeof executor !== "undefined") {
-        await new Promise((resolve, reject) => {
-          executor.submit(() => {
-            this.setBlockAt(block.position, block).then(resolve).catch(reject);
-          });
-        });
-      } else {
-        await this.setBlockAt(block.position, block);
+    const value = yOrBlock;
+    const chunk = Chunk.isAir(value) ? this.getChunkAt(posOrX) : this.getOrCreateChunkAt(posOrX);
+    return chunk?.setBlockAt(posOrX, value);
+  }
+  async placeMultiBlock(multiblock, useBatches = true, executor) {
+    const keys = /* @__PURE__ */ new Map();
+    for (const block of multiblock.blocks) {
+      if (Chunk.isAir(block)) continue;
+      const key = AssetKey.parse("blockstates", block.type);
+      keys.set(key.serialize(), key);
+    }
+    await BlockStates.getAll(keys.values());
+    if (!useBatches) {
+      for (const block of multiblock.blocks) await this.setBlockAt(block.position, block);
+      return;
+    }
+    const positions = /* @__PURE__ */ new Map();
+    for (const block of multiblock.blocks) {
+      const key = block.position.join(",");
+      const group = positions.get(key);
+      if (group) group.push(block);
+      else positions.set(key, [block]);
+    }
+    const groups = [...positions.values()];
+    const queue = executor ?? new BatchedExecutor();
+    try {
+      for (let i = 0; i < groups.length; i += queue.batch) {
+        const results = await Promise.allSettled(groups.slice(i, i + queue.batch).map(
+          (blocks) => queue.submit(async () => {
+            for (const block of blocks) await this.setBlockAt(block.position, block);
+          })
+        ));
+        const failure = results.find((result) => result.status === "rejected");
+        if (failure?.status === "rejected") throw failure.reason;
       }
-    };
-    for (let block of multiblock.blocks) {
-      await place(block);
+    } finally {
+      if (!executor) queue.stop();
     }
   }
   async clear() {
-    console.log("CHUNKS", this._chunks);
-    for (let chunk of this._chunks.values()) {
-      console.log("dispose chunk", chunk);
-      await chunk?.dispose();
+    for (const chunk of this._chunks.values()) {
+      await chunk.dispose();
     }
     this._chunks.clear();
   }
@@ -7316,8 +7321,7 @@ var MineRenderWorld = class {
     return chunk;
   }
   getChunkAt(pos) {
-    const index = this.worldPosToChunkIndex(pos);
-    return this._chunks[index];
+    return this._chunks.get(this.worldPosToChunkKey(pos));
   }
   static worldToScenePosition(pos) {
     return new Vector310(
@@ -7333,12 +7337,6 @@ var MineRenderWorld = class {
       pos.z / 16
     );
   }
-  worldPosToChunkIndex(pos) {
-    const chunkX = Math.floor(pos.x / 16);
-    const chunkY = Math.floor(pos.y / 16);
-    const chunkZ = Math.floor(pos.z / 16);
-    return chunkZ * this._size * this._size + chunkY * this._size + chunkX;
-  }
   worldPosToChunkKey(pos) {
     const chunkX = Math.floor(pos.x / 16);
     const chunkY = Math.floor(pos.y / 16);
@@ -7346,12 +7344,9 @@ var MineRenderWorld = class {
     return `${chunkX}_${chunkY}_${chunkZ}`;
   }
   validatePosBounds(pos) {
-    if (pos.x < 0) throw new Error("x<0");
-    if (pos.y < 0) throw new Error("y<0");
-    if (pos.z < 0) throw new Error("z<0");
-    if (pos.x > this._blockSize) throw new Error("x>" + this._blockSize);
-    if (pos.y > this._blockSize) throw new Error("y>" + this._blockSize);
-    if (pos.z > this._blockSize) throw new Error("z>" + this._blockSize);
+    if (![pos.x, pos.y, pos.z].every(Number.isInteger)) {
+      throw new RangeError("Block coordinates must be integers");
+    }
   }
 };
 
@@ -7364,16 +7359,6 @@ var Chunk = class _Chunk {
     this.x = x;
     this.y = y;
     this.z = z;
-    let anchor = new Object3D6();
-    this._anchor = anchor;
-    scene.add(anchor);
-    anchor.position.set(this.x * 256 + 128 - 8, this.y * 256 + 128 - 8, this.z * 256 + 128 - 8);
-    addBox3WireframeToObject(
-      new Box32(new Vector311(0, 0, 0), new Vector311(256, 256, 256)),
-      anchor,
-      16776960,
-      4
-    );
   }
   getBlockAt(posOrX, y, z) {
     if (typeof posOrX == "number") {
@@ -7399,7 +7384,7 @@ var Chunk = class _Chunk {
     return this.setBlockInChunkAt(pos, block, worldPos);
   }
   /**
-   * Set block at a _chunk_ position (0-16)
+   * Set a block at integer chunk-local coordinates from 0 to 15.
    */
   async setBlockInChunkAt(pos, block, worldPos) {
     if (typeof worldPos === "undefined") {
@@ -7412,15 +7397,13 @@ var Chunk = class _Chunk {
       current.object.removeFromScene();
       delete this._blocks[index];
     }
-    if (typeof block === "undefined" || typeof block.type === "undefined" || block.type === "air") return void 0;
+    if (!block || _Chunk.isAir(block)) return void 0;
     const blockState = await BlockStates.get(AssetKey.parse("blockstates", block.type));
     if (blockState) {
       const blockObject = await this.scene.addBlock(blockState, {
         mergeMeshes: true,
         instanceMeshes: true,
-        wireframe: true,
         maxInstanceCount: 2e3
-        //TODO: setting this to high kills FPS; increase on demand somehow?
       });
       if (block.properties) {
         await blockObject.setState(block.properties);
@@ -7436,23 +7419,21 @@ var Chunk = class _Chunk {
     return void 0;
   }
   async clear() {
-    let promises = [];
-    let pos = new Vector311();
-    for (let x = 0; x < 16; x++) {
-      for (let z = 0; z < 16; z++) {
-        for (let y = 0; y < 16; y++) {
-          promises.push(this.setBlockInChunkAt(pos.set(x, y, z), void 0));
-        }
-      }
-    }
-    return Promise.all(promises).then((ignored) => {
-    });
+    this._blocks.forEach((block) => block.object.removeFromScene());
+    this._blocks.length = 0;
   }
   async dispose() {
     await this.clear();
-    this.scene.remove(this._anchor);
+  }
+  static isAir(block) {
+    if (typeof block === "undefined" || typeof block.type === "undefined") return true;
+    const key = AssetKey.parse("blockstates", block.type);
+    return key.namespace === "minecraft" && !key.type && ["air", "cave_air", "void_air"].includes(key.path);
   }
   static chunkPosToBlockIndex(pos) {
+    if ([pos.x, pos.y, pos.z].some((value) => !Number.isInteger(value) || value < 0 || value >= 16)) {
+      throw new RangeError("Block coordinates must be integers from 0 to 15 within a chunk");
+    }
     return pos.y * 16 * 16 + pos.z * 16 + pos.x;
   }
   worldPosToChunkPos(pos) {