feature/gltf-pipeline-v2 #18

NotEvil · 2026-04-17T02:07:38Z

NotEvil commented

2026-04-17 02:07:38 +00:00

No description provided.

NotEvil added 19 commits 2026-04-17 02:07:39 +00:00

feat(gltf): add suggestBoneName + knownBoneNames helpers to GltfBoneMapper ad74d320be

feat(datadriven): make animation_bones optional — absent means permissive 7ef85b4e92

fix(gltf): select mesh by 'Item' name convention with fallback 6dad447c05

fix(gltf): strip armature prefix from bone names in FurnitureGlbParser d7f8bf6c72

Strip pipe-delimited armature prefixes (e.g., "MyRig|body" -> "body")
from bone names in parseSeatSkeleton and remapSeatAnimations, matching
the existing stripping already present in GlbParser. Without this,
isKnownBone checks fail when Blender exports prefixed bone names.

feat(gltf): accept custom bones — remove 11-bone filter, simplify joint remap 8af58b5dd5

feat(validation): add diagnostic data model — GlbDiagnostic, GlbValidationResult, GlbDiagnosticRegistry eb759fefff

feat(validation): add GlbValidator — structural validation from JSON chunk c0c53f9504

perf(gltf): add skinning cache — skip re-skinning when pose is unchanged 17269f51f8

feat(validation): add GlbValidationReloadListener — validates GLBs on resource reload ca4cbcad12

feat(validation): add /tiedup validate client command for GLB diagnostics 3f6e04edb0

fix(validation): reduce log spam + add OOM guard + check correct mesh for WEIGHTS_0 9dfd2d1724

- DataDrivenItemParser: downgrade animation_bones absence log from INFO to
  DEBUG (was spamming for every item without the optional field)
- GlbValidator: read 12-byte GLB header first and reject files declaring
  totalLength > 50 MB before allocating (prevents OOM on malformed GLBs)
- GlbValidator: WEIGHTS_0 check now uses the same mesh selection logic as
  GlbParser (prefer "Item", fallback to last non-Player) instead of
  blindly checking the first mesh

feat(validation): show toast notification when GLB errors are detected on reload b0766fecc6

fix(animation): free bones only enabled for Full-prefixed animations 229fc66340

Previously, any GLB with keyframes on free bones would animate them,
even for standard animations like Idle. This caused accidental bone
hijacking — e.g., handcuffs freezing the player's head because the
artist keyframed all bones in Blender.

Now the Full prefix (FullIdle, FullStruggle, FullWalk) is enforced:
only Full-prefixed animations can animate free bones. Standard
animations (Idle, Struggle, Walk) only animate owned bones.

This aligns the code with the documented convention in ARTIST_GUIDE.md.

fix(animation): preserve head tracking in Full animations — head never enabled as free bone 3d57d83a5b

feat(animation): add FullHead convention — opt-in head animation in Full animations 806a1e732d

FullStruggle, FullWalk etc. animate body+legs but preserve head tracking.
FullHeadStruggle, FullHeadWalk etc. also animate the head.
The 'Head' keyword in the animation name is the opt-in signal.

fix(animation): extend resolver fallback chain to include FullHead variants e56e6dd551

The resolver now tries FullHead* before Full* at each fallback step.
Example: FullHeadStruggle → FullStruggle → Struggle → FullHeadIdle → FullIdle → Idle

Previously FullHead* names were dead — the resolver never constructed them,
so animations named FullHeadStruggle were unreachable.

fix(animation): variant randomness no longer permanently cached + fix FullHead false-positive a3287b7db8

Two fixes from the animation audit:

1. Variant cache key now includes the resolved animation name (e.g., Struggle.2).
   Previously, the cache key only used context+parts, so the first random variant
   pick was reused forever. Now each variant gets its own cache entry, and a fresh
   random pick happens each time the context changes.

2. FullHead check changed from contains("Head") to startsWith("gltf_FullHead")
   to prevent false positives on names like FullOverhead or FullAhead.

fix(animation): fix variant caching bug in multi-item path — same fix as single-item 168c0675bb

docs(artist-guide): update for Full/FullHead conventions, custom bones, frame 0 behavior, validation tools f37600783a

NotEvil merged commit cc8adfe015 into develop

2026-04-17 02:07:45 +00:00

NotEvil referenced this issue from a commit

2026-04-17 02:07:46 +00:00

Merge pull request 'feature/gltf-pipeline-v2' (#18) from feature/gltf-pipeline-v2 into develop

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: NotEvil/TiedUp-#18