Recommendation: call the attachments "session-attachment-2023-04-16-0123456-1.jpeg" .. "session-attachment-2023-04-16-0123456-<n>.jpeg"

types/Attachment.ts
getSuggestedFilename already has an index parameter for this, which is passed along from the calling function save.
but this parameter is not used in the calling function.
util/attachmentsUtil.ts
saveAttachmentToDisk.

saveAttachmentToDisk should have a new parameter index, which should be "" at
components/conversation/message/message-content/MessageAttachment.tsx
components/lightbox/LightboxGallery.tsx changes to:

    void saveAttachmentToDisk({ ...mediaItem, conversationId: selectedConversation,  
        index: props.selectedIndex });

components/conversation/message/message-content/MessageContextMenu.tsx
changes to:

        index: targetAttachmentIndex,

I'll try to get a patch for this.

Work in progress
RubenGarcia#1

If somebody else wants to tackle this, please do. This is my first patch here, so I will take a while to get used to the project.

@slrslr with this the first attachment keeps the name, but following attachments will have 001,002, ... added to them so the names are now unique.

I also have a branch which adds support to download all attachments in a message:
https://github.com/RubenGarcia/session-desktop/tree/DownloadAll
Would you be interested in that functionality?

Any new update regarding this change?

The patch needs to be approved by a developer, and I have not received any input yet.
Feel free to use the patch when you compile session.

Hey, thanks for the PR!
I made some comments to that PR, let me know how it goes.

Also, have a look at #2928 when you have time.