Update app.py

app.py

@@ -5,15 +5,27 @@ import base64
 from PIL import Image
 import io
 import datetime
+import smtplib
+from email.mime.text import MIMEText
+from email.mime.multipart import MIMEMultipart
 
 # Configuration from environment variables
 OPENROUTER_API_KEY = os.getenv("OPENROUTER_API_KEY")
 DEFAULT_REPORT_EMAIL = os.getenv("REPORT_EMAIL")
-BREVO_API_KEY = os.getenv("BREVO_API_KEY")
+
+# Email configuration - SMTP
+BREVO_SMTP_SERVER = os.getenv("BREVO_SMTP_SERVER")
+BREVO_SMTP_PORT = int(os.getenv("BREVO_SMTP_PORT", "587"))
+BREVO_SMTP_USERNAME = os.getenv("BREVO_SMTP_USERNAME")
+BREVO_SMTP_PASSWORD = os.getenv("BREVO_SMTP_PASSWORD")
 BREVO_FROM_EMAIL = os.getenv("BREVO_FROM_EMAIL", "noreply@yourdomain.com")
 
+# Email configuration - API (fallback)
+BREVO_API_KEY = os.getenv("BREVO_API_KEY")
+
 # Testing/Demo Mode Configuration
-DEMO_MODE = os.getenv("DEMO_MODE", "true").lower() == "true"  # Enable demo mode by default
+# Disable demo mode if SMTP or API is configured
+DEMO_MODE = os.getenv("DEMO_MODE", "false").lower() == "true" if (BREVO_SMTP_SERVER or BREVO_API_KEY) else True
 
 # Store conversation history and settings
 conversation_history = []
@@ -23,17 +35,17 @@ current_report_email = DEFAULT_REPORT_EMAIL
 departments = {
     "administration": {
         "name": "Administration",
-        "email": os.getenv("ADMIN_EMAIL", "nahdi.hassene34+admin@gmail.com"),
+        "email": os.getenv("ADMIN_EMAIL", "admin@demo.com"),
         "responsibilities": "Overall management, decision making, and coordination"
     },
     "production": {
         "name": "Production", 
-        "email": os.getenv("PRODUCTION_EMAIL", "nahdi.hassene34+production@gmail.com"),
+        "email": os.getenv("PRODUCTION_EMAIL", "production@demo.com"),
         "responsibilities": "Manufacturing, quality control, and production planning"
     },
     "magazine": {
         "name": "Magazine Management",
-        "email": os.getenv("MAGAZINE_EMAIL", "nahdi.hassene34+magazine@demo.com"),
+        "email": os.getenv("MAGAZINE_EMAIL", "magazine@demo.com"),
         "responsibilities": "Content creation, publishing, and distribution"
     }
 }
@@ -47,12 +59,38 @@ def update_report_email(new_email):
     else:
         return "❌ Please enter a valid email address"
 
+def send_email_smtp(to_email, subject, html_content, from_name="AI Coordination System"):
+    """Send email using SMTP"""
+    try:
+        if not all([BREVO_SMTP_SERVER, BREVO_SMTP_USERNAME, BREVO_SMTP_PASSWORD]):
+            return "❌ SMTP not configured - missing server, username, or password"
+        
+        # Create message
+        msg = MIMEMultipart('alternative')
+        msg['Subject'] = subject
+        msg['From'] = f"{from_name} <{BREVO_FROM_EMAIL}>"
+        msg['To'] = to_email
+        
+        # Attach HTML content
+        html_part = MIMEText(html_content, 'html')
+        msg.attach(html_part)
+        
+        # Send email
+        with smtplib.SMTP(BREVO_SMTP_SERVER, BREVO_SMTP_PORT) as server:
+            server.starttls()
+            server.login(BREVO_SMTP_USERNAME, BREVO_SMTP_PASSWORD)
+            server.send_message(msg)
+        
+        return f"✅ Email sent to {to_email} (via SMTP)"
+        
+    except Exception as e:
+        return f"❌ SMTP error: {str(e)}"
+
 def send_email_brevo_api(to_email, subject, html_content, from_name="AI Coordination System"):
     """Send email using Brevo API"""
     try:
