Fix ratelimit errors when using fast refreshing applications

Author: Adikso

bashbot/core/settings.py

@@ -35,6 +35,7 @@ def load(self, path=DEFAULT_CONFIG_PATH):
         self.add_default('terminal.delete_messages', True)
         self.add_default('terminal.submit_by_default', True)
         self.add_default('terminal.interactive_by_default', False)
+        self.add_default('terminal.max_refresh_frequency', 0.5)
 
         # [terminal.interactive]
         self.add_default('terminal.interactive.delete_messages', True)

bashbot/terminal/terminal.py

@@ -45,7 +45,8 @@ def __init__(self, name: str,
 
         self.fd = None
         self.content = None
-        self.content_update = False
+
+        self.refresh_timer = None
 
     def open(self):
         pid, self.fd = os.forkpty()
@@ -69,7 +70,14 @@ def close(self):
         os.close(self.fd)
 
     def refresh(self, loop=None):
-        execute_async(loop or asyncio.get_event_loop(), self.on_change(self, self.content))
+        if self.refresh_timer and self.refresh_timer.is_alive():
+            return
+
+        interval = settings().get('terminal.max_refresh_frequency')
+        self.refresh_timer = threading.Timer(interval, lambda: {
+            execute_async(loop or asyncio.get_event_loop(), self.on_change(self, self.content))
+        })
+        self.refresh_timer.start()
 
     def send_input(self, data: str):
         if self.state != TerminalState.OPEN:
@@ -100,7 +108,6 @@ def __monitor_pty(self, loop):
             while output:
                 self.stream.feed(output)
                 self.content = '\n'.join(self.screen.display)
-                self.content_update = True
 
                 if self.on_change:
                     self.refresh(loop)

config.toml.example

@@ -17,6 +17,7 @@ delete_on_close = false
 delete_messages = true
 submit_by_default = true
 interactive_by_default = false
+max_refresh_frequency = 0.5
 
 [other]
 check_for_updates = true