Skip to content

transferables for encoder#589

Closed
cpeddecord wants to merge 1 commit intoremote-uifrom
transferables-for-encoder
Closed

transferables for encoder#589
cpeddecord wants to merge 1 commit intoremote-uifrom
transferables-for-encoder

Conversation

@cpeddecord
Copy link

implementing a subset of widely available transferable objects (MDN Link) for RPC's basic encoder.

@cpeddecord cpeddecord requested a review from lemonmade August 12, 2025 21:30
@github-actions
Copy link
Contributor

We detected some changes in packages/*/package.json or packages/*/src, and there are no updates in the .changeset directory.
If the changes are user-facing and should cause a version bump, run yarn changeset to track your changes and include them in the next release CHANGELOG.
If you are making simple updates to repo configuration, examples, or documentation, you do not need to add a changeset.

@cpeddecord cpeddecord requested a review from alxclark August 12, 2025 21:30
@lemonmade
Copy link
Member

I would not personally want this to be the default behavior, since it is a destructive operation that the host has no way to control. In my more modern RPC library, I added a dedicated library function that lets a consumer mark transferable properties: https://github.com/lemonmade/quilt/blob/main/packages/threads/source/transfer.ts#L9 (though I would use a WeakSet to keep track of these, if I were doing it again today).

@cpeddecord cpeddecord closed this Sep 24, 2025
@cpeddecord cpeddecord deleted the transferables-for-encoder branch September 24, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants