Looks like an excellent approach with maximum immutability and minimal exceptions. Let's try it.

Sweet! Thanks again, @markstos

@jeremy-daley-kr I found there was a competing PR submitted to handle the binary type mangling:

#762

That PR returns a the binary object as a Uint8Array, while your PR returns the binary object as a Buffer that contains Uint8Array's. This was reflected when I tried to merge both and got this test failure about the different types:

• secret: Buffer(6) [Uint8Array] [

• secret: Uint8Array(6) [

Returning Uint8Array(6) seems simpler and preferable. Could you review why this is returning a buffer instead?

@markstos I might be a bit confused what the expected behavior is then, because this is the repo that was linked from @Maia-Everett original issue (#761), and it uses Buffer.from to assert equality:
https://codeberg.org/Linneris/config-yaml-binary-bug/src/branch/main/src/index.js

@markstos Also, I just checked out v3.3.7 (before the Proxy stuff was added), and added the test snippet you linked to... It gives Buffer(6) [Uint8Array].

Since that's a new feature request/breaking change, can you consider publishing a new release in the meantime?

@markstos Friendly bump ^^

@jdmarshall has been added as a co-maintainer to help move things along. We are evaluating plans for the next release now. Thanks for your patience with our volunteer team.

@jdmarshall has been added as a co-maintainer to help move things along. We are evaluating plans for the next release now. Thanks for your patience with our volunteer team.

Thanks, @markstos !