refactor: new validation rule interface

hauler-manifest.yaml

@@ -57,4 +57,4 @@ metadata:
 spec:
   files:
     - name: validatorctl
-      path: https://github.com/validator-labs/validatorctl/releases/download/v0.1.1/validator-linux-ARCH
+      path: https://github.com/validator-labs/validatorctl/releases/download/v0.1.2/validator-linux-ARCH

pkg/validationresult/validation_result.go

@@ -26,16 +26,16 @@
 	Patch(ctx context.Context, obj client.Object, opts ...patch.Option) error
 }
 
-// ValidationRule is an interface for validation rules.
-type ValidationRule interface {
+// Validator is an interface for building ValidationResults.
+type Validator interface {
 	client.Object
 	GetKind() string
 	PluginCode() string
 	ResultCount() int
 }
 
-// Build creates a new ValidationResult for a specific ValidationRule.
-func Build(r ValidationRule) *v1alpha1.ValidationResult {
+// Build creates a new ValidationResult for a specific Validator.
+func Build(r Validator) *v1alpha1.ValidationResult {
 	return &v1alpha1.ValidationResult{
 		TypeMeta: metav1.TypeMeta{
 			APIVersion: v1alpha1.APIVersion,
@@ -61,8 +61,8 @@
 	}
 }
 
-// Name returns the name of a ValidationResult for a specific ValidationRule.
-func Name(r ValidationRule) string {
+// Name returns the name of a ValidationResult for a specific Validator.
+func Name(r Validator) string {
 	name := fmt.Sprintf("validator-plugin-%s-%s", r.PluginCode(), r.GetName())
 	return util.Sanitize(name)
 }

pkg/validationrule/validation_rule.go

@@ -0,0 +1,38 @@
+// Package validationrule describes validation rules.
+package validationrule
+
+// Interface defines validation rule behavior.
+type Interface interface {
+	// Name returns the name of the rule.
+	Name() string
+	// SetName sets the name of the rule if it is a rule that requires manually specifying its name.
+	// This should be a no-op for rules that automatically generate their name.
+	SetName(string)
+	// RequiresName returns whether the validation rule requires its name to be manually specified.
+	// This should return false for rules that automatically generate their name.
+	RequiresName() bool
+}
+
+// ManuallyNamed can be embedded into a rule struct to indicate that the rule requires its name to
+// be manually specified.
+type ManuallyNamed struct{}
+
+// RequiresName returns true.
+func (ManuallyNamed) RequiresName() bool {
+	return true
+}
+
+// AutomaticallyNamed can be embedded into a rule struct to indicate that the rule does not require
+// its name to be manually specified because it is automatically generated from other data in the
+// rule.
+type AutomaticallyNamed struct{}
+
+// RequiresName returns false.
+func (AutomaticallyNamed) RequiresName() bool {
+	return false
+}
+
+// SetName is a no-op because the rule does not support manually specifying its name.
+func (AutomaticallyNamed) SetName(string) {
+	// no-op
+}