arguments.py

@@ -29,11 +29,3 @@ class ArgumentParser:
             await utils.reply(message, f"```\n{self.print_help()}```")
         except Exception as e:
             await utils.reply(message, f"`{e}`")
-
-
-def range_type(string, min=0, max=100):
-    value = int(string)
-    if min <= value <= max:
-        return value
-    else:
-        raise argparse.ArgumentTypeError("value not in range %s-%s" % (min, max))

commands/__init__.py

@@ -1,6 +1,21 @@
+import importlib
+import inspect
+
+from state import reloaded_modules
+
 from . import bot, tools, utils, voice
 from .utils import *
 
 
 def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()
+
     globals().update({k: v for k, v in vars(utils).items() if not k.startswith("_")})

commands/bot.py

@@ -1,18 +1,12 @@
+import importlib
+import inspect
 import time
 
 import arguments
 import commands
+import constants
 import utils
-from state import start_time
-
-
-async def help(message):
-    await utils.reply(
-        message,
-        ", ".join(
-            [f"`{command.value}`" for command in commands.Command.__members__.values()]
-        ),
-    )
+from state import reloaded_modules, start_time
 
 
 async def uptime(message):
@@ -33,28 +27,16 @@ async def uptime(message):
     if args.since:
         await utils.reply(message, f"{round(start_time)}")
     else:
-        format_plural = lambda noun, count: noun if count == 1 else noun + "s"
+        await utils.reply(message, f"up {round(time.time() - start_time)} seconds")
 
-        segments = []
-        duration = time.time() - start_time
 
-        days, duration = divmod(duration, 86400)
-        if days >= 1:
-            days = int(days)
-            segments.append(f"{days} {format_plural('day', days)}")
-
-        hours, duration = divmod(duration, 3600)
-        if hours >= 1:
-            hours = int(hours)
-            segments.append(f"{hours} {format_plural('hour', hours)}")
-
-        minutes, duration = divmod(duration, 60)
-        if minutes >= 1:
-            minutes = int(minutes)
-            segments.append(f"{minutes} {format_plural('minute', minutes)}")
-
-        seconds = int(duration)
-        if seconds > 0:
-            segments.append(f"{seconds} {format_plural('second', seconds)}")
-
-        await utils.reply(message, f"up {', '.join(segments)}")
+def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()

commands/tools.py

@@ -1,8 +1,11 @@
+import importlib
+import inspect
 import re
 
 import arguments
 import commands
-import utils
+import constants
+from state import reloaded_modules
 
 
 async def clear(message):
@@ -60,3 +63,15 @@ async def clear(message):
         )
     except:
         pass
+
+
+def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()

commands/utils.py

@@ -6,7 +6,6 @@ import constants
 class Command(enum.Enum):
     CLEAR = "clear"
     EXECUTE = "execute"
-    HELP = "help"
     JOIN = "join"
     LEAVE = "leave"
     PAUSE = "pause"

commands/voice.py

@@ -1,10 +1,12 @@
-import functools
+import importlib
+import inspect
 
 import arguments
 import commands
+import constants
 import utils
-import youtubedl
-from state import client, player_current, player_queue
+import ytdlp
+from state import client, playback_queue, reloaded_modules
 
 
 async def queue_or_play(message):
@@ -12,8 +14,8 @@ async def queue_or_play(message):
     if not command_allowed(message):
         return
 
-    if message.guild.id not in player_queue:
-        player_queue[message.guild.id] = []
+    if message.guild.id not in playback_queue:
+        playback_queue[message.guild.id] = []
 
     tokens = commands.tokenize(message.content)
     parser = arguments.ArgumentParser(
@@ -26,28 +28,19 @@ async def queue_or_play(message):
         action="store_true",
         help="clear all queued songs",
     )
-    parser.add_argument(
-        "-v",
-        "--volume",
-        default=50,
-        type=functools.partial(arguments.range_type, min=0, max=150),
-        metavar="[0-150]",
-        help="the volume level (0 - 150)",
-    )
     if not (args := await parser.parse_args(message, tokens)):
         return
 
     if args.clear:
