Update app.py

app.py

@@ -44,9 +44,15 @@ else:
         unsafe_allow_html=True
     )
 
-# Initialize authentication state
+# Initialize authentication and state
 if "authenticated" not in st.session_state:
     st.session_state.authenticated = False
+if "messages" not in st.session_state:
+    st.session_state["messages"] = []
+if "last_user_input" not in st.session_state:
+    st.session_state.last_user_input = ""
+if "last_assistant_response" not in st.session_state:
+    st.session_state.last_assistant_response = ""
 
 # Login screen
 if not st.session_state.authenticated:
@@ -63,14 +69,11 @@ if not st.session_state.authenticated:
         else:
             st.error("Incorrect username or password. Please try again.")
 
-# Main chat UI using streamlit-chat
+# Main chat UI
 else:
-    if "messages" not in st.session_state:
-        st.session_state["messages"] = []
-
     st.divider()
 
-    # Display chat history with avatars
+    # Display chat history
     for i, msg in enumerate(st.session_state["messages"]):
         if msg["role"] == "assistant":
             st.markdown(f'<img src="https://www.carfind.co.za/images/Carfind-Icon.svg" width="30" style="vertical-align: middle;"> <strong>Carfind Assistant:</strong> {msg["content"]}', unsafe_allow_html=True)
@@ -91,59 +94,63 @@ else:
     with col_clear:
         if st.button("🗑️", help="Clear chat"):
             st.session_state.messages = []
+            st.session_state.last_user_input = ""
+            st.session_state.last_assistant_response = ""
             st.rerun()
 
-    # Handle OpenAI assistant (with loop protection)
-    if user_input and openai_key:
-        if len(st.session_state.messages) == 0 or user_input != st.session_state.messages[-1]["content"]:
-            st.session_state.messages.append({"role": "user", "content": user_input})
-
-            with st.spinner("Thinking and typing... 💬"):
-                client = OpenAI(api_key=openai_key)
-                ASSISTANT_ID = "asst_5gQR21fOsmHil11FGBzEArA7"
-
-                def save_transcript(messages):
-                    with open("chat_logs.txt", "a") as log:
-                        log.write(f"\n--- New Chat ({datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S')}) ---\n")
-                        for msg in messages:
-                            log.write(f"{msg['role'].capitalize()}: {msg['content']}\n")
-                        log.write("--- End Chat ---\n")
-
-                try:
-                    thread = client.beta.threads.create()
-                    thread_id = thread.id
-
-                    client.beta.threads.messages.create(
+    # Only trigger on new distinct input
+    if user_input and user_input != st.session_state.last_user_input and openai_key:
+        st.session_state.last_user_input = user_input
+        st.session_state.messages.append({"role": "user", "content": user_input})
+
+        with st.spinner("Thinking and typing... 💬"):
+            client = OpenAI(api_key=openai_key)
+            ASSISTANT_ID = "asst_5gQR21fOsmHil11FGBzEArA7"
+
+            def save_transcript(messages):
+                with open("chat_logs.txt", "a") as log:
+                    log.write(f"\n--- New Chat ({datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S')}) ---\n")
+                    for msg in messages:
+                        log.write(f"{msg['role'].capitalize()}: {msg['content']}\n")
+                    log.write("--- End Chat ---\n")
+
+            try:
+                thread = client.beta.threads.create()
+                thread_id = thread.id
+
+                client.beta.threads.messages.create(
+                    thread_id=thread_id,
+                    role="user",
+                    content=user_input
+                )
+
+                run = client.beta.threads.runs.create(
+                    thread_id=thread_id,
+                    assistant_id=ASSISTANT_ID
+                )
+
+                while True:
+                    run_status = client.beta.threads.runs.retrieve(
                         thread_id=thread_id,
-                        role="user",
-                        content=user_input
+                        run_id=run.id
                     )
+                    if run_status.status == "completed":
+                        break
+                    time.sleep(1)
 
-                    run = client.beta.threads.runs.create(
-                        thread_id=thread_id,
-                        assistant_id=ASSISTANT_ID
-                    )
-
-                    while True:
-                        run_status = client.beta.threads.runs.retrieve(
-                            thread_id=thread_id,
-                            run_id=run.id
-                        )
-                        if run_status.status == "completed":
-                            break
-                        time.sleep(1)
-
-                    messages_response = client.beta.threads.messages.list(thread_id=thread_id)
-                    assistant_reply = messages_response.data[0].content[0].text.value
+                messages_response = client.beta.threads.messages.list(thread_id=thread_id)
+                assistant_reply = messages_response.data[0].content[0].text.value.strip()
 
-                    if assistant_reply.strip().lower() != user_input.strip().lower():
-                        st.session_state.messages.append({"role": "assistant", "content": assistant_reply})
-                        save_transcript(st.session_state.messages)
+                if (assistant_reply.lower() != user_input.lower() and
+                    assistant_reply != st.session_state.last_assistant_response):
+                    st.session_state.messages.append({"role": "assistant", "content": assistant_reply})
+                    st.session_state.last_assistant_response = assistant_reply
+                    save_transcript(st.session_state.messages)
 
-                    st.rerun()
+                st.rerun()
 
-                except Exception as e:
-                    st.error(f"An error occurred: {str(e)}")
+            except Exception as e:
+                st.error(f"An error occurred: {str(e)}")
 
     elif not openai_key:
         st.error("OpenAI key not found. Please ensure it is set as a Hugging Face secret.")