Merge pull request 'v4.1.0' (#189) from dev into main

Reviewed-on: #189

README.md

@@ -16,7 +16,6 @@ There are different sub-packages available:
 * pyrogram - Telegram bots with Pyrogram's fork "Pyrofork"
 * pycord - Discord bots with Pycord
 * speed - Performance improvements
-* cache - Support for Redis and Memcached
 * dev - Dependencies for package development purposes
 
 You can freely choose any sub-package you want, as well as add multiple (comma-separated) or none at all.

docs/Makefile

@@ -1,20 +0,0 @@
-# Minimal makefile for Sphinx documentation
-#
-
-# You can set these variables from the command line, and also
-# from the environment for the first two.
-SPHINXOPTS    ?=
-SPHINXBUILD   ?= sphinx-build
-SOURCEDIR     = .
-BUILDDIR      = _build
-
-# Put it first so that "make" without argument is like "make help".
-help:
-	@$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
-
-.PHONY: help Makefile
-
-# Catch-all target: route all unknown targets to Sphinx using the new
-# "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
-%: Makefile
-	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)

docs/conf.py

@@ -1,61 +0,0 @@
-# Configuration file for the Sphinx documentation builder.
-#
-# For the full list of built-in configuration values, see the documentation:
-# https://www.sphinx-doc.org/en/master/usage/configuration.html
-import os
-import sys
-from datetime import datetime
-from typing import List, Dict, Any
-
-import libbot
-
-sys.path.insert(0, os.path.abspath(".."))
-
-
-# -- Project information -----------------------------------------------------
-# https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information
-
-project: str = "LibBotUniversal"
-author: str = libbot.__author__
-version: str = libbot.__version__
-release: str = version
-copyright: str = f"{datetime.now().year} {author}"
-
-html_title: str = f"{project} v{version} Documentation"
-
-
-# -- General configuration ---------------------------------------------------
-# https://www.sphinx-doc.org/en/master/usage/configuration.html#general-configuration
-
-extensions: List[str] = [
-    "sphinx.ext.autodoc",
-    "sphinx.ext.autosummary",
-    "sphinx.ext.viewcode",
-    "sphinx.ext.viewcode",
-    "sphinx.ext.napoleon",
-    "sphinx_copybutton",
-]
-
-templates_path: List[str] = ["_templates"]
-exclude_patterns: List[str] = ["_build", "Thumbs.db", ".DS_Store"]
-
-
-# -- Options for HTML output -------------------------------------------------
-# https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-html-output
-
-html_theme: str = "furo"
-html_static_path: List[str] = ["_static"]
-
-html_theme_options: Dict[str, Any] = {
-    "source_repository": "https://git.end-play.xyz/profitroll/LibBotUniversal",
-    "source_branch": "main",
-    "source_directory": "docs/",
-    "light_css_variables": {
-        "font-stack": "'Outfit', sans-serif",
-    },
-}
-
-resource_links: Dict[str, Any] = {
-    "issues": "https://git.end-play.xyz/profitroll/LibBotUniversal/issues",
-    "examples": "https://git.end-play.xyz/profitroll/LibBotUniversal/tree/main/examples",
-}

docs/index.rst

@@ -1,29 +0,0 @@
-Welcome to LibBotUniversal
-=============================
-
-LibBotUniversal (AKA libbot) simplifies development and maintenance of different chatbots.
-
-Getting started
-*****************************
-
-There are a few pages to take a look at if you're just getting started.
-
-* Installing the library
-* Creating your first bot
-* Examples of different bots
-
-Getting help
-*****************************
-
-If you are looking for help or want to report a bug, we got you covered.
-
-* Bug reports are being submitted on the repo's issue tracker.
-
-Package
-*****************************
-
-.. toctree::
-   :maxdepth: 1
-   :caption: API reference
-
-   modules

docs/make.bat

