bot/add_character.py

@@ -1,12 +1,11 @@
 import requests
 from misskey.exceptions import MisskeyAPIException
 from client import client_connection
-from db_utils import insert_character
+from db_utils import get_db_connection
 
 def add_character(name: str, rarity: int, weight: float, image_url: str) -> tuple[int, str]:
     """
-    Adds a character to the database, uploading the image from a public URL to
+    Adds a character to the database, uploading the image from a public URL to the bot's Misskey Drive.
-    the bot's Misskey Drive.
 
     Args:
         name (str): Character name.
@@ -21,29 +20,41 @@ def add_character(name: str, rarity: int, weight: float, image_url: str) -> tupl
         ValueError: If inputs are invalid.
         RuntimeError: If image download/upload or database operation fails.
     """
-    # Validate inputs
-    if not name or not name.strip():
-        raise ValueError("Character name cannot be empty.")
-    if not isinstance(rarity, int) or rarity < 1:
-        raise ValueError("Rarity must be a positive integer.")
-    if not isinstance(weight, (int, float)) or weight <= 0:
-        raise ValueError("Weight must be a positive number.")
-    if not image_url:
-        raise ValueError("Image URL must be provided.")
-
-    # Download image
-    response = requests.get(image_url, stream=True, timeout=30)
-    if response.status_code != 200:
-        raise RuntimeError(f"Failed to download image from {image_url}")
-
-    # Upload to bot's Drive
-    mk = client_connection()
     try:
-        media = mk.drive_files_create(response.raw)
+        # Validate inputs
-        file_id = media["id"]
+        if not name or not name.strip():
-    except MisskeyAPIException as e:
+            raise ValueError("Character name cannot be empty.")
-        raise RuntimeError(f"Failed to upload image to bot's Drive: {e}") from e
+        if not isinstance(rarity, int) or rarity < 1:
+            raise ValueError("Rarity must be a positive integer.")
+        if not isinstance(weight, (int, float)) or weight <= 0:
+            raise ValueError("Weight must be a positive number.")
+        if not image_url:
+            raise ValueError("Image URL must be provided.")
 
-    # Insert into database
+        # Download image
-    character_id = insert_character(name.strip(), rarity, float(weight), file_id)
+        response = requests.get(image_url, stream=True, timeout=30)
-    return character_id, file_id
+        if response.status_code != 200:
+            raise RuntimeError(f"Failed to download image from {image_url}")
+
+        # Upload to bot's Drive
+        mk = client_connection()
+        try:
+            media = mk.drive_files_create(response.raw)
+            file_id = media["id"]
+        except MisskeyAPIException as e:
+            raise RuntimeError(f"Failed to upload image to bot's Drive: {e}") from e
+
+        # Insert into database
+        conn = get_db_connection()
+        cur = conn.cursor()
+        cur.execute(
+            'INSERT INTO characters (name, rarity, weight, file_id) VALUES (?, ?, ?, ?)',
+            (name.strip(), rarity, float(weight), file_id)
+        )
+        conn.commit()
+        character_id = cur.lastrowid
+
+        return character_id, file_id
+    finally:
+        if 'conn' in locals():
+            conn.close()

bot/bot_app.py

@@ -1,7 +1,6 @@
 import time
 import misskey as misskey
 from client import client_connection
-import db_utils as db
 
 from config import NOTIFICATION_POLL_INTERVAL
 from notification import process_notifications
@@ -9,9 +8,6 @@ from notification import process_notifications
 if __name__ == '__main__':
     # Initialize the Misskey client
     client = client_connection()
-    # Connect to DB
-    db.connect()
-
     print('Listening for notifications...')
     while True:
         if not process_notifications(client):

bot/db_utils.py

@@ -1,79 +1,68 @@
-from random import choices
 import sqlite3
 import config
 
 DB_PATH = config.DB_PATH
-CONNECTION: sqlite3.Connection
-CURSOR: sqlite3.Cursor
 
-def connect() -> None:
+def get_db_connection():
     '''Creates a connection to the database'''
-    print('Connecting to the database...')
+    conn = sqlite3.connect(DB_PATH)
-    global CONNECTION
+    conn.row_factory = sqlite3.Row
-    global CURSOR
+    return conn
-    CONNECTION = sqlite3.connect(DB_PATH, autocommit=True)
-    CONNECTION.row_factory = sqlite3.Row
-    CURSOR = CONNECTION.cursor()
-
-def get_random_character():
-    ''' Gets a random character from the database'''
-    CURSOR.execute('SELECT * FROM characters')
-    characters = CURSOR.fetchall()
-
-    if not characters:
-        return None, None, None, None
-
-    weights = [c['weight'] for c in characters]
-    chosen = choices(characters, weights=weights, k=1)[0]
-
-    return chosen['id'], chosen['name'], chosen['file_id'], chosen['rarity']
 
 def get_or_create_user(username):
     '''Retrieves an ID for a given user, if the user does not exist, it will be
     created.'''
