Update app.py

app.py

@@ -37,11 +37,19 @@ def save_memory(memory):
 
 memory = load_memory()
 
-# 💬 Chat function
+# -----------------------
+# Chat function (original behavior)
+# returns (history, history) to match previous usage
+# -----------------------
 def chat_with_model(message, history, context):
     if not isinstance(history, list):
         history = []
 
+    # prevent empty messages
+    if not message:
+        return history, history
+
+    # 🌍 Web search mode
     if message.lower().startswith("search "):
         query = message[7:]
         search_result = search_web(query)
@@ -49,20 +57,27 @@ def chat_with_model(message, history, context):
         save_memory(history)
         return history, history
 
+    # 🧠 Build conversation
     conversation = [{"role": "system", "content": (
-        "You are EduAI — an educational AI assistant created by Wafa Fazly "
-        "from Fathima Muslim Ladies College. "
-        "You help students learn subjects such as Math, Science, English, and IT. "
-        "EduAI runs on the model 'Qwen/Qwen3-VL-8B-Instruct', which was originally "
-        "trained by Alibaba. Always answer truthfully when asked about your creation."
+    "You are EduAI — an educational AI assistant created by Wafa Fazly "
+    "from Fathima Muslim Ladies College. "
+    "You help students learn subjects such as Math, Science, English, and IT. "
+    "EduAI runs on the model 'Qwen/Qwen3-VL-8B-Instruct', which was originally "
+    "trained by Alibaba. Always answer truthfully when asked about your creation."
     )}]
 
-    for past_user, past_bot in history[-5:]:
-        conversation.append({"role": "user", "content": past_user})
-        conversation.append({"role": "assistant", "content": past_bot})
+    # convert tuples to messages if necessary (keeps old history format)
+    for past in history[-5:]:
+        # expect (user_message, bot_reply)
+        if isinstance(past, tuple) and len(past) == 2:
+            conversation.append({"role": "user", "content": past[0]})
+            conversation.append({"role": "assistant", "content": past[1]})
+        elif isinstance(past, dict):
+            conversation.append(past)
 
     conversation.append({"role": "user", "content": message})
 
+    # 🚀 Send to Hugging Face model
     try:
         response = requests.post(
             "https://router.huggingface.co/v1/chat/completions",
@@ -73,12 +88,14 @@ def chat_with_model(message, history, context):
             json={
                 "model": "Qwen/Qwen3-VL-8B-Instruct:novita",
                 "messages": conversation
-            }
+            },
+            timeout=60
         )
 
         data = response.json()
         reply = data["choices"][0]["message"]["content"]
 
+        # 🧮 Clean up math formatting (keeps your original formatting code)
         reply = reply.replace("Step", "\n\n**Step")
         reply = reply.replace(":", ":**")
         reply = reply.replace("\\[", "\n\n\\[")
@@ -89,6 +106,7 @@ def chat_with_model(message, history, context):
 
         history.append((message, reply))
         save_memory(history)
+        # IMPORTANT: return a pair (history, history) because other code expects two outputs
         return history, history
 
     except Exception as e:
@@ -108,7 +126,51 @@ def clear_memory():
         os.remove(MEMORY_FILE)
     return [], "🧹 Chat memory cleared! Start fresh."
 
-# 🎨 Gradio Interface (UI Improved)
+# -----------------------
+# Pause / Send wrappers (FIXED)
+# -----------------------
+
+# send handler that respects paused state (ALWAYS returns (chat_history, textbox_clear))
+def send_handler(message, history, context, paused_state):
+    if paused_state:
+        # Do not call the model when paused — append a friendly hint
+        if not isinstance(history, list):
+            history = []
+        history.append((None, "⏸️ Chat is paused. Click Resume to continue."))
+        return history, ""  # update chatbot and clear textbox
+    # Not paused: call original chat handler and adapt its returns to (chat_history, textbox_clear)
+    hist_pair = chat_with_model(message, history, context)  # returns (history, history)
+    # unpack safely
+    if isinstance(hist_pair, tuple) and len(hist_pair) == 2:
+        hist = hist_pair[0]
+    else:
+        hist = hist_pair
+    return hist, ""
+
+
+# toggle pause/resume and update UI (state + chat + button text + send button disabled)
+def toggle_pause(paused_state, history):
+    new_state = not bool(paused_state)
+    if not isinstance(history, list):
+        history = []
+
+    if new_state:
+        # now paused
+        history.append((None, "⏸️ Chat paused. Send is disabled."))
+        pause_btn_update = gr.Button.update(value="▶ Resume")
+        send_btn_update = gr.Button.update(disabled=True)
+    else:
+        # resumed
+        history.append((None, "▶️ Chat resumed. You can send messages now."))
+        pause_btn_update = gr.Button.update(value="⏸ Pause")
+        send_btn_update = gr.Button.update(disabled=False)
+
+    # return new pause state, updated chat history, and two UI updates (pause button & send button)
+    return new_state, history, pause_btn_update, send_btn_update
+
+# -----------------------
+# Build UI (unchanged layout; pause added)
+# -----------------------
 with gr.Blocks(theme=gr.themes.Soft(primary_hue="violet")) as iface:
     gr.Markdown(
         """
@@ -165,7 +227,6 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="violet")) as iface:
                     {"left": "\\[", "right": "\\]", "display": True}
                 ]
             )
-
             msg = gr.Textbox(
                 label="💭 Type your question here...",
                 placeholder="Ask EduAI anything about your studies..."
@@ -174,12 +235,20 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="violet")) as iface:
             with gr.Row():
                 send = gr.Button("✨ Send Message")
                 pause = gr.Button("⏸ Pause", variant="secondary")
+                # state to keep track of pause (False = running, True = paused)
+                pause_state = gr.State(False)
 
     # 🪄 Event handlers
     subj.change(update_context, inputs=subj, outputs=context_display)
     planner.change(update_context, inputs=planner, outputs=context_display)
     lang.change(update_context, inputs=lang, outputs=context_display)
-    send.click(chat_with_model, inputs=[msg, chatbot, context_display], outputs=[chatbot, chatbot])
+
+    # send now uses send_handler and respects pause_state; outputs: chatbot and clears textbox
+    send.click(send_handler, inputs=[msg, chatbot, context_display, pause_state], outputs=[chatbot, msg])
+
     clear_btn.click(clear_memory, outputs=[chatbot, context_display])
 
+    # pause toggles pause_state, updates chatbot with a message, updates pause button label and disables/enables send
+    pause.click(toggle_pause, inputs=[pause_state, chatbot], outputs=[pause_state, chatbot, pause, send])
+
 iface.launch()