mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-11-06 17:27:31 +00:00
## 为什么合并(用户口径:更新部分应独立为子系统)
之前 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、拒绝目录当包
118 lines
4.5 KiB
Go
118 lines
4.5 KiB
Go
package main
|
||
|
||
import (
|
||
"strings"
|
||
"testing"
|
||
|
||
"github.com/JianFeeeee/HologramAgent/internal/update"
|
||
)
|
||
|
||
// ── 版本比较的判据 ────────────────────────────────────────────
|
||
//
|
||
// 这些是 OTA 敢不敢按 -apply 的依据。判错一次就可能把新版换成旧版,
|
||
// 或让"内容其实没变"被当成升级成功。
|
||
|
||
func TestCompareVersions(t *testing.T) {
|
||
cases := []struct {
|
||
name string
|
||
cur *update.BinVersion
|
||
nv *update.BinVersion
|
||
wantSub string // 结论文案必须包含的片段
|
||
}{
|
||
{
|
||
name: "正常升级",
|
||
cur: &update.BinVersion{Version: "1.3.0", Commit: "aaa"},
|
||
nv: &update.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||
wantSub: "可升级",
|
||
},
|
||
{
|
||
name: "降级必须显式警告",
|
||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||
nv: &update.BinVersion{Version: "1.3.0", Commit: "bbb"},
|
||
wantSub: "降级",
|
||
},
|
||
{
|
||
name: "同版本同 commit = 无需升级",
|
||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||
nv: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||
wantSub: "完全相同",
|
||
},
|
||
{
|
||
// ★ 这条最容易被忽略:版本号没变但内容变了(同版本重发)。
|
||
// 不报出来会让用户以为"升级了其实没升"或反之。
|
||
name: "同版本不同 commit 必须点明",
|
||
cur: &update.BinVersion{Version: "1.4.0", Commit: "aaa"},
|
||
nv: &update.BinVersion{Version: "1.4.0", Commit: "bbb"},
|
||
wantSub: "同版本重发",
|
||
},
|
||
{
|
||
// 非标准形态(1.4.0dev):走字符串回退,不能说"降级"
|
||
name: "非标准版本号走字符串比较",
|
||
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 := update.CompareVersions(c.cur, c.nv)
|
||
if !strings.Contains(got, c.wantSub) {
|
||
t.Errorf("%s: 结论应含 %q,实际 %q", c.name, c.wantSub, got)
|
||
}
|
||
}
|
||
}
|
||
|
||
// 判据:读不到当前版本时**不能**阻断(第一次升级必然撞到)。
|
||
//
|
||
// 这是实测踩到的:线上内核是加 --version 之前的版本,第一次探测必然失败。
|
||
// 若此时直接退出,所有用户第一次升级都要人工介入。
|
||
func TestCompareVersions_UnknownCurrentDoesNotBlock(t *testing.T) {
|
||
got := update.CompareVersions(nil, &update.BinVersion{Version: "1.4.0", Commit: "bbb"})
|
||
if strings.Contains(got, "❌") {
|
||
t.Errorf("当前版本未知不应判失败,实际: %s", got)
|
||
}
|
||
if !strings.Contains(got, "跳过比较") {
|
||
t.Errorf("应说明跳过比较,实际: %s", got)
|
||
}
|
||
}
|
||
|
||
// ── 版本解析 ─────────────────────────────────────────────────
|
||
|
||
func TestParseSemver(t *testing.T) {
|
||
// 只测**可观察行为**(返回 nil 与否),不探私有字段 ——
|
||
// 跨包断言内部字段会把重构锁死,而"解析对不对"最终要由
|
||
// CompareVersions 的行为体现(上面那组用例已覆盖)。
|
||
if update.ParseSemver("1.4.0") == nil {
|
||
t.Error("1.4.0 应能解析")
|
||
}
|
||
// dev 后缀不算正式语义版本(否则 1.4.0dev 会被当成 1.4.0 比大小)
|
||
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", "1.4.x", "1.4.4.4"} {
|
||
if v := update.ParseSemver(bad); v != nil {
|
||
t.Errorf("%q 不应被解析成语义版本,实际 %+v", bad, v)
|
||
}
|
||
}
|
||
}
|
||
|
||
// ── waiter 兜底解析 ──────────────────────────────────────────
|
||
|
||
func TestFirstSemverLike(t *testing.T) {
|
||
// waiter 的真实输出形态
|
||
line := "waiter 1.4.0 (commit 98850e1, built 2026-10-17T09:20:25Z)"
|
||
if got := update.FirstSemverLike(line); got != "1.4.0" {
|
||
t.Errorf("应从 waiter 输出取到 1.4.0,实际 %q", got)
|
||
}
|
||
// 不能把日期当版本号
|
||
if got := update.FirstSemverLike("built 2026-10-17"); got != "" {
|
||
t.Errorf("日期不应被识别为版本号,实际 %q", got)
|
||
}
|
||
}
|
||
|
||
|