Viper flags should use conts

[ninjamast3r]

To avoid getting flags out of sync if we ever change them, the viper names should use constants. So instead of:

cmd.Flags().String("foo", "", "it does stuff")
...
foo := viper.GetString("foo")

do this:

const fooFlag := "foo"
cmd.Flags().String(fooFlag , "", "it does stuff")
...
foo := viper.GetString(fooFlag )

Keep the flags related to whatever commands they reference. No need to reference variables from way across the project.

[caproven]

Cobra/viper related & could possibly be looked at when handling this issue:

I've noticed cli/cmd/root.go has 2 calls of viper.AutomaticEnv(). I haven't looked any further to see if one could be safely deleted, but imagine this is unintentional.