feat: expose AttachmentQueue's AttachmentContext#985
Conversation
🦋 Changeset detectedLatest commit: 0bedefe The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
simolus3
left a comment
There was a problem hiding this comment.
We need to run pnpm exec api-extractor run --local to update the API snapshot. Also AttachmentService should be marked as @alpha instead of @internal now since this makes it part of the public API.
It's nice to see the API Extractor in action! After properly examining the current state of the various components. I updated this PR to not expose |
Some
AttachmentQueueusers have a use case for customsaveFilemethods. TheattachmentServiceexposes a context to mutateattachmentrecords in a safe manner. Unfortunately, theattachmentServiceis currently marked asprivatewhich means extending classes cannot access it.The
AttachmentsServicecontains functionality to watch active attachments and obtain anAttachmentContextvia a mutex. Users won't likely need to watch the attachment state, but they could need anAttachmentContext- which exposes methods for reading and manipulatingAttachmentRecords.This makes
AttachmentContexta public API and adds a small method toAttachmentQueueto access anAttachmentContext.An example use case for requiring access is available at #983