stop emoji fade

app.py

@@ -88,6 +88,12 @@ custom_css = """
     min-width: 120px;
 }
 
+/* Prevent fades/transitions and enforce full opacity for emoji displays */
+.emoji-box, .emoji-box * {
+    transition: none !important;
+    opacity: 1 !important;
+}
+
 .chat-container {
     border-radius: 15px;
     overflow: hidden;
@@ -121,7 +127,7 @@ with gr.Blocks(title="Emoji AI Avatar") as demo:
             with gr.Row(elem_classes="emoji-container"):
                 with gr.Column(elem_classes="emoji-box"):
                     user_emoji_display = gr.HTML(
-                        get_emoji_html("😐", "You: waiting..."),
+                        get_emoji_html("😐", "You"),
                         label="Your Emoji"
                     )
                 with gr.Column(elem_classes="emoji-box"):
@@ -266,7 +272,7 @@ with gr.Blocks(title="Emoji AI Avatar") as demo:
     def update_user_emoji_live(text: str):
         """Update user emoji in real-time as they type - EVERY KEYSTROKE"""
         if not text or not text.strip():
-            return get_emoji_html("😐", "You: waiting...")
+            return get_emoji_html("😐", "You")
         
         # Analyze sentiment on every keystroke for live updates
         # The analyzer now focuses on the LAST SENTENCE for accuracy
@@ -331,12 +337,16 @@ with gr.Blocks(title="Emoji AI Avatar") as demo:
         """
         Stream chat response with MCP as grounding context.
         MCP provides context/skills that enhance Gemini's responses.
