refactor(cli): replace bind() with explicit field assignment

Remove the reflection-based bind() helper and replace it with direct
struct field assignment in each command handler. This eliminates a
hidden coupling between struct field declaration order and CLI argument
position, and fixes a latent runtime panic caused by calling
SetString() on authorizer.Role, a named string type that reflection
does not treat as a plain string.

Also validates the role argument early in memberAdd, returning a clear
error message instead of silently passing RoleInvalid downstream, and
fixes cobra.RangeArgs(2, 4) to cobra.RangeArgs(2, 3) in
namespaceCreate since --type is a flag, not a positional argument.

Author: geovannewashington

cli/cmd/cmd.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/shellhub-io/shellhub/cli/pkg/inputs"
 	"github.com/shellhub-io/shellhub/cli/services"
+	"github.com/shellhub-io/shellhub/pkg/api/authorizer"
 	"github.com/shellhub-io/shellhub/pkg/uuid"
 	"github.com/spf13/cobra"
 )
@@ -38,25 +39,23 @@ func namespaceCreate(service services.Services) *cobra.Command {
 		Long: `Creates a new namespace in the system using the provided namespace name, associated owner's username, and an optional tenant ID and Type.
 The owner must be a valid username within the system. If a tenant ID is provided, it should be in UUID format.`,
 		Example: `cli namespace create dev john_doe --type=team`,
-		Args:    cobra.RangeArgs(2, 4),
+		Args:    cobra.RangeArgs(2, 3),
 		RunE: func(cmd *cobra.Command, args []string) error {
-			// Avoid panic when TenantID isn't provided.
-
-			if len(args) == 2 {
-				args = append(args, "")
+			namespaceType, err := cmd.Flags().GetString("type")
+			if err != nil {
+				return err
 			}
 
-			var input inputs.NamespaceCreate
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.NamespaceCreate{
+				Namespace: args[0],
+				Owner:     args[1],
+				TenantID:  "",
+				Type:      namespaceType,
 			}
 
-			typeNamespace, err := cmd.Flags().GetString("type")
-			if err != nil {
-				return err
+			if len(args) == 3 {
+				input.TenantID = args[2]
 			}
-			input.Type = typeNamespace
 
 			namespace, err := service.NamespaceCreate(cmd.Context(), &input)
 			if err != nil {
@@ -86,10 +85,8 @@ func namespaceDelete(service services.Services) *cobra.Command {
 		Example: `cli namespace delete dev`,
 		Args:    cobra.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.NamespaceDelete
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.NamespaceDelete{
+				Namespace: args[0],
 			}
 
 			if err := service.NamespaceDelete(cmd.Context(), &input); err != nil {
@@ -328,10 +325,15 @@ and the role indicates the permissions that the member will have within that nam
 		Example: `cli member add myuser mynamespace observer`,
 		Args:    cobra.ExactArgs(3),
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.MemberAdd
+			role := authorizer.RoleFromString(args[2])
+			if role == authorizer.RoleInvalid {
+				return fmt.Errorf("invalid role %q, valid roles are: owner, administrator, operator, observer", args[2])
+			}
 
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.MemberAdd{
+				Username:  args[0],
+				Namespace: args[1],
+				Role:      role,
 			}
 
 			namespace, err := service.NamespaceAddMember(cmd.Context(), &input)
@@ -359,10 +361,9 @@ The username identifies the member to be removed, and the namespace specifies wh
 		Example: `cli member remove john_doe dev`,
 		Args:    cobra.ExactArgs(2),
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.MemberRemove
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.MemberRemove{
+				Username:  args[0],
+				Namespace: args[1],
 			}
 
 			namespace, err := service.NamespaceRemoveMember(cmd.Context(), &input)

cli/cmd/namespace.go

@@ -41,14 +41,13 @@ The username must be unique, and the password must meet the system's security re
 		Example: `cli user create john_doe Secret123!- john.doe@test.com
 cli user create john_doe Secret123!- john.doe@test.com --admin`,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.UserCreate
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.UserCreate{
+				Username: args[0],
+				Password: args[1],
+				Email:    args[2],
+				Admin:    admin,
 			}
 
-			input.Admin = admin
-
 			user, err := service.UserCreate(cmd.Context(), &input)
 			if err != nil {
 				return err
@@ -75,10 +74,9 @@ func userResetPassword(service services.Services) *cobra.Command {
 		Long:    `Updates the password for an existing user identified by the given username.`,
 		Example: `cli user password john_doe Secret123!-`,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.UserUpdate
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.UserUpdate{
+				Username: args[0],
+				Password: args[1],
 			}
 
 			if err := service.UserUpdate(cmd.Context(), &input); err != nil {
@@ -101,10 +99,8 @@ func userDelete(service services.Services) *cobra.Command {
 		Long:    `Deletes a user and all associated data from the system based on the provided username.`,
 		Example: `cli user delete john_doe`,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			var input inputs.UserDelete
-
-			if err := bind(args, &input); err != nil {
-				return err
+			input := inputs.UserDelete{
+				Username: args[0],
 			}
 
 			if err := service.UserDelete(cmd.Context(), &input); err != nil {