Reorg

[notmandatory]

No description provided.

[brh28]

Thoughts on using different constructors to build the request objects. Example here:

pub fn read(&mut self) -> Result<ReadResponse, Error> {
        let read_cmd = ReadCommand::for_satscard(self.card_nonce.clone());
        let read_response = self.transport.transmit_read(read_cmd);
        if let Ok(read_response) = &response {
            self.card_nonce = read_response.card_nonce.clone();
        }
        response
    }
    ```

[notmandatory]

Hey sorry for all the refactoring on top of your changes. But the process has been really helpful in finding a cleaner design. In the latest commit I added Authentication and SharedCommands traits which should consolidate most of the duplicated code and make it clear where new commands need to go. Shared commands that are called the same way for all the card types go in the SharedCommands trait, and the rest go in the specific impl for each card.

If this looks OK to you then I'll merge it. Adding the rest of the commands can go in new PRs.

Besides the remaining commands the other near-term TODOs I see are:

• verify all all encrypted and/or signed response data
• make an example CLI (using clap-rs) for easier manual testing

[brh28]

LGTM. Gives some structure which should make development easier