Refactor, save video_id in playlist_tracks and use it to detect existance.

Signed-off-by: Abdulkadir Furkan Şanlı <me@abdulocra.cy>

main.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-"""parkerbot: Matrix bot to generate YouTube (music) playlists from links sent to a channel."""
+"""ParkerBot"""
 
 import argparse
 import asyncio
@@ -18,37 +18,46 @@ from nio import AsyncClient, RoomMessageText, SyncResponse, UploadResponse
 
 DATA_DIR = os.getenv("DATA_DIR", "./")
 DB_PATH = os.path.join(DATA_DIR, "parkerbot.sqlite3")
-TOKEN_PATH = os.path.join(DATA_DIR, "sync_token")
 PICKLE_PATH = os.path.join(DATA_DIR, "token.pickle")
+TOKEN_PATH = os.path.join(DATA_DIR, "sync_token")
 
 MATRIX_SERVER = os.getenv("MATRIX_SERVER")
 MATRIX_ROOM = os.getenv("MATRIX_ROOM")
 MATRIX_USER = os.getenv("MATRIX_USER")
 MATRIX_PASSWORD = os.getenv("MATRIX_PASSWORD")
 
-YOUTUBE_PLAYLIST_TITLE = os.getenv("YOUTUBE_PLAYLIST_TITLE")
 YOUTUBE_CLIENT_SECRETS_FILE = os.getenv("YOUTUBE_CLIENT_SECRETS_FILE")
-YOUTUBE_API_SERVICE_NAME = "youtube"
+YOUTUBE_PLAYLIST_TITLE = os.getenv("YOUTUBE_PLAYLIST_TITLE")
-YOUTUBE_API_VERSION = "v3"
 
-conn = sqlite3.connect(DB_PATH)
+
-cursor = conn.cursor()
+def connect_db():
+    """Connect to DB and return connection and cursor."""
+    conn = sqlite3.connect(DB_PATH)
+    cursor = conn.cursor()
+    return conn, cursor
 
 
 def parse_arguments():
     """Parse command line arguments."""
     parser = argparse.ArgumentParser(
-        description="Matrix bot to generate YouTube (music) playlists from links sent to a channel."
+        description=(
+            "Matrix bot to generate YouTube (music) playlists from links sent "
+            "to a channel."
+        )
     )
     parser.add_argument(
         "--backwards-sync",
         action="store_true",
-        help="Run backwards sync on start (this may cause you to exceed your daily API quota).",
+        help=(
+            "Run backwards sync on start. This most probably will cause you to "
+            "exceed your YouTube daily API quota, and other hidden YouTube rate"
+            " limits."
+        ),
     )
     return parser.parse_args()
 
 
