fix(cli): remove duplicate error output and suppress usage on runtime errors (#466)

- Add SilenceUsage to root command so runtime errors don't print usage info
- Remove manual error print in main() that duplicated Cobra's automatic output
- Make whoami print "not logged in" as info instead of returning it as error
- Fix dash ping double-printing error details then a separate "Error: ping failed"

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: GenerQAQ

src/client/acontext-cli/cmd/dash_ping.go

@@ -16,15 +16,19 @@ func init() {
 		RunE: func(cmd *cobra.Command, args []string) error {
 			// 1. Verify that we have a local key for the resolved project
 			if dashProject == "" {
-				return fmt.Errorf("no project selected\n\nTo fix this, run:\n  acontext dash projects select")
+				fmt.Println(tui.RenderWarning("No project selected"))
+				fmt.Println("\nTo fix this, run:\n  acontext dash projects select")
+				return nil
 			}
 
 			ks, err := auth.LoadKeyStore()
 			if err != nil {
 				return fmt.Errorf("failed to load credentials: %w", err)
 			}
 			if ks.Keys[dashProject] == "" {
-				return fmt.Errorf("no API key found in credentials.json for project %s\n\nTo fix this, run:\n  acontext dash projects select --project %s --api-key <sk-ac-...>\n\nThe API key can be found on the Acontext Dashboard:\n  https://dash.acontext.io", dashProject, dashProject)
+				fmt.Println(tui.RenderWarning(fmt.Sprintf("No API key found in credentials.json for project %s", dashProject)))
+				fmt.Printf("\nTo fix this, run:\n  acontext dash projects select --project %s --api-key <sk-ac-...>\n\nThe API key can be found on the Acontext Dashboard:\n  https://dash.acontext.io\n", dashProject)
+				return nil
 			}
 
 			// 2. Check API connectivity
@@ -36,7 +40,7 @@ func init() {
 			if err := client.Ping(cmd.Context()); err != nil {
 				fmt.Printf("Ping failed for project %s: %v\n", dashProject, err)
 				fmt.Printf("Fix with: acontext dash projects select --project %s --api-key <sk-ac-...>\n", dashProject)
-				return fmt.Errorf("ping failed")
+				return nil
 			}
 
 			fmt.Println(tui.RenderSuccess(fmt.Sprintf("Project %s is reachable. Setup complete.", dashProject)))

src/client/acontext-cli/cmd/login.go

@@ -197,7 +197,8 @@ func runLogout(cmd *cobra.Command, args []string) error {
 func runWhoami(cmd *cobra.Command, args []string) error {
 	af, err := auth.MustLoad()
 	if err != nil {
-		return err
+		fmt.Println("Not logged in — run 'acontext login' first")
+		return nil
 	}
 
 	// Validate token with Supabase and refresh if needed
@@ -206,7 +207,8 @@ func runWhoami(cmd *cobra.Command, args []string) error {
 		// If session was auto-cleared (e.g. another device consumed the refresh token),
 		// show "not logged in" instead of the underlying error.
 		if !auth.IsLoggedIn() {
-			return fmt.Errorf("not logged in — run 'acontext login' first")
+			fmt.Println("Not logged in — run 'acontext login' first")
+			return nil
 		}
 		return err
 	}

src/client/acontext-cli/main.go

@@ -36,7 +36,6 @@ func main() {
 	}
 
 	if cmdErr := rootCmd.Execute(); cmdErr != nil {
-		fmt.Fprintf(os.Stderr, "Error: %v\n", cmdErr)
 		executedCmd, _, _ := rootCmd.Find(os.Args[1:])
 		if executedCmd == nil {
 			executedCmd = rootCmd
@@ -111,8 +110,9 @@ func buildCommandPath(cmd *cobra.Command) string {
 }
 
 var rootCmd = &cobra.Command{
-	Use:   "acontext",
-	Short: "Acontext CLI - Agent Skills as a Memory Layer",
+	Use:          "acontext",
+	SilenceUsage: true,
+	Short:        "Acontext CLI - Agent Skills as a Memory Layer",
 	Long: `Acontext CLI is a command-line tool for quickly creating Acontext projects.
 	
 It helps you: