fix (test app for location selector fixes)

app.py

@@ -216,38 +216,192 @@
 #     uvicorn.run(app, host="0.0.0.0", port=7860)
 
 """
-Alert Summary Generator Backend
-Modified FastAPI app with location-based alert generation
+Alert Summary Generator Backend - Standalone Version
+FastAPI app with embedded pipeline and location data
 """
 
 from fastapi import FastAPI, HTTPException
 from fastapi.middleware.cors import CORSMiddleware
 from pydantic import BaseModel
-from typing import Optional, Dict, Any
+from typing import Optional, Dict, Any, List
 import os
 from datetime import datetime
+from openai import OpenAI
 
-# Import existing pipeline components
-from src.pipeline import FarmerChatPipeline
-from src.client import get_openai_client
+# ============================================================================
+# OPENAI CLIENT SETUP
+# ============================================================================
 
-app = FastAPI(title="Farmer.chat Alert Summary API")
+def get_openai_client():
+    """Initialize OpenAI client with API key from environment"""
+    api_key = os.getenv("OPENAI_API_KEY")
+    if not api_key:
+        raise ValueError("OPENAI_API_KEY environment variable not set")
+    return OpenAI(api_key=api_key)
+
+
+# ============================================================================
+# PIPELINE COMPONENTS
+# ============================================================================
+
+class QueryRouter:
+    """Routes queries to appropriate MCP servers"""
+    
+    def __init__(self, client):
+        self.client = client
+    
+    def route_query(self, query: str, location: Dict[str, float]) -> Dict[str, Any]:
+        """Determine which servers to call based on query"""
+        prompt = f"""Given the farmer query: "{query}"
+Location: {location['latitude']}, {location['longitude']}
+
+Determine which data sources are needed. Return JSON:
+{{
+    "weather": true/false,
+    "soil": true/false, 
+    "water": true/false,
+    "elevation": true/false,
+    "pests": true/false
+}}"""
+        
+        response = self.client.chat.completions.create(
+            model="gpt-4",
+            messages=[{"role": "user", "content": prompt}],
+            temperature=0
+        )
+        
+        # Parse routing decision
+        import json
+        try:
+            routing = json.loads(response.choices[0].message.content)
+        except:
+            # Default: query all servers for alerts
+            routing = {
+                "weather": True,
+                "soil": True,
+                "water": True,
+                "elevation": True,
+                "pests": True
+            }
+        
+        return routing
+
+
+class MCPExecutor:
+    """Executes parallel calls to MCP servers"""
+    
+    def __init__(self, client):
+        self.client = client
+    
+    def execute_parallel(self, routing: Dict[str, bool], location: Dict[str, float]) -> Dict[str, str]:
+        """Execute MCP server calls in parallel"""
+        results = {}
+        
+        # Simulate MCP server calls (replace with actual server calls)
+        if routing.get("weather"):
+            results["weather"] = f"Weather data for {location['latitude']}, {location['longitude']}"
+        
+        if routing.get("soil"):
+            results["soil"] = f"Soil data for {location['latitude']}, {location['longitude']}"
+        
+        if routing.get("water"):
+            results["water"] = f"Water availability for {location['latitude']}, {location['longitude']}"
+        
+        if routing.get("elevation"):
+            results["elevation"] = f"Elevation data for {location['latitude']}, {location['longitude']}"
+        
+        if routing.get("pests"):
+            results["pests"] = f"Pest risk data for {location['latitude']}, {location['longitude']}"
+        
+        return results
+
+
+class ResponseCompiler:
+    """Compiles MCP results into coherent response"""
+    
+    def __init__(self, client):
+        self.client = client
+    
+    def compile_response(self, query: str, mcp_results: Dict[str, str], location: Dict[str, float]) -> str:
+        """Compile MCP results into final response"""
+        
+        # Format MCP results for context
+        context = "\n\n".join([f"{k.upper()}: {v}" for k, v in mcp_results.items()])
+        
+        prompt = f"""You are an agricultural assistant. Compile this data into a comprehensive alert summary.
+
+FARMER QUERY: {query}
+LOCATION: {location['latitude']}, {location['longitude']}
+
+DATA FROM SOURCES:
+{context}
+
+Provide a comprehensive agricultural alert covering:
+1. Current weather conditions and forecast
+2. Soil health and recommendations  
+3. Water availability status
+4. Elevation/topography considerations
+5. Pest risks and preventive measures
+
+Be specific, actionable, and farmer-friendly."""
+
+        response = self.client.chat.completions.create(
+            model="gpt-4",
+            messages=[{"role": "user", "content": prompt}],
+            temperature=0.7
+        )
+        
+        return response.choices[0].message.content
+
+
+class HindiTranslator:
+    """Translates responses to Hindi"""
+    
+    def __init__(self, client):
+        self.client = client
+    
+    def translate(self, text: str, target_lang: str = "en") -> str:
+        """Translate text if needed"""
+        if target_lang == "en":
+            return text
+        
+        # Add Hindi translation logic here
+        return text
+
+
+class FarmerChatPipeline:
+    """Main pipeline orchestrating all stages"""
+    
+    def __init__(self, client, location: Dict[str, float]):
+        self.client = client
+        self.location = location
+        self.router = QueryRouter(client)
+        self.executor = MCPExecutor(client)
+        self.compiler = ResponseCompiler(client)
+        self.translator = HindiTranslator(client)
+    
+    def process_query(self, query: str, language: str = "en") -> str:
+        """Process query through full pipeline"""
+        
+        # Stage 1: Route query
+        routing = self.router.route_query(query, self.location)
+        
+        # Stage 2: Execute MCP calls
+        mcp_results = self.executor.execute_parallel(routing, self.location)
+        
+        # Stage 3: Compile response
+        response = self.compiler.compile_response(query, mcp_results, self.location)
+        
+        # Stage 4: Translate if needed
+        final_response = self.translator.translate(response, language)
+        
+        return final_response
 
