fix: Resolve failing linter checks (#1361)

* ci: weekly check.

updates:
- [github.com/charliermarsh/ruff-pre-commit: v0.0.254 → v0.0.261](astral-sh/ruff-pre-commit@v0.0.254...v0.0.261)
- [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0)

* fix:  Resolve failing linter checks

* ci: correct from checks.

* scream: I swear this bot is trying to break my commits on purpose

* Bump max-complexity to a number we're unlikely to hit

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: silasary

.pre-commit-config.yaml

@@ -30,12 +30,12 @@ repos:
     - id: check-merge-conflict
       name: Merge Conflicts
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: 'v0.0.254'
+    rev: 'v0.0.261'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
   - repo: https://github.com/psf/black
-    rev: 23.1.0
+    rev: 23.3.0
     hooks:
       - id: black
         name: Black Formatting

interactions/api/gateway/websocket.py

@@ -164,7 +164,7 @@ async def send_json(self, data: dict, bypass=False) -> None:
         serialized = FastJson.dumps(data)
         await self.send(serialized, bypass)
 
-    async def receive(self, force: bool = False) -> str:
+    async def receive(self, force: bool = False) -> str:  # noqa: C901
         """
         Receive a full event payload from the WebSocket.
 

interactions/api/http/http_client.py

@@ -334,7 +334,7 @@ def _process_payload(
         form_data.add_field("payload_json", FastJson.dumps(payload))
         return form_data
 
-    async def request(
+    async def request(  # noqa: C901
         self,
         route: Route,
         payload: list | dict | None = None,

interactions/api/voice/voice_gateway.py

@@ -100,7 +100,7 @@ async def run(self) -> None:
             # possible race conditions to consider.
             await self.dispatch_opcode(data, op)
 
-    async def receive(self, force=False) -> str:
+    async def receive(self, force=False) -> str:  # noqa: C901
         buffer = bytearray()
 
         while True:

interactions/client/client.py

@@ -1691,7 +1691,7 @@ async def _run_slash_command(self, command: SlashCommand, ctx: "InteractionConte
         return await command(ctx, **ctx.kwargs)
 
     @processors.Processor.define("raw_interaction_create")
-    async def _dispatch_interaction(self, event: RawGatewayEvent) -> None:
+    async def _dispatch_interaction(self, event: RawGatewayEvent) -> None:  # noqa: C901
         """
         Identify and dispatch interaction of slash commands or components.
 

interactions/ext/debug_extension/debug_exec.py

@@ -95,7 +95,7 @@ async def handle_exec_result(self, ctx: ModalContext, result: Any, value: Any, b
         if result is None:
             result = value or "No Output!"
 
-        if isinstance(result, Message):
+        elif isinstance(result, Message):
             try:
                 e = debug_embed("Exec", timestamp=result.created_at, url=result.jump_url)
                 e.description = result.content
@@ -109,13 +109,13 @@ async def handle_exec_result(self, ctx: ModalContext, result: Any, value: Any, b
             except Exception:
                 return await ctx.send(result.jump_url)
 
-        if isinstance(result, Embed):
+        elif isinstance(result, Embed):
             return await ctx.send(embeds=[result])
 
-        if isinstance(result, File):
+        elif isinstance(result, File):
             return await ctx.send(file=result)
 
-        if isinstance(result, Paginator):
+        elif isinstance(result, Paginator):
             return await result.send(ctx)
 
         if hasattr(result, "__iter__"):

interactions/ext/jurigged.py

@@ -112,7 +112,7 @@ def jurigged_prerun(self, _path: str, cf: CodeFile) -> None:
             self.bot.logger.debug(f"Caching {cf.module_name}")
             self.command_cache = get_all_commands(cf.module)
 
-    def jurigged_postrun(self, _path: str, cf: CodeFile) -> None:
+    def jurigged_postrun(self, _path: str, cf: CodeFile) -> None:  # noqa: C901
         """
         Jurigged postrun event.
 

interactions/ext/prefixed_commands/command.py

@@ -465,7 +465,7 @@ def is_subcommand(self) -> bool:
         """Return whether this command is a subcommand or not."""
         return bool(self.parent)
 
-    def _parse_parameters(self) -> None:
+    def _parse_parameters(self) -> None:  # noqa: C901
         """
         Parses the parameters that this command has into a form i.py can use.
 
@@ -668,7 +668,7 @@ def wrapper(func: Callable) -> Self:
 
         return wrapper
 
-    async def call_callback(self, callback: Callable, ctx: "PrefixedContext") -> None:
+    async def call_callback(self, callback: Callable, ctx: "PrefixedContext") -> None:  # noqa: C901
         """
         Runs the callback of this command.
 

interactions/ext/prefixed_commands/manager.py

@@ -243,7 +243,7 @@ async def _handle_ext_unload(self, event: ExtensionUnload) -> None:
             self.remove_command(name)
 
     @listen("raw_message_create", is_default_listener=True)
-    async def _dispatch_prefixed_commands(self, event: RawGatewayEvent) -> None:
+    async def _dispatch_prefixed_commands(self, event: RawGatewayEvent) -> None:  # noqa: C901
         """Determine if a prefixed command is being triggered, and dispatch it."""
         # don't waste time processing this if there are no prefixed commands
         if not self.commands:

interactions/models/discord/guild.py

@@ -1664,6 +1664,7 @@ async def ban(
             warn(
                 "delete_message_days is deprecated and will be removed in a future update",
                 DeprecationWarning,
+                stacklevel=2,
             )
             delete_message_seconds = delete_message_days * 3600
         await self._client.http.create_guild_ban(self.id, to_snowflake(user), delete_message_seconds, reason=reason)