refactor: Remove browser polyfills - Githubissues

liftedinit / many-js

JavaScript implementation of the MANY protocol

Apache License 2.0

5 stars 7 forks source link

refactor: Remove browser polyfills #99

Closed stanleyjones closed 1 year ago

stanleyjones commented 1 year ago

This PR removes the need for browser polyfills in web apps that import the ManyJS library.

Specifically:

makeRandomBytes is somewhat intelligent and only imports Node's crypto library if it's not found in the browser
All instances of Node's Buffer class have been replaced with the universal Uint8Array
New helpers to replace Buffer.compare, Buffer.from, Buffer.toString (along with tests!)
Updated documentation and types to refer to CborData instead of buffers