Update app.py

app.py

@@ -15,7 +15,9 @@ from matching import calculate_local_matches
 from feedback import save_feedback
 from ui_styles import BASE_CSS
 
-# ---------- Streamlit Config ----------
+# ----------------------------------------------
+# Streamlit Config
+# ----------------------------------------------
 st.set_page_config(
     page_title="AI Skill Swap",
     page_icon="🤝",
@@ -26,21 +28,24 @@ st.markdown(f"<style>{BASE_CSS}</style>", unsafe_allow_html=True)
 
 store = ProfileStore()
 
-# ---------- Header ----------
+# ----------------------------------------------
+# Header Section
+# ----------------------------------------------
 col_logo, col_title = st.columns([1, 6])
 with col_logo:
     try:
-        st.image("logo.jpg", width=90)  # Use uploaded logo
+        # Use uploaded logo if available
+        st.image("logo.jpg", width=90)
     except:
-        st.image(
-            "https://via.placeholder.com/100x100/4F46E5/FFFFFF?text=🤝",
-            width=90,
-        )
+        st.image("https://via.placeholder.com/100x100/4F46E5/FFFFFF?text=🤝", width=90)
+
 with col_title:
     st.title("AI Skill Swap")
     st.caption("Teach what you know • Learn what you love • Match intelligently")
 
-# ---------- Sidebar: Profile Management ----------
+# ----------------------------------------------
+# Sidebar: Profile Management
+# ----------------------------------------------
 with st.sidebar:
     st.header("👤 Profile Manager")
 
@@ -50,10 +55,11 @@ with st.sidebar:
     selected_user = st.selectbox(
         "Select profile",
         ["— Create New —"] + usernames,
-        key="selected_user",
+        key="selected_user"
     )
 
     st.markdown("### 📝 Profile Details")
+
     username = st.text_input(
         "Username",
         value="" if selected_user == "— Create New —" else selected_user,
@@ -96,23 +102,23 @@ with st.sidebar:
         key="avatar_uploader"
     )
 
-    # ---------- Avatar Preview ----------
+    # ---------- Avatar Preview & Size Check
     if avatar_file:
-        st.image(avatar_file, width=150, caption="Preview of avatar")
-        if avatar_file.size > 800 * 1024:  # 800KB max
-            st.error("File exceeds maximum size of 800KB!")
+        st.image(avatar_file, width=150, caption="Preview Avatar")
+        if avatar_file.size > 800 * 1024:  # 800KB limit
+            st.error("File exceeds 800KB!")
             avatar_file = None
 
     st.markdown("---")
     col_create, col_update = st.columns(2)
 
-    # ---------- CREATE PROFILE ----------
+    # ---------- Create Button
     with col_create:
-        if st.button("➕ Create", use_container_width=True, key="create_button"):
+        if st.button("➕ Create", key="create_button"):
             if not username.strip():
                 st.error("Username is required!")
             elif store.find_by_username(username):
-                st.error("Profile already exists. Use Update.")
+                st.error("Already exists — use Update!")
             else:
                 avatar_path = None
                 if avatar_file:
@@ -137,22 +143,24 @@ with st.sidebar:
                 ok, msg = store.add_or_update(profile)
                 if ok:
                     st.success(msg)
-                    # Clear fields
-                    for key in ["offers_text", "wants_text", "availability", "preferences", "username_input"]:
+                    # Clear fields on create
+                    for key in ["offers_text", "wants_text", "availability", "preferences"]:
                         if key in st.session_state:
                             del st.session_state[key]
                     if "avatar_uploader" in st.session_state:
                         del st.session_state["avatar_uploader"]
+                    # Clear previous matches
+                    st.session_state["matches"] = []
                     st.experimental_rerun()
                 else:
                     st.error(msg)
 
-    # ---------- UPDATE PROFILE ----------
+    # ---------- Update Button
     with col_update:
-        if st.button("💾 Update", use_container_width=True, key="update_button"):
+        if st.button("💾 Update", key="update_button"):
             existing = store.find_by_username(username)
             if not existing:
-                st.error("Profile does not exist. Use Create.")
+                st.error("Profile does not exist.")
             else:
                 existing.offers = normalize_skill_list(offers_text)
                 existing.wants = normalize_skill_list(wants_text)
@@ -175,34 +183,33 @@ with st.sidebar:
                     del st.session_state["avatar_uploader"]
                 st.experimental_rerun()
 
-    # ---------- DELETE PROFILE ----------
+    # ---------- Delete Button
     if selected_user != "— Create New —":
-        if st.button("🗑️ Delete Profile", type="secondary", key="delete_button"):
-            profiles = store.load_all()
-            profile_to_delete = next((p for p in profiles if p.username == selected_user), None)
-            if profile_to_delete:
-                profiles.remove(profile_to_delete)
-                store.save_all(profiles)
-                st.warning(f"Profile '{selected_user}' deleted")
-                for key in ["offers_text", "wants_text", "availability", "preferences", "username_input"]:
+        if st.button("🗑️ Delete Profile", key="delete_button"):
+            ok, msg = store.delete(selected_user)
+            if ok:
+                st.warning(msg)
+                # Clear sidebar fields
+                for key in ["username_input", "offers_input", "wants_input", "availability_input", "preferences_input"]:
                     if key in st.session_state:
                         del st.session_state[key]
                 if "avatar_uploader" in st.session_state:
                     del st.session_state["avatar_uploader"]
                 st.experimental_rerun()
             else:
-                st.error("Profile not found")
+                st.error(msg)
 
-# ---------- MAIN CONTENT ----------
+# ----------------------------------------------
+# Main Content - Community Profiles
+# ----------------------------------------------
 left, right = st.columns([2, 3])
 
-# ---------- COMMUNITY PROFILES ----------
 with left:
     st.subheader("🌍 Community Profiles")
     profiles = store.load_all()
 
     if not profiles:
-        st.info("No profiles yet. Create one from the sidebar.")
+        st.info("No profiles yet. Create one above.")
     else:
         for p in profiles:
             st.markdown("<div class='card'>", unsafe_allow_html=True)
@@ -212,10 +219,7 @@ with left:
                 if p.avatar and Path(p.avatar).exists():
                     st.image(p.avatar, width=120)
                 else:
-                    st.image(
-                        "https://via.placeholder.com/120",
-                        width=120,
-                    )
+                    st.image("https://via.placeholder.com/120", width=120)
 
             with cols[1]:
                 st.markdown(f"**{p.username}**")
@@ -224,7 +228,9 @@ with left:
                 st.markdown(f"🎯 **Wants:** {', '.join(p.wants) or '—'}")
             st.markdown("</div>", unsafe_allow_html=True)
 
-# ---------- AI MATCHMAKING ----------
+# ----------------------------------------------
+# AI Matchmaking (Profile Cards)
+# ----------------------------------------------
 with right:
     st.subheader("🤖 AI Matchmaking")
 
@@ -232,75 +238,73 @@ with right:
         st.info("Add profiles to enable matchmaking.")
     else:
         pick = st.selectbox(
-            "Choose your profile",
+            "Match for profile",
             [p.username for p in profiles],
             key="matchmaking_select"
         )
 
         if st.button("✨ Find Best Matches", key="find_matches_button"):
-            with st.spinner("Analyzing profiles..."):
-                current = store.find_by_username(pick)
-                matches = []
-
-                try:
-                    sys_msg, user_msg = make_prompt_for_matching(current, profiles, 3)
-                    raw_matches = ask_groq_for_matches(sys_msg, user_msg)
-                    # Enrich Groq matches with full profile info
-                    for rm in raw_matches:
-                        profile = store.find_by_username(rm.get("username", ""))
-                        if profile:
-                            matches.append({
-                                "username": profile.username,
-                                "offers": profile.offers,
-                                "wants": profile.wants,
-                                "avatar": profile.avatar,
-                                "score": int(float(rm.get("score", 0)) * 100),
-                                "reason": rm.get("reason", "AI Match")
-                            })
-                except Exception:
-                    matches = calculate_local_matches(
-                        current,
-                        [p for p in profiles if p.id != current.id],
-                        3,
-                    )
-                    # Convert score 0-1 to 0-100%
-                    for m in matches:
-                        m["score"] = int(m["score"] * 100)
-
-                # Display matches
-                for m in matches:
-                    st.markdown("<div class='card'>", unsafe_allow_html=True)
-                    cols = st.columns([1, 4])
-                    with cols[0]:
-                        if m.get("avatar") and Path(m["avatar"]).exists():
-                            st.image(m["avatar"], width=100)
-                        else:
-                            st.image(
-                                "https://via.placeholder.com/100",
-                                width=100
-                            )
-                    with cols[1]:
-                        st.markdown(f"### {m.get('username')}")
-                        st.markdown(f"🧠 **Offers:** {', '.join(m.get('offers', [])) or '—'}")
-                        st.markdown(f"🎯 **Wants:** {', '.join(m.get('wants', [])) or '—'}")
-                        st.markdown(f"**Match Score:** {m.get('score', 0)}%")
-                        st.markdown(f"Match Reason: {m.get('reason', 'AI Match')}")
-                    st.markdown("</div>", unsafe_allow_html=True)
-
-# ---------- FEEDBACK ----------
+            current = store.find_by_username(pick)
+
+            try:
+                sys_msg, user_msg = make_prompt_for_matching(current, profiles, 3)
+                raw_matches = ask_groq_for_matches(sys_msg, user_msg)
+            except Exception:
+                raw_matches = calculate_local_matches(
+                    current,
+                    [p for p in profiles if p.id != current.id],
+                    3,
+                )
+
+            # Enrich returned matches
+            enriched = []
+            for rm in raw_matches:
+                prof = store.find_by_username(rm.get("username", ""))
+                if prof:
+                    enriched.append({
+                        "username": prof.username,
+                        "offers": prof.offers,
+                        "wants": prof.wants,
+                        "avatar": prof.avatar,
+                        "score": int(float(rm.get("score", 0)) * 100),
+                        "reason": rm.get("reason", "AI Match")
+                    })
+            st.session_state["matches"] = enriched
+
+        matches_list = st.session_state.get("matches", [])
+        for m in matches_list:
+            st.markdown("<div class='card'>", unsafe_allow_html=True)
+            cols = st.columns([1, 4])
+
+            with cols[0]:
+                if m.get("avatar") and Path(m["avatar"]).exists():
+                    st.image(m["avatar"], width=120)
+                else:
+                    st.image("https://via.placeholder.com/120", width=120)
+
+            with cols[1]:
+                st.markdown(f"### {m.get('username')}")
+                st.markdown(f"🧠 **Offers:** {', '.join(m.get('offers', [])) or '—'}")
+                st.markdown(f"🎯 **Wants:** {', '.join(m.get('wants', [])) or '—'}")
+                st.markdown(f"**Match Score:** {m.get('score', 0)}%")
+                st.markdown(f"**Reason:** {m.get('reason', '')}")
+
+            st.markdown("</div>", unsafe_allow_html=True)
+
+# ----------------------------------------------
+# Feedback
+# ----------------------------------------------
 with st.sidebar:
     st.markdown("---")
     with st.expander("💬 Feedback"):
         fb = st.text_area("Your feedback", key="feedback_text")
         if st.button("Submit Feedback", key="submit_feedback_button"):
             if fb.strip():
-                save_feedback(
-                    {
-                        "timestamp": time.time(),
-                        "feedback": fb,
-                        "username": username if username.strip() else "anonymous",
-                    }
-                )
+                save_feedback({
+                    "timestamp": time.time(),
+                    "feedback": fb,
+                    "username": username if username.strip() else "anonymous",
+                })
                 st.success("Thank you!")
                 if "feedback_text" in st.session_state:
                     del st.session_state["feedback_text"]