feat(client): render viewed images in work logs

[maria-rcks]

Expanded image-read work-log rows currently show only file paths, and persisted T3 attachment paths fail when treated as workspace files. This keeps the shared work-log selector, resolves each viewed image to either a workspace-file or attachment asset, reuses the signed asset renderers on web and mobile, and removes duplicate web grouping code.

Evidence

Before: an expanded Read 1 file row falls back to Image unavailable for a persisted T3 attachment.

After: the isolated T3 preview resolved the same path shape as an attachment asset, returned the SVG with HTTP 200, and rendered it without the unavailable fallback. The T3 snapshot and recording endpoints timed out, so the after screenshot is still unverified.

Checks

• vp run --filter @t3tools/client-runtime typecheck
• vp run --filter @t3tools/web typecheck
• vp run --filter @t3tools/mobile typecheck
• targeted Vite+ formatting and lint
• bundled runtime assertions for persisted attachment paths and workspace paths with query/fragment suffixes
• focused Vite+ test collection is currently blocked before collection by TypeError: Cannot read properties of undefined (reading 'config')

Built with gpt-5.6-sol in T3 Code.

---

Note

Medium Risk
Changes asset resolution and signed URL loading for work-log previews across web and mobile; incorrect path classification could still show unavailable images but does not touch auth or data persistence.

Overview
Expanded read work-log rows can now show an inline image preview instead of only a path string. Shared logic in client-runtime picks eligible image paths from entry detail (workEntryViewedImagePath) and resolves them to signed attachment or workspace-file assets (resolveViewedImageAsset), including persisted T3 attachment paths that previously broke when treated as workspace files.

Web and mobile reuse the same asset image components (ChatMarkdownAssetImage / generalized ThreadMarkdownImage) inside expanded work-log detail, with URL fragments preserved for markdown-style sources. Web MessagesTimeline.logic drops duplicated tool-group summarization and re-exports it from work-log/presentation; dynamic “Read file” tool rows are matched case-insensitively.

^{Reviewed by Cursor Bugbot for commit 22b6107. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Render viewed images in work-log entries on mobile and web

• Generalizes ThreadMarkdownImage (mobile) and ChatMarkdownAssetImage (web) to accept either an attachment or a workspace-file resource, and to append optional srcFragment (hash/query) to the resolved asset URL.
• Adds workEntryViewedImagePath and resolveViewedImageAsset in presentation.ts to detect when a work-log entry reflects an image read and to classify the source into an AssetResource.
• Expanded work-log rows on both platforms now render the resolved image above the detail text via the pluggable renderImage (mobile) or ChatMarkdownAssetImage (web) callbacks.
• Replaces local tool/action normalization utilities in MessagesTimeline.logic.ts with re-exports from @t3tools/client-runtime/work-log/presentation.
• Fixes toolGroupAction to classify dynamic tool calls named "Read File" (case-insensitive, whitespace-trimmed) as a read action.
• Risk: ThreadMarkdownImage props changed from threadId/path to a resource prop; any out-of-tree callers of the old signature will break.

^{Macroscope summarized 22b6107.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 28bccd1d-df94-4fa3-a05c-fe123e8e3334

📥 Commits

Reviewing files that changed from the base of the PR and between 932970e and ddf9de1.

📒 Files selected for processing (2)

• apps/mobile/src/features/threads/ThreadFeed.tsx
• apps/web/src/components/chat/MessagesTimeline.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

The change adds shared detection for viewed workspace image paths in work-log entries. Web and mobile thread views render eligible images above expanded work-log details. Web image rendering is exported for timeline reuse.

Changes

Workspace image previews

Layer / File(s)	Summary
Work-log image path contract packages/client-runtime/src/work-log/presentation.ts, packages/client-runtime/src/work-log/presentation.test.ts, apps/web/src/components/chat/MessagesTimeline.logic.ts	The presentation layer detects trimmed, single-path image details for read entries. Dynamic read-file titles are matched case-insensitively. Timeline helpers now come from the shared presentation module.
Web work-log image preview apps/web/src/components/ChatMarkdown.tsx, apps/web/src/components/chat/MessagesTimeline.tsx	Expanded web work-log entries render eligible image previews with basename alt text, a 16rem maximum height, and image expansion handling.
Mobile work-log image preview apps/mobile/src/features/threads/thread-work-log.tsx, apps/mobile/src/features/threads/ThreadFeed.tsx	Expanded mobile work-log rows render eligible images above detail text through the forwarded workspace image renderer.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to ddf9d

The PR adds inline previews for qualifying viewed images in expanded work-log rows on web and mobile while reusing existing signed image rendering. No actionable merge-blocking risk remains beyond normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant ThreadView
  participant WorkLogPresentation
  participant WorkspaceImageRenderer
  ThreadView->>WorkLogPresentation: derive viewed image path from work entry
  WorkLogPresentation-->>ThreadView: return image path or null
  ThreadView->>WorkspaceImageRenderer: render eligible image preview
  WorkspaceImageRenderer-->>ThreadView: handle image expansion

Loading

Suggested reviewers: juliusmarminge, t3dotgg

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 11.11% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 9 functions across 7 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the main change: rendering viewed images in work logs.
Description check	✅ Passed	The description explains what changed, why it changed, UI evidence, validation steps, and known limitations. It is mostly complete, although it uses a Checks section instead of the template's Checklis…

Full details: Description check

Explanation

The description explains what changed, why it changed, UI evidence, validation steps, and known limitations. It is mostly complete, although it uses a Checks section instead of the template's Checklist heading.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 932970e. Configure here.}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR adds automatic inline rendering and previewing of viewed images in work logs across web and mobile, including new signed-asset resolution for persisted attachments. The cross-platform capability and shared work-log behavior changes extend beyond a small isolated fix and merit human review.

^{You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

</antml de="">

^{Posted via Macroscope — UI Consistency}