+        
+        EMOJI DISPLAY: Keeps previous emoji stable until new emotion is detected.
+        No 'thinking' or intermediate states shown.
         """
         if not message.strip():
+            # No message — don't overwrite existing emoji displays
             yield (
                 history,
-                get_emoji_html("😐", "You: neutral"),
-                get_emoji_html("😐", "AI: neutral"),
+                None,
+                None,
             )
             return
         
@@ -348,31 +358,22 @@ with gr.Blocks(title="Emoji AI Avatar") as demo:
         # Create new history with user message
         new_history = list(history) + [{"role": "user", "content": message}]
         
-        # Initial state - show MCP context gathering if enabled
-        if use_mcp and mcp_client.connected:
-            current_ai_emoji = "🔌"
-            current_ai_label = f"AI + MCP ({mcp_client.space_name}): gathering context..."
-        else:
-            current_ai_emoji = "🤔"
-            current_ai_label = "AI: thinking..."
+        # STABLE EMOJI: Keep previous AI emoji unchanged until a new emotion is detected
+        current_ai_emoji = None
+        current_ai_label = None
+        last_yielded_ai_html = None
         
+        # don't overwrite the AI emoji at stream start; leave it unchanged until we detect a real emotion
         yield (
             new_history,
             user_emoji_html,
-            get_emoji_html(current_ai_emoji, current_ai_label),
+            None,
         )
         
-        # Get MCP context if enabled
+        # Get MCP context if enabled (no emoji change during context gathering)
         mcp_context = ""
         if use_mcp and mcp_client.connected:
             mcp_context = mcp_client.get_context_for_llm(message)
-            if mcp_context:
-                # Update status to show we got context
-                yield (
-                    new_history,
-                    user_emoji_html,
-                    get_emoji_html("🧠", f"AI + MCP: processing with context..."),
-                )
         
         # Build the enhanced message with MCP context
         if mcp_context:
@@ -404,19 +405,25 @@ Please answer the user's question, incorporating the MCP context where relevant.
             if elapsed < STREAM_DELAY:
                 time.sleep(STREAM_DELAY - elapsed)
             last_yield_time = time.time()
-            
-            # Update AI emoji every 2 chunks
+
+            # Update AI emoji every 2 chunks - ONLY when emotion changes
             if chunk_count % 2 == 0:
-                partial_sentiment = sentiment_analyzer.analyze(full_response)
-                detected_emotion = partial_sentiment["label"]
-                if detected_emotion != last_emotion and detected_emotion:
-                    last_emotion = detected_emotion
-                    current_ai_emoji = emoji_mapper.get_emoji(detected_emotion)
-                    # Show MCP indicator if using MCP
-                    if mcp_context:
-                        current_ai_label = f"AI+MCP: {detected_emotion}"
-                    else:
-                        current_ai_label = f"AI: {detected_emotion}"
+                    partial_sentiment = sentiment_analyzer.analyze(full_response)
+                    detected_emotion = partial_sentiment["label"]
+                    # Only update emoji if emotion actually changed (not empty)
+                    if detected_emotion and detected_emotion != "neutral" and detected_emotion != last_emotion:
+                        last_emotion = detected_emotion
+                        current_ai_emoji = emoji_mapper.get_emoji(detected_emotion)
+                        # Show MCP indicator if using MCP
+                        if mcp_context:
+                            current_ai_label = f"AI+MCP: {detected_emotion}"
+                        else:
+                            current_ai_label = f"AI: {detected_emotion}"
+                    elif detected_emotion == "neutral" and last_emotion == "":
+                        # First detection is neutral - update label but keep neutral emoji
+                        last_emotion = "neutral"
+                        current_ai_emoji = "😐"
+                        current_ai_label = "AI: neutral"
             
             # Add MCP indicator to response if context was used
             display_response = full_response
@@ -429,11 +436,21 @@ Please answer the user's question, incorporating the MCP context where relevant.
                 {"role": "user", "content": message},
                 {"role": "assistant", "content": display_response}
             ]
-            
+
+            # Only update ai_emoji_display when AI emoji actually changed
+            ai_html_to_yield = None
+            if current_ai_emoji is not None:
+                ai_html_to_yield = get_emoji_html(current_ai_emoji, current_ai_label)
+                # If identical to last yielded HTML, avoid updating to prevent visual flicker
+                if ai_html_to_yield == last_yielded_ai_html:
+                    ai_html_to_yield = None
+                else:
+                    last_yielded_ai_html = ai_html_to_yield
+
             yield (
                 display_history,
                 user_emoji_html,
-                get_emoji_html(current_ai_emoji, current_ai_label),
+                ai_html_to_yield,
             )
         
         # Final sentiment analysis
@@ -502,10 +519,6 @@ Please answer the user's question, incorporating the MCP context where relevant.
         lambda: "",
         None,
         msg,
-    ).then(
-        lambda: get_emoji_html("😐", "You: waiting..."),
-        None,
-        user_emoji_display,
     )
     
     submit_btn.click(
@@ -516,10 +529,6 @@ Please answer the user's question, incorporating the MCP context where relevant.
         lambda: "",
         None,
         msg,
-    ).then(
-        lambda: get_emoji_html("😐", "You: waiting..."),
-        None,
-        user_emoji_display,
     )
     
     # Clear button
@@ -527,7 +536,7 @@ Please answer the user's question, incorporating the MCP context where relevant.
     
     def clear_chat():
         gemini.reset_chat()  # Reset Gemini chat history too
-        return [], get_emoji_html("😐", "You: waiting..."), get_emoji_html("😐", "AI: neutral")
+        return [], get_emoji_html("😐", "You"), get_emoji_html("😐", "AI: neutral")
     
     clear_btn.click(
         clear_chat,
@@ -592,13 +601,23 @@ if __name__ == "__main__":
     port = _choose_port(preferred=preferred_port, start=7861, end=7870)
     
     if port:
-        print(f"🚀 Starting Gradio on 0.0.0.0:{port}")
+        print(f"🚀 Attempting to start Gradio on 0.0.0.0:{port}")
     else:
         print(f"🚀 Starting Gradio with auto-assigned port")
 
-    demo.launch(
-        server_name="0.0.0.0",
-        server_port=port,
-        share=False,
-        css=custom_css,
-    )
+    # Try launching on the chosen port, but gracefully fallback to auto-assignment
+    # if Gradio raises OSError (port collision/race condition).
+    try:
+        demo.launch(
+            server_name="0.0.0.0",
+            server_port=port,
+            share=False,
+            css=custom_css,
+        )
+    except OSError as e:
+        print(f"⚠️  Failed to bind to port {port}: {e}. Falling back to auto-assigned port.")
+        demo.launch(
+            server_name="0.0.0.0",
+            share=False,
+            css=custom_css,
+        )