Fix is with Youtube summary

Author: MrDesjardins

routes/queue.py

@@ -125,8 +125,14 @@ def play_next_in_queue():
         # Add type-specific fields
         if next_item.type == "summary":
             response["week_year"] = next_item.week_year
+            logger.info(
+                f"Playing next item (summary): {next_item.title} - week_year: {next_item.week_year}"
+            )
         else:
             response["youtube_id"] = next_item.youtube_id
+            logger.info(
+                f"Playing next item (youtube): {next_item.title} - video_id: {next_item.youtube_id}"
+            )
 
         return JSONResponse(response)
     except Exception as e:

routes/stream.py

@@ -110,6 +110,14 @@ def stream_video(request: StreamRequest):
 
     logger.info(f"🎬 /stream requested for video: {video_id}")
 
+    # Validate video ID
+    if not video_id or video_id.strip() == "":
+        logger.error("Received empty video_id in /stream request")
+        raise HTTPException(
+            status_code=400,
+            detail="Invalid request: video_id is required and cannot be empty",
+        )
+
     # Fetch video metadata and save to database
     try:
         metadata = get_video_metadata(video_id)

services/database.py

@@ -370,7 +370,16 @@ def get_queue() -> List[QueueItem]:
         """)
 
         rows = cursor.fetchall()
-        return [QueueItem.from_db_row(row) for row in rows]
+        items = [QueueItem.from_db_row(row) for row in rows]
+
+        # Log queue items for debugging
+        for item in items:
+            logger.debug(
+                f"Queue item #{item.id}: type={item.type}, "
+                f"week_year={item.week_year}, title={item.title}"
+            )
+
+        return items
 
 
 def get_next_in_queue() -> Optional[QueueItem]:
@@ -630,7 +639,8 @@ def add_summary_to_queue(week_year: str) -> int:
 
         record_id = cursor.lastrowid
         logger.info(
-            f"Added summary to queue (position {next_position}): {summary.title}"
+            f"Added summary to queue (position {next_position}): {summary.title} "
+            f"[type=summary, week_year={week_year}]"
         )
         return record_id
 

services/models.py

@@ -62,6 +62,10 @@ class QueueItem:
     @classmethod
     def from_db_row(cls, row) -> "QueueItem":
         """Create instance from database row."""
+        # Get type from row, defaulting to 'youtube' only if None
+        # This preserves explicit 'summary' types while handling backward compatibility
+        item_type = row["type"] if row["type"] is not None else "youtube"
+
         return cls(
             id=row["id"],
             youtube_id=row["youtube_id"],
@@ -70,7 +74,7 @@ def from_db_row(cls, row) -> "QueueItem":
             thumbnail_url=row["thumbnail_url"],
             position=row["position"],
             created_at=row["created_at"],
-            type=row["type"] or "youtube",
+            type=item_type,
             week_year=row["week_year"],
         )
 
@@ -86,7 +90,9 @@ def to_dict(self) -> dict:
             "created_at": self.created_at,
             "type": self.type,
         }
-        if self.week_year:
+        # Always include week_year for summary items (even if None)
+        # This ensures the frontend can properly identify summary types
+        if self.type == "summary":
             result["week_year"] = self.week_year
         return result