- Deploy script and migration optimization to avoid too much clutter.

Author: MrDesjardins

deploy.sh

@@ -0,0 +1,21 @@
+#!/bin/bash
+
+# Deploy script - push to GitHub and update remote server
+# Usage: ./deploy.sh [commit message]
+
+set -e
+
+REMOTE_HOST="10.0.0.181"
+REMOTE_USER="pdesjardins"
+REMOTE_DIR="/home/pdesjardins/code/audio-stream-server"
+
+# Push to GitHub
+echo "Pushing to GitHub..."
+git push origin main
+
+# SSH into remote and run update script
+echo "Deploying to $REMOTE_HOST..."
+ssh "$REMOTE_USER@$REMOTE_HOST" "cd $REMOTE_DIR && ./update.sh"
+
+echo ""
+echo "Deploy complete!"

migrate_add_audio_duration.py

@@ -20,13 +20,6 @@ def migrate_add_audio_duration():
         print(f"❌ Database not found at {db_path}. Nothing to migrate.")
         return
 
-    # Create backup
-    backup_path = db_path.with_suffix(
-        f"{db_path.suffix}.backup_{datetime.now().strftime('%Y%m%d_%H%M%S')}"
-    )
-    shutil.copy2(db_path, backup_path)
-    print(f"✅ Created backup: {backup_path}")
-
     conn = sqlite3.connect(db_path)
     cursor = conn.cursor()
 
@@ -41,6 +34,13 @@ def migrate_add_audio_duration():
             )
             return
 
+        # Create backup only when migration is actually needed
+        backup_path = db_path.with_suffix(
+            f"{db_path.suffix}.backup_{datetime.now().strftime('%Y%m%d_%H%M%S')}"
+        )
+        shutil.copy2(db_path, backup_path)
+        print(f"✅ Created backup: {backup_path}")
+
         # Add the new column (nullable, since existing records won't have this data)
         print("Adding 'audio_duration_seconds' column...")
         cursor.execute(

migrate_add_metadata.py

@@ -39,15 +39,27 @@ def column_exists(cursor, table_name, column_name):
 
 def migrate():
     """Add channel and thumbnail_url columns to tables."""
-    backup_path = backup_database()
-
     if not os.path.exists(DB_PATH):
         logger.info("No database to migrate. New schema will be created on first run.")
         return
 
     conn = sqlite3.connect(DB_PATH)
     cursor = conn.cursor()
 
+    # Check if any columns are missing before creating a backup
+    needs_migration = (
+        not column_exists(cursor, "play_history", "channel")
+        or not column_exists(cursor, "play_history", "thumbnail_url")
+        or not column_exists(cursor, "queue", "channel")
+        or not column_exists(cursor, "queue", "thumbnail_url")
+    )
+    if not needs_migration:
+        logger.info("All columns already exist. No migration needed.")
+        conn.close()
+        return
+
+    backup_path = backup_database()
+
     try:
         # Migrate play_history table
         if not column_exists(cursor, "play_history", "channel"):

migrate_add_weekly_summary.py

@@ -56,17 +56,17 @@ def migrate_database():
         print(f"✗ Database not found at {DB_PATH}")
         return False
 
-    # Create backup
-    backup_path = backup_database()
-
     try:
         conn = sqlite3.connect(DB_PATH)
 
-        # Check if migration needed
+        # Check if migration needed before creating a backup
         if not check_if_migration_needed(conn):
             conn.close()
             return True
 
+        # Create backup only when migration is actually needed
+        backup_path = backup_database()
+
         cursor = conn.cursor()
 
         # Create weekly_summaries table

update.sh

@@ -74,7 +74,21 @@ uv run python migrate_add_weekly_summary.py
 uv run python -c "from services.database import init_database; init_database(); print('Database schema updated successfully')"
 
 echo ""
-echo "Step 6: Restarting service..."
+echo "Step 6: Cleaning up old database backups..."
+echo "----------------------------------------"
+KEEP_BACKUPS=5
+BACKUPS=$(ls -t audio_history.db.backup_* 2>/dev/null)
+BACKUP_COUNT=$(echo "$BACKUPS" | grep -c . 2>/dev/null || echo 0)
+if [ "$BACKUP_COUNT" -gt "$KEEP_BACKUPS" ]; then
+    TO_DELETE=$((BACKUP_COUNT - KEEP_BACKUPS))
+    echo "$BACKUPS" | tail -n +"$((KEEP_BACKUPS + 1))" | xargs rm -f
+    echo "✓ Removed $TO_DELETE old backup(s), kept $KEEP_BACKUPS most recent"
+else
+    echo "✓ $BACKUP_COUNT backup(s) found, no cleanup needed"
+fi
+
+echo ""
+echo "Step 7: Restarting service..."
 echo "----------------------------------------"
 if [ "$SERVICE_RUNNING" = true ]; then
     echo "Restarting audio-stream service..."