use more reliable bot entity instead of player.entity

Author: zardoy

Diff for: src/mineflayer/playerState.ts

@@ -77,9 +77,9 @@ export class PlayerStateManager implements IPlayerState {
 
   // #region Movement and Physics State
   private updateState () {
-    if (!bot.player?.entity || this.disableStateUpdates) return
+    if (!bot?.entity || this.disableStateUpdates) return
 
-    const { velocity } = bot.player.entity
+    const { velocity } = bot.entity
     const isOnGround = bot.entity.onGround
     const VELOCITY_THRESHOLD = 0.01
     const SPRINTING_VELOCITY = 0.15
@@ -138,7 +138,7 @@ export class PlayerStateManager implements IPlayerState {
   }
 
   getPosition (): Vec3 {
-    return bot.player?.entity.position ?? new Vec3(0, 0, 0)
+    return bot.entity?.position ?? new Vec3(0, 0, 0)
   }
   // #endregion
 

Diff for: src/sounds/botSoundSystem.ts

@@ -132,9 +132,9 @@ subscribeKey(miscUiState, 'gameLoaded', async () => {
 
   let lastStepSound = 0
   const movementHappening = async () => {
-    if (!bot.player || !soundMap) return // no info yet
+    if (!bot.entity || !soundMap) return // no info yet
     const VELOCITY_THRESHOLD = 0.1
-    const { x, z, y } = bot.player.entity.velocity
+    const { x, z, y } = bot.entity.velocity
     if (bot.entity.onGround && Math.abs(x) < VELOCITY_THRESHOLD && (Math.abs(z) > VELOCITY_THRESHOLD || Math.abs(y) > VELOCITY_THRESHOLD)) {
       // movement happening
       if (Date.now() - lastStepSound > 300) {