Update app.py

app.py

@@ -17,12 +17,11 @@ client = OpenAI(api_key=OPENAI_API_KEY, base_url=BASE_URL)
 def push(text):
     try:
         if not PUSHOVER_TOKEN or not PUSHOVER_USER:
-            print("Pushover not configured:", text)
             return
         requests.post(
             "https://api.pushover.net/1/messages.json",
             data={"token": PUSHOVER_TOKEN, "user": PUSHOVER_USER, "message": text},
-            timeout=10
+            timeout=5
         )
     except:
         pass
@@ -49,7 +48,7 @@ tools = [
                 "properties": {
                     "email": {"type": "string"},
                     "name": {"type": "string"},
-                    "notes": {"type": "string"},
+                    "notes": {"type": "string"}
                 },
                 "required": ["email"]
             }
@@ -59,7 +58,7 @@ tools = [
         "type": "function",
         "function": {
             "name": "record_unknown_question",
-            "description": "Record an unknown question.",
+            "description": "Record unknown question.",
             "parameters": {
                 "type": "object",
                 "properties": {"question": {"type": "string"}},
@@ -78,21 +77,21 @@ class Me:
         if os.path.exists("me/summary.txt"):
             self.summary = open("me/summary.txt", "r", encoding="utf-8").read()
 
-        pdf = "me/Ayush_linkdin.pdf"
-        if os.path.exists(pdf):
+        pdf_path = "me/Ayush_linkdin.pdf"
+        if os.path.exists(pdf_path):
             text = []
-            reader = PdfReader(pdf)
-            for p in reader.pages:
-                t = p.extract_text()
+            reader = PdfReader(pdf_path)
+            for page in reader.pages:
+                t = page.extract_text()
                 if t:
                     text.append(t)
             self.linkedin = "\n\n".join(text)
 
     def system_prompt(self):
         return f"""
-You are acting as {self.name}. Answer questions about his background, skills and experience.
-If unknown, use record_unknown_question.
-If user is interested, ask for email and call record_user_details.
+You are acting as {self.name}. You answer questions about his background, skills and experience.
+If unsure, use record_unknown_question.
+If user is interested, ask for an email and call record_user_details.
 
 Summary:
 {self.summary}
@@ -104,9 +103,8 @@ LinkedIn:
     def chat(self, message, history):
         messages = [{"role": "system", "content": self.system_prompt()}]
 
-        # Add history
-        for role, content in history:
-            messages.append({"role": role, "content": content})
+        for role, text in history:
+            messages.append({"role": role, "content": text})
 
         messages.append({"role": "user", "content": message})
 
@@ -116,16 +114,18 @@ LinkedIn:
                 messages=messages,
                 tools=tools,
                 tool_choice="auto",
-                max_tokens=600
+                max_tokens=400
             )
 
-            msg = response.choices[0].message
-            finish = response.choices[0].finish_reason
+            choice = response.choices[0]
+            msg = choice.message
+            finish = choice.finish_reason
 
             if finish == "tool_calls":
-                for tool_call in msg.tool_calls:
-                    func = tool_call.function
-                    result = globals()[func.name](**json.loads(func.arguments))
+                for call in msg.tool_calls:
+                    fname = call.function.name
+                    args = json.loads(call.function.arguments)
+                    result = globals()[fname](**args)
                     messages.append({"role": "tool", "content": json.dumps(result)})
                 continue
 
@@ -134,11 +134,11 @@ LinkedIn:
 
 me = Me()
 
-def respond(user_message, chat_history):
-    output = me.chat(user_message, chat_history)
-    chat_history.append(("user", user_message))
-    chat_history.append(("assistant", output))
-    return "", chat_history
+def respond(user_message, history):
+    bot_reply = me.chat(user_message, history)
+    history.append(("user", user_message))
+    history.append(("assistant", bot_reply))
+    return "", history
 
 
 with gr.Blocks(css="""
@@ -155,7 +155,7 @@ with gr.Blocks(css="""
         btn_projects = gr.Button("Latest Projects")
 
     chatbot = gr.Chatbot(height=450)
-    user_input = gr.Textbox(placeholder="Type here...")
+    user_input = gr.Textbox(placeholder="Type your message...")
 
     user_input.submit(respond, [user_input, chatbot], [user_input, chatbot])
 
@@ -168,6 +168,5 @@ with gr.Blocks(css="""
     btn_projects.click(lambda: "Show Ayush Tyagi’s latest projects.", None, user_input)
     btn_projects.click(respond, [user_input, chatbot], [user_input, chatbot])
 
-
 if __name__ == "__main__":
-    ui.launch(server_name="0.0.0.0", server_port=int(os.environ.get("PORT", 7860)))
+    ui.launch()