@needle-tools/three-animation-pointer 1.1.0 → 1.1.1

package/CHANGELOG.md

@@ -4,6 +4,9 @@ All notable changes to this package will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [1.1.1] - 2026-06-12
+- Fix: morph target (blend shape) weight animation was silently dropped for multi-material meshes without skin/bones. The multi-material morph branch only bound children that were `SkinnedMesh`; such meshes load as a `Group` of plain `Mesh` children, so no track was created (and the empty-but-truthy result skipped the fallback). Now any child with `morphTargetInfluences` is bound.
+
 ## [1.1.0] - 2026-06-01
 - Add: support for `/nodes/{}/weights/{}` to animate individual morph target weights (works with multi-material meshes / primitives) — by @0b5vr in [#1](https://github.com/needle-tools/three-animation-pointer/pull/1)
 

package/README.md

@@ -6,6 +6,7 @@ This package adds a [three.js](https://github.com/mrdoob/three.js) GLTFLoader pl
 - all material properties 💎 (including texture transforms) 
 - cameras 🎥
 - lights 💡
+- morph target weights 😀 (all weights or individual ones via `/nodes/{}/weights/{}`)
 - custom code/scripts 📄
 
 

package/dist/three-animation-pointer.js

@@ -1,17 +1,17 @@
-import { SkinnedMesh as S, InterpolateDiscrete as w, InterpolateLinear as F, AnimationClip as M, PropertyBinding as C, QuaternionKeyframeTrack as W, ColorKeyframeTrack as B, VectorKeyframeTrack as D, BooleanKeyframeTrack as E, NumberKeyframeTrack as O } from "three";
+import { InterpolateDiscrete as w, InterpolateLinear as F, AnimationClip as S, PropertyBinding as C, QuaternionKeyframeTrack as M, ColorKeyframeTrack as W, VectorKeyframeTrack as B, BooleanKeyframeTrack as D, NumberKeyframeTrack as E } from "three";
 const d = {
   node: "node",
   material: "material",
   camera: "camera",
   light: "light"
-}, y = "KHR_animation_pointer", U = {
+}, y = "KHR_animation_pointer", O = {
   // We use a custom interpolant (GLTFCubicSplineInterpolation) for CUBICSPLINE tracks. Each
   // keyframe track will be initialized with a default interpolation type, then modified.
   CUBICSPLINE: void 0,
   LINEAR: F,
   STEP: w
 };
-class G {
+class z {
   /** @type {import("three/examples/jsm/loaders/GLTFLoader").GLTFParser} */
   constructor(a) {
     this.name = y, this.parser = a, this.animationPointerResolver = null;
@@ -29,10 +29,10 @@ class G {
   }
   loadAnimationTargetFromChannelWithAnimationPointer(a) {
     var m;
-    j();
+    L();
     const t = a.target, r = t.extensions && t.extensions[y] && t.path && t.path === "pointer";
     if (!r) return null;
-    let e, n = d.node, s;
+    let e, s = d.node, n;
     if (r) {
       const h = t.extensions[y];
       let o = h.pointer;
@@ -40,13 +40,13 @@ class G {
         console.warn("Invalid path", h, t);
         return;
       }
-      if (o.startsWith("/materials/") ? n = d.material : o.startsWith("/extensions/KHR_lights_punctual/lights/") ? n = d.light : o.startsWith("/cameras/") && (n = d.camera), s = this._tryResolveTargetId(o, n), s === null || isNaN(s)) {
-        console.warn("Failed resolving animation node id: " + s, o);
+      if (o.startsWith("/materials/") ? s = d.material : o.startsWith("/extensions/KHR_lights_punctual/lights/") ? s = d.light : o.startsWith("/cameras/") && (s = d.camera), n = this._tryResolveTargetId(o, s), n === null || isNaN(n)) {
+        console.warn("Failed resolving animation node id: " + n, o);
         return;
       }
-      switch (n) {
+      switch (s) {
         case d.material:
-          const T = ("/materials/" + s.toString() + "/").length, g = o.substring(0, T);
+          const T = ("/materials/" + n.toString() + "/").length, g = o.substring(0, T);
           switch (e = o.substring(T), e) {
             case "pbrMetallicRoughness/baseColorFactor":
               e = "color";
@@ -133,7 +133,7 @@ class G {
           o = g + e;
           break;
         case d.node:
-          const _ = ("/nodes/" + s.toString() + "/").length, b = o.substring(0, _);
+          const _ = ("/nodes/" + n.toString() + "/").length, b = o.substring(0, _);
           if (e = o.substring(_), e.startsWith("weights/")) {
             const f = e.substring(8);
             /^\d+$/.test(f) ? e = "morphTargetInfluences[" + f + "]" : console.warn(y + ": invalid weight index", e, o);
@@ -158,7 +158,7 @@ class G {
           o = b + e;
           break;
         case d.light:
-          const u = ("/extensions/KHR_lights_punctual/lights/" + s.toString() + "/").length;
+          const u = ("/extensions/KHR_lights_punctual/lights/" + n.toString() + "/").length;
           switch (e = o.substring(u), e) {
             case "color":
               break;
@@ -174,10 +174,10 @@ class G {
               e = "distance";
               break;
           }
-          o = "/lights/" + s.toString() + "/" + e;
+          o = "/lights/" + n.toString() + "/" + e;
           break;
         case d.camera:
-          const l = ("/cameras/" + s.toString() + "/").length, k = o.substring(0, l);
+          const l = ("/cameras/" + n.toString() + "/").length, k = o.substring(0, l);
           switch (e = o.substring(l), e) {
             case "perspective/yfov":
               e = "fov";
@@ -205,17 +205,17 @@ class G {
       }
       (m = this.animationPointerResolver) != null && m.resolvePath && (o = this.animationPointerResolver.resolvePath(o)), t.extensions[y].pointer = o;
     }
-    if (s == null || isNaN(s)) {
-      console.warn("Failed resolving animation node id: " + s, t);
+    if (n == null || isNaN(n)) {
+      console.warn("Failed resolving animation node id: " + n, t);
       return;
     }
     let i;
-    return n === d.node ? i = this.parser.getDependency("node", s) : n === d.material ? i = this.parser.getDependency("material", s) : n === d.light ? i = this.parser.getDependency("light", s) : n === d.camera ? i = this.parser.getDependency("camera", s) : console.error("Unhandled type", n), i;
+    return s === d.node ? i = this.parser.getDependency("node", n) : s === d.material ? i = this.parser.getDependency("material", n) : s === d.light ? i = this.parser.getDependency("light", n) : s === d.camera ? i = this.parser.getDependency("camera", n) : console.error("Unhandled type", s), i;
   }
-  createAnimationTracksWithAnimationPointer(a, t, r, e, n) {
+  createAnimationTracksWithAnimationPointer(a, t, r, e, s) {
     var b;
-    if (!(n.extensions && n.extensions[y] && n.path && n.path === "pointer")) return null;
-    let i = n.extensions[y].pointer;
+    if (!(s.extensions && s.extensions[y] && s.path && s.path === "pointer")) return null;
+    let i = s.extensions[y].pointer;
     if (!i) return null;
     const m = [];
     i = i.replaceAll("/", ".");
@@ -225,7 +225,7 @@ class G {
       const u = h[3];
       if (a.type === "Group") {
         for (const l of a.children)
-          l instanceof S && l.morphTargetInfluences && (h[3] = l.name, h[4] = u, _(this.parser));
+          l.morphTargetInfluences && (h[3] = l.name, h[4] = u, _(this.parser));
         return m;
       }
     }
@@ -244,21 +244,21 @@ class G {
       switch (r.itemSize) {
         case 1:
           Object.prototype.toString.call(r.array) === "[object Uint8Array]" && // UNSIGNED_BYTE as a strong hint
-          g(a, i) ? (l = E, k = !0) : l = O;
+          g(a, i) ? (l = D, k = !0) : l = E;
           break;
         case 2:
         case 3:
-          l = D;
+          l = B;
           break;
         case 4:
-          i.endsWith(".quaternion") ? l = W : l = B;
+          i.endsWith(".quaternion") ? l = M : l = W;
           break;
       }
       if (!l) {
         console.warn("Unsupported output accessor format", r);
         return;
       }
-      const f = e.interpolation !== void 0 ? U[e.interpolation] : F;
+      const f = e.interpolation !== void 0 ? O[e.interpolation] : F;
       let c = u._getArrayFromAccessor(r);
       i.endsWith(".fov") && (c = c.map((p) => p / Math.PI * 180)), k && (c = c.map((p) => p > 0));
       const R = new l(
@@ -293,9 +293,9 @@ class G {
    * @return {Promise<AnimationClip>}
    */
   loadAnimation(a) {
-    const t = this, r = this.parser.json, e = this.parser, n = r.animations[a], s = n.name ? n.name : "animation_" + a, i = [], m = [], h = [], o = [], T = [];
-    for (let g = 0, _ = n.channels.length; g < _; g++) {
-      const b = n.channels[g], u = n.samplers[b.sampler], l = b.target, k = n.parameters !== void 0 ? n.parameters[u.input] : u.input, f = n.parameters !== void 0 ? n.parameters[u.output] : u.output;
+    const t = this, r = this.parser.json, e = this.parser, s = r.animations[a], n = s.name ? s.name : "animation_" + a, i = [], m = [], h = [], o = [], T = [];
+    for (let g = 0, _ = s.channels.length; g < _; g++) {
+      const b = s.channels[g], u = s.samplers[b.sampler], l = b.target, k = s.parameters !== void 0 ? s.parameters[u.input] : u.input, f = s.parameters !== void 0 ? s.parameters[u.output] : u.output;
       let c = t.loadAnimationTargetFromChannelWithAnimationPointer(b);
       c || (c = t.loadAnimationTargetFromChannel(b)), i.push(c), m.push(e.getDependency("accessor", k)), h.push(e.getDependency("accessor", f)), o.push(u), T.push(l);
     }
@@ -316,42 +316,42 @@ class G {
           for (let K = 0; K < I.length; K++)
             f.push(I[K]);
       }
-      return new M(s, void 0, f);
+      return new S(n, void 0, f);
     });
   }
 }
-let H = !1, L = null;
-function j() {
+let H = !1, U = null;
+function L() {
   if (H) return;
   H = !0;
-  const P = L || (L = C.findNode);
+  const P = U || (U = C.findNode);
   C.findNode = function(a, t) {
     if (!t) return P(a, t);
     if (t.startsWith(".materials.")) {
-      const r = t.substring(11).substring(t.indexOf(".")), e = r.indexOf("."), n = e < 0 ? r : r.substring(0, e);
-      let s = null;
+      const r = t.substring(11).substring(t.indexOf(".")), e = r.indexOf("."), s = e < 0 ? r : r.substring(0, e);
+      let n = null;
       return a.traverse((i) => {
-        s !== null || i.type !== "Mesh" && i.type !== "SkinnedMesh" || i.material && (i.material.uuid === n || i.material.name === n) && (s = i.material, s !== null && (r.endsWith(".map") ? s = s.map : r.endsWith(".emissiveMap") && (s = s.emissiveMap)));
-      }), s;
+        n !== null || i.type !== "Mesh" && i.type !== "SkinnedMesh" || i.material && (i.material.uuid === s || i.material.name === s) && (n = i.material, n !== null && (r.endsWith(".map") ? n = n.map : r.endsWith(".emissiveMap") && (n = n.emissiveMap)));
+      }), n;
     } else if (t.startsWith(".nodes.") || t.startsWith(".lights.") || t.startsWith(".cameras.")) {
       const r = t.split(".");
       let e;
-      for (let n = 1; n < r.length; n++) {
-        const s = r[n];
-        if (s.length == 36)
-          e = a.getObjectByProperty("uuid", s);
-        else if (e && e[s]) {
-          const m = Number.parseInt(s);
-          let h = s;
+      for (let s = 1; s < r.length; s++) {
+        const n = r[s];
+        if (n.length == 36)
+          e = a.getObjectByProperty("uuid", n);
+        else if (e && e[n]) {
+          const m = Number.parseInt(n);
+          let h = n;
           m >= 0 && (h = m), e = e[h];
         } else {
-          const m = a.getObjectByName(s);
+          const m = a.getObjectByName(n);
           m && (e = m);
         }
       }
       if (!e) {
-        const n = P(a, r[2]);
-        return n || console.warn(y + ": Property binding not found", t, a, a.name, r), n;
+        const s = P(a, r[2]);
+        return s || console.warn(y + ": Property binding not found", t, a, a.name, r), s;
       }
       return e;
     }
@@ -359,6 +359,6 @@ function j() {
   };
 }
 export {
-  G as GLTFAnimationPointerExtension,
-  G as default
+  z as GLTFAnimationPointerExtension,
+  z as default
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@needle-tools/three-animation-pointer",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "KHR_animation_pointer support for three.js",
   "type": "module",
   "main": "dist/three-animation-pointer.js",
@@ -46,4 +46,4 @@
     "registry": "https://registry.npmjs.org/",
     "access": "public"
   }
-}
+}

package/src/GLTFLoaderAnimationPointer.js

@@ -7,7 +7,6 @@ import {
 	PropertyBinding,
 	QuaternionKeyframeTrack,
 	VectorKeyframeTrack,
-	SkinnedMesh,
 	BooleanKeyframeTrack
 } from 'three';
 
@@ -408,12 +407,13 @@ export class GLTFAnimationPointerExtension {
 			if ( node.type === 'Group' ) {
 
 				if ( _animationPointerDebug )
-					console.log( 'Detected multi-material skinnedMesh export', animationPointerPropertyPath, node );
+					console.log( 'Detected multi-material mesh export', animationPointerPropertyPath, node );
 
-				// We assume the children are skinned meshes
+				// Bind any child that has morph influences. The children may be plain
+				// Mesh (no skin/bones) or SkinnedMesh, so we must not require SkinnedMesh.
 				for ( const ch of node.children ) {
 
-					if ( ch instanceof SkinnedMesh && ch.morphTargetInfluences ) {
+					if ( ch.morphTargetInfluences ) {
 
 						parts[ 3 ] = ch.name;
 						parts[ 4 ] = property;