From e0ee0c43bbfb20261552f8e50af9842d022922ec Mon Sep 17 00:00:00 2001 From: JianFeeeee Date: Fri, 9 Oct 2026 07:32:00 +0800 Subject: [PATCH] =?UTF-8?q?fix(hmapdev):=20SDK=20=E6=A8=A1=E5=9D=97?= =?UTF-8?q?=E8=AF=86=E5=88=AB=E8=A6=81=E8=AE=A4=E6=96=B0=E6=97=A7=E4=B8=A4?= =?UTF-8?q?=E4=B8=AA=20module=20=E5=90=8D=EF=BC=88=E8=BF=81=E7=A7=BB?= =?UTF-8?q?=E6=BC=8F=E6=94=B9=E7=9A=84=E9=80=BB=E8=BE=91=E5=88=A4=E5=AE=9A?= =?UTF-8?q?=EF=BC=89?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 上一提交（b6a6168）只替换了 import 路径，**漏了用旧 module 名做逻辑判定 的五处**。CI 当场抓到一处（`TestPrintVersionReportsInjectedVersion`）， 但那是测试断言；另外三处是**产品逻辑**，CI 抓不到，是静默失效。 ## 漏改的地方 | 位置 | 原判定 | 失效后果 | |---|---|---| | `cmd_build.go` | `Contains(mod,"homeagent-sdk")` | 认不出 SDK 依赖 ⇒ **不写 replace** | | `cmd_skill.go` | `Contains(gm,"homeagent-sdk")` | 认不出 SDK 仓 ⇒ 不安装 skills | | `yaegi/interp.go` ×2 | `Contains(line,"homeagent-sdk")` | `findSDKGoPath` 找不到 SDK ⇒ 退化到兜底 | 迁移后这些 `Contains` **恒为假**（新名是 `homeagentsdk`，没有连字符）。 三者都**不报错**，只是行为悄悄变成另一条分支——比直接崩更难发现。 ## 关键：不能只认新名 本机实测存量插件工作区的 go.mod **全部仍是旧名**： ``` /home/program/qq-plugin (无 go.mod) /home/program/vanblog-plugin require gitcode.com/JianFeeeee/homeagent-sdk /home/program/mcquery-plugin require gitcode.com/JianFeeeee/homeagent-sdk /home/program/fileproc-plugin require gitcode.com/JianFeeeee/homeagent-sdk /home/program/agentmail/plugins/ require gitcode.com/JianFeeeee/homeagent-sdk ``` 用户已创建的项目不会因为我们改仓就自动更新。所以判定必须 **两种都接受**，而不是把旧名换成新名（那只是把 bug 从"认不出新"换成"认不出旧"）。 ## 改动 - `meta` 补 `SDKModule` / `LegacySDKModule` 常量（原注释引用了 `SDKModule` 但常量并不存在——注释先于实现） - 新增 `tools/hmapdev/sdkmodule.go`：`isSDKModule` / `sdkModulePath` / `containsSDKModule` - 新增 `tools/hmapdev/yaegi/sdkmodule.go`：同语义的最小副本 （yaegi 是独立子包，不能 import 父包） - 三处逻辑判定改用上述助手 ## 判据（可杀变异，逐个实测） `sdkmodule_test.go` + `yaegi/sdkmodule_test.go`：覆盖新名/旧名/带版本号的 旧名/空串/核心仓（`HomeAgent`，不是 SDK）/无关模块。 这三个判定点**此前零测试覆盖**——这正是它们能静默失效的原因。 变异验证： - 只认新名 → FAIL「旧名漏认会让存量插件静默走错分支」 - `containsSDKModule` 只认新名 → FAIL「必须认旧名」 ## 验证 - SDK 根 module + hmapdev module：`go build` / `go test` 全通过 - 20 个 example `go vet` 全通过 - `go mod verify` → all modules verified --- meta/meta.go | 15 +++++++ tools/hmapdev/cmd_build.go | 2 +- tools/hmapdev/cmd_skill.go | 2 +- tools/hmapdev/main_version_test.go | 3 +- tools/hmapdev/sdkmodule.go | 61 +++++++++++++++++++++++++++ tools/hmapdev/sdkmodule_test.go | 58 +++++++++++++++++++++++++ tools/hmapdev/yaegi/interp.go | 4 +- tools/hmapdev/yaegi/sdkmodule.go | 31 ++++++++++++++ tools/hmapdev/yaegi/sdkmodule_test.go | 26 ++++++++++++ 9 files changed, 197 insertions(+), 5 deletions(-) create mode 100644 tools/hmapdev/sdkmodule.go create mode 100644 tools/hmapdev/sdkmodule_test.go create mode 100644 tools/hmapdev/yaegi/sdkmodule.go create mode 100644 tools/hmapdev/yaegi/sdkmodule_test.go diff --git a/meta/meta.go b/meta/meta.go index 9ed1cf0..c6553c6 100644 --- a/meta/meta.go +++ b/meta/meta.go @@ -59,6 +59,21 @@ var ( // SDKName 是 SDK 名称。 SDKName = "HomeAgent SDK" + // SDKModule 是本 SDK 仓当前的 Go module path。 + // + // ★ 2026-10-08 随托管地迁移：`gitcode.com/JianFeeeee/homeagent-sdk` + // → `github.com/JianFeeeee/homeagentsdk`（仓库迁到 GitHub）。 + // + // ❗已有插件的 go.mod 里写的仍是旧路径（本机实测 qq/vanblog/mcquery/ + // fileproc/mail-bridge 五个插件工作区全是 gitcode.com/...）。 + // 因此凡是要「识别 SDK 模块」的判定必须**两种都接受** + // （见 hmapdev 的 sdkModulePath 助手），不能只认新名—— + // 只认新名会让旧插件被当成「不是 SDK 模块」而静默走错分支。 + SDKModule = "github.com/JianFeeeee/homeagentsdk" + + // LegacySDKModule 是迁移前的 SDK module path，仍需被识别。 + LegacySDKModule = "gitcode.com/JianFeeeee/homeagent-sdk" + // CoreModule 是核心仓的 Go module path，供 hmapdev 生成 go.mod 时使用。 // // ★ 2026-10-08 随主仓托管地迁移而更新：仓库在 GitHub diff --git a/tools/hmapdev/cmd_build.go b/tools/hmapdev/cmd_build.go index c3160f7..ba01055 100644 --- a/tools/hmapdev/cmd_build.go +++ b/tools/hmapdev/cmd_build.go @@ -411,7 +411,7 @@ func ensureGoMod(plg *PlgConfig, sdkPath string) string { mod = parts[0] } } - if mod != "" && strings.Contains(mod, "homeagent-sdk") { + if mod != "" && isSDKModule(mod) { sdkModule = mod break } diff --git a/tools/hmapdev/cmd_skill.go b/tools/hmapdev/cmd_skill.go index 3842366..ff34329 100644 --- a/tools/hmapdev/cmd_skill.go +++ b/tools/hmapdev/cmd_skill.go @@ -130,7 +130,7 @@ func repoSkillsFromExe(exe string) string { if st, err := os.Stat(cand); err == nil && st.IsDir() { // 确认这确实像 SDK 仓（有 go.mod 且 module 是 homeagent-sdk） if gm, err := os.ReadFile(filepath.Join(dir, "go.mod")); err == nil && - strings.Contains(string(gm), "homeagent-sdk") { + containsSDKModule(string(gm)) { return cand } } diff --git a/tools/hmapdev/main_version_test.go b/tools/hmapdev/main_version_test.go index 321a04f..301b214 100644 --- a/tools/hmapdev/main_version_test.go +++ b/tools/hmapdev/main_version_test.go @@ -34,7 +34,8 @@ func TestPrintVersionReportsInjectedVersion(t *testing.T) { if !strings.Contains(out, "deadbee") { t.Fatalf("提交号未出现在输出里:\n%s", out) } - if !strings.Contains(out, "HomeAgent") && !strings.Contains(out, "homeagent-sdk") { + if !strings.Contains(out, "HomeAgent") && !strings.Contains(out, "homeagent-sdk") && + !strings.Contains(out, "homeagentsdk") { t.Fatalf("输出里没有 SDK 模块标识:\n%s", out) } } diff --git a/tools/hmapdev/sdkmodule.go b/tools/hmapdev/sdkmodule.go new file mode 100644 index 0000000..4b69bdb --- /dev/null +++ b/tools/hmapdev/sdkmodule.go @@ -0,0 +1,61 @@ +package main + +import "strings" + +// sdkModulePath 报告 mod 是否为 HomeAgent SDK 的 module path， +// 是则返回规范化的模块名。 +// +// # 为何要同时认两个名字（2026-10-08） +// +// SDK 仓已从 gitcode 迁到 GitHub，module 路径随之变成 +// `github.com/JianFeeeee/homeagentsdk`。但**存量插件的 go.mod 里写的仍是旧路径** +// `gitcode.com/JianFeeeee/homeagent-sdk` —— 本机实测 qq / vanblog / mcquery / +// fileproc / homeagent-mail-bridge 五个插件工作区全部如此（用户已创建的项目 +// 不可能因为我们改了仓就自动更新）。 +// +// 所以任何「识别 SDK 模块」的判定都必须**两种都接受**： +// 只认新名 ⇒ 旧插件被当成「这不是 SDK 依赖」⇒ 静默走错分支 +// （不写 replace、认不出 SDK 仓、yaegi 找不到 SDK 源码路径）。 +// +// 这类失效**不报错**，正是最危险的一种：迁移代码路径改完了， +// 逻辑判定漏改，功能悄悄坏掉而测试仍然绿。 +// +// 为何用 Contains 而不是 ==：module 行可能带版本号后缀 +// （`gitcode.com/.../v1.3.0`）或是 require 块内的裸写法。 +func sdkModulePath(mod string) (string, bool) { + mod = strings.TrimSpace(mod) + if mod == "" { + return "", false + } + for _, name := range []string{ + "github.com/JianFeeeee/homeagentsdk", + "gitcode.com/JianFeeeee/homeagent-sdk", + } { + if strings.Contains(mod, name) { + return name, true + } + } + return "", false +} + +// isSDKModule 是 sdkModulePath 的布尔形式（调用方不关心规范名时用）。 +func isSDKModule(mod string) bool { + _, ok := sdkModulePath(mod) + return ok +} + +// containsSDKModule 报告一段文本（如 go.mod 全文）里是否出现 SDK module 名。 +// +// 与 isSDKModule 分开：那个判「单个 module path 是不是 SDK」， +// 这个判「整份文件里有没有引用 SDK」——用途不同，别合并。 +func containsSDKModule(text string) bool { + for _, name := range []string{ + "github.com/JianFeeeee/homeagentsdk", + "gitcode.com/JianFeeeee/homeagent-sdk", + } { + if strings.Contains(text, name) { + return true + } + } + return false +} diff --git a/tools/hmapdev/sdkmodule_test.go b/tools/hmapdev/sdkmodule_test.go new file mode 100644 index 0000000..5502412 --- /dev/null +++ b/tools/hmapdev/sdkmodule_test.go @@ -0,0 +1,58 @@ +package main + +import "testing" + +// sdkModule 识别必须**同时认新旧两个 module 名**。 +// +// # 为何这条判据重要（2026-10-08） +// +// SDK 仓从 gitcode 迁到 GitHub 后 module 路径变成 +// `github.com/JianFeeeee/homeagentsdk`。我（AI）做迁移时只替换了 import 路径， +// **漏了用旧名做逻辑判定的地方**——`cmd_build.go` 用 +// `strings.Contains(mod, "homeagent-sdk")` 判断「这是不是 SDK 依赖」， +// 迁移后恒为假。 +// +// 后果是**静默**的：不写 replace、认不出 SDK 仓，没有任何报错。 +// 这类失效只有实测才会暴露，而当时没有任何测试覆盖这三个判定点。 +// +// 存量插件的 go.mod 里写的仍是旧名（本机实测 qq / vanblog / mcquery / +// fileproc / mail-bridge 五个工作区全是 `gitcode.com/...`）， +// 所以**旧名必须继续被认**，不能简单换名字。 +func TestSDKModulePathAcceptsBothNames(t *testing.T) { + cases := []struct { + mod string + want bool + }{ + // 当前名 + {"github.com/JianFeeeee/homeagentsdk", true}, + {"github.com/JianFeeeee/homeagentsdk/sdk", true}, + // 迁移前旧名（存量插件仍在用，必须继续认） + {"gitcode.com/JianFeeeee/homeagent-sdk", true}, + {"gitcode.com/JianFeeeee/homeagent-sdk/sdk", true}, + {"gitcode.com/JianFeeeee/homeagent-sdk v1.3.0", true}, + // 不应误判 + {"", false}, + {"github.com/JianFeeeee/HomeAgent", false}, // 核心仓，不是 SDK + {"github.com/other/sdk", false}, + {" ", false}, + } + for _, c := range cases { + if got := isSDKModule(c.mod); got != c.want { + t.Errorf("isSDKModule(%q) = %v, want %v\n"+ + " 旧名漏认会让存量插件静默走错分支（不写 replace / 找不到 SDK 路径）", + c.mod, got, c.want) + } + } +} + +func TestContainsSDKModuleAcceptsBothNames(t *testing.T) { + if !containsSDKModule("module foo\nrequire github.com/JianFeeeee/homeagentsdk v0.0.0\n") { + t.Error("containsSDKModule 必须认当前名") + } + if !containsSDKModule("require gitcode.com/JianFeeeee/homeagent-sdk v0.0.0\n") { + t.Error("containsSDKModule 必须认旧名（存量插件 go.mod 仍是旧名）") + } + if containsSDKModule("module foo\n\ngo 1.21\n") { + t.Error("不含 SDK 依赖的 go.mod 不应被误认为 SDK 仓") + } +} diff --git a/tools/hmapdev/yaegi/interp.go b/tools/hmapdev/yaegi/interp.go index 914d0b3..9f91a93 100644 --- a/tools/hmapdev/yaegi/interp.go +++ b/tools/hmapdev/yaegi/interp.go @@ -28,10 +28,10 @@ func findSDKGoPath(pluginDir string) string { if data, err := os.ReadFile(gm); err == nil { for _, line := range strings.Split(string(data), "\n") { line = strings.TrimSpace(line) - if strings.HasPrefix(line, "replace ") && strings.Contains(line, "homeagent-sdk") { + if strings.HasPrefix(line, "replace ") && isSDKModule(line) { parts := strings.Fields(line) for _, p := range parts { - if strings.Contains(p, "homeagent-sdk") && strings.Contains(p, string(filepath.Separator)) { + if isSDKModule(p) && strings.Contains(p, string(filepath.Separator)) { return filepath.Dir(filepath.Dir(p)) } } diff --git a/tools/hmapdev/yaegi/sdkmodule.go b/tools/hmapdev/yaegi/sdkmodule.go new file mode 100644 index 0000000..5c717dd --- /dev/null +++ b/tools/hmapdev/yaegi/sdkmodule.go @@ -0,0 +1,31 @@ +package yaegi + +import "strings" + +// sdkModuleNames 是 HomeAgent SDK 的 module path：**当前名 + 迁移前旧名**。 +// +// # 为何必须两个都认（2026-10-08） +// +// SDK 仓已从 gitcode 迁到 GitHub，module 路径变成 +// `github.com/JianFeeeee/homeagentsdk`；但**存量插件的 go.mod 里写的仍是旧路径** +// `gitcode.com/JianFeeeee/homeagent-sdk`（本机实测 qq / vanblog / mcquery / +// fileproc / mail-bridge 五个插件工作区全是旧名）。用户已建的项目不会因为 +// 我们改了仓就自动更新。 +// +// 本包的 findSDKGoPath 要从插件 go.mod 的 replace 里推出 SDK 源码根目录。 +// 只认新名 ⇒ 旧插件查不到路径 ⇒ 退化成「找不到 SDK」的兜底分支， +// 而那是**静默**的：没有报错，只是行为不再是预期那条。 +var sdkModuleNames = []string{ + "github.com/JianFeeeee/homeagentsdk", + "gitcode.com/JianFeeeee/homeagent-sdk", +} + +// isSDKModule 报告 s 是否含 SDK 模块名。 +func isSDKModule(s string) bool { + for _, n := range sdkModuleNames { + if strings.Contains(s, n) { + return true + } + } + return false +} diff --git a/tools/hmapdev/yaegi/sdkmodule_test.go b/tools/hmapdev/yaegi/sdkmodule_test.go new file mode 100644 index 0000000..87e1e40 --- /dev/null +++ b/tools/hmapdev/yaegi/sdkmodule_test.go @@ -0,0 +1,26 @@ +package yaegi + +import "testing" + +// isSDKModule 必须同时认新旧两个 SDK module 名。 +// +// 迁移（gitcode → github）后只认新名会让 findSDKGoPath 对**存量插件** +// （go.mod 里仍是旧名）找不到 SDK 源码路径，且**静默**退化到兜底分支。 +func TestIsSDKModuleAcceptsBothNames(t *testing.T) { + cases := []struct { + s string + want bool + }{ + {"replace github.com/JianFeeeee/homeagentsdk => ../sdk", true}, + {"replace gitcode.com/JianFeeeee/homeagent-sdk => /tmp/sdk-clone", true}, + {"github.com/JianFeeeee/homeagentsdk/sdk", true}, + {"gitcode.com/JianFeeeee/homeagent-sdk/sdk", true}, + {"replace github.com/other/thing => ../x", false}, + {"", false}, + } + for _, c := range cases { + if got := isSDKModule(c.s); got != c.want { + t.Errorf("isSDKModule(%q) = %v, want %v", c.s, got, c.want) + } + } +}