hermes - 💡(How to fix) Fix [Bug] create_session 静默失败 + 消息重复写入 — 会话从 state.db 消失 [1 participants]

try: session_id = await state.create_session(...) except Exception as e: logger.warning(f"Failed to create session: {e}") # ← 静默失败，没有补救 return # session_id 未设置，后续 append_message 跳过

TL;DR

Fix the silent failure in create_session() by adding retry logic and error escalation, and update _last_flushed_db_idx after successful message writes to prevent duplicate message writes.

Guidance

• For Bug 1, modify the create_session() exception handling to include retry logic and escalate the error from logger.warning to logger.error to make the issue more visible.
• For Bug 2, update the _last_flushed_db_idx cursor after successful message writes to prevent re-writing all messages and ensure accurate message counts.
• Verify the fixes by checking the session counts in state.db and the message counts in the JSON files, and ensure that session_search returns all expected sessions.
• To mitigate the issue temporarily, consider reducing the concurrency of Hermes instances or implementing a queue to handle session creations and message writes.

Example

try:
    session_id = await state.create_session(...)
except Exception as e:
    logger.error(f"Failed to create session: {e}")  # Escalate error
    # Add retry logic here, e.g., retry after a short delay
    # ...

Notes

The provided guidance assumes that the issue is caused by the described bugs and that the fixes will resolve the problems. However, additional testing and verification are necessary to ensure that the fixes work as expected and do not introduce new issues.

Recommendation

Apply the suggested workarounds, specifically adding retry logic and error escalation to create_session() and updating _last_flushed_db_idx after successful message writes, to address the identified bugs and prevent session loss and duplicate message writes.