Update LootGoal to prevent bot getting stuck

Core/ClassConfig/ClassConfiguration.cs

@@ -238,9 +238,8 @@ private static void SetBaseActions(
 
                 user.Key = baseAction.Key;
 
-                //if (!string.IsNullOrEmpty(@default.Requirement))
-                //    user.Requirement += " " + @default.Requirement;
-                //user.Requirements.AddRange(@default.Requirements);
+                if (!string.IsNullOrEmpty(baseAction.Requirement))
+                    user.Requirement += " " + baseAction.Requirement;
 
                 if (user.BeforeCastDelay == @default.BeforeCastDelay)
                     user.BeforeCastDelay = baseAction.BeforeCastDelay;

Core/ClassConfig/ClassConfigurationBaseActions.cs

@@ -34,14 +34,16 @@ public sealed partial class ClassConfiguration
         Key = "I", // Interact.Key
         Name = nameof(Approach),
         PressDuration = 10,
-        BaseAction = true
+        BaseAction = true,
+        Requirement = "!SoftTargetDead"
     };
 
     public KeyAction AutoAttack { get; } = new()
     {
         Key = "I", // Interact.Key
         Name = nameof(AutoAttack),
-        BaseAction = true
+        BaseAction = true,
+        Requirement = "!AutoAttacking && !SoftTargetDead"
     };
 
     public KeyAction TargetLastTarget { get; } = new()

Core/Goals/CombatGoal.cs

@@ -203,6 +203,7 @@ private void FindNewTarget()
                     ResetCooldowns();
 
                     logger.LogWarning("Found new target!");
+                    wait.Update();
                     return;
                 }
 

Core/Goals/ConsumeCorpseGoal.cs

@@ -23,9 +23,8 @@ public ConsumeCorpseGoal(ILogger<ConsumeCorpseGoal> logger,
         {
             AddPrecondition(GoapKey.consumablecorpsenearby, true);
         }
-        AddPrecondition(GoapKey.pulled, false);
-        AddPrecondition(GoapKey.dangercombat, false);
-        AddPrecondition(GoapKey.incombat, false);
+        AddPrecondition(GoapKey.damagedone, false);
+        AddPrecondition(GoapKey.damagetaken, false);
 
         AddPrecondition(GoapKey.producedcorpse, true);
         AddPrecondition(GoapKey.consumecorpse, false);

Core/Goals/LootGoal.cs

@@ -18,6 +18,7 @@ public sealed partial class LootGoal : GoapGoal, IGoapEventListener
 
     private const int MAX_TIME_TO_REACH_MELEE = 10000;
     private const int MAX_TIME_TO_DETECT_LOOT = 2 * CastingHandler.GCD;
+    private const int MAX_TIME_TO_RESET_LOOT = 800;
 
     private readonly ILogger<LootGoal> logger;
     private readonly ConfigurableInput input;
@@ -68,14 +69,20 @@ public LootGoal(ILogger<LootGoal> logger,
         this.state = state;
 
         this.token = cts.Token;
+        AddPrecondition(GoapKey.pulled, false);
         AddPrecondition(GoapKey.dangercombat, false);
         AddPrecondition(GoapKey.shouldloot, true);
         AddEffect(GoapKey.shouldloot, false);
     }
 
     public override void OnEnter()
     {
-        WaitForLootReset();
+        float elapsedMs = WaitForLootReset();
+        if (elapsedMs < 0)
+        {
+            LogLootStatusDidNotChangedInTime(logger, elapsedMs);
+            return;
+        }
 
         if (combatLog.DamageTakenCount() == 0)
         {
@@ -100,9 +107,9 @@ public override void OnEnter()
         ClearTargetIfNeeded();
     }
 
-    private void WaitForLootReset()
+    private float WaitForLootReset()
     {
-        wait.While(LootReset);
+        return wait.Until(MAX_TIME_TO_RESET_LOOT, LootStatusIsCorpse);
     }
 
     private void WaitForLosingTarget()
@@ -121,6 +128,7 @@ private void CaptureStateBeforeLoot()
 
     private void CheckInventoryFull()
     {
+        ClearTargetIfNeeded();
         if (!bagReader.BagsFull())
             return;
 
@@ -147,10 +155,8 @@ private bool TryLoot()
                 return true;
             }
         }
-        if (!input.KeyboardOnly) {
-            return LootMouse();
-        }
-        return false;
+
+        return !input.KeyboardOnly && LootMouse();
     }
 
     private void HandleSuccessfulLoot()
@@ -396,8 +402,8 @@ private bool MoveToTargetAndReached()
         return bits.Target() && playerReader.MinRangeZero();
     }
 
-    private bool LootReset() =>
-        (LootStatus)playerReader.LootEvent.Value != LootStatus.CORPSE;
+    private bool LootStatusIsCorpse() =>
+        (LootStatus)playerReader.LootEvent.Value == LootStatus.CORPSE;
 
     #region Logging
 
@@ -453,5 +459,11 @@ private void LogWarning(string text)
         Message = "Keyboard loot failed! Has target ? {hasTarget}")]
     static partial void LogKeyboardLootFailed(ILogger logger, bool hasTarget);
 
+    [LoggerMessage(
+        EventId = 0137,
+        Level = LogLevel.Error,
+        Message = "LootGoal failed to start due LootStatus did not changed within the expected time window {elapsedMs}ms")]
+    static partial void LogLootStatusDidNotChangedInTime(ILogger logger, float elapsedMs);
+
     #endregion
 }

Core/Requirement/RequirementFactory.cs

@@ -143,6 +143,10 @@ public RequirementFactory(IServiceProvider sp, ClassConfiguration classConfig)
             { "TargetsPet", playerReader.TargetsPet },
             { "TargetsNone", playerReader.TargetsNone },
 
+            // Soft Target
+            { "SoftTarget", bits.SoftInteract },
+            { "SoftTargetDead", bits.SoftInteract_Dead },
+
             { AddVisible, npcNameFinder._PotentialAddsExist },
             { "InCombat", bits.Combat },
 

README.md

@@ -1642,6 +1642,8 @@ Allow requirements about what buffs/debuffs you have or the target has or in gen
 | `"TargetsMe"` | The target currently targets the player |
 | `"TargetsPet"` | The target currently targets the player's pet |
 | `"TargetsNone"` | The target currently has not target |
+| `"SoftTarget"` | The player has an available soft target |
+| `"SoftTargetDead"` | The player has an available soft target which is dead |
 | `"AddVisible"` | Around the target there are possible additional NPCs |
 | `"InCombat"` | Player in combat. |
 | `"TargetCastingSpell"` | Target casts any spell |