Update app.py

app.py

@@ -63,13 +63,13 @@ tab1, tab2 = st.tabs(["💬 Chat Assistant", "🖼️ Visual Reference Search"])
 
 # ------------------ Tab 1: Chat Assistant ------------------
 with tab1:
+    # Sidebar
     with st.sidebar:
         st.header("🧪 Pathology Tools")
+        
         if st.button("🧹 Clear Chat"):
-            st.session_state.messages = []
-            st.session_state.thread_id = None
-            st.session_state.image_urls = []
-            st.session_state.pending_prompt = None
+            for k in ["messages", "thread_id", "image_urls", "pending_prompt"]:
+                st.session_state[k] = [] if k.endswith("s") else None
             st.rerun()
 
         show_image = st.toggle("📸 Show Images", value=True)
@@ -81,16 +81,15 @@ with tab1:
         if section:
             st.session_state.pending_prompt = f"Summarize or list key points from section: {section}"
 
-        actions = [
+        action = st.selectbox("Common Pathology Queries", [
             "Select an action...",
             "List histological features of inflammation",
             "Summarize features of carcinoma",
             "List muscle types and features",
             "Extract diagnostic markers",
             "Summarize embryology stages"
-        ]
-        action = st.selectbox("Common Pathology Queries", actions)
-        if action != actions[0]:
+        ])
+        if action != "Select an action...":
             st.session_state.pending_prompt = action
 
     chat_col, image_col = st.columns([2, 1])
@@ -98,17 +97,19 @@ with tab1:
     with chat_col:
         st.markdown("### 💬 Ask a Pathology-Specific Question")
         user_input = st.chat_input("Example: What are features of squamous cell carcinoma?")
+
+        # Append input or pending prompt
         if user_input:
             st.session_state.messages.append({"role": "user", "content": user_input})
         elif st.session_state.pending_prompt:
             st.session_state.messages.append({"role": "user", "content": st.session_state.pending_prompt})
             st.session_state.pending_prompt = None
 
+        # Trigger assistant run
         if st.session_state.messages and st.session_state.messages[-1]["role"] == "user":
             try:
-                if st.session_state.thread_id is None:
-                    thread = client.beta.threads.create()
-                    st.session_state.thread_id = thread.id
+                if not st.session_state.thread_id:
+                    st.session_state.thread_id = client.beta.threads.create().id
 
                 client.beta.threads.messages.create(
                     thread_id=st.session_state.thread_id,
@@ -129,39 +130,39 @@ with tab1:
                         time.sleep(1)
 
                 if status.status == "completed":
-                    messages = client.beta.threads.messages.list(thread_id=st.session_state.thread_id)
-                    for m in reversed(messages.data):
+                    responses = client.beta.threads.messages.list(thread_id=st.session_state.thread_id).data
+                    for m in reversed(responses):
                         if m.role == "assistant":
                             reply = m.content[0].text.value.strip()
-                            is_duplicate = any(
-                                reply in msg["content"] or msg["content"] in reply
-                                for msg in st.session_state.messages if msg["role"] == "assistant"
-                            )
-                            if not is_duplicate:
+
+                            # De-dupe reply
+                            if not any(reply in msg["content"] or msg["content"] in reply
+                                       for msg in st.session_state.messages if msg["role"] == "assistant"):
                                 st.session_state.messages.append({"role": "assistant", "content": reply})
-                                image_matches = re.findall(
+
+                                # Image detection
+                                images = re.findall(
                                     r'https://raw\.githubusercontent\.com/AndrewLORTech/witspathologai/main/[^\s\n"]+\.png',
                                     reply
                                 )
-                                st.session_state.image_urls = image_matches
+                                st.session_state.image_urls = images
                             break
                 else:
-                    st.error("❌ Assistant failed to respond.")
+                    st.error("❌ Assistant failed to complete.")
                 st.rerun()
             except Exception as e:
                 st.error(f"❌ Error: {e}")
 
+        # Display conversation
         for msg in st.session_state.messages:
             with st.chat_message(msg["role"]):
                 st.markdown(msg["content"], unsafe_allow_html=True)
 
-        # 🧠 Follow-up Suggestions
+        # Optional follow-ups
         if st.session_state.messages and st.session_state.messages[-1]["role"] == "assistant":
-            latest_reply = st.session_state.messages[-1]["content"]
-            match = re.search(r"Some Possible Questions:\s*(.*?)(?=\n#{2,}|\\Z)", latest_reply, re.DOTALL)
-            if match:
-                block = match.group(1)
-                questions = [line.strip(" -•") for line in block.splitlines() if line.strip().startswith(("-", "•"))]
+            last = st.session_state.messages[-1]["content"]
+            if "Some Possible Questions:" in last:
+                questions = re.findall(r"[-•]\s*(.*)", last)
                 if questions:
                     st.markdown("#### 💡 Follow-Up Suggestions")
                     for q in questions:
@@ -171,16 +172,13 @@ with tab1:
 
     with image_col:
         if show_image and st.session_state.image_urls:
-            st.markdown("### 🖼️ Image(s)")
-            for raw_url in st.session_state.image_urls:
+            st.markdown("### 🖼️ Images")
+            for url in st.session_state.image_urls:
                 try:
-                    r = requests.get(raw_url, timeout=5)
-                    r.raise_for_status()
-                    img = Image.open(BytesIO(r.content))
-                    st.image(img, caption=f"📷 {raw_url.split('/')[-1]}", use_container_width=True)
+                    img = Image.open(BytesIO(requests.get(url, timeout=5).content))
+                    st.image(img, caption=url.split("/")[-1], use_container_width=True)
                 except Exception:
-                    continue
-
+                    st.warning(f"⚠️ Failed to load image: {url}")
 
 # ------------------ Tab 2: Visual Reference Search ------------------
 with tab2: