Update app.py

app.py

@@ -19,17 +19,26 @@ from typing import Dict, List, Any
 # Load environment variables
 load_dotenv()
 
-# Environment-based configuration
+# Environment-based configuration with Spaces compatibility
 SAMBANOVA_API_KEY = os.getenv("SAMBANOVA_API_KEY", "")
 MODAL_TOKEN_ID = os.getenv("MODAL_TOKEN_ID", "")
 MODAL_TOKEN_SECRET = os.getenv("MODAL_TOKEN_SECRET", "")
 USDA_NASS_API_KEY = os.getenv("USDA_NASS_API_KEY", "")
-GRADIO_SERVER_PORT = int(os.getenv("GRADIO_SERVER_PORT", "7864"))
+OPENWEATHER_API_KEY = os.getenv("OPENWEATHER_API_KEY", "")
+# Hugging Face Spaces uses port 7860 by default
+GRADIO_SERVER_PORT = int(os.getenv("GRADIO_SERVER_PORT", "7860"))
 GRADIO_SERVER_NAME = os.getenv("GRADIO_SERVER_NAME", "0.0.0.0")
-GRADIO_SHARE = os.getenv("GRADIO_SHARE", "false").lower() == "false"
+GRADIO_SHARE = os.getenv("GRADIO_SHARE", "false").lower() == "true"  # Disable share for Spaces
 DEBUG_MODE = os.getenv("DEBUG_MODE", "false").lower() == "true"
 CONTEXT7_ENABLED = os.getenv("CONTEXT7_ENABLED", "true").lower() == "true"
 
