Update app.py

app.py

@@ -65,26 +65,24 @@ else:
 # ------------------ Tab 1: Chat Assistant ------------------
 with tab1:
     with st.sidebar:
-        st.header("🔧 Tools")
-        if st.button("🧹 Clear Chat"):
-            for key in ["messages", "thread_id", "image_url", "image_updated", "pending_prompt"]:
-                st.session_state[key] = [] if key == "messages" else None if "url" in key else False
-            st.rerun()
+        st.header("🧪 Pathology Tools")
 
-        if st.button("🚪 Logout"):
-            st.session_state.clear()
+        if st.button("🧹 Clear Chat"):
+            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)
+
         keyword = st.text_input("Keyword Search", placeholder="e.g. mitosis, carcinoma")
         if st.button("🔎 Search") and keyword:
             st.session_state.pending_prompt = f"Find clauses or references related to: {keyword}"
 
         section = st.text_input("Section Lookup", placeholder="e.g. Connective Tissue")
-        if st.button("📑 Lookup") and section:
+        if section:
             st.session_state.pending_prompt = f"Summarize or list key points from section: {section}"
 
-        action = st.selectbox("📂 Common Pathology Queries", [
+        action = st.selectbox("Common Pathology Queries", [
             "Select an action...",
             "List histological features of inflammation",
             "Summarize features of carcinoma",
@@ -92,19 +90,19 @@ with tab1:
             "Extract diagnostic markers",
             "Summarize embryology stages"
         ])
-        if action and action != "Select an action...":
+        if action != "Select an action...":
             st.session_state.pending_prompt = action
-            st.rerun()
 
     chat_col, image_col = st.columns([2, 1])
+
     with chat_col:
         st.markdown("### 💬 Ask a Pathology-Specific Question")
         user_input = st.chat_input("Example: What are features of squamous cell carcinoma?")
-        if user_input or st.session_state.pending_prompt:
-            st.session_state.messages.append({
-                "role": "user",
-                "content": user_input or st.session_state.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
 
         if st.session_state.messages and st.session_state.messages[-1]["role"] == "user":
@@ -125,44 +123,68 @@ with tab1:
 
                 with st.spinner("🔬 Analyzing..."):
                     while True:
-                        status = client.beta.threads.runs.retrieve(thread_id=st.session_state.thread_id, run_id=run.id)
+                        status = client.beta.threads.runs.retrieve(
+                            thread_id=st.session_state.thread_id, run_id=run.id
+                        )
                         if status.status in ("completed", "failed", "cancelled"):
                             break
                         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()
-                            if not any(reply in msg["content"] or msg["content"] in reply for msg in st.session_state.messages if msg["role"] == "assistant"):
+                            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(r'https://raw\.githubusercontent\.com/AndrewLORTech/witspathologai/main/[^\s\n"]+\.png', reply)
-                                if image_matches:
-                                    st.session_state.image_url = image_matches[0]
-                                    st.session_state.image_updated = True
+                                # Extract image URLs from response
+                                images = re.findall(
+                                    r'https://raw\.githubusercontent\.com/AndrewLORTech/witspathologai/main/[^\s\n"]+\.png',
+                                    reply
+                                )
+                                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 messages
         for msg in st.session_state.messages:
             with st.chat_message(msg["role"]):
                 st.markdown(msg["content"], unsafe_allow_html=True)
 
-    with image_col:
-        if show_image and st.session_state.image_url:
-            try:
-                r = requests.get(st.session_state.image_url)
-                r.raise_for_status()
-                img = Image.open(BytesIO(r.content))
-                st.image(img, caption="📄 Referenced Image", use_container_width=True)
-            except Exception as e:
-                st.error(f"🖼️ Failed to load image: {e}")
+        # Display follow-up questions
+        if st.session_state.messages and st.session_state.messages[-1]["role"] == "assistant":
+            last = st.session_state.messages[-1]["content"]
+            if "Some Possible Questions:" in last:
+                # Filter only lines that are actual questions
+                all_lines = re.findall(r"[-•]\s*(.*)", last)
+                questions = [line for line in all_lines if line.strip().endswith("?")]
+
+                if questions:
+                    st.markdown("#### 💡 Follow-Up Suggestions")
+                    for q in questions:
+                        if st.button(f"📌 {q}"):
+                            st.session_state.pending_prompt = q
+                            st.rerun()
+                else:
+                    st.markdown("#### 💡 No follow-up questions detected in the assistant's response.")
 
+    with image_col:
+        if show_image and st.session_state.image_urls:
+            st.markdown("### 🖼️ Images")
+            for url in st.session_state.image_urls:
+                try:
+                    img = Image.open(BytesIO(requests.get(url, timeout=5).content))
+                    st.image(img, caption=url.split("/")[-1], use_container_width=True)
+                except Exception:
+                    st.warning(f"⚠️ Failed to load image: {url}")
 
 # ------------------ Tab 2: Visual Reference Search ------------------
 import urllib.parse