-        player_queue[message.guild.id] = []
-        await utils.add_check_reaction(message)
+        playback_queue[message.guild.id] = []
+        await message.add_reaction("✅")
         return
     elif query := args.query:
         try:
             async with message.channel.typing():
-                player = await youtubedl.YTDLSource.from_url(
+                player = await ytdlp.YTDLSource.from_url(
                     query, loop=client.loop, stream=True
                 )
-                player.volume = float(args.volume) / 100.0
         except Exception as e:
             await utils.reply(
                 message,
@@ -55,16 +48,15 @@ async def queue_or_play(message):
             )
             return
 
-        player_queue[message.guild.id].insert(
-            0, {"player": player, "queuer": message.author.id}
+        playback_queue[message.guild.id].append(
+            {"player": player, "queuer": message.author.id}
         )
 
         if (
             not message.guild.voice_client.is_playing()
             and not message.guild.voice_client.is_paused()
         ):
-            await utils.reply(message, f"**now playing:** `{player.title}`")
-            play_next(message)
+            await play_next(message)
         else:
             await utils.reply(
                 message,
@@ -80,29 +72,29 @@ async def queue_or_play(message):
                 )
             else:
                 generate_currently_playing = (
-                    lambda: f"**0.** {'**paused:** ' if message.guild.voice_client.is_paused() else ''}`{message.guild.voice_client.source.title}` (<@{player_current[message.guild.id]['queuer']}>)"
+                    lambda: f"**0.** {'**paused:** ' if message.guild.voice_client.is_paused() else ''}`{message.guild.voice_client.source.title}`"
                 )
                 if (
-                    not player_queue[message.guild.id]
+                    not playback_queue[message.guild.id]
                     and not message.guild.voice_client.source
                 ):
                     await utils.reply(
                         message,
                         "nothing is playing or queued!",
                     )
-                elif not player_queue[message.guild.id]:
+                elif not playback_queue[message.guild.id]:
                     await utils.reply(message, generate_currently_playing())
                 elif not message.guild.voice_client.source:
                     await utils.reply(
                         message,
-                        generate_queue_list(player_queue[message.guild.id]),
+                        generate_queue_list(playback_queue[message.guild.id]),
                     )
                 else:
                     await utils.reply(
                         message,
                         generate_currently_playing()
                         + "\n"
-                        + generate_queue_list(player_queue[message.guild.id]),
+                        + generate_queue_list(playback_queue[message.guild.id]),
                     )
         else:
             await utils.reply(
@@ -115,15 +107,14 @@ async def skip(message):
     if not command_allowed(message):
         return
 
-    if not player_queue[message.guild.id]:
+    if not playback_queue[message.guild.id]:
         message.guild.voice_client.stop()
         await utils.reply(
             message,
             "the queue is empty now!",
         )
     else:
-        message.guild.voice_client.stop()
-        await utils.add_check_reaction(message)
+        await play_next(message)
 
 
 async def join(message):
@@ -174,7 +165,8 @@ async def volume(message):
     parser.add_argument(
         "volume",
         nargs="?",
-        type=functools.partial(arguments.range_type, min=0, max=150),
+        type=int,
+        choices=range(0, 151),
         metavar="[0-150]",
         help="the volume level (0 - 150)",
     )
@@ -188,24 +180,28 @@ async def volume(message):
         )
         return
 
-    if args.volume is None:
+    if args.volume:
+        message.guild.voice_client.source.volume = float(args.volume) / 100.0
+        await utils.reply(
+            message,
+            f"{args.volume}",
+        )
+    else:
         await utils.reply(
             message,
             f"{int(message.guild.voice_client.source.volume * 100)}",
         )
-    else:
-        message.guild.voice_client.source.volume = float(args.volume) / 100.0
-        await utils.add_check_reaction(message)
 
 
-def play_next(message, once=False):
-    message.guild.voice_client.stop()
-    if player_queue[message.guild.id]:
-        queued = player_queue[message.guild.id].pop()
-        player_current[message.guild.id] = queued
+async def play_next(message):
+    while playback_queue[message.guild.id]:
+        queued = playback_queue[message.guild.id].pop()
+        await ensure_joined(message)
+        message.guild.voice_client.stop()
         message.guild.voice_client.play(
-            queued["player"], after=lambda _: play_next(message) if not once else None
+            queued["player"], after=lambda e: print(f"player error: {e}") if e else None
         )
+        await message.channel.send(f"**now playing:** {queued['player'].title}")
 
 
 async def ensure_joined(message):
@@ -229,3 +225,15 @@ def generate_queue_list(queue: list):
             for i, queued in enumerate(queue)
         ]
     )
