add project validation function, errors and apply for list command

Author: Benjosh95

internal/cmd/ske/cluster/list/list.go

@@ -13,12 +13,12 @@ import (
 	"github.com/stackitcloud/stackit-cli/internal/pkg/flags"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/globalflags"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/print"
-	"github.com/stackitcloud/stackit-cli/internal/pkg/projectname"
 	serviceEnablementClient "github.com/stackitcloud/stackit-cli/internal/pkg/services/service-enablement/client"
 	serviceEnablementUtils "github.com/stackitcloud/stackit-cli/internal/pkg/services/service-enablement/utils"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/services/ske/client"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/tables"
 	"github.com/stackitcloud/stackit-cli/internal/pkg/utils"
+	"github.com/stackitcloud/stackit-cli/internal/pkg/validation"
 
 	"github.com/spf13/cobra"
 	"github.com/stackitcloud/stackit-sdk-go/services/ske"
@@ -57,6 +57,12 @@ func NewCmd(params *params.CmdParams) *cobra.Command {
 				return err
 			}
 
+			// Validate project ID (exists and user has access)
+			projectLabel, err := validation.ValidateProject(ctx, params.Printer, params.CliVersion, cmd, model.ProjectId)
+			if err != nil {
+				return err
+			}
+
 			// Configure API client
 			apiClient, err := client.ConfigureClient(params.Printer, params.CliVersion)
 			if err != nil {
@@ -91,14 +97,6 @@ func NewCmd(params *params.CmdParams) *cobra.Command {
 				clusters = clusters[:*model.Limit]
 			}
 
-			projectLabel := model.ProjectId
-			if len(clusters) == 0 {
-				projectLabel, err = projectname.GetProjectName(ctx, params.Printer, params.CliVersion, cmd)
-				if err != nil {
-					params.Printer.Debug(print.ErrorLevel, "get project name: %v", err)
-				}
-			}
-
 			return outputResult(params.Printer, model.OutputFormat, projectLabel, clusters)
 		},
 	}
@@ -113,9 +111,6 @@ func configureFlags(cmd *cobra.Command) {
 
 func parseInput(p *print.Printer, cmd *cobra.Command) (*inputModel, error) {
 	globalFlags := globalflags.Parse(p, cmd)
-	if globalFlags.ProjectId == "" {
-		return nil, &errors.ProjectIdError{}
-	}
 
 	limit := flags.FlagToInt64Pointer(p, cmd, limitFlag)
 	if limit != nil && *limit < 1 {

internal/pkg/errors/errors.go

@@ -178,6 +178,16 @@ To list all profiles, run:
   $ stackit config profile list`
 
 	FILE_ALREADY_EXISTS = `file %q already exists in the export path. Delete the existing file or define a different export path`
+
+	PROJECT_NOT_FOUND = `the project %[2]q (ID: %[1]s) does not exist.
+
+To list all available projects, run:
+  $ stackit project list`
+
+	PROJECT_ACCESS_DENIED = `you don't have access to the project %[2]q (ID: %[1]s).
+
+To list all available projects, run:
+  $ stackit project list`
 )
 
 type ServerNicAttachMissingNicIdError struct {
@@ -499,3 +509,21 @@ type FileAlreadyExistsError struct {
 }
 
 func (e *FileAlreadyExistsError) Error() string { return fmt.Sprintf(FILE_ALREADY_EXISTS, e.Filename) }
+
+type ProjectNotFoundError struct {
+	ProjectId    string
+	ProjectLabel string
+}
+
+func (e *ProjectNotFoundError) Error() string {
+	return fmt.Sprintf(PROJECT_NOT_FOUND, e.ProjectId, e.ProjectLabel)
+}
+
+type ProjectAccessDeniedError struct {
+	ProjectId    string
+	ProjectLabel string
+}
+
+func (e *ProjectAccessDeniedError) Error() string {
+	return fmt.Sprintf(PROJECT_ACCESS_DENIED, e.ProjectId, e.ProjectLabel)
+}

internal/pkg/validation/project.go

@@ -0,0 +1,61 @@
+package validation
+
+import (
+	"context"
+	"fmt"
+	"net/http"
+
+	"github.com/stackitcloud/stackit-cli/internal/pkg/errors"
+	"github.com/stackitcloud/stackit-cli/internal/pkg/print"
+	"github.com/stackitcloud/stackit-cli/internal/pkg/projectname"
+	"github.com/stackitcloud/stackit-cli/internal/pkg/services/resourcemanager/client"
+
+	"github.com/spf13/cobra"
+	"github.com/stackitcloud/stackit-sdk-go/core/oapierror"
+)
+
+// ValidateProject validates that the project ID is not empty, exists, and the user has access to it.
+// It returns the project name for display purposes.
+func ValidateProject(ctx context.Context, p *print.Printer, cliVersion string, cmd *cobra.Command, projectId string) (string, error) {
+	// Check if project ID is empty
+	if projectId == "" {
+		return "", &errors.ProjectIdError{}
+	}
+
+	// Configure Resource Manager API client
+	apiClient, err := client.ConfigureClient(p, cliVersion)
+	if err != nil {
+		return "", fmt.Errorf("configure resource manager client: %w", err)
+	}
+
+	// Try to get project details to validate existence and access
+	req := apiClient.GetProject(ctx, projectId)
+	resp, err := req.Execute()
+	if err != nil {
+		// Check for specific HTTP status codes
+		if httpErr, ok := err.(*oapierror.GenericOpenAPIError); ok {
+			switch httpErr.StatusCode {
+			case http.StatusNotFound:
+				// Try to get project name for better error message
+				projectLabel := projectId
+				if projectName, nameErr := projectname.GetProjectName(ctx, p, cliVersion, cmd); nameErr == nil {
+					projectLabel = projectName
+				}
+				return "", &errors.ProjectNotFoundError{ProjectId: projectId, ProjectLabel: projectLabel}
+			case http.StatusForbidden:
+				// Try to get project name for better error message
+				projectLabel := projectId
+				if projectName, nameErr := projectname.GetProjectName(ctx, p, cliVersion, cmd); nameErr == nil {
+					projectLabel = projectName
+				}
+				return "", &errors.ProjectAccessDeniedError{ProjectId: projectId, ProjectLabel: projectLabel}
+			case http.StatusUnauthorized:
+				return "", &errors.AuthError{}
+			}
+		}
+		return "", fmt.Errorf("validate project: %w", err)
+	}
+
+	// Project exists and user has access, returning the project name
+	return *resp.Name, nil
+}