fix(govet): inline deprecated reflect.Ptr to reflect.Pointer

The govet inline analyzer flags reflect.Ptr as a deprecated alias.
reflect.Pointer was introduced in Go 1.18 as the preferred name and
both constants share the same underlying value, so this is a pure
rename. Without the change, golangci-lint blocks merges on every
file that still references reflect.Ptr (10 occurrences across 4
files; the run reports only 3 due to the default
max-same-issues=3 cap).

Author: charlesgreen

internal/llm/validator/model_validator.go

@@ -61,7 +61,7 @@ func WrapValidatorFunc(fn func(*ValidationContext, any, ValidatorFunc) (any, err
 func ValidateFieldMatch(field1, field2 string) func(any) error {
 	return func(v any) error {
 		val := reflect.ValueOf(v)
-		if val.Kind() == reflect.Ptr {
+		if val.Kind() == reflect.Pointer {
 			val = val.Elem()
 		}
 
@@ -88,7 +88,7 @@ func ValidateFieldMatch(field1, field2 string) func(any) error {
 func ValidateDateRange(startField, endField string, message string) func(any) error {
 	return func(v any) error {
 		val := reflect.ValueOf(v)
-		if val.Kind() == reflect.Ptr {
+		if val.Kind() == reflect.Pointer {
 			val = val.Elem()
 		}
 
@@ -128,7 +128,7 @@ func ValidateDateRange(startField, endField string, message string) func(any) er
 func ValidateConditionalRequired(conditionField string, conditionValue any, requiredField string) func(any) error {
 	return func(v any) error {
 		val := reflect.ValueOf(v)
-		if val.Kind() == reflect.Ptr {
+		if val.Kind() == reflect.Pointer {
 			val = val.Elem()
 		}
 
@@ -159,7 +159,7 @@ func ValidateConditionalRequired(conditionField string, conditionValue any, requ
 func ValidateAtLeastOne(fields ...string) func(any) error {
 	return func(v any) error {
 		val := reflect.ValueOf(v)
-		if val.Kind() == reflect.Ptr {
+		if val.Kind() == reflect.Pointer {
 			val = val.Elem()
 		}
 
@@ -182,7 +182,7 @@ func ValidateAtLeastOne(fields ...string) func(any) error {
 func ValidateMutuallyExclusive(fields ...string) func(any) error {
 	return func(v any) error {
 		val := reflect.ValueOf(v)
-		if val.Kind() == reflect.Ptr {
+		if val.Kind() == reflect.Pointer {
 			val = val.Elem()
 		}
 

pkg/llm/provider/structured.go

@@ -411,7 +411,7 @@ func SchemaFromStruct(t reflect.Type) *Schema {
 
 func generateSchemaFromType(t reflect.Type) *Schema {
 	// Handle pointers
-	if t.Kind() == reflect.Ptr {
+	if t.Kind() == reflect.Pointer {
 		t = t.Elem()
 	}
 

pkg/mcp/server.go

@@ -210,7 +210,7 @@ func generateSchemaFromType(v any) (Schema, error) {
 	t := reflect.TypeOf(v)
 
 	// Handle pointer types
-	if t.Kind() == reflect.Ptr {
+	if t.Kind() == reflect.Pointer {
 		t = t.Elem()
 	}
 
@@ -267,7 +267,7 @@ func generateSchemaFromType(v any) (Schema, error) {
 // typeToSchemaField converts a reflect.Type to a SchemaField
 func typeToSchemaField(t reflect.Type) SchemaField {
 	// Handle pointer types
-	if t.Kind() == reflect.Ptr {
+	if t.Kind() == reflect.Pointer {
 		t = t.Elem()
 	}
 

pkg/mcp/typed_tools.go

@@ -93,7 +93,7 @@ func generateSchema[T any]() Schema {
 	typ := reflect.TypeOf(t)
 
 	// Handle pointer types
-	if typ.Kind() == reflect.Ptr {
+	if typ.Kind() == reflect.Pointer {
 		typ = typ.Elem()
 	}
 
@@ -151,7 +151,7 @@ func parseJSONFieldName(tag string, defaultName string) string {
 // goTypeToJSONType converts Go types to JSON schema types
 func goTypeToJSONType(t reflect.Type) string {
 	// Handle pointer types
-	if t.Kind() == reflect.Ptr {
+	if t.Kind() == reflect.Pointer {
 		t = t.Elem()
 	}