fix dockerfile and bot

Dockerfile

@@ -1,6 +1,8 @@
 FROM python:3.12-slim
 
-RUN apt-get clean && rm -rf /var/lib/apt/lists/*
+# Install curl for health check
+RUN apt-get update && apt-get install -y curl && \
+    apt-get clean && rm -rf /var/lib/apt/lists/*
 
 # Set up a new user named "user" with user ID 1000
 RUN useradd -m -u 1000 user
@@ -30,7 +32,7 @@ RUN pip install --no-cache-dir --upgrade pip
 COPY --chown=user requirements.txt $HOME/app/
 
 # Install Python dependencies in user space
-RUN pip install --no-cache-dir -r requirements.txt
+RUN pip install --no-cache-dir --upgrade -r requirements.txt
 
 # Copy source code with proper ownership
 COPY --chown=user src/ $HOME/app/src/
@@ -49,9 +51,10 @@ EXPOSE 7860
 
 # Health check
 HEALTHCHECK --interval=30s --timeout=10s --start-period=60s --retries=3 \
-    CMD curl --fail http://localhost:7860/ || exit 1
+    CMD curl --fail http://localhost:7860/health || exit 1
 
 # Run the application when the container starts HF
-CMD ["uvicorn", "src.telegram_bot:app", "--host", "0.0.0.0", "--port", "7860"]
+#CMD ["uvicorn", "src.telegram_bot:app", "--host", "0.0.0.0", "--port", "7860"]
+CMD ["python", "telegram_bot.py"]
 # Uncomment the line below to run the application locally
 #CMD ["python", "-m", "telegram_bot"]

src/telegram_bot.py

@@ -10,23 +10,23 @@ from fastapi import FastAPI, Request
 import uvicorn
 import httpx
 
+
 logging.basicConfig(level=logging.INFO)
 logger = logging.getLogger(__name__)
 
 load_dotenv()
 
-# Fix 1: Use consistent token name (BOT_TOKEN is standard for HF Spaces)
+
 BOT_TOKEN = os.getenv("BOT_TOKEN") or os.getenv("TELEGRAM_TOKEN")
 if not BOT_TOKEN:
     logger.error("BOT_TOKEN not found! Please add it in Space Settings > Repository secrets")
     raise ValueError("BOT_TOKEN not found in environment variables")
 
-# Fix 2: Correct SPACE_ID format
 SPACE_ID = os.environ.get('SPACE_ID', 'researchengineering-news_sentiment_analyzer')
 PORT = int(os.environ.get('PORT', 7860))
 
-# Fix 3: Create application GLOBALLY so webhook can access it
-application = Application.builder().token(BOT_TOKEN).build()
+
+application = None
 
 
 def format_news_for_telegram(news_json: list[dict[str, Any]]) -> str:
@@ -80,15 +80,38 @@ async def run_crew(update: Update, context: ContextTypes.DEFAULT_TYPE):
         await update.message.reply_text(f"Sorry, there was an error fetching news: {str(e)}")
 
 
-# Add handlers to the global application
-application.add_handler(CommandHandler("start", start))
-application.add_handler(CommandHandler("help", help_command))
-application.add_handler(CommandHandler("run", run_crew))
+# Initialize the Telegram application
+async def init_telegram_app():
+    global application
+    application = Application.builder().token(BOT_TOKEN).build()
+
+    # Add handlers
+    application.add_handler(CommandHandler("start", start))
+    application.add_handler(CommandHandler("help", help_command))
+    application.add_handler(CommandHandler("run", run_crew))
+
+    # Initialize the application
+    await application.initialize()
+    logger.info("Telegram application initialized successfully")
 
 # Create FastAPI app
 app = FastAPI(title="Financial News Bot", description="Telegram bot for financial news")
 
 
+@app.on_event("startup")
+async def startup_event():
+    """Initialize Telegram bot on FastAPI startup"""
+    await init_telegram_app()
+
+
+@app.on_event("shutdown")
+async def shutdown_event():
+    """Cleanup on shutdown"""
+    global application
+    if application:
+        await application.shutdown()
+
+
 @app.post(f"/{BOT_TOKEN}")
 async def webhook(request: Request):
     """Handle incoming webhook from Telegram"""
@@ -113,7 +136,8 @@ async def root():
         "status": "Financial News Bot is running!",
         "webhook_url": f"https://{SPACE_ID}.hf.space/{BOT_TOKEN[:10]}...",
         "space_id": SPACE_ID,
-        "available_endpoints": ["/", "/set_webhook", "/webhook_info"]
+        "available_endpoints": ["/", "/set_webhook", "/webhook_info", "/health"],
+        "bot_initialized": application is not None
     }