Update app.py

app.py

@@ -5,85 +5,116 @@ import openai
 # --------------- PRESETS ---------------
 
 PRESETS = {
-    "Executive Summary":
-        {
-            "system": (
-                "You are an executive briefing assistant. "
-                "Turn complex text into a concise 3–5 bullet executive summary "
-                "with clear insights and recommendations."
-            ),
-            "user": "Paste a long report, transcript, or document here and I'll summarize it."
-        },
-    "Polished Email Reply":
-        {
-            "system": (
-                "You are a professional communications assistant. "
-                "Write clear, concise, and friendly emails suitable for executives. "
-                "Preserve tone and key details."
-            ),
-            "user": "Paste the email you received and a few notes on how you want to respond."
-        },
-    "Meeting Notes → Action Plan":
-        {
-            "system": (
-                "You convert messy meeting notes into a clear action-oriented summary. "
-                "Always include: 1) Key decisions, 2) Action items with owners and timelines, "
-                "3) Risks or next steps."
-            ),
-            "user": "Paste your rough meeting notes or bullet points and I'll structure them."
-        },
-    "Idea Generator / Brainstorm":
-        {
-            "system": (
-                "You are a creative strategist. Generate structured ideas with titles, "
-                "descriptions, and next steps."
-            ),
-            "user": "Describe what you want to build, improve, or launch. I'll propose ideas."
-        },
+    "Executive Summary": {
+        "system": (
+            "You are an executive briefing assistant. "
+            "Turn complex text into a concise 3–5 bullet executive summary "
+            "with clear insights and recommendations."
+        ),
+        "user": "Paste a long report, transcript, or document here and I'll summarize it.",
+    },
+    "Polished Email Reply": {
+        "system": (
+            "You are a professional communications assistant. "
+            "Write clear, concise, and friendly emails suitable for executives. "
+            "Preserve tone and key details."
+        ),
+        "user": "Paste the email you received and a few notes on how you want to respond.",
+    },
+    "Meeting Notes → Action Plan": {
+        "system": (
+            "You convert messy meeting notes into a clear action-oriented summary. "
+            "Always include: 1) Key decisions, 2) Action items with owners and timelines, "
+            "3) Risks or next steps."
+        ),
+        "user": "Paste your rough meeting notes or bullet points and I'll structure them.",
+    },
+    "Idea Generator / Brainstorm": {
+        "system": (
+            "You are a creative strategist. Generate structured ideas with titles, "
+            "descriptions, and next steps."
+        ),
+        "user": "Describe what you want to build, improve, or launch. I'll propose ideas.",
+    },
 }
 
+
 # --------------- CORE CHAT LOGIC ---------------
 
-def run_completion(api_key, model, system_prompt, user_prompt, temperature, history):
+def run_completion(api_key, model, system_prompt, user_prompt, history):
+    """
+    Core call to the OpenAI Chat Completions API.
+
+    IMPORTANT:
+    - Does NOT raise errors for missing API key; returns a friendly message instead.
+    - Does NOT send temperature or max_tokens/max_completion_tokens, to avoid
+      model-specific 400 errors like you saw with GPT-5.
+    """
+
+    history = history or []
+
     if not api_key:
-        raise gr.Error("🔑 Please save your API key first.")
+        # No exception; just return a message in the output area
+        msg = "❌ No API key found. Paste your key in the API Key field and click **Save Key** first."
+        history.append(("System", msg))
+        return history, msg
 
     if not user_prompt.strip():
-        raise gr.Error("✏️ Add a prompt or select a preset before running.")
+        msg = "⚠️ Please enter a prompt or paste some content before running."
+        history.append(("System", msg))
+        return history, msg
+
+    # Configure OpenAI
+    openai.api_key = api_key.strip()
 
-    openai.api_key = api_key
+    # Build messages for the chat completion
     messages = []
     if system_prompt.strip():
         messages.append({"role": "system", "content": system_prompt.strip()})
     messages.append({"role": "user", "content": user_prompt.strip()})
 
-    # GPT-5 models use max_completion_tokens instead of max_tokens
-    response = openai.chat.completions.create(
-        model=model,
-        messages=messages,
-        temperature=float(temperature),
-        max_completion_tokens=1200,
-    )
+    try:
+        # Minimal, model-agnostic call: no temperature / max_* params
+        response = openai.chat.completions.create(
+            model=model.strip(),
+            messages=messages,
+        )
+        answer = (response.choices[0].message.content or "").strip()
+        if not answer:
+            answer = "⚠️ The model returned an empty response. Try again with more context."
+
+    except Exception as e:
+        answer = f"❌ Error while calling the model:\n\n`{e}`"
 
-    answer = response.choices[0].message.content.strip()
-    history = history or []
     history.append(("You", user_prompt))
     history.append(("Assistant", answer))
+
     return history, answer
 
 
 # --------------- HELPERS ---------------
 
-def save_api_key(raw_key):
-    cleaned = raw_key.strip()
+def save_api_key(raw_key: str):
+    """
+    Save the API key for this session.
+
+    We still allow non 'sk-' prefixes so people can drop in OpenRouter or other provider keys.
+    """
+    cleaned = (raw_key or "").strip()
     if not cleaned:
-        raise gr.Error("🔐 Please paste a valid API key.")
+        raise gr.Error("🔐 Please paste a valid API key before saving.")
+
     if not cleaned.startswith("sk-"):
         gr.Info("Key saved, but it doesn’t start with 'sk-'. Double-check your provider format.")
+
+    # Return key for State + clear textbox with a new placeholder
     return cleaned, gr.update(value="", placeholder="API key saved ✔")
 
 
-def load_preset(preset_name):
+def load_preset(preset_name: str):
+    """
+    Load system + user text for a selected preset.
+    """
     if not preset_name or preset_name not in PRESETS:
         return gr.update(), gr.update()
     preset = PRESETS[preset_name]
@@ -96,21 +127,21 @@ CUSTOM_CSS = """
 #zen-root {
     background: radial-gradient(circle at top left, #111827, #020617 65%);
     color: #f3f4f6;
-    font-family: 'Inter', system-ui, sans-serif;
+    font-family: 'Inter', system-ui, -apple-system, BlinkMacSystemFont, sans-serif;
     min-height: 100vh;
 }
 .zen-card {
     border-radius: 20px;
-    background: rgba(17, 24, 39, 0.6);
-    border: 1px solid rgba(255,255,255,0.08);
-    backdrop-filter: blur(16px);
-    box-shadow: 0 0 40px rgba(0,0,0,0.35);
-    padding: 1.75rem;
+    background: rgba(15, 23, 42, 0.82);
+    border: 1px solid rgba(148, 163, 184, 0.35);
+    backdrop-filter: blur(18px);
+    box-shadow: 0 24px 60px rgba(0, 0, 0, 0.55);
+    padding: 1.8rem;
 }
 .zen-pill {
     border-radius: 9999px;
-    padding: 0.3rem 0.9rem;
-    border: 1px solid rgba(255,255,255,0.15);
+    padding: 0.28rem 0.9rem;
+    border: 1px solid rgba(129, 140, 248, 0.65);
     color: #a5b4fc;
     font-size: 0.75rem;
     text-transform: uppercase;
@@ -119,18 +150,29 @@ CUSTOM_CSS = """
 .zen-title {
     font-size: 1.9rem;
     font-weight: 700;
-    letter-spacing: -0.02em;
+    letter-spacing: -0.025em;
+}
+.zen-subtitle {
+    font-size: 0.9rem;
+    color: #9ca3af;
 }
 """
 
-with gr.Blocks(css=CUSTOM_CSS, elem_id="zen-root", fill_height=True, title="ZEN Promptboard — GPT-5") as demo:
+with gr.Blocks(
+    css=CUSTOM_CSS,
+    elem_id="zen-root",
+    fill_height=True,
+    title="ZEN Promptboard — GPT-5",
+) as demo:
+
+    # Header
     gr.Markdown(
         """
 <div class="zen-card">
   <div class="zen-pill">ZEN VANGUARD • GPT-5 CONSOLE</div>
   <div class="zen-title" style="margin-top: 0.8rem;">Professional AI Promptboard</div>
-  <div style="margin-top: 0.5rem; color:#9ca3af;">
-    Transform unstructured content into polished professional outputs — with GPT-5 as your assistant.
+  <div class="zen-subtitle" style="margin-top: 0.5rem;">
+    Transform unstructured content into polished, ready-to-send executive artifacts.
   </div>
 </div>
         """
@@ -139,14 +181,17 @@ with gr.Blocks(css=CUSTOM_CSS, elem_id="zen-root", fill_height=True, title="ZEN
     api_key_state = gr.State("")
 
     with gr.Row(equal_height=True):
+        # LEFT COLUMN: API + model + presets
         with gr.Column(scale=1):
             with gr.Group():
-                gr.Markdown("### 🔐 Connect")
+                gr.Markdown("### 🔐 1. Connect")
+
                 api_key_input = gr.Textbox(
                     label="API Key",
-                    placeholder="Paste your GPT-5 key here (not stored on server)",
+                    placeholder="Paste your GPT-style key here (not stored on server)",
                     type="password",
                 )
+
                 with gr.Row():
                     save_btn = gr.Button("Save Key", variant="primary")
                     key_status = gr.Markdown("Key not saved yet.")
@@ -154,40 +199,49 @@ with gr.Blocks(css=CUSTOM_CSS, elem_id="zen-root", fill_height=True, title="ZEN
                 model_name = gr.Textbox(
                     label="Model ID",
                     value="gpt-5",
-                    info="Default: GPT-5. You may switch to gpt-4o or gpt-4.1-mini if needed.",
-                )
-                temperature = gr.Slider(
-                    0.0, 1.2, 0.5, 0.05, label="Creativity"
+                    info="Default: gpt-5. You can switch to gpt-4o, gpt-4.1-mini, etc. if needed.",
                 )
 
             with gr.Group():
-                gr.Markdown("### ⚡ Presets")
+                gr.Markdown("### ⚡ 2. Presets")
+
                 preset_radio = gr.Radio(
                     label="Quick workflows",
                     choices=list(PRESETS.keys()),
                     interactive=True,
                 )
 
+        # RIGHT COLUMN: prompts + output
         with gr.Column(scale=2):
             system_box = gr.Textbox(
                 label="System / Role Instructions",
-                placeholder="How should the AI behave?",
+                placeholder="How should the AI behave? (e.g., 'You are my executive assistant...')",
                 lines=4,
             )
+
             user_box = gr.Textbox(
                 label="Your Prompt or Notes",
-                placeholder="Paste text, meeting notes, or raw input...",
+                placeholder="Paste content, meeting notes, or a draft you want rewritten...",
                 lines=8,
             )
+
             run_btn = gr.Button("Run Prompt", variant="primary")
+
             output_md = gr.Markdown("Output will appear here.")
-            history_chat = gr.Chatbot(label="History", height=300)
 
-    # --- Wiring ---
+            history_chat = gr.Chatbot(
+                label="History",
+                height=300,
+                type="tuples",          # explicit to avoid the deprecation warning
+                show_copy_button=True,
+            )
+
+    # --------------- WIRES / EVENTS ---------------
 
-    def _save_and_ack(key):
+    def _save_and_ack(key: str):
         stored_key, placeholder_update = save_api_key(key)
-        return stored_key, "✅ API key stored for this session.", placeholder_update
+        status_msg = "✅ API key stored for this session."
+        return stored_key, status_msg, placeholder_update
 
     save_btn.click(
         _save_and_ack,
@@ -201,20 +255,18 @@ with gr.Blocks(css=CUSTOM_CSS, elem_id="zen-root", fill_height=True, title="ZEN
         outputs=[system_box, user_box],
     )
 
-    def _run(api_state, model, system, user, temp, history):
-        history, answer = run_completion(
+    def _run(api_state, model, system, user, history):
+        return run_completion(
             api_key=api_state,
             model=model,
             system_prompt=system,
             user_prompt=user,
-            temperature=temp,
             history=history,
         )
-        return history, answer
 
     run_btn.click(
         _run,
-        inputs=[api_key_state, model_name, system_box, user_box, temperature, history_chat],
+        inputs=[api_key_state, model_name, system_box, user_box, history_chat],
         outputs=[history_chat, output_md],
     )