@@ -1,35 +0,0 @@
-@ECHO OFF
-
-pushd %~dp0
-
-REM Command file for Sphinx documentation
-
-if "%SPHINXBUILD%" == "" (
-	set SPHINXBUILD=sphinx-build
-)
-set SOURCEDIR=.
-set BUILDDIR=_build
-
-%SPHINXBUILD% >NUL 2>NUL
-if errorlevel 9009 (
-	echo.
-	echo.The 'sphinx-build' command was not found. Make sure you have Sphinx
-	echo.installed, then set the SPHINXBUILD environment variable to point
-	echo.to the full path of the 'sphinx-build' executable. Alternatively you
-	echo.may add the Sphinx directory to PATH.
-	echo.
-	echo.If you don't have Sphinx installed, grab it from
-	echo.https://www.sphinx-doc.org/
-	exit /b 1
-)
-
-if "%1" == "" goto help
-
-%SPHINXBUILD% -M %1 %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%
-goto end
-
-:help
-%SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%
-
-:end
-popd

docs/modules.rst

@@ -1,7 +0,0 @@
-libbot
-======
-
-.. toctree::
-   :maxdepth: 5
-
-   libbot

pyproject.toml

@@ -1,5 +1,5 @@
 [build-system]
-requires = ["setuptools>=77.0.3", "wheel"]
+requires = ["setuptools>=62.6", "wheel"]
 build-backend = "setuptools.build_meta"
 
 [project]
@@ -9,11 +9,11 @@ authors = [{ name = "Profitroll" }]
 description = "Universal bot library with functions needed for basic Discord/Telegram bot development."
 readme = "README.md"
 requires-python = ">=3.11"
