Make AcceptQuest/CompleteQuest skippable if their relevant quest has been accepted/completed

QuestPaths/quest-v1.json

@@ -184,7 +184,8 @@
                       "ChocoboUnlocked",
                       "AetheryteShortcutIfInSameTerritory",
                       "NotTargetable",
-                      "ItemNotInInventory"
+                      "ItemNotInInventory",
+                      "WakingSandsMainArea"
                     ]
                   }
                 },

Questionable.Model/V1/Converter/SkipConditionConverter.cs

@@ -13,5 +13,6 @@ public sealed class SkipConditionConverter() : EnumConverter<ESkipCondition>(Val
         { ESkipCondition.AetheryteShortcutIfInSameTerritory, "AetheryteShortcutIfInSameTerritory" },
         { ESkipCondition.NotTargetable, "NotTargetable" },
         { ESkipCondition.ItemNotInInventory, "ItemNotInInventory" },
+        { ESkipCondition.WakingSandsMainArea, "WakingSandsMainArea" },
     };
 }

Questionable.Model/V1/ESkipCondition.cs

@@ -14,4 +14,7 @@ public enum ESkipCondition
     AetheryteShortcutIfInSameTerritory,
     NotTargetable,
     ItemNotInInventory,
+
+    // TODO: This is an indication the whole skip bit should be optimized/parameterized to some extent
+    WakingSandsMainArea,
 }

Questionable/Controller/Steps/Shared/SkipCondition.cs

@@ -28,7 +28,9 @@ internal static class SkipCondition
                 step.SkipIf.Where(x => x != ESkipCondition.AetheryteShortcutIfInSameTerritory).ToList();
             if (relevantConditions.Count == 0 &&
                 step.CompletionQuestVariablesFlags.Count == 0 &&
-                step.RequiredQuestVariables.Count == 0)
+                step.RequiredQuestVariables.Count == 0 &&
+                step.PickUpQuestId == null &&
+                step.NextQuestId == null)
                 return null;
 
             return serviceProvider.GetRequiredService<CheckTask>()
@@ -143,6 +145,28 @@ internal static class SkipCondition
                 }
             }
 
+            if (Step.SkipIf.Contains(ESkipCondition.WakingSandsMainArea))
+            {
+                var position = clientState.LocalPlayer!.Position;
+                if (position.X < 24)
+                {
+                    logger.LogInformation("Skipping step, as we're not in the Solar");
+                    return true;
+                }
+            }
+
+            if (Step.PickUpQuestId != null && gameFunctions.IsQuestAcceptedOrComplete(Step.PickUpQuestId.Value))
+            {
+                logger.LogInformation("Skipping step, as we have already picked up the relevant quest");
+                return true;
+            }
+
+            if (Step.TurnInQuestId != null && gameFunctions.IsQuestComplete(Step.TurnInQuestId.Value))
+            {
+                logger.LogInformation("Skipping step, as we have already completed the relevant quest");
+                return true;
+            }
+
             return false;
         }
 

Questionable/Controller/Steps/Shared/WaitAtEnd.cs

@@ -108,20 +108,26 @@ internal static class WaitAtEnd
                     ];
 
                 case EInteractionType.AcceptQuest:
-                    return
+                {
-                    [
+                    var accept = serviceProvider.GetRequiredService<WaitQuestAccepted>()
-                        serviceProvider.GetRequiredService<WaitQuestAccepted>()
+                        .With(step.PickUpQuestId ?? quest.QuestId);
-                            .With(step.PickUpQuestId ?? quest.QuestId),
+                    var delay = serviceProvider.GetRequiredService<WaitDelay>();
-                        serviceProvider.GetRequiredService<WaitDelay>()
+                    if (step.PickUpQuestId != null)
-                    ];
+                        return [accept, delay, Next(quest, sequence)];
+                    else
+                        return [accept, delay];
+                }
 
                 case EInteractionType.CompleteQuest:
-                    return
+                {
-                    [
+                    var complete = serviceProvider.GetRequiredService<WaitQuestCompleted>()
-                        serviceProvider.GetRequiredService<WaitQuestCompleted>()
+                        .With(step.TurnInQuestId ?? quest.QuestId);
-                            .With(step.TurnInQuestId ?? quest.QuestId),
+                    var delay = serviceProvider.GetRequiredService<WaitDelay>();
-                        serviceProvider.GetRequiredService<WaitDelay>()
+                    if (step.NextQuestId != null)
-                    ];
+                        return [complete, delay, Next(quest, sequence)];
+                    else
+                        return [complete, delay];
+                }
 
                 case EInteractionType.Interact:
                 default: