This is fantastic, really a beautiful interface. And the readme in the repositories are easy to use and well written.
Apologies for the length but these are my genuine first impressions and there’s no second chance at that, so even if some of this later turns out to be misconceived I think it still has some value.
--test-coins maybe a message about how pk/sk might be used or what to do with it (if anything). Great to see those values being displayed but maybe some more info about why could be good. But also I appreciate the concise interface, so a bit of give and take in this.
--allow-all-auth maybe could have single-letter flag of
-y like many package managers have for assume yes. Maybe consider changing the long name too;
--noconfirm which to me is ambiguous. Seems like assume yes is fairly common terminology.
Would you consider allowing pk/sk in config file as an alternative to secret/password? This might make it easier to utilise other secret derivation schemes such as bip39 instead of the maidsafe derivation with secret/password.
./safe auth creates
~/.safe/credentials with permissions
664, I think
600 would be more appropriate.
Stepping back a bit, safe_auth is a good name but what is this package really doing from the user perspective? I ask because
safe has a command
auth so there may appear to be some mixing of concepts, especially to new users. I’d say maybe safe_auth is more of a control center or safety filter or bodyguard or protector or something like that. I feel that _auth suffix doesn’t have enough weight behind it. A little bikeshedding, but also a little way to remove mental load and help the intuitions of newcomers.
Is there a way to specify
safe_auth --daemon for a different port from 41805? I ran the daemon on 31805 and couldn’t seem to get
safe to talk to it.
safe cat and others use a table layout. Just wondering if this table format can be suppressed so the output can be piped, ie just a list of items like
ls? I think maybe the most expected default behaviour is no table, and use
--pretty to add layout.