Skip to content

Comments

fix: recognize native tool_result format in TUI history loader#484

Merged
bug-ops merged 2 commits intomainfrom
feat/m26.1/fix-tool-result-history-display
Feb 17, 2026
Merged

fix: recognize native tool_result format in TUI history loader#484
bug-ops merged 2 commits intomainfrom
feat/m26.1/fix-tool-result-history-display

Conversation

@bug-ops
Copy link
Owner

@bug-ops bug-ops commented Feb 17, 2026

Summary

  • TUI history loader now recognizes [tool_result: id] format used by native tool_use providers (Claude, OpenAI)
  • Previously these messages were displayed as User role instead of Tool role on session restore
  • Added test for the new format

Test plan

  • Existing load_history tests pass
  • New test load_history_recognizes_tool_result_format added
  • Manual: restart TUI session with history containing native tool calls, verify they render as Tool messages

Messages from native tool_use providers (Claude, OpenAI) are persisted
as [tool_result: id] instead of [tool output: name]. The TUI history
parser now handles this format so restored messages render as Tool
role instead of User.
@github-actions github-actions bot added rust bug Something isn't working size/S labels Feb 17, 2026
@bug-ops bug-ops enabled auto-merge (squash) February 17, 2026 21:29
@bug-ops bug-ops merged commit 3e18471 into main Feb 17, 2026
18 checks passed
@bug-ops bug-ops deleted the feat/m26.1/fix-tool-result-history-display branch February 17, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working rust size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant