refactor: cache guild info and guild channels

this lowers the amount of requests we make and reduces the chance of ratelimiting

Author: onerandomusername

botstrap.py

@@ -84,11 +84,36 @@ def __init__(self, guild_id: int | str):
         )
         self.guild_id = guild_id
         self._app_info: dict[str, Any] | None = None
+        self._guild_info: dict[str, Any] | None = None
+        self._guild_channels: list[dict[str, Any]] | None = None
 
     @staticmethod
     def _raise_for_status(response: Response) -> None:
         response.raise_for_status()
 
+    @property
+    def guild_info(self) -> dict[str, Any]:
+        """Fetches the guild's information."""
+        if self._guild_info is None:
+            response = self.get(f"/guilds/{self.guild_id}")
+            self._guild_info = cast("dict[str, Any]", response.json())
+        return self._guild_info
+
+    @property
+    def guild_channels(self) -> list[dict[str, Any]]:
+        """Fetches the guild's channels."""
+        if self._guild_channels is None:
+            response = self.get(f"/guilds/{self.guild_id}/channels")
+            self._guild_channels = cast("list[dict[str, Any]]", response.json())
+        return self._guild_channels
+
+    def get_channel(self, id: int | str) -> dict[str, Any]:
+        """Fetches a channel by its ID."""
+        for channel in self.guild_channels:
+            if channel["id"] == str(id):
+                return channel
+        raise KeyError(f"Channel with ID {id} not found.")
+
     @property
     def app_info(self) -> dict[str, Any]:
         """Fetches the application's information."""
@@ -117,7 +142,7 @@ def upgrade_application_flags_if_necessary(self) -> bool:
     def check_if_in_guild(self) -> bool:
         """Check if the bot is a member of the guild."""
         try:
-            _ = self.get(f"/guilds/{self.guild_id}")
+            _ = self.guild_info
         except HTTPStatusError as e:
             if e.response.status_code == 403 or e.response.status_code == 404:
                 return False
@@ -130,15 +155,14 @@ def upgrade_server_to_community_if_necessary(
         announcements_channel_id_: int | str,
     ) -> None:
         """Fetches server info & upgrades to COMMUNITY if necessary."""
-        response = self.get(f"/guilds/{self.guild_id}")
-        payload = response.json()
+        payload = self.guild_info
 
         if COMMUNITY_FEATURE not in payload["features"]:
             log.info("This server is currently not a community, upgrading.")
             payload["features"].append(COMMUNITY_FEATURE)
             payload["rules_channel_id"] = rules_channel_id_
             payload["public_updates_channel_id"] = announcements_channel_id_
-            self.patch(f"/guilds/{self.guild_id}", json=payload)
+            self._guild_info = self.patch(f"/guilds/{self.guild_id}", json=payload).json()
             log.info(f"Server {self.guild_id} has been successfully updated to a community.")
 
     def create_forum_channel(self, channel_name_: str, category_id_: int | str | None = None) -> str:
@@ -152,22 +176,20 @@ def create_forum_channel(self, channel_name_: str, category_id_: int | str | Non
         log.info(f"New forum channel: {channel_name_} has been successfully created.")
         return forum_channel_id
 
-    def is_forum_channel(self, channel_id_: str) -> bool:
+    def is_forum_channel(self, channel_id: str) -> bool:
         """A boolean that indicates if a channel is of type GUILD_FORUM."""
-        response = self.get(f"/channels/{channel_id_}")
-        return response.json()["type"] == GUILD_FORUM_TYPE
+        return self.get_channel(channel_id)["type"] == GUILD_FORUM_TYPE
 
-    def delete_channel(self, channel_id_: str | int) -> None:
+    def delete_channel(self, channel_id: str | int) -> None:
         """Delete a channel."""
-        log.info(f"Channel python-help: {channel_id_} is not a forum channel and will be replaced with one.")
-        self.delete(f"/channels/{channel_id_}")
+        log.info("Channel python-help: %s is not a forum channel and will be replaced with one.", channel_id)
+        self.delete(f"/channels/{channel_id}")
 
     def get_all_roles(self) -> dict[str, int]:
         """Fetches all the roles in a guild."""
         result = SilencedDict(name="Roles dictionary")
 
-        response = self.get(f"guilds/{self.guild_id}/roles")
-        roles = response.json()
+        roles = self.guild_info["roles"]
 
         for role in roles:
             name = "_".join(part.lower() for part in role["name"].split(" ")).replace("-", "_")
@@ -182,10 +204,7 @@ def get_all_channels_and_categories(self) -> tuple[dict[str, str], dict[str, str
         channels = SilencedDict(name="Channels dictionary")
         categories = SilencedDict(name="Categories dictionary")
 
-        response = self.get(f"guilds/{self.guild_id}/channels")
-        server_channels = response.json()
-
-        for channel in server_channels:
+        for channel in self.guild_channels:
             channel_type = channel["type"]
             name = "_".join(part.lower() for part in channel["name"].split(" ")).replace("-", "_")
             if re.match(off_topic_channel_name_regex, name):