-license = "GPL-3.0"
-license-files = ["LICENSE"]
+license = { text = "GPLv3" }
 classifiers = [
     "Development Status :: 3 - Alpha",
     "Intended Audience :: Developers",
+    "License :: OSI Approved :: GNU General Public License v3 (GPLv3)",
     "Operating System :: OS Independent",
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
@@ -28,7 +28,6 @@ dependencies = { file = "requirements/_.txt" }
 
 [tool.setuptools.dynamic.optional-dependencies]
 dev = { file = "requirements/dev.txt" }
-docs = { file = "requirements/docs.txt" }
 pycord = { file = "requirements/pycord.txt" }
 pyrogram = { file = "requirements/pyrogram.txt" }
 speed = { file = "requirements/speed.txt" }

requirements/_.txt

@@ -1,2 +1,2 @@
 aiofiles>=23.0.0
-typing-extensions~=4.14.0
+typing-extensions~=4.12.2

requirements/cache.txt

@@ -1,2 +1,2 @@
 pymemcache~=4.0.0
-redis~=6.2.0
+redis~=5.2.1

requirements/dev.txt

@@ -1,12 +1,12 @@
 black==25.1.0
 build==1.2.2.post1
 isort==5.13.2
-mypy==1.16.1
-pylint==3.3.7
-pytest-asyncio==1.0.0
-pytest-cov==6.2.1
-pytest==8.4.1
-tox==4.27.0
+mypy==1.15.0
+pylint==3.3.4
+pytest-asyncio==0.25.3
+pytest-cov==6.0.0
+pytest==8.3.4
+tox==4.24.0
 twine==6.1.0
-types-aiofiles==24.1.0.20250606
-types-ujson==5.10.0.20250326
+types-aiofiles==24.1.0.20241221
+types-ujson==5.10.0.20240515

requirements/docs.txt

@@ -1,3 +0,0 @@
-furo==2024.8.6
-sphinx==8.1.3
-sphinx_copybutton==0.5.2

src/libbot/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "4.3.0"
+__version__ = "4.1.0"
 __license__ = "GPL3"
 __author__ = "Profitroll"
 

src/libbot/cache/classes/cache.py

@@ -1,5 +1,5 @@
 from abc import ABC, abstractmethod
-from typing import Any, Dict, Optional
+from typing import Any, Dict
 
 import pymemcache
 import redis
@@ -27,16 +27,16 @@ class Cache(ABC):
         pass
 
     @abstractmethod
-    def set_json(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
+    def set_json(self, key: str, value: Any) -> None:
         # TODO This method must also carry out ObjectId conversion!
         pass
 
     @abstractmethod
-    def set_string(self, key: str, value: str, ttl_seconds: Optional[int] = None) -> None:
+    def set_string(self, key: str, value: str) -> None:
         pass
 
     @abstractmethod
-    def set_object(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
+    def set_object(self, key: str, value: Any) -> None:
         pass
 
     @abstractmethod

src/libbot/cache/classes/cache_memcached.py

@@ -1,6 +1,6 @@
 import logging
 from logging import Logger
-from typing import Dict, Any, Optional
+from typing import Dict, Any
 
 from pymemcache import Client
 
@@ -13,30 +13,21 @@ logger: Logger = logging.getLogger(__name__)
 class CacheMemcached(Cache):
     client: Client
 
-    def __init__(
-        self, client: Client, prefix: Optional[str] = None, default_ttl_seconds: Optional[int] = None
-    ) -> None:
-        self.client: Client = client
-        self.prefix: str | None = prefix
-        self.default_ttl_seconds: int = default_ttl_seconds if default_ttl_seconds is not None else 0
+    def __init__(self, client: Client):
+        self.client = client
 
         logger.info("Initialized Memcached for caching")
 
     @classmethod
-    def from_config(cls, engine_config: Dict[str, Any], prefix: Optional[str] = None) -> "CacheMemcached":
+    def from_config(cls, engine_config: Dict[str, Any]) -> "CacheMemcached":
         if "uri" not in engine_config:
             raise KeyError(
                 "Cache configuration is invalid. Please check if all keys are set (engine: memcached)"
             )
 
-        return cls(Client(engine_config["uri"], default_noreply=True), prefix=prefix)
-
-    def _get_prefixed_key(self, key: str) -> str:
-        return key if self.prefix is None else f"{self.prefix}_{key}"
+        return cls(Client(engine_config["uri"], default_noreply=True))
 
     def get_json(self, key: str) -> Any | None:
-        key = self._get_prefixed_key(key)
-
         try:
             result: Any | None = self.client.get(key, None)
 
@@ -52,8 +43,6 @@ class CacheMemcached(Cache):
         return None if result is None else _string_to_json(result)
 
     def get_string(self, key: str) -> str | None:
-        key = self._get_prefixed_key(key)
-
         try:
             result: str | None = self.client.get(key, None)
 
@@ -72,39 +61,27 @@ class CacheMemcached(Cache):
     def get_object(self, key: str) -> Any | None:
         raise NotImplementedError()
 
-    def set_json(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
-        key = self._get_prefixed_key(key)
-
+    def set_json(self, key: str, value: Any) -> None:
         try:
-            self.client.set(
-                key,
-                _json_to_string(value),
-                expire=self.default_ttl_seconds if ttl_seconds is None else ttl_seconds,
-            )
+            self.client.set(key, _json_to_string(value))
             logger.debug("Set json cache key '%s'", key)
         except Exception as exc:
             logger.error("Could not set json cache key '%s' due to: %s", key, exc)
             return None
 
-    def set_string(self, key: str, value: str, ttl_seconds: Optional[int] = None) -> None:
-        key = self._get_prefixed_key(key)
-
+    def set_string(self, key: str, value: str) -> None:
         try:
-            self.client.set(
-                key, value, expire=self.default_ttl_seconds if ttl_seconds is None else ttl_seconds
-            )
+            self.client.set(key, value)
             logger.debug("Set string cache key '%s'", key)
         except Exception as exc:
             logger.error("Could not set string cache key '%s' due to: %s", key, exc)
             return None
 
     # TODO Implement binary serialization
-    def set_object(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
+    def set_object(self, key: str, value: Any) -> None:
         raise NotImplementedError()
 
     def delete(self, key: str) -> None:
-        key = self._get_prefixed_key(key)
-
         try:
             self.client.delete(key)
             logger.debug("Deleted cache key '%s'", key)

src/libbot/cache/classes/cache_redis.py

@@ -1,11 +1,11 @@
 import logging
 from logging import Logger
-from typing import Dict, Any, Optional
+from typing import Dict, Any
 
 from redis import Redis
 
 from .cache import Cache
-from ..utils._objects import _json_to_string, _string_to_json
+from ..utils._objects import _string_to_json, _json_to_string
 
 logger: Logger = logging.getLogger(__name__)
 
@@ -13,30 +13,21 @@ logger: Logger = logging.getLogger(__name__)
 class CacheRedis(Cache):
     client: Redis
 
-    def __init__(
-        self, client: Redis, prefix: Optional[str] = None, default_ttl_seconds: Optional[int] = None
-    ) -> None:
-        self.client: Redis = client
-        self.prefix: str | None = prefix
-        self.default_ttl_seconds: int | None = default_ttl_seconds
+    def __init__(self, client: Redis):
+        self.client = client
 
         logger.info("Initialized Redis for caching")
 
     @classmethod
-    def from_config(cls, engine_config: Dict[str, Any], prefix: Optional[str] = None) -> Any:
+    def from_config(cls, engine_config: Dict[str, Any]) -> Any:
         if "uri" not in engine_config:
             raise KeyError(
                 "Cache configuration is invalid. Please check if all keys are set (engine: memcached)"
             )
 
-        return cls(Redis.from_url(engine_config["uri"]), prefix=prefix)
-
-    def _get_prefixed_key(self, key: str) -> str:
-        return key if self.prefix is None else f"{self.prefix}_{key}"
+        return cls(Redis.from_url(engine_config["uri"]))
 
     def get_json(self, key: str) -> Any | None:
-        key = self._get_prefixed_key(key)
-
         try:
             result: Any | None = self.client.get(key)
 
@@ -52,8 +43,6 @@ class CacheRedis(Cache):
         return None if result is None else _string_to_json(result)
 
     def get_string(self, key: str) -> str | None:
-        key = self._get_prefixed_key(key)
-
         try:
             result: str | None = self.client.get(key)
 
@@ -72,37 +61,27 @@ class CacheRedis(Cache):
     def get_object(self, key: str) -> Any | None:
         raise NotImplementedError()
 
-    def set_json(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
-        key = self._get_prefixed_key(key)
-
+    def set_json(self, key: str, value: Any) -> None:
         try:
-            self.client.set(
-                key,
-                _json_to_string(value),
-                ex=self.default_ttl_seconds if ttl_seconds is None else ttl_seconds,
-            )
+            self.client.set(key, _json_to_string(value))
             logger.debug("Set json cache key '%s'", key)
         except Exception as exc:
             logger.error("Could not set json cache key '%s' due to: %s", key, exc)
             return None
 
-    def set_string(self, key: str, value: str, ttl_seconds: Optional[int] = None) -> None:
-        key = self._get_prefixed_key(key)
-
+    def set_string(self, key: str, value: str) -> None:
         try:
-            self.client.set(key, value, ex=self.default_ttl_seconds if ttl_seconds is None else ttl_seconds)
+            self.client.set(key, value)
             logger.debug("Set string cache key '%s'", key)
         except Exception as exc:
             logger.error("Could not set string cache key '%s' due to: %s", key, exc)
             return None
 
     # TODO Implement binary serialization
-    def set_object(self, key: str, value: Any, ttl_seconds: Optional[int] = None) -> None:
+    def set_object(self, key: str, value: Any) -> None:
         raise NotImplementedError()
 
     def delete(self, key: str) -> None:
-        key = self._get_prefixed_key(key)
-
         try:
             self.client.delete(key)
             logger.debug("Deleted cache key '%s'", key)

src/libbot/cache/manager/manager.py

@@ -1,4 +1,4 @@
-from typing import Dict, Any, Literal, Optional
+from typing import Dict, Any, Literal
 
 from ..classes import CacheMemcached, CacheRedis
 
@@ -6,7 +6,6 @@ from ..classes import CacheMemcached, CacheRedis
 def create_cache_client(
     config: Dict[str, Any],
     engine: Literal["memcached", "redis"] | None = None,
-    prefix: Optional[str] = None,
 ) -> CacheMemcached | CacheRedis:
     if engine not in ["memcached", "redis"] or engine is None:
         raise KeyError(f"Incorrect cache engine provided. Expected 'memcached' or 'redis', got '{engine}'")
@@ -18,8 +17,8 @@ def create_cache_client(
 
     match engine:
         case "memcached":
-            return CacheMemcached.from_config(config["cache"][engine], prefix=prefix)
+            return CacheMemcached.from_config(config["cache"][engine])
         case "redis":
-            return CacheRedis.from_config(config["cache"][engine], prefix=prefix)
+            return CacheRedis.from_config(config["cache"][engine])
         case _:
             raise KeyError(f"Cache implementation for the engine '{engine}' is not present.")

src/libbot/errors/config.py

@@ -4,8 +4,8 @@ from typing import Any, List, Optional
 class ConfigKeyError(Exception):
     """Raised when config key is not found.
 
-    Args:
-        key (str | List[str]): Missing config key
+    ### Attributes:
+        * key (`str | List[str]`): Missing config key.
     """
 
     def __init__(self, key: str | List[str]) -> None:
@@ -21,9 +21,9 @@ class ConfigKeyError(Exception):
 class ConfigValueError(Exception):
     """Raised when config key's value is invalid.
 
-    Args:
-        key (str | List[str]): Invalid config key
-        value (Optional[Any]): Key's correct value
+    ### Attributes:
+        * key (`str | List[str]`): Invalid config key.
+        * value (`Optional[Any]`): Key's correct value.
     """
 
     def __init__(self, key: str | List[str], value: Optional[Any] = None) -> None:

src/libbot/i18n/_functions.py

@@ -18,16 +18,16 @@ def _(
     locale: str | None = "en",
     locales_root: str | Path = Path("locale"),
 ) -> Any:
-    """Get value of the locale string.
+    """Get value of locale string
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locale (str | None): Locale to be looked up in. Defaults to `"en"`
-        locales_root (str | Path, optional):Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locale (`str | None`): Locale to looked up in. Defaults to `"en"`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        Any: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
+    ### Returns:
+        * `Any`: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
     """
     if locale is None:
         locale: str = config_get("locale")
@@ -58,16 +58,16 @@ async def _(
     locale: str | None = "en",
     locales_root: str | Path = Path("locale"),
 ) -> Any:
-    """Get value of the locale string.
+    """Get value of locale string
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locale (str | None): Locale to be looked up in. Defaults to `"en"`
-        locales_root (str | Path, optional):Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locale (`str | None`): Locale to looked up in. Defaults to `"en"`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        Any: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
+    ### Returns:
+        * `Any`: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
     """
     locale: str = config_get("locale") if locale is None else locale
 
@@ -94,16 +94,17 @@ async def _(
 
 @asyncable
 def in_all_locales(key: str, *args: str, locales_root: str | Path = Path("locale")) -> List[Any]:
-    """Get value of the provided key and path in all available locales.
+    """Get value of the provided key and path in all available locales
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locales_root (str | Path, optional): Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        List[Any]: List of values in all locales
+    ### Returns:
+        * `List[Any]`: List of values in all locales
     """
+
     output: List[Any] = []
 
     for locale in _get_valid_locales(locales_root):
@@ -127,15 +128,15 @@ def in_all_locales(key: str, *args: str, locales_root: str | Path = Path("locale
 
 @in_all_locales.asynchronous
 async def in_all_locales(key: str, *args: str, locales_root: str | Path = Path("locale")) -> List[Any]:
-    """Get value of the provided key and path in all available locales.
+    """Get value of the provided key and path in all available locales
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locales_root (str | Path, optional): Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        List[Any]: List of values in all locales
+    ### Returns:
+        * `List[Any]`: List of values in all locales
     """
 
     output: List[Any] = []
@@ -160,17 +161,20 @@ async def in_all_locales(key: str, *args: str, locales_root: str | Path = Path("
 
 
 @asyncable
-def in_every_locale(key: str, *args: str, locales_root: str | Path = Path("locale")) -> Dict[str, Any]:
-    """Get value of the provided key and path in every available locale with locale tag.
+def in_every_locale(
+    key: str, *args: str, locales_root: str | Path = Path("locale")
+) -> Dict[str, Any]:
+    """Get value of the provided key and path in every available locale with locale tag
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locales_root (str | Path, optional): Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        Dict[str, Any]: Locale is a key, and it's value from locale file is a value
+    ### Returns:
+        * `Dict[str, Any]`: Locale is a key, and it's value from locale file is a value
     """
+
     output: Dict[str, Any] = {}
 
     for locale in _get_valid_locales(locales_root):
@@ -196,15 +200,15 @@ def in_every_locale(key: str, *args: str, locales_root: str | Path = Path("local
 async def in_every_locale(
     key: str, *args: str, locales_root: str | Path = Path("locale")
 ) -> Dict[str, Any]:
-    """Get value of the provided key and path in every available locale with locale tag.
+    """Get value of the provided key and path in every available locale with locale tag
 
-    Args:
-        key (str): The last key of the locale's keys path
-        *args (str): Path to key like: `dict[args][key]`
-        locales_root (str | Path, optional): Folder where locales are located. Defaults to `Path("locale")`
+    ### Args:
+        * key (`str`): The last key of the locale's keys path.
+        * *args (`str`): Path to key like: `dict[args][key]`.
+        * locales_root (`str | Path`, *optional*): Folder where locales are located. Defaults to `Path("locale")`.
 
-    Returns:
-        Dict[str, Any]: Locale is a key, and it's value from locale file is a value
+    ### Returns:
+        * `Dict[str, Any]`: Locale is a key, and it's value from locale file is a value
     """
 
     output: Dict[str, Any] = {}

src/libbot/i18n/classes/bot_locale.py

@@ -14,11 +14,6 @@ class BotLocale:
         default_locale: str | None = "en",
         locales_root: str | Path = Path("locale"),
     ) -> None:
-        """
-        Args:
-            default_locale (str | None): Default bot's locale. Defaults to `"en"`
-            locales_root (str | Path): Folder where locales are located. Defaults to `Path("locale")`
-        """
         if isinstance(locales_root, str):
             locales_root = Path(locales_root)
         elif not isinstance(locales_root, Path):
@@ -35,15 +30,15 @@ class BotLocale:
             self.locales[locale] = json_read(Path(f"{locales_root}/{locale}.json"))
 
     def _(self, key: str, *args: str, locale: str | None = None) -> Any:
-        """Get value of locale string.
+        """Get value of locale string
 
-        Args:
-            key (str): The last key of the locale's keys path
-            *args (str): Path to key like: `dict[args][key]`
-            locale (str | None, optional): Locale to be looked up in. Defaults to config's `"locale"` value
+        ### Args:
+            * key (`str`): The last key of the locale's keys path
+            * *args (`str`): Path to key like: `dict[args][key]`
+            * locale (`str | None`, *optional*): Locale to looked up in. Defaults to config's `"locale"` value
 
-        Returns:
-            Any: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
+        ### Returns:
+            * `Any`: Value of provided locale key. Is usually `str`, `Dict[str, Any]` or `List[Any]`
         """
         if locale is None:
             locale: str = self.default
@@ -69,14 +64,14 @@ class BotLocale:
             return f'⚠️ Locale in config is invalid: could not get "{key}" in {args} from locale "{locale}"'
 
     def in_all_locales(self, key: str, *args: str) -> List[Any]:
-        """Get value of the provided key and path in all available locales.
+        """Get value of the provided key and path in all available locales
 
-        Args:
-            key (str): The last key of the locale's keys path
-            *args (str): Path to key like: `dict[args][key]`
+        ### Args:
+            * key (`str`): The last key of the locale's keys path.
+            * *args (`str`): Path to key like: `dict[args][key]`.
 
-        Returns:
-            List[Any]: List of values in all locales
+        ### Returns:
+            * `List[Any]`: List of values in all locales
         """
         output: List[Any] = []
 
@@ -99,14 +94,14 @@ class BotLocale:
         return output
 
     def in_every_locale(self, key: str, *args: str) -> Dict[str, Any]:
-        """Get value of the provided key and path in every available locale with locale tag.
+        """Get value of the provided key and path in every available locale with locale tag
 
-        Args:
-            key (str): The last key of the locale's keys path
-            *args (str): Path to key like: `dict[args][key]`
+        ### Args:
+            * key (`str`): The last key of the locale's keys path.
+            * *args (`str`): Path to key like: `dict[args][key]`.
 
-        Returns:
-            Dict[str, Any]: Locale is a key, and it's value from locale file is a value
+        ### Returns:
+            * `Dict[str, Any]`: Locale is a key, and it's value from locale file is a value
         """
         output: Dict[str, Any] = {}
 

src/libbot/pycord/classes/bot.py

@@ -19,27 +19,16 @@ logger: Logger = logging.getLogger(__name__)
 
 
 class PycordBot(Bot):
-    # TODO Write a docstring
     @override
     def __init__(
-        self,
-        *args,
-        config: Dict[str, Any] | None = None,
-        config_path: str | Path = Path("config.json"),
-        locales_root: str | Path | None = None,
-        scheduler: AsyncIOScheduler | BackgroundScheduler | None = None,
-        **kwargs,
+            self,
+            *args,
+            config: Dict[str, Any] | None = None,
+            config_path: str | Path = Path("config.json"),
+            locales_root: str | Path | None = None,
+            scheduler: AsyncIOScheduler | BackgroundScheduler | None = None,
+            **kwargs,
     ):
-        """
-
-        Args:
-            *args:
-            config:
-            config_path:
-            locales_root:
-            scheduler:
-            **kwargs:
-        """
         self.config: Dict[str, Any] = config if config is not None else json_read(config_path)
 
         super().__init__(
@@ -62,30 +51,15 @@ class PycordBot(Bot):
 
         self.scheduler: AsyncIOScheduler | BackgroundScheduler | None = scheduler
 
-    # TODO Write a docstring
     @override
     async def start(self, token: str, reconnect: bool = True, scheduler_start: bool = True) -> None:
-        """
-
-        Args:
-            token:
-            reconnect:
-            scheduler_start:
-        """
         if self.scheduler is not None and scheduler_start:
             self.scheduler.start()
 
         await super().start(token, reconnect=reconnect)
 
-    # TODO Write a docstring
     @override
     async def close(self, scheduler_shutdown: bool = True, scheduler_wait: bool = True) -> None:
-        """
-
-        Args:
-            scheduler_shutdown:
-            scheduler_wait:
-        """
         if self.scheduler is not None and scheduler_shutdown:
             self.scheduler.shutdown(scheduler_wait)
 

src/libbot/pycord/utils/color.py

@@ -16,24 +16,20 @@ def _hex_from_int(color_int: int) -> str:
 
 
 def color_from_hex(hex_string: str) -> Colour:
-    """Convert valid hexadecimal string to :class:`discord.Colour`.
+    """Convert valid hexadecimal string to discord.Colour.
 
-    Args:
-        hex_string (str): Hexadecimal string to convert into :class:`discord.Colour` object
-
-    Returns:
-        Colour: :class:`discord.Colour` object
+    :param hex_string: Hexadecimal string to convert into Colour object
+    :type hex_string: str
+    :return: Colour object
     """
     return Colour(_int_from_hex(hex_string))
 
 
 def hex_from_color(color: Colour) -> str:
-    """Convert :class:`discord.Colour` to hexadecimal string.
+    """Convert discord.Colour to hexadecimal string.
 
-    Args:
-        color (Colour): :class:`discord.Colour` object to convert into the string
-
-    Returns:
-        str: Hexadecimal string in #XXXXXX format
+    :param color: Colour object to convert into the string
+    :type color: Colour
+    :return: Hexadecimal string in #XXXXXX format
     """
     return _hex_from_int(color.value)

src/libbot/pyrogram/classes/client.py

@@ -6,7 +6,7 @@ from logging import Logger
 from os import cpu_count, getpid
 from pathlib import Path
 from time import time
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List
 
 from typing_extensions import override
 
@@ -46,28 +46,27 @@ logger: Logger = logging.getLogger(__name__)
 
 
 class PyroClient(Client):
-    # TODO Write a docstring
     @override
     def __init__(
-        self,
-        name: str = "bot_client",
-        owner: int | None = None,
-        config: Dict[str, Any] | None = None,
-        config_path: str | Path = Path("config.json"),
-        api_id: int | None = None,
-        api_hash: str | None = None,
-        bot_token: str | None = None,
-        workers: int = min(32, cpu_count() + 4),
-        locales_root: str | Path | None = None,
-        plugins_root: str = "plugins",
-        plugins_exclude: List[str] | None = None,
-        sleep_threshold: int = 120,
-        max_concurrent_transmissions: int = 1,
-        commands_source: Dict[str, dict] | None = None,
-        scoped_commands: bool | None = None,
-        i18n_bot_info: bool = False,
-        scheduler: AsyncIOScheduler | BackgroundScheduler | None = None,
-        **kwargs,
+            self,
+            name: str = "bot_client",
+            owner: int | None = None,
+            config: Dict[str, Any] | None = None,
+            config_path: str | Path = Path("config.json"),
+            api_id: int | None = None,
+            api_hash: str | None = None,
+            bot_token: str | None = None,
+            workers: int = min(32, cpu_count() + 4),
+            locales_root: str | Path | None = None,
+            plugins_root: str = "plugins",
+            plugins_exclude: List[str] | None = None,
+            sleep_threshold: int = 120,
+            max_concurrent_transmissions: int = 1,
+            commands_source: Dict[str, dict] | None = None,
+            scoped_commands: bool | None = None,
+            i18n_bot_info: bool = False,
+            scheduler: AsyncIOScheduler | BackgroundScheduler | None = None,
+            **kwargs,
     ):
         self.config: Dict[str, Any] = config if config is not None else json_read(config_path)
 
@@ -120,12 +119,6 @@ class PyroClient(Client):
 
     @override
     async def start(self, register_commands: bool = True, scheduler_start: bool = True) -> None:
-        """Start the bot and it's services.
-
-        Args:
-            register_commands (bool, optional): Register commands on start. Defaults to `True`
-            scheduler_start (bool, optional): Start the scheduler on start. Defaults to `True`
-        """
         await super().start()
 
         self.start_time = time()
@@ -216,15 +209,8 @@ class PyroClient(Client):
 
     @override
     async def stop(
-        self, exit_completely: bool = True, scheduler_shutdown: bool = True, scheduler_wait: bool = True
+            self, exit_completely: bool = True, scheduler_shutdown: bool = True, scheduler_wait: bool = True
     ) -> None:
-        """Stop the bot and it's services.
-
-        Args:
-            exit_completely (bool, optional): Exit the program. Defaults to `True`
-            scheduler_shutdown (bool): Shutdown the scheduler. Defaults to `True`
-            scheduler_wait (bool): Wait for tasks to finish. Defaults to `True`
-        """
         try:
             await self.send_message(
                 chat_id=(
@@ -251,10 +237,10 @@ class PyroClient(Client):
                 raise SystemExit("Bot has been shut down, this is not an application error!") from exc
 
     async def collect_commands(self) -> List[CommandSet] | None:
-        """Gather list of the bot's commands.
+        """Gather list of the bot's commands
 
-        Returns:
-            List[CommandSet]: List of the commands' sets
+        ### Returns:
+            * `List[CommandSet]`: List of the commands' sets.
         """
         command_sets = None
 
@@ -322,7 +308,7 @@ class PyroClient(Client):
             # in it, if there are any. Then adds them to self.commands
             for handler in self.dispatcher.groups[0]:
                 if isinstance(handler, MessageHandler) and (
-                    hasattr(handler.filters, "base") or hasattr(handler.filters, "other")
+                        hasattr(handler.filters, "base") or hasattr(handler.filters, "other")
                 ):
                     for entry in [handler.filters.base, handler.filters.other]:
                         if hasattr(entry, "commands"):
@@ -333,13 +319,13 @@ class PyroClient(Client):
         return command_sets
 
     def add_command(
-        self,
-        command: str,
+            self,
+            command: str,
     ) -> None:
-        """Add command to the bot's internal commands list.
+        """Add command to the bot's internal commands list
 
-        Args:
-            command (str): Command's name
+        ### Args:
+            * command (`str`)
         """
         self.commands.append(
             PyroCommand(
@@ -352,12 +338,9 @@ class PyroClient(Client):
             command,
         )
 
-    async def register_commands(self, command_sets: Optional[List[CommandSet]] = None) -> None:
-        """Register commands stored in bot's 'commands' attribute.
+    async def register_commands(self, command_sets: List[CommandSet] | None = None) -> None:
+        """Register commands stored in bot's 'commands' attribute"""
 
-        Args:
-            command_sets (List[CommandSet], optional): List of command sets. Commands will be parsed from bot's 'commands' attribute if not provided
-        """
         if command_sets is None:
             commands = [
                 BotCommand(command=command.command, description=command.description)
@@ -382,12 +365,9 @@ class PyroClient(Client):
                 language_code=command_set.language_code,
             )
 
-    async def remove_commands(self, command_sets: Optional[List[CommandSet]] = None) -> None:
-        """Remove commands stored in bot's 'commands' attribute.
+    async def remove_commands(self, command_sets: List[CommandSet] | None = None) -> None:
+        """Remove commands stored in bot's 'commands' attribute"""
 
-        Args:
-            command_sets (List[CommandSet], optional): List of command sets. Commands with scope :class:`pyrogram.types.BotCommandScopeDefault` will be removed if not provided
-        """
         if command_sets is None:
             logger.info("Removing commands with a default scope 'BotCommandScopeDefault'")
             await self.delete_bot_commands(BotCommandScopeDefault())