fix: move run_worker_first validator out of shared validator package

Author: maxwellpeterson

…mvalidator/run_worker_first_validator.go …ers_script/run_worker_first_validator.gointernal/customvalidator/run_worker_first_validator.go renamed to internal/services/workers_script/run_worker_first_validator.go internal/customvalidator/run_worker_first_validator.go renamed to internal/services/workers_script/run_worker_first_validator.go

@@ -1,4 +1,4 @@
-package customvalidator
+package workers_script
 
 import (
 	"context"
@@ -15,12 +15,6 @@ var _ validator.Dynamic = runWorkerFirstValidator{}
 // - A list/tuple of strings (for path rules)
 type runWorkerFirstValidator struct{}
 
-// RunWorkerFirst returns a validator that ensures the dynamic value is either
-// a boolean or a list/tuple containing only string elements.
-func RunWorkerFirst() validator.Dynamic {
-	return runWorkerFirstValidator{}
-}
-
 func (v runWorkerFirstValidator) Description(ctx context.Context) string {
 	return v.MarkdownDescription(ctx)
 }
@@ -49,7 +43,7 @@ func (v runWorkerFirstValidator) ValidateDynamic(ctx context.Context, req valida
 		if _, isStringElement := listType.ElemType.(basetypes.StringType); isStringElement {
 			return // List of strings is valid
 		}
-		resp.Diagnostics.AddAttributeError(req.Path, "Invalid list element type", 
+		resp.Diagnostics.AddAttributeError(req.Path, "Invalid list element type",
 			"When using a list, all elements must be strings containing path rules")
 		return
 	}
@@ -59,7 +53,7 @@ func (v runWorkerFirstValidator) ValidateDynamic(ctx context.Context, req valida
 		// Verify all elements are strings
 		for i, elemType := range tupleType.ElemTypes {
 			if _, isString := elemType.(basetypes.StringType); !isString {
-				resp.Diagnostics.AddAttributeError(req.Path, "Invalid tuple element type", 
+				resp.Diagnostics.AddAttributeError(req.Path, "Invalid tuple element type",
 					fmt.Sprintf("Element at index %d must be a string, got %T", i, elemType))
 				return
 			}
@@ -68,6 +62,6 @@ func (v runWorkerFirstValidator) ValidateDynamic(ctx context.Context, req valida
 	}
 
 	// If we get here, the type is not supported
-	resp.Diagnostics.AddAttributeError(req.Path, "Invalid type for run_worker_first", 
+	resp.Diagnostics.AddAttributeError(req.Path, "Invalid type for run_worker_first",
 		fmt.Sprintf("Expected boolean or list/tuple of strings, got %T", valueType))
 }

internal/services/workers_script/schema.go

@@ -6,7 +6,6 @@ import (
 	"context"
 
 	"github.com/cloudflare/terraform-provider-cloudflare/internal/customfield"
-	"github.com/cloudflare/terraform-provider-cloudflare/internal/customvalidator"
 	"github.com/hashicorp/terraform-plugin-framework-jsontypes/jsontypes"
 	"github.com/hashicorp/terraform-plugin-framework-timetypes/timetypes"
 	"github.com/hashicorp/terraform-plugin-framework-validators/resourcevalidator"
@@ -125,11 +124,9 @@ func ResourceSchema(ctx context.Context) schema.Schema {
 								},
 							},
 							"run_worker_first": schema.DynamicAttribute{
-								Description: "When a boolean true, requests will always invoke the Worker script. Otherwise, attempt to serve an asset matching the request, falling back to the Worker script. When a list of strings, contains path rules to control routing to either the Worker or assets. Glob (*) and negative (!) rules are supported. Rules must start with either '/' or '!/'. At least one non-negative rule must be provided, and negative rules have higher precedence than non-negative rules.",
-								Optional:    true,
-								Validators: []validator.Dynamic{
-									customvalidator.RunWorkerFirst(),
-								},
+								Description:   "When a boolean true, requests will always invoke the Worker script. Otherwise, attempt to serve an asset matching the request, falling back to the Worker script. When a list of strings, contains path rules to control routing to either the Worker or assets. Glob (*) and negative (!) rules are supported. Rules must start with either '/' or '!/'. At least one non-negative rule must be provided, and negative rules have higher precedence than non-negative rules.",
+								Optional:      true,
+								Validators:    []validator.Dynamic{runWorkerFirstValidator{}},
 								CustomType:    customfield.NormalizedDynamicType{},
 								PlanModifiers: []planmodifier.Dynamic{customfield.NormalizeDynamicPlanModifier()},
 							},