+# Detect if running in Hugging Face Spaces
+IS_SPACES = os.getenv("SPACE_ID") is not None
+if IS_SPACES:
+    print("🤗 Running in Hugging Face Spaces environment")
+    GRADIO_SHARE = False  # Never share when in Spaces
+    DEBUG_MODE = False    # Disable debug in production Spaces
+
 # MCP Server Configuration
 MCP_SERVER_ENABLED = True
 MCP_TOOLS_AVAILABLE = [
@@ -55,7 +64,7 @@ class MCPAgriculturalAI:
     async def generate_analysis(self, prompt: str, context: Dict) -> str:
         """Generate real AI analysis using SambaNova API"""
         if not self.available:
-            return "AI analysis unavailable - API key not configured"
+            return self._get_fallback_analysis(prompt, context)
             
         try:
             headers = {
@@ -80,7 +89,7 @@ class MCPAgriculturalAI:
                 "max_tokens": 2000
             }
             
-            async with httpx.AsyncClient(timeout=30.0) as client:
+            async with httpx.AsyncClient(timeout=10.0) as client:  # Reduced timeout for Spaces
                 response = await client.post(
                     f"{self.base_url}/chat/completions",
                     headers=headers,
@@ -91,10 +100,50 @@ class MCPAgriculturalAI:
                     result = response.json()
                     return result["choices"][0]["message"]["content"]
                 else:
-                    return f"AI API Error: {response.status_code} - {response.text}"
+                    return self._get_fallback_analysis(prompt, context)
                     
         except Exception as e:
-            return f"AI Analysis Error: {str(e)}"
+            print(f"AI API Error (falling back to template): {str(e)}")
+            return self._get_fallback_analysis(prompt, context)
+    
+    def _get_fallback_analysis(self, prompt: str, context: Dict) -> str:
+        """Provide fallback analysis when AI API is unavailable"""
+        location = context.get("location", {})
+        region = context.get("region", {})
+        farm = context.get("farm", {})
+        
+        lat = location.get("lat", 0)
+        lon = location.get("lon", 0)
+        region_name = region.get("name", "Unknown Region")
+        
+        return f"""
+### 🌾 Agricultural Analysis (Fallback Mode)
+
+**Location**: {lat:.4f}°N, {lon:.4f}°E ({region_name})
+**Status**: AI analysis temporarily unavailable - using expert templates
+
+**Crop Recommendations:**
+• **Wheat**: Excellent choice for temperate climates
+• **Corn**: High yield potential with proper irrigation
+• **Barley**: Good rotation crop with disease resistance
+
+**Economic Projections:**
+• Expected revenue: €2,500-3,500/hectare
+• Operating costs: €1,200-1,600/hectare
+• Net profit potential: €1,300-1,900/hectare
+
+**Risk Assessment:**
+• Weather risk: Moderate (use crop insurance)
+• Market volatility: Low to moderate
+• Disease pressure: Standard prevention recommended
+
+**Sustainability Score: 80/100**
+• Water efficiency: Good
+• Soil health: Maintained with rotation
+• Carbon impact: Neutral to positive
+
+*Note: This is a template analysis. For AI-powered insights, please configure API keys.*
+"""
     
     def get_system_status(self) -> Dict:
         """Get comprehensive system status for MCP"""
@@ -1099,16 +1148,74 @@ if __name__ == "__main__":
     print(f"📍 Server Configuration: {GRADIO_SERVER_NAME}:{GRADIO_SERVER_PORT}")
     print(f"🔧 Environment: {'Production' if not DEBUG_MODE else 'Development'}")
     print(f"🤖 MCP Server: {'✅ Enabled' if MCP_SERVER_ENABLED else '❌ Disabled'}")
-    print(f"🔑 Environment file: {'.env loaded' if os.path.exists('.env') else 'using defaults'}")
+    print(f"🔑 Environment file: {'.env loaded' if os.path.exists('.env') else 'using environment variables'}")
     
-    # Create and launch the MCP-enabled application
-    app = create_mcp_application()
+    if IS_SPACES:
+        print("🤗 Optimized for Hugging Face Spaces deployment")
+        print("💡 Tip: Set API keys in Spaces Settings > Repository secrets for full functionality")
     
-    app.launch(
-        server_name=GRADIO_SERVER_NAME,
-        server_port=GRADIO_SERVER_PORT,
-        share=GRADIO_SHARE,
-        show_error=DEBUG_MODE,
-        inbrowser=True,
-        favicon_path=None
-    ) 
+    # Create and launch the MCP-enabled application
+    try:
+        app = create_mcp_application()
+        
+        # Spaces-optimized launch configuration
+        if IS_SPACES:
+            # Simplified launch for Spaces
+            app.launch(
+                server_name="0.0.0.0",
+                server_port=7860,
+                share=False,
+                show_error=True,
+                inbrowser=False,
+                favicon_path=None,
+                prevent_thread_lock=False
+            )
+        else:
+            # Full configuration for local/other deployments
+            app.launch(
+                server_name=GRADIO_SERVER_NAME,
+                server_port=GRADIO_SERVER_PORT,
+                share=GRADIO_SHARE,
+                show_error=DEBUG_MODE,
+                inbrowser=True,
+                favicon_path=None
+            )
+    except Exception as e:
+        print(f"❌ Error launching application: {e}")
+        if IS_SPACES:
+            print("🔧 Creating minimal fallback interface...")
+            # Create a minimal fallback interface for Spaces
+            import gradio as gr
+            
+            def fallback_message():
+                return """
+# 🌾 CropCortex MCP Server - Startup Error
+
+The application encountered an error during startup. This is likely due to:
+
+1. **Missing API Keys**: Configure SambaNova API key in Spaces settings
+2. **Dependencies**: Some packages may need to be installed
+3. **Environment**: Check that all required environment variables are set
+
+## Quick Fix:
+1. Go to your Space settings
+2. Add `SAMBANOVA_API_KEY` in Repository secrets
+3. Restart the Space
+
+## Fallback Mode:
+The application is running in basic mode with limited functionality.
+                """
+            
+            fallback_app = gr.Interface(
+                fn=lambda: fallback_message(),
+                inputs=[],
+                outputs=gr.Markdown(),
+                title="CropCortex MCP - Fallback Mode",
+                description="Agricultural Intelligence Platform (Limited Mode)"
+            )
+            
+            fallback_app.launch(
+                server_name="0.0.0.0",
+                server_port=7860,
+                share=False
+            )