-def define_tables():
+def define_tables(conn, cursor):
     """Define tables for use with program."""
     with conn:
         cursor.execute(
@@ -66,11 +75,12 @@ def define_tables():
                 playlist_id TEXT UNIQUE,
                 creation_date DATE)"""
         )
-        cursor.execute(
+        cursor.execute(  # TODO: Write migration script to add video_id.
             """CREATE TABLE IF NOT EXISTS playlist_tracks (
                 id INTEGER PRIMARY KEY AUTOINCREMENT,
                 playlist_id INTEGER,
                 message_id INTEGER,
+                video_id TEXT,
                 FOREIGN KEY (playlist_id) REFERENCES playlists(id),
                 FOREIGN KEY (message_id) REFERENCES messages(id),
                 UNIQUE (playlist_id, message_id))"""
@@ -102,8 +112,8 @@ def get_authenticated_service():
     return build("youtube", "v3", credentials=credentials)
 
 
-def get_monday_date(timestamp):
+def monday_date(timestamp):
-    """Get Monday of the week for the given timestamp. Weeks start on Monday."""
+    """Return Monday of week for given timestamp. Weeks start on Monday."""
     date = datetime.datetime.fromtimestamp(timestamp / 1000, datetime.UTC)
     return date - datetime.timedelta(days=date.weekday())
 
@@ -128,30 +138,26 @@ def make_playlist(youtube, title):
     return response["id"]
 
 
-def get_or_make_playlist(youtube, monday_date):
+def get_or_make_playlist(conn, cursor, youtube, playlist_date):
     """Get ID of playlist for given Monday's week, make if doesn't exist."""
-    playlist_title = f"{YOUTUBE_PLAYLIST_TITLE} {monday_date.strftime('%Y-%m-%d')}"
+    title = f"{YOUTUBE_PLAYLIST_TITLE} {playlist_date.strftime('%Y-%m-%d')}"
 
-    # Check if playlist exists in the database
+    cursor.execute("SELECT playlist_id FROM playlists WHERE title = ?", (title,))
-    cursor.execute(
-        "SELECT playlist_id FROM playlists WHERE title = ?", (playlist_title,)
-    )
     row = cursor.fetchone()
     if row:
-        return row[0]  # Playlist already exists
+        return row[0]
 
-    # If not, make a new playlist on YouTube and save it in the database
+    playlist_id = make_playlist(youtube, title)
-    playlist_id = make_playlist(youtube, playlist_title)
     with conn:
-        cursor.execute(
+        cursor.execute(  # TODO: https://docs.python.org/3/library/sqlite3.html#default-adapters-and-converters-deprecated
             "INSERT INTO playlists (title, playlist_id, creation_date) VALUES (?, ?, ?)",
-            (playlist_title, playlist_id, monday_date),
+            (title, playlist_id, playlist_date),
         )
 
     return playlist_id
 
 
-def add_video_to_playlist(youtube, playlist_id, video_id, retry_count=3):
+def add_video_to_playlist(youtube, playlist_id, video_id, retry_count=6):
     """Add video to playlist."""
     for attempt in range(retry_count):
         try:
@@ -178,34 +184,15 @@ def is_music(youtube, video_id):
 
     # Check if the video category is Music (typically category ID 10)
     return video_details["items"][0]["snippet"]["categoryId"] in (
-        "10",
+        "10",  # music
-        "24",
+        "24",  # entertainment
-    )  # music, entertainment
+    )
 
 
-async def message_callback(client, room, event):
+async def send_intro_message(client, sender, room_id):
-    """Event handler for received messages."""
+    """Sends introduction message in reply to sender, in room with room_id."""
-    youtube_link_pattern = r"(https?://(?:www\.|music\.)?youtube\.com/(?!playlist\?list=)watch\?v=[\w-]+|https?://youtu\.be/[\w-]+)"
-    sender = event.sender
-    if sender != MATRIX_USER:
-        body = event.body.strip()
-        timestamp = event.server_timestamp
-        room_id = room.room_id
-        monday_date = get_monday_date(timestamp)
-        youtube = get_authenticated_service()
-        playlist_id = get_or_make_playlist(youtube, monday_date)
-        youtube_links = re.findall(youtube_link_pattern, body)
-
-        timestamp_sec = datetime.datetime.fromtimestamp(
-            event.server_timestamp / 1000, datetime.UTC
-        )  # milisec to sec
-        current_time = datetime.datetime.now(datetime.UTC)
-
-        if body == "!parkerbot" and current_time - timestamp_sec < datetime.timedelta(
-            seconds=30
-        ):
     intro_message = (
-                "Hi, I'm ParkerBot! I generate YouTube playlists from links "
+        f"Hi {sender}, I'm ParkerBot! I generate YouTube playlists from links "
         "sent to this channel. You can find my source code here: "
         "https://git.abdulocra.cy/abdulocracy/parkerbot"
     )
@@ -214,7 +201,9 @@ async def message_callback(client, room, event):
         message_type="m.room.message",
         content={"msgtype": "m.text", "body": intro_message},
     )
-            with open("./parker.gif", "rb") as gif_file:  # this is broken as shit
+
+    # TODO: Figure out how to properly send GIF, this is broken as shit.
+    with open("./parker.gif", "rb") as gif_file:
         response = await client.upload(gif_file, content_type="image/gif")
     if isinstance(response, UploadResponse):
         print("Image was uploaded successfully to server. ")
@@ -232,9 +221,9 @@ async def message_callback(client, room, event):
     else:
         print(f"Failed to upload image. Failure response: {response}")
 
-        if body == "!pow" and current_time - timestamp_sec < datetime.timedelta(
+
-            seconds=30
+async def send_playlist_of_week(client, sender, room_id, playlist_id):
-        ):
+    """Sends playlist of the week in reply to sender, in room with room_id."""
     playlist_link = f"https://www.youtube.com/playlist?list={playlist_id}"
     reply_msg = f"{sender}, here's the playlist of the week: {playlist_link}"
     await client.room_send(
@@ -243,74 +232,120 @@ async def message_callback(client, room, event):
         content={"msgtype": "m.text", "body": reply_msg},
     )
 
+
+async def message_callback(conn, cursor, youtube, client, room, event):
+    """Event handler for received messages."""
+    sender = event.sender
+    if sender != MATRIX_USER:
+        body = event.body.strip()
+        timestamp = event.server_timestamp
+        playlist_id = get_or_make_playlist(
+            conn, cursor, youtube, monday_date(timestamp)
+        )
+
+        timestamp_sec = datetime.datetime.fromtimestamp(
+            event.server_timestamp / 1000, datetime.UTC  # millisec to sec
+        )
+        current_time = datetime.datetime.now(datetime.UTC)
+        recent = current_time - timestamp_sec < datetime.timedelta(seconds=30)
+
+        if body == "!parkerbot" and recent:
+            await send_intro_message(client, sender, room.room_id)
+            return
+
+        if body == "!pow" and recent:
+            await send_playlist_of_week(client, sender, room.room_id, playlist_id)
+            return
+
+        youtube_link_pattern = (
+            r"(https?://(?:www\.|music\.)?youtube\.com/(?!playlist\?list=)watch"
+            r"\?v=[\w-]+|https?://youtu\.be/[\w-]+)"
+        )
+        youtube_links = re.findall(youtube_link_pattern, body)
+
         for link in youtube_links:
             video_id = link.split("v=")[-1].split("&")[0].split("/")[-1]
             if is_music(youtube, video_id):
+                message_id = record_message(conn, cursor, sender, link, timestamp)
+                if in_playlist(cursor, video_id, playlist_id):
+                    print(f"Track is already in this week's playlist: {link}")
+                else:
+                    # Add video to playlist and record it in the database
+                    add_video_to_playlist(youtube, playlist_id, video_id)
+                    with conn:
+                        cursor.execute(
+                            (
+                                "INSERT INTO playlist_tracks (playlist_id, message_id, video_id) "
+                                "VALUES (?, ?, ?)"
+                            ),
+                            (playlist_id, message_id, video_id),
+                        )
+                    print(f"Added track to this week's playlist: {link}")
+
+
+def in_playlist(cursor, video_id, playlist_id):
+    """Checks if video is in playlist."""
+    cursor.execute(
+        "SELECT id FROM playlist_tracks WHERE video_id = ? AND playlist_id = ?",
+        (video_id, playlist_id),
+    )
+    if cursor.fetchone():
+        return True
+    return False
+
+
+def record_message(conn, cursor, sender, link, timestamp):
+    """Records message to messages table in DB, returns ID."""
     try:
+        with conn:
             cursor.execute(
                 "INSERT INTO messages (sender, message, timestamp) VALUES (?, ?, ?)",
                 (sender, link, timestamp),
             )
-                    conn.commit()
+        print(f"Saved message: {sender} {link} {timestamp}")
-                    print(f"Saved YouTube link from {sender}: {link}")
     except sqlite3.IntegrityError as e:
         if "UNIQUE constraint failed" in str(e):
             print(f"Entry already exists: {sender} {link} {timestamp}")
         else:
             raise e
 
-                # Check if the link is already added to any playlist
-                cursor.execute("SELECT id FROM messages WHERE message = ?", (link,))
-                message_row = cursor.fetchone()
-
-                if message_row:
     cursor.execute(
-                        "SELECT id FROM playlist_tracks WHERE message_id = ? AND playlist_id = ?",
+        "SELECT id FROM messages WHERE sender = ? AND message = ? AND timestamp = ?",
-                        (message_row[0], playlist_id),
+        (sender, link, timestamp),
     )
-                    track_row = cursor.fetchone()
+    return cursor.fetchone()[0]
-
-                    if track_row:
-                        print(f"Track already in playlist: {link}")
-                    else:
-                        # Add video to playlist and record it in the database
-                        add_video_to_playlist(youtube, playlist_id, video_id)
-                        with conn:
-                            cursor.execute(
-                                "INSERT INTO playlist_tracks (playlist_id, message_id) VALUES (?, ?)",
-                                (playlist_id, message_row[0]),
-                            )
-                        print(f"Added track to playlist: {link}")
 
 
 async def sync_callback(response):
-    """Save Matrix sync token."""
+    """Saves Matrix sync token."""
-    # Save the sync token to a file or handle it as needed
+    with open(TOKEN_PATH, "w", encoding="utf-8") as f:
-    with open(TOKEN_PATH, "w") as f:
         f.write(response.next_batch)
 
 
 def load_sync_token():
-    """Get an existing Matrix sync token if it exists."""
+    """Gets saved Matrix sync token if it exists."""
     try:
-        with open(TOKEN_PATH, "r") as file:
+        with open(TOKEN_PATH, "r", encoding="utf-8") as file:
             return file.read().strip()
     except FileNotFoundError:
         return None
 
 
-async def get_client():
+async def get_client(conn, cursor, youtube):
     """Returns configured and logged in Matrix client."""
     client = AsyncClient(MATRIX_SERVER, MATRIX_USER)
     client.add_event_callback(
-        lambda room, event: message_callback(client, room, event), RoomMessageText
+        lambda room, event: message_callback(
+            conn, cursor, youtube, client, room, event
+        ),
+        RoomMessageText,
     )
     client.add_response_callback(sync_callback, SyncResponse)
     print(await client.login(MATRIX_PASSWORD))
     return client
 
 
-async def backwards_sync(client, room, start_token):
+async def backwards_sync(conn, cursor, youtube, client, room, start_token):
     """Fetch and process historical messages from a given room."""
     print("Starting to process channel log...")
     from_token = start_token
@@ -322,7 +357,7 @@ async def backwards_sync(client, room, start_token):
         # Process each message
         for event in response.chunk:
             if isinstance(event, RoomMessageText):
-                await message_callback(client, room, event)
+                await message_callback(conn, cursor, youtube, client, room, event)
 
         # Break if there are no more messages to fetch
         if not response.end or response.end == from_token:
@@ -335,14 +370,17 @@ async def backwards_sync(client, room, start_token):
 async def main():
     """Get DB and Matrix client ready, and start syncing."""
     args = parse_arguments()
-    define_tables()
+    conn, cursor = connect_db()
-    client = await get_client()
+    define_tables(conn, cursor)
+    youtube = get_authenticated_service()
+    client = await get_client(conn, cursor, youtube)
     sync_token = load_sync_token()
 
+    # This is incredibly dumb and most probably will exceed your YouTube API quota.
     if args.backwards_sync:
         init_sync = await client.sync(30000)
         room = await client.room_resolve_alias(MATRIX_ROOM)
-        await backwards_sync(client, room, init_sync.next_batch)
+        await backwards_sync(conn, cursor, youtube, client, room, init_sync.next_batch)
 
     await client.sync_forever(30000, full_state=True, since=sync_token)