fix: preserve CLI session tool metadata

[Michaelyklam]

Thinking Path

• CLI sessions now appear in the WebUI sidebar, but issue bug(cli-sessions): tool calls and tool results are not rendered when opening a CLI session — get_cli_session_messages() drops the tool_calls / tool_call_id / tool_name columns #1772 showed that opening one loses the tool-heavy parts of the transcript.
• The server loader was reading only role, content, and timestamp from state.db.messages, while the agent stores tool calls/results in adjacent columns.
• The frontend already knows how to render assistant tool_calls and role='tool' results when that metadata reaches the browser.
• The narrow fix is to preserve that metadata server-side, then refresh old imported sidecars that have the same row count but stripped tool metadata.

What Changed

• Extended api.models.get_cli_session_messages() to read and pass through CLI tool metadata:
  • assistant tool_calls
  • tool-result tool_call_id, tool_name, and name
  • reasoning fields where present
• Kept the loader defensive against older/partial state.db schemas by checking available columns before selecting optional fields.
• Updated the existing CLI import refresh path so previously imported sessions with stripped metadata are rebuilt even when the refreshed transcript has the same number of rows.
• Added regression tests with an isolated SQLite state.db fixture covering both fresh loads and same-length metadata refreshes.
• Added API-level evidence JSON for the tool-call/result shape preserved for rendering.

Fixes #1772

Why It Matters

CLI sessions are often short and tool-heavy. Dropping tool-call metadata makes those transcripts look nearly empty in the WebUI, even though the underlying Hermes state has the full action/result history. This restores parity with WebUI-native tool rendering without changing the frontend rendering model.

Verification

• python -m pytest tests/test_cli_session_tool_metadata.py -q initially failed before the fix with missing tool_calls / no same-length refresh.
• /home/michael/.hermes/hermes-agent/venv/bin/python -m pytest tests/test_cli_session_tool_metadata.py tests/test_session_import_cli_fallback_model.py -q → 10 passed
• /home/michael/.hermes/hermes-agent/venv/bin/python -m pytest tests/test_cli_session_tool_metadata.py tests/test_session_import_cli_fallback_model.py tests/test_cron_session_title.py -q → 15 passed
• git diff --check → passed
• env -u HERMES_CONFIG_PATH -u HERMES_WEBUI_HOST /home/michael/.hermes/hermes-agent/venv/bin/python -m pytest tests/ -q → 4689 passed, 4 skipped, 3 xpassed, 1 warning, 8 subtests passed in 378.92s
• Raw evidence URL check → HTTP 200, 567 bytes
• GitHub Actions on PR fix: preserve CLI session tool metadata #1778 → test (3.11), test (3.12), and test (3.13) all passed

Evidence / UI media

API-level evidence for the server payload that the existing WebUI tool-card renderer consumes:

https://raw.githubusercontent.com/Michaelyklam/hermes-webui/fix/issue-1772-cli-tool-calls/docs/pr-media/1772/cli-tool-metadata-api-evidence.json

This is a server/session-loader fix, not a visual layout change, so no before/after screenshot is included.

Risks / Follow-ups

• The SQL query now probes messages columns before selecting optional metadata, so older state DBs should continue to degrade safely.
• Existing imports with same-length stripped metadata are refreshed only when the fresh transcript adds CLI tool metadata and otherwise matches after stripping timing/tool metadata.
• No live service restart was performed.

Model Used

• Provider: OpenAI Codex
• Model: gpt-5.5
• AI-assisted implementation and verification using Hermes CLI tools, pytest, git, and GitHub CLI.

[nesquena-hermes]

Thanks @Michaelyklam — this shipped in v0.51.16 (commit 697a7a1) as part of a 3-PR full-sweep batch. Stage rebased your branch onto current master, ran the full pre-release gate (4695 pytest, browser tests, Opus advisor verdict), and merged via release PR #1781.

GitHub didn't auto-close because the merge commit only references the squash-merged stage branch, not your fork's commit directly — closing manually for hygiene.

Live now on existing installs after git pull + restart.

Release notes: https://github.com/nesquena/hermes-webui/releases/tag/v0.51.16