fix: Folia thread-safety for rotation reads and death model cleanup

[arkarang]

Summary

• EntityTracker: updateBaseEntity()에서 asyncTaskLater → taskLater(entity)로 변경 — entity state 읽기를 owning region thread에서 실행
• EntityBodyRotator: pitch(), bodyYaw(), headYaw(), yaw(), onWalk() 직접 호출을 throttleTickFloat/throttleTickBoolean으로 래핑 — EXECUTOR 워커 스레드에서 NMS 핸들 직접 읽기 방지 (Folia 스레드 소유권 위반 해결)
• EntityManager (Bukkit + Fabric): 사망 핸들러에서 forRemoval(true)를 항상 설정 — death 애니메이션 유무와 무관하게 트래커가 사망 상태로 표시됨
• EntityTrackerRegistry#despawn(): forRemoval 트래커를 스킵하지 않고 close() 호출 — 엔티티가 월드에서 제거될 때 확실히 정리

Context

회전값 throttle (EntityBodyRotator)

bodyRotation0(), stableBodyYaw(), head/body supplier 람다에서 entity.pitch(), entity.bodyYaw() 등이 EXECUTOR 10ms tick에서 직접 호출됨. Folia에서는 이 값들이 Region Thread에서만 원자적으로 업데이트되므로, 다른 스레드에서 읽으면 stale read 또는 스레드 소유권 검증 실패 발생.

기존 FunctionUtil.throttleTickFloat 패턴(AtomicLong.compareAndSet + volatile cache)을 활용하여 50ms 간격 thread-safe 캐싱 적용. EntityTracker에서 damageTick, walkSpeed 등은 이미 이 패턴 사용 중이었으나 EntityBodyRotator는 미적용 상태였음.

사망 모델 잔류 (EntityManager + EntityTrackerRegistry)

death 애니메이션이 없는 몹 사망 시:

1. animate("death") → false 반환
2. 기존: forRemoval(true) 미호출 → 트래커 활성 상태 유지
3. EntityRemoveFromWorldEvent → despawn() → forRemoval 안 되어있으니 despawn() 호출 → entity.dead() 시점에 따라 close() 안 될 수 있음

수정 후:

1. forRemoval(true) 항상 설정
2. despawn()에서 forRemoval 트래커를 close() → EntityRemoveFromWorldEvent 시점에 확실히 정리

Test plan

• Folia 서버에서 모델 적용 엔티티 이동/회전 싱크 확인
• 빠른 방향 전환 시 모델 회전 추종 확인
• 사망 애니메이션 없는 몹 사망 → 모델 사라지는지 확인
• 사망 애니메이션 있는 몹 사망 → 애니메이션 재생 후 모델 사라지는지 확인
• 청크 경계에서 몹 사망 → 모델 잔류 없는지 확인
• 서버 로그에 Thread failed main thread check 없는지 확인

🤖 Generated with Claude Code

[toxicity188]

이미 최종값인 head랑 body 부터 throttle이 걸려있는데 왜 넣은 거임?

[arkarang]

AI 피셜:

headSupplier/bodySupplier는 최종 합성값(Quaternionf/ModelRotation)에 throttle이 걸려있는데, throttle이 만료되서 람다가
실행될 때 entity.pitch(), entity.bodyYaw() 등이 EXECUTOR 스레드에서 NMS 핸들(handle().xRot 등)을 직접 읽어요.
EntityTracker에서 damageTick, walkSpeed에 이미 같은 패턴이 적용되어있는 것과 동일한 이유입니다.

그렇데

[toxicity188]

그거 엔티티 사망 때 애니메이션이 있으면 모델 삭제를 미루기 위한 flag에 가까운데 테스트 해본 거 맞음?
death가 없더라도 엔티티가 remove 되기 전까진 사망했다고 안 지워질 텐데

[arkarang]

하하 AI의 대침공 너도 당해봐라

[arkarang]

맞아요 그래서 수정했습니다. 20틱 fallback은 제거하고, forRemoval(true)를 항상 설정 + despawn()에서 forRemoval 트래커를
스킵하지 않고 close() 하도록 변경했어요. death 애니메이션 있으면 콜백에서 close, 없으면 EntityRemoveFromWorldEvent →
despawn() → close() 흐름으로 정리됩니다.

그렇데

[arkarang]

근데 MobModel 관련해서 MythicMobs 쪽 호환성 이슈도 있어보이긴 해서 저도 테스트 좀 더 해봐야할듯.

주말에 테스트하고 결과알랴줌