-        # Demo mode: simulate email sending
-        if DEMO_MODE or not BREVO_API_KEY:
-            return f"🧪 [DEMO MODE] Email simulated to {to_email}"
+        if not BREVO_API_KEY:
+            return "❌ Brevo API not configured"
         
         response = requests.post(
             "https://api.brevo.com/v3/smtp/email",
@@ -74,13 +112,33 @@ def send_email_brevo_api(to_email, subject, html_content, from_name="AI Coordina
         )
         
         if response.status_code == 201:
-            return f"✅ Email sent to {to_email}"
+            return f"✅ Email sent to {to_email} (via API)"
         else:
             error_msg = response.json().get('message', 'Unknown error')
             return f"❌ Failed to send email: {error_msg}"
             
     except Exception as e:
-        return f"❌ Email error: {str(e)}"
+        return f"❌ Email API error: {str(e)}"
+
+def send_email(to_email, subject, html_content, from_name="AI Coordination System"):
+    """Send email using available method (SMTP or API)"""
+    # Demo mode: simulate email sending
+    if DEMO_MODE:
+        return f"🧪 [DEMO MODE] Email simulated to {to_email}"
+    
+    # Try SMTP first (preferred)
+    if BREVO_SMTP_SERVER:
+        result = send_email_smtp(to_email, subject, html_content, from_name)
+        # If SMTP fails, try API as fallback
+        if "❌" in result and BREVO_API_KEY:
+            return send_email_brevo_api(to_email, subject, html_content, from_name)
+        return result
+    
+    # Fall back to API if SMTP not configured
+    if BREVO_API_KEY:
+        return send_email_brevo_api(to_email, subject, html_content, from_name)
+    
+    return "❌ No email service configured (need SMTP or API credentials)"
 
 def get_mock_ai_response(user_message, department, user_name, has_image=False):
     """Generate a mock AI response for demo/testing mode"""
@@ -473,9 +531,9 @@ def send_department_email(sender_dept, recipient_dept, subject, message, user_na
 </body>
 </html>"""
         
-        # Send email using Brevo API
+        # Send email using SMTP or API
         from_name = f"{departments[sender_dept]['name']}"
-        email_status = send_email_brevo_api(recipient_email, f"[{urgency}] {subject}", html_content, from_name)
+        email_status = send_email(recipient_email, f"[{urgency}] {subject}", html_content, from_name)
         return email_status
     
     except Exception as e:
@@ -484,11 +542,14 @@ def send_department_email(sender_dept, recipient_dept, subject, message, user_na
 def send_email_report(conversation_data):
     """Send comprehensive coordination report to administration"""
     # Demo mode check
-    if DEMO_MODE or not BREVO_API_KEY or not current_report_email:
-        if DEMO_MODE:
-            return "🧪 [DEMO MODE] Report email simulated successfully"
-        else:
-            return "Email reporting not configured - missing Brevo API key or report email"
+    if DEMO_MODE:
+        return "🧪 [DEMO MODE] Report email simulated successfully"
+    
+    if not current_report_email:
+        return "❌ Report email not configured - please set report email address"
+    
+    if not (BREVO_SMTP_SERVER or BREVO_API_KEY):
+        return "❌ Email service not configured - need SMTP or API credentials"
     
     try:
         # Calculate department activity
@@ -590,8 +651,8 @@ def send_email_report(conversation_data):
 </body>
 </html>"""
         
-        # Send email using Brevo API
-        email_status = send_email_brevo_api(current_report_email, subject, html_content, "AI Coordination System")
+        # Send email using SMTP or API
+        email_status = send_email(current_report_email, subject, html_content, "AI Coordination System")
         return email_status
     
     except Exception as e:
@@ -768,7 +829,15 @@ def get_system_status():
     status_lines.append("**System Configuration:**")
     status_lines.append(f"• Demo Mode: {'🧪 ENABLED' if DEMO_MODE else '✅ DISABLED'}")
     status_lines.append(f"• OpenRouter API: {'✅ Configured' if OPENROUTER_API_KEY else '❌ Not configured (using demo)'}")
-    status_lines.append(f"• Brevo Email API: {'✅ Configured' if BREVO_API_KEY else '❌ Not configured (simulated)'}")
+    
+    # Email configuration status
+    if BREVO_SMTP_SERVER:
+        status_lines.append(f"• Email (SMTP): ✅ Configured ({BREVO_SMTP_SERVER}:{BREVO_SMTP_PORT})")
+    elif BREVO_API_KEY:
+        status_lines.append(f"• Email (API): ✅ Configured")
+    else:
+        status_lines.append(f"• Email: ❌ Not configured (simulated)")
+    
     status_lines.append(f"• Report Email: {current_report_email if current_report_email else '❌ Not set'}")
     return "\n".join(status_lines)