fix: flush_memories history destruction, OpenClaw overwrite, set dedup, chat_id guard

[aaronlab]

Summary

• run_agent.py (HIGH): Fix a latent bug in flush_memories where messages.pop() removes the sentinel before the finally block runs. When memory_tool_def is None (no memory tool registered), the finally while-loop fails to find the sentinel and keeps popping until the entire messages list is empty — destroying all conversation history. Fix: let the finally block handle cleanup as designed.
• hermes_cli/setup.py: The comment on line 2624 says "no overwrite" but code passes overwrite=True. This silently overwrites existing Hermes configuration with stale OpenClaw data when setup wizard is run again. Fix: align code with comment (overwrite=False).
• hermes_cli/model_switch.py: Remove duplicate "opencode-go" entry from set literal on line 665 (copy-paste error).
• tools/send_message_tool.py: Wrap int(chat_id) and int(thread_id) in try/except to prevent unhandled ValueError when Telegram channel name resolution returns non-numeric values (e.g., "@channelname" format).

Test plan

• Verify flush_memories with no memory tool registered does not corrupt messages list
• Verify OpenClaw migration preserves existing Hermes config files
• Verify switch_model with all three OpenCode provider variants
• Verify _send_telegram with non-numeric chat_id returns clean error

🤖 Generated with Claude Code

[teknium1]

Thanks for the thorough multi-part fix, @aaronlab — closing as redundant. All four changes are dead or already-fixed on current main:

1. flush_memories history destruction — the entire flush_memories function was removed in PR refactor(memory): remove flush_memories entirely #15696 (merged Apr 25). The code path your fix targeted no longer exists.
2. hermes_cli/setup.py overwrite=True — main already passes overwrite=False (with the explanatory comment your fix wanted): the OpenClaw migration was hardened in PR feat(claw-migrate): harden OpenClaw import with plan-first apply, redaction, and pre-migration backup #16911.
3. hermes_cli/model_switch.py duplicate "opencode-go" — the set literal is already {"opencode-zen", "opencode-go", "opencode"} with no duplicate.
4. tools/send_message_tool.py int(chat_id) guard — the bare int(chat_id) was replaced with normalize_telegram_chat_id(), which handles the exact @username case you flagged even better than a try/except: it passes the username through to the Bot API (which accepts it) instead of returning an error, so the send still succeeds.

Good catches on real issues — they just all got resolved independently in the weeks since you opened this. Appreciate the contribution.