The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 4, 2026 7:37pm

PR Summary

Medium Risk
Moderate UI/runtime risk: changes how preview URLs are derived across draft restore, optimistic messages, and display mapping, and introduces inline <video> rendering which could affect performance or MIME handling.

Overview
Fixes Mothership attachment preview restoration by deriving previewUrl from the persisted attachment key via a shared getMothershipAttachmentPreviewUrl helper (used in display-message, use-chat optimistic messages, draft restore, and queued-message restore).

Adds video preview support: video files now get preview URLs and render as a <video preload="metadata"> thumbnail (both in the input attachment grid and in sent message attachments), while non-previewable files continue to fall back to the document pill/icon UI.

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

Greptile Summary

This PR fixes missing attachment previews on Mothership draft restore and adds video preview support (input chip + sent message) by extracting a shared getMothershipAttachmentPreviewUrl helper and wiring it into draft-restore, queued-message-restore, optimistic user messages, and display-message — eliminating four copies of the old image-only inline pattern. Video rendering uses <video preload="metadata"> with a document-icon fallback layer behind it for environments where the first frame doesn't paint automatically.

Confidence Score: 5/5

Safe to merge — the fix is well-scoped and the only finding is a P2 accessibility nit on the video element in chat-message-attachments.

All findings are P2 (missing aria-label on the <video> in chat-message-attachments). The core bug fix and deduplication are correct, and the new video rendering path mirrors the accepted approach already applied to attached-files-list.

apps/sim/app/workspace/[workspaceId]/home/components/chat-message-attachments.tsx — video element missing accessible name.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[File attached / Draft restored] --> B{Has key?}
    B -- Yes --> C[getMothershipAttachmentPreviewUrl]
    B -- No uploading --> D[URL.createObjectURL blob URL]
    C --> E{media_type?}
    D --> E
    E -- image/ --> F[previewUrl set]
    E -- video/ --> F
    E -- other --> G[previewUrl = undefined]
    F --> H{Render location}
    G --> I[FileAttachmentPill / icon chip]
    H -- attached-files-list input chip --> J{isVideo?}
    H -- chat-message-attachments sent msg --> K{isVideo?}
    J -- Yes --> L[icon fallback + video element]
    J -- No --> M[img element]
    K -- Yes --> N[icon fallback + video element]
    K -- No --> O[img element]

_{Reviews (2): Last reviewed commit: "fix(mothership): icon fallback behind vi..." | Re-trigger Greptile}

@greptile

@cursor review