fenrir

Что лучше вытаскивать данные из БД(sqlite) или хранить в неких переменных?

Если данные критичные и недопустима их потеря - хранить в БД, если допустимо их потерять из-за ошибок, отключений и прочего - хранить в переменных. Если и то и то, накапливать какое-то количество данных и скидывать в БД сразу батчем по таймеру или по достижению определенного размера.

async def on_message(self, message: Message): ...

on_message это событие, которое срабатывает на каждое сообщение, каждого пользователя. На среднего размера боте с несколькими сотнями серверов в активное вечернее время оно может дергаться тысячи раз в секунду, как сами думаете, насколько адекватно в таком событии ходить в базу? Насколько часто меняются каналы, чтобы каждый раз их доставать из БД? Знаешь ли условия, при которых данные в базе меняются и их нужно повторно перечитать?

guild_id = message.guild.id

Как думаете, что лежит в guild, если боту пишут в личные сообщения?

Либо включайте голову и думайте почему вы делаете то или иное действие: нужны ли вам данные на чтение или на запись, могут ли они одновременно измениться, обрабатываются ли корнер кейсы и прочее или пишите как пишется, оптимизацией займетесь если дойдете до того, что она понадобится.

Подскажите, как лучше для оптимизации процессов, и где можно на будущие брать полезную информацию о python.

Читать книги, слушать подкасты, ходить на конференции.

sqlite3.OperationalError: no such column: telegram_id

Помочь с переводом ошибки? В таблице нет колонки telegram_id.

С учетом того, что у тебя таблица создается с правилом IF NOT EXISTS вероятно ранее уже создана таблица без этой колонки. Добавить первичный ключ в существующую таблицу можно только при помощи определенных танцев с бубном, вероятно стоит удалить таблицу и создать заново.

1) Записать в БД время, когда нужно разбанить и id пользователя
2) Сделать обычный бан
3) Периодически (скажем раз в минуту) ходить в базу и доставать записи для которых время разбана меньше текущего времени
4) Выполнить разбан всех полученных id, удалить записи из БД

sqlite не имеет встроенного формата для дат и времени, так что я бы рекомендовал хранить время как INT, преобразуя datetime в timestamp.

Потому что нужно читать ошибку, а затем число открывающих и закрывающих скобок, если за вас это не делает IDE.
И не используйте f строки в sql запросах, на эту тему есть множество ответов.

Ответ на конкретный вопрос как убрать какие-то символы - посмотрите документацию по replace(), translate(), join() и что такое срез строк.

Ответ на реальный вопрос, который вы не смогли сформулировать: не использовать список для admins, когда нужна просто строка и не делать SELECT *, когда нужен только ID

В таблицу добавить ключевое поле GUILD_ID
Соответственно запрос изменится на:

select = '''SELECT * FROM "users" WHERE id = %s AND guild_id = %s'''
cur.execute(select, (member.id, guild.id))