Update app.py

app.py

@@ -1,84 +1,79 @@
-import gradio as gr
-import os
-import requests
-
-# Load GROQ API key from Hugging Face Secrets
-GROQ_API_KEY = os.environ.get("GROQ_API_KEY")
-GROQ_API_URL = "https://api.groq.com/openai/v1/chat/completions"
-MODEL_NAME = "llama3-8b-8192"
-
-# System prompt defines chatbot personality
-SYSTEM_PROMPT = """
-You are an expert Programming Tutor.
-Explain programming concepts in a simple, beginner-friendly way.
-Provide examples when helpful and keep answers concise.
-"""
-
-# Build messages safely
-def build_messages(message, chat_history):
-    messages = [{"role": "system", "content": SYSTEM_PROMPT}]
-    if chat_history is not None:
-        for item in chat_history:
-            if isinstance(item, tuple) and len(item) == 2:
-                user_msg = str(item[0])
-                bot_msg = str(item[1])
-                messages.append({"role": "user", "content": user_msg})
-                messages.append({"role": "assistant", "content": bot_msg})
-    messages.append({"role": "user", "content": str(message)})
-    return messages
-
-# Respond function for Gradio
-def respond(message, chat_history, temperature):
-    # Ensure chat_history is list of tuples
-    safe_history = []
-    if chat_history is not None:
-        for item in chat_history:
-            if isinstance(item, tuple) and len(item) == 2:
-                safe_history.append((str(item[0]), str(item[1])))
-
-    messages = build_messages(message, safe_history)
-
-    headers = {
-        "Authorization": f"Bearer {GROQ_API_KEY}",
-        "Content-Type": "application/json"
-    }
-
-    response = requests.post(
-        GROQ_API_URL,
-        headers=headers,
-        json={
-            "model": MODEL_NAME,
-            "messages": messages,
-            "temperature": temperature
-        }
-    )
-
-    if response.status_code == 200:
-        bot_reply = response.json()["choices"][0]["message"]["content"]
-    else:
-        bot_reply = f"Error {response.status_code}: {response.text}"
-
-    safe_history.append((str(message), str(bot_reply)))
-    return "", safe_history
-
-# Gradio UI
-with gr.Blocks() as demo:
-    gr.Markdown("## 💻 Programming Tutor Chatbot (Powered by GROQ)")
-
-    chatbot = gr.Chatbot()
-    state = gr.State([])
-
-    msg = gr.Textbox(label="Ask a programming question")
-    temperature = gr.Slider(
-        minimum=0.1,
-        maximum=1.0,
-        value=0.7,
-        step=0.1,
-        label="Response Creativity"
-    )
-    clear = gr.Button("Clear Chat")
-
-    msg.submit(respond, [msg, state, temperature], [msg, chatbot])
-    clear.click(lambda: ([], []), None, [chatbot, state])
-
+import gradio as gr
+import os
+import requests
+
+# Load API key from Hugging Face Secrets
+GROQ_API_KEY = os.environ.get("GROQ_API_KEY")
+GROQ_API_URL = "https://api.groq.com/openai/v1/chat/completions"
+MODEL_NAME = "llama3-8b-8192"
+
+SYSTEM_PROMPT = """
+You are an expert Programming Tutor.
+Explain programming concepts in a simple, beginner-friendly way.
+Provide examples when helpful and keep answers concise.
+"""
+
+# Build messages in proper format
+def build_messages(user_input, chat_history):
+    messages = [{"role": "system", "content": SYSTEM_PROMPT}]
+    
+    # Add previous chat history safely
+    for item in chat_history or []:
+        if isinstance(item, tuple) and len(item) == 2:
+            user_msg = str(item[0])
+            bot_msg = str(item[1])
+            messages.append({"role": "user", "content": user_msg})
+            messages.append({"role": "assistant", "content": bot_msg})
+    
+    # Add latest user input
+    messages.append({"role": "user", "content": str(user_input)})
+    return messages
+
+# Call GROQ API
+def query_groq(user_input, chat_history, temperature):
+    messages = build_messages(user_input, chat_history)
+
+    headers = {
+        "Authorization": f"Bearer {GROQ_API_KEY}",
+        "Content-Type": "application/json"
+    }
+
+    payload = {
+        "model": MODEL_NAME,
+        "messages": messages,
+        "temperature": temperature
+    }
+
+    response = requests.post(GROQ_API_URL, headers=headers, json=payload)
+
+    if response.status_code == 200:
+        return response.json()["choices"][0]["message"]["content"]
+    else:
+        return f"Error {response.status_code}: {response.text}"
+
+# Gradio respond function
+def respond(user_input, chat_history, temperature):
+    safe_history = []
+    for item in chat_history or []:
+        if isinstance(item, tuple) and len(item) == 2:
+            safe_history.append((str(item[0]), str(item[1])))
+
+    bot_reply = query_groq(user_input, safe_history, temperature)
+    safe_history.append((str(user_input), str(bot_reply)))
+    return "", safe_history
+
+# Gradio UI
+with gr.Blocks() as demo:
+    gr.Markdown("## 💻 Programming Tutor Chatbot (Powered by GROQ)")
+
+    chatbot = gr.Chatbot()
+    state = gr.State([])
+
+    msg = gr.Textbox(label="Ask a programming question")
+    temperature = gr.Slider(0.1, 1.0, value=0.7, step=0.1, label="Response Creativity")
+    clear = gr.Button("Clear Chat")
+
+    msg.submit(respond, [msg, state, temperature], [msg, chatbot])
+    clear.click(lambda: ([], []), None, [chatbot, state])
+
 demo.launch()