fix(startup): simplify backend readiness logic for HF Spaces

app.py

@@ -2740,34 +2740,38 @@ if __name__ == "__main__":
     # Start backend services if running in HF Spaces
     if is_hf_space:
         def start_mcp_server():
-            """Start MCP server in background process."""
+            """Start MCP server in a background process."""
             try:
                 import sys
                 import subprocess
-                # Use subprocess.Popen to run in background
-                subprocess.Popen([
-                    sys.executable, "-m", "uvicorn",
-                    "backend.mcp_server.server:app",
-                    "--host", "0.0.0.0",
-                    "--port", os.getenv("MCP_PORT", "8900"),
-                    "--log-level", "info"
-                ], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+                # Use subprocess.Popen to run in background and surface logs in HF Spaces
+                subprocess.Popen(
+                    [
+                        sys.executable, "-m", "uvicorn",
+                        "backend.mcp_server.server:app",
+                        "--host", "0.0.0.0",
+                        "--port", os.getenv("MCP_PORT", "8900"),
+                        "--log-level", "info",
+                    ]
+                )
             except Exception as e:
                 print(f"Warning: Could not start MCP server: {e}")
         
         def start_fastapi_server():
-            """Start FastAPI server in background process."""
+            """Start FastAPI server in a background process."""
             try:
                 import sys
                 import subprocess
-                # Use subprocess.Popen to run in background
-                subprocess.Popen([
-                    sys.executable, "-m", "uvicorn",
-                    "backend.api.main:app",
-                    "--host", "0.0.0.0",
-                    "--port", os.getenv("API_PORT", "8000"),
-                    "--log-level", "info"
-                ], stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+                # Use subprocess.Popen to run in background and surface logs in HF Spaces
+                subprocess.Popen(
+                    [
+                        sys.executable, "-m", "uvicorn",
+                        "backend.api.main:app",
+                        "--host", "0.0.0.0",
+                        "--port", os.getenv("API_PORT", "8000"),
+                        "--log-level", "info",
+                    ]
+                )
             except Exception as e:
                 print(f"Warning: Could not start FastAPI server: {e}")
         
@@ -2777,23 +2781,10 @@ if __name__ == "__main__":
         api_thread = threading.Thread(target=start_fastapi_server, daemon=True)
         
         mcp_thread.start()
-        time.sleep(2)  # Give MCP server time to start
+        time.sleep(4)  # Give MCP server time to start
         api_thread.start()
-        time.sleep(3)  # Give FastAPI time to start
-        
-        # Wait for services to be ready
-        max_attempts = 30
-        for attempt in range(max_attempts):
-            try:
-                response = requests.get("http://localhost:8000/health", timeout=2)
-                if response.status_code == 200:
-                    print("✅ FastAPI backend is ready!")
-                    break
-            except:
-                if attempt < max_attempts - 1:
-                    time.sleep(1)
-                else:
-                    print("⚠️ FastAPI backend may not be ready, continuing anyway...")
+        # Give FastAPI extra time to start on first cold boot (model downloads etc.)
+        time.sleep(10)
     
     demo.launch(
         server_name=server_name,