fix(logs): simplify log file retrieval to avoid some errors

Author: HitaloM

.github/copilot-instructions.md

@@ -2,12 +2,12 @@
 
 We are building a Python project using Hydrogram (a Pyrogram fork) for Telegram bot development. When generating or suggesting code, follow these guidelines:
 
-1. Use Python 3.13 features appropriately:
+1. Use latest Python features appropriately:
    - Type hints with the latest typing module features
-   - Pattern matching with match/case for handling different message types
+   - Pattern matching and structural pattern matching
    - F-strings for all string formatting
    - Walrus operator (:=) when it improves readability
-   - Use structural pattern matching for command handling
+   - And all other latest Python features
 
 2. Follow PEP 8 style guidelines:
    - 4 spaces for indentation
@@ -16,15 +16,18 @@ We are building a Python project using Hydrogram (a Pyrogram fork) for Telegram
    - PascalCase for classes
    - UPPER_CASE for constants
    - Use positional arguments for logging statements instead of f-strings
+   - Code style consistency using Ruff.
 
 3. Project architecture:
    - Keep functions small and focused on a single responsibility
    - Use type hints consistently on all functions and methods
-   - Prefer composition over inheritance
+   - Prefer nheritance over composition when appropriate
    - Use async/await for all I/O operations
+   - Use context managers for file and resource management
+   - Use async context managers for async I/O operations
 
 4. Code organization:
-   - Organize related functionality into separate modules
+   - Organize code into modules and packages
    - Use relative imports within the package
    - Follow a consistent naming pattern for handler functions
 

src/korone/modules/sudo/handlers/logs.py

@@ -1,10 +1,13 @@
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright (c) 2025 Hitalo M. <https://github.com/HitaloM>
 
-import operator
+import contextlib
 from datetime import UTC, datetime
+from io import BytesIO
 
+import aiofiles
 from hydrogram import Client
+from hydrogram.errors import MessageDeleteForbidden
 from hydrogram.types import Message
 
 from korone.decorators import router
@@ -14,30 +17,38 @@
 
 @router.message(Command("logs", disableable=False) & IsSudo)
 async def handle_logs_command(client: Client, message: Message) -> None:
-    if not (log_files := list(LOG_DIR.glob("korone.log*"))):
+    log_files = list(LOG_DIR.glob("korone.log*"))
+    if not log_files:
         await message.reply("No log files found.")
         return
 
-    file_times = {file: file.stat().st_mtime for file in log_files}
-    match max(file_times.items(), key=operator.itemgetter(1)):
-        case (most_recent_file, mtime):
-            file_size = round(most_recent_file.stat().st_size / (1024 * 1024), 2)
-            timestamp = datetime.fromtimestamp(mtime, UTC).strftime("%Y-%m-%d %H:%M:%S")
-            caption = (
-                f"<b>📄 Log File</b>\n"
-                f"<b>File:</b> <code>{most_recent_file.name}</code>\n"
-                f"<b>Size:</b> <code>{file_size} MB</code>\n"
-                f"<b>Last Modified:</b> <code>{timestamp} UTC</code>"
-            )
+    most_recent_file = max(log_files, key=lambda f: f.stat().st_mtime)
+    file_size = round(most_recent_file.stat().st_size / (1024 * 1024), 2)
+    timestamp = datetime.fromtimestamp(most_recent_file.stat().st_mtime, UTC).strftime(
+        "%Y-%m-%d %H:%M:%S"
+    )
+    caption = (
+        f"<b>📄 Log File</b>\n"
+        f"<b>File:</b> <code>{most_recent_file.name}</code>\n"
+        f"<b>Size:</b> <code>{file_size} MB</code>\n"
+        f"<b>Last Modified:</b> <code>{timestamp} UTC</code>"
+    )
 
     status_message = await message.reply("Uploading log file...")
 
     try:
+        async with aiofiles.open(most_recent_file, "rb") as file:
+            file_buffer = BytesIO(await file.read())
+            file_buffer.seek(0)
+
         await message.reply_document(
-            document=str(most_recent_file),
+            document=file_buffer,
             caption=caption,
             file_name=most_recent_file.name,
         )
-        await status_message.delete()
+
+        with contextlib.suppress(MessageDeleteForbidden):
+            await status_message.delete()
+
     except Exception as exc:
-        await status_message.edit(f"Error sending log file: {exc!s}")
+        await status_message.edit(f"Error sending log file: {exc}")