When you dont want to litter on stackoverflow

Всм, он свой id не сможет поменять, без доступа к бд телеграмма. Как и твой код, если не получит доступ к серверу на котором ты его разместил. В твоём коде уязвимостей не вижу. Хотя id лучше брать с базы данных или env.

Код безопасный

https://discordpy.readthedocs.io/en/stable/api.htm...
https://discordpy.readthedocs.io/en/stable/api.htm...

if role.mention == bad_role:
role.mention
``` Returns a string that allows you to mention the member. ```
Я думаю что проблема в условии. Вместо перебора лучше сделать условие

guild_roles_ids = [role.id for role in guild.roles]
if role.id in guild_roles_ids: ...

Та и само условие лучше вынести в отдельную функцию.
И что делат роль плохой "bad_role"? Я думаю тут лучше назвать что то вдухе forbidden_role, not_allowed_role, role_ with_restrict_access

Пусть бот поставит по одной нужной реакции под своё сообщение, только не забудь не считать их.

return ctx.message.author.id == 418786573283491853 # , а не 
return ctx.message.author.id = = 418786573283491853

import asyncio

asyns def():
         ...
         await asyncio.sleep(30)
         ....

Помогло?

Сервер в discord.py это guild, а не server, прочитай ошибку - тут написано что member не имеет атрибута server. Используй member.guild! https://discordpy.readthedocs.io/en/latest/api.htm...

Для отправки сообщение через каждых 30 минут используйте библиотеку schedule. Чтобы включать и выключать функцию сделайте переменную для функции, и в том случае если она ровна false - код просто не выполнять. Также этого можно добиться с помощью асинхронности. То есть сделать функцию которая вызывается асинхронно и меняет эту переменную, если это нужно.

Ты не внимательно прочитал документацию schudle, там нужно обновлять информацию каждую секунду

while True:
    schedule.run_pending()
    time.sleep(1)

Полный пример:

def r_check_mes(message):
    x = message.text
    if x == 'Включить':
        schedule.every(2).seconds.do(lambda: job(message))
    elif x == 'Выключить':
        schedule.cancel_job(lambda: job(message))
    while True:
           schedule.run_pending()
           time.sleep(1)

Во первых вместо time.sleep используйте schedule, по скольку ваше выполнение через неделю может сожрать гиг памяти. Во вторых, а не проще просто создать переменную и в том случае если она = False просто не выполнять цикл. Можно сделать программу асинхронной, что бы она не завершилась, после указания нечего не делать, и ждала следующих указаний. А и добавь логи с декоратором который будет ловить ошибки, а интервал бери с конфиг файла, который тебе нужно создать. Подозреваю что там очень плохая структура проекта ( не удивлюсь если ты писал всё в одном файле ) Если хочешь, то создай git репу и пул реквест, я сделаю заметки.

Во первых, не там должно быть не айди бота, а его токен, как я знаю. Во вторых, а почему у тебя отступ в строку после декоратора. А как ты эту функцию вообще вызывал. Я так понимаю, что это команда в discord. Ты что-то писал в текстовом канале на сервере к которому подключён бот?
Пример что нужно написать в канале:
*hello
Стоп, стоп, стоп, а ты саму команду то добавил?
bot.add_command(hello)