ouyangbo/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 11 Packages Projects Releases Wiki Activity

fix(honcho): persist one-sided turns under the empty-content guard

Browse Source 
The containment commit skipped the whole turn when either side was
empty, which would drop a real user message on interrupted or
tool-only turns. Keep the guard for fully-empty turns only and skip
empty sides individually inside the sync loop.
This commit is contained in:
Erosika
2026-08-10 18:32:29 -04:00
committed by kshitij
parent d610b238c6
commit 08b3312031
1 changed files with 12 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/memory/honcho/__init__.py
+12 -5
View File
@@ -1432,16 +1432,23 @@ class HonchoMemoryProvider(MemoryProvider):
msg_limit = self._config.message_max_chars if self._config else 25000
clean_user_content = sanitize_context(user_content or "").strip()
clean_assistant_content = sanitize_context(assistant_content or "").strip()
if not clean_user_content or not clean_assistant_content:
# Skip only when the whole turn is empty. An interrupted or tool-only
# turn can legitimately have an empty assistant side; the user's
# message must still be persisted (the manager already drops
# empty-user turns upstream). Empty sides are skipped per-loop below
# so we never write empty-string messages either.
if not clean_user_content and not clean_assistant_content:
return
def _sync():
try:
session = self._manager.get_or_create(self._session_key)
for chunk in self._chunk_message(clean_user_content, msg_limit):
session.add_message("user", chunk)
for chunk in self._chunk_message(clean_assistant_content, msg_limit):
session.add_message("assistant", chunk)
if clean_user_content:
for chunk in self._chunk_message(clean_user_content, msg_limit):
session.add_message("user", chunk)
if clean_assistant_content:
for chunk in self._chunk_message(clean_assistant_content, msg_limit):
session.add_message("assistant", chunk)
self._manager._flush_session(session)
except Exception as e:
logger.debug("Honcho sync_turn failed: %s", e)