-    CURSOR.execute('SELECT id FROM users WHERE username = ?', (username,))
+    conn = get_db_connection()
-    user = CURSOR.fetchone()
+    conn.row_factory = sqlite3.Row
+    cur = conn.cursor()
+    cur.execute('SELECT id FROM users WHERE username = ?', (username,))
+    user = cur.fetchone()
     if user:
+        conn.close()
         return user[0]
 
     # New user starts with has_rolled = False
-    CURSOR.execute(
+    cur.execute(
         'INSERT INTO users (username, has_rolled) VALUES (?, ?)',
         (username, False)
     )
-    user_id = CURSOR.lastrowid
+    conn.commit()
+    user_id = cur.lastrowid
+    conn.close()
     return user_id
 
-def insert_character(name: str, rarity: int, weight: float, file_id: str) -> int:
+def add_pull(user_id, character_id):
-    '''Inserts a character'''
-    CURSOR.execute(
-        'INSERT INTO characters (name, rarity, weight, file_id) VALUES (?, ?, ?, ?)',
-        (name, rarity, weight, file_id)
-    )
-    character_id = CURSOR.lastrowid
-    return character_id if character_id else 0
-
-def insert_pull(user_id, character_id):
     '''Creates a pull in the database'''
-    CURSOR.execute(
+    conn = get_db_connection()
-        'INSERT INTO pulls (user_id, character_id) VALUES (?, ?)',
+    cur = conn.cursor()
-        (user_id, character_id)
+    cur.execute('INSERT INTO pulls (user_id, character_id) VALUES (?, ?)', (user_id, character_id))
-    )
+    conn.commit()
+    conn.close()
 
 def get_last_rolled_at(user_id):
     '''Gets the timestamp when the user last rolled'''
-    CURSOR.execute("SELECT timestamp FROM pulls WHERE user_id = ? ORDER BY timestamp DESC", \
+    conn = get_db_connection()
+    cur = conn.cursor()
+    cur.execute("SELECT timestamp FROM pulls WHERE user_id = ? ORDER BY timestamp DESC", \
             (user_id,))
-    row = CURSOR.fetchone()
+    row = cur.fetchone()
+    conn.close()
     return row[0] if row else None
 
 
 def get_config(key):
     '''Reads the value for a specified config key from the db'''
-    CURSOR.execute("SELECT value FROM config WHERE key = ?", (key,))
+    conn = get_db_connection()
-    row = CURSOR.fetchone()
+    cur = conn.cursor()
+    cur.execute("SELECT value FROM config WHERE key = ?", (key,))
+    row = cur.fetchone()
+    conn.close()
     return row[0] if row else None
 
 def set_config(key, value):
     '''Writes the value for a specified config key to the db'''
-    CURSOR.execute("INSERT OR REPLACE INTO config (key, value) VALUES (?, ?)", (key, value))
+    conn = get_db_connection()
+    cur = conn.cursor()
+    cur.execute("INSERT OR REPLACE INTO config (key, value) VALUES (?, ?)", (key, value))
+    conn.commit()
+    conn.close()

bot/response.py

@@ -1,8 +1,25 @@
+import random
 from datetime import datetime, timedelta, timezone
-from db_utils import get_or_create_user, insert_pull, get_last_rolled_at, get_random_character
+from db_utils import get_or_create_user, add_pull, get_db_connection, get_last_rolled_at
 from add_character import add_character
 from config import GACHA_ROLL_INTERVAL
 
+def get_character():
+    ''' Gets a random character from the database'''
+    conn = get_db_connection()
+    cur = conn.cursor()
+    cur.execute('SELECT * FROM characters')
+    characters = cur.fetchall()
+    conn.close()
+
+    if not characters:
+        return None, None, None, None
+
+    weights = [c['weight'] for c in characters]
+    chosen = random.choices(characters, weights=weights, k=1)[0]
+
+    return chosen['id'], chosen['name'], chosen['file_id'], chosen['rarity']
+
 def do_roll(full_user):
     '''Determines whether the user can roll, then pulls a random character'''
     user_id = get_or_create_user(full_user)
@@ -33,12 +50,12 @@ def do_roll(full_user):
 
             return f'{full_user} ⏱️ Please wait another {remaining_duration} before rolling again.'
 
-    character_id, character_name, file_id, rarity = get_random_character()
+    character_id, character_name, file_id, rarity = get_character()
 
     if not character_id:
         return f'{full_user} Uwaaa... something went wrong! No characters found. 😿'
 
-    insert_pull(user_id,character_id)
+    add_pull(user_id,character_id)
     stars = '⭐️' * rarity
     return([f"@{full_user} 🎲 Congrats! You rolled {stars} **{character_name}**\n\
             She's all yours now~ 💖✨",[file_id]])