klbr/khoj
1
0
Fork 0
mirror of https://github.com/khoaliber/khoj.git synced 2026-03-08 05:39:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix the to_notify decider automation chat actor. Add detailed logging

Browse Source
This commit is contained in:
Debanjum
2024-12-20 13:27:29 -08:00
parent 03b4667acb
commit dcc5073d16
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/khoj/routers/helpers.py
View File

@@ -1651,12 +1651,15 @@ def should_notify(original_query: str, executed_query: str, ai_response: str, us
with timer("Chat actor: Decide to notify user of automation response", logger): with timer("Chat actor: Decide to notify user of automation response", logger):
try: try:
# TODO Replace with async call so we don't have to maintain a sync version # TODO Replace with async call so we don't have to maintain a sync version
response = send_message_to_model_wrapper_sync(to_notify_or_not, user) response = send_message_to_model_wrapper_sync(to_notify_or_not, user=user)
should_notify_result = "no" not in response.lower() should_notify_result = "no" not in response.lower()
logger.info(f'Decided to {"not " if not should_notify_result else ""}notify user of automation response.') logger.info(f'Decided to {"not " if not should_notify_result else ""}notify user of automation response.')
return should_notify_result return should_notify_result
except: except Exception as e:
logger.warning(f"Fallback to notify user of automation response as failed to infer should notify or not.") logger.warning(
f"Fallback to notify user of automation response as failed to infer should notify or not. {e}",
exc_info=True,
)
return True return True