clean up cli command parsing, remove unnecessary code

Author: chenosaurus

cmd/lk/room.go

@@ -179,7 +179,7 @@ var (
 						},
 						&cli.BoolFlag{
 							Name:  "auto-subscribe",
-							Usage: "Automatically subscribe to published tracks. Default is false.",
+							Usage: "Automatically subscribe to published tracks.",
 							Value: false,
 						},
 					},
@@ -789,10 +789,7 @@ func joinRoom(ctx context.Context, cmd *cli.Command) error {
 		return err
 	}
 
-	autoSubscribe, err := extractFlagAsBool(cmd, "auto-subscribe")
-	if err != nil {
-		return err
-	}
+	autoSubscribe := cmd.Bool("auto-subscribe")
 
 	participantIdentity := cmd.String("identity")
 

cmd/lk/utils.go

@@ -142,14 +142,6 @@ func extractFlagOrArg(c *cli.Command, flag string) (string, error) {
 	return value, nil
 }
 
-func extractFlagAsBool(c *cli.Command, flag string) (bool, error) {
-	if c.IsSet(flag) {
-		return c.Bool(flag), nil
-	}
-
-	return false, nil
-}
-
 type loadParams struct {
 	requireURL bool
 }

version.go

@@ -15,5 +15,5 @@
 package livekitcli
 
 const (
-	Version = "2.4.5"
+	Version = "2.4.6"
 )