Update app.py

app.py

@@ -40,13 +40,10 @@ CAMPAIGN_CONFIG_FILE = "campaign_rules.json"
 # 1. CAMPAIGN MANAGER & LOGIC
 # =====================================================
 def load_campaign_rules():
-    """Loads rules from JSON. Auto-migrates old list-based format to new include/exclude format."""
     if os.path.exists(CAMPAIGN_CONFIG_FILE):
         try:
             with open(CAMPAIGN_CONFIG_FILE, "r") as f:
                 rules = json.load(f)
-                
-                # Backward compatibility: Convert flat lists into include/exclude dicts
                 for campaign, config in rules.items():
                     if isinstance(config, list):
                         rules[campaign] = {
@@ -74,39 +71,32 @@ def get_campaign_match(text, rules):
     if not text or not isinstance(text, str):
         return None
         
-    # Convert URL delimiters to spaces and pad the string for EXACT word matching
-    # Example: "/blog/ai-readiness" -> " blog ai readiness "
-    search_text = " " + re.sub(r'[\-_/.,?=&+#]', ' ', text.lower()) + " "
+    # Clean text: replace all URL symbols with spaces and pad edges
+    clean_text = " " + re.sub(r'[\-_/.,?=&+#]', ' ', text.lower()) + " "
     
     for campaign_name, config in rules.items():
         includes = config.get("include", [])
         excludes = config.get("exclude", [])
         
-        # 1st: Check Exclusions (Exact Word Match)
+        # 1st: Check Exclusions (Exact Match)
         has_exclusion = False
         for ex in excludes:
             if not ex.strip(): continue
-            # Format keyword same way (e.g. "asp.net" -> "asp net")
-            kw = re.sub(r'[\-_/.,?=&+#]', ' ', ex.lower().strip())
-            if f" {kw} " in search_text:
+            kw = " " + re.sub(r'[\-_/.,?=&+#]', ' ', ex.lower().strip()) + " "
+            if kw in clean_text:
                 has_exclusion = True
                 break
                 
         if has_exclusion:
             continue
             
-        # 2nd: Check Inclusions (Exact Word Match)
-        has_inclusion = False
+        # 2nd: Check Inclusions (Exact Match)
         for inc in includes:
             if not inc.strip(): continue
-            kw = re.sub(r'[\-_/.,?=&+#]', ' ', inc.lower().strip())
-            if f" {kw} " in search_text:
-                has_inclusion = True
-                break
+            kw = " " + re.sub(r'[\-_/.,?=&+#]', ' ', inc.lower().strip()) + " "
+            if kw in clean_text:
+                return campaign_name
                 
-        if has_inclusion:
-            return campaign_name
-            
     return None
 
 def apply_business_logic(df):
@@ -125,7 +115,7 @@ def apply_business_logic(df):
             match = get_campaign_match(row["landing_page_path"], rules)
             if match: return match
             
-        # Priority 2: Exit Page (Only checked if Landing Page DID NOT match)
+        # Priority 2: Exit Page
         if row.get("exit_page_path"):
             match = get_campaign_match(row["exit_page_path"], rules)
             if match: return match
@@ -348,10 +338,12 @@ def build_dashboard(df, preset, top_n):
     if df is None or df.empty: return px.bar(title="No Data")
     if "Campaign" not in df.columns: df = apply_business_logic(df)
     
-    # RESTORED EXACT ORIGINAL CHART LOGIC
     if preset == "Key Campaigns Bifurcation (MS Tech, Fintech, Adtech)":
-        target = ["MS Tech", "Fintech", "Adtech"]
-        filtered = df[df["Campaign"].isin(target)].copy()
+        target_lower = ["ms tech", "fintech", "adtech"]
+        
+        # Case Insensitive Matching (Fix for empty chart)
+        filtered = df[df["Campaign"].astype(str).str.lower().isin(target_lower)].copy()
+        
         if filtered.empty: return px.bar(title="No Data for Key Campaigns")
                 
         agg = filtered.groupby("Campaign").agg(
@@ -530,16 +522,13 @@ with gr.Blocks(title="Leadfeeder Campaign Pro") as demo:
                 if not target_camp:
                     return "❌ Error: Campaign name cannot be empty.", gr.update()
                     
-                # 1. Update the dictionary format
                 rules[target_camp] = {"include": clean_inc, "exclude": clean_exc}
                 
-                # 2. Save file locally first
                 with open(CAMPAIGN_CONFIG_FILE, "w") as f:
                     json.dump(rules, f, indent=4)
                     
                 status_message = f"✅ Saved locally! Updated keywords for '{target_camp}'."
                 
-                # 3. Push to Hugging Face automatically
                 if HF_TOKEN and SPACE_ID:
                     try:
                         api = HfApi(token=HF_TOKEN)