Update app/app.py

app/app.py

@@ -18,9 +18,10 @@ from fastapi import FastAPI, Request
 from fastapi.responses import HTMLResponse, StreamingResponse
 from fastapi.templating import Jinja2Templates
 
-# Use relative imports because these modules are in the same 'app' package.
-from .price_fetcher import PriceFetcher
-from .gemini_analyzer import GeminiAnalyzer
+# Use relative imports if your files are in an 'app' package,
+# or direct imports if they are at the root. Assuming root for simplicity now.
+from price_fetcher import PriceFetcher
+from gemini_analyzer import GeminiAnalyzer
 from newsapi import NewsApiClient
 
 
@@ -38,15 +39,17 @@ async def lifespan(app: FastAPI):
         app.state.gemini_analyzer = GeminiAnalyzer(client=client)
         app.state.news_api = NewsApiClient(api_key=os.getenv("NEWS_API_KEY"))
 
-        # Create a queue for the real-time signal feed, attached to the app's state.
+        # =================== FIX APPLIED HERE (1/2) ===================
+        # We only need ONE queue for the autonomous signal stream.
         app.state.signal_queue: asyncio.Queue = asyncio.Queue()
+        # =============================================================
 
         # Create cancellable background tasks for periodic updates.
         price_task = asyncio.create_task(
             run_periodic_updates(app.state.price_fetcher, interval_seconds=60)
         )
         news_task = asyncio.create_task(
-            run_periodic_news_analysis(app, interval_seconds=900)  # Run every 15 minutes
+            run_periodic_news_analysis(app, interval_seconds=900)  # Check news every 15 minutes
         )
         
         print("🚀 Sentinel TradeFlow Protocol started successfully.")
@@ -83,8 +86,18 @@ async def run_periodic_news_analysis(app: FastAPI, interval_seconds: int):
                 title = article.get('title')
                 if title and "[Removed]" not in title:
                     analysis = await analyzer.analyze_text(title)
+                    # Don't proceed if Gemini returned an error
+                    if analysis.get("error"):
+                        print(f"Skipping article due to Gemini error: {analysis['reason']}")
+                        continue
+                    
                     analysis['url'] = article.get('url')
+                    # =================== FIX APPLIED HERE (2/2) ===================
+                    # Ensure the result is put into the one and only signal_queue.
                     await app.state.signal_queue.put(analysis)
+                    print(f"✅ Signal generated and queued for: {title}")
+                    # =============================================================
+
         except Exception as e:
             print(f"❌ Error during news fetching or analysis: {e}")
             
@@ -131,19 +144,16 @@ async def serve_dashboard(request: Request):
 @app.get("/api/signals/stream")
 async def signal_stream(request: Request):
     """SSE stream for the automated Signal Stream."""
-    # ====================================================================
-    #                       FIX APPLIED HERE
-    # ====================================================================
-    # Access the shared application state via `request.app.state`, not `request.state`.
+    # This was already correct from the last fix, but we confirm it points to signal_queue.
     queue: asyncio.Queue = request.app.state.signal_queue
-    # ====================================================================
-    
     async def event_generator():
+        # Let's send a confirmation that the stream is connected and ready
+        yield f"event: message\ndata: <div hx-swap-oob='innerHTML' id='signal-stream-container'><p>Status: ONLINE - Listening for new market signals...</p></div>\n\n"
         while True:
             payload = await queue.get()
             html = render_signal_card(payload)
             data_payload = html.replace('\n', '')
-            # Use a generic 'message' event, which HTMX listens to by default
             sse_message = f"event: message\ndata: {data_payload}\n\n"
             yield sse_message
+            
     return StreamingResponse(event_generator(), media_type="text/event-stream")