Skip to content

fix: show live tool result images outside event groups#60

Merged
jmfederico merged 6 commits into
mainfrom
fix/pr-53-live-tool-result-images
Jul 14, 2026
Merged

fix: show live tool result images outside event groups#60
jmfederico merged 6 commits into
mainfrom
fix/pr-53-live-tool-result-images

Conversation

@jmfederico

Copy link
Copy Markdown
Owner

Summary

  • keep native tool-result images visible outside collapsed EVENTS groups for both hydrated history and live sessions
  • reconcile provisional tool.end images with authoritative message.end content without duplication or metadata loss
  • support text-plus-image and image-only results while retaining one technical execution representation
  • present visible images as clearly labeled standard chat cards using the assistant color scheme
  • keep late-loading images pinned only while the reader remains at the bottom

Credit

This builds on #53 and retains @kanazawahere's original commit, grouping fix, tests, and authorship in the branch history.

Validation

  • npm run verify: 167 test files, 1124 passed, 2 skipped
  • npm run build
  • npm run changelog:status: patch bump recognized
  • synthetic merge with current origin/main: 185 test files, 1335 passed, 2 skipped, production build passed
  • git diff --check

Supersedes #53.

@jmfederico jmfederico merged commit cc4443e into main Jul 14, 2026
2 checks passed
@jmfederico jmfederico deleted the fix/pr-53-live-tool-result-images branch July 14, 2026 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants