From aaaa4b4d6418ac52596008bca94958aaa695101f Mon Sep 17 00:00:00 2001 From: Mathias Date: Mon, 14 Sep 2026 20:06:00 +0200 Subject: [PATCH] fix(lint): reflect.Ptr -> reflect.Pointer, unblocking task check `task check` has been red at HEAD since 2bb1fef adopted the canonical .golangci.yml from infra/ci-templates, which enables govet's `inline` analyzer. Nothing to do with the change it blocks; reproduced against a clean HEAD before touching it. reflect.Ptr is the pre-Go-1.18 spelling of reflect.Pointer and is a deprecated alias for it. Identical constant, no behaviour change. Separate commit because it is unrelated to the auth work that found it. Co-Authored-By: Claude Opus 5 --- internal/tools/tool.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/tools/tool.go b/internal/tools/tool.go index f561f68..8ceebdc 100644 --- a/internal/tools/tool.go +++ b/internal/tools/tool.go @@ -36,7 +36,7 @@ func parseArgs(raw json.RawMessage, dst any) error { // allowlist check. Returns a typed ErrValidation naming the missing field. func validateRequiredIdentifiers(dst any) error { v := reflect.ValueOf(dst) - if v.Kind() != reflect.Ptr || v.IsNil() { + if v.Kind() != reflect.Pointer || v.IsNil() { return nil } v = v.Elem()