Update app.py

app.py

@@ -1,22 +1,3 @@
-import os
-from openai import OpenAI
-from datetime import datetime
-import gradio as gr
-import time
-import openai # Redundant import
-
-# --- Constants ---
-# Default model might need to align with what client.responses.create supports
-DEFAULT_MODEL = "gpt-4.1" # As per your example
-MAX_HISTORY_LENGTH = 5 # History formatting will be manual and limited
-
-# --- API Key and Client Initialization ---
-API_KEY = os.getenv("OPENAI_API_KEY")
-if not API_KEY:
-    print("Error: OPENAI_API_KEY environment variable not set.")
-    # Handle missing key appropriately (e.g., disable UI, raise error)
-client = OpenAI(api_key=API_KEY)
-
 # --- Helper Functions ---
 
 # !!! WARNING: This function is adapted to the requested format and LOSES features !!!
@@ -26,65 +7,92 @@ def get_openai_response_simplified(prompt, model=DEFAULT_MODEL, system_prompt=""
     NOTE: This is NON-STREAMING and handles history/system prompt crudely.
     Advanced parameters (temp, top_p etc.) are NOT supported by this structure.
     """
-    today_day = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
+    print("--- Entering get_openai_response_simplified ---") # DEBUG
+    print(f"Received prompt: {prompt}") # DEBUG
+    print(f"Received model: {model}") # DEBUG
+    print(f"Received history (first few): {chat_history[:2] if chat_history else 'None'}") # DEBUG
 
-    # --- Attempt to manually format history and system prompt into 'input' ---
+    today_day = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
     formatted_input = ""
-    # Add system prompt if provided
     effective_system_prompt = f"Today's date is: {today_day}. {system_prompt}".strip()
     if effective_system_prompt:
-         # How best to include system prompt? Prepend? Specific tags? Unknown.
          formatted_input += f"System: {effective_system_prompt}\n\n"
-
-    # Add chat history (simple concatenation)
     if chat_history:
         for turn in chat_history:
              if len(turn) == 2 and turn[0] is not None and turn[1] is not None:
                  formatted_input += f"User: {turn[0]}\nAssistant: {turn[1]}\n"
+    formatted_input += f"User: {prompt}\nAssistant:"
 
-    # Add the current user prompt
-    formatted_input += f"User: {prompt}\nAssistant:" # Prompt the model for the next turn
+    print(f"Formatted Input for API: \n{formatted_input}\n---") # DEBUG
 
     try:
-        # *** Using the requested client.responses.create format ***
-        # NOTE: This assumes client.responses.create actually exists and works this way.
-        # NOTE: Parameters like temperature, top_p, max_tokens are NOT included here
-        #       as they are not part of the provided example format.
+        print("Attempting client.responses.create call...") # DEBUG
+        # Check if method seems to exist before calling (basic check)
+        if not (hasattr(client, 'responses') and hasattr(client.responses, 'create')):
+             print("ERROR: client.responses.create method NOT FOUND!") # DEBUG
+             return "Error: The API call structure 'client.responses.create' is not available in the OpenAI client."
+
         response = client.responses.create(
             model=model,
             input=formatted_input
         )
-        # Assuming the response object has an 'output_text' attribute
-        return response.output_text
-
-    # Error handling might need adjustment based on how client.responses.create fails
+        print(f"API call supposedly succeeded. Response object: {response}") # DEBUG
+
+        # Check if response has the expected attribute BEFORE accessing it
+        if hasattr(response, 'output_text'):
+            output = response.output_text
+            print(f"Extracted output_text: {output}") # DEBUG
+            print("--- Exiting get_openai_response_simplified (Success) ---") # DEBUG
+            return output
+        else:
+            print("ERROR: Response object does NOT have 'output_text' attribute.") # DEBUG
+            print("--- Exiting get_openai_response_simplified (Error) ---") # DEBUG
+            return "Error: API response format is unexpected (missing output_text)."
+
+    # Keep specific error handling
     except openai.APIConnectionError as e:
-        print(f"OpenAI API request failed: {e}")
+        print(f"ERROR caught in get_openai_response_simplified: APIConnectionError: {e}") # DEBUG
         return f"Error: Could not connect to OpenAI API. {e}"
     except openai.RateLimitError as e:
-        print(f"OpenAI API request failed: {e}")
+        print(f"ERROR caught in get_openai_response_simplified: RateLimitError: {e}") # DEBUG
         return f"Error: Rate limit exceeded. Please try again later. {e}"
     except openai.AuthenticationError as e:
-        print(f"OpenAI API request failed: {e}")
+        print(f"ERROR caught in get_openai_response_simplified: AuthenticationError: {e}") # DEBUG
         return f"Error: Authentication failed. Check your API key. {e}"
     except openai.APIStatusError as e:
-        print(f"OpenAI API request failed: {e}")
+        print(f"ERROR caught in get_openai_response_simplified: APIStatusError: {e}") # DEBUG
         return f"Error: OpenAI API returned an error (Status: {e.status_code}). {e}"
     except AttributeError as e:
-        print(f"Error accessing response or client method: {e}")
-        return f"Error: The API call structure 'client.responses.create' or its response format might be incorrect or not available. {e}"
+        # This might catch the error if the initial check fails or response has wrong structure
+        print(f"ERROR caught in get_openai_response_simplified: AttributeError: {e}") # DEBUG
+        return f"Error: Problem with API call structure or response format. {e}"
     except Exception as e:
-        print(f"An unexpected error occurred: {e}")
+        print(f"ERROR caught in get_openai_response_simplified: Unexpected Exception: {e}") # DEBUG
         return f"An unexpected error occurred: {e}"
 
+
 # !!! WARNING: This update function is now NON-STREAMING !!!
 def update_ui_simplified(message, chat_history, model, system_prompt, history_length):
     """Updates the Gradio UI WITHOUT streaming."""
+    print("\n--- Entering update_ui_simplified ---") # DEBUG
+    print(f"Received message: {message}") # DEBUG
+    print(f"Current chat_history length: {len(chat_history)}") # DEBUG
+    print(f"Requested history_length: {history_length}") # DEBUG
+
     if not message:
+        print("Empty message received, returning.") # DEBUG
         return "", chat_history # Return original history if message is empty
 
+    # Ensure history_length is an int for slicing
+    try:
+        hist_len_int = int(history_length)
+    except ValueError:
+        print(f"Warning: Invalid history_length '{history_length}', defaulting to {MAX_HISTORY_LENGTH}")
+        hist_len_int = MAX_HISTORY_LENGTH
+
     # Keep only the specified length of history for the *next* call
-    history_for_api = chat_history[-int(history_length):] if history_length > 0 else []
+    history_for_api = chat_history[-hist_len_int:] if hist_len_int > 0 else []
+    print(f"History slice for API length: {len(history_for_api)}") # DEBUG
 
     # Call the simplified, non-streaming function
     bot_response = get_openai_response_simplified(
@@ -93,105 +101,55 @@ def update_ui_simplified(message, chat_history, model, system_prompt, history_le
         system_prompt=system_prompt,
         chat_history=history_for_api # Pass the potentially trimmed history
     )
+    print(f"Received from API helper: {bot_response}") # DEBUG
 
     # Append the user message and the *complete* bot response
+    # Make sure chat_history is treated as a list
+    if chat_history is None:
+       chat_history = []
     chat_history.append((message, bot_response))
+    print(f"Appended to chat_history. New length: {len(chat_history)}") # DEBUG
 
     # Update UI only once with the full response
     # Always display history based on the slider length for visibility
-    visible_history = chat_history[-int(history_length):] if history_length > 0 else []
+    visible_history = chat_history[-hist_len_int:] if hist_len_int > 0 else []
+    print(f"Visible history length: {len(visible_history)}") # DEBUG
+    print("--- Exiting update_ui_simplified ---") # DEBUG
     return "", visible_history # Clear input, return updated history
 
-# --- Gradio Interface (Modified for Simplified API Call) ---
-with gr.Blocks(theme=gr.themes.Soft()) as demo:
-    # Keep your Markdown, titles, etc.
-    gr.Markdown("# Chat (Simplified API Demo)")
-    gr.Markdown("---")
-    gr.Markdown("Using a simplified, non-streaming API call structure.")
-    gr.Markdown("---")
-
-    # ... (rest of your Markdown) ...
-
-    gr.Markdown("Chat below (Note: Responses will appear all at once): 👇")
-
-    with gr.Row():
-        with gr.Column(scale=4):
-            chatbot = gr.Chatbot(
-                label="Chat Window",
-                show_label=False,
-                avatar_images=(
-                    "https://cdn-icons-png.flaticon.com/512/1077/1077114.png", # User
-                    "https://cdn-icons-png.flaticon.com/512/8649/8649540.png" # AI
-                ),
-                render_markdown=True,
-                height=500,
-                bubble_full_width=False
-            )
-            msg = gr.Textbox(
-                label="Your Message",
-                placeholder="Type your message here and press Enter...",
-                scale=4,
-                show_label=False,
-                container=False
-            )
-
-            # Accordion remains, but parameters might not be used by the simplified API call
-            with gr.Accordion("Advanced Options (May Not Apply to Simplified API)", open=False):
-                model_select = gr.Dropdown(
-                    label="Model",
-                    # Ensure gpt-4.1 is a valid choice if used
-                    choices=["gpt-4.1", "gpt-4o-mini-2024-07-18", "gpt-3.5-turbo-0125", "gpt-4o"],
-                    value=DEFAULT_MODEL,
-                    interactive=True
-                )
-                # These sliders are kept for UI, but won't be passed to the simplified API call
-                temperature_slider = gr.Slider(label="Temperature (Not Used)", minimum=0.0, maximum=2.0, value=1.0, step=0.1, interactive=True)
-                top_p_slider = gr.Slider(label="Top P (Not Used)", minimum=0.0, maximum=1.0, value=1.0, step=0.05, interactive=True)
-                frequency_penalty_slider = gr.Slider(label="Frequency Penalty (Not Used)", minimum=-2.0, maximum=2.0, value=0.0, step=0.1, interactive=True)
-                presence_penalty_slider = gr.Slider(label="Presence Penalty (Not Used)", minimum=-2.0, maximum=2.0, value=0.0, step=0.1, interactive=True)
-                system_prompt_textbox = gr.Textbox(label="System Prompt", placeholder="e.g., You are a helpful assistant.", lines=3, interactive=True)
-                history_length_slider = gr.Slider(label="Chat History Length (Affects Input & Display)", minimum=1, maximum=20, value=MAX_HISTORY_LENGTH, step=1, interactive=True)
-
-
-            with gr.Row():
-                 clear = gr.Button("Clear Chat")
-                 send = gr.Button("Send Message", variant="primary")
-
-
-    # --- Event Handlers (Using Simplified Functions) ---
-    # Define inputs, excluding sliders not used by the simplified function
+# --- Gradio Interface (Ensure Simplified Handlers are Used) ---
+# ... (Keep the Gradio Blocks definition as in the previous example) ...
+
+# --- Event Handlers (Check these carefully) ---
     inputs_simplified = [
         msg, chatbot, model_select, system_prompt_textbox,
-        history_length_slider
+        history_length_slider # Make sure history_length_slider is included
     ]
-    outputs = [msg, chatbot] # Outputs remain the same
+    outputs = [msg, chatbot]
 
-    # Connect send button click
     send.click(
-        update_ui_simplified, # Use the non-streaming UI update function
-        inputs=inputs_simplified,
+        update_ui_simplified, # Ensure this is the correct function
+        inputs=inputs_simplified, # Ensure this list matches the function args
         outputs=outputs,
         queue=True
     )
-
-    # Connect textbox submit (Enter key)
     msg.submit(
-        update_ui_simplified, # Use the non-streaming UI update function
-        inputs=inputs_simplified,
+        update_ui_simplified, # Ensure this is the correct function
+        inputs=inputs_simplified, # Ensure this list matches the function args
         outputs=outputs,
         queue=True
     )
-
-    # Connect clear button
     clear.click(lambda: (None, []), None, outputs=[msg, chatbot], queue=False)
 
-    gr.Examples(
-        examples=["Tell me about the latest AI developments", "Write a short story about a friendly robot", "Explain black holes simply"],
-        inputs=msg,
-        label="Example Prompts"
-    )
+# ... (Rest of the Gradio code and launch) ...
 
-# --- Launch ---
 if __name__ == "__main__":
-    demo.queue()
-    demo.launch()
+    # Make sure API key is loaded (add check if needed)
+    if not API_KEY:
+        print("FATAL ERROR: OPENAI_API_KEY is not set. The application cannot function.")
+        # Optionally, you could display this error in the Gradio UI as well
+        # and prevent the launch or disable input components.
+    else:
+        print("OpenAI API Key found.")
+        demo.queue()
+        demo.launch()