updated app.py

app.py

@@ -742,10 +742,37 @@ def send_voice_message_v6(message, chat_history):
 def generate_podcast_ui(doc_ids, style, duration, voice1, voice2):
     """UI wrapper for podcast generation"""
     try:
+        # Add detailed logging
+        logger.info(f"generate_podcast_ui called with:")
+        logger.info(f"  doc_ids: {doc_ids} (type: {type(doc_ids)})")
+        logger.info(f"  style: {style}")
+        logger.info(f"  duration: {duration}")
+        logger.info(f"  voice1: {voice1}")
+        logger.info(f"  voice2: {voice2}")
+        
+        # Handle various input formats
+        if doc_ids is None:
+            logger.warning("doc_ids is None")
+            return ("⚠️ Please select at least one document", None, "No documents selected", "")
+        
+        # Convert to list if needed
+        if isinstance(doc_ids, str):
+            logger.info(f"Converting string doc_id to list: {doc_ids}")
+            doc_ids = [doc_ids]
+        
+        # Check if empty
         if not doc_ids or len(doc_ids) == 0:
+            logger.warning(f"doc_ids is empty or has length 0: {doc_ids}")
+            return ("⚠️ Please select at least one document", None, "No documents selected", "")
+        
+        # Filter out None or empty string values
+        doc_ids = [doc_id for doc_id in doc_ids if doc_id and doc_id.strip()]
+        
+        if not doc_ids:
+            logger.warning("After filtering, no valid doc_ids remain")
             return ("⚠️ Please select at least one document", None, "No documents selected", "")
         
-        logger.info(f"Generating podcast: {len(doc_ids)} docs, {style}, {duration}min")
+        logger.info(f"Generating podcast with {len(doc_ids)} valid documents: {doc_ids}")
         
         result = mcp_server.run_async(
             mcp_server.generate_podcast_async(
@@ -757,12 +784,16 @@ def generate_podcast_ui(doc_ids, style, duration, voice1, voice2):
             )
         )
         
+        logger.info(f"Podcast generation result: success={result.get('success')}")
+        
         if result.get("success"):
             audio_file = result.get("audio_file")
             transcript = result.get("transcript", "Transcript not available")
             message = result.get("message", "Podcast generated!")
             formatted_transcript = f"## Podcast Transcript\n\n{transcript}"
             
+            logger.info(f"Podcast generated successfully: {audio_file}")
+            
             return (
                 f"✅ {message}",
                 audio_file,
@@ -771,9 +802,10 @@ def generate_podcast_ui(doc_ids, style, duration, voice1, voice2):
             )
         else:
             error = result.get("error", "Unknown error")
+            logger.error(f"Podcast generation failed: {error}")
             return (f"❌ Error: {error}", None, "Generation failed", "")
     except Exception as e:
-        logger.error(f"Podcast UI error: {str(e)}")
+        logger.error(f"Podcast UI error: {str(e)}", exc_info=True)
         return (f"❌ Error: {str(e)}", None, "An error occurred", "")
 
 def load_dashboard_stats():
@@ -1165,13 +1197,17 @@ def create_gradio_interface():
                         with gr.Group():
                             gr.Markdown("#### 📚 Select Content")
                             
+                            # IMPORTANT: CheckboxGroup that stores document IDs
                             podcast_doc_selector = gr.CheckboxGroup(
-                                choices=get_document_choices(),
+                                choices=get_document_choices(),  # List of (label, value) tuples
                                 label="Documents to Include",
                                 info="Choose 1-5 documents for best results",
-                                interactive=True
-                                
+                                interactive=True,
+                                value=[]  # Start with empty selection
                             )
+                            
+                            # Debug display (optional - remove in production)
+                            gr.Markdown("*Selected document IDs will be used for podcast generation*")
                         
                         with gr.Accordion("🎨 Podcast Settings", open=True):
                             with gr.Row():
@@ -1240,11 +1276,11 @@ def create_gradio_interface():
                                 value="*Transcript will appear after generation...*"
                             )
                 
-                # Event handlers
+                # Event handler - Connect the button to the function
                 generate_podcast_btn.click(
                     fn=generate_podcast_ui,
                     inputs=[
-                        doc_dropdown_content,
+                        podcast_doc_selector,
                         podcast_style,
                         podcast_duration,
                         host1_voice_selector,