fix(honcho): strip whitespace from conclusion and delete_id inputs

Models may send whitespace-only strings like {"conclusion": " "} which
pass bool() but create meaningless conclusions. Strip both inputs so
whitespace-only values are treated as empty.

Adds tests for whitespace-only conclusion and delete_id.

Reviewed-by: @erosika
This commit is contained in:
kshitijk4poor 2026-04-16 22:19:54 +05:30 committed by kshitij
parent 4377d7da0d
commit fe3e68f572
2 changed files with 26 additions and 2 deletions

View file

@ -1008,8 +1008,8 @@ class HonchoMemoryProvider(MemoryProvider):
return json.dumps({"result": "\n\n".join(parts) or "No context available."})
elif tool_name == "honcho_conclude":
delete_id = args.get("delete_id")
conclusion = args.get("conclusion", "")
delete_id = (args.get("delete_id") or "").strip()
conclusion = args.get("conclusion", "").strip()
peer = args.get("peer", "user")
has_delete_id = bool(delete_id)