refactor(Core/Movement): Remove 70y dispatch cap and isMoving lastPath guard

src/Ai/Base/Actions/MovementActions.cpp

@@ -3334,33 +3334,18 @@ bool MovementAction::ExecuteTravelPlan(TravelPlan& state)
         case PathNodeType::NODE_NODE:
         {
             // Batch consecutive walk points into one spline. Capped at
-            // 20 points OR ~70y of accumulated distance — whichever
+            // 20 points per dispatch as a cheap upper bound on per-tick
-            // comes first. The distance cap gives the planner regular
+            // work; stepIdx advances exactly in step with what's
-            // re-evaluation points without committing the whole
+            // dispatched, so the next tick picks up from the cutoff.
-            // remaining route up front; stepIdx advances exactly in
-            // step with what's actually dispatched, so the next tick
-            // picks up from the truncation point.
             static constexpr uint32 MAX_SPLINE_POINTS = 20;
-            static constexpr float MAX_BATCH_LENGTH = 70.0f;
             state.walkPoints.clear();
-            float accumulated = 0.f;
             while (state.stepIdx < state.steps.size() && state.walkPoints.size() < MAX_SPLINE_POINTS)
             {
                 const PathNodePoint& wp = state.steps[state.stepIdx];
                 if (wp.type != PathNodeType::NODE_PATH && wp.type != PathNodeType::NODE_NODE)
                     break;
-                G3D::Vector3 next(wp.point.GetPositionX(), wp.point.GetPositionY(), wp.point.GetPositionZ());
+                state.walkPoints.push_back(G3D::Vector3(wp.point.GetPositionX(),
-                if (!state.walkPoints.empty())
+                    wp.point.GetPositionY(), wp.point.GetPositionZ()));
-                {
-                    accumulated += (next - state.walkPoints.back()).length();
-                    if (accumulated >= MAX_BATCH_LENGTH)
-                    {
-                        state.walkPoints.push_back(next);
-                        state.stepIdx++;
-                        break;
-                    }
-                }
-                state.walkPoints.push_back(next);
                 state.stepIdx++;
             }
 

src/Ai/World/Rpg/Action/NewRpgBaseAction.cpp

@@ -78,17 +78,12 @@ bool NewRpgBaseAction::MoveFarTo(WorldPosition dest)
         }
     }
 
-    // 10% lastPath reuse — route commitment across combat
+    // 10% lastPath reuse — if the cached path's endpoint is still
-    // interruptions. If the cached path's endpoint is still close
+    // close (within 10%) to the new dest AND we're not nearly there,
-    // (within 10%) to the new dest AND bot is mid-flight toward it
+    // keep the existing route. Matches cmangos ResolveMovePath.
-    // (>10y away AND currently moving), reuse silently. The
-    // bot->isMoving() guard prevents reuse from short-circuiting
-    // when bot is stopped between dispatches — in that case we
-    // MUST fall through to dispatch the next leg, not return true
-    // and stand still.
     {
         LastMovement& lastMove = AI_VALUE(LastMovement&, "last movement");
-        if (bot->isMoving() && !lastMove.lastPath.empty())
+        if (!lastMove.lastPath.empty())
         {
             WorldPosition lastBack = lastMove.lastPath.getBack();
             if (lastBack.GetMapId() == dest.GetMapId())
@@ -177,30 +172,6 @@ bool NewRpgBaseAction::MoveFarTo(WorldPosition dest)
 
             if (points.size() >= 2)
             {
-                // Cap dispatched path length at ~70y. MoveFarTo's
-                // early-exit (top of function) lets the active spline
-                // run until bot is within 10y of its endpoint, then
-                // replans from the new position. Capping per-dispatch
-                // distance gives the planner regular re-evaluation
-                // points without the per-tick replan cost of fully
-                // unbounded chunks.
-                {
-                    constexpr float maxDispatchLength = 70.0f;
-                    float accumulated = 0.f;
-                    size_t cutoff = points.size();
-                    for (size_t i = 1; i < points.size(); ++i)
-                    {
-                        accumulated += (points[i] - points[i - 1]).length();
-                        if (accumulated >= maxDispatchLength)
-                        {
-                            cutoff = i + 1;
-                            break;
-                        }
-                    }
-                    if (cutoff < points.size())
-                        points.resize(cutoff);
-                }
-
                 LOG_INFO("playerbots", "[MoveFar] {} mmap-path | dis={:.0f} | endDist={:.0f} | wp={}",
                     bot->GetName(), dis, endDistToDest, (uint32)points.size());
                 EmitDebugMove("MoveFar", "mmap",