mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-10-03 15:53:56 +00:00
webui: 修复 '删除源/服务器' 只置 <nil> 不真删的问题
前端 deleteSource/deleteMCPServer 通过 PUT value:null 删除,但后端 handleSettings PUT 只 SetCore/SetPlugin(fmt.Sprint(nil) → 字面 '<nil>'),导致 core.llm.sources.<name>.* 等键残留不可达的 <nil> 行,污染 LLM 源与探活。 - SettingsAPI 新增 RemoveCore/RemovePlugin;settingsImpl 接入 ConfigRegistry.Delete - webui handleSettings PUT:body.Value==null 时改走删除分支(核心表/插件表均适配) - 现状验证:mocktest.* 残留已从本机 config.db 清除,sources=3,无无效源
This commit is contained in:
@ -11,6 +11,10 @@ type SettingsAPI interface {
|
||||
DefsPlugin(plugin, prefix string) []*ConfigDef
|
||||
// Remove 删除本插件配置中的单个键(插件删除时清理自身配置用)。
|
||||
Remove(key string) error
|
||||
// RemoveCore 删除核心配置表中的单个键。
|
||||
RemoveCore(key string) error
|
||||
// RemovePlugin 删除另一个插件配置表中的单个键。
|
||||
RemovePlugin(plugin, key string) error
|
||||
}
|
||||
|
||||
type ConfigDef = pubsdk.ConfigDef
|
||||
|
||||
@ -17,19 +17,33 @@ func NewSettings(name string, reg *internalConfig.ConfigRegistry) SettingsAPI {
|
||||
}
|
||||
|
||||
func (s *settingsImpl) Get(key string) (interface{}, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
return s.reg.PluginConfig(s.pluginName).Get(key)
|
||||
}
|
||||
func (s *settingsImpl) Set(key string, value interface{}) error {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.PluginConfig(s.pluginName).Set(key, value)
|
||||
}
|
||||
func (s *settingsImpl) Remove(key string) error {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.PluginConfig(s.pluginName).Remove(key)
|
||||
}
|
||||
func (s *settingsImpl) RemoveCore(key string) error {
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.Delete(coreKey(key))
|
||||
}
|
||||
func (s *settingsImpl) List(prefix string) ([]string, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
return s.reg.PluginConfig(s.pluginName).List(prefix)
|
||||
}
|
||||
func coreKey(key string) string {
|
||||
@ -40,15 +54,21 @@ func coreKey(key string) string {
|
||||
}
|
||||
|
||||
func (s *settingsImpl) GetCore(key string) (interface{}, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
return s.reg.Get(coreKey(key))
|
||||
}
|
||||
func (s *settingsImpl) SetCore(key string, value interface{}) error {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.Set(coreKey(key), value)
|
||||
}
|
||||
func (s *settingsImpl) ListCore(prefix string) ([]string, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
p := coreKey(prefix)
|
||||
if p == "core." {
|
||||
p = "core."
|
||||
@ -56,19 +76,33 @@ func (s *settingsImpl) ListCore(prefix string) ([]string, error) {
|
||||
return s.reg.List(p), nil
|
||||
}
|
||||
func (s *settingsImpl) GetPlugin(plugin, key string) (interface{}, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
return s.reg.PluginConfig(plugin).Get(key)
|
||||
}
|
||||
func (s *settingsImpl) SetPlugin(plugin, key string, value interface{}) error {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.PluginConfig(plugin).Set(key, value)
|
||||
}
|
||||
func (s *settingsImpl) RemovePlugin(plugin, key string) error {
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.PluginConfig(plugin).Remove(key)
|
||||
}
|
||||
func (s *settingsImpl) ListPlugin(plugin, prefix string) ([]string, error) {
|
||||
if s.reg == nil { return nil, nil }
|
||||
if s.reg == nil {
|
||||
return nil, nil
|
||||
}
|
||||
return s.reg.PluginConfig(plugin).List(prefix)
|
||||
}
|
||||
func (s *settingsImpl) RegisterDef(def ConfigDef) {
|
||||
if s.reg == nil { return }
|
||||
if s.reg == nil {
|
||||
return
|
||||
}
|
||||
s.reg.PluginConfig(s.pluginName).RegisterDef(internalConfig.ConfigDef{
|
||||
Key: def.Key, Type: def.Type, DisplayName: def.DisplayName, Description: def.Description,
|
||||
Category: def.Category, Options: def.Options,
|
||||
@ -76,7 +110,9 @@ func (s *settingsImpl) RegisterDef(def ConfigDef) {
|
||||
})
|
||||
}
|
||||
func (s *settingsImpl) Defs(prefix string) []*ConfigDef {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
defs := s.reg.PluginConfig(s.pluginName).ListDefs(prefix)
|
||||
out := make([]*ConfigDef, len(defs))
|
||||
for i, d := range defs {
|
||||
@ -90,11 +126,15 @@ func (s *settingsImpl) Defs(prefix string) []*ConfigDef {
|
||||
return out
|
||||
}
|
||||
func (s *settingsImpl) Dump() map[string]interface{} {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return s.reg.Dump()
|
||||
}
|
||||
func (s *settingsImpl) Plugins() []string {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
names := s.reg.ListPlugins()
|
||||
result := make([]string, 0, len(names)+1)
|
||||
result = append(result, "core")
|
||||
@ -102,11 +142,15 @@ func (s *settingsImpl) Plugins() []string {
|
||||
return result
|
||||
}
|
||||
func (s *settingsImpl) DefsCore(prefix string) []*ConfigDef {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return mapDefs(s.reg.ListDefs(prefix))
|
||||
}
|
||||
func (s *settingsImpl) DefsPlugin(plugin, prefix string) []*ConfigDef {
|
||||
if s.reg == nil { return nil }
|
||||
if s.reg == nil {
|
||||
return nil
|
||||
}
|
||||
return mapDefs(s.reg.PluginConfig(plugin).ListDefs(prefix))
|
||||
}
|
||||
|
||||
@ -122,13 +166,19 @@ func mapDefs(defs []*internalConfig.ConfigDef) []*ConfigDef {
|
||||
}
|
||||
|
||||
func stringifyDefault(v interface{}) string {
|
||||
if v == nil { return "" }
|
||||
if v == nil {
|
||||
return ""
|
||||
}
|
||||
switch x := v.(type) {
|
||||
case string: return x
|
||||
case string:
|
||||
return x
|
||||
case bool:
|
||||
if x { return "true" }
|
||||
if x {
|
||||
return "true"
|
||||
}
|
||||
return "false"
|
||||
default: return fmt.Sprint(v)
|
||||
default:
|
||||
return fmt.Sprint(v)
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user