-
xmr-prTyLauriente opened pull request #4674: wizard: add Trezor Safe 7 support
-
xmr-pr
-
br-m<jpk68:matrix.org> selsta: I agree with you that the LLM-written C++ would be unmaintainable, however I do think it would be less ideal to have it in Rust
-
selstawhy?
-
br-m<jpk68:matrix.org> The reason is that Trezor's Rust library is a no-std reference impl which only covers the framing and Noise handshake
-
br-m<jpk68:matrix.org> It has nothing to do with Protobuf, USB, Bluetooth, etc.
-
br-m<jpk68:matrix.org> All of the crypto stuff we need to implement this in C++ is already in libsodium
-
br-m<jpk68:matrix.org> I'm pretty sure it can be done with only ~2,000 new lines of C++, not accounting for new Protobuf definitions and integration into the existing device state machine thing
-
selstacan you add a comment to the PR? maybe he can slop a prototype with lobsodium
-
br-m<jpk68:matrix.org> Trezor's Rust crate would also have to be vendored or added as a submodule; the former would require constant updates and the latter would increase the time needed to clone the repo by 3x