Update app/app.py

app/app.py

@@ -18,14 +18,10 @@ from fastapi import FastAPI, Request
 from fastapi.responses import HTMLResponse, StreamingResponse
 from fastapi.templating import Jinja2Templates
 
-# ====================================================================
-#                       FIX APPLIED HERE
-# ====================================================================
 # Use relative imports because these modules are in the same 'app' package.
 from .price_fetcher import PriceFetcher
 from .gemini_analyzer import GeminiAnalyzer
 from newsapi import NewsApiClient
-# ====================================================================
 
 
 # --- Application Lifespan for Resource Management ---
@@ -37,20 +33,17 @@ async def lifespan(app: FastAPI):
     lifespan context manager.
     """
     async with httpx.AsyncClient() as client:
-        # Instantiate and store all services in the application state.
         app.state.price_fetcher = PriceFetcher(client=client, coins=["bitcoin", "ethereum", "dogecoin"])
         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
         app.state.signal_queue: asyncio.Queue = asyncio.Queue()
 
-        # Create cancellable background tasks for periodic updates.
+        # Create cancellable background tasks. Let's use a shorter timer for testing.
         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=300)  # Check news every 5 minutes for debugging
         )
         
         print("🚀 Sentinel TradeFlow Protocol started successfully.")
@@ -72,42 +65,57 @@ async def run_periodic_updates(fetcher: PriceFetcher, interval_seconds: int):
         await asyncio.sleep(interval_seconds)
 
 async def run_periodic_news_analysis(app: FastAPI, interval_seconds: int):
-    """Fetches, analyzes, and queues top crypto news periodically."""
+    """Fetches, analyzes, and queues top crypto news periodically with detailed logging."""
     while True:
-        print("📰 Fetching latest crypto news for automated analysis...")
+        print("📰 [1/5] Fetching latest crypto news...")
         try:
             top_headlines = app.state.news_api.get_everything(
-                q='bitcoin OR ethereum OR crypto OR blockchain',
+                q='bitcoin OR ethereum OR "binance coin" OR solana OR ripple OR cardano',
                 language='en',
                 sort_by='publishedAt',
                 page_size=5
             )
+            
+            articles = top_headlines.get('articles', [])
+            print(f"📰 [2/5] NewsAPI call successful. Found {len(articles)} articles.")
+            
+            if not articles:
+                print("📰 [SKIP] No new articles found in this cycle.")
+                await asyncio.sleep(interval_seconds)
+                continue
+
             analyzer: GeminiAnalyzer = app.state.gemini_analyzer
-            for article in top_headlines.get('articles', []):
+            for article in articles:
                 title = article.get('title')
-                if title and "[Removed]" not in title:
-                    analysis = await analyzer.analyze_text(title)
-                    if analysis.get("error"):
-                        print(f"Skipping article due to Gemini error: {analysis['reason']}")
-                        continue
-                    
-                    analysis['url'] = article.get('url')
-                    await app.state.signal_queue.put(analysis)
-                    print(f"✅ Signal generated and queued for: {title}")
+                print(f"📰 [3/5] Processing article: '{title}'")
+
+                if not title or "[Removed]" in title:
+                    print(f"📰 [SKIP] Article has no title or was removed.")
+                    continue
+                
+                print(f"📰 [4/5] Sending to Gemini for analysis...")
+                analysis = await analyzer.analyze_text(title)
+                
+                if analysis.get("error"):
+                    print(f"❌ [SKIP] Gemini analysis failed for '{title}'. Reason: {analysis.get('reason')}")
+                    continue
+                
+                analysis['url'] = article.get('url')
+                await app.state.signal_queue.put(analysis)
+                print(f"✅ [5/5] Signal generated and queued for: '{title}'")
 
         except Exception as e:
-            print(f"❌ Error during news fetching or analysis: {e}")
+            print(f"❌❌❌ CRITICAL ERROR in news analysis loop: {e}")
             
+        print(f"📰 Loop finished. Waiting for {interval_seconds} seconds.")
         await asyncio.sleep(interval_seconds)
 
 # --- FastAPI App Initialization ---
 
 app = FastAPI(title="Sentinel TradeFlow Protocol", lifespan=lifespan)
-# This path is relative to the root where the uvicorn command is run
 templates = Jinja2Templates(directory="templates")
 
 # --- HTML Rendering Helper ---
-
 def render_signal_card(payload: dict) -> str:
     """Renders a dictionary of analysis into a styled HTML card."""
     s = payload
@@ -133,7 +141,6 @@ def render_signal_card(payload: dict) -> str:
     """
 
 # --- API Endpoints ---
-
 @app.get("/", response_class=HTMLResponse)
 async def serve_dashboard(request: Request):
     """Serves the main interactive dashboard from `index.html`."""