Fixed expander nesting issue in HF Expert analysis section

app.py

@@ -48,12 +48,6 @@ if "tts_enabled" not in st.session_state:
 if "voice_input" not in st.session_state:
     st.session_state.voice_input = ""
 
-# DEBUG: Validate session messages (fixed syntax)
-for idx, msg in enumerate(st.session_state.messages):
-    if not isinstance(msg, dict) or "role" not in msg or "content" not in msg:
-        st.error(f"🚨 Malformed message at index {idx}: {msg}")
-        st.stop()
-
 # Sidebar layout redesign
 with st.sidebar:
     st.title("🧠 AI Life Coach")
@@ -146,22 +140,19 @@ with st.sidebar:
         st.divider()
         
         st.subheader("🤖 HF Expert Analysis")
-        col1, col2 = st.columns([3, 1])
-        with col1:
-            st.markdown("""
-                **HF Expert Features:**
-                - Analyzes entire conversation history
-                - Performs web research when needed
-                - Provides deep insights and recommendations
-                - Acts as expert consultant in your conversation
-            """)
-        with col2:
-            if st.button("🧠 Activate HF Expert",
-                         key="activate_hf_expert_sidebar",
-                         help="Send conversation to HF endpoint for deep analysis",
-                         use_container_width=True,
-                         disabled=st.session_state.is_processing):
-                st.session_state.hf_expert_requested = True
+        st.markdown("""
+            **HF Expert Features:**
+            - Analyzes entire conversation history
+            - Performs web research when needed
+            - Provides deep insights and recommendations
+            - Acts as expert consultant in your conversation
+        """)
+        if st.button("🧠 Activate HF Expert",
+                     key="activate_hf_expert_sidebar",
+                     help="Send conversation to HF endpoint for deep analysis",
+                     use_container_width=True,
+                     disabled=st.session_state.is_processing):
+            st.session_state.hf_expert_requested = True
 
 # Main interface
 st.title("🧠 AI Life Coach")
@@ -212,58 +203,58 @@ if st.session_state.messages and len(st.session_state.messages) > 0:
                          use_container_width=True,
                          disabled=st.session_state.is_processing):
                 st.session_state.hf_expert_requested = True
-        
-        # Show HF expert analysis when requested
-        if st.session_state.get("hf_expert_requested", False):
-            with st.spinner("🧠 HF Expert analyzing conversation..."):
-                try:
-                    # Get conversation history
-                    user_session = session_manager.get_session("default_user")
-                    conversation_history = user_session.get("conversation", [])
-                    
-                    # Show what HF expert sees
-                    with st.expander("📋 HF Expert Input", expanded=False):
-                        st.markdown("**Conversation History Sent to HF Expert:**")
-                        for i, msg in enumerate(conversation_history[-10:]):  # Last 10 messages
-                            st.markdown(f"**{msg['role'].capitalize()}:** {msg['content'][:100]}{'...' if len(msg['content']) > 100 else ''}")
-                    
-                    # Request HF analysis
-                    hf_analysis = coordinator.manual_hf_analysis(
-                        "default_user",
-                        conversation_history
-                    )
-                    
-                    if hf_analysis:
-                        # Display HF expert response with clear indication
-                        with st.chat_message("assistant"):
-                            st.markdown("### 🤖 HF Expert Analysis")
-                            st.markdown(hf_analysis)
-                        
-                        # Add research/web search decisions
-                        research_needs = coordinator.determine_web_search_needs(conversation_history)
-                        if research_needs["needs_search"]:
-                            st.info(f"🔍 **Research Needed:** {research_needs['reasoning']}")
-                            if st.button("🔎 Perform Web Research", key="web_research_button"):
-                                # Perform web search
-                                with st.spinner("🔎 Searching for current information..."):
-                                    # Add web search logic here
-                                    st.success("✅ Web research completed!")
-                        
-                        # Add to message history with HF expert tag
-                        st.session_state.messages.append({
-                            "role": "assistant",
-                            "content": hf_analysis,
-                            "timestamp": datetime.now().strftime("%H:%M:%S"),
-                            "source": "hf_expert",
-                            "research_needs": research_needs
-                        })
-                        
-                        st.session_state.hf_expert_requested = False
+
+# Show HF expert analysis when requested (outside of the expander)
+if st.session_state.get("hf_expert_requested", False):
+    with st.spinner("🧠 HF Expert analyzing conversation..."):
+        try:
+            # Get conversation history
+            user_session = session_manager.get_session("default_user")
+            conversation_history = user_session.get("conversation", [])
+            
+            # Show what HF expert sees in a separate expander
+            with st.expander("📋 HF Expert Input", expanded=False):
+                st.markdown("**Conversation History Sent to HF Expert:**")
+                for i, msg in enumerate(conversation_history[-10:]):  # Last 10 messages
+                    st.markdown(f"**{msg['role'].capitalize()}:** {msg['content'][:100]}{'...' if len(msg['content']) > 100 else ''}")
+            
+            # Request HF analysis
+            hf_analysis = coordinator.manual_hf_analysis(
+                "default_user",
+                conversation_history
+            )
+            
+            if hf_analysis:
+                # Display HF expert response with clear indication
+                with st.chat_message("assistant"):
+                    st.markdown("### 🤖 HF Expert Analysis")
+                    st.markdown(hf_analysis)
                 
-                except Exception as e:
-                    user_msg = translate_error(e)
-                    st.error(f"❌ HF Expert analysis failed: {user_msg}")
-                    st.session_state.hf_expert_requested = False
+                # Add research/web search decisions
+                research_needs = coordinator.determine_web_search_needs(conversation_history)
+                if research_needs["needs_search"]:
+                    st.info(f"🔍 **Research Needed:** {research_needs['reasoning']}")
+                    if st.button("🔎 Perform Web Research", key="web_research_button"):
+                        # Perform web search
+                        with st.spinner("🔎 Searching for current information..."):
+                            # Add web search logic here
+                            st.success("✅ Web research completed!")
+                
+                # Add to message history with HF expert tag
+                st.session_state.messages.append({
+                    "role": "assistant",
+                    "content": hf_analysis,
+                    "timestamp": datetime.now().strftime("%H:%M:%S"),
+                    "source": "hf_expert",
+                    "research_needs": research_needs
+                })
+                
+                st.session_state.hf_expert_requested = False
+        
+        except Exception as e:
+            user_msg = translate_error(e)
+            st.error(f"❌ HF Expert analysis failed: {user_msg}")
+            st.session_state.hf_expert_requested = False
 
 # Chat input - FIXED VERSION (moved outside of tabs)
 user_input = st.chat_input("Type your message here...", disabled=st.session_state.is_processing)