fix(cli): login and logout output improvements

* Display errors using functions from `output`.
* Add `output.PrintlnOK` for printing affirmative messages.
* Use non-emoji symbols with ansi coloring.

cli/cmd/login/login.go

@@ -3,7 +3,6 @@ package login
 import (
 	"context"
 	"fmt"
-	"log"
 	"net/http"
 	"os"
 
@@ -22,15 +21,16 @@ var LoginCmd = &cobra.Command{
 		if user == nil {
 			Login(auth.NumerousTenantAuthenticator, cmd.Context())
 		} else {
-			fmt.Println("✅ Great, you are already logged in!")
+			output.PrintlnOK("Great, you are already logged in!")
 		}
 	},
 }
 
 func Login(a auth.Authenticator, ctx context.Context) *auth.User {
 	state, err := a.GetDeviceCode(ctx, http.DefaultClient)
 	if err != nil {
-		log.Fatal(err)
+		output.PrintErrorDetails("Error getting device code", err)
+		os.Exit(1)
 	}
 
 	fmt.Printf(`You are logging into Numerous.
@@ -46,7 +46,8 @@ Press Enter to continue...
 `, state.UserCode)
 
 	if _, err = fmt.Scanln(); err != nil {
-		log.Fatal(err)
+		output.PrintErrorDetails("Error getting keystroke", err)
+		os.Exit(1)
 	}
 
 	if err := a.OpenURL(state.VerificationURI); err != nil {
@@ -58,17 +59,15 @@ Press Enter to continue...
 
 	result, err := a.WaitUntilUserLogsIn(ctx, http.DefaultClient, state)
 	if err != nil {
-		log.Fatal(err)
+		output.PrintErrorDetails("Error waiting for login", err)
+		os.Exit(1)
 	}
 
 	if err := a.StoreAccessToken(result.AccessToken); err != nil {
-		output.PrintError(
-			"Login failed.",
-			"Error occurred storing access token in your keyring.\nError details: %s",
-			err.Error(),
-		)
+		output.PrintErrorDetails("Login failed. Could not store credentials in keyring.", err)
 		os.Exit(1)
 	}
+
 	if err := a.StoreRefreshToken(result.RefreshToken); err != nil {
 		output.PrintError(
 			"Error occurred storing refresh token in your keyring.",
@@ -77,7 +76,7 @@ Press Enter to continue...
 		)
 	}
 
-	fmt.Println("🎉 You are now logged in to Numerous!")
+	output.PrintlnOK("You are now logged in to Numerous!")
 
 	return a.GetLoggedInUserFromKeyring()
 }

cli/cmd/logout/logout.go

@@ -1,7 +1,6 @@
 package logout
 
 import (
-	"fmt"
 	"net/http"
 	"os"
 
@@ -34,7 +33,7 @@ func logout(a auth.Authenticator) error {
 		output.PrintUnknownError(err)
 		return err
 	}
-	fmt.Println("✅ Successfully logged out!")
+	output.PrintlnOK("You are now logged out.")
 
 	return nil
 }

cli/cmd/output/ok.go

@@ -0,0 +1,9 @@
+package output
+
+import "fmt"
+
+// Prints message as a line prefixed with a green checkmark. Additional
+// arguments are used for formatting.
+func PrintlnOK(message string, args ...any) {
+	fmt.Printf(ansiGreen+checkmark+ansiReset+" "+message+"\n", args...)
+}