Skip to content

fix(tui): render local images reliably - #182

Merged
clabby merged 1 commit into
cl/image-cleanupfrom
cl/image-rendering
Aug 22, 2026
Merged

fix(tui): render local images reliably#182
clabby merged 1 commit into
cl/image-cleanupfrom
cl/image-rendering

Conversation

@clabby

@clabby clabby commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • render local Markdown images inline only when the terminal supports a native image protocol
  • degrade unsupported backends to ordinary clickable Markdown links
  • accept absolute scratchpad and other local paths outside the workspace
  • move metadata, decode, resize, and protocol preparation off the render thread
  • bound image work and caches while preserving independent placement sizes
  • hydrate restored transcripts without blocking first paint
  • lazily retransmit images after focus and resume without changing their dimensions
  • avoid redraw-time filesystem work and unnecessary image reconstruction

Testing

  • cargo check --all-features
  • just check-fmt
  • just clippy
  • just test (946 tests)

Stack

  1. feat(prompt): link local images with Markdown #179 — Markdown image-link prompting
  2. This PR
  3. chore(release): prepare 0.6.6 #180 — prepare 0.6.6

Depends on #179. No matching open issue exists to close.

@clabby clabby changed the title cl/image rendering fix(tui): render local images reliably Aug 22, 2026
@codspeed-hq

codspeed-hq Bot commented Aug 22, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 18 untouched benchmarks
⏩ 31 skipped benchmarks1


Comparing cl/image-rendering (ab9ca94) with cl/image-cleanup (23c2078)

Open in CodSpeed

Footnotes

  1. 31 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@clabby
clabby merged commit 92e731b into main Aug 22, 2026
16 checks passed
@clabby
clabby deleted the cl/image-rendering branch August 22, 2026 04:21
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.

1 participant