xbot 五大方向重构 — 终审报告

审核机构：门下省
审核日期：2026-03-19
审核范围：6 大代码变更方向，24 个新增/修改文件
审核基准：/workspace/xbot/plan/xbot-major-refactor-design.md (v3)

所有包编译、静态分析、测试均无错误。

涉及文件：agent/channel_prompt.go、agent/channel_prompt_test.go、agent/middleware.go、agent/agent.go、channel/feishu.go、main.go

涉及文件：llm/types.go、llm/anthropic.go、llm/anthropic_test.go、agent/middleware.go

涉及文件：agent/progress.go、agent/progress_test.go、agent/engine.go、agent/reply.go、agent/reply_test.go

涉及文件：agent/registry.go、agent/command_builtin.go、storage/sqlite/registry.go、storage/sqlite/registry_test.go、storage/sqlite/db.go

修改意见：

1. 在 agent/agent.go 添加 SetRegistryManager(rm *RegistryManager) 方法
2. 在 main.go 初始化 RegistryManager 并注入到 Agent
3. 注入时机：在 agentLoop := agent.New(...) 之后、Run() 之前

涉及文件：channel/capability.go、channel/capability_test.go、agent/settings.go、agent/settings_test.go、storage/sqlite/user_settings.go、agent/command_builtin.go、channel/feishu.go

修改意见：

1. 在 agent/agent.go 添加 SetSettingsService(svc *SettingsService) 方法
2. 在 main.go 初始化 SettingsService 并注入到 Agent
3. （架构建议）考虑将 SettingsCapability / UIBuilder 接口从 channel 包提取为独立接口包，避免 agent → channel 的直接依赖

涉及文件：llm/types.go、llm/anthropic.go、llm/anthropic_test.go、llm/openai.go

已在方向 2 中一并审核。补充：

方向 1-3、6 实现质量优秀，代码风格一致，架构设计合理。但方向 4 和 5 存在同一类系统性遗漏：新增的服务字段（registryManager、settingsSvc）在 Agent struct 中声明、在 command_builtin.go 中消费，但缺少公开 setter 方法且 main.go 未注入初始化代码。这导致 8 个新增命令（/publish、/unpublish、/browse、/install、/uninstall、/my、/set、/menu）在运行时全部因 nil check 而静默返回提示信息，功能完全不生效。

1. agent/agent.go — 添加两个 setter：

   func (a *Agent) SetRegistryManager(rm *RegistryManager) { a.registryManager = rm }
   func (a *Agent) SetSettingsService(svc *SettingsService) { a.settingsSvc = svc }
   

2. main.go — 在 agentLoop := agent.New(...) 之后添加初始化：

   // 注入 RegistryManager
   sharedDB, _ := sqlite.Open(dbPath)  // 复用已有连接
   agentLoop.SetRegistryManager(agent.NewRegistryManager(sharedDB, xbotDir))
   
   // 注入 SettingsService
   agentLoop.SetSettingsService(agent.NewSettingsService(
       sqlite.NewUserSettingsService(sharedDB),
       disp,  // channel.Dispatcher 用于查询 SettingsCapability
   ))
   

修复完成后可免审提交（门下省确认修改范围可控）。

门下省 · 审核完毕