Improving the official APIs? about algonaut

[ivanschuetz]

A partly "Rust specific" example:

LogicSignature is modeled after the algod REST API:

pub struct LogicSignature {
    #[serde(rename = "l")]
    pub logic: Vec<u8>,

    pub sig: Option<Signature>,
    pub msig: Option<MultisigSignature>,

    #[serde(rename = "arg")]
    pub args: Vec<Vec<u8>>,
}

this struct is used in the domain too, which isn't ideal. As far I understand having both sig and msig set is illegal (it at least fails the verification). A sum type would represent this better:

enum Sig {
    Sig(Signature),
    Msig(MultisigSignature),
}

New domain type:

pub struct LogicSignature {
    pub logic: Vec<u8>,
    pub sig: Option<Sig>,
    pub args: Vec<Vec<u8>>,
}

Now we can't have the illegal state. It could be expanded further, to not need an optional:

enum Sig {
    ContractAccount
    Sig(Signature),
    Msig(MultisigSignature),
}

pub struct LogicSignature {
    pub logic: Vec<u8>,
    pub sig: Sig,
    pub args: Vec<Vec<u8>>,
}

And now we can remove already almost half of the Java SDK's verification method and related Unit Tests

The problem with these type of changes is that e.g. Java doesn't have sum types, so they can't be proposed there. @manuelmauro thoughts?

[ivanschuetz]

Cleared internally:

• The SDK will prioritize idiomatic Rust.
• For refactorings not related with above, we will submit issues to the Go SDK, when it's possible to keep the APIs similar.