Refactor: binary-sv2::serde-sv2 crate

[rrybarczyk]

Background

This task is an outcome of the protocols Rust docs issues tracked in #845.

While documenting protocols::v2::binary-sv2::serde-sv2 in #1011, areas of potential code debt were identified. This issue servers as a place to list out these items to then be addressed in an organized manner. The initial Rust documentation effort was an immediate action, while a refactoring (which implies breaking API changes) is not so urgent priority, so for now we should leave this in the backlog for an appropriate moment in the future.

Identified Potential Code Debt

TODO

[jbesraa]

IIUC serde-sv2 and non-serde-sv2 are both an implementation of the binary-sv2 crate. IMO it makes sense to merge this issue with #1131, #1132 and #1133, leaving #1331 as the main issue.

[plebhash]

#845 (comment)