Update app.py

app.py

@@ -234,21 +234,26 @@ def process_message(
     global client
     
     if client is None:
-        return history + [(message, "❌ Please configure your API key first in the Settings tab.")], "", ""
+        return history + [(message if message else "No message", "❌ Please configure your API key first in the Settings tab.")], "", "❌ API key not configured"
     
     if not message.strip() and not files:
-        return history + [(message, "⚠️ Please enter a message or upload files.")], "", ""
+        return history + [("", "⚠️ Please enter a message or upload files.")], "", "⚠️ No input provided"
     
     status_messages = []
     
     try:
+        # Ensure history is a list
+        if history is None:
+            history = []
+        
         # Build messages array
         messages = []
         
         # Add conversation history
         for user_msg, assistant_msg in history:
-            messages.append({"role": "user", "content": user_msg})
-            if assistant_msg:
+            if user_msg:  # Only add if user message exists
+                messages.append({"role": "user", "content": user_msg if user_msg else "..."})
+            if assistant_msg:  # Only add if assistant message exists
                 messages.append({"role": "assistant", "content": assistant_msg})
         
         # Build current message content
@@ -261,18 +266,23 @@ def process_message(
             
             for file in files:
                 if file is not None:
-                    file_blocks, status = process_file(file)
-                    content.extend(file_blocks)
-                    status_messages.append(status)
-                    file_count += 1
-                    
-                    # Count pages for PDFs
-                    if status.startswith("✅") and "page(s)" in status:
-                        try:
-                            pages = int(status.split("converted to ")[1].split(" page(s)")[0])
-                            total_pages += pages
-                        except:
-                            pass
+                    try:
+                        file_blocks, status = process_file(file)
+                        if file_blocks:  # Only add if we got valid blocks
+                            content.extend(file_blocks)
+                            status_messages.append(status)
+                            file_count += 1
+                            
+                            # Count pages for PDFs
+                            if status and status.startswith("✅") and "page(s)" in status:
+                                try:
+                                    pages = int(status.split("converted to ")[1].split(" page(s)")[0])
+                                    total_pages += pages
+                                except:
+                                    pass
+                    except Exception as file_error:
+                        error_msg = f"❌ Error processing file: {str(file_error)}"
+                        status_messages.append(error_msg)
             
             if file_count > 0:
                 file_summary = f"📎 {file_count} file(s) uploaded"
@@ -281,9 +291,13 @@ def process_message(
                 content.insert(0, {"type": "text", "text": file_summary})
         
         # Add text message
-        if message.strip():
+        if message and message.strip():
             content.append({"type": "text", "text": message})
         
+        # If no content at all, return error
+        if not content:
+            return history + [(message if message else "", "❌ No valid content to process")], "", "❌ No valid content"
+        
         messages.append({"role": "user", "content": content})
         
         # Prepare API call parameters
@@ -298,20 +312,45 @@ def process_message(
         if enable_reasoning:
             api_params["extra_body"] = {"reasoning": {"enabled": True}}
         
-        # Make API call
-        response = client.chat.completions.create(**api_params)
+        # Make API call with additional error handling
+        try:
+            response = client.chat.completions.create(**api_params)
+        except Exception as api_error:
+            error_msg = f"❌ API Error: {str(api_error)}"
+            return history + [(message if message else "", error_msg)], "", error_msg
+        
+        # Check if response is valid
+        if not response:
+            error_message = "❌ Error: Received None response from API"
+            return history + [(message if message else "", error_message)], "", error_message
         
-        assistant_message = response.choices[0].message.content
+        if not hasattr(response, 'choices') or not response.choices or len(response.choices) == 0:
+            error_message = "❌ Error: Received empty response from API"
+            return history + [(message if message else "", error_message)], "", error_message
+        
+        # Get the assistant message safely
+        try:
+            assistant_message = response.choices[0].message.content
+            if not assistant_message:
+                assistant_message = "⚠️ Model returned an empty response"
+        except (AttributeError, IndexError) as e:
+            assistant_message = f"❌ Error extracting response: {str(e)}"
         
         # Extract reasoning if available
         reasoning_text = ""
-        if enable_reasoning and hasattr(response.choices[0].message, 'reasoning_details'):
-            reasoning_details = response.choices[0].message.reasoning_details
-            if reasoning_details:
-                reasoning_text = f"**🧠 Reasoning Process:**\n{json.dumps(reasoning_details, indent=2)}"
+        if enable_reasoning:
+            try:
+                if hasattr(response.choices[0].message, 'reasoning_details'):
+                    reasoning_details = response.choices[0].message.reasoning_details
+                    if reasoning_details:
+                        reasoning_text = f"**🧠 Reasoning Process:**\n{json.dumps(reasoning_details, indent=2)}"
+            except Exception as reasoning_error:
+                # Reasoning extraction failed, but that's okay
+                pass
         
         # Update history
-        new_history = history + [(message, assistant_message)]
+        display_message = message if message and message.strip() else "[Files uploaded]"
+        new_history = history + [(display_message, assistant_message)]
         
         # Combine status messages
         combined_status = "\n".join(status_messages) if status_messages else "✅ Message processed successfully"
@@ -319,8 +358,13 @@ def process_message(
         return new_history, reasoning_text, combined_status
         
     except Exception as e:
-        error_message = f"❌ Error: {str(e)}"
-        return history + [(message, error_message)], "", error_message
+        error_message = f"❌ Error: {str(e)}\n\nType: {type(e).__name__}"
+        import traceback
+        error_detail = traceback.format_exc()
+        print(f"Full error: {error_detail}")  # For debugging
+        
+        display_message = message if message and message.strip() else "[Error occurred]"
+        return history + [(display_message, error_message)], "", error_message
 
 def clear_conversation():
     """Clear conversation history"""