@needle-tools/three-animation-pointer 1.0.5 → 1.0.6

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.0.5] - 2025-10-15
+- Fix: Remove leftover log
+
 ## [1.0.5] - 2025-10-15
 - Add: support for boolean animation tracks (when using ubyte type)
 

package/dist/three-animation-pointer.js

@@ -1,4 +1,4 @@
-import { SkinnedMesh as S, InterpolateDiscrete as M, InterpolateLinear as F, AnimationClip as W, PropertyBinding as C, QuaternionKeyframeTrack as w, ColorKeyframeTrack as D, VectorKeyframeTrack as B, BooleanKeyframeTrack as E, NumberKeyframeTrack as O } from "three";
+import { SkinnedMesh as S, InterpolateDiscrete as M, InterpolateLinear as F, AnimationClip as W, PropertyBinding as C, QuaternionKeyframeTrack as w, ColorKeyframeTrack as B, VectorKeyframeTrack as D, BooleanKeyframeTrack as E, NumberKeyframeTrack as O } from "three";
 const d = {
   node: "node",
   material: "material",
@@ -29,7 +29,7 @@ class G {
   }
   loadAnimationTargetFromChannelWithAnimationPointer(a) {
     var u;
-    L();
+    j();
     const t = a.target, r = t.extensions && t.extensions[R] && t.path && t.path === "pointer";
     if (!r) return null;
     let e, n = d.node, s;
@@ -46,7 +46,7 @@ class G {
       }
       switch (n) {
         case d.material:
-          const T = ("/materials/" + s.toString() + "/").length, g = o.substring(0, T);
+          const T = ("/materials/" + s.toString() + "/").length, f = o.substring(0, T);
           switch (e = o.substring(T), e) {
             case "pbrMetallicRoughness/baseColorFactor":
               e = "color";
@@ -130,7 +130,7 @@ class G {
               e = "specularColor";
               break;
           }
-          o = g + e;
+          o = f + e;
           break;
         case d.node:
           const k = ("/nodes/" + s.toString() + "/").length, l = o.substring(0, k);
@@ -173,8 +173,8 @@ class G {
           o = "/lights/" + s.toString() + "/" + e;
           break;
         case d.camera:
-          const f = ("/cameras/" + s.toString() + "/").length, b = o.substring(0, f);
-          switch (e = o.substring(f), e) {
+          const g = ("/cameras/" + s.toString() + "/").length, b = o.substring(0, g);
+          switch (e = o.substring(g), e) {
             case "perspective/yfov":
               e = "fov";
               break;
@@ -222,9 +222,9 @@ class G {
       return u;
     }
     k(this.parser);
-    function g(l, c) {
+    function f(l, c) {
       try {
-        const f = c.split(".").filter(Boolean), m = f[f.length - 1].replace(/\[.*\]$/, "");
+        const g = c.split(".").filter(Boolean), m = g[g.length - 1].replace(/\[.*\]$/, "");
         return m in l ? typeof l[m] == "boolean" : !1;
       } catch {
         return !1;
@@ -232,19 +232,18 @@ class G {
     }
     function k(l) {
       i = h.join(".");
-      let c, f = !1;
+      let c, g = !1;
       switch (r.itemSize) {
         case 1:
-          const _ = Object.prototype.toString.call(r.array) === "[object Uint8Array]" && // UNSIGNED_BYTE as a strong hint
-          g(a, i);
-          _ ? console.log("Assuming boolean animation for", i, "based on property type and accessor type") : console.log("Not a boolean target", i), _ ? (c = E, f = !0) : c = O;
+          Object.prototype.toString.call(r.array) === "[object Uint8Array]" && // UNSIGNED_BYTE as a strong hint
+          f(a, i) ? (c = E, g = !0) : c = O;
           break;
         case 2:
         case 3:
-          c = B;
+          c = D;
           break;
         case 4:
-          i.endsWith(".quaternion") ? c = w : c = D;
+          i.endsWith(".quaternion") ? c = w : c = B;
           break;
       }
       if (!c) {
@@ -253,7 +252,7 @@ class G {
       }
       const b = e.interpolation !== void 0 ? U[e.interpolation] : F;
       let m = l._getArrayFromAccessor(r);
-      i.endsWith(".fov") && (m = m.map((x) => x / Math.PI * 180)), f && (m = m.map((x) => x > 0));
+      i.endsWith(".fov") && (m = m.map((_) => _ / Math.PI * 180)), g && (m = m.map((_) => _ > 0));
       const p = new c(
         i,
         t.array,
@@ -261,16 +260,16 @@ class G {
         b
       );
       if (b === "CUBICSPLINE" && l._createCubicSplineTrackInterpolant(p), u.push(p), i && r.itemSize === 4 && i.startsWith(".materials.") && i.endsWith(".color")) {
-        const x = new Float32Array(m.length / 4);
-        for (let _ = 0, P = m.length / 4; _ < P; _ += 1)
-          x[_] = m[_ * 4 + 3];
-        const y = new c(
+        const _ = new Float32Array(m.length / 4);
+        for (let y = 0, P = m.length / 4; y < P; y += 1)
+          _[y] = m[y * 4 + 3];
+        const x = new c(
           i.replace(".color", ".opacity"),
           t.array,
-          x,
+          _,
           b
         );
-        b === "CUBICSPLINE" && l._createCubicSplineTrackInterpolant(p), u.push(y);
+        b === "CUBICSPLINE" && l._createCubicSplineTrackInterpolant(p), u.push(x);
       }
     }
     return u;
@@ -287,10 +286,10 @@ class G {
    */
   loadAnimation(a) {
     const t = this, r = this.parser.json, e = this.parser, n = r.animations[a], s = n.name ? n.name : "animation_" + a, i = [], u = [], h = [], o = [], T = [];
-    for (let g = 0, k = n.channels.length; g < k; g++) {
-      const l = n.channels[g], c = n.samplers[l.sampler], f = l.target, b = n.parameters !== void 0 ? n.parameters[c.input] : c.input, m = n.parameters !== void 0 ? n.parameters[c.output] : c.output;
+    for (let f = 0, k = n.channels.length; f < k; f++) {
+      const l = n.channels[f], c = n.samplers[l.sampler], g = l.target, b = n.parameters !== void 0 ? n.parameters[c.input] : c.input, m = n.parameters !== void 0 ? n.parameters[c.output] : c.output;
       let p = t.loadAnimationTargetFromChannelWithAnimationPointer(l);
-      p || (p = t.loadAnimationTargetFromChannel(l)), i.push(p), u.push(e.getDependency("accessor", b)), h.push(e.getDependency("accessor", m)), o.push(c), T.push(f);
+      p || (p = t.loadAnimationTargetFromChannel(l)), i.push(p), u.push(e.getDependency("accessor", b)), h.push(e.getDependency("accessor", m)), o.push(c), T.push(g);
     }
     return Promise.all([
       Promise.all(i),
@@ -298,14 +297,14 @@ class G {
       Promise.all(h),
       Promise.all(o),
       Promise.all(T)
-    ]).then(function(g) {
-      const k = g[0], l = g[1], c = g[2], f = g[3], b = g[4], m = [];
-      for (let p = 0, x = k.length; p < x; p++) {
-        const y = k[p], _ = l[p], P = c[p], N = f[p], K = b[p];
-        if (y === void 0) continue;
-        y.updateMatrix && (y.updateMatrix(), y.matrixAutoUpdate = !0);
-        let v = t.createAnimationTracksWithAnimationPointer(y, _, P, N, K);
-        if (v || (v = e._createAnimationTracks(y, _, P, N, K)), v)
+    ]).then(function(f) {
+      const k = f[0], l = f[1], c = f[2], g = f[3], b = f[4], m = [];
+      for (let p = 0, _ = k.length; p < _; p++) {
+        const x = k[p], y = l[p], P = c[p], K = g[p], N = b[p];
+        if (x === void 0) continue;
+        x.updateMatrix && (x.updateMatrix(), x.matrixAutoUpdate = !0);
+        let v = t.createAnimationTracksWithAnimationPointer(x, y, P, K, N);
+        if (v || (v = e._createAnimationTracks(x, y, P, K, N)), v)
           for (let A = 0; A < v.length; A++)
             m.push(v[A]);
       }
@@ -313,11 +312,11 @@ class G {
     });
   }
 }
-let H = !1, j = null;
-function L() {
+let H = !1, L = null;
+function j() {
   if (H) return;
   H = !0;
-  const I = j || (j = C.findNode);
+  const I = L || (L = C.findNode);
   C.findNode = function(a, t) {
     if (!t) return I(a, t);
     if (t.startsWith(".materials.")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@needle-tools/three-animation-pointer",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "KHR_animation_pointer support for three.js",
   "type": "module",
   "main": "dist/three-animation-pointer.js",

package/src/GLTFLoaderAnimationPointer.js

@@ -457,15 +457,6 @@ export class GLTFAnimationPointerExtension {
 
 					const looksLikeBool = isUInt8 && // UNSIGNED_BYTE as a strong hint
 						isBooleanTarget(node, animationPointerPropertyPath);
-
-						// Log once per target property if it looks like a boolean
-						if (looksLikeBool) {
-							console.log('Assuming boolean animation for', animationPointerPropertyPath, 'based on property type and accessor type');
-						}
-						else
-						{
-							console.log('Not a boolean target', animationPointerPropertyPath);
-						}
 					if (looksLikeBool) {
 						TypedKeyframeTrack = BooleanKeyframeTrack;
 						convertToBoolean = true; // map 0 => false, >0 => true