Huge delay using Navmesh

[tight swallow]

Player NavMeshController

using Mirror;
using UnityEngine;
using UnityEngine.AI;

namespace Gameplay.Features.Avatars
{
    public class PlayerNavMeshMovement : NetworkBehaviour
    {
        public NavMeshAgent Agent { get; private set; }

        [Header("Movement")] 
        public Camera playerCamera;
        [SerializeField] private LayerMask _layerMask;

        void Awake()
        {
            Agent = GetComponent<NavMeshAgent>();
        }
        
        public override void OnStartClient()
        {
            if (Agent != null)
            {
                Agent.enabled = true;

                if (!isLocalPlayer)
                {
                    Agent.updatePosition = false;
                    Agent.updateRotation = false;
                }
            }
        }
        
        // Runs only on the server when the object spawns
        public override void OnStartServer()
        {
            if (Agent != null) Agent.enabled = true;
        }

        public override void OnStartLocalPlayer()
        {
            playerCamera = Camera.main;
        }

        void Update()
        {
            if (!isLocalPlayer) return;
            
            if (Input.GetMouseButtonDown(0))
            {
                if (Physics.Raycast(playerCamera.ScreenPointToRay(Input.mousePosition), out RaycastHit hit, 2000f, _layerMask))
                {
                    Agent.SetDestination(hit.point);
                    CmdMoveToPoint(hit.point);
                }
            }
        }

        [Command]
        void CmdMoveToPoint(Vector3 destination)
        {
            Agent.SetDestination(destination);
        }
    }
}

[wary anvil]

Agent should be disabled in the player prefab and only enabled in OnStartServer override, and only set destination inside the CmdMoveToPoint, not in clients at all. Remove what you have in OnStartClient entirely. Also, assign the agent in OnValidate instead of Awake.

[tight swallow]

Thanks @wary anvil I was trying to use that mixed approach to have a kind of client side prediction so players will have immediate feedback, but I think I've failed miserably. Really appreaciate your help. You don't remember me but like 5 years ago you helped me with my graduation thesis 🥲

Btw, is there a way to have immediate movement on the client like I tried?

[wary anvil]

Sure...change everything to Client-To-Server, only enable the agent in OnStartLocalPlayer along with the player move script, let NetworkTransform update the server and other clients. You can still spot check on the server for players being off the navmesh.