chore: cleanup

Signed-off-by: Kevin Conner <kev.conner@getupcloud.com>

Author: Kevin Conner

k8s/authorizer.go

@@ -186,7 +186,7 @@ func (r *ResourceCheck) Receive(function string, overload string, args []ref.Val
 				return types.NewErr("namespace already invoked")
 			}
 			if namespace, ok := getString(args[0].Value()); ok {
-				resourceCheck := &*r
+				resourceCheck := r
 				initResourceReceiver(resourceCheck, &namespace, resourceCheck.name, r.noSubresource)
 				return resourceCheck
 			}
@@ -198,7 +198,7 @@ func (r *ResourceCheck) Receive(function string, overload string, args []ref.Val
 				if len(name) == 0 {
 					return r
 				}
-				resourceCheck := &*r
+				resourceCheck := r
 				initResourceReceiver(resourceCheck, resourceCheck.namespace, &name, r.noSubresource)
 				return resourceCheck
 			}

k8s/conversion.go

@@ -23,7 +23,7 @@ func convertToMap(val any) (map[string]any, error) {
 		return nil, err
 	}
 	result := map[string]any{}
-	yaml.Unmarshal(bytes, result)
+	err = yaml.Unmarshal(bytes, result)
 	if err != nil {
 		return nil, err
 	}

k8s/evals.go

@@ -121,7 +121,7 @@ type EvalResponse struct {
 	Validations              []*EvalResult   `json:"validations,omitempty"`
 	AuditAnnotations         []*EvalResult   `json:"auditAnnotations,omitempty"`
 	WebhookMatchConditions   [][]*EvalResult `json:"webhookMatchConditions,omitempty"`
-	Cost                     *uint64         `json:"cost, omitempty"`
+	Cost                     *uint64         `json:"cost,omitempty"`
 }
 
 func getResults(val *ref.Val) (any, *string) {

k8s/validatingadmissionpolicy.go

@@ -165,7 +165,7 @@ func EvalValidatingAdmissionPolicy(policyInput, oldObjectInput, objectValueInput
 	// 'authorizer' - A CEL Authorizer. May be used to perform authorization checks for the principal (user or service account) of the request.
 	// 'authorizer.requestResource' - A CEL ResourceCheck constructed from the 'authorizer' and configured with the request resource.
 
-	matchConditionsEnvOptions := append([]cel.EnvOption(nil), celEnvOptions...)
+	matchConditionsEnvOptions := append([]cel.EnvOption{}, celEnvOptions...)
 	matchConditionsEnvOptions = append(matchConditionsEnvOptions, matchConditionsCelVars...)
 	matchConditionsEnv, err := cel.NewEnv(matchConditionsEnvOptions...)
 	if err != nil {
@@ -214,7 +214,7 @@ func EvalValidatingAdmissionPolicy(policyInput, oldObjectInput, objectValueInput
 
 	// run validations only if matchConditions pass
 	if matchConditions {
-		validationEnvOptions := append([]cel.EnvOption(nil), celEnvOptions...)
+		validationEnvOptions := append([]cel.EnvOption{}, celEnvOptions...)
 		validationEnvOptions = append(validationEnvOptions, validationCelVars...)
 		validationEnv, err := cel.NewEnv(validationEnvOptions...)
 		if err != nil {

k8s/webhook.go

@@ -84,7 +84,7 @@ func EvalWebhook(webhookInput, oldObjectInput, objectValueInput, requestInput, a
 	// 'authorizer' - A CEL Authorizer. May be used to perform authorization checks for the principal (user or service account) of the request.
 	// 'authorizer.requestResource' - A CEL ResourceCheck constructed from the 'authorizer' and configured with the request resource.
 
-	matchConditionsEnvOptions := append([]cel.EnvOption(nil), celEnvOptions...)
+	matchConditionsEnvOptions := append([]cel.EnvOption{}, celEnvOptions...)
 	matchConditionsEnvOptions = append(matchConditionsEnvOptions, matchConditionsCelVars...)
 	matchConditionsEnv, err := cel.NewEnv(matchConditionsEnvOptions...)
 	if err != nil {