Go 1.27 vet 为什么提示错误包装指针类型

如果 Go 1.27 的 vet 报出 %w wants operand of error type NotFound, not pointer type *NotFound (defeats errors.Is)，通常不是 vet 不认识指针，而是错误类型 NotFound 用值接收者实现了 error，代码却把 *NotFound 放进了 %w。这样包装后的动态类型变成指针，调用方若按值类型做 errors.Is、errors.As 或类型断言，就可能永远匹配不到。

这条 vet 提示到底在检查什么

Go 1.27 的 printf 分析器把 fmt.Errorf 的 %w 也纳入了检查。下面的类型声明使用值接收者，因此两个类型都满足 error：

package main

import (
    "errors"
    "fmt"
)

type NotFound struct { ID int }

// 值接收者让 NotFound 和 *NotFound 都拥有 Error 方法。
func (e NotFound) Error() string { return fmt.Sprintf("record %d not found", e.ID) }

func load() error {
    p := &NotFound{ID: 7}
    // 这里的动态类型是 *NotFound，Go 1.27 vet 会提醒它可能破坏按值匹配。
    return fmt.Errorf("load: %w", p)
}

func findValue(err error) bool {
    var target NotFound
    // target 的地址是 *NotFound，但目标错误类型是 NotFound。
    return errors.As(err, &target)
}

关键不在于“指针一定不能包装”，而在于包装实参的具体类型和业务代码准备匹配的类型不一致。errors.As 会沿着包装树寻找可赋值的具体错误类型；*NotFound 不会自动变成 NotFound。

先看 Error 方法的接收者，再决定修复方向

排查时先跳到错误类型的 Error 方法。值接收者意味着值和指针都进入方法集；指针接收者则通常只有指针满足 error。可以按下面的表快速判断：

如果错误只是携带少量不可变字段，最简单的改法是包装值，并把匹配目标也设为值：

func load() error {
    // 让包装树中的动态类型保持为 NotFound。
    return fmt.Errorf("load: %w", NotFound{ID: 7})
}

func isNotFound(err error) bool {
    var target NotFound
    // errors.As 的第二个参数仍然要传目标变量的地址。
    return errors.As(err, &target)
}

若错误需要共享可变状态，或者项目已经把指针类型作为稳定契约，也可以保留指针包装，但匹配必须写成：

func isPointerNotFound(err error) bool {
    var target *NotFound
    // &target 的类型是 **NotFound，用来接收包装树中的 *NotFound。
    return errors.As(err, &target)
}

用一轮小检查确认告警已经消失

修改后不要只看编译是否通过。编译器允许这两种动态类型，真正容易遗漏的是调用方的错误分类逻辑。建议按三层复查：

1. 先运行 go vet ./...，确认同一行的 %w 指针提示消失；go test 在 Go 1.27 中也会默认调用部分 vet 检查。
2. 为 errors.As 写一个最小断言，分别覆盖包装值和包装指针的路径；如果使用 errors.Is，确认比较目标与动态类型及自定义 Is 方法契约一致。
3. 检查 go.mod 的 go 指令和 CI 使用的 Go 版本，避免本地不提示、升级后突然在测试阶段失败。

临时使用 go test -vet=off 只能隐藏诊断，不会改变包装后的动态类型，也不能修复调用方的匹配逻辑。若项目暂时需要兼容旧工具链，应在兼容矩阵中分别跑测试，而不是把这个检查永久关闭。

把修复规则写进代码评审清单

这类告警适合形成一条很短的评审规则：先问错误类型的 Error 接收者，再看 %w 的实参，最后对照调用方的 errors.Is、errors.As 或 errors.AsType 目标。只有三者的类型契约一致，错误包装才真正保留了可分类性。

如果没有明确的指针语义，优先包装值类型；如果必须返回指针，就在接口文档和测试中明确调用方要匹配 *E。这样升级 Go 1.27 后，vet 的提示就能变成一次有价值的类型契约检查，而不是简单地改掉一颗星号。

相关问题

为什么 errors.As 的第二个参数看起来还要传指针？

第二个参数是“接收匹配结果的变量地址”。匹配值类型 E 时传 *E，匹配指针类型 *E 时则传 **E；它与被包装错误的动态类型不是同一个概念。

把 Error 改成指针接收者能不能消除提示？

这可能改变接口契约，但不是机械修复。改完后 E 通常不再实现 error，调用方、返回值和测试都要按 *E 重新检查。

只运行 go vet 不运行测试可以吗？

不建议。vet 能发现可疑包装，但无法替代针对 errors.Is、errors.As 和业务分支的行为测试。