+
+
+def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()

constants.py

@@ -3,20 +3,7 @@ import os
 EMBED_COLOR = 0xFF6600
 OWNERS = [531392146767347712]
 PREFIX = "%"
-RELOADABLE_MODULES = [
-    "arguments",
-    "commands",
-    "commands.bot",
-    "commands.tools",
-    "commands.utils",
-    "commands.voice",
-    "constants",
-    "core",
-    "events",
-    "utils",
-    "voice",
-    "youtubedl",
-]
+RELOAD_BLACKLISTED_MODULES = ["re", "argparse"]
 
 YTDL_OPTIONS = {
     "default_search": "auto",

core.py

@@ -1,136 +1,7 @@
-import asyncio
-import contextlib
-import importlib
-import inspect
-import io
-import textwrap
-import traceback
-
-import disnake_paginator
-
-import commands
-import constants
-import core
-import utils
-from state import command_locks
+message_handlers = {}
 
 
-async def on_message(message):
-    tokens = commands.tokenize(message.content)
-    if not tokens:
-        return
-    matched = commands.match_token(tokens[0])
-    if not matched:
-        return
-
-    if len(matched) > 1:
-        await utils.reply(
-            message,
-            f"ambiguous command, could be {' or '.join([f'`{match.value}`' for match in matched])}",
-        )
-        return
-
-    C = commands.Command
-    try:
-        match matched[0]:
-            case C.RELOAD if message.author.id in constants.OWNERS:
-                reloaded_modules = set()
-                for module in filter(
-                    lambda v: inspect.ismodule(v)
-                    and v.__name__ in constants.RELOADABLE_MODULES,
-                    globals().values(),
-                ):
-                    core.rreload(reloaded_modules, module)
-
-                await utils.add_check_reaction(message)
-            case C.EXECUTE if message.author.id in constants.OWNERS:
-                code = message.content[len(tokens[0]) + 1 :].strip().strip("`")
-                for replacement in ["python", "py"]:
-                    if code.startswith(replacement):
-                        code = code[len(replacement) :]
-
-                stdout = io.StringIO()
-                try:
-                    with contextlib.redirect_stdout(stdout):
-                        if "#globals" in code:
-                            exec(
-                                f"async def run_code():\n{textwrap.indent(code, '   ')}",
-                                globals(),
-                            )
-                            await globals()["run_code"]()
-                        else:
-                            dictionary = dict(locals(), **globals())
-                            exec(
-                                f"async def run_code():\n{textwrap.indent(code, '   ')}",
-                                dictionary,
-                                dictionary,
-                            )
-                            await dictionary["run_code"]()
-                        output = stdout.getvalue()
-                except Exception as e:
-                    output = "`" + str(e) + "`"
-
-                output = utils.filter_secrets(output)
-
-                if len(output) > 2000:
-                    output = output.replace("`", "\\`")
-                    pager = disnake_paginator.ButtonPaginator(
-                        prefix=f"```\n",
-                        suffix="```",
-                        color=constants.EMBED_COLOR,
-                        segments=disnake_paginator.split(output),
-                        invalid_user_function=utils.invalid_user_handler,
-                    )
-                    await pager.start(
-                        disnake_paginator.wrappers.MessageInteractionWrapper(message)
-                    )
-                elif len(output.strip()) == 0:
-                    await utils.add_check_reaction(message)
-                else:
-                    await message.channel.send(output)
-            case C.CLEAR | C.PURGE if message.author.id in constants.OWNERS:
-                await commands.tools.clear(message)
-            case C.JOIN:
-                await commands.voice.join(message)
-            case C.LEAVE:
-                await commands.voice.leave(message)
-            case C.QUEUE | C.PLAY:
-                if message.guild.id not in command_locks:
-                    command_locks[message.guild.id] = asyncio.Lock()
-                async with command_locks[message.guild.id]:
-                    await commands.voice.queue_or_play(message)
-            case C.SKIP:
-                await commands.voice.skip(message)
-            case C.RESUME:
-                await commands.voice.resume(message)
-            case C.PAUSE:
-                await commands.voice.pause(message)
-            case C.VOLUME:
-                await commands.voice.volume(message)
-            case C.HELP:
-                await commands.bot.help(message)
-            case C.UPTIME:
-                await commands.bot.uptime(message)
-    except Exception as e:
-        await utils.reply(
-            message,
-            f"exception occurred while processing command: ```\n{''.join(traceback.format_exception(e)).replace('`', '\\`')}```",
-        )
-
-
-def rreload(reloaded_modules, module):
-    reloaded_modules.add(module.__name__)
-    importlib.reload(module)
-    if "__reload_module__" in dir(module):
-        module.__reload_module__()
-
-    with contextlib.suppress(AttributeError):
-        for submodule in filter(
-            lambda v: inspect.ismodule(v)
-            and v.__name__ in constants.RELOADABLE_MODULES
-            and v.__name__ not in reloaded_modules,
-            map(lambda attr: getattr(module, attr), dir(module)),
-        ):
-            rreload(reloaded_modules, submodule)
-
-    importlib.reload(module)
+async def trigger_message_handlers(event_type: str, *data):
+    if event_type in message_handlers:
+        for message_handler in message_handlers[event_type]:
+            await message_handler(*data)

events.py

@@ -1,7 +1,113 @@
-dynamic_handlers = {}
+import contextlib
+import importlib
+import inspect
+import io
+import textwrap
+import traceback
+
+import disnake_paginator
+
+import commands
+import constants
+import utils
+from state import reloaded_modules
 
 
-async def trigger_dynamic_handlers(event_type: str, *data):
-    if event_type in dynamic_handlers:
-        for message_handler in dynamic_handlers[event_type]:
-            await message_handler(*data)
+async def on_message(message):
+    tokens = commands.tokenize(message.content)
+    if not tokens:
+        return
+    matched = commands.match_token(tokens[0])
+    if not matched:
+        return
+
+    if len(matched) > 1:
+        await utils.reply(
+            message,
+            f"ambiguous command, could be {' or '.join([f'`{match.value}`' for match in matched])}",
+        )
+        return
+
+    C = commands.Command
+    try:
+        match matched[0]:
+            case C.EXECUTE if message.author.id in constants.OWNERS:
+                code = message.content[len(tokens[0]) + 1 :].strip().strip("`")
+                for replacement in ["python", "py"]:
+                    if code.startswith(replacement):
+                        code = code[len(replacement) :]
+
+                stdout = io.StringIO()
+                try:
+                    with contextlib.redirect_stdout(stdout):
+                        if "#globals" in code:
+                            exec(
+                                f"async def run_code():\n{textwrap.indent(code, '   ')}",
+                                globals(),
+                            )
+                            await globals()["run_code"]()
+                        else:
+                            dictionary = dict(locals(), **globals())
+                            exec(
+                                f"async def run_code():\n{textwrap.indent(code, '   ')}",
+                                dictionary,
+                                dictionary,
+                            )
+                            await dictionary["run_code"]()
+                        output = stdout.getvalue()
+                except Exception as e:
+                    output = "`" + str(e) + "`"
+
+                output = utils.filter_secrets(output)
+
+                if len(output) > 2000:
+                    output = output.replace("`", "\\`")
+                    pager = disnake_paginator.ButtonPaginator(
+                        prefix=f"```\n",
+                        suffix="```",
+                        color=constants.EMBED_COLOR,
+                        segments=disnake_paginator.split(output),
+                        invalid_user_function=utils.invalid_user_handler,
+                    )
+                    await pager.start(
+                        disnake_paginator.wrappers.MessageInteractionWrapper(message)
+                    )
+                elif len(output.strip()) == 0:
+                    await message.add_reaction("✅")
+                else:
+                    await message.channel.send(output)
+            case C.CLEAR | C.PURGE:
+                await commands.tools.clear(message)
+            case C.JOIN:
+                await commands.voice.join(message)
+            case C.LEAVE:
+                await commands.voice.leave(message)
+            case C.QUEUE | C.PLAY:
+                await commands.voice.queue_or_play(message)
+            case C.SKIP:
+                await commands.voice.skip(message)
+            case C.RESUME:
+                await commands.voice.resume(message)
+            case C.PAUSE:
+                await commands.voice.pause(message)
+            case C.VOLUME:
+                await commands.voice.volume(message)
+            case C.UPTIME:
+                await commands.bot.uptime(message)
+    except Exception as e:
+        await utils.reply(
+            message,
+            f"exception occurred while processing command: ```\n{''.join(traceback.format_exception(e)).replace('`', '\\`')}```",
+        )
+
+
+def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()

main.py

@@ -1,9 +1,12 @@
+import importlib
+import inspect
 import time
 
+import commands
 import constants
 import core
 import events
-from state import client, start_time
+from state import client, reloaded_modules, start_time
 
 
 @client.event
@@ -13,19 +16,37 @@ async def on_ready():
 
 @client.event
 async def on_message_edit(before, after):
-    await events.trigger_dynamic_handlers("on_message_edit", before, after)
+    await core.trigger_message_handlers("on_message_edit", before, after)
 
     await on_message(after)
 
 
 @client.event
 async def on_message(message):
-    await events.trigger_dynamic_handlers("on_message", message)
+    await core.trigger_message_handlers("on_message", message)
+
+    global reloaded_modules
 
     if not message.content.startswith(constants.PREFIX):
         return
 
-    await core.on_message(message)
+    if message.author.id in constants.OWNERS and commands.match(message.content) == [
+        commands.Command.RELOAD
+    ]:
+        for name, module in globals().items():
+            if (
+                inspect.ismodule(module)
+                and name not in constants.RELOAD_BLACKLISTED_MODULES
+            ):
+                importlib.reload(module)
+                if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                    reloaded_modules.add(name)
+                    module.__reload_module__()
+        reloaded_modules.clear()
+        await message.add_reaction("✅")
+        return
+
+    await events.on_message(message)
 
 
 client.run(constants.SECRETS["TOKEN"])

state.py

@@ -2,12 +2,11 @@ import time
 
 import disnake
 
-player_queue = {}
-player_current = {}
-command_locks = {}
+start_time = time.time()
+
+playback_queue = {}
+reloaded_modules = set()
 
 intents = disnake.Intents.default()
 intents.message_content = True
 client = disnake.Client(intents=intents)
-
-start_time = time.time()

utils.py

@@ -3,10 +3,6 @@ import disnake
 import constants
 
 
-async def add_check_reaction(message):
-    await message.add_reaction("✅")
-
-
 async def reply(message, *args):
     await message.reply(*args, allowed_mentions=disnake.AllowedMentions.none())
 

ytdlp.py

@@ -1,10 +1,13 @@
 import asyncio
+import importlib
+import inspect
 from typing import Any, Optional
 
 import disnake
 import yt_dlp
 
 import constants
+from state import reloaded_modules
 
 ytdl = yt_dlp.YoutubeDL(constants.YTDL_OPTIONS)
 
@@ -35,12 +38,22 @@ class YTDLSource(disnake.PCMVolumeTransformer):
         return cls(
             disnake.FFmpegPCMAudio(
                 data["url"] if stream else ytdl.prepare_filename(data),
-                before_options="-vn -reconnect 1",
+                options="-vn -reconnect 1",
             ),
             data=data,
         )
 
 
 def __reload_module__():
+    for name, module in globals().items():
+        if (
+            inspect.ismodule(module)
+            and name not in constants.RELOAD_BLACKLISTED_MODULES
+        ):
+            importlib.reload(module)
+            if "__reload_module__" in dir(module) and name not in reloaded_modules:
+                reloaded_modules.add(name)
+                module.__reload_module__()
+
     global ytdl
     ytdl = yt_dlp.YoutubeDL(constants.YTDL_OPTIONS)