fix: add stepaction as a valid kind in the hub resolver

The hub resolver should consider StepAction as a valid kind.
This change ensures that the controller considers a stepaction as a valid kind
resolving through a hub resolver.

test: validate step actions in hub resolver

Author: waveywaves

pkg/remoteresolution/resolver/hub/resolver_test.go

@@ -60,7 +60,16 @@ func TestValidate(t *testing.T) {
 			version:      "bar",
 			catalog:      "baz",
 			hubType:      ArtifactHubType,
-		}, {
+		},
+		{
+			testName:     "stepaction validation",
+			kind:         "stepaction",
+			resourceName: "foo",
+			version:      "bar",
+			catalog:      "baz",
+			hubType:      ArtifactHubType,
+		},
+		{
 			testName:     "tekton type validation",
 			kind:         "task",
 			resourceName: "foo",
@@ -134,7 +143,7 @@ func TestValidateConflictingKindName(t *testing.T) {
 		hubType string
 	}{
 		{
-			kind:    "not-taskpipeline",
+			kind:    "not-taskpipelineorstepaction",
 			name:    "foo",
 			version: "bar",
 			catalog: "baz",

pkg/resolution/resolver/hub/resolver.go

@@ -22,6 +22,7 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"slices"
 	"strings"
 
 	goversion "github.com/hashicorp/go-version"
@@ -45,6 +46,8 @@ const (
 	disabledError = "cannot handle resolution request, enable-hub-resolver feature flag not true"
 )
 
+var supportedKinds = []string{"task", "pipeline", "stepaction"}
+
 // Resolver implements a framework.Resolver that can fetch files from OCI bundles.
 //
 // Deprecated: Use [github.com/tektoncd/pipeline/pkg/remoteresolution/resolver/hub.Resolver] instead.
@@ -359,8 +362,8 @@ func validateParams(ctx context.Context, paramsMap map[string]string, tektonHubU
 		missingParams = append(missingParams, ParamVersion)
 	}
 	if kind, ok := paramsMap[ParamKind]; ok {
-		if kind != "task" && kind != "pipeline" {
-			return errors.New("kind param must be task or pipeline")
+		if !isSupportedKind(kind) {
+			return fmt.Errorf("kind param must be one of: %s", strings.Join(supportedKinds, ", "))
 		}
 	}
 	if hubType, ok := paramsMap[ParamType]; ok {
@@ -439,3 +442,7 @@ func resolveVersionConstraint(ctx context.Context, paramsMap map[string]string,
 	}
 	return ret, nil
 }
+
+func isSupportedKind(kindValue string) bool {
+	return slices.Contains[[]string, string](supportedKinds, kindValue)
+}

pkg/resolution/resolver/hub/resolver_test.go

@@ -62,7 +62,16 @@ func TestValidateParams(t *testing.T) {
 			version:      "bar",
 			catalog:      "baz",
 			hubType:      ArtifactHubType,
-		}, {
+		},
+		{
+			testName:     "stepaction validation",
+			kind:         "stepaction",
+			resourceName: "foo",
+			version:      "bar",
+			catalog:      "baz",
+			hubType:      ArtifactHubType,
+		},
+		{
 			testName:     "tekton type validation",
 			kind:         "task",
 			resourceName: "foo",
@@ -148,7 +157,7 @@ func TestValidateParamsConflictingKindName(t *testing.T) {
 		hubType string
 	}{
 		{
-			kind:    "not-taskpipeline",
+			kind:    "not-taskpipelineorstepaction",
 			name:    "foo",
 			version: "bar",
 			catalog: "baz",