@powersync/attachments StorageAdapter method parameters are a bit limited

[hunterm-wbc]

As a user, when I'm implementing my own StorageAdapter, I find the parameters defined by the interface to be a bit limiting. Specifically, the uploadFile, downloadFile and deleteFile methods. In order to interface with my own backend, I need more information than just the filename or uri, I specifically need some of the information from the attachment record, including the additional columns I've defined.

For my own use case, I've resorted to stuffing any information I need to process the file to the filename of the record, which works - however this seems a bit hacky.

Is there any possibility of having this interface expanded to provide pass alone the attachment record?

[stevensJourney]

That's a fair point. We could, and should pass the entire attachment record to those methods. The more recent attachment helpers in other SDKs do this. We'll keep this issue open till the enhancements have been made :)