如何创建Golang lint规则要求goroutine通过带recover的包装启动？

一、创建自定义golangci-lint规则的步骤 #

golangci-lint支持通过AST分析扩展自定义规则，你可以按以下步骤实现强制使用指定包装函数启动goroutine的检查：

1. 初始化规则项目
   使用golangci-lint的规则模板生成项目骨架：

   golangci-lint new github.com/your-namespace/your-repo/rules/guardgoroutine
   

2. 编写AST分析逻辑
   在生成的规则文件中，实现对go语句的检测：

   • 遍历AST中的GoStmt节点
   • 判断该goroutine是否通过指定的包装函数（如GuardGo/SafeGo）调用
   • 如果是直接调用go func() {...}()，则抛出lint警告或错误

   核心判断逻辑示例：

   func (l *Linter) VisitNode(node ast.Node) {
       goStmt, ok := node.(*ast.GoStmt)
       if !ok {
           return
       }
       // 检查go语句是否是调用指定包装函数
       callExpr, ok := goStmt.Call.Fun.(*ast.CallExpr)
       if !ok {
           l.report(goStmt, "goroutine must be started via GuardGo/SafeGo wrapper")
           return
       }
       ident, ok := callExpr.Fun.(*ast.Ident)
       if !ok || (ident.Name != "GuardGo" && ident.Name != "SafeGo") {
           l.report(goStmt, "goroutine must be started via GuardGo/SafeGo wrapper")
       }
   }
   

3. 配置启用规则
   在你的.golangci.yml中添加自定义规则引用：

   linters-settings:
     custom:
       guardgoroutine:
         path: github.com/your-namespace/your-repo/rules/guardgoroutine
         description: Enforce goroutines to be started with panic-safe wrappers
         original-url: github.com/your-namespace/your-repo/rules/guardgoroutine
   linters:
     enable:
       - guardgoroutine
   

4. 测试规则
   编写测试用例，覆盖合法调用（使用包装函数）和违规调用（直接go语句）场景，验证规则正确性。

二、带崩溃恢复与上下文取消的包装函数示例 #

以下是两种符合需求的包装函数实现：

示例1：带自定义onRecover回调的版本 #

package utils

import (
	"context"
	"log"
)

// GuardGo 启动带崩溃恢复的goroutine，触发panic时执行onRecover回调并取消上下文
func GuardGo(ctx context.Context, onRecover func(ctx context.Context, err interface{}), fn func(context.Context)) {
	ctx, cancel := context.WithCancel(ctx)
	defer cancel()

	go func() {
		defer func() {
			if r := recover(); r != nil {
				log.Printf("goroutine recovered from panic: %v", r)
				onRecover(ctx, r)
				cancel()
			}
		}()

		fn(ctx)
	}()
}

示例2：内置上下文取消与日志的简化版本 #

package utils

import (
	"context"
	"log"
)

// SafeGo 自动处理goroutine崩溃，触发时取消上下文并记录错误日志
func SafeGo(ctx context.Context, fn func(context.Context)) {
	ctx, cancel := context.WithCancel(ctx)
	defer cancel()

	go func() {
		defer func() {
			if r := recover(); r != nil {
				log.Printf("goroutine panic recovered: %v", r)
				cancel()
			}
		}()

		fn(ctx)
	}()
}

三、不建议强制全局使用这类规则的原因 #

虽然这类包装函数能避免单个goroutine崩溃导致进程退出，但强制全局应用存在不少问题：

• 隐藏逻辑错误：Goroutine的panic通常是程序bug（如nil指针、数组越界），强制恢复会掩盖这些问题，让错误长期存在而难以被发现和修复。
• 上下文滥用风险：如果多个goroutine共享同一个上下文，单个goroutine崩溃就取消上下文会导致其他正常执行的goroutine被意外终止，破坏业务流程的完整性。
• 性能损耗：每个goroutine都额外添加defer和recover操作，在高并发场景下会累计一定的性能开销。
• 违背Go设计哲学：Go鼓励显式处理错误，panic仅用于不可恢复的严重错误，全局强制恢复相当于用“容错”替代“纠错”，不符合语言的设计初衷。
• 边界场景覆盖不足：goroutine内部启动的子goroutine无法被外层的恢复逻辑覆盖，依然可能导致进程崩溃；同时recover无法捕获运行时致命错误（如内存耗尽、栈溢出）。

内容的提问来源于stack exchange，提问作者D. Maul