-# CORS middleware
-app.add_middleware(
-    CORSMiddleware,
-    allow_origins=["*"],
-    allow_credentials=True,
-    allow_methods=["*"],
-    allow_headers=["*"],
-)
 
-# Initialize OpenAI client and pipeline
-openai_client = get_openai_client()
-DEFAULT_LOCATION = {"latitude": 25.5941, "longitude": 85.1376}  # Patna default
-pipeline = FarmerChatPipeline(openai_client, DEFAULT_LOCATION)
+# ============================================================================
+# BIHAR LOCATION DATA
+# ============================================================================
 
-# Bihar location data
 BIHAR_DATA = {
     "Araria": ["Araria", "Forbesganj", "Jokihat", "Raniganj"],
     "Arwal": ["Arwal", "Kaler", "Karpi", "Kurtha"],
@@ -265,188 +419,223 @@ BIHAR_DATA = {
     "Jehanabad": ["Jehanabad", "Ghoshi", "Makhdumpur", "Modanganj"],
     "Kaimur": ["Bhabua", "Mohania", "Ramgarh", "Chainpur"],
     "Katihar": ["Katihar", "Barsoi", "Manihari", "Pranpur"],
-    "Khagaria": ["Khagaria", "Gogri", "Mansi", "Alauli"],
-    "Kishanganj": ["Kishanganj", "Bahadurganj", "Thakurganj", "Kochadhaman"],
-    "Lakhisarai": ["Lakhisarai", "Barahiya", "Halsi", "Surajgarha"],
-    "Madhepura": ["Madhepura", "Bihariganj", "Murliganj", "Udakishunganj"],
-    "Madhubani": ["Madhubani", "Jhanjharpur", "Benipatti", "Phulparas"],
-    "Munger": ["Munger", "Jamalpur", "Kharagpur", "Tarapur"],
-    "Muzaffarpur": ["Muzaffarpur", "Motipur", "Kanti", "Saraiya"],
-    "Nalanda": ["Bihar Sharif", "Rajgir", "Hilsa", "Islampur"],
-    "Nawada": ["Nawada", "Rajauli", "Hisua", "Warsaliganj"],
-    "Patna": ["Patna", "Danapur", "Masaurhi", "Phulwari", "Barh"],
-    "Purnia": ["Purnia", "Banmankhi", "Dhamdaha", "Baisi"],
-    "Rohtas": ["Sasaram", "Dehri", "Bikramganj", "Nokha"],
-    "Saharsa": ["Saharsa", "Simri Bakhtiyarpur", "Salkhua", "Sonbarsa"],
-    "Samastipur": ["Samastipur", "Rosera", "Dalsinghsarai", "Patori"],
-    "Saran": ["Chhapra", "Marhaura", "Dariapur", "Amnour"],
-    "Sheikhpura": ["Sheikhpura", "Barbigha", "Chewara", "Ghatkusumbha"],
-    "Sheohar": ["Sheohar", "Piprahi", "Tariyani", "Dumri Katsari"],
-    "Sitamarhi": ["Sitamarhi", "Bairgania", "Belsand", "Parihar"],
-    "Siwan": ["Siwan", "Maharajganj", "Goriakothi", "Barharia"],
-    "Supaul": ["Supaul", "Birpur", "Triveniganj", "Nirmali"],
-    "Vaishali": ["Hajipur", "Mahnar", "Lalganj", "Raghopur"],
-    "West Champaran": ["Bettiah", "Bagaha", "Narkatiaganj", "Ramnagar"],
+    "Khagaria": ["Khagaria", "Parbatta", "Alauli", "Beldaur"],
+    "Kishanganj": ["Kishanganj", "Bahadurganj", "Thakurganj", "Dighalbank"],
+    "Lakhisarai": ["Lakhisarai", "Halsi", "Suryagarha", "Pipariya"],
+    "Madhepura": ["Madhepura", "Udakishanganj", "Murliganj", "Alamnagar"],
+    "Madhubani": ["Madhubani", "Jhanjharpur", "Benipatti", "Jainagar"],
+    "Munger": ["Munger", "Jamalpur", "Asarganj", "Tarapur"],
+    "Muzaffarpur": ["Muzaffarpur", "Sitamarhi", "Minapur", "Bochaha"],
+    "Nalanda": ["Bihar Sharif", "Rajgir", "Hilsa", "Biharsharif"],
+    "Nawada": ["Nawada", "Rajauli", "Akbarpur", "Hisua"],
+    "Patna": ["Patna", "Danapur", "Fatuha", "Khagaul"],
+    "Purnia": ["Purnia", "Dhamdaha", "Kasba", "Banmankhi"],
+    "Rohtas": ["Sasaram", "Dehri", "Bikramganj", "Nasriganj"],
+    "Saharsa": ["Saharsa", "Sonbarsa", "Simri Bakhtiarpur", "Mahishi"],
+    "Samastipur": ["Samastipur", "Rosera", "Dalsinghsarai", "Pusa"],
+    "Saran": ["Chapra", "Marhaura", "Amnour", "Sonepur"],
+    "Sheikhpura": ["Sheikhpura", "Barbigha", "Ariari", "Shekhopur"],
+    "Sheohar": ["Sheohar", "Dumri Katsari", "Piprahi", "Tariyani"],
+    "Sitamarhi": ["Sitamarhi", "Pupri", "Belsand", "Bathnaha"],
+    "Siwan": ["Siwan", "Maharajganj", "Mairwa", "Darauli"],
+    "Supaul": ["Supaul", "Nirmali", "Triveniganj", "Chhatapur"],
+    "Vaishali": ["Hajipur", "Mahua", "Lalganj", "Desri"],
+    "West Champaran": ["Bettiah", "Bagaha", "Narkatiaganj", "Lauriya"]
 }
 
-# Location coordinates
 LOCATIONS = {
-    "alauli": {'latitude': 25.6392, 'longitude': 86.3854},
-    "amarpur": {'latitude': 25.0217, 'longitude': 86.8932},
-    "amnour": {'latitude': 25.9264, 'longitude': 84.9129},
-    "araria": {'latitude': 26.1346, 'longitude': 87.4654},
-    "arrah": {'latitude': 25.5603, 'longitude': 84.6632},
-    "arwal": {'latitude': 25.2369, 'longitude': 84.6738},
-    "aurangabad": {'latitude': 24.7863, 'longitude': 84.4145},
-    "bagaha": {'latitude': 27.059, 'longitude': 84.2065},
-    "bahadurganj": {'latitude': 26.2702, 'longitude': 87.8373},
-    "baheri": {'latitude': 26.0043, 'longitude': 86.0405},
-    "baikunthpur": {'latitude': 26.2817, 'longitude': 84.7428},
-    "bairgania": {'latitude': 26.7401, 'longitude': 85.2755},
-    "baisi": {'latitude': 25.8396, 'longitude': 87.7509},
-    "bakhri": {'latitude': 25.5743, 'longitude': 86.244},
-    "banka": {'latitude': 24.8669, 'longitude': 86.9126},
-    "banmankhi": {'latitude': 25.8976, 'longitude': 87.1659},
-    "barahat": {'latitude': 24.8869, 'longitude': 87.0296},
-    "barahiya": {'latitude': 25.2636, 'longitude': 86.0055},
-    "barauli": {'latitude': 26.3966, 'longitude': 84.6005},
-    "barauni": {'latitude': 25.4315, 'longitude': 86.0284},
-    "barbigha": {'latitude': 25.207, 'longitude': 85.7715},
-    "barh": {'latitude': 25.451, 'longitude': 85.7002},
-    "barharia": {'latitude': 26.3156, 'longitude': 84.456},
-    "barsoi": {'latitude': 25.6206, 'longitude': 87.9744},
-    "begusarai": {'latitude': 25.4139, 'longitude': 86.1349},
-    "belhar": {'latitude': 24.906, 'longitude': 86.6074},
-    "belsand": {'latitude': 26.4323, 'longitude': 85.3946},
-    "benipatti": {'latitude': 26.4435, 'longitude': 85.9105},
-    "benipur": {'latitude': 26.0754, 'longitude': 86.1245},
-    "bettiah": {'latitude': 26.8023, 'longitude': 84.5074},
-    "bhabua": {'latitude': 25.0763, 'longitude': 83.6336},
-    "bhagalpur": {'latitude': 25.2495, 'longitude': 86.9828},
-    "bihar sharif": {'latitude': 25.1939, 'longitude': 85.5209},
-    "bihariganj": {'latitude': 25.741, 'longitude': 86.9758},
-    "bikramganj": {'latitude': 25.2184, 'longitude': 84.2324},
-    "birpur": {'latitude': 26.5152, 'longitude': 87.0085},
-    "bodh gaya": {'latitude': 24.6776, 'longitude': 84.9727},
-    "buxar": {'latitude': 25.5612, 'longitude': 83.9813},
-    "chainpur": {'latitude': 24.9811, 'longitude': 83.4203},
-    "chakia": {'latitude': 26.4704, 'longitude': 84.9933},
-    "chausa": {'latitude': 25.4514, 'longitude': 83.8621},
-    "chewara": {'latitude': 25.054, 'longitude': 85.9335},
-    "chhapra": {'latitude': 25.7784, 'longitude': 84.7515},
-    "dalsinghsarai": {'latitude': 25.6727, 'longitude': 85.8124},
-    "danapur": {'latitude': 25.6359, 'longitude': 85.0474},
-    "darbhanga": {'latitude': 26.157, 'longitude': 85.8995},
-    "dariapur": {'latitude': 25.8275, 'longitude': 85.0565},
-    "daudnagar": {'latitude': 25.044, 'longitude': 84.4323},
-    "dehri": {'latitude': 24.9259, 'longitude': 84.1729},
-    "dhaka": {'latitude': 26.6924, 'longitude': 85.1986},
-    "dhamdaha": {'latitude': 25.7378, 'longitude': 87.2338},
-    "dumraon": {'latitude': 25.5238, 'longitude': 84.16},
-    "dumri katsari": {'latitude': 26.4691, 'longitude': 85.2195},
-    "forbesganj": {'latitude': 26.2784, 'longitude': 87.2988},
-    "gaya": {'latitude': 24.7964, 'longitude': 85.008},
-    "ghatkusumbha": {'latitude': 25.2127, 'longitude': 85.9308},
-    "ghoshi": {'latitude': 25.1296, 'longitude': 85.0976},
-    "gogri": {'latitude': 25.4294, 'longitude': 86.7107},
-    "gopalganj": {'latitude': 26.5382, 'longitude': 84.4589},
-    "goriakothi": {'latitude': 26.2448, 'longitude': 84.575},
-    "hajipur": {'latitude': 25.6906, 'longitude': 85.209},
-    "halsi": {'latitude': 25.0332, 'longitude': 86.0554},
-    "hilsa": {'latitude': 25.3007, 'longitude': 85.2494},
-    "hisua": {'latitude': 24.8457, 'longitude': 85.3954},
-    "islampur": {'latitude': 25.0969, 'longitude': 85.2604},
-    "jagdishpur": {'latitude': 25.4819, 'longitude': 84.4569},
-    "jale": {'latitude': 26.3563, 'longitude': 85.769},
-    "jamalpur": {'latitude': 25.3139, 'longitude': 86.4897},
-    "jamui": {'latitude': 24.9669, 'longitude': 86.174},
-    "jehanabad": {'latitude': 25.2265, 'longitude': 84.9911},
-    "jhajha": {'latitude': 24.7729, 'longitude': 86.3863},
-    "jhanjharpur": {'latitude': 26.2872, 'longitude': 86.3404},
-    "jokihat": {'latitude': 26.0816, 'longitude': 87.5935},
-    "kahalgaon": {'latitude': 25.2652, 'longitude': 87.2835},
-    "kaler": {'latitude': 25.1549, 'longitude': 84.5414},
-    "kanti": {'latitude': 26.1819, 'longitude': 85.2711},
-    "karpi": {'latitude': 25.1714, 'longitude': 84.7256},
-    "kateya": {'latitude': 26.5723, 'longitude': 84.0811},
-    "katihar": {'latitude': 25.5434, 'longitude': 87.569},
-    "khagaria": {'latitude': 25.5064, 'longitude': 86.4663},
-    "kharagpur": {'latitude': 25.133, 'longitude': 86.5248},
-    "kishanganj": {'latitude': 26.1014, 'longitude': 87.9508},
-    "kochadhaman": {'latitude': 26.1014, 'longitude': 87.9508},
-    "kurtha": {'latitude': 25.1333, 'longitude': 84.8113},
-    "lakhisarai": {'latitude': 25.1678, 'longitude': 86.0594},
-    "lalganj": {'latitude': 25.8521, 'longitude': 85.1896},
-    "madhepura": {'latitude': 25.926, 'longitude': 86.8204},
-    "madhubani": {'latitude': 26.3489, 'longitude': 86.0772},
-    "maharajganj": {'latitude': 26.141, 'longitude': 84.5318},
-    "mahnar": {'latitude': 25.622, 'longitude': 85.5091},
-    "makhdumpur": {'latitude': 25.0739, 'longitude': 84.9879},
-    "manihari": {'latitude': 25.3414, 'longitude': 87.6195},
-    "mansi": {'latitude': 25.546, 'longitude': 86.5504},
-    "marhaura": {'latitude': 25.9302, 'longitude': 84.795},
-    "masaurhi": {'latitude': 25.3516, 'longitude': 84.9873},
-    "modanganj": {'latitude': 25.2591, 'longitude': 85.1584},
-    "mohania": {'latitude': 25.1677, 'longitude': 83.6216},
-    "motihari": {'latitude': 26.6507, 'longitude': 84.9115},
-    "motipur": {'latitude': 26.2571, 'longitude': 85.1689},
-    "munger": {'latitude': 25.3774, 'longitude': 86.4731},
-    "murliganj": {'latitude': 25.8764, 'longitude': 86.9369},
-    "muzaffarpur": {'latitude': 26.1183, 'longitude': 85.3858},
-    "nabinagar": {'latitude': 24.6428, 'longitude': 84.0968},
-    "narkatiaganj": {'latitude': 27.0969, 'longitude': 84.4585},
-    "nathnagar": {'latitude': 25.2184, 'longitude': 86.9177},
-    "nawada": {'latitude': 24.9348, 'longitude': 85.5488},
-    "nirmali": {'latitude': 26.3789, 'longitude': 86.7324},
-    "nokha": {'latitude': 25.0832, 'longitude': 84.0932},
-    "obra": {'latitude': 24.9377, 'longitude': 84.4326},
-    "parihar": {'latitude': 26.7512, 'longitude': 85.6799},
-    "patna": {'latitude': 25.6093, 'longitude': 85.1235},
-    "patori": {'latitude': 25.622, 'longitude': 85.5869},
-    "phulparas": {'latitude': 26.386, 'longitude': 86.4547},
-    "phulwari": {'latitude': 25.543, 'longitude': 85.0523},
-    "piprahi": {'latitude': 26.3489, 'longitude': 86.0772},
-    "piro": {'latitude': 25.3383, 'longitude': 84.3541},
-    "pranpur": {'latitude': 25.4876, 'longitude': 87.7082},
-    "purnia": {'latitude': 25.7774, 'longitude': 87.4731},
-    "raghopur": {'latitude': 25.5714, 'longitude': 85.3241},
-    "rajauli": {'latitude': 24.6343, 'longitude': 85.5609},
-    "rajgir": {'latitude': 25.03, 'longitude': 85.4207},
-    "ramgarh": {'latitude': 25.304, 'longitude': 83.6808},
-    "ramnagar": {'latitude': 27.2909, 'longitude': 84.2936},
-    "raniganj": {'latitude': 26.0553, 'longitude': 87.3077},
-    "raxaul": {'latitude': 26.9869, 'longitude': 84.8447},
-    "rosera": {'latitude': 25.7798, 'longitude': 86.0425},
-    "sabour": {'latitude': 25.2282, 'longitude': 87.0636},
-    "saharsa": {'latitude': 25.8759, 'longitude': 86.5927},
-    "salkhua": {'latitude': 25.6653, 'longitude': 86.5498},
-    "samastipur": {'latitude': 25.8597, 'longitude': 85.7839},
-    "saraiya": {'latitude': 26.0701, 'longitude': 85.1843},
-    "sasaram": {'latitude': 24.951, 'longitude': 84.0149},
-    "shahpur": {'latitude': 25.6593, 'longitude': 84.4295},
-    "sheikhpura": {'latitude': 25.1493, 'longitude': 85.8766},
-    "sheohar": {'latitude': 26.5158, 'longitude': 85.2849},
-    "sherghati": {'latitude': 24.5939, 'longitude': 84.8479},
-    "sikandra": {'latitude': 24.9251, 'longitude': 86.0462},
-    "simri": {'latitude': 25.6499, 'longitude': 84.1384},
-    "simri bakhtiyarpur": {'latitude': 25.7209, 'longitude': 86.5975},
-    "sitamarhi": {'latitude': 26.5943, 'longitude': 85.5004},
-    "siwan": {'latitude': 26.2185, 'longitude': 84.3585},
-    "sonbarsa": {'latitude': 25.7013, 'longitude': 86.7794},
-    "sono": {'latitude': 24.6998, 'longitude': 86.2794},
-    "supaul": {'latitude': 26.0846, 'longitude': 86.5785},
-    "surajgarha": {'latitude': 25.188, 'longitude': 86.2868},
-    "tarapur": {'latitude': 25.1039, 'longitude': 86.647},
-    "tariyani": {'latitude': 26.5158, 'longitude': 85.2849},
-    "teghra": {'latitude': 25.4727, 'longitude': 85.9368},
-    "tekari": {'latitude': 24.9411, 'longitude': 84.843},
-    "thakurganj": {'latitude': 26.4275, 'longitude': 88.129},
-    "triveniganj": {'latitude': 26.127, 'longitude': 86.8873},
-    "udakishunganj": {'latitude': 25.926, 'longitude': 86.8204},
-    "warsaliganj": {'latitude': 24.9348, 'longitude': 85.5488},
+    "Araria": {"latitude": 26.1523, "longitude": 87.5167},
+    "Forbesganj": {"latitude": 26.3023, "longitude": 87.2664},
+    "Jokihat": {"latitude": 25.8998, "longitude": 87.2686},
+    "Raniganj": {"latitude": 26.0537, "longitude": 87.5333},
+    "Arwal": {"latitude": 25.2560, "longitude": 84.6819},
+    "Kaler": {"latitude": 25.1960, "longitude": 84.6219},
+    "Karpi": {"latitude": 25.2360, "longitude": 84.7019},
+    "Kurtha": {"latitude": 25.3160, "longitude": 84.6619},
+    "Aurangabad": {"latitude": 24.7521, "longitude": 84.3742},
+    "Daudnagar": {"latitude": 25.0337, "longitude": 84.4007},
+    "Obra": {"latitude": 24.9923, "longitude": 84.4342},
+    "Nabinagar": {"latitude": 24.6087, "longitude": 84.1269},
+    "Banka": {"latitude": 24.8893, "longitude": 86.9220},
+    "Amarpur": {"latitude": 25.0393, "longitude": 86.9020},
+    "Barahat": {"latitude": 24.8393, "longitude": 87.0020},
+    "Belhar": {"latitude": 24.9393, "longitude": 86.9620},
+    "Begusarai": {"latitude": 25.4182, "longitude": 86.1347},
+    "Bakhri": {"latitude": 25.4582, "longitude": 86.0547},
+    "Barauni": {"latitude": 25.4751, "longitude": 86.0458},
+    "Teghra": {"latitude": 25.5082, "longitude": 85.9347},
+    "Bhagalpur": {"latitude": 25.2425, "longitude": 86.9842},
+    "Sabour": {"latitude": 25.2375, "longitude": 87.0542},
+    "Nathnagar": {"latitude": 25.1225, "longitude": 87.0042},
+    "Kahalgaon": {"latitude": 25.1925, "longitude": 87.2142},
+    "Arrah": {"latitude": 25.5560, "longitude": 84.6631},
+    "Jagdishpur": {"latitude": 25.4660, "longitude": 84.4231},
+    "Piro": {"latitude": 25.3260, "longitude": 84.4031},
+    "Shahpur": {"latitude": 25.6060, "longitude": 84.4031},
+    "Buxar": {"latitude": 25.5641, "longitude": 83.9778},
+    "Dumraon": {"latitude": 25.5541, "longitude": 84.1478},
+    "Chausa": {"latitude": 25.5241, "longitude": 83.9178},
+    "Simri": {"latitude": 25.6141, "longitude": 84.0478},
+    "Darbhanga": {"latitude": 26.1542, "longitude": 85.8978},
+    "Baheri": {"latitude": 26.0442, "longitude": 85.8378},
+    "Jale": {"latitude": 26.2042, "longitude": 85.8578},
+    "Benipur": {"latitude": 26.1142, "longitude": 85.9478},
+    "Motihari": {"latitude": 26.6484, "longitude": 84.9194},
+    "Raxaul": {"latitude": 26.9784, "longitude": 84.8494},
+    "Chakia": {"latitude": 26.4184, "longitude": 85.0494},
+    "Dhaka": {"latitude": 26.6784, "longitude": 85.1694},
+    "Gaya": {"latitude": 24.7955, "longitude": 85.0002},
+    "Bodh Gaya": {"latitude": 24.6955, "longitude": 84.9902},
+    "Tekari": {"latitude": 24.9455, "longitude": 85.0402},
+    "Sherghati": {"latitude": 24.5655, "longitude": 84.7902},
+    "Gopalganj": {"latitude": 26.4685, "longitude": 84.4388},
+    "Barauli": {"latitude": 26.3785, "longitude": 84.5788},
+    "Baikunthpur": {"latitude": 26.5285, "longitude": 84.3588},
+    "Kateya": {"latitude": 26.4285, "longitude": 84.6388},
+    "Jamui": {"latitude": 24.9272, "longitude": 86.2231},
+    "Jhajha": {"latitude": 24.7772, "longitude": 86.3731},
+    "Sikandra": {"latitude": 24.9672, "longitude": 86.0631},
+    "Sono": {"latitude": 24.8372, "longitude": 86.1431},
+    "Jehanabad": {"latitude": 25.2078, "longitude": 84.9869},
+    "Ghoshi": {"latitude": 25.1478, "longitude": 84.9269},
+    "Makhdumpur": {"latitude": 25.2478, "longitude": 85.0469},
+    "Modanganj": {"latitude": 25.2678, "longitude": 84.9069},
+    "Bhabua": {"latitude": 25.0405, "longitude": 83.6074},
+    "Mohania": {"latitude": 25.1305, "longitude": 83.4774},
+    "Ramgarh": {"latitude": 24.9505, "longitude": 83.6874},
+    "Chainpur": {"latitude": 25.2005, "longitude": 83.7474},
+    "Katihar": {"latitude": 25.5394, "longitude": 87.5839},
+    "Barsoi": {"latitude": 25.3794, "longitude": 87.8839},
+    "Manihari": {"latitude": 25.3394, "longitude": 87.6239},
+    "Pranpur": {"latitude": 25.6894, "longitude": 87.7239},
+    "Khagaria": {"latitude": 25.5022, "longitude": 86.4665},
+    "Parbatta": {"latitude": 25.5422, "longitude": 86.5865},
+    "Alauli": {"latitude": 25.4622, "longitude": 86.3465},
+    "Beldaur": {"latitude": 25.5622, "longitude": 86.4265},
+    "Kishanganj": {"latitude": 26.1046, "longitude": 87.9475},
+    "Bahadurganj": {"latitude": 26.2646, "longitude": 88.1175},
+    "Thakurganj": {"latitude": 26.0446, "longitude": 87.8275},
+    "Dighalbank": {"latitude": 25.9046, "longitude": 87.9875},
+    "Lakhisarai": {"latitude": 25.1678, "longitude": 86.0927},
+    "Halsi": {"latitude": 25.2278, "longitude": 86.0327},
+    "Suryagarha": {"latitude": 25.1078, "longitude": 86.1527},
+    "Pipariya": {"latitude": 25.2078, "longitude": 86.1327},
+    "Madhepura": {"latitude": 25.9207, "longitude": 86.7940},
+    "Udakishanganj": {"latitude": 25.9807, "longitude": 86.6740},
+    "Murliganj": {"latitude": 25.8907, "longitude": 86.9940},
+    "Alamnagar": {"latitude": 25.9607, "longitude": 86.7340},
+    "Madhubani": {"latitude": 26.3561, "longitude": 86.0644},
+    "Jhanjharpur": {"latitude": 26.2661, "longitude": 86.2844},
+    "Benipatti": {"latitude": 26.5961, "longitude": 86.1444},
+    "Jainagar": {"latitude": 26.2061, "longitude": 86.1644},
+    "Munger": {"latitude": 25.3753, "longitude": 86.4734},
+    "Jamalpur": {"latitude": 25.3153, "longitude": 86.4934},
+    "Asarganj": {"latitude": 25.1453, "longitude": 86.6834},
+    "Tarapur": {"latitude": 25.0253, "longitude": 86.6334},
+    "Muzaffarpur": {"latitude": 26.1225, "longitude": 85.3906},
+    "Sitamarhi": {"latitude": 26.5925, "longitude": 85.4806},
+    "Minapur": {"latitude": 26.0625, "longitude": 85.2906},
+    "Bochaha": {"latitude": 26.0025, "longitude": 85.5306},
+    "Bihar Sharif": {"latitude": 25.1979, "longitude": 85.5238},
+    "Rajgir": {"latitude": 25.0279, "longitude": 85.4238},
+    "Hilsa": {"latitude": 25.3179, "longitude": 85.2838},
+    "Biharsharif": {"latitude": 25.1979, "longitude": 85.5238},
+    "Nawada": {"latitude": 24.8834, "longitude": 85.5387},
+    "Rajauli": {"latitude": 25.0634, "longitude": 85.6387},
+    "Akbarpur": {"latitude": 24.8234, "longitude": 85.4587},
+    "Hisua": {"latitude": 24.8334, "longitude": 85.4187},
+    "Patna": {"latitude": 25.5941, "longitude": 85.1376},
+    "Danapur": {"latitude": 25.6341, "longitude": 85.0476},
+    "Fatuha": {"latitude": 25.5041, "longitude": 85.3076},
+    "Khagaul": {"latitude": 25.5741, "longitude": 85.0476},
+    "Purnia": {"latitude": 25.7771, "longitude": 87.4753},
+    "Dhamdaha": {"latitude": 25.8871, "longitude": 87.5853},
+    "Kasba": {"latitude": 25.8471, "longitude": 87.5353},
+    "Banmankhi": {"latitude": 25.8871, "longitude": 87.1953},
+    "Sasaram": {"latitude": 24.9520, "longitude": 84.0328},
+    "Dehri": {"latitude": 24.9020, "longitude": 84.1828},
+    "Bikramganj": {"latitude": 25.2120, "longitude": 84.2628},
+    "Nasriganj": {"latitude": 25.0520, "longitude": 84.1228},
+    "Saharsa": {"latitude": 25.8769, "longitude": 86.5956},
+    "Sonbarsa": {"latitude": 25.9269, "longitude": 86.7356},
+    "Simri Bakhtiarpur": {"latitude": 25.9569, "longitude": 86.3556},
+    "Mahishi": {"latitude": 25.9969, "longitude": 86.4756},
+    "Samastipur": {"latitude": 25.8647, "longitude": 85.7817},
+    "Rosera": {"latitude": 25.7947, "longitude": 85.9317},
+    "Dalsinghsarai": {"latitude": 25.6647, "longitude": 85.8317},
+    "Pusa": {"latitude": 25.9847, "longitude": 85.6717},
+    "Chapra": {"latitude": 25.7805, "longitude": 84.7477},
+    "Marhaura": {"latitude": 25.9705, "longitude": 84.8677},
+    "Amnour": {"latitude": 25.8905, "longitude": 84.9077},
+    "Sonepur": {"latitude": 25.6905, "longitude": 85.1777},
+    "Sheikhpura": {"latitude": 25.1391, "longitude": 85.8354},
+    "Barbigha": {"latitude": 25.2191, "longitude": 85.7354},
+    "Ariari": {"latitude": 25.0591, "longitude": 85.9554},
+    "Shekhopur": {"latitude": 25.1391, "longitude": 85.8354},
+    "Sheohar": {"latitude": 26.5184, "longitude": 85.2959},
+    "Dumri Katsari": {"latitude": 26.5784, "longitude": 85.1959},
+    "Piprahi": {"latitude": 26.4684, "longitude": 85.4159},
+    "Tariyani": {"latitude": 26.5584, "longitude": 85.2359},
+    "Sitamarhi": {"latitude": 26.5925, "longitude": 85.4806},
+    "Pupri": {"latitude": 26.4725, "longitude": 85.7006},
+    "Belsand": {"latitude": 26.4425, "longitude": 85.4006},
+    "Bathnaha": {"latitude": 26.5925, "longitude": 85.5306},
+    "Siwan": {"latitude": 26.2195, "longitude": 84.3564},
+    "Maharajganj": {"latitude": 26.1095, "longitude": 84.5064},
+    "Mairwa": {"latitude": 26.2295, "longitude": 84.1664},
+    "Darauli": {"latitude": 26.1595, "longitude": 84.1464},
+    "Supaul": {"latitude": 26.1260, "longitude": 86.6050},
+    "Nirmali": {"latitude": 26.3160, "longitude": 86.5850},
+    "Triveniganj": {"latitude": 26.2160, "longitude": 87.0250},
+    "Chhatapur": {"latitude": 26.2160, "longitude": 86.9050},
+    "Hajipur": {"latitude": 25.6851, "longitude": 85.2095},
+    "Mahua": {"latitude": 25.9651, "longitude": 85.2895},
+    "Lalganj": {"latitude": 25.8751, "longitude": 85.1695},
+    "Desri": {"latitude": 25.6051, "longitude": 85.4895},
+    "Bettiah": {"latitude": 26.8022, "longitude": 84.5025},
+    "Bagaha": {"latitude": 27.0922, "longitude": 84.0925},
+    "Narkatiaganj": {"latitude": 26.4322, "longitude": 84.7925},
+    "Lauriya": {"latitude": 26.9822, "longitude": 84.3125}
 }
 
 
+# ============================================================================
+# FASTAPI APP
+# ============================================================================
+
+app = FastAPI(title="Farmer.chat Alert Summary API")
+
+# CORS middleware
+app.add_middleware(
+    CORSMiddleware,
+    allow_origins=["*"],
+    allow_credentials=True,
+    allow_methods=["*"],
+    allow_headers=["*"],
+)
+
+# Initialize pipeline
+openai_client = None
+pipeline = None
+
+@app.on_event("startup")
+async def startup_event():
+    """Initialize OpenAI client and pipeline on startup"""
+    global openai_client, pipeline
+    try:
+        openai_client = get_openai_client()
+        DEFAULT_LOCATION = {"latitude": 25.5941, "longitude": 85.1376}
+        pipeline = FarmerChatPipeline(openai_client, DEFAULT_LOCATION)
+        print("✓ Pipeline initialized successfully")
+    except Exception as e:
+        print(f"✗ Pipeline initialization failed: {e}")
+        raise
+
+
+# ============================================================================
+# API MODELS
+# ============================================================================
+
 class LocationRequest(BaseModel):
     location_name: str
     district: Optional[str] = None
@@ -455,91 +644,103 @@ class LocationRequest(BaseModel):
 class AlertResponse(BaseModel):
     location: str
     coordinates: Dict[str, float]
+    district: Optional[str] = None
     alert_summary: str
     timestamp: str
-    district: Optional[str] = None
 
 
+# ============================================================================
+# API ENDPOINTS
+# ============================================================================
+
 @app.get("/")
 async def root():
+    """Root endpoint"""
     return {
         "message": "Farmer.chat Alert Summary API",
         "version": "1.0.0",
         "endpoints": {
-            "/locations": "Get all available districts and villages",
-            "/generate-alert": "Generate alert summary for a location"
+            "/locations": "GET - List all districts and villages",
+            "/generate-alert": "POST - Generate alert for location",
+            "/health": "GET - Health check"
         }
     }
 
 
-@app.get("/locations")
-async def get_locations():
-    """Get all available districts and villages in Bihar"""
+@app.get("/health")
+async def health_check():
+    """Health check endpoint"""
     return {
-        "state": "Bihar",
-        "districts": BIHAR_DATA,
-        "total_districts": len(BIHAR_DATA),
-        "total_villages": sum(len(villages) for villages in BIHAR_DATA.values())
+        "status": "healthy",
+        "pipeline": "initialized" if pipeline else "not_initialized",
+        "timestamp": datetime.now().isoformat()
     }
 
 
+@app.get("/locations")
+async def get_locations():
+    """Get all districts and villages"""
+    return {"districts": BIHAR_DATA}
+
+
 @app.post("/generate-alert", response_model=AlertResponse)
 async def generate_alert(request: LocationRequest):
-    """
-    Generate comprehensive alert summary for a specific location.
-    Automatically queries all MCP servers (Weather, Soil, Water, Elevation, Pests).
-    """
+    """Generate alert summary for selected location"""
+    
+    if not pipeline:
+        raise HTTPException(status_code=500, detail="Pipeline not initialized")
+    
+    # Find location coordinates
+    location_name = request.location_name.strip()
+    coordinates = None
+    
+    # Case-insensitive search
+    for loc_key, loc_coords in LOCATIONS.items():
+        if loc_key.lower() == location_name.lower():
+            coordinates = loc_coords
+            break
+    
+    if not coordinates:
+        raise HTTPException(
+            status_code=404,
+            detail=f"Location '{location_name}' not found in database"
+        )
+    
+    # Update pipeline location
+    pipeline.location = coordinates
+    
+    # Create comprehensive query for all MCP servers
+    query = f"""Generate a comprehensive agricultural alert summary for {location_name} covering:
+1. Current weather conditions and 7-day forecast
+2. Soil health analysis and fertilizer recommendations
+3. Groundwater availability and irrigation status
+4. Elevation and topography considerations for farming
+5. Current pest risks and preventive measures
+
+Provide actionable insights for farmers."""
+    
     try:
-        # Get coordinates for the location
-        location_key = request.location_name.lower()
-        coords = LOCATIONS.get(location_key)
-        
-        if not coords:
-            raise HTTPException(
-                status_code=404,
-                detail=f"Location '{request.location_name}' not found in database"
-            )
-        
-        # Update pipeline location
-        pipeline.location = coords
-        
-        # Create comprehensive query for alert generation
-        alert_query = f"""Generate a comprehensive agricultural alert summary for {request.location_name}, Bihar.
-        
-Please provide:
-1. Weather conditions and forecast
-2. Soil analysis and recommendations
-3. Water availability status
-4. Elevation and topography considerations
-5. Pest risk assessment and preventive measures
-
-Format the response as a clear, actionable alert summary for farmers."""
-        
-        # Process through the pipeline
-        result = await pipeline.process(alert_query, coords)
+        # Generate alert
+        alert_summary = pipeline.process_query(query)
         
         return AlertResponse(
-            location=request.location_name.title(),
-            coordinates=coords,
-            alert_summary=result,
-            timestamp=datetime.utcnow().isoformat(),
-            district=request.district
+            location=location_name,
+            coordinates=coordinates,
+            district=request.district,
+            alert_summary=alert_summary,
+            timestamp=datetime.now().isoformat()
         )
-        
-    except HTTPException:
-        raise
+    
     except Exception as e:
-        raise HTTPException(status_code=500, detail=str(e))
-
+        raise HTTPException(
+            status_code=500,
+            detail=f"Error generating alert: {str(e)}"
+        )
 
-@app.get("/health")
-async def health_check():
-    return {
-        "status": "healthy",
-        "timestamp": datetime.utcnow().isoformat(),
-        "pipeline_initialized": pipeline is not None
-    }
 
+# ============================================================================
+# RUN SERVER
+# ============================================================================
 
 if __name__ == "__main__":
     import uvicorn