fix review findings

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

Author: sivchari

pkg/analysis/helpers/inspector/inspector.go

@@ -108,8 +108,7 @@ func (i *inspector) InspectFields(inspectField func(field *ast.Field, stack []as
 
 		markerSet := i.markers.FieldMarkers(field)
 
-		schemalessMarker := markerSet.Get(markersconsts.SchemaLessMarker)
-		if len(schemalessMarker) > 0 {
+		if isSchemalessType(markerSet) {
 			// If the field is marked as schemaless, we don't need to inspect it.
 			return false
 		}
@@ -172,6 +171,12 @@ func isItemsType(structType *ast.StructType) bool {
 	return true
 }
 
+func isSchemalessType(markerSet markers.MarkerSet) bool {
+	// Check if the field is marked as schemaless.
+	schemalessMarker := markerSet.Get(markersconsts.KubebuilderSchemaLessMarker)
+	return len(schemalessMarker) > 0
+}
+
 // printDebugInfo prints debug information about the field that caused a panic during inspection.
 // This function is designed to allow us to help identify which fields are causing issues during inspection.
 func printDebugInfo(field *ast.Field) string {

pkg/markers/markers.go

@@ -155,6 +155,9 @@ const (
 
 	// KubebuilderListMapKeyMarker is the marker used to specify the key field for map-type lists.
 	KubebuilderListMapKeyMarker = "listMapKey"
+
+	// KubebuilderSchemaLessMarker is the marker that indicates that a struct is schemaless.
+	KubebuilderSchemaLessMarker = "kubebuilder:validation:Schemaless"
 )
 
 const (
@@ -164,8 +167,3 @@ const (
 	// K8sRequiredMarker is the marker that indicates that a field is required in k8s declarative validation.
 	K8sRequiredMarker = "k8s:required"
 )
-
-const (
-	// SchemaLessMarker is the marker that indicates that a struct is schemaless.
-	SchemaLessMarker = "kubebuilder:validation:Schemaless"
-)