feat(cmd): accept short flag -H for --host in both server and client commands

Ayman Bagabas created 1 month ago

Change summary

internal/cmd/root.go   |  3 +--
internal/cmd/server.go | 10 +++++-----
2 files changed, 6 insertions(+), 7 deletions(-)

Detailed changes

internal/cmd/root.go

@@ -39,7 +39,7 @@ func init() {
 	rootCmd.Flags().BoolP("help", "h", false, "Help")
 	rootCmd.Flags().BoolP("yolo", "y", false, "Automatically accept all permissions (dangerous mode)")
 
-	rootCmd.Flags().StringVar(&clientHost, "host", server.DefaultHost(), "Connect to a specific crush server host (for advanced users)")
+	rootCmd.Flags().StringVarP(&clientHost, "host", "H", server.DefaultHost(), "Connect to a specific crush server host (for advanced users)")
 
 	rootCmd.AddCommand(runCmd)
 	rootCmd.AddCommand(updateProvidersCmd)
@@ -83,7 +83,6 @@ crush -y
 		case "unix", "npipe":
 			_, err := os.Stat(hostURL.Host)
 			if err != nil && errors.Is(err, fs.ErrNotExist) {
-				slog.Info("Starting server...", "host", clientHost)
 				if err := startDetachedServer(cmd); err != nil {
 					return err
 				}

internal/cmd/server.go

@@ -17,6 +17,11 @@ import (
 
 var serverHost string
 
+func init() {
+	serverCmd.Flags().StringVarP(&serverHost, "host", "H", server.DefaultHost(), "Server host (TCP or Unix socket)")
+	rootCmd.AddCommand(serverCmd)
+}
+
 var serverCmd = &cobra.Command{
 	Use:   "server",
 	Short: "Start the Crush server",
@@ -90,8 +95,3 @@ var serverCmd = &cobra.Command{
 		return nil
 	},
 }
-
-func init() {
-	serverCmd.Flags().StringVar(&serverHost, "host", server.DefaultHost(), "Server host (TCP or Unix socket)")
-	rootCmd.AddCommand(serverCmd)
-}