Sayedc patch 1 - #40

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@main.py`:
- Around line 5-7: Check that TOKEN (set from os.getenv("TELEGRAM_BOT_TOKEN"))
is not None/empty before constructing telebot.TeleBot; if missing, log or print
a clear error and exit (e.g., raise SystemExit or call sys.exit(1)) so
telebot.TeleBot(TOKEN) is never called with a None value. Update the
initialization flow around the TOKEN variable and the telebot.TeleBot(...) call
to perform this early validation and fail fast with a helpful message.
- Around line 9-12: The handler handle_message registered with
`@bot.message_handler`(func=lambda message: True) assumes message.text is always a
string, but message.text can be None for non-text updates; modify the handler to
either restrict the decorator to text-only messages (use content_types=['text']
instead of func=lambda message: True) or add a guard at the start of
handle_message that checks message.text is not None (e.g., if message.text is
None: return) before using "http" in message.text; reference handle_message,
`@bot.message_handler`, and message.text when applying the change.
- Around line 16-28: The code uses a fixed filename 'video.mp4' when creating
ydl_opts and downloading with yt_dlp.YoutubeDL, causing race conditions and
orphaned files; update the download/output filename to a unique per-request name
(e.g., incorporate message.message_id or a uuid) when building ydl_opts/outtmpl,
use that unique path for opening and passing to bot.send_video, and move the
os.remove cleanup into a finally block (or try/except/finally) around the
download and send operations so the temporary file is always removed even if
bot.send_video or download raises.

In `@requirements.txt`:
- Around line 1-2: requirements.txt lists python-telegram-bot but the code
imports telebot (used in main.py and commands.py) which is provided by
pyTelegramBotAPI; replace the python-telegram-bot entry with pyTelegramBotAPI
(keep yt-dlp) so pip installs the package that provides the telebot module and
avoids ImportError at import telebot in main.py/commands.py.

---

Nitpick comments:
In `@main.py`:
- Around line 30-31: Replace the broad except Exception as e /
bot.reply_to(message, f"❌ حصل خطأ: {e}") pattern with specific exception
handlers where possible (e.g., ValueError, KeyError, network-related exceptions)
and, for any remaining catch-all, log the full exception server-side using
logger.exception or logging.exception to capture stack trace, then send a
generic user-facing message via bot.reply_to (e.g., "❌ حدث خطأ غير متوقع، يرجى
المحاولة لاحقًا") to avoid leaking internal details; target the except block
around bot.reply_to in main.py and update the exception variable usage
accordingly (keep the detailed log but remove e from the message shown to
users).
- Around line 36-37: Import and invoke register_commands from commands.py in
main.py so the bot command menu is registered: add an import for
register_commands and call register_commands(bot) before starting polling (i.e.,
before bot.infinity_polling() or immediately after bot.remove_webhook()),
referencing the register_commands function and the existing bot.infinity_polling
call; alternatively, if the commands file is no longer needed, remove
commands.py and its tests instead of calling register_commands.