check only identical marker

Signed-off-by: sivchari <shibuuuu5@gmail.com>

Author: sivchari

pkg/analysis/duplicatemarkers/analyzer.go

@@ -20,6 +20,7 @@ import (
 	"go/ast"
 
 	"golang.org/x/tools/go/analysis"
+	"k8s.io/apimachinery/pkg/util/sets"
 
 	kalerrors "sigs.k8s.io/kube-api-linter/pkg/analysis/errors"
 	"sigs.k8s.io/kube-api-linter/pkg/analysis/helpers/extractjsontags"
@@ -47,37 +48,54 @@ func run(pass *analysis.Pass) (any, error) {
 	}
 
 	inspect.InspectFields(func(field *ast.Field, _ []ast.Node, _ extractjsontags.FieldTagInfo, markersAccess markers.Markers) {
-		if len(field.Names) == 0 {
-			return
-		}
 		checkField(pass, field, markersAccess)
 	})
 
+	inspect.InspectTypeSpec(func(typeSpec *ast.TypeSpec, markersAccess markers.Markers) {
+		checkTypeSpec(pass, typeSpec, markersAccess)
+	})
+
 	return nil, nil //nolint:nilnil
 }
 
 func checkField(pass *analysis.Pass, field *ast.Field, markersAccess markers.Markers) {
-	set := markersAccess.FieldMarkers(field)
+	if field == nil || len(field.Names) == 0 {
+		return
+	}
 
-	fieldName := field.Names[0].Name
+	fieldMarkers := markersAccess.FieldMarkers(field)
 
-	for _, marker := range set.UnsortedList() {
-		// TODO: Add check whether the marker is a duuplicate or not.
+	set := sets.New[string]()
+
+	for _, marker := range fieldMarkers.UnsortedList() {
+		if !set.Has(marker.String()) {
+			set.Insert(marker.String())
+			continue
+		}
 		pass.Report(analysis.Diagnostic{
 			Pos:     field.Pos(),
-			Message: fmt.Sprintf("%s has duplicated markers %s", fieldName, marker.String()),
-			SuggestedFixes: []analysis.SuggestedFix{
-				{
-					Message: fmt.Sprintf("should remove `// +%s`", marker.String()),
-					TextEdits: []analysis.TextEdit{
-						{
-							Pos:     marker.Pos,
-							End:     marker.End,
-							NewText: nil,
-						},
-					},
-				},
-			},
+			Message: fmt.Sprintf("%s has duplicated markers %s", field.Names[0].Name, marker.String()),
+		})
+	}
+}
+
+func checkTypeSpec(pass *analysis.Pass, typeSpec *ast.TypeSpec, markersAccess markers.Markers) {
+	if typeSpec == nil {
+		return
+	}
+
+	typeMarkers := markersAccess.TypeMarkers(typeSpec)
+
+	set := sets.New[string]()
+
+	for _, marker := range typeMarkers.UnsortedList() {
+		if !set.Has(marker.String()) {
+			set.Insert(marker.String())
+			continue
+		}
+		pass.Report(analysis.Diagnostic{
+			Pos:     typeSpec.Pos(),
+			Message: fmt.Sprintf("%s has duplicated markers %s", typeSpec.Name.Name, marker.String()),
 		})
 	}
 }

pkg/analysis/duplicatemarkers/testdata/src/a/a.go

@@ -10,7 +10,7 @@ type Enum string // want "Enum has duplicated markers kubebuilder:validation:Enu
 
 // +kubebuilder:validation:MaxLength=10
 // +kubebuilder:validation:MaxLength=11
-type MaxLength int // want "MaxLength has duplicated markers kubebuilder:validation:MaxLength"
+type MaxLength int
 
 // +kubebuilder:object:root=true
 // +kubebuilder:subresource:status
@@ -43,14 +43,14 @@ type DuplicateMarkerSpec struct { // want "DuplicateMarkerSpec has duplicated ma
 
 	// +kubebuilder:validation:MaxLength=11
 	// +kubebuilder:validation:MaxLength=10
-	DuplicatedMaxlength int `json:"maxlength"` // want "DuplicatedMaxlength has duplicated markers kubebuilder:validation:MaxLength"
+	DuplicatedMaxlength int `json:"maxlength"`
 
 	// +listType=map
 	// +listMapKey=primaryKey
 	// +listMapKey=secondaryKey
 	// +listType=map
 	// +required
-	DuplicatedListTypeMap Map `json:"duplicatedListTypeMap"` // want "DuplicatedListTypeMap has duplicated markers listType=map, listMapKey=primaryKey, listMapKey=secondaryKey"
+	DuplicatedListTypeMap Map `json:"duplicatedListTypeMap"` // want "DuplicatedListTypeMap has duplicated markers listType=map"
 
 	// +optional
 	// +kubebuilder:validation:XValidation:rule="self >= 1 && self <= 3",message="must be 1 to 5"