Update app.py

app.py

@@ -208,6 +208,40 @@ def save_feedback(entry: Dict[str, Any]):
     data.append(entry)
     FEEDBACK_FILE.write_text(json.dumps(data, indent=2, ensure_ascii=False), encoding="utf-8")
 
+# ---------- Local matching fallback ----------
+def calculate_local_matches(current: Profile, candidates: List[Profile], top_k: int = 3) -> List[Dict[str, Any]]:
+    matches = []
+    for cand in candidates:
+        if cand.id == current.id:
+            continue
+            
+        score = 0
+        offers_set = set(o.lower() for o in cand.offers)
+        wants_set = set(w.lower() for w in cand.wants)
+        cur_offers = set(o.lower() for o in current.offers)
+        cur_wants = set(w.lower() for w in current.wants)
+
+        score += len(offers_set & cur_wants) * 30
+        score += len(cur_offers & wants_set) * 30
+        
+        if cand.availability and current.availability:
+            if cand.availability.lower() in current.availability.lower() or current.availability.lower() in cand.availability.lower():
+                score += 20
+                
+        if cand.preferences and current.preferences:
+            if cand.preferences.lower() in current.preferences.lower() or current.preferences.lower() in cand.preferences.lower():
+                score += 20
+
+        matches.append({
+            "id": cand.id,
+            "username": cand.username,
+            "score": min(100, score),
+            "reason": "Local scoring",
+            "avatar": cand.avatar
+        })
+
+    return sorted(matches, key=lambda x: x["score"], reverse=True)[:top_k]
+
 # ---------- Streamlit UI ----------
 st.set_page_config(page_title="AI Skill Swap", page_icon="🤝", layout="wide")
 
@@ -338,51 +372,77 @@ with col2:
         if st.button("Run AI matchmaking"):
             with st.spinner("Finding matches..."):
                 current = store.find_by_username(pick)
+                
+                if not current:
+                    st.error("Selected profile not found!")
+                    st.stop()
 
-                sys_ins, user_msg = make_prompt_for_matching(current, profiles, top_k)
                 try:
+                    # Try AI matching first
+                    sys_ins, user_msg = make_prompt_for_matching(current, profiles, top_k)
                     matches = ask_groq_for_matches(sys_ins, user_msg)
-                except:
-                    # fallback local scoring
-                    matches = []
-                    for cand in profiles:
-                        if cand.id == current.id:
-                            continue
-                        score = 0
-                        offers_set = set(o.lower() for o in cand.offers)
-                        wants_set = set(w.lower() for w in cand.wants)
-                        cur_offers = set(o.lower() for o in current.offers)
-                        cur_wants = set(w.lower() for w in current.wants)
-
-                        score += len(offers_set & cur_wants) * 30
-                        score += len(cur_offers & wants_set) * 30
-                        if cand.availability and current.availability and cand.availability.lower() in current.availability.lower():
-                            score += 20
-                        if cand.preferences and current.preferences and cand.preferences.lower() in current.preferences.lower():
-                            score += 20
-
-                        matches.append({
-                            "id": cand.id,
-                            "username": cand.username,
-                            "score": min(100, score),
-                            "reason": "Local scoring",
-                            "avatar": cand.avatar
-                        })
-
-                    matches = sorted(matches, key=lambda x: x["score"], reverse=True)[:top_k]
+                    
+                    # Ensure matches have score field
+                    for match in matches:
+                        if "score" not in match:
+                            match["score"] = 0
+                except Exception as e:
+                    st.warning(f"AI matching failed: {e}. Using local matching...")
+                    # Fallback to local matching
+                    candidates = [p for p in profiles if p.id != current.id]
+                    matches = calculate_local_matches(current, candidates, top_k)
 
                 st.markdown("<div class='card'><b>Top Matches</b></div>", unsafe_allow_html=True)
 
                 for m in matches:
+                    if not isinstance(m, dict):
+                        continue
+                        
                     c1, c2 = st.columns([1, 5])
                     with c1:
-                        if m.get("avatar") and Path(m["avatar"]).exists():
-                            st.image(m["avatar"], width=72)
+                        avatar = m.get("avatar")
+                        if avatar and Path(avatar).exists():
+                            st.image(avatar, width=72)
                         else:
                             st.image("https://via.placeholder.com/72?text=User", width=72)
 
                     with c2:
-                        st.markdown(f"### {m['username']}")
-                        st.markdown(f"**Match score:** <span class='match-score'>{m['score']}%</span>", unsafe_allow_html=True)
-                        st.progress(int(m["score"]) / 100)
-                        st.markdown("*🎉 Yass! We found a buddy to swap your skills with!*")
+                        st.markdown(f"### {m.get('username', 'Unknown User')}")
+                        
+                        # Safely get score with default value
+                        score = m.get("score", 0)
+                        if not isinstance(score, (int, float)):
+                            try:
+                                score = float(score)
+                            except:
+                                score = 0
+                        score = max(0, min(100, score))
+                        
+                        st.markdown(f"**Match score:** <span class='match-score'>{score}%</span>", unsafe_allow_html=True)
+                        st.progress(score / 100)
+                        
+                        reason = m.get("reason", "AI matchmaking")
+                        st.caption(f"*{reason}*")
+
+# Feedback button (floating)
+st.markdown("""
+<div class='feedback-floating' onclick="document.querySelector('.feedback-form').style.display='block'">💬</div>
+""", unsafe_allow_html=True)
+
+# Hidden feedback form
+with st.sidebar:
+    st.markdown("---")
+    with st.expander("💬 Feedback"):
+        feedback = st.text_area("Your feedback")
+        if st.button("Submit Feedback"):
+            if feedback.strip():
+                save_feedback({
+                    "timestamp": time.time(),
+                    "feedback": feedback.strip(),
+                    "username": st.session_state.get("username", "anonymous")
+                })
+                st.success("Thank you for your feedback!")
+                time.sleep(1)
+                st.rerun()
+            else:
+                st.warning("Please enter some feedback.")