From 854ef6d5ba21245862a2e3d816e6fe6e2130b194 Mon Sep 17 00:00:00 2001 From: Your Name Date: Tue, 17 Mar 2026 17:40:38 +0530 Subject: [PATCH 1/3] Fix: Hide sensitive configuration errors from UI --- backend/agents.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/backend/agents.py b/backend/agents.py index ecba861..901a525 100644 --- a/backend/agents.py +++ b/backend/agents.py @@ -578,4 +578,5 @@ async def handle_chat(self, session_id: str, query: str, reset: bool = False) -> print(f"Error in handle_chat: {e}") import traceback traceback.print_exc() - return f"Error: {e}" + return "I'm sorry, I encountered an internal server error and cannot process your request right now." + From 04e342fafbb8ca4b22a556431183165207bd020c Mon Sep 17 00:00:00 2001 From: Your Name Date: Fri, 20 Mar 2026 15:19:42 +0530 Subject: [PATCH 2/3] Enhance: Replace generic prints with logger.exception() as suggested in review --- backend/agents.py | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/backend/agents.py b/backend/agents.py index 901a525..70fb109 100644 --- a/backend/agents.py +++ b/backend/agents.py @@ -10,6 +10,15 @@ from ks_search_tool import general_search, general_search_async, global_fuzzy_keyword_search from retrieval import get_retriever +import logging + +# Configure logger +logger = logging.getLogger(__name__) +if not logger.handlers: + _h = logging.StreamHandler() + _h.setFormatter(logging.Formatter("%(asctime)s [%(levelname)s] %(name)s: %(message)s")) + logger.addHandler(_h) + logger.setLevel(logging.INFO) # LLM (Gemini) client setup try: @@ -574,9 +583,7 @@ async def handle_chat(self, session_id: str, query: str, reset: bool = False) -> if len(self.chat_history[session_id]) > 20: self.chat_history[session_id] = self.chat_history[session_id][-20:] return response_text - except Exception as e: - print(f"Error in handle_chat: {e}") - import traceback - traceback.print_exc() + except Exception: + logger.exception("Error in handle_chat") return "I'm sorry, I encountered an internal server error and cannot process your request right now." From 1c8b1e5c870cb096947fc40d9e225b375c6e3a95 Mon Sep 17 00:00:00 2001 From: Your Name Date: Sat, 21 Mar 2026 19:46:57 +0530 Subject: [PATCH 3/3] Resolve agent logging conflict manually --- backend/agents.py | 8 +++----- backend/package-lock.json | 6 ++++++ 2 files changed, 9 insertions(+), 5 deletions(-) create mode 100644 backend/package-lock.json diff --git a/backend/agents.py b/backend/agents.py index 4fa1a47..10788c9 100644 --- a/backend/agents.py +++ b/backend/agents.py @@ -585,8 +585,6 @@ async def handle_chat(self, session_id: str, query: str, reset: bool = False) -> if len(self.chat_history[session_id]) > 20: self.chat_history[session_id] = self.chat_history[session_id][-20:] return response_text - except Exception as e: - print(f"Error in handle_chat: {e}") - import traceback - traceback.print_exc() - return f"Error: {e}" + except Exception: + logger.exception("Error in handle_chat") + return "I'm sorry, I encountered an internal server error and cannot process your request right now." diff --git a/backend/package-lock.json b/backend/package-lock.json new file mode 100644 index 0000000..dfb18f1 --- /dev/null +++ b/backend/package-lock.json @@ -0,0 +1,6 @@ +{ + "name": "backend", + "lockfileVersion": 3, + "requires": true, + "packages": {} +}