Update app/app.py

app/app.py

@@ -16,7 +16,7 @@ from fastapi.responses import HTMLResponse, StreamingResponse
 from fastapi.templating import Jinja2Templates
 from pydantic import BaseModel, constr
 
-# These relative imports will now work correctly.
+# Import our modular, asynchronous service classes using relative paths
 from .price_fetcher import PriceFetcher
 from .sentiment import SentimentAnalyzer
 
@@ -38,6 +38,7 @@ async def lifespan(app: FastAPI):
     # On startup:
     async with httpx.AsyncClient() as client:
         # Instantiate and store our services in the application state.
+        # This makes them accessible in any request handler via `request.app.state`.
         app.state.price_fetcher = PriceFetcher(client=client, coins=["bitcoin", "ethereum", "dogecoin"])
         app.state.sentiment_analyzer = SentimentAnalyzer(client=client)
         app.state.request_counter = 0
@@ -48,7 +49,7 @@ async def lifespan(app: FastAPI):
         )
         
         print("🚀 CryptoSentinel AI started successfully.")
-        yield
+        yield  # The application is now running and ready to accept requests.
         
         # On shutdown:
         print("⏳ Shutting down background tasks...")
@@ -68,44 +69,78 @@ async def run_periodic_updates(fetcher: PriceFetcher, interval_seconds: int):
 # --- FastAPI App Initialization ---
 
 app = FastAPI(title="CryptoSentinel AI", lifespan=lifespan)
-# This path assumes the app is run from the root directory.
-templates = Jinja2Templates(directory="app/templates")
+
+# ====================================================================
+#                       FIX APPLIED HERE
+# ====================================================================
+# The 'templates' directory is at the root of the project, not inside 'app'.
+# This path is relative to the directory where the run command is executed.
+templates = Jinja2Templates(directory="templates")
+# ====================================================================
+
 
 # --- API Endpoints ---
-# ... (The rest of the file is unchanged) ...
+
 @app.get("/", response_class=HTMLResponse)
 async def serve_dashboard(request: Request):
+    """Serves the main interactive dashboard from `index.html`."""
     return templates.TemplateResponse("index.html", {"request": request})
 
 @app.get("/api/prices", response_class=HTMLResponse)
 async def get_prices_fragment(request: Request):
+    """Returns an HTML fragment with the latest cached crypto prices for HTMX."""
     price_fetcher: PriceFetcher = request.app.state.price_fetcher
     prices = price_fetcher.get_current_prices()
+
     html_fragment = ""
     for coin, price in prices.items():
+        # Format the price nicely, handling the initial '--' state
         price_str = f"${price:,.2f}" if isinstance(price, (int, float)) else price
         html_fragment += f"<div><strong>{coin.capitalize()}:</strong> {price_str}</div>"
+    
     return HTMLResponse(content=html_fragment)
 
 @app.post("/api/sentiment")
-async def analyze_sentiment(payload: SentimentRequest, request: Request, background_tasks: BackgroundTasks):
+async def analyze_sentiment(
+    payload: SentimentRequest,
+    request: Request,
+    background_tasks: BackgroundTasks
+):
+    """
+    Validates and queues a text for sentiment analysis. The heavy lifting is
+    done in the background to ensure the API responds instantly.
+    """
     analyzer: SentimentAnalyzer = request.app.state.sentiment_analyzer
     request.app.state.request_counter += 1
     request_id = request.app.state.request_counter
+
+    # The actual API call to Hugging Face will run after this response is sent.
     background_tasks.add_task(analyzer.compute_and_publish, payload.text, request_id)
+    
     return HTMLResponse(content="<small>Queued for analysis...</small>")
 
 @app.get("/api/sentiment/stream")
 async def sentiment_stream(request: Request):
+    """
+    Establishes a Server-Sent Events (SSE) connection. It efficiently pushes
+    new sentiment results as HTML fragments to the client as they become available.
+    """
     analyzer: SentimentAnalyzer = request.app.state.sentiment_analyzer
+
     async def event_generator():
+        # Clear the initial "waiting..." message on the client.
+        # hx-swap-oob="innerHTML" swaps this div out-of-band without affecting the target.
         yield f"event: sentiment_update\ndata: <div id='sentiment-results' hx-swap-oob='innerHTML'></div>\n\n"
+        
+        # Listen for new results from the analyzer's internal queue.
         async for result_payload in analyzer.stream_results():
             try:
                 result = result_payload['result']
                 label = str(result.get('label', 'NEUTRAL')).lower()
                 score = result.get('score', 0.0) * 100
                 text = result_payload['text']
+                
+                # Dynamically build the HTML fragment to be sent to the client.
                 html_fragment = f"""
                 <div>
                     <blockquote>{text}</blockquote>
@@ -116,16 +151,22 @@ async def sentiment_stream(request: Request):
                     </p>
                 </div>
                 """
+                # First, process the string to remove newlines.
                 data_payload = html_fragment.replace('\n', '')
+                
+                # Then, use the clean variable in the f-string to build the message.
                 sse_message = f"event: sentiment_update\ndata: {data_payload}\n\n"
+                
                 yield sse_message
+
             except (KeyError, TypeError):
-                continue
+                continue # Ignore malformed payloads
+
     return StreamingResponse(event_generator(), media_type="text/event-stream")
 
 # This block is now mostly for IDEs, the primary run method is the uvicorn command.
 if __name__ == "__main__":
-    # Note: Running this file directly (`python app/app.py`) will fail due to relative imports.
-    # Use the command: `uvicorn app.app:app --reload` from the project root.
+    # Note: Running this file directly (`python app/main.py`) will fail due to relative imports.
+    # Use the command: `uvicorn app.main:app --reload` from the project root.
     print("To run this application, use the command from the root directory:")
-    print("uvicorn app.app:app --host 0.0.0.0 --port 7860 --reload")
+    print("uvicorn app.main:app --host 0.0.0.0 --port 7860 --reload")