Improvement on fishbot + config + idling bot

bots/fishing/FishingAgent.py

@@ -5,16 +5,23 @@ import time
 import cv2 as cv
 import pyautogui
 from core.Logger import Logger
+from core.Player import PLAYER_STATE
 
 class FishingAgent:
-  def __init__(self, monitor) -> None:
-    self.logger = Logger("Fishing Agent")
-    self.retry = 0
-    self.monitor = monitor
-    self.lureLoc = None
+  def __init__(self, monitor, player) -> None:
     self.fishingBobberImg = cv.imread("assets/fishing/Bobber.jpg", cv.IMREAD_GRAYSCALE)
     self.fishingIconImg = cv.imread("assets/fishing/FishIcon.jpg", cv.IMREAD_GRAYSCALE)
     self.fishingThread = None
+    self.logger = Logger("Fishing Agent")
+    self.monitor = monitor
+    self.lureLoc = None
+    self.player = player
+    self.retry = 0
+
+  def startFishing(self):
+    self.player.changeState(PLAYER_STATE.FISHING)
+    while self.player.state is PLAYER_STATE.FISHING:
+      self.castLure()
 
   def castLure(self):
     self.retry = 0
@@ -35,7 +42,7 @@ class FishingAgent:
     if (res == -1):
       if (self.retry > 4):
         self.logger.log("Took too to find lure long, retrying fishing from the beginning")
-        self.castLure()
+        return
       else:  
         self.logger.log("Trying again in 1 second...")
         self.retry += 1
@@ -57,9 +64,9 @@ class FishingAgent:
       if (res != -1 and abs(res[1] - self.lureLoc[1]) > 20):
         pyautogui.click()
         time.sleep(0.5)
-        self.castLure()
+        return
       elif (self.retry > 200):
-        self.castLure()
+        return
 
   def run(self):
     self.logger.log("Starting fishing thread in 3 seconds...")
@@ -70,8 +77,8 @@ class FishingAgent:
     time.sleep(1)
     
     self.fishingThread = Thread(
-        target=self.castLure, 
+        target=self.startFishing, 
         args=(),
         name="fishing thread",
-        daemon=True)    
+        daemon=True)
     self.fishingThread.start()