@umicat/three-sdk 0.16.0 → 0.16.1

package/dist/SceneLoader3D.js

@@ -315,15 +315,32 @@ export async function loadScene3D(scene3d, manifest, opts = {}) {
         dispose() {
             for (const m of mixers)
                 m.stopAllAction();
+            // Textures were never freed here — only geometries/materials — because
+            // every prior consumer (a game) loads a scene once per session. The
+            // Scene Previewer (platform-ui) reloads on every click-to-a-different-
+            // scene and every manual refresh, so the leak that used to be
+            // session-lifetime now actually accumulates. A Set guards against
+            // disposing a texture twice when two materials share one (e.g. an
+            // atlas'd model).
+            const disposedTextures = new Set();
+            const disposeMaterial = (mat) => {
+                for (const value of Object.values(mat)) {
+                    if (value instanceof THREE.Texture && !disposedTextures.has(value)) {
+                        disposedTextures.add(value);
+                        value.dispose();
+                    }
+                }
+                mat.dispose();
+            };
             scene.traverse((o) => {
                 const mesh = o;
                 if (mesh.geometry)
                     mesh.geometry.dispose();
                 const mat = mesh.material;
                 if (Array.isArray(mat))
-                    mat.forEach((m) => m.dispose());
-                else
-                    mat?.dispose();
+                    mat.forEach(disposeMaterial);
+                else if (mat)
+                    disposeMaterial(mat);
             });
         },
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umicat/three-sdk",
-  "version": "0.16.0",
+  "version": "0.16.1",
   "description": "Three.js runtime for Umicat games: the scene3d design format, its loader with physics, a kinematic character controller, and the Umicat platform via @umicat/platform-sdk.",
   "type": "module",
   "main": "dist/index.js",