mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-10-02 15:23:57 +00:00
fix(lua): 内置适配器按内容自动更新,替代「文件已存在就跳过」
## 原机制是这次全部误判的根源
if _, err := os.Stat(dstPath); err == nil { continue }
**升级二进制永远不更新已部署的适配器文件。** 于是"改了仓库 ≠ 生产生效",
而这个机制让同类问题可以长期潜伏:
2026-08-26 15:46 生产 openai.lua 手工补上 stream_index 透传
2026-08-26 16:10 cfd1653 提交,说明里写了但代码没改这个文件
修复当天先在生产落地、32 分钟后才提交入库(漏了这个文件),此后一个月里
两端都没人发现 —— 生产不报问题(它有),仓库的判据也测不到(直接构造 Go
结构体,绕过适配器)。而"升级不覆盖"意味着即使仓库补上修复,已部署的
老实例也不会拿到。
## 新判据(按内容,不按存在)
文件不存在 ⇒ 写
有历史清单且盘上 == 上次内嵌 ⇒ 覆盖(只是没跟上新版本)
有历史清单但盘上 != 上次内嵌 ⇒ 不动 + 日志(用户改过)
无历史清单(首跑/从旧版本升级) ⇒ 不动,只补缺失文件(与旧行为一致)
"上次内嵌的版本"记在 `DataDir/adapters/.bundled`(`<name>\t<sha256>`)。
⚠️ 为什么不能无条件覆盖:adapter_path 是可配置项,用户可以把 adapter_path
指向自己维护的适配器。静默覆盖等于丢弃他们的修改,而且**没有报错**。
## 判据(两个方向都要测)
- TestWriteBundledAdaptersSkipsUserModified 用户改过的**必须保留**,
且改完仍能正常加载(fixture 必须功能完整,否则会因为缺钩子函数而失败 ——
那是 fixture 问题,不是保护逻辑问题)
- TestWriteBundledAdaptersUpdatesStale 落后于新内嵌的**必须被更新**,
且**幂等**(三跑不再改写任何文件)
★ 第二条是必要的:只测保护的话,**一个"永远不覆盖任何文件"的实现也能全绿**
—— 而那正是要修的病。
## 代价(必须知道)
**升级到本版本的这一次,已部署实例的适配器不会更新**(没有历史清单可比)。
从第二次升级起自动生效。要立刻生效就删掉 DataDir/adapters 让内核重新解包。
对本次修的 8 个适配器而言:生产此刻用不到(三个源 llmsproxy/visionllm/
justworker 全是 openai.lua),所以不影响运行;将来启用 deepseek 等源时,
自然就是修复版。
## 顺带
`bundledAdapterNames` 从 writeBundledAdapters 里提出来成包级变量 ——
writeBundledAdapters 与体检判据共用,避免两处各写一份而漏掉某个
(漏掉的后果是该适配器永远不会被更新)。
This commit is contained in:
@ -2,7 +2,6 @@ package lua
|
||||
|
||||
import (
|
||||
"encoding/json"
|
||||
"path/filepath"
|
||||
"testing"
|
||||
)
|
||||
|
||||
@ -122,7 +121,7 @@ func TestAllBundledAdaptersStreamToolCallStatus(t *testing.T) {
|
||||
}
|
||||
var supported, nested, unsupported []string
|
||||
|
||||
for _, name := range bundledAdapterNames(t) {
|
||||
for _, name := range bundledAdapterNames {
|
||||
vm := NewVM(t.TempDir())
|
||||
loadBundled(t, vm, name)
|
||||
out, err := vm.CallTransformStreamChunk(name, openAIMultiToolChunk)
|
||||
@ -183,18 +182,3 @@ func loadBundled(t *testing.T, vm *VM, name string) {
|
||||
t.Fatalf("加载 %s 失败: %v", name, err)
|
||||
}
|
||||
}
|
||||
|
||||
func bundledAdapterNames(t *testing.T) []string {
|
||||
t.Helper()
|
||||
ents, err := bundledAdapters.ReadDir("adapters")
|
||||
if err != nil {
|
||||
t.Fatalf("读 adapters 目录失败: %v", err)
|
||||
}
|
||||
var out []string
|
||||
for _, e := range ents {
|
||||
if filepath.Ext(e.Name()) == ".lua" {
|
||||
out = append(out, e.Name()[:len(e.Name())-4])
|
||||
}
|
||||
}
|
||||
return out
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user