From ff3e4c5cfe6ce48f23e870c31fda1a353b079d78 Mon Sep 17 00:00:00 2001 From: Mudkip <518069+mudkipme@users.noreply.github.com> Date: Mon, 23 Feb 2026 20:09:39 +0800 Subject: [PATCH] fix: avoid truncating memo batch attachments (#5654) --- store/attachment.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/store/attachment.go b/store/attachment.go index f9897d08e..41f84abfa 100644 --- a/store/attachment.go +++ b/store/attachment.go @@ -80,11 +80,12 @@ func (s *Store) CreateAttachment(ctx context.Context, create *Attachment) (*Atta func (s *Store) ListAttachments(ctx context.Context, find *FindAttachment) ([]*Attachment, error) { // Set default limits to prevent loading too many attachments at once - if find.Limit == nil && find.GetBlob { + shouldApplyDefaultLimit := find.Limit == nil && len(find.MemoIDList) == 0 + if shouldApplyDefaultLimit && find.GetBlob { // When fetching blobs, we should be especially careful with limits defaultLimit := 10 find.Limit = &defaultLimit - } else if find.Limit == nil { + } else if shouldApplyDefaultLimit { // Even without blobs, let's default to a reasonable limit defaultLimit := 100 find.Limit = &defaultLimit