Nice job @anon78698497, some other comments (it’d be nice if we could comment in the PR, is there a way to enable that?):
- Perhaps we can adapt the
prompt_userhelper to accept an optional arg (e.g.Option<&str>) which specifies what to ask the user to enter, so we have a single helper function covering whatprompt_confirmationdoes too. - I see
let mut paths: Vec<String> = vec![];but we only work on a single path which is the~/.safe, so perhaps we don’t need it, or if we are gonna populate it with several paths then declare it up front asVec<PathBuf>?
As a general comment, shouldn’t we try to remove only the data but not the executables? as it stands it’s more like an uninstallation rather than a reset, I was assuming we wanted something which can clear all data stored by nodes, CLI credentials, but not the actual apps/executables ?