Fix: Add test route and improve report-demo route with better error handling

chatkit/backend/app/main.py

@@ -239,6 +239,21 @@ async def health_check():
     return {"status": "healthy", "service": "ClassLens"}
 
 
+@app.get("/test-routes")
+async def test_routes():
+    """Test endpoint to verify routes are working."""
+    return {
+        "message": "Routes are working!",
+        "report_demo_route": "/report-demo",
+        "available_paths": [
+            "/health",
+            "/api/health",
+            "/report-demo",
+            "/test-routes"
+        ]
+    }
+
+
 @app.get("/report-template.html")
 async def serve_report_template():
     """Serve the report template HTML file."""
@@ -272,7 +287,7 @@ async def serve_report_demo():
     possible_paths = [p for p in possible_paths if p is not None]
     
     for template_path in possible_paths:
-        if template_path.exists():
+        if template_path and template_path.exists():
             return FileResponse(
                 template_path,
                 media_type="text/html",
@@ -283,8 +298,15 @@ async def serve_report_demo():
     current_dir = base_dir
     files_in_dir = list(current_dir.glob("*")) if current_dir.exists() else []
     searched_paths = [str(p) for p in possible_paths]
+    debug_info = {
+        "searched_paths": searched_paths,
+        "current_dir": str(current_dir),
+        "files_in_dir": [str(f.name) for f in files_in_dir[:10]],
+        "static_dir": str(STATIC_DIR) if STATIC_DIR.exists() else "does not exist",
+        "__file__": str(Path(__file__)),
+    }
     return Response(
-        content=f"Report template not found.\nSearched: {searched_paths}\nCurrent dir: {current_dir}\nFiles in dir: {[str(f) for f in files_in_dir[:10]]}",
+        content=f"Report template not found.\nDebug info:\n{json.dumps(debug_info, indent=2)}",
         status_code=404,
         media_type="text/plain"
     )
@@ -314,8 +336,9 @@ if STATIC_DIR.exists():
     @app.get("/{full_path:path}")
     async def serve_spa_routes(full_path: str):
         """Catch-all route to serve React SPA for client-side routing."""
-        # Don't serve SPA for API routes
-        if full_path.startswith(("api/", "auth/", "chatkit", "report-demo")):
+        # Don't serve SPA for API routes or specific endpoints
+        excluded_paths = ("api/", "auth/", "chatkit", "report-demo", "test-routes", "health")
+        if full_path.startswith(excluded_paths) or full_path in excluded_paths:
             return Response(status_code=404)
         
         # Check if it's a static file