mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-11-06 17:27:31 +00:00
refactor(update): 内核与插件更新合并为 update 子系统（三端共享一个底层）
## 为什么合并（用户口径：更新部分应独立为子系统）
之前 kernelupgrade / pluginupdate 两个包，但它们共享三样东西，
分开写就要各复制一份，然后慢慢不一致：
① **版本比较** —— 合并前共 6 个函数做同一件事：
kernel 侧 ParseSemver / CmpSemver / versionNewer / trimTagV
plugin 侧 trimV / isNewer / semverNums（还自带一份正则）
且**行为不一致**：kernel 侧只挡 len<2，"1.4" 漏过去被填成 1.4.0；
plugin 侧的正则要求两段所以也漏。两份都把 "1.4" 当合法版本。
② **两阶段交互**（检查 → 确认 → 执行）—— 同一套用户口径。
③ **失败要响亮** —— 都不静默降级。
## 收敛结果
internal/update/
doc.go 子系统头：职责边界、三端共享契约、为何不分家
semver.go ParseSemver / CmpSemver / Newer / TrimTag（**唯一一份**）
kernel.go 内核升级闭环（解包→校验→备份→替换→重启→复核→回滚）
release.go GitHub Release 查询
plugin.go 插件升级（查仓库→拉源码→hmapdev 编译→产物）
三端共享：
agent 工具（plugin_update / kernel_update） ┐
WebAPI（/plugins/update、/kernel/update） ├─ internal/update
CLI（homed-upgrade） ┘
入口层只做参数解析与结果呈现，不含业务逻辑 —— 否则会出现
"agent 说没有新版、WebUI 却升了级"的分裂状态。
## 合并时抓到的两个真实 bug
① **ParseSemver 只挡 len<2** ⇒ "1.4" 被零值填成 1.4.0，
「1.4 → 1.4.0」被判完全相同（说"无需升级"），而两者内容未必相同。
现在要求恰好三段。
② **不认 v 前缀** ⇒ GitHub release tag 全是 v1.4.0 形态，
CLI 对"当前 v1.4.0 vs 包内 v1.4.0"判不可比、只能退回字符串比较。
现在 TrimTag 去 v 前缀后再比。
## 新增入口
· plugin_update（agent 工具）+ GET/POST /plugins/update
—— 两阶段：apply=false 只检查（列出 updatable / no_repo 两类，
后者是"未声明仓库、无法自动更新"的内置/私有插件）；
apply=true 需显式 plugins=all 或名单（不默认全升）
· kernel_update（agent 工具）+ GET/POST /kernel/update
—— 两阶段同构；apply 只收本机包路径（URL 下载是 CLI 职责，
agent 不该在推理中发起长下载）
判据：
· internal/update: Newer / TrimTag / ParseSemver 的边界（含上述两个 bug 的回归）
· pluginmgr 入口层 5 条：检查形状、apply 必须显式给范围、
kernel apply 必须给 pkg 且指向 homed-upgrade、拒绝目录当包
This commit is contained in:
JianFeeeee
@ -4,7 +4,7 @@ import (
|
||||
"strings"
|
||||
"testing"
|
||||
|
||||
"github.com/JianFeeeee/HologramAgent/internal/kernelupgrade"
|
||||
"github.com/JianFeeeee/HologramAgent/internal/update"
|
||||
)
|
||||
|
||||
// ── 版本比较的判据 ────────────────────────────────────────────
|
||||
@ -15,46 +15,46 @@ import (
|
||||
func TestCompareVersions(t *testing.T) {
|
||||
cases := []struct {
|
||||
name string
|
||||
cur *kernelupgrade.BinVersion
|
||||
nv *kernelupgrade.BinVersion
|
||||
cur *update.BinVersion
|
||||
nv *update.BinVersion
|
||||
wantSub string // 结论文案必须包含的片段
|
||||
}{
|
||||
{
|
||||
name: "正常升级",
|
||||
cur: &kernelupgrade.BinVersion{Version: "1.3.0", Commit: "aaa"},
|
||||
nv: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||||
cur: &update.BinVersion{Version: "1.3.0", Commit: "aaa"},
|
||||
nv: &update.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||||
wantSub: "可升级",
|
||||
},
|
||||
{
|
||||
name: "降级必须显式警告",
|
||||
cur: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &kernelupgrade.BinVersion{Version: "1.3.0", Commit: "bbb"},
|
||||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &update.BinVersion{Version: "1.3.0", Commit: "bbb"},
|
||||
wantSub: "降级",
|
||||
},
|
||||
{
|
||||
name: "同版本同 commit = 无需升级",
|
||||
cur: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
wantSub: "完全相同",
|
||||
},
|
||||
{
|
||||
// ★ 这条最容易被忽略：版本号没变但内容变了（同版本重发）。
|
||||
// 不报出来会让用户以为"升级了其实没升"或反之。
|
||||
name: "同版本不同 commit 必须点明",
|
||||
cur: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &update.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||||
wantSub: "同版本重发",
|
||||
},
|
||||
{
|
||||
// 非标准形态（1.4.0dev）：走字符串回退，不能说"降级"
|
||||
name: "非标准版本号走字符串比较",
|
||||
cur: &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &kernelupgrade.BinVersion{Version: "1.4.0dev", Commit: "bbb"},
|
||||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||||
nv: &update.BinVersion{Version: "1.4.0dev", Commit: "bbb"},
|
||||
wantSub: "commit 不同",
|
||||
},
|
||||
}
|
||||
for _, c := range cases {
|
||||
got := kernelupgrade.CompareVersions(c.cur, c.nv)
|
||||
got := update.CompareVersions(c.cur, c.nv)
|
||||
if !strings.Contains(got, c.wantSub) {
|
||||
t.Errorf("%s: 结论应含 %q，实际 %q", c.name, c.wantSub, got)
|
||||
}
|
||||
@ -66,7 +66,7 @@ func TestCompareVersions(t *testing.T) {
|
||||
// 这是实测踩到的：线上内核是加 --version 之前的版本，第一次探测必然失败。
|
||||
// 若此时直接退出，所有用户第一次升级都要人工介入。
|
||||
func TestCompareVersions_UnknownCurrentDoesNotBlock(t *testing.T) {
|
||||
got := kernelupgrade.CompareVersions(nil, &kernelupgrade.BinVersion{Version: "1.4.0", Commit: "bbb"})
|
||||
got := update.CompareVersions(nil, &update.BinVersion{Version: "1.4.0", Commit: "bbb"})
|
||||
if strings.Contains(got, "❌") {
|
||||
t.Errorf("当前版本未知不应判失败，实际: %s", got)
|
||||
}
|
||||
@ -81,16 +81,20 @@ func TestParseSemver(t *testing.T) {
|
||||
// 只测**可观察行为**（返回 nil 与否），不探私有字段 ——
|
||||
// 跨包断言内部字段会把重构锁死，而"解析对不对"最终要由
|
||||
// CompareVersions 的行为体现（上面那组用例已覆盖）。
|
||||
if kernelupgrade.ParseSemver("1.4.0") == nil {
|
||||
if update.ParseSemver("1.4.0") == nil {
|
||||
t.Error("1.4.0 应能解析")
|
||||
}
|
||||
// dev 后缀不算正式语义版本（否则 1.4.0dev 会被当成 1.4.0 比大小）
|
||||
if v := kernelupgrade.ParseSemver("1.4.0dev"); v != nil {
|
||||
if v := update.ParseSemver("1.4.0dev"); v != nil {
|
||||
t.Errorf("1.4.0dev 应回退字符串比较，实际解析成 %+v", v)
|
||||
}
|
||||
// ★ "v1.4.0" 现在**算**合法：GitHub 的 release tag 就是 v 前缀形态
|
||||
//（hologramagent 的 tag 全部形如 v1.4.0）。早期实现不认 v 前缀，
|
||||
// 于是 CLI 对"当前 v1.4.0 vs 包内 v1.4.0"判不可比、只能退回字符串比较。
|
||||
//
|
||||
// 不完整串同样不算
|
||||
for _, bad := range []string{"", "1", "1.4", "v1.4.0", "1.4.x"} {
|
||||
if v := kernelupgrade.ParseSemver(bad); v != nil {
|
||||
for _, bad := range []string{"", "1", "1.4", "v1.4", "1.4.x", "1.4.4.4"} {
|
||||
if v := update.ParseSemver(bad); v != nil {
|
||||
t.Errorf("%q 不应被解析成语义版本，实际 %+v", bad, v)
|
||||
}
|
||||
}
|
||||
@ -101,11 +105,11 @@ func TestParseSemver(t *testing.T) {
|
||||
func TestFirstSemverLike(t *testing.T) {
|
||||
// waiter 的真实输出形态
|
||||
line := "waiter 1.4.0 (commit 98850e1, built 2026-10-17T09:20:25Z)"
|
||||
if got := kernelupgrade.FirstSemverLike(line); got != "1.4.0" {
|
||||
if got := update.FirstSemverLike(line); got != "1.4.0" {
|
||||
t.Errorf("应从 waiter 输出取到 1.4.0，实际 %q", got)
|
||||
}
|
||||
// 不能把日期当版本号
|
||||
if got := kernelupgrade.FirstSemverLike("built 2026-10-17"); got != "" {
|
||||
if got := update.FirstSemverLike("built 2026-10-17"); got != "" {
|
||||
t.Errorf("日期不应被识别为版本号，实际 %q", got)
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user