From bd79c92753e04d539c12a73f1a39422e50c6ab9f Mon Sep 17 00:00:00 2001 From: Vellum Assistant Date: Wed, 25 Feb 2026 14:06:31 -0500 Subject: [PATCH] fix: skip full video download for lazy attachment thumbnails Co-Authored-By: Claude --- .../MediaEmbeds/InlineVideoAttachmentView.swift | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/clients/macos/vellum-assistant/Features/Chat/MediaEmbeds/InlineVideoAttachmentView.swift b/clients/macos/vellum-assistant/Features/Chat/MediaEmbeds/InlineVideoAttachmentView.swift index ba823ff612f..075a87217fd 100644 --- a/clients/macos/vellum-assistant/Features/Chat/MediaEmbeds/InlineVideoAttachmentView.swift +++ b/clients/macos/vellum-assistant/Features/Chat/MediaEmbeds/InlineVideoAttachmentView.swift @@ -176,19 +176,11 @@ struct InlineVideoAttachmentView: View { let url = safeTempURL() do { try data.write(to: url) } catch { return } fileURL = url - } else if attachment.isLazyLoad, - let port = daemonHttpPort, - let attachmentId = attachment.id.isEmpty ? nil : attachment.id { - // File-backed attachment: fetch from content endpoint. - let url = safeTempURL() - do { - let bytes = try await fetchAttachmentContent(port: port, attachmentId: attachmentId) - try bytes.write(to: url) - } catch { - return - } - fileURL = url } else { + // For lazy (file-backed) attachments, don't download the full video + // just for a thumbnail — large recordings (100MB+) cause unnecessary + // network traffic and memory pressure. The view already shows a + // play-button placeholder when thumbnailImage is nil. return }