Extract ValidateIndexNameArgs util function

Author: azzazzel

internal/pkg/cli/command/index/configure.go

@@ -2,10 +2,9 @@ package index
 
 import (
 	"context"
-	"errors"
-	"strings"
 
 	"github.com/pinecone-io/cli/internal/pkg/utils/exit"
+	"github.com/pinecone-io/cli/internal/pkg/utils/index"
 	"github.com/pinecone-io/cli/internal/pkg/utils/msg"
 	"github.com/pinecone-io/cli/internal/pkg/utils/pcio"
 	"github.com/pinecone-io/cli/internal/pkg/utils/presenters"
@@ -31,19 +30,7 @@ func NewConfigureIndexCmd() *cobra.Command {
 		Use:     "configure <name>",
 		Short:   "Configure an existing index with the specified configuration",
 		Example: "",
-		Args: func(cmd *cobra.Command, args []string) error {
-			if len(args) == 0 {
-				// TODO: start interactive mode. For now just return an error.
-				return errors.New("please provide an index name")
-			}
-			if len(args) > 1 {
-				return errors.New("please provide only one index name")
-			}
-			if strings.TrimSpace(args[0]) == "" {
-				return errors.New("index name cannot be empty")
-			}
-			return nil
-		},
+		Args:    index.ValidateIndexNameArgs,
 		Run: func(cmd *cobra.Command, args []string) {
 			options.name = args[0]
 			runConfigureIndexCmd(options)

internal/pkg/cli/command/index/create.go

@@ -2,12 +2,11 @@ package index
 
 import (
 	"context"
-	"errors"
-	"strings"
 
 	"github.com/MakeNowJust/heredoc"
 	"github.com/pinecone-io/cli/internal/pkg/utils/docslinks"
 	"github.com/pinecone-io/cli/internal/pkg/utils/exit"
+	"github.com/pinecone-io/cli/internal/pkg/utils/index"
 	"github.com/pinecone-io/cli/internal/pkg/utils/log"
 	"github.com/pinecone-io/cli/internal/pkg/utils/msg"
 	"github.com/pinecone-io/cli/internal/pkg/utils/pcio"
@@ -90,18 +89,7 @@ func NewCreateIndexCmd() *cobra.Command {
 		# create an integrated index
 		$ pc index create my-index --dimension 1536 --metric cosine --cloud aws --region us-east-1 --model multilingual-e5-large --field_map text=chunk_text
 		`),
-		Args: func(cmd *cobra.Command, args []string) error {
-			if len(args) == 0 {
-				return errors.New("please provide an index name")
-			}
-			if len(args) > 1 {
-				return errors.New("please provide only one index name")
-			}
-			if strings.TrimSpace(args[0]) == "" {
-				return errors.New("index name cannot be empty")
-			}
-			return nil
-		},
+		Args: index.ValidateIndexNameArgs,
 		Run: func(cmd *cobra.Command, args []string) {
 			options.name = args[0]
 			runCreateIndexCmd(options)

internal/pkg/cli/command/index/delete.go

@@ -2,10 +2,10 @@ package index
 
 import (
 	"context"
-	"errors"
 	"strings"
 
 	"github.com/pinecone-io/cli/internal/pkg/utils/exit"
+	"github.com/pinecone-io/cli/internal/pkg/utils/index"
 	"github.com/pinecone-io/cli/internal/pkg/utils/msg"
 	"github.com/pinecone-io/cli/internal/pkg/utils/sdk"
 	"github.com/pinecone-io/cli/internal/pkg/utils/style"
@@ -22,19 +22,7 @@ func NewDeleteCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "delete <name>",
 		Short: "Delete an index",
-		Args: func(cmd *cobra.Command, args []string) error {
-			if len(args) == 0 {
-				// TODO: start interactive mode. For now just return an error.
-				return errors.New("please provide an index name")
-			}
-			if len(args) > 1 {
-				return errors.New("please provide only one index name")
-			}
-			if strings.TrimSpace(args[0]) == "" {
-				return errors.New("index name cannot be empty")
-			}
-			return nil
-		},
+		Args:  index.ValidateIndexNameArgs,
 		Run: func(cmd *cobra.Command, args []string) {
 			options.name = args[0]
 			ctx := context.Background()

internal/pkg/cli/command/index/describe.go

@@ -1,10 +1,10 @@
 package index
 
 import (
-	"errors"
 	"strings"
 
 	"github.com/pinecone-io/cli/internal/pkg/utils/exit"
+	"github.com/pinecone-io/cli/internal/pkg/utils/index"
 	"github.com/pinecone-io/cli/internal/pkg/utils/msg"
 	"github.com/pinecone-io/cli/internal/pkg/utils/pcio"
 	"github.com/pinecone-io/cli/internal/pkg/utils/presenters"
@@ -25,19 +25,7 @@ func NewDescribeCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "describe <name>",
 		Short: "Get configuration and status information for an index",
-		Args: func(cmd *cobra.Command, args []string) error {
-			if len(args) == 0 {
-				// TODO: start interactive mode. For now just return an error.
-				return errors.New("please provide an index name")
-			}
-			if len(args) > 1 {
-				return errors.New("please provide only one index name")
-			}
-			if strings.TrimSpace(args[0]) == "" {
-				return errors.New("index name cannot be empty")
-			}
-			return nil
-		},
+		Args:  index.ValidateIndexNameArgs,
 		Run: func(cmd *cobra.Command, args []string) {
 			options.name = args[0]
 			pc := sdk.NewPineconeClient()

internal/pkg/utils/index/validation.go

@@ -0,0 +1,23 @@
+package index
+
+import (
+	"errors"
+	"strings"
+
+	"github.com/spf13/cobra"
+)
+
+// ValidateIndexNameArgs validates that exactly one non-empty index name is provided as a positional argument.
+// This is the standard validation used across all index commands (create, describe, delete, configure).
+func ValidateIndexNameArgs(cmd *cobra.Command, args []string) error {
+	if len(args) == 0 {
+		return errors.New("please provide an index name")
+	}
+	if len(args) > 1 {
+		return errors.New("please provide only one index name")
+	}
+	if strings.TrimSpace(args[0]) == "" {
+		return errors.New("index name cannot be empty")
+	}
+	return nil
+}