fix(edge): 删掉 lifecycle 的第三份 safego 副本并接上 panic observer——Edge 生命周期 goroutine 的 panic 从此可观测

[DeliciousBuding]

结论先说

edge-server/internal/lifecycle/safego.go 是一份活的第三份 panic 恢复副本，守着 Edge 最关键的 13 处 goroutine 启动点；而 safego.SetPanicObserver 全仓只有 Hub 注册。结果：Edge 生命周期 goroutine 的 panic 只写日志、不进任何指标 —— 仪表盘与告警完全看不见，只能靠有人去 grep journal。同时 pkg/safego 的包注释用完成时态声称"两边已统一"，这是假的。

关联：#2154（Feynman 文档/架构探索批 P1；本 PR 同时是 #2230 的后续——那个 PR 给 pkg/safego 加了 RecoverInto，而 Edge 这份副本拿不到）

核实到的事实

事实	证据
lifecycle 私有副本活着且守着关键路径	safego.go 的 safeGo/recoverPanickedGoroutine；13 处调用点：run、hubAck、hubDoneEnqueue、hubFailEnqueue、hubCallbackQueue、resultAggregator、resultAggregatorTimeout、watchRunProcess、cancelGrace、build 阶段 5 处
同仓其它 edge 包早已用 pkg 版	adapters/orchestrator/orchestrator_dispatch_interceptor.go:184、api/handlers_events.go:107、store/file_store.go:92 → 一个进程两套恢复路径
Edge 无 observer	SetPanicObserver 唯一注册点 = hub-server/internal/metrics/metrics.go:18（init）
pkg 文档假声明	pkg/safego/safego.go 包注释："Both servers previously kept near-identical copies … This shared package unifies the recovery path"
无任何文档引用被删文件	全仓 .md grep lifecycle/safego = 0 命中

红证据（修复前）

panic_recovery_test.go:294: a recovered lifecycle panic never reached safego's
PanicObserver — Edge goroutine panics are invisible to metrics and alerting
--- FAIL: TestLifecycleGoroutinePanicReachesPanicObserver (3.00s)

该测试复用既有 TestFireHubCallbacksRecoverPanic 的 panicking CallbackReporter 夹具，注册 observer 后触发 fireHubAck，断言 observer 收到且 label 是 hubAck（label 是告警唯一的归因信息）。

改动

1. 删除 lifecycle/safego.go，13 处调用点改 safego.SafeGo（行为等价：pkg 版是超集，多一条 observer 派发），同步更新引用旧标识符的注释。
2. 新增指标 edge_goroutine_panic_recoveries_total{goroutine}：与既有 edge_http_panic_recoveries_total 合成"请求路径 / goroutine 路径"两半，也与 Hub 的 goroutine_panic_recoveries 对齐 —— 运维可用同一阈值告警两边。label 必须是静态调用点字面量（pkg/safego 对 name 的契约），空值归一到 unknown，注释里写明动态 label 会让 series 无界。
3. (*EdgeMetrics).InstallPanicObserver()：nil 安全；在 httpserver.buildEventBusAndMetrics 里于任何 run 可启动之前安装。
4. pkg/safego 包注释改成事实：点名 lifecycle 那份副本、它守的 goroutine 清单、造成的可观测性缺口、现在两边的 observer 安装点，并写明 hook 进程级唯一、二次安装会替换。
5. lifecycle/safego_test.go → panic_recovery_test.go（被测对象已不在本包），文件头写明分工：launcher 本身由 pkg/safego 自测；本文件钉两条属于本包的性质——① 生命周期 goroutine 确实都走它，② 恢复出来的 panic 可被观测。

覆盖是转移，不是消失（如实说明）

删掉了两个直测已删除的私有 launcher 的用例（TestSafeGoRecoversPanic / TestSafeGoRunsNormalFunc）：pkg/safego 有等价测试。本包保留的 4 路 fireHub* panic 测试仍断言 4 条恢复日志（期望串随 pkg 的日志消息更新）。

一处自陈：我第一版测试有竞态

defer close(done) 在 fn 内层先执行，而 observer 在 SafeGo 自己的 defer 里才跑 → <-done 后立刻读计数器恒为 0（pkg/safego 自己的测试注释也警告过这点）。改为带 2s 截止时间的轮询，并把原因写进注释。

门禁（全部跑在本 HEAD）

门禁	结果
edge go build ./... / go vet ./...	干净
go test ./internal/lifecycle ./internal/metrics ./internal/httpserver ./internal/api ./internal/store ./internal/adapters/orchestrator -count=1	6 包 ok
go test ./internal/lifecycle -race -count=1	ok 34.5s，无 DATA RACE
golangci-lint run（lifecycle/metrics/httpserver）	仅 1 条既有 gocyclo（process_executor_build.go:37 复杂度 22；本 PR 只替换了其中调用名，复杂度未变）
verify-edge-lint-ratchet.py	PASS（9 findings，全在基线）
gosec -fmt=json ./... | verify-gosec-gates.sh	CLEAN
verify-doc-ssot / verify-conventions	rc=0
pkg go test ./...	ok
git diff --check	干净

证据等级：L0（单测 + -race + 静态门禁）。

未做活体 panic 注入：新指标是 CounterVec，首次恢复 panic 前不产生 series，所以活体只能验证 Edge 重启健康 + /v1/metrics 可服务；接线正确性由上述两个 metrics 测试与 lifecycle observer 测试证明。合并后我会重建 bin/edge 并做这两项活体检查。

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: caaa14f5-6895-4037-bfa5-16